Skip to content

fix: WebMCP execution input compatibility for v0.5.1 - #15

Merged
TueJon merged 2 commits into
mainfrom
fix/webmcpify-state-of-art-20260914
Sep 14, 2026
Merged

TueJon merged 2 commits into
mainfrom
fix/webmcpify-state-of-art-20260914

Conversation

@TueJon

@TueJon TueJon commented Sep 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • adapt the verification template and visual Workbench to current object-shaped executeTool() input
  • preserve Chrome 150 JSON-string compatibility with a temporary side-effect-free probe
  • invoke every real application tool exactly once; update ambient types, proof, docs, manifests, changelog, and v0.5.1 notes

Evidence

  • WebMCP CG draft 97da8f5: optional any input object
  • Chrome imperative guide updated 2026-09-11: object input; JSON strings deprecated from Chrome 155
  • WPT commits 1a21db90ad and c94abb33b8 cover object input and object-shaped enumerated schemas
  • official webmcp-tools PR #447 moved to object input with legacy compatibility

Verification

  • npm run check — 43/43
  • npm run workbench:verify — Chromium desktop/mobile/native/CSP gates pass
  • npm run proof:verify — native Chrome 150, detected JSON-string mode, UI delta/invalid input/cleanup pass
  • npm audit --omit=dev — 0 vulnerabilities

No merge, tag, release, package publication, or directory mutation is part of this PR.

@TueJon TueJon added bug Something isn't working webmcp WebMCP agent surface labels Sep 14, 2026
@TueJon TueJon self-assigned this Sep 14, 2026

@TueJon TueJon left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

skills/webmcpify/templates/webmcp.spec.ts:71 and skills/webmcpify/references/verify.md:76 — the remaining runner guidance still says native getTools() always returns a stringified inputSchema, while this release correctly documents and supports object-shaped schemas in current native implementations. The shipped parsing logic already handles both shapes, so this is non-blocking, but these comments can mislead agents adapting the template.

Closure checklist:

  • Update the listTools comment in skills/webmcpify/templates/webmcp.spec.ts to describe older-string/current-object compatibility.
  • Update the LLM-runner bullet in skills/webmcpify/references/verify.md to say native inputSchema may be either shape.

@TueJon
TueJon marked this pull request as draft September 14, 2026 14:43
@TueJon
TueJon marked this pull request as ready for review September 14, 2026 14:44
@TueJon

TueJon commented Sep 14, 2026

Copy link
Copy Markdown
Owner Author

Resolved the documentation-consistency caveat at b8a4881:

  • skills/webmcpify/templates/webmcp.spec.ts now describes older-string/current-object schema compatibility.
  • skills/webmcpify/references/verify.md now documents both native inputSchema shapes.

Verification: npm run check passes all 43 tests; the stale native-string-only wording is absent repo-wide.

@TueJon TueJon left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

No blocking findings. The prior two-item schema-shape documentation checklist is fully closed, with no regression in the compatibility adapter or release surfaces.
Verified with npm run check (43/43), native Chrome 150 proof, and the Workbench browser matrix.

@TueJon
TueJon merged commit ed7ab5f into main Sep 14, 2026
2 checks passed
@TueJon
TueJon deleted the fix/webmcpify-state-of-art-20260914 branch September 22, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working webmcp WebMCP agent surface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant