test(fixtures): add execution recovery contract pack v1 - #97
Open
mohamedwael201193 wants to merge 1 commit into
Open
test(fixtures): add execution recovery contract pack v1#97mohamedwael201193 wants to merge 1 commit into
mohamedwael201193 wants to merge 1 commit into
Conversation
Adds golden status envelopes and recovery rules for KeeperHub#53. Does not implement --require-verified (see KeeperHub#95).
suisuss
added a commit
that referenced
this pull request
Aug 11, 2026
…98) * docs: KEEP-1132 add contribution guide and require an accepted issue This repo had no CONTRIBUTING.md. Adds one covering the real build (make build/test/lint, the sync-version step, `go generate ./docs/` which CI fails on drift) and ISSUES.md as the contribution policy. Anything that changes command behaviour needs an issue marked `accepted` before the pull request, referenced in the PR title after the conventional-commit type (`feat: #97 description`). The type prefix still drives release-please. Issue forms and pr-issue-link.yml match the keeperhub setup, retargeted to main since this repo has no staging branch. Needs before this is useful: the `accepted`, `needs-triage` and `no-issue-required` labels. * docs: KEEP-1132 require reason, scope and plan on every issue Mirrors the keeperhub change. Issue forms restructured around three required parts: reason (command, output, expectation and its source, what it costs), scope (commands affected, siblings checked, one problem), and plan (the filer's proposal, which triage may replace in a comment before accepting). Adds `confirmed` so reproducing an issue is separable from deciding to fix it. * docs: KEEP-1132 never bounce an issue for its format Mirrors the keeperhub change: nothing applies retroactively, a filer is never asked to restate what they already said, and compound issues are split by us with the reporter credited on each part.
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.
Summary
--require-verified(see feat(execute): require chain-verified receipts in status #95).Test plan
testdata/execution_recovery_v1/in Go testsdocs/execution-recovery-v1/contract.mdRefs: #53