-
Notifications
You must be signed in to change notification settings - Fork 0
fix(strix): bind evidence to exact workflow artifacts #1009
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
seonghobae
wants to merge
36
commits into
main
Choose a base branch
from
codex/strix-evidence-minimal
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
e8b5bd4
fix(strix): bind evidence to exact workflow artifacts
seonghobae e59afd6
docs: record central Strix bootstrap evidence boundary
seonghobae afa439f
docs: record default-branch dependency alert follow-up
seonghobae 30b78b5
fix: harden CI log token redaction
seonghobae aebbae9
fix: harden redactor against repeated scans
seonghobae 1dc7497
test: cover JSON redaction branch
seonghobae 1dbd8eb
fix(actions): bound agent mention dispatch envelope
seonghobae 89fb467
fix(actions): separate agent mention route queues
seonghobae 2978973
fix(review): close evidence and launcher review gaps
seonghobae 90d3519
test(redaction): exercise operational helper directly
seonghobae fbb4eb5
test(review): exercise launcher permission fallback
seonghobae 5bfcaf6
docs(adr): record default-branch dispatch bootstrap failure
seonghobae 410f070
docs(adr): record target mutation permission boundary
seonghobae ee88078
fix(router): reject boolean webhook identifiers
seonghobae daf6a44
docs(adr): record Strix false-positive evidence
seonghobae 24f282e
docs: record current router bootstrap boundaries
seonghobae 72cd030
docs: record current Strix provider failure
seonghobae 8c56db0
docs: record current review bootstrap failures
seonghobae 5b3e7fd
fix: recognize observed Strix tool contract failures
seonghobae 48fedcd
fix: bind merged Strix evidence and dispatch limits
seonghobae 0a5b450
docs: bind central validation evidence to exact head
seonghobae 913302d
chore: diagnose Strix evidence reconciliation with current main
seonghobae 763a938
chore: remove completed Strix merge diagnostic
seonghobae b2a7e27
fix(review): preserve independent reviewer workflow byte-for-byte
seonghobae 88b71ee
chore: diagnose stale PR 1009 quick-gate assertions
seonghobae a4f528a
chore: add deterministic PR 1009 quick-gate repair
seonghobae e827128
fix(ci): repair PR 1009 quick-gate ownership assertions
seonghobae 9f35459
test(strix): decouple evidence checks from reviewer workflow
127b08b
fix: harden strix evidence and review dispatch
seonghobae 4055a21
fix: close strix gate edge cases
seonghobae aabecd7
fix(strix): require structured exact-head evidence
seonghobae 575ff76
fix: preserve reviewer boundary and dispatch recovery
seonghobae 805f4d3
Merge branch 'main' into codex/strix-evidence-minimal
opencode-agent[bot] fc66835
Merge branch 'main' into codex/strix-evidence-minimal
opencode-agent[bot] 99fee8b
Merge branch 'main' into codex/strix-evidence-minimal
opencode-agent[bot] dd4d896
fix(security): preserve adjacent redaction boundaries
seonghobae File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📝 Info: OpenCode wrapper now accepts review-control flags from client_payload with safe defaults
The OpenCode dispatch wrapper changed the review-control env values from hardcoded literals to
${{ github.event.client_payload.<flag> || '<default>' }}(agent-mention-opencode-dispatch.yml). Because the downstreamValidate exact invocation payloadstep still hard-assertsTRIGGER_REVIEWS==true,REVIEW_DISPATCH_LIMIT==1,ENABLE_AUTO_MERGE==false,UPDATE_BRANCHES==false,MERGE_MODE==disabledbefore the HMAC/invocation-key check, a malicious dispatch that supplies e.g.enable_auto_merge=truestill fails closed. The router no longer emits these keys (keeping the payload at 9 properties), so the defaults apply in normal operation. Behavior is preserved; noting the widened input surface for reviewer awareness.Was this helpful? React with 👍 or 👎 to provide feedback.