[lilyshen0722]

Written by Quire, a Commonly agent — https://commonly.me/v2/pods/6aaee2af1e8446ae11fb1457

TASK-017 — Bring your own pages: walk record decides each one

Tier 0, docs only (plus one unit-test list). The README already dropped Hermes and DeepSeek Harness; the docs site still published both BYO pages. Each page stays only with a dated walk record linked here.

DeepSeek Harness: kept

• In-repo walk record: docs/cli/dsh.md (lines 82–83): walked with a non-admin Commonly account on macOS with CLI 0.1.117, then re-walked on CLI 0.1.122 on 2026-10-23. The page names CLI 0.1.122 and dsh 0.2.1-rc.2, which match the re-walk.
• Merged PR fix(scout): guide DeepSeek Harness connection #2404 (fix(scout): guide DeepSeek Harness connection) records the same non-admin macOS walk.
• Gap, for GTM to rule on: neither record cites a pod message in which an @mention was answered. The walk record shows the attach and run path, not the reply. If the press needs the reply cited, the page needs a message ID from the re-walk.

Hermes: unlisted

• The only Hermes walk on record is PA-1, cited in PR fix(scout): list Hermes as a command-line connect, with its walk and caveats (TASK-429) #2569's body (10-22 16:30Z, @mention answered in 56 s). That walk does not cover this page:
  • it ran on a commit before the probe fix (the PR body says so);
  • it used a provider other than OpenRouter, while the page tells the reader to use an OpenRouter key.
• Removed: the page file, its nav entry, and its card on the Bring your own index. The index description no longer names Hermes.
• To restore: re-walk on the current head with the setup the page describes, then put that record in the PR body.

Test

backend/__tests__/unit/scripts/docsSiteFacts.test.js: Hermes moves from the required nav list to the not-listed list, so the test enforces the unlist.

Verification

• npx jest __tests__/unit/scripts/docsSiteFacts.test.js (Node 26 locally): 6/6 pass.
• node --check on the test file and git diff --check: pass.
• docs-site/docs.json parses.
• cd docs-site && npx -y mint@4.2.1028 broken-links: no broken links.
• grep -rn hermes docs-site: no references remain.

Not changed

• frontend/public/llms.txt has no Hermes or DeepSeek Harness line, so nothing came off it.
• The in-app BYO copy belongs to TASK-012.

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: Team-Commonly/commonly
• Branch: main
• File: .coderabbit.yaml
• Commit: 31289b5
• Layers: Repository YAML
• Review profile: QUIET
• Plan: Advanced
• Run ID: d8055956-2318-4ca6-9131-a6b700cbe197

Commits

Reviewing files that changed from the base of the PR and between 31289b5 and 6b6e3af.

Files selected for processing: 4

• backend/__tests__/unit/scripts/docsSiteFacts.test.js
• docs-site/bring-your-own/hermes.mdx
• docs-site/bring-your-own/index.mdx
• docs-site/docs.json

Files with no reviewable changes: 2

• docs-site/docs.json
• docs-site/bring-your-own/hermes.mdx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 1 remain after this review.

Walkthrough

The Hermes Bring Your Own Agent guide was deleted. Its page description, guide card, and navigation entry were removed. The navigation test now expects Hermes to be absent.

Changes: Hermes BYO documentation

Layer / File(s)	Summary
Remove Hermes guide and listings docs-site/bring-your-own/hermes.mdx, docs-site/bring-your-own/index.mdx, docs-site/docs.json, backend/__tests__/unit/scripts/docsSiteFacts.test.js	The Hermes setup guide and its references were removed. The navigation test now checks that Hermes is absent.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Possibly related PRs

• Team-Commonly/commonly#2480: Added the Hermes guide, card, and navigation entry that this change removes.
• Team-Commonly/commonly#2582: Updated the Hermes guide that this change removes.
• Team-Commonly/commonly#2569: Added Hermes to Scout CLI guidance, which is related to the Hermes BYO documentation.

Merge Risk: Minimal · up to 6b6e3

No actionable merge-blocking risk remains in this documentation-only change.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly summarizes the main documentation change: removing the Hermes BYO page while retaining the DeepSeek Harness page based on its walk record.
Description check		The description directly explains the Hermes removal, the DeepSeek Harness decision, the test changes, and the reported verification results.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}