Skip to content

fix: update remaining opencode references in help output and CLI#547

Open
VJ-yadav wants to merge 1 commit intoAltimateAI:mainfrom
VJ-yadav:fix/help-branding-opencode-refs
Open

fix: update remaining opencode references in help output and CLI#547
VJ-yadav wants to merge 1 commit intoAltimateAI:mainfrom
VJ-yadav:fix/help-branding-opencode-refs

Conversation

@VJ-yadav
Copy link
Copy Markdown

@VJ-yadav VJ-yadav commented Mar 28, 2026

Summary

Fixes #416 — Updates user-facing opencode references to altimate-code across help text, CLI commands, and mDNS discovery.

Changed (8 files, 19 replacements):

  • thread.ts — TUI help descriptions
  • uninstall.ts — uninstall command description
  • pr.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 domain opencode.localaltimate-code.local
  • thread.test.ts — test expectations for mDNS domain
  • types.gen.ts — JSDoc mDNS domain reference

Intentionally NOT changed: Internal identifiers (@opencode/Auth, ProviderID.opencode, OPENCODE_* env vars), @opencode-ai/* package names, .opencode config directories, npm registry names in uninstall, User-Agent/X-Title protocol headers.

Test Plan

  • bun turbo typecheck passes across all 5 packages
  • bun test test/cli/tui/thread.test.ts passes with updated mDNS expectations
  • Manual: altimate-code --help shows no opencode in descriptions
  • Manual: altimate-code pr messages reference altimate-code

Checklist

  • Follows conventional commit format
  • No breaking changes to internal APIs or env vars
  • Tests updated for changed behavior

Summary by CodeRabbit

  • Chores
    • Updated all CLI command descriptions and user-facing messages to reference "altimate-code" instead of "opencode", including command help text and output messages.
    • Updated default mDNS domain and service naming from "opencode" to "altimate-code" for network discovery and local service identification.

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>
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@github-actions
Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

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.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 4c0edf43-dd26-4f51-b374-213635dbfbfd

📥 Commits

Reviewing files that changed from the base of the PR and between 33c331b and 377e484.

⛔ Files ignored due to path filters (1)
  • packages/sdk/js/src/v2/gen/types.gen.ts is excluded by !**/gen/**
📒 Files selected for processing (7)
  • packages/opencode/src/cli/cmd/pr.ts
  • packages/opencode/src/cli/cmd/tui/thread.ts
  • packages/opencode/src/cli/cmd/uninstall.ts
  • packages/opencode/src/cli/network.ts
  • packages/opencode/src/config/config.ts
  • packages/opencode/src/server/mdns.ts
  • packages/opencode/test/cli/tui/thread.test.ts

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
PR and TUI Command Descriptions
packages/opencode/src/cli/cmd/pr.ts, packages/opencode/src/cli/cmd/tui/thread.ts
Updated command describe strings from "opencode" to "altimate-code"; PR command also updated session detection log message and import command reference from opencode import to altimate-code import.
Uninstall Command Description
packages/opencode/src/cli/cmd/uninstall.ts
Updated describe string for uninstall command from referencing "opencode" to "altimate-code".
mDNS Configuration and Defaults
packages/opencode/src/cli/network.ts, packages/opencode/src/config/config.ts, packages/opencode/src/server/mdns.ts
Updated mDNS domain defaults and descriptions from opencode.local to altimate-code.local across CLI options, config schema, and service publishing logic.
Test Fixtures
packages/opencode/test/cli/tui/thread.test.ts
Updated test setup to use altimate-code.local as default mDNS domain value.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested labels

contributor

Poem

🐰 A hop, skip, and brand new name,
From opencode to altimate's fame!
Strings and domains all freshly aligned,
No logic changed, just clarity refined! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The PR title concisely and clearly describes the main change: updating outdated opencode references to altimate-code in help text and CLI commands.
Description check ✅ Passed The PR description follows the template with Summary, Test Plan, and Checklist sections; clearly explains the changes across 8 files and provides context about intentional non-changes.
Linked Issues check ✅ Passed The PR directly addresses issue #416 by replacing user-visible opencode references with altimate-code in CLI help, TUI hints, mDNS domains, and the pr command binary spawn invocation—resolving incorrect instructions.
Out of Scope Changes check ✅ Passed All changes are in-scope and directly support the branding update objective; internal identifiers, package names, config directories, and protocol headers were intentionally preserved as documented.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Altimate code Hints on starting TUI are referred from upstream provider (OC)

1 participant