claude: Add a note in CONTRIBUTING.md stating that routing is now done automatically without command prefix. #9

2026-08-23T15:36:46Z

claude-bot commented

2026-08-23 15:36:46 +00:00

Requested via Matrix:

Add a note in CONTRIBUTING.md stating that routing is now done automatically without command prefix.

Requested via Matrix: > Add a note in CONTRIBUTING.md stating that routing is now done automatically without command prefix.

claude-bot added 1 commit 2026-08-23 15:36:46 +00:00

claude: Add a note in CONTRIBUTING.md stating that routing is now done automatically without command prefix. 01580222f9

claude-bot added 1 commit 2026-08-23 15:36:47 +00:00

claude: Add a note in CONTRIBUTING.md stating that routing is now done automatically without command prefix. 01580222f9

claude-bot commented

2026-08-23 15:37:05 +00:00

The claims in the doc check out — agent/src/router.js exists and does the chat/code_task classification, and the prior commit confirms prefixes were dropped in favor of routing through LiteLLM.

Since this is a brand-new CONTRIBUTING.md (not an edit to an existing file), there's not much surface for bugs, but a few notes:

Correctness/accuracy

Minor: the note says routing goes "via the LiteLLM gateway and agent/src/router.js's classifier," but per the referenced commit, only the chat path is auto-routed through LiteLLM (openrouter/auto) — code_task requests bypass LiteLLM entirely and go straight to the Claude Code CLI (runChatTask()). The current phrasing could read as if LiteLLM handles all routing/all messages, which slightly overstates its role. Worth tightening to something like: "classified by router.js into chat (routed via the LiteLLM gateway) or code_task (handled directly by the Claude Code CLI)."

Simplification/reuse

Nothing to flag — it's a small, self-contained doc addition with no code changes, no duplication, and no efficiency concerns.

Overall this is a low-risk documentation-only PR; the one suggestion above is optional wording precision, not a blocker.

The claims in the doc check out — `agent/src/router.js` exists and does the chat/code_task classification, and the prior commit confirms prefixes were dropped in favor of routing through LiteLLM. Since this is a brand-new `CONTRIBUTING.md` (not an edit to an existing file), there's not much surface for bugs, but a few notes: **Correctness/accuracy** - Minor: the note says routing goes "via the LiteLLM gateway and `agent/src/router.js`'s classifier," but per the referenced commit, only the *chat* path is auto-routed through LiteLLM (`openrouter/auto`) — `code_task` requests bypass LiteLLM entirely and go straight to the Claude Code CLI (`runChatTask()`). The current phrasing could read as if LiteLLM handles all routing/all messages, which slightly overstates its role. Worth tightening to something like: "classified by `router.js` into chat (routed via the LiteLLM gateway) or code_task (handled directly by the Claude Code CLI)." **Simplification/reuse** - Nothing to flag — it's a small, self-contained doc addition with no code changes, no duplication, and no efficiency concerns. Overall this is a low-risk documentation-only PR; the one suggestion above is optional wording precision, not a blocker.

You are not authorized to merge this pull request.

This pull request can be merged automatically.

This branch is out-of-date with the base branch

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin claude/9b0ae7ad:claude/9b0ae7ad

git checkout claude/9b0ae7ad

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: william/gitops-automation#9