Repository navigation
deps: bump @opencode/codemode to 2.2.5 - #4
Merged
Merged
Conversation
Pin @opencode/codemode at 2.2.5 and effect at 4.0.6. Document the current baseline and keep for...in rejected because deleted keys are still visited. Co-authored-by: hyein.sophia.cho <hyein.sophia.cho@gmail.com>
 [Medium impact] The dependency update appears safe to merge; no actionable issues were found.
|
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
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.
Versions
@opencode/codemode: 2.2.1 → 2.2.5 (exact pin, no range)effect: 4.0.1 → 4.0.6 (exact pin).npm view @opencode/codemode@2.2.5 dependenciesreportseffect4.0.6andacorn8.15.0. There is no peer dependency to reconcile.acornstays 8.15.0versionstays 0.1.5npm
dist-tags.latestis2.2.5(published 2026-10-30T23:12:36.996Z). No open pull request existed for this version or for an older codemode bump.Behavior
No runtime change. The tool bridge, limits, preflight decisions, and result contract stay the same, and
for...inis still rejected before execution.CodeMode 2.2.5 visits inherited enumerable string keys in
for...in(2.2.1 returned only own keys) and still visits keys that are deleted during iteration.route.jsonly updates the comment that records that reason.test/extension.test.jsnow expects the pinned version2.2.5.Docs
Current-baseline mentions now name 2.2.5:
README.md(backed-by version, admitted syntax, and thefor...inreason)CHANGELOG.md(new Unreleased entry; the existing 2.2.1 Unreleased note is unchanged)TESTING.md(release pin and this run’s results, including Effect 4.0.6)Historical changelog entries and older verification sections are unchanged.
Tests
Node v22.23.4.
npm testafter the pin, lockfile, version assertion, comment, and doc edits:isolated real Pi SDK sessionwithoutPI_OPENCODE_SDK)npm run test:pi-versionson the 2.2.5 pin and lockfile (before the comment and doc edits, which do not change execution):Links