Repository navigation
Correct thirteen 0.1.12 changelog lines that overstated or missed a condition - #58
Merged
Merged
Conversation
…ondition A review of the published notes against the 0.1.12 code found these lines claimed more than the code does: - target_turn_start does not carry latency or interrupted; only the end does - a tool failure's error is in the output; the 500-character error field is only for a call that stops the loop - doctor probes run by absolute path, not all from fixed system paths, and the shell lookup can take up to 3 seconds - transcript URL masking covers http(s) URLs and database or broker connection strings, not every URL - status lines are masked and compacted too, not only warnings, errors and progress - the failed-upload warning follows the evidence path line - the update UI-control event is only recorded when /update hands off - a Homebrew upgrade refreshes taps first, not "always" Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
The 0.1.12 release notes went out with thirteen lines that claim more than the 0.1.12 code does. This corrects them in
CHANGELOG.md; nothing else changes.What was wrong
target_turn_startdoesn't carry latency orinterrupted; onlytarget_turn_enddoes.rook doctorruns its probes by absolute path, not all from fixed system paths: the shell's version check uses the pathpsreports first. The parent-process walk that finds the shell can take up to 3 seconds, not a flat 1.5.updateUI-control event is recorded only when/updatehands off torook update, not for/update auto.brew.envcan still turn that off).The GitHub Release's summary paragraph has the same kind of overstatement ("a full record", "masks credentials … before anything is written"). It needs the matching edit to the release text, which isn't part of this PR.
🤖 Generated with Claude Code