feat(cli): stacked-push verify walker via --from - #11
Conversation
CI now walks github.event.before..HEAD so stacked landings and direct pushes are checked, not only PR events. Fixes #10. Know-Code-Verified: dd5af611db71140513ff456be388cfe2655cf77e9f8bc168595b0e43dd46c547
PR SummaryMedium Risk Overview The CLI gains GitHub integration now triggers verify on Docs and tests document the PR vs push split, merge-button behavior, and receipt-mode push caveats; smoke CI adds a Reviewed by Cursor Bugbot for commit b50c9b2. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 860d5b6. Configure here.
An outdated GitHub merge-commit landing included unrelated mainline files in the tree-pair. Compare the feature tip instead. Know-Code-Verified: 882aa43dc477157b21d0c71611be261ab8f8591aefa8fd4651f864ce6bdeb2db

Summary
know-code verify --from <oid>: walkfrom..HEAD, split byKnow-Code-Verifiedhash, and check each run as a historical tree-pair (parent-of-first → tip). Trailerless merges attach; trailerless linear commits fail closed. One-non-merge runs also accept the empty-tree (index) hash; later pushes in the same range match first-parent ancestors of the run start.know-code init --workflow, and the composite action to run on push to the base branch and passgithub.event.before. PR verify is unchanged (head.sha, no--from). All-zerosbeforeskips the walk.Fixes #10.
Test plan
npm test— especiallyverify-walk.test.tsand the GitHub-shaped cases inverify.test.tshead.shaand runs bareknow-code verifymainrunsknow-code verify --from "$BEFORE"(skip ifbeforeis all zeros)