Repository navigation
feat(opencode): expose running version, provider and model in the env block - #51911
Open
PascalBourdier wants to merge 2 commits into
Open
PascalBourdier wants to merge 2 commits into
PascalBourdier wants to merge 2 commits into
Conversation
github-actions
Bot
added
needs:compliance
Contributor
|
Hey! Your PR title Please update it to start with one of:
Where See CONTRIBUTING.md for details. |
Contributor
|
The following comment was made by an LLM, it may be inaccurate: |
PascalBourdier
force-pushed
the
feat/env-block-version-model
branch
from
September 28, 2026 17:08
8c7c515 to
086aca7
Compare
1 task done
… block Agents operating under 'state your engine and version' conventions (AGENTS.md PR traceability rules) cannot reliably know their own harness version: it is absent from the model context, which invites hallucinated values (observed: an opencode 1.18.30 agent declared 'opencode 0.11.x', anchored on a stale doc example). Add to the structured <env> block: - OpenCode version: the build-time InstallationVersion (falls back to 'local' in dev builds) - Provider: the provider ID (the canonical, actionable form used in opencode.json provider/model references) - Model: the models.dev display name, with the exact API id in parentheses Also fix the adjacent prose line: it called the API id 'the model named', which is an identifier, not a name (e.g. openrouter z-ai/glm-5.3) — it now uses the display name, keeping the canonical providerID/api.id reference as the 'exact model ID'. Fixes anomalyco#51909
PascalBourdier
force-pushed
the
feat/env-block-version-model
branch
from
September 28, 2026 17:18
086aca7 to
c0df7a6
Compare
PascalBourdier
changed the title
PascalBourdier
marked this pull request as ready for review
September 28, 2026 17:23
PascalBourdier
changed the title
Assert that environment(model) exposes the OpenCode version, the provider and the model identity (display name + exact API id) in the <env> block, and that the adjacent prose line uses the display name. Runs through the testEffect instance scaffolding (withTmpdirInstance), which provides the InstanceState context environment() reads.
github-actions
Bot
removed
the
needs:compliance
Contributor
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
This was referenced Sep 28, 2026
|
The Head Fork leaf kvnloo#189 recorded fail→pass on That locks those three fields in the env block. It does not cover other system-prompt sections. |
3 of 6 tasks
1 task done
Open
1 task done
This branch has not been deployed
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
2 participants
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.
Issue for this PR
Closes #51909
Type of change
What does this PR do?
The model context never includes the running OpenCode version, so agents asked to "state your engine and version" (a common AGENTS.md traceability rule) tend to guess: an opencode 1.18.30 agent was observed writing
opencode 0.11.xin a PR description, anchored on a stale doc example. The model id only appears as prose in the main system prompt, which subagents and custom agents may not receive.This adds three lines to the structured
<env>block of the session system prompt:OpenCode version: ${InstallationVersion}— the build-time constantProvider: ${model.providerID}— the canonical form used in opencode.json referencesModel: ${model.name} (${model.api.id})— models.dev display name + exact API idThe adjacent prose line is also fixed: it called the API id "the model named" (an id, not a name — e.g.
z-ai/glm-5.3); it now uses the display name and keepsproviderID/api.idas the "exact model ID".All values are already in scope of
environment(model)— no new service dependency.Context: #9065 asked for the same (model identity + version); only the model part landed (88bcd04, prose line), then it was stale-closed with an invite to re-open. #51909 is that re-open.
How did you verify your code works?
environment(model)inpackages/opencode/test/session/system.test.ts(asserts the three env lines and the fixed prose line); it runs through thetestEffectinstance scaffolding, which provides theInstanceStatecontext the function reads.bun run typecheck(turbo, all 30 packages) — green.bun test test/session/system.test.ts— 7 pass, 0 fail (6 existing + the new test; no existing assertion touches the env block or the prose line).bun run lint(oxlint) — no new findings from this change (the reported error and warnings are pre-existing on untouched lines).Screenshots / recordings
N/A — not a UI change.
Checklist