fix: rename opencode references to altimate-code in test files - #24
Merged
Conversation
Update test files to use the correct names after the config loader was renamed from opencode to altimate-code: - `opencode.json` → `altimate-code.json` - `.opencode/` → `.altimate-code/` - `.git/opencode` → `.git/altimate-code` - `OPENCODE_*` env vars → `ALTIMATE_CLI_*` - Cache dir `opencode` → `altimate-code` - Schema URL `opencode.ai` → `altimate-code.dev` Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
Comment on lines
10
to
+19
| import { Filesystem } from "../../src/util/filesystem" | ||
|
|
||
| // Get managed config directory from environment (set in preload.ts) | ||
| const managedConfigDir = process.env.OPENCODE_TEST_MANAGED_CONFIG_DIR! | ||
| const managedConfigDir = process.env.ALTIMATE_CLI_TEST_MANAGED_CONFIG_DIR! | ||
|
|
||
| afterEach(async () => { | ||
| await fs.rm(managedConfigDir, { force: true, recursive: true }).catch(() => {}) | ||
| }) | ||
|
|
||
| async function writeManagedSettings(settings: object, filename = "opencode.json") { | ||
| async function writeManagedSettings(settings: object, filename = "altimate-code.json") { |
There was a problem hiding this comment.
Bug: User-facing documentation and theme configuration files were not updated with the new altimate-code naming, retaining old opencode references, which will cause user confusion and misconfiguration.
Severity: MEDIUM
Suggested Fix
Update the documentation files packages/altimate-code/test/AGENTS.md and packages/altimate-code/src/acp/README.md to reflect the new altimate-code naming for configurations and environment variables. In all theme JSON files, change the $schema URL from https://opencode.ai/theme.json to https://altimate-code.dev/theme.json.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: packages/altimate-code/test/config/config.test.ts#L10-L19
Potential issue: The pull request, which intends to rename references from "opencode" to
"altimate-code", is incomplete. Several user-facing documentation and configuration
files still contain the old naming. Specifically,
`packages/altimate-code/test/AGENTS.md` and `packages/altimate-code/src/acp/README.md`
instruct users to use old configuration names and environment variables like
`OPENCODE_ENABLE_QUESTION_TOOL`. This will cause confusion and configuration failures.
Additionally, all 35 theme JSON files under
`packages/altimate-code/src/cli/cmd/tui/context/theme/` reference an outdated schema
URL, `https://opencode.ai/theme.json`, which may break IDE validation and indicates an
incomplete migration.
Did we get this right? 👍 / 👎 to inform future reviews.
- Fix build.ts solid-plugin import to use bare specifier for monorepo hoisting - Update agent tests: "build" → "builder", "plan" → "analyst" for disabled fallback - Fix well-known config mock URL in config.test.ts - Fix message-v2 test: "OpenCode" → "Altimate CLI" - Fix retry.test.ts: replace unsupported test.concurrent with test - Fix read.test.ts: update agent name to "builder" - Fix agent-color.test.ts: update config keys to "builder" - Fix registry.test.ts: remove unpublished plugin dep from test fixture - Skip adding plugin dependency in local dev mode (installDependencies) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Update theme schema URL from opencode.ai to altimate-code.dev (33 files) - Rename opencode references in ACP README.md and AGENTS.md docs - Update test fixture tmp dir prefix to altimate-code-test- - Install warehouse extras in Python CI for duckdb/boto3 test deps Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…tests/ - Allow SqlGuardResult.data to be None (fixes lineage.check Pydantic error) - Set testpaths = ["tests"] in pyproject.toml to exclude src/test_local.py from pytest collection (it's a source module, not a test) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove unused imports in server.py (duplicate imports, unused models) - Remove unused `json` import in schema/cache.py - Remove unused `os` import in sql/feedback_store.py - Add noqa for keyring availability check import Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
anandgupta42
added a commit
that referenced
this pull request
Mar 17, 2026
* fix: rename opencode references to altimate-code in all test files Update test files to use the correct names after the config loader was renamed from opencode to altimate-code: - `opencode.json` → `altimate-code.json` - `.opencode/` → `.altimate-code/` - `.git/opencode` → `.git/altimate-code` - `OPENCODE_*` env vars → `ALTIMATE_CLI_*` - Cache dir `opencode` → `altimate-code` - Schema URL `opencode.ai` → `altimate-code.dev` Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: resolve remaining test failures and build import issue - Fix build.ts solid-plugin import to use bare specifier for monorepo hoisting - Update agent tests: "build" → "builder", "plan" → "analyst" for disabled fallback - Fix well-known config mock URL in config.test.ts - Fix message-v2 test: "OpenCode" → "Altimate CLI" - Fix retry.test.ts: replace unsupported test.concurrent with test - Fix read.test.ts: update agent name to "builder" - Fix agent-color.test.ts: update config keys to "builder" - Fix registry.test.ts: remove unpublished plugin dep from test fixture - Skip adding plugin dependency in local dev mode (installDependencies) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: address Sentry review comments and Python CI deps - Update theme schema URL from opencode.ai to altimate-code.dev (33 files) - Rename opencode references in ACP README.md and AGENTS.md docs - Update test fixture tmp dir prefix to altimate-code-test- - Install warehouse extras in Python CI for duckdb/boto3 test deps Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: Python CI — SqlGuardResult allows None data, restrict pytest to tests/ - Allow SqlGuardResult.data to be None (fixes lineage.check Pydantic error) - Set testpaths = ["tests"] in pyproject.toml to exclude src/test_local.py from pytest collection (it's a source module, not a test) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: resolve ruff lint errors in Python engine - Remove unused imports in server.py (duplicate imports, unused models) - Remove unused `json` import in schema/cache.py - Remove unused `os` import in sql/feedback_store.py - Add noqa for keyring availability check import Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
opencode.json,.opencode/,.git/opencode,OPENCODE_*env vars, andopencodecache directoriesChanges
opencode.json/opencode.jsoncaltimate-code.json/altimate-code.jsonc.opencode/.altimate-code/.git/opencode.git/altimate-codeOPENCODE_CALLERALTIMATE_CLI_CALLEROPENCODE_CONFIG_DIRALTIMATE_CLI_CONFIG_DIROPENCODE_CONFIG_CONTENTALTIMATE_CLI_CONFIG_CONTENTOPENCODE_DISABLE_PROJECT_CONFIGALTIMATE_CLI_DISABLE_PROJECT_CONFIGOPENCODE_TEST_HOMEALTIMATE_CLI_TEST_HOMEOPENCODE_TEST_MANAGED_CONFIG_DIRALTIMATE_CLI_TEST_MANAGED_CONFIG_DIROPENCODE_MODELS_PATHALTIMATE_CLI_MODELS_PATHopencode.ai/config.jsonaltimate-code.dev/config.jsonopencodealtimate-codeTest plan
build→buildermismatch, network-dependent test, timeout)🤖 Generated with Claude Code