Repository navigation
chore: release v0.5.1 - #142
Merged
Merged
Conversation
Bump the patch version after merging the runtime session identity fix (#134) and refresh dependencies with `bun update`. Also relax the `/goal` wiring assertion so it tolerates the optional runtime argument that #134 added to `sendAgentCommand`. The assertion matches the hook source text, so the new third argument made it fail even though the wiring is correct. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UUVQrLSerMv3An8J2WT2ex
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
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.
Prepares the
v0.5.1patch release after merging #134.Changes
versionto0.5.1.bun update. Direct ranges moved forjs-yaml(^5.4.3→^5.5.0),react/react-dom(^19.3.0→^19.3.1),@tauri-apps/cli(^2.12.2→^2.12.3) andpostcss(^8.5.30→^8.5.31), plus transitive updates inbun.lock. The@oh-my-pi/*packages stay on~18.10.4./goalwiring assertion inlib/goal-mode.wiring.test.mjs.Why the test change
#134 added an optional runtime argument to
sendAgentCommand:That test asserts against the hook's source text, and its regex required the call to close right after
args, so the extra argument made it fail even though the wiring is correct. The pattern now tolerates the optional third argument. This test is not modified by #134 and passed before it, so the failure was a regression introduced by that merge;bun testis not part ofci.yml, which is why it went unnoticed.Verification
Run on this branch, after
bun update:bun install --frozen-lockfile— clean, no lockfile drift.bun run typecheck— passes.bun run build— passes (warnings only).bun test— 616 pass, 1 skip, 0 fail across 617 tests in 98 files.bun run lintstill reports pre-existing errors; it already fails onmainand is not part of CI.The GitNexus
impact/detect_changessteps required byCLAUDE.mdcould not be run: the GitNexus MCP server is not connected in this session.Generated by Claude Code