Skip to content

Pin and harden the qualified CLI action - #11

Merged
morgaesis merged 4 commits into
mainfrom
codex/action-v0.6.1-compat
Jul 14, 2026
Merged

Pin and harden the qualified CLI action#11
morgaesis merged 4 commits into
mainfrom
codex/action-v0.6.1-compat

Conversation

@morgaesis

Copy link
Copy Markdown
Contributor

Pins v0.6.1, invokes bounded review, requires explicit model or trusted config, supports Anthropic and private endpoint authentication, and fails closed on malformed envelopes and untrusted artifacts or signers. The local contract tests, published release assets, and hostile audit pass; the pinned CLI intentionally contains no hosted default model.

@postil-dev postil-dev Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 finding worth a look. It does not block this merge.

Review details

Comment thread .github/workflows/ci.yml
model:
name: fixture/not-contacted
YAML
: >"$empty_diff"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warn Remove POSTIL IGNORE REPOSITORY MODEL CONFIG
warn · confidence 0.70 · kind: risk

The diff removes the POSTIL_IGNORE_REPOSITORY_MODEL_CONFIG environment variable setting. Verify that this variable was not previously used to control behavior in the review command and that its removal does not introduce any unintended side effects. Check the code that processes the review command to confirm that the variable is no longer needed.

@morgaesis
morgaesis merged commit 413a451 into main Jul 14, 2026
12 checks passed
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