Skip to content

fix(workflow): keep the run marker in the closing trailer block - #10

Merged
decebal merged 1 commit into
mainfrom
fix/snapshot-budget
Nov 10, 2026
Merged

decebal merged 1 commit into
mainfrom
fix/snapshot-budget

Conversation

@decebal

@decebal decebal commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Managed commits now put the Allstack-Run marker in the message's closing trailer block. It was a separate -m paragraph, so git stopped seeing a message's own trailers (Release-note:, Signed-off-by:) as trailers. After prose or a bare subject the marker starts its own block.
  • The workspace fixture's git calls get a 60 s deadline. The large-checkout case stages 10,010 files in one call and ran past the old 10 s deadline at load average 23, failing with empty stderr.
  • The Quality macOS job moves to macos-15. GitHub refuses macos-14 ("The configuration 'macos-14-arm64-us-default' is not supported"), which cancelled every Quality run since 2026-11-07.

The snapshot-budget and shared-tracker work this branch started with landed through #8 (ba8eb65), so it is gone from this diff.

Test plan

  • cargo test --all-targets on d2c54e3: all suites pass, 2 bootstrap tests ignored (need a real Chronis)
  • cargo fmt --check, cargo clippy --all-targets -- -D warnings
  • Quality on 08894e8: ubuntu-24.04, macos-15 and source-and-dependencies pass
  • Longhand's own check-msg (the commit-msg hook tool): rejects the old marker shape with "no Release-note: trailer", accepts the new one. The old shape's rejection is what stopped run 1596a6d9 at commit on 2026-11-08 (three receipts)
  • Snapshot bound on real Longhand: run 1596a6d9 on the #1846 branch recorded started and two checkpointed events, and claimed its task in the primary tracker from a linked worktree

Limits

  • Not run: a managed commit on Longhand with a binary from this branch. Run 1596a6d9 was abandoned and #1846 was committed by hand, so the new shape is proven against the hook tool only.
  • Which binary started that run is not recorded; it had to carry the budget code to pass the default bound.
  • That run also left handoff and recover refusing on an unresolved commit intent. Not addressed here.

🤖 Generated with Claude Code

@decebal
decebal force-pushed the fix/snapshot-budget branch from 6996b87 to d2c54e3 Compare November 9, 2026 16:14
@decebal decebal changed the title fix(enrollment): support large linked worktrees on current main fix(workflow): keep the run marker in the closing trailer block Nov 9, 2026
Managed commits passed the Allstack-Run marker as a separate -m, which
makes it its own paragraph. Git reads trailers only from the last
paragraph, so a message ending in Release-note: or Signed-off-by: lost
those trailers to any repository rule that requires them. The marker now
joins an existing closing trailer block, and starts its own block after
prose or a bare subject.

Also gives the workspace fixture's git calls a 60 s deadline. The
large-checkout case stages 10,010 files in one git add and ran past the
10 s deadline at load average 23, failing with empty stderr.

Moves the Quality macOS job to macos-15. GitHub now refuses macos-14 with
"The configuration 'macos-14-arm64-us-default' is not supported", which
has cancelled every Quality run since 2026-11-07.
@decebal
decebal force-pushed the fix/snapshot-budget branch from d2c54e3 to 08894e8 Compare November 9, 2026 17:27
@decebal
decebal marked this pull request as ready for review November 9, 2026 23:50
@decebal
decebal merged commit 24db226 into main Nov 10, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant