Skip to content

feat(landing-page): agent design pages — all 21 supported agents#4050

Merged
xne998808-ai merged 12 commits into
mainfrom
feat/landing-gemini-design
Jun 11, 2026
Merged

feat(landing-page): agent design pages — all 21 supported agents#4050
xne998808-ai merged 12 commits into
mainfrom
feat/landing-gemini-design

Conversation

@xne998808-ai

@xne998808-ai xne998808-ai commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Why

The /agents/ hub previously linked to rich design pages for only four agents. This completes the family: every coding agent Open Design's runtime supports now has its own long-form design page, so visitors searching " design" land on a real, localized page that drives to Open Design instead of a bare hub anchor. The hub was also missing four agents the runtime actually supports — they're added here.

What users will see

  • 17 new agent design pages at /agents/<agent>-design/ (on top of the existing 4), one per supported coding agent: Gemini CLI, GitHub Copilot CLI, Qwen Code, Grok CLI, Kimi CLI, DeepSeek TUI, Trae CLI, Aider, Antigravity, DeepSeek Reasonix, Hermes, Devin for Terminal, Pi, Kiro CLI, Kilo, Mistral Vibe CLI, Qoder CLI.
  • Each page: hand-structured how-to long-form (what it is, the design angle, setup, reference-to-UI, extend, a comparison table, pitfalls, and with Open Design), an FAQ with JSON-LD, and a warm editorial hero — the agent's real mark where one exists, a clean wordmark where it does not.
  • The /agents/ hub now lists all 21 agents (four were missing) and the top-nav Agent dropdown links into every page. Fixes an incorrect vendor label for Pi.
  • Localized across all 18 landing locales, including a Traditional Chinese (zh-tw) fix so the agent guides render Traditional instead of Simplified.

Surface area

  • UI — new long-form pages under apps/landing-page (/agents/<agent>-design/); the /agents/ hub and Agent nav dropdown link into them
  • i18n — agent guides across all landing locales (en hand-checked, zh translated, the other 15 machine-translated and merged into the split agent-guides.part-*.i18n.ts files — now 15 parts to stay under the 1 MiB changed-file blob guard; a Traditional Chinese override added under the zh-tw entry)
  • Content accuracy — every fact, command, model name, and resource URL was verified against the agent's own runtime def (apps/daemon/src/runtimes/defs/*) and official sources; sparse agents are described accurately-but-generally rather than with invented specifics
  • Keyboard shortcut / CLI / API / Extension point / new dependency / default behavior change — n/a

Validation

  • astro check (typecheck): 0 errors.
  • Static build: clean — 9,879 pages, including all 21 agents × 18 locales.
  • All changed agent-guides*.ts files are < 1 MiB (blob guard).
  • Spot-checked rendering: rich layout, hero images, localized JSON-LD, zh-tw Traditional output, and the hub/nav links, locally.
  • Resource URLs verified to resolve (a couple host-blocked URLs cross-checked via search).

Notes

Built with the od-agent-design-page workflow that codifies this page family (content structure, logo-fused hero, the 18-locale split-file localization, and the hub/nav wiring).

Add a rich long-form /agents/gemini-design/ page for Google's Gemini CLI,
mirroring the existing agent detail pages: hand-structured how-to content
(what it is, the multimodal + 1M-context design angle, setup, screenshot-to-UI,
GEMINI.md / MCP, a comparison table, pitfalls, and with Open Design), an FAQ
with JSON-LD, and a hero illustration with the Gemini mark fused into the
artwork. Wire it into the /agents/ hub via DETAIL_ROUTES and the Agent nav
dropdown.

Localize across every landing locale: en is hand-written, zh hand-reviewed, and
the other 16 are machine-translated and merged into the split agent-guides part
files (kept under the 1 MiB changed-file blob guard).
@lefarcen lefarcen requested a review from PerishCode June 10, 2026 06:36
@lefarcen lefarcen added size/XXL PR changes 1500+ lines risk/medium Medium risk: regular code changes type/feature New feature labels Jun 10, 2026

@lefarcen lefarcen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @xne998808-ai! 👋 Gemini CLI is a solid next entry for the /agents/ family — the multimodal + 1M-context angle gives it a clear identity distinct from the other pages, and routing localized traffic to a proper landing page rather than a bare hub anchor is exactly the right call.

