fix: update remaining opencode references in help output and CLI#547
fix: update remaining opencode references in help output and CLI#547VJ-yadav wants to merge 1 commit intoAltimateAI:mainfrom
Conversation
Replace user-visible "opencode" branding in CLI --help descriptions, mDNS defaults, and pr command binary spawns with "altimate-code". Closes AltimateAI#417, relates to AltimateAI#416. Co-Authored-By: Vijay Yadav <vijay@studentsucceed.com>
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThis PR updates user-facing CLI command descriptions, configuration strings, and default mDNS domain values from "opencode" to "altimate-code" across the CLI package and configuration layer, maintaining all existing functionality while aligning messaging with altimate-code branding. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested labels
Poem
🚥 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 unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Fixes #416 — Updates user-facing
opencodereferences toaltimate-codeacross help text, CLI commands, and mDNS discovery.Changed (8 files, 19 replacements):
thread.ts— TUI help descriptionsuninstall.ts— uninstall command descriptionpr.ts— user-facing messages +spawn("opencode", ...)→spawn("altimate-code", ...)(functional fix — old binary name fails on rebranded installs)network.ts,config.ts,mdns.ts— mDNS domainopencode.local→altimate-code.localthread.test.ts— test expectations for mDNS domaintypes.gen.ts— JSDoc mDNS domain referenceIntentionally NOT changed: Internal identifiers (
@opencode/Auth,ProviderID.opencode,OPENCODE_*env vars),@opencode-ai/*package names,.opencodeconfig directories, npm registry names in uninstall,User-Agent/X-Titleprotocol headers.Test Plan
bun turbo typecheckpasses across all 5 packagesbun test test/cli/tui/thread.test.tspasses with updated mDNS expectationsaltimate-code --helpshows noopencodein descriptionsaltimate-code prmessages referencealtimate-codeChecklist
Summary by CodeRabbit