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
Contributor

@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

    • Default mDNS domain changed to altimate-code.local.
    • Service names and status messages now use the altimate-code prefix.
    • PR/CLI session flow now launches and displays altimate-code tooling/status text.
  • Tests

    • Test inputs and mocks updated to use the new altimate-code mDNS domain and naming.

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

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

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: 265437c5-1a23-48c5-8dc8-0084782b0d66

📥 Commits

Reviewing files that changed from the base of the PR and between dd6b929 and 19c42d1.

⛔ Files ignored due to path filters (1)
  • packages/sdk/js/src/v2/gen/types.gen.ts is excluded by !**/gen/**
📒 Files selected for processing (5)
  • packages/opencode/src/cli/cmd/pr.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
✅ Files skipped from review due to trivial changes (4)
  • packages/opencode/src/server/mdns.ts
  • packages/opencode/test/cli/tui/thread.test.ts
  • packages/opencode/src/cli/network.ts
  • packages/opencode/src/config/config.ts
🚧 Files skipped from review as they are similar to previous changes (1)
  • packages/opencode/src/cli/cmd/pr.ts

📝 Walkthrough

Walkthrough

The PR renames runtime targets and mDNS defaults from "opencode" to "altimate-code": CLI PR command now imports and spawns altimate-code, mDNS defaults and published service names switch to altimate-code.local / altimate-code-<port>, and tests/config docs updated accordingly.

Changes

Cohort / File(s) Summary
PR Command Execution
packages/opencode/src/cli/cmd/pr.ts
Replaced invocations and user-facing strings from opencode to altimate-code: importer call, spawned TUI, and non-zero-exit error message. Session URL detection and sessionId extraction unchanged.
Network CLI & Config docs
packages/opencode/src/cli/network.ts, packages/opencode/src/config/config.ts
Changed CLI option default description and Zod .describe() from opencode.local to altimate-code.local.
mDNS Publish
packages/opencode/src/server/mdns.ts
Default Bonjour host and service name fallbacks changed to altimate-code.local and altimate-code-<port> respectively.
Tests
packages/opencode/test/cli/tui/thread.test.ts
Updated test fixture mDNS domain values from opencode.local to altimate-code.local.

Sequence Diagram(s)

sequenceDiagram
  participant User as "User"
  participant CLI as "opencode CLI\n(packages/opencode/src/cli/cmd/pr.ts)"
  participant Importer as "altimate-code\n(import subcommand)"
  participant TUI as "altimate-code TUI\n(spawned process)"

  rect rgba(200,220,255,0.5)
    User->>CLI: run `pr` command with PR/session URL
    CLI->>Importer: Process.text(["altimate-code","import", sessionUrl])
    Importer-->>CLI: stdout with sessionId
    CLI->>TUI: spawn("altimate-code", ["tui","--session", sessionId])
    TUI-->>User: TUI UI appears
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

contributor

Poem

🐰 I hopped through code and changed the name,

swapped opencode for altimate-code fame,
a little nudge, a tiny tweak,
now services bloom and tests will speak,
I munch a carrot — build and peak!

🚥 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 accurately reflects the main change: updating branding references from 'opencode' to 'altimate-code' in user-facing help output and CLI commands.
Description check ✅ Passed The PR description is comprehensive, covering summary, detailed changes across 8 files, intentional exclusions, completed and pending test verification, and a proper checklist aligned with the template.
Linked Issues check ✅ Passed All changes directly address issue #416 requirements: user-facing 'opencode' references updated to 'altimate-code' in TUI hints and CLI help text, mDNS domain updated, tests adjusted, and internal identifiers properly preserved.
Out of Scope Changes check ✅ Passed All changes are directly in-scope for issue #416; no extraneous modifications detected. Changes are limited to user-facing references and related test expectations.

✏️ 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.

@VJ-yadav VJ-yadav force-pushed the fix/help-branding-opencode-refs branch from 377e484 to ade8d48 Compare April 3, 2026 06:10
@dev-punia-altimate
Copy link
Copy Markdown

❌ Tests — Failures Detected

TypeScript — 15 failure(s)

  • connection_refused [2.50ms]
  • timeout [2.32ms]
  • permission_denied [2.50ms]
  • parse_error [2.22ms]
  • oom [2.48ms]
  • network_error [2.32ms]
  • auth_failure [2.42ms]
  • rate_limit [2.56ms]
  • internal_error [2.62ms]
  • empty_error [0.23ms]
  • connection_refused [0.15ms]
  • timeout [0.08ms]
  • permission_denied [0.06ms]
  • parse_error [0.08ms]
  • oom [0.06ms]

cc @VJ-yadav
Tested at ade8d48d | Run log | Powered by QA Autopilot

@anandgupta42
Copy link
Copy Markdown
Contributor

Review — Ready for merge

Scope: Fixes #416 — replaces user-facing opencode references with altimate-code across help text, CLI commands, and mDNS discovery. 19 replacements across 8 files.

Analysis

  • Mostly branding text updates (thread.ts TUI help, uninstall description, pr.ts user-facing messages)
  • Functional fix: pr.tsspawn("opencode", ...)spawn("altimate-code", ...). This was an actual bug on rebranded installs where the old binary name doesn't exist. Legit runtime fix hidden in a rename PR
  • mDNS domain opencode.localaltimate-code.local across network.ts, config.ts, mdns.ts + test expectations in thread.test.ts + JSDoc in types.gen.ts
  • Correctly preserved: internal identifiers (@opencode/Auth, ProviderID.opencode, OPENCODE_* env vars), @opencode-ai/* package names, .opencode config dirs, npm registry names, User-Agent/X-Title protocol headers. This is the right call — breaking those would cascade across tooling and CI
  • Scope discipline is solid — only user-visible surface changed

Files changed (6): thread.ts, uninstall.ts, pr.ts, network.ts, config.ts, mdns.ts + test (+15 −15)

CI status (commit ade8d48d)

  • ✅ TypeScript, Marker Guard, anti-slop, check-standards, check-compliance, GitGuardian — all green
  • ⚠️ centralized-test-results — same batch-wide flakiness

Recommendation
Ready for merge — rebase on main. Clean rebrand with a bonus functional fix. No changes requested.

cc @anandgupta42 — ready for /release once rebased.

@anandgupta42 anandgupta42 force-pushed the fix/help-branding-opencode-refs branch from 92424f7 to 27e9fd4 Compare April 5, 2026 06:41
@anandgupta42
Copy link
Copy Markdown
Contributor

Consensus code-review applied — no changes needed

Ran a 4-participant consensus review (Claude + GPT 5.4 + Gemini 3.1 Pro + self-review).

Verdict: APPROVE — mostly user-facing branding + one functional fix (spawn("opencode"…)spawn("altimate-code"…) in pr.ts). mDNS domain renames are consistently applied across network.ts, config.ts, mdns.ts, thread.test.ts, and generated SDK types.

GPT flagged two MAJOR concerns that I investigated and assessed as false positives:

  1. "Session import URL mismatch" — the opncd.ai/s/... regex at pr.ts:87 is correct. opncd.ai IS altimate-code's intentional share-link shortener, as confirmed by the rebrand markers in script/upstream/analyze.ts and script/upstream/merge.ts. The comment "altimate-code session link" is accurate.
  2. "No test coverage for spawn change" — writing a full mock-test for spawn() + gh CLI + git is disproportionate for a literal binary-name change. This change is exercised by any real-world altimate-code pr invocation and would fail loudly on rebranded installs if incorrect.

The intentionally-preserved opencode references (@opencode/Auth, ProviderID.opencode, OPENCODE_* env vars, @opencode-ai/* package names, .opencode config dirs, User-Agent/X-Title protocol headers) are correctly left untouched — breaking those would cascade across tooling.

All CI green. Ready for merge as-is.

@VJ-yadav
Copy link
Copy Markdown
Contributor Author

Hi @anandgupta42 — approved + consensus review passed on Apr 5, CI green. Just needs a rebase (behind main, no conflicts). Could you merge when you get a moment?

@VJ-yadav VJ-yadav force-pushed the fix/help-branding-opencode-refs branch from 27e9fd4 to dd6b929 Compare April 13, 2026 20:11
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>
@VJ-yadav VJ-yadav force-pushed the fix/help-branding-opencode-refs branch from dd6b929 to 19c42d1 Compare April 14, 2026 00:43
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)

3 participants