One small thing before pool review picks this up: the Notes section already says astro check and the static build are green — worth promoting that to a proper ## Validation section so the reviewer can scan it without reading to the bottom:

## Validation
- `astro check` — green  
- Static build — green  
- 18-locale coverage confirmed; all `agent-guides*.ts` files < 1 MiB

Otherwise the body reads well. @PerishCode has been assigned for review.

@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

🚀 Landing page preview

This PR is deployed to a Cloudflare Pages preview — not staging or production:

@PerishCode PerishCode left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I reviewed the new Gemini CLI landing-page route, localized route wrapper, /agents/ hub and header wiring, the expanded agent-guide copy, and the new WebP assets. The implementation matches the existing agent-detail page family, the new links resolve through the locale-aware helpers, the i18n content files parse under Node 24, and the added images are valid WebP assets. Nice work keeping this page consistent with the existing localized agent surface while adding a full long-form guide.

Note: I could not rerun pnpm --filter @open-design/landing-page typecheck in this prepared worktree because node_modules is absent and astro is not installed locally, but the lightweight syntax checks for the changed TypeScript content files passed.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

The hand-authored INFO_PAGE_COPY['zh-tw'] entry spreads the zh (Simplified)
copy and overrides specific sections, but it never overrode `agentGuides` — so
every agent detail page rendered Simplified Chinese under /zh-tw/ instead of
Traditional (the part-file localization is only consumed by the locales that go
through compactInfoPageCopy, which zh-tw does not).

Add a Traditional Chinese `agentGuides` override to the zh-tw entry (all five
agents, converted with Taiwan terminology) and drop zh-tw from the generated
part files since it never used them. This also fixes the four existing agents
(claude-code, codex, cursor, opencode) that were already shipping Simplified
text on their zh-tw pages.
…Grok, Kimi, DeepSeek)

Extend the agent detail family with five well-documented agents, each built from
the same rich long-form template as the existing pages:

- GitHub Copilot CLI, Qwen Code, Grok CLI, Kimi CLI, DeepSeek TUI
- /agents/<slug>-design/ with hand-structured how-to content (what it is, the
  design angle, setup, screenshot-to-UI, extend, comparison, pitfalls, with
  Open Design), an FAQ with JSON-LD, and a warm editorial hero illustration with
  each agent's real mark fused into the artwork
- wired into the /agents/ hub (DETAIL_ROUTES) and the Agent nav dropdown

Every fact, command, model name, and resource URL was verified against official
sources; nothing fabricated. Localized across all landing locales: en
hand-checked, zh translated, the other 15 machine-translated and merged into the
split agent-guides part files (now 8 parts to stay under the 1 MiB blob guard),
and a Traditional Chinese override added for each agent under the zh-tw entry.
@PerishCode

Copy link
Copy Markdown
Contributor

@xne998808-ai I'm holding off on generating review comments for #4050 because this pull request has merge conflicts right now.

Please resolve the conflicts with main and push the updated branch. Once that's done, request or wait for the review to run again and I'll take another look.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

…d agents

Add the remaining eleven agent detail pages so /agents/ covers every coding
agent Open Design's runtime supports (the hub was previously missing four):

- Trae CLI, Aider, Antigravity, DeepSeek Reasonix (added to the hub + pages)
- Hermes, Devin for Terminal, Pi, Kiro CLI, Kilo, Mistral Vibe CLI, Qoder CLI

Each is built from the same rich long-form template (what it is, the design
angle, setup, reference-to-UI, extend, comparison, pitfalls, with Open Design),
an FAQ with JSON-LD, and a warm editorial hero — the agent's real mark where one
exists, a clean wordmark where it does not. Every fact, command, model name, and
resource URL was verified against the agent's own runtime def and official
sources; sparse agents are described accurately-but-generally rather than with
invented specifics. Also fixes the hub's incorrect vendor for Pi.

Wired into the /agents/ hub (now 21 agents) and the Agent nav dropdown.
Localized across all landing locales: en hand-checked, zh translated, the other
15 machine-translated and merged into the split agent-guides part files (now 15
parts to stay under the 1 MiB blob guard), with a Traditional Chinese override
added for each agent under the zh-tw entry.
@PerishCode

Copy link
Copy Markdown
Contributor

@xne998808-ai I'm holding off on generating review comments for #4050 because this pull request has merge conflicts right now.

Please resolve the conflicts with main and push the updated branch. Once that's done, request or wait for the review to run again and I'll take another look.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

…esign

# Conflicts:
#	apps/landing-page/app/info-page-i18n.ts
@xne998808-ai xne998808-ai reopened this Jun 10, 2026
@xne998808-ai xne998808-ai changed the title feat(landing-page): add the Gemini CLI agent design page feat(landing-page): agent design pages — all 21 supported agents Jun 10, 2026
@lefarcen lefarcen requested a review from PerishCode June 10, 2026 23:25
…n.ts

With all 21 agents inlined, info-page-i18n.ts grew to ~1.5 MiB and tripped the
changed-file blob guard (1 MiB). Move the English and Simplified-Chinese
agentGuides maps into agent-guides.en.i18n.ts / agent-guides.zh.i18n.ts and
import them back, mirroring how the other locales' guides already live in split
part files. No behavior change — same data, just hoisted. All changed files are
now under the blob guard.

@lefarcen lefarcen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @xne998808-ai — covering all 21 supported agents in one go is a solid milestone! One remaining template gap: the Screenshots section currently points to a local preview command, but since UI is checked in Surface area, a couple of attached images (hero + one representative page layout) would let reviewers gauge the visual footprint without spinning up the dev server. Happy to unblock once those are in. 🙌

@nettee nettee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I verified the new hub/detail wiring locally, reran pnpm --filter @open-design/landing-page typecheck, and completed a full static pnpm --filter @open-design/landing-page build. The routes and pages all build, but I found two user-visible consistency issues in the new agent-hub metadata; details are inline.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Location: apps/landing-page/app/pages/agents/index.astro RIGHT lines 66-69

Adding these new slugs without extending localizedCredentialLabels leaves the localized hubs with fallback strings like BYOK · ByteDance / BYOK · community instead of real credential copy. After building this branch, /zh-tw/agents/ and /uk/agents/ render those fallbacks for Trae CLI, Aider, Antigravity, and Reasonix, and the pi entry in the same locale map still shows the old interactive-login text. That makes the PR’s new agents read half-localized and keeps the Pi fix from landing outside English. Please add localized entries for the new slugs here (and update pi) or derive the label from the localized guide copy so the hub cannot drift.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

Comment thread apps/landing-page/app/_components/header.tsx Outdated

@Siri-Ray Siri-Ray left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I reviewed the expanded landing-page agent hub, the new agent detail route family, localized wrappers, guide-data split files, and the new WebP assets. The route/content structure is in good shape overall; I found one small user-visible metadata count drift to clean up. Thanks for pushing the agent page family across the full supported set.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Location: apps/landing-page/app/pages/agents/index.astro RIGHT line 66

Adding these four agents makes the hub render 21 cards via allAgents.length, but the hub metadata still comes from copy.agents.title / copy.agents.description, which currently say “17 BYOK adapters” in English and the zh/zh-tw overrides. That means the browser title, meta description, and localized search snippets can advertise 17 while the page body and this PR now advertise 21. Please update those agent hub title/description strings in info-page-i18n.ts for en, zh, and zh-tw (or derive the count from the same source as the heading) so the metadata cannot drift from the rendered agent list.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

…the data

Address review feedback on the agent hub metadata:

- The browser title / meta description hard-coded "17 BYOK adapters" while the
  hub now renders 21 cards. Switch the en/zh/zh-tw strings to a {count}
  placeholder and substitute allAgents.length in the page, so the metadata can
  no longer drift from the rendered list (also fixes a pre-existing unsubstituted
  {count} in the ja title).
- Localized hubs showed "BYOK · <vendor>" fallbacks for the four newly added
  agents and a stale interactive-login string for Pi. Fall back to each agent's
  localized guide credential before the BYOK·vendor placeholder, and drop the
  stale Pi entries from the per-locale map so every agent shows a real
  localized credential.
