[NO-MERGE][rewrite] Interfaces Protocol v1 conformance slice (WS5)#136
Draft
adamcohenhillel wants to merge 18 commits into
Draft
[NO-MERGE][rewrite] Interfaces Protocol v1 conformance slice (WS5)#136adamcohenhillel wants to merge 18 commits into
adamcohenhillel wants to merge 18 commits into
Conversation
Vendored ifp/1 validator+machine+fixtures, all 37 commands re-expressed as v1 tool declarations with migration table, lifecycle adapter (propose/approve/reject/result with params_hash binding), 86 new test cases wired into check.sh, v0 manifests marked deprecated/superseded_by. check.sh fully green. Remainder in WS5-REMAINDER.md. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Do not merge
Branch-only work under the Interfaces v1 rewrite plan (see silentspeech-app PR #138). Merge requires the separate Part VI acceptance review.
Note: this branch is cut at
59ba9f65(tip ofrefactor/app-os-boundary, which contains all of main plus thePhoneToolGatewayseam and adaptive surfaces) — so this PR includes that unmerged branch's commits plus the WS5 work on top.Contents
runtime/protocol/ifp1/— vendored TypeScript validator + conformance machine + fixture corpus (80 cases incl. shared canonical-JSON/params-hash vectors),openphone-tools.ifp1.jsonre-expressing all 37 v0 commands as ifp/1 tool declarations (risk carried over exactly: 8 low / 24 medium / 5 high), andMIGRATION.mdwith the full command-to-tool table.adapter.mjs) —runtime.tool.requestedbecomesaction.propose; confirmation resolution becomes approve/reject with exactparams_hashbinding; results carrybinding_mismatch/action_expiredhandling.runtime.confirmation.requireddeliberately has no wire message: confirmation is surface-local authority (spec 8.2)."deprecated": true, "superseded_by": "ifp1/") — not deleted; the Java side still reads them. Deletion gate in the remainder.scripts/check.sh; full check suite green including the Java compile contract checks.Spec bugs found here (wrong params_hash example, undefined canonical JSON) were fixed upstream in the shared corpus.
Remainder:
WS5-REMAINDER.md(Java RuntimeToolBridge/PhoneToolGateway port, Ed25519 pairing promotion from the OpenClaw adapter, OpenClaw/MCP/CLI manifest switch, old-manifest deletion gate, iOS runtime note).🤖 Generated with Claude Code