Repository navigation
Conversation
TueJon
commented
Nov 9, 2026
TueJon
left a comment
Owner
Author
There was a problem hiding this comment.
Round 1 review of head 16c076e (docs-only, +28/−28 across 5 files). No prior reviews; judged on this diff.
Sweep: every changed surface traced — docs/site-tools/index.html (requirements, troubleshooting, FAQ, sources), docs/index.html (client-proof bullet, new probe paragraph), llms.txt, webmcp-agent-skill/index.html (en+de), tests/pages.test.mjs. No code, workflow, deploy, schema or secret changes; no external effects.
- Product fit: the refresh removes a model/workspace matrix and a "no forms/iframes" claim the current official guide no longer carries, replacing both with dated, hedged copy ("documentation check, not a live account test"). Matches the repo's experimental-honesty bar.
- Stale-claim sweep: no remaining
GPT-5.6 Sol/GPT-6 Sol/Luna/Edu/not Declarative APIstrings in html/txt. Remaining2026-10-12refs are the separate Chrome origin-trial evidence paragraphs with their own dates — correctly untouched. - Bilingual parity: en+de spans updated together on both bilingual pages.
docs/index.htmlhas no German mirror for the touched bullets, so no parity gap. - Test guard: updated facts plus a
doesNotMatchagainst the superseded matrix;node --test tests/*.test.mjspasses 53/53 locally and CItestis SUCCESS at this head. - Cited dates: WebMCP draft
2026-11-02confirmed against the live spec page; OpenAI guide URL live. TheexposedTosentence correctly separates agent discovery from in-page cross-origin gates. The new Chrome 156 probe paragraph is dated, scoped, and states its non-claims explicitly. - Generated files:
index.html/de/and tool contracts untouched, so no regen owed.
No blocking or non-blocking findings. Ship it.
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
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.
The public docs still say ChatGPT Site tools exclude HTML forms and iframe registrations and repeat a model/workspace matrix removed from the current official guide.
Refresh the dated Site tools page, documentation index, bilingual skill overview and llms.txt from the official OpenAI documentation checked 2026-11-09. Keep account/workspace and provider-origin permissions explicit, distinguish agent discovery from in-page cross-origin gates, and record current scalar/outputSchema limitations without claiming account or WPT verification.
Validation: 53/53 dependency-free tests pass, including the updated public-copy consistency check. Homepage, generated pages, tool/discovery contracts, layout and deploy configuration are unchanged. This PR prepares the copy only; merging/deploying is outside the task's authorization. Source compatibility changes are tracked in TueJon/webmcpify#26.