chore: scrub named other-CLI references from the codebase - #307
Conversation
AGENTS.MD described Zero as "in the same family as Claude Code, Codex,
and Droid". Because AGENTS.MD is injected into the model as project
context, the agent parroted that sentence verbatim when asked who it is,
naming other agents in its own identity.
Reword it to describe Zero on its own terms ("an open-source terminal
coding agent") with no comparison to other products.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✅ Files skipped from review due to trivial changes (5)
WalkthroughDocumentation and inline comments were updated to remove specific agent-name references and replace them with generalized wording about multi-provider terminal agents and configurable TUI behavior. ChangesDocumentation wording update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
Comment |
Zero automated PR reviewVerdict: No blockers found Blockers
Validation
ScopeHead: This deterministic review checks validation status and basic diff hygiene. A human reviewer still owns product judgment and design quality. |
Follow-up to the AGENTS.MD wording fix: sweep the rest of the tree for
comments and docs that named specific other CLIs and replace them with
neutral phrasing ("comparable terminal agents" / generic positioning).
- internal/tui/rendering.go, model.go, session_controls.go (+ test):
drop the named other-agent references from doc comments; the behaviour
and the existing generic "reference agents/TUIs" phrasing are unchanged.
- docs/PRD.md: replace the named competitor in the target/vision with
generic "multi-provider terminal agents".
The ChatGPT Codex provider integration keeps its name — that is a real
product Zero connects to, not a reference to another tool's source.
What
Scrub every named other-CLI reference out of the codebase and replace it with neutral phrasing. Two motivations:
AGENTS.MDdescribed Zero as "in the same family as Claude Code, Codex, and Droid." SinceAGENTS.MDis injected into the model as project context, the agent parroted that line verbatim when asked "who are you?" — naming other agents in its own identity.Changes
AGENTS.MD— Zero is now described on its own terms ("an open-source terminal coding agent"), no comparison.internal/tui/rendering.go,model.go,session_controls.go(+ test) — doc comments that named other CLIs now say "comparable terminal agents"; the existing generic "reference agents/TUIs" phrasing is unchanged. Comment-only, no behaviour change.docs/PRD.md— target/vision use generic "multi-provider terminal agents" instead of a named product.Claude CodeandCursorare kept as legitimate public-competitor positioning.Not touched (intentionally)
internal/providers/openai/codex*.go) keeps its name — that's a real product Zero connects to, not a reference to another tool's source.factory.go,ClientFactory) — ordinary software term.Verification
go build ./...+gofmtclean. A tracked-file sweep foropencode|openclaude|vix|droidreturns nothing.Summary by CodeRabbit