Skip to content

Fix canonical Service Kit module manifests - #567

Merged
LioRael merged 1 commit into
mainfrom
fix/service-kit-canonical-manifest-digests
Aug 13, 2026
Merged

Fix canonical Service Kit module manifests#567
LioRael merged 1 commit into
mainfrom
fix/service-kit-canonical-manifest-digests

Conversation

@LioRael

@LioRael LioRael commented Aug 13, 2026

Copy link
Copy Markdown
Owner

What changed

  • serialize TypeScript Module Manifests with the Rust canonical optional-field rules
  • retain non-enumerable compatibility accessors for empty admin and requires
  • include the canonical empty runtime schedules field
  • add a patch changeset for @lenso/service-kit

Why

Provider releases generated by Service Kit could hash JSON that differed from the Rust Host's canonical reserialization. The Host then rejected otherwise valid releases with a manifest digest mismatch.

Validation

  • pnpm --dir sdk/typescript check
  • pnpm changeset status
  • fresh TypeScript Service generation and Provider release validation
  • real Host startup and Provider registration

Copilot AI lite review requested due to automatic review settings August 13, 2026 11:03
@changeset-bot

changeset-bot Bot commented Aug 13, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ded16d4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@lenso/service-kit Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@LioRael
LioRael merged commit b510af8 into main Aug 13, 2026
3 checks passed
@LioRael
LioRael deleted the fix/service-kit-canonical-manifest-digests branch August 13, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants