commit verification workflow#10581

Merged

macneale4 merged 25 commits intomainfrom

macneale4-claude/verify-workflow

Mar 5, 2026

Commits on Mar 5, 2026

actions/commit: export RunCommitVerification and add ErrCommitVerificationFailed

macneale4
and
claude
committed
cherry_pick: halt with dirty state on verification failure

macneale4
and
claude
committed
cherry_pick: surface ErrCommitVerificationFailed clearly on --continue

macneale4
and
claude
committed
cherry_pick: add verification_failures column and fix preMergeWorking bug

macneale4
and
claude
committed
rebase: halt on verification failure with dirty state preserved

macneale4
and
claude
committed
merge: halt on verification failure with dirty state preserved

macneale4
and
claude
committed
enginetest: add commit verification tests for cherry-pick, merge, rebase

macneale4
and
claude
committed
merge+rebase CLI: handle verification failure halt correctly

macneale4
and
claude
committed
bats: add commit verification tests for cherry-pick, merge, rebase

macneale4
and
claude
committed
Move all commit verification bats tests to commit_verification.bats

macneale4
and
claude
committed
Define CommitVerificationFailedPrefix constant to replace hardcoded strings

macneale4
and
claude
committed
Remove pre-emptive RunCommitVerification in cherry_pick.go

macneale4
and
claude
committed
Fix recomentation after failing cherry-pick
macneale4
committed
Make cherry-pick more resilient to mismatched versions
macneale4
committed
BAD CLAUDE
macneale4
committed
Surface specific verification failure details in cherry-pick CLI output

macneale4
and
claude
committed
Update engine tests: cherry-pick verification failure now returns error not row

macneale4
and
claude
committed
Update bats tests 11 and 12: specific cherry-pick verification assertions

macneale4
and
claude
committed
Fix missed engine test: cherry-pick tests-pass scenario also returns error not row

macneale4
and
claude
committed
Remove verification_failures column from dolt_cherry_pick() stored procedure result

macneale4
and
claude
committed
cleanup of bats tests
macneale4
committed
Remove pre-emptive RunCommitVerification from merge, make helpers private

macneale4
and
claude
committed
forgot || false on a couple tests
macneale4
committed
manual review/cleanup
macneale4
committed
PR Feedback for additional comment
macneale4
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

commit verification workflow#10581

commit verification workflow#10581
macneale4 merged 25 commits intomainfrom
macneale4-claude/verify-workflow

Commits on Mar 5, 2026

actions/commit: export RunCommitVerification and add ErrCommitVerificationFailed

cherry_pick: halt with dirty state on verification failure

cherry_pick: surface ErrCommitVerificationFailed clearly on --continue

cherry_pick: add verification_failures column and fix preMergeWorking bug

rebase: halt on verification failure with dirty state preserved

merge: halt on verification failure with dirty state preserved

enginetest: add commit verification tests for cherry-pick, merge, rebase

merge+rebase CLI: handle verification failure halt correctly

bats: add commit verification tests for cherry-pick, merge, rebase

Move all commit verification bats tests to commit_verification.bats

Define CommitVerificationFailedPrefix constant to replace hardcoded strings

Remove pre-emptive RunCommitVerification in cherry_pick.go

Fix recomentation after failing cherry-pick

Make cherry-pick more resilient to mismatched versions

BAD CLAUDE

Surface specific verification failure details in cherry-pick CLI output

Update engine tests: cherry-pick verification failure now returns error not row

Update bats tests 11 and 12: specific cherry-pick verification assertions

Fix missed engine test: cherry-pick tests-pass scenario also returns error not row

Remove verification_failures column from dolt_cherry_pick() stored procedure result

cleanup of bats tests

Remove pre-emptive RunCommitVerification from merge, make helpers private

forgot || false on a couple tests

manual review/cleanup

PR Feedback for additional comment