chore: bump version to 4.0.3 - #677
Closed
aselim31 wants to merge 6 commits into
Closed
Conversation
Contributor
|
This PR is missing a linked issue reference. Please add one of the following to the PR description:
If this is a trivial change that doesn't need an issue, ask a maintainer to add the |
The 2>&1 redirect merged stderr warnings into the JSON capture. ANSI-colored warnings from Dotbot.Settings caused ConvertFrom-Json to fail with 'Invalid property identifier character: \'. Use 2>$null to discard stderr so only the JSON stdout is parsed.
All five status -Json captures in Test-Components.ps1 used 2>&1 which merges stderr warnings into the JSON stdout. Any ANSI-colored warning (e.g. from Dotbot.Settings) prefixed the JSON, causing ConvertFrom-Json to fail. Changed all five captures to 2>$null so only clean JSON stdout is captured and parsed.
Line 5742: reverted 2>null back to 2>&1 — the nested-git test asserts that $phase4NestedGitOutput contains 'DOTBOT_HOME is not set', which comes from stderr. Discarding stderr broke that assertion. Line 156: changed double-quoted regex with backtick-dollar to a single-quoted pattern with \$. In a double-quoted string, backtick-dollar produces $, but in .NET regex $ is the end-of-line anchor, not a literal dollar. Single-quoted '\$' correctly produces the literal-dollar escape.
Collaborator
Author
|
Cancelling — version bump will go through the scheduled bump-release workflow instead. |
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.
Closes #678
Summary
version.jsonfrom4.0.2→4.0.3docs/release-notes/v4.0.3.mdRelease notes
7 bugs fixed · 3 new features · 9 issues closed
Fixes: process-state BotRoot vs worktree, orphaned process tree cleanup, MCP stdin busy-loop, verify-chain wrong cwd, status handler double response, Windows PATH (git/claude MISSING), integration branch pushed N times per run
New:
dotbot logscommand, v3.5 YAML auto-migration, quality gate hook beforedoneTest plan
version.jsonreads4.0.3after merge🤖 Generated with Claude Code