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

    • Updated default mDNS domain configuration to altimate-code.local.
    • Service names and status messages now use the altimate-code prefix.
    • PR command workflow now integrates with altimate-code tooling and displays matching status text.
  • Tests

    • Test inputs updated to reflect the new default mDNS domain.

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

Warning

Rate limit exceeded

@anandgupta42 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 29 minutes and 53 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 29 minutes and 53 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 97b9ee97-6897-4f12-8a8c-543158c1959f

📥 Commits

Reviewing files that changed from the base of the PR and between 92424f7 and 27e9fd4.

⛔ 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
📝 Walkthrough

Walkthrough

This PR updates branding and runtime targets from "opencode" to "altimate-code": CLI PR command now imports/spawns altimate-code, mDNS defaults and service names change 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 user-facing strings and runtime invocations from opencode to altimate-code: importer call, TUI spawn, and non-zero-exit error message. PR body session URL matching remains unchanged.
Network & Server Configuration
packages/opencode/src/cli/network.ts, packages/opencode/src/config/config.ts, packages/opencode/src/server/mdns.ts
Changed default mDNS domain in CLI option and Zod .describe() to altimate-code.local; MDNS.publish now defaults host/name to altimate-code.local and altimate-code-<port>.
Test Fixtures
packages/opencode/test/cli/tui/thread.test.ts
Updated test inputs to use mdns-domain / mdnsDomain = 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 URL)
    CLI->>Importer: Process.text(["altimate-code","import", sessionUrl])
    Importer-->>CLI: stdout includes 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

🐰 A tiny hop from old to new,

opencode swapped for altimate hue,
Services renamed, the TUI's due,
I nibble carrots — build and brew! 🥕

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is comprehensive and follows the template structure with Summary, Test Plan, and Checklist sections. However, it is missing the required 'PINEAPPLE' marker at the top as specified in the description template for AI-generated contributions. Add 'PINEAPPLE' at the very beginning of the PR description before the Summary section, as required by the repository template for AI-generated contributions.
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 (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating opencode references to altimate-code in help output and CLI across multiple files.
Linked Issues check ✅ Passed The PR fully addresses issue #416 by replacing user-facing opencode references with altimate-code in help text, CLI commands (pr.ts), and mDNS domain names, while preserving internal identifiers as required.
Out of Scope Changes check ✅ Passed All changes are within scope and directly address issue #416. The PR correctly avoids modifying internal identifiers, package names, and configuration directories while updating only user-facing references.

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

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>
@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.

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