ReviewOS

also looking at this

stacks/uplink

docs(homepage): copy that survives a second engine

#57
Merged glennmichael123 wants to merge docs/homepage-two-engines into main
2 files +10 -10
11 days ago

Closes #19. Copy only: no new views, no route changes, no engine selection logic.

What was wrong

Uplink has answered texts with Codex since #7, and the page still said the answers came from Claude Code. Most of the strings only named one engine and could be reworded. Three stated facts that are true of Claude and of no other agent, so rewording could not fix them.

The API key answer gave claude setup-token and the Keychain as the way in. That is Claude's path, not Codex's, which keeps its own credentials and leaves Uplink holding nothing. Each engine gets its own clause now, checked against codex-engine.ts and settings.ts rather than written from the old answer.

The cost answer billed against "your own Claude plan". It is the plan you already have, Claude or ChatGPT.

The bento cell named CLAUDE.md as the per-repository instruction file. It is a filename rather than an engine's name, which is why it is easy to skim past, but a second engine reads a different one. It names both.

What deliberately still says Claude

The hero headline and the <title>. The issue's requirement was copy that survives a second engine, not the page's strongest line flattened into "your agent". The title is also 55 characters, and listing both would push it past what search results show.

Two alt texts, because alt text follows the image rather than the copy. The menubar screenshot and the social card describe what those images actually show, and editing the alt before the image is retaken makes it describe something that is not there.

Two things the issue did not know about

It was written before some of this moved, so for the record:

  • The download subhead and one FAQ already name both engines. Chris got there first.
  • The head is now /var/www/reviewos-reviewos/releases/6c73186/resources/views/layouts/app.stx:3619:2285: include error: Error loading include file site-head: ENOENT: no such file or directory, open '/var/www/reviewos-reviewos/releases/6c73186/resources/views/partials/site-head.stx' ╭──────────────────────────────────────────────────────────────╮ │  ERROR  Include Error │ ╰──────────────────────────────────────────────────────────────╯ File: app.stx (line 3619, col 2285) Message: Error loading include file site-head: ENOENT: no such file or directory, open '/var/www/reviewos-reviewos/releases/6c73186/resources/views/partials/site-head.stx' Context:  3617: 3618: > 3619:

    Closes #19. Copy only: no new views, no route changes, no engine selection logic.

    What was wrong

    Uplink has answered texts with Codex since #7, and the page still said the answers came from Claude Code. Most of the strings only named one engine and could be reworded. Three stated facts that are true of Claude and of no other agent, so rewording could not fix them.

    The API key answer gave claude setup-token and the Keychain as the way in. That is Claude's path, not Codex's, which keeps its own credentials and leaves Uplink holding nothing. Each engine gets its own clause now, checked against codex-engine.ts and settings.ts rather than written from the old answer.

    The cost answer billed against "your own Claude plan". It is the plan you already have, Claude or ChatGPT.

    The bento cell named CLAUDE.md as the per-repository instruction file. It is a filename rather than an engine's name, which is why it is easy to skim past, but a second engine reads a different one. It names both.

    What deliberately still says Claude

    The hero headline and the <title>. The issue's requirement was copy that survives a second engine, not the page's strongest line flattened into "your agent". The title is also 55 characters, and listing both would push it past what search results show.

    Two alt texts, because alt text follows the image rather than the copy. The menubar screenshot and the social card describe what those images actually show, and editing the alt before the image is retaken makes it describe something that is not there.

    Two things the issue did not know about

    It was written before some of this moved, so for the record:

    • The download subhead and one FAQ already name both engines. Chris got there first.
    • The head is now /var/www/reviewos-reviewos/releases/6c73186/resources/views/layouts/app.stx:3708:34: include error: Error loading include file site-head: ENOENT: no such file or directory, open '/var/www/reviewos-reviewos/releases/6c73186/resources/views/partials/site-head.stx' ╭──────────────────────────────────────────────────────────────╮ │  ERROR  Include Error │ ╰──────────────────────────────────────────────────────────────╯ File: app.stx (line 3708, col 34) Message: Error loading include file site-head: ENOENT: no such file or directory, open '/var/www/reviewos-reviewos/releases/6c73186/resources/views/partials/site-head.stx' Context:  3706: 3707: > 3708: 3709: 3710:  , and that partial carries its own schema.org description which named Claude Code. It renders on this page, so it is fixed here. Its og:image:alt is left, by the alt-text rule above.

    Out of scope and still naming Claude alone: dashboard.stx and thanks.stx. Different pages, not this issue.

    Verification

    • 155 unit tests pass, lint clean
    • stx typecheck on the view: 22 template expressions, 0 errors
    • no em-dashes or en-dashes, verified by grep

    🤖 Generated with Claude Code

    3620: 3621: 
    , and that partial carries its own schema.org description which named Claude Code. It renders on this page, so it is fixed here. Its og:image:alt is left, by the alt-text rule above.

Out of scope and still naming Claude alone: dashboard.stx and thanks.stx. Different pages, not this issue.

Verification

  • 155 unit tests pass, lint clean
  • stx typecheck on the view: 22 template expressions, 0 errors
  • no em-dashes or en-dashes, verified by grep

🤖 Generated with Claude Code

2 changed files on the files tab, with 0 review threads.

Sign in to comment on this pull request.