The hub cards used 'Qwen' / 'Grok' / 'Cursor Agent' while the nav dropdown and
detail-page breadcrumbs use 'Qwen Code' / 'Grok CLI' / 'Cursor', so the same
routes shipped two public names (and the shorter ones leaked into the hub
JSON-LD). Update the hub tiers to the detail-page names so hub, nav, and
JSON-LD all agree.

@Siri-Ray Siri-Ray left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I reviewed the current landing-page update across the expanded /agents/ hub, the Agent dropdown wiring, the 21 agent detail route family, localized route wrappers, split guide-data files, and referenced WebP assets. The current head resolves the prior hub metadata/name drift by deriving the count from the rendered agent list and keeping the hub/dropdown labels aligned, and the new route/image references are structurally consistent. I could not run the full Astro typecheck in this prepared worktree because node_modules is not installed, but lightweight source checks passed: git diff --check, Node syntax checks for the changed TypeScript guide files, SLUG/ROUTE consistency checks for the agent detail pages, and verification that all referenced agent WebP files exist. Thanks for the broad localization and route coverage here.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

nettee
nettee previously requested changes Jun 11, 2026

@nettee nettee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I rechecked the current head against the expanded /agents/ hub, the extracted guide sources, and the new localized detail routes. The count/name drift from the earlier head is fixed, but I still found two user-visible content/localization gaps that should be corrected before this lands; details are inline.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Location: apps/landing-page/app/pages/agents/index.astro RIGHT line 63

Qoder's hub metadata is still out of sync with the canonical metadata elsewhere in this PR. The new Qoder guide copy describes it as an Alibaba product that authenticates with a Qoder account, and apps/daemon/src/runtimes/defs/qoder.ts still points users at qodercli login / QODER_PERSONAL_ACCESS_TOKEN. Shipping the main /agents/ card as vendor: 'community' plus Provider keys via Qoder config (BYOK) sends users down the wrong onboarding path from the hub, which undermines the PR's stated content-accuracy goal. Please align this row with the same source of truth used by the guide/runtime metadata, e.g. vendor: 'Alibaba' and a credential string that points to the Qoder account login flow.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

Location: apps/landing-page/app/pages/agents/gemini-design/index.astro RIGHT line 48

These new localized detail pages still hardcode the second breadcrumb label as Agents, and the same literal is used again in the visible breadcrumb nav below. On /zh/..., /ja/..., /uk/..., etc. the page body localizes, but this crumb and its BreadcrumbList schema entry stay English, so the PR does not actually ship fully localized navigation/structured data for the new routes. Please source that label from the locale-aware agents copy (for example copy.agents.breadcrumb) in both places, or factor the breadcrumb through a shared helper so every new detail page stays localized consistently.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

@nettee nettee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I reviewed the current head across the expanded /agents/ hub, the Agent dropdown wiring, the split guide-data loader, the localized route wrappers, and the new detail-page family. The previous localization and metadata regressions on earlier heads are fixed; I found one remaining merge-safe maintainability/content-accuracy issue in the agent registry wiring, detailed inline.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Location: apps/landing-page/app/pages/agents/index.astro RIGHT lines 26-37

The new hub registry is still drifting from the runtime registry it says it mirrors. The comment on this changed block says these names match each RuntimeAgentDef, but this head now ships Cursor here while the actual adapter name is still Cursor Agent in apps/daemon/src/runtimes/defs/cursor-agent.ts, and the same shortened label is duplicated again in the header dropdown. That leaves the landing page advertising a different name than the product/runtime surface users have to select later, and it shows this agent list can still fall out of sync even after the earlier hub/header fix on this PR. Please either align these labels with the runtime names, or move the landing-page agent metadata to a shared source/test so the hub, dropdown, and runtime defs cannot drift independently.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

@Siri-Ray Siri-Ray left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I reviewed the current landing-page head across the expanded /agents/ hub, Agent dropdown, localized detail wrappers, split guide-data loader, rich detail-page template, and referenced WebP assets. The earlier localization and Qoder metadata issues are addressed; I found one remaining merge-safe registry-label drift to clean up. Thanks for pushing this broad agent-page family forward.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Location: apps/landing-page/app/pages/agents/index.astro RIGHT line 37

