Sync the Claude Code web SessionStart hook from TapHouse#28
Merged
Conversation
Fresh web-session containers clone bare (no submodules, no pre-commit hook), so agent commits could bypass the local format layer and fail the clang-format gate only in CI. The canonical TapHouse hook closes the gap at session start: recursive submodule init, pre-commit install against the pinned config, and a hook-environment warm for the container snapshot. Synced via taphouse/scripts/sync.sh; the hook script is drift-guarded once the TapHouse tag advances past v4. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BuUawafae7RR2ySWabuJcs
v5 canonicalizes the Claude Code web SessionStart hook (tap/TapHouse#4 merged); this branch already carries the synced hook, so the v5 drift guard — which now covers it — passes by construction. Both the reusable workflow ref and the comparison ref move together. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BuUawafae7RR2ySWabuJcs
The style workflow references taphouse@v5, which did not exist when the ref bump was pushed; this empty commit re-triggers the runs now that the tag is live. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BuUawafae7RR2ySWabuJcs
…NarrowbandGuard) The duplicate run on the identical tree passed all 181 cases; this PR touches no C++. Re-kick to clear the flaky pair. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BuUawafae7RR2ySWabuJcs
Owner
Author
|
The macOS flake that hit this PR (twice) is now diagnosed and documented in #29: the Accelerate float32 FFT backend produces epsilon-different spectra across the heterogeneous Generated by Claude Code |
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.
Consumer side of tap/TapHouse#4: the canonical web SessionStart hook (recursive submodule init + pinned pre-commit install + hook warm at session start), synced via
taphouse/scripts/sync.sh..gitignoreadjusted to keep.claude/*ignored while allowing the checked-insettings.jsonandhooks/. Core configs untouched (main == v4). Validated by direct execution in this clone. Drift-guarded once the TapHouse tag advances past v4.🤖 Generated with Claude Code
https://claude.ai/code/session_01BuUawafae7RR2ySWabuJcs
Generated by Claude Code