Skip to content

chore: upgrade opencode to 2.0.29 and openchamber to 2.3.2 - #24

Merged
icco merged 3 commits into
mainfrom
chore/upgrade-deps
Oct 14, 2026
Merged

icco merged 3 commits into
mainfrom
chore/upgrade-deps

Conversation

@icco

@icco icco commented Oct 14, 2026

Copy link
Copy Markdown
Owner

Why

Upgrades opencode 2.0.27 to 2.0.29, openchamber 2.2.0 to 2.3.2, and effect 4.0.2 to 4.0.4. Every other pinned tool, image, plugin, MCP package, and action is already at its latest release.

Both releases are newer than pnpm's minimum release age, so they are excluded by exact version after a changelog review. The config moves to the stable top-level policies key.

Risks

opencode 2.0.29 adds no database migrations, so rolling back to 2.0.27 is safe. openchamber 2.3.2 ships opencode 2.0.28 client packages against the 2.0.29 server.

icco and others added 2 commits October 14, 2026 14:36
Both releases are under pnpm's minimum release age, so the excludes move from the 2.0.27 opencode packages to 2.0.29 and add @openchamber/web and @openchamber/sdk 2.3.2 after reviewing their changelogs. Every other pinned tool, image, plugin, MCP package, and action is already at its latest release.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
OpenCode 2.0.29 stabilizes top-level policies; experimental.policies still merges in but is the legacy spelling. The README now states that /etc/opencode/opencode.json overrides global and project config, which 2.0.29 documents.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@icco
icco marked this pull request as ready for review October 14, 2026 14:37
@icco
icco requested a balanced review from Copilot October 14, 2026 14:37
@icco icco self-assigned this Oct 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The provider security boundary lacks an integration assertion proving the migrated policy is enforced.

1 open finding
What changed in this PR

Upgrades OpenCode, OpenChamber, and Effect while adopting stable provider policies and documenting system-config precedence.

Changes:

  • Updates pinned runtime and development dependencies.
  • Migrates provider restrictions to top-level policies.
  • Refreshes release-age exclusions and lock data.
File Description
Dockerfile Updates runtime tool versions.
README.md Clarifies configuration precedence.
opencode.jsonc Migrates provider policies.
package.json Updates development dependencies.
pnpm-lock.yaml Regenerates dependency resolutions.
pnpm-workspace.yaml Updates release-age exclusions.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread opencode.jsonc
}
]
}
"policies": [

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

🤖 babysit-pr: Fixed in 78431a3: the integration test now checks that every model from /api/model belongs to google or github-copilot. /api/provider only lists connected providers, so it is empty in CI with or without the policy. With policies removed from the image config, /api/model returns OpenCode's 10 free models and the new assertion fails; with them it passes.

/api/provider lists only connected providers, so it is empty in CI either way. /api/model lists OpenCode's free models when provider.use is not enforced, so checking that every model belongs to google or github-copilot fails if the policies stop loading.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@icco
icco requested a balanced review from Copilot October 14, 2026 14:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

Versions, configuration migration, lockfile updates, and integration coverage are consistent.

1 open finding
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

🧠 Review effort: Balanced

@icco
icco merged commit bceb065 into main Oct 14, 2026
10 checks passed
@icco
icco deleted the chore/upgrade-deps branch October 14, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants