fix: audit and correct the compare pages — real APIs, valid HTML, honest examples#36
Merged
Merged
Conversation
…est examples Follow-up to the content restructure (#34, #35), which only touched one overclaiming table row per compare page. A full pass turned up more: Fabricated code samples: all three pages had "integration" code importing a package that has never existed (@traverse-framework/runtime) with an invented TraverseRuntime.init()/.execute() API. A developer copying any of these would hit a wall immediately. - vs-function-calling: the "using them together via MCP" section invented a hand-rolled MCP tool wrapper. The real story is simpler and better — traverse-mcp -- stdio already exposes every governed capability as an MCP tool, no wrapper code needed. Replaced with the real invocation and a link to /agents.html. Also fixed an example contract's placement array, which listed ai-pipeline as a valid value. - vs-microservices: replaced the fake Rust sample with the real traverse-embedder crate API (BundleEmbedder::init, .submit, .subscribe). - vs-serverless: the sample was a Cloudflare Worker, i.e. edge — a placement Traverse hasn't shipped. Reframed as a Rust AWS Lambda handler using the same real traverse-embedder crate, which sidesteps the unverified edge-WASM question entirely by being a real, ordinary Rust process. Malformed HTML: all three pages' CTA bands had a stray </section> landing mid-block (evidently copy-pasted), closing the section early and leaving the button row and final closing tags orphaned outside it. Browsers silently recover from this, but it's invalid structure that happened to look fine by accident. Fixed the tag order in all three so the section actually closes where the markup says it does. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
enricopiovesan
enabled auto-merge (squash)
July 21, 2026 03:17
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
Follow-up to the content restructure (#34, #35), which only touched one overclaiming table row per compare page during the sitewide AI-pipeline sweep. Went back and gave all three
/compare/*pages the full audit — these are exactly the pages a skeptical technical evaluator reads closest, so they deserved the same scrutiny as everything else.Fabricated code samples
All three pages had "integration" code importing a package that has never existed —
@traverse-framework/runtime— with an inventedTraverseRuntime.init()/.execute()API. Anyone who copied any of these would hit a wall immediately.traverse-mcp -- stdioalready exposes every governed capability as an MCP tool, no wrapper code needed. Replaced with the real invocation and a link to/agents.html. Also fixed an example contract whoseplacementarray listedai-pipelineas a valid value.traverse-embeddercrate API (BundleEmbedder::init,.submit,.subscribe).traverse-embeddercrate, which sidesteps the edge-WASM question entirely by just being an ordinary Rust process.Malformed HTML
All three pages' CTA bands had a stray
</section>landing mid-block (evidently copy-pasted from one page to the next), closing the section early and leaving the button row and final closing tags orphaned outside it. Browsers silently recover from this kind of thing, so it happened to render fine by accident — but it's invalid structure. Fixed the tag order in all three so the section closes where the markup says it does.Test plan
npm run buildclean after each of the three edits (caught nothing this time — no repeat of the escaping bug from feat: content restructure phase 2 — remove Solutions/Use-cases, honest sitewide language #35)dist/output: all three CTA bands close correctly, both buttons present and visible, real API names (BundleEmbedder,traverse_embedder,lambda_runtime) present in the rendered code samples, zero remaining references to the fabricatedTraverseRuntime/@traverse-framework/runtimeanywhere insrc/🤖 Generated with Claude Code