docs: correct the unified-log guidance in CLAUDE.md - #110
Merged
Merged
Conversation
The "agent's Bash sandbox cannot read the unified log" rule was a misdiagnosis that cost real debugging time this session. zsh has a `log` builtin that shadows /usr/bin/log, so a bare `log show` is swallowed and every predicate looks empty. Calling /usr/bin/log by absolute path works fine, so the instruction to hand traces to the user is unnecessary. Record the two gotchas that are real: dext lines surface as process "kernel" rather than the dext's own process name (filtering on the latter returns nothing while the driver is logging), and sub-Default levels need --info --debug. Also record the inverse lesson: silence from a dext is a finding. Grep for the guaranteed "LogConfig initialized" line per instance, and read the crash reports early -- a crashed dext restarts degraded while ioreg still shows its services matched and active, which disguises the failure as a bug in whatever layer you were working on. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> (cherry picked from commit cecd77a7bd8f53112b606d5419649d7608ee8b24)
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
2 participants
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.
CLAUDE.md: correct the unified-log guidance
CLAUDE.md says the agent's Bash sandbox "cannot read the unified log" because
log showreturns zero lines. That was a misdiagnosis: zsh has alogbuiltin that shadows/usr/bin/log, so a barelog show …is swallowed and every predicate looks empty. Calling/usr/bin/logby absolute path works, so the instruction to hand traces to the user isn't needed.The PR keeps the two gotchas that are real:
process == "kernel", not under the dext's own process name.--info --debug.It also adds the inverse lesson: a silent dext is evidence. Grep for the guaranteed
LogConfig initializedline for each instance, and check/Library/Logs/DiagnosticReports/net.mrmidi.ASFW.ASFWDriver-*.ipsearly. A crashed dext restarts degraded whileioregstill shows its services active, which makes the failure look like a bug in whatever layer you're working on.Docs only.
🤖 Generated with Claude Code