Skip to content

[G01] Strengthen broker process fixture oracles - #166

Merged
jjangg96 merged 4 commits into
mainfrom
codex/issue-1-offline-evidence-2026-11-13
Nov 13, 2026
Merged

jjangg96 merged 4 commits into
mainfrom
codex/issue-1-offline-evidence-2026-11-13

Conversation

@jjangg96

Copy link
Copy Markdown
Contributor

Summary

  • Strengthen tagged separate-process broker fixtures to verify synthetic App JWT RS256 signatures and hold the first observed workflow identity stable.
  • Capture request URI, selected non-secret headers, authorization class, and body length/hash without retaining credentials or bodies.
  • Require attempt and account-admission lifecycle intent tied to exact attempt/journal inode identities before the fixture records token-mint effects.
  • Add negative cases for missing ledger records, wrong lifecycle kind, and mismatched attempt or journal inode; document the exact offline result.
  • Keep production provenance, conditional Scale Set cleanup, trusted runtime, and the separately authorized live canary unresolved. No live canary was run.

Validation

  • Go 1.27.3: go test -race -p=1 -count=1 -tags=g01_pair_fixture -run '^TestProcessBrokerHTTPRequiresAdmissionIntentBeforeMint$' ./livecanary — PASS for all four refusal cases.
  • Build cache, module cache, and temporary files were kept on /Volumes/worktree.
  • git diff --check — PASS.

Refs #1. Issue #1 remains open and Project #2 remains In progress.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-13T09:13:19.939131Z 0dd5400 Manual request
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@jjangg96

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Nice work!

Reviewed commit: 0dd54000ac

ℹ️ 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".

@jjangg96
jjangg96 merged commit 4b9b7be into main Nov 13, 2026
1 check passed
@jjangg96
jjangg96 deleted the codex/issue-1-offline-evidence-2026-11-13 branch November 13, 2026 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant