Add closure-run identity correlation - #26
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6c969c6786
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| check: ## Run the full repo validation battery | ||
| @$(MAKE) --no-print-directory closure-identity CLOSURE_PHASE=check CLOSURE_PARENT_COMMAND="make check" |
There was a problem hiding this comment.
Record the required HIGR documentation
AGENTS.md requires every change to complete the Document step by updating HYPOTHESES.md, LEARNINGS.md, and HANDOFF.md; this change adds a new closure identity surface and validation target without updating those ledgers. Without that hypothesis/validation/handoff record, the new workflow behavior cannot be traced through the repo’s required HIGR process.
Useful? React with 👍 / 👎.
Summary
tools/emit_closure_identity.pyto emit comparable local and GitHub closure-run identity JSON fields:closure_run_id,closure_phase,closure_trigger,evidence_reuse_key,parent_command,github_run_id, andgithub_run_attempt.make closure-identity/make checkand adds the focusedmake test-closure-identityregression totools/test_all.sh.Issue #164 / Repo-Star Replay Context
CustomerNewsletter:Makefile, .github/workflows/ci.yml, .github/workflows/pages.yml.Refs briancl2/build-meta-analysis#164
Refs briancl2/build-meta-analysis#589
Validation
make test-closure-identitypython3 /Users/briancl/repos/repo-auditor/scripts/as_signature_scan.py AS-34 /Users/briancl/repos/CustomerNewsletter->fired=false,closure_run_identity_gap_count=0python3 -m py_compile tools/emit_closure_identity.pygit diff --checkmake check-> 33 required suites passed, 20 private-fixture suites skippedmake review-> No findingsBounded Non-Claims