fix: stabilize hosted documentation follow-ups - #405
Merged
Merged
Conversation
Record the verified shared formatter boundary, mutable hosted-document contract, snapshot preservation, and end-to-end acceptance workflow.
Drop search evidence ranges from generated follow-ups for mutable hosted documentation URLs while preserving repository snapshot ranges and explicit caller bounds. Update durable CLI, MCP, and Agent Skill guidance for the current-content contract.
Describe mutable hosted targets, snapshot-addressed repository docs, and full heading-subtree reads in both current and deprecated CLI help surfaces.
Document heading-subtree semantics across MCP and Agent Skill surfaces, remove vacuous CLI text assertions, and validate bounds-free hosted follow-ups in live MCP smoke coverage.
Teach the shared smoke corpus to return hosted documentation evidence and prove the live assertion rejects generated follow-ups that replay search bounds.
Use the explicit Express documentation site and follow one documented search continuation before requiring hosted evidence, with coverage for the in-progress response shape.
Key the hosted-doc smoke continuation on the completion flag and model the required empty results array in its in-progress fixture.
Capture the completed implementation, validation, smoke limitations, and clean review evidence before draft PR delivery.
Close the implementation plan with the delivered draft PR while preserving the explicit merge and release boundary.
Replace the earlier keychain-blocked smoke limitation with the successful real-site search-to-read evidence and retain the unrelated full-suite limitation.
jlitola
marked this pull request as ready for review
September 18, 2026 05:56
Capture the refreshed Express corpus evidence showing exact fragment forwarding and bounds-free heading-subtree reads.
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.
Summary
documentation_pagetargetsVerification
bun test— 4,826 passbun run typecheckbun run format:checkbun run lint— passes with 8 pre-existing warnings inrepository-target.tsbun run buildbun run plugins:checkbun run validate:packagesbun run smoke:cli:builtbun run smoke:mcp:builtLive evidence
After resolving a local keychain prompt, a live source search for
routingagainstsite:expressjs.comreturned hosteddocumentation_pagehits with observed locator ranges and bounds-free generatedfollowUpvalues. Human-readable CLI output displayed the exact hosted target without coordinates. An unbounded live read of the returnedhttps://expressjs.com/en/5x/guide/routing/target then returned the current page from line 1 through its reported total; the 16,278-line aggregate target also completed through source and built CLIs with valid JSON.After the Express corpus refresh, a live
route methodssearch emittedhttps://expressjs.com/en/5x/guide/routing/#route-methodswith search evidence lines 56–66. The generated follow-up preserved that fragment exactly and omitted bounds. Executing it returned lines 56–112 beginning at## Route methods, confirming the complete heading subtree rather than replaying the narrower search coordinates.The full authenticated MCP smoke progressed through live package calls but stopped on an unrelated
pkg_upgrade_reviewsuccess assertion before reaching its documentation block. The focused real-site search-to-read path is verified; a full live MCP-suite pass is not claimed.Targeted agent evaluation was also attempted. Claude's isolated harness was not logged in. Codex invoked
searchanddocs_list, then timed out after 302 seconds while those service calls remained in progress; its available tool-call and metrics artifacts were inspected, with no final or isolation artifact produced.Scope
No backend, remote server, release, deployment, retry, snapshot, compatibility, or historical-storage changes are included.