This changed row now advertises the adapter as Cursor, but the comment immediately above says these labels stay in lockstep with each RuntimeAgentDef.name, and the actual runtime definition still exposes name: 'Cursor Agent' in apps/daemon/src/runtimes/defs/cursor-agent.ts. The header dropdown changed to the same shortened label as well, so users can see one name on the landing surface and a different one when selecting the runtime in-product. Please either restore the canonical Cursor Agent label here and in the header entry, or move these landing-page labels behind a shared registry/check so the hub, dropdown, and runtime names cannot drift independently.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

@lefarcen

Copy link
Copy Markdown
Contributor

Hey @xne998808-ai — thanks for pushing the follow-up.

The earlier blockers from @nettee are cleared on this head. The only remaining reviewer feedback I see on 0d40e39 is the merge-safe registry-name drift both @nettee and @Siri-Ray called out: the landing hub/header now say Cursor, while the runtime still exposes Cursor Agent. Worth tightening before merge so the marketing surface and in-product runtime picker stay in sync.

Reviewer noted the hub still drifted from the runtime registry it documents:
the card/dropdown shipped 'Cursor' while `runtimes/defs/cursor-agent.ts` names
the adapter 'Cursor Agent'. Rename the Cursor hub card, nav dropdown entry, and
detail-page breadcrumb (all locales) to 'Cursor Agent' so the landing page
advertises the same name users select in the app, and reword the registry
comment to describe the display-name contract instead of overstating a literal
1:1 mirror.
@xne998808-ai

Copy link
Copy Markdown
Collaborator Author

Fixed in 2ddcf56 — aligned the Cursor agent name to the runtime adapter. The hub card, the Agent dropdown, and the detail-page breadcrumb (all locales) now read Cursor Agent, matching runtimes/defs/cursor-agent.ts, so the landing page advertises the same name users select in the app. Verified on the built branch: /agents/ and /ja/agents/cursor-design/ (visible crumb + BreadcrumbList JSON-LD) both render Cursor Agent. Also reworded the registry comment to describe the display-name contract (hub ↔ dropdown ↔ breadcrumb ↔ RuntimeAgentDef) rather than overstating a literal mirror. (qwen already matches the def as Qwen Code; kept Grok CLI as the established user-facing name for the grok adapter.)

@lefarcen

Copy link
Copy Markdown
Contributor

Hey @xne998808-ai — thanks, that closes the remaining registry-name drift I had flagged on the previous head.

At this point the current branch is down to CI finishing and a fresh reviewer pass on 2ddcf563. Once one of the pool reviewers signs off on this head, it should be in good shape from the maintainer side.

@Siri-Ray Siri-Ray left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I reviewed the current head across the expanded /agents hub, Agent dropdown, localized route wrappers, long-form detail-page template, split guide-data files, and referenced WebP assets. The prior follow-up issues around Qoder credentials, localized breadcrumbs/CTA labels, and Cursor naming are addressed on this head; I also checked the route/header/hub consistency, wrapper coverage, image references, file sizes, and whitespace. I could not run the full Astro typecheck because this prepared worktree has no node_modules/astro installed, but the lightweight checks did not surface any actionable issues. Thanks for the broad agent-page and localization work here.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

@nettee nettee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I rechecked the current landing-page head across the expanded /agents/ hub, the Agent dropdown wiring, and the new detail-page family. The earlier localization/content blockers are resolved; I found two remaining merge-safe consistency issues in the new agent surface that are worth tightening before merge.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Location: apps/landing-page/app/pages/agents/index.astro RIGHT line 49

This new hub row still breaks the runtime-name contract the comment above describes. The landing surface now calls this adapter Grok CLI, but the actual runtime definition is still name: 'Grok Build' in apps/daemon/src/runtimes/defs/grok-build.ts, and the header dropdown was updated to the same Grok CLI label. That means users can arrive from /agents/grok-design/ looking for Grok CLI and then fail to find the same name in the product runtime picker, even though the PR body says the page metadata was verified against the runtime defs. Please either align the landing-page label with the current runtime name, or rename the runtime from one shared source/test so the hub, dropdown, and runtime picker cannot drift again.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

Location: apps/landing-page/app/pages/agents/gemini-design/index.astro RIGHT line 72

These new detail pages still pass active="home" into Layout. In app/_components/header.tsx, active="home" highlights the Product menu, while the Agent dropdown only highlights when active === 'agent'. As shipped, every new /agents/*-design/ page renders with the wrong top-nav active state even though this PR is introducing the dedicated Agent hub and dropdown. Please switch these detail routes to active="agent" (and apply the same change to the other generated *-design pages, plus the /agents/ hub) so the navigation state matches the route family.

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

Inline comment could not be anchored: inline anchor is outside the PR diff anchorable ranges

@lefarcen

Copy link
Copy Markdown
Contributor

Hey @xne998808-ai — thanks for the latest pass here.

The earlier Cursor Agent drift is fixed on 2ddcf563. On the current head, the remaining public feedback I see is @nettee's two merge-safe consistency follow-ups:

  1. Grok CLI on the landing hub/dropdown still doesn't match the current runtime name Grok Build.
  2. The new *-design pages still pass active="home" into Layout, so the top nav highlights Product instead of Agent on those routes.

Separately, CI is still red on this head (Validate workspace gate and Runner CI), so it isn't merge-ready yet even though @Siri-Ray has already approved.

Two more review-flagged consistency fixes:

- Every /agents/*-design/ detail page and the /agents/ hub passed active="home"
  to Layout, which highlights the Product menu instead of the Agent dropdown.
  Switch them to active="agent" so the top-nav active state matches the route.
- The hub/dropdown/guide called the grok adapter "Grok CLI" while the runtime
  def is name: 'Grok Build'. Rename it to "Grok Build" across the landing
  surface (hub, dropdown, breadcrumb, and guide copy in every locale) so the
  advertised name matches the runtime picker users select in the app.
@xne998808-ai

Copy link
Copy Markdown
Collaborator Author

Both fixed in 9354db7:

  • Nav active state — every /agents/*-design/ detail page and the /agents/ hub now pass active="agent" instead of active="home", so the top nav highlights the Agent dropdown rather than the Product menu. Verified the built gemini-design page renders the Agent nav item with is-active.
  • Grok name — renamed the grok adapter to Grok Build across the landing surface (hub card, nav dropdown, breadcrumb, and guide copy in every locale) to match runtimes/defs/grok-build.ts. Built /agents/grok-design/ now titles as "Grok Build for design". That brings all three flagged adapters in line with the runtime names (Qwen Code / Grok Build / Cursor Agent).

@Siri-Ray Siri-Ray left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@xne998808-ai I reviewed the current landing-page head across the expanded /agents hub, Agent dropdown, localized route wrappers, the 21 detail-page family, split guide-data files, and referenced WebP assets. The latest head addresses the previous Qoder credential, CTA localization, Cursor/Grok naming, and active-nav feedback; I also checked hub/header/route consistency, localized wrapper coverage, WebP references, changed-file size limits, whitespace, and TypeScript syntax for the split guide files. I could not run the full Astro typecheck in this prepared worktree because node_modules is missing and astro is not installed, but the lightweight checks did not surface actionable issues. Thanks for the thorough route and localization work here. 🙌

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

@lefarcen

Copy link
Copy Markdown
Contributor

Hey @xne998808-ai — thanks for turning those around quickly.

The two follow-ups I called out on 2ddcf563 are addressed on 9354db7: the landing surface now uses Grok Build consistently with the runtime, and the *-design pages / /agents/ hub are on active="agent" so the top-nav state matches the route family.

From here the branch looks down to CI finishing and refreshed reviewer state on this head.

@xne998808-ai xne998808-ai dismissed stale reviews from nettee and nettee June 11, 2026 07:26

Dismissing as outdated: every issue in this review was fixed in later commits — breadcrumb localization and the Qoder hub metadata were corrected in 9e67f44. Your own subsequent reviews on 0d40e39 and 2ddcf56 confirmed the earlier localization/content blockers are resolved, and the current head (9354db7) carries a fresh code-owner approval from Siri-Ray.

@xne998808-ai xne998808-ai added this pull request to the merge queue Jun 11, 2026
Merged via the queue into main with commit eab3926 Jun 11, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk/medium Medium risk: regular code changes size/XXL PR changes 1500+ lines type/feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants