Skip to content

fix: confirm Skills responses with filtered stream values - #1301

Merged
blove merged 2 commits into
mainfrom
blove/skills-public-confirmation
Oct 12, 2026
Merged

blove merged 2 commits into
mainfrom
blove/skills-public-confirmation

Conversation

@blove

@blove blove commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Deployed Skills values streams can omit private metadata that remains in saved checkpoints. This made successful native React runs appear unconfirmed. Allow only missing live skills_metadata and skills_load_errors; observed fields, ordinary state, messages, tools, ownership, terminal controls, and the loaded checkpoint retain their exact checks.

Add filtered and unfiltered native submit/load regression coverage and rejection cases for conflicting or incomplete state. Update the Skills guides and generated public guidance to distinguish streamed values from checkpoint evidence.

Validation:

  • Skills unit tests: 110 passed.
  • Skills browser tests: 9 passed, with 11 installed SDK/compiled graph fixture tests.
  • Skills lint and production build passed; lint reports 38 test warnings.
  • Website unit tests and generated context passed.
  • Offline replay of the captured public stream through the real native session now confirms the exact saved checkpoint.

@vercel

vercel Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
threadplane Ready Ready Preview Oct 12, 2026 3:30am UTC

Request Review

@blove
blove merged commit a240834 into main Oct 12, 2026
37 checks passed

This branch was successfully deployed

1 active deployment
Preview – threadplane — 5800ca85 Deployed Oct 12, 2026 by vercel[bot]
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.

1 participant