Skip to content

Add a narrow public-source release check - #61

Merged
VladimirBrejcha merged 15 commits into
mainfrom
codex/narrow-public-source-release-audit
Aug 22, 2026
Merged

Add a narrow public-source release check#61
VladimirBrejcha merged 15 commits into
mainfrom
codex/narrow-public-source-release-audit

Conversation

@VladimirBrejcha

Copy link
Copy Markdown
Collaborator

What

  • add a thin public-source-release-audit skill with explicit automated, semantic, and GitHub-native boundaries
  • replace the inline leak scan with a 157-line checker that reads exact Git-index blobs and also scans non-ignored worktree source
  • add 10 focused regression tests and include the skill in the 39-skill managed baseline

Real problem solved

The previous verifier enumerated tracked paths but read their worktree contents. A staged credential could therefore be hidden from local verification by an unstaged clean replacement. The new checker reads the candidate blobs directly, neutralizes Git environment redirects, and separately scans local source for early feedback.

Findings contain only a path and category. Matched values and sensitive filenames are redacted.

Deliberate boundaries

This does not parse workflows or shells, model execution or taint, crawl history, or reproduce GitHub policy. Ignored files and submodule contents are out of scope. Customer information, internal context, and other semantic disclosures require actual review. Live repository controls are confirmed through read-only GitHub commands and reported separately.

Verification

  • skill validator passed
  • 10/10 focused tests passed, including staged/worktree masking in both directions, Git-index redirection, redaction, symlinks, supported formats, safe path precision, and unresolved-index failure
  • complete scripts/verify.sh passed
  • mechanism-audit verdict for the bounded automated promise: holds; no P0 or P1 fixes

@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 18cdea1f9f

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/SKILL.md
Handle root homes, bounded blob caching, non-fetching partial clones, explicit LFS review, and classic branch protection without expanding into generalized parsing.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3dcd7b7c9f

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Reject symlinked parents, scan raw Git path bytes safely, and cover PGP headers plus serialized Windows home paths without expanding the audit architecture.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0085174f01

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/SKILL.md
Comment thread public-source-release-audit/scripts/check_current_source.rb
Normalize subdirectory inputs, suppress repository fsmonitor hooks, and encode the classic-protection branch parameter without adding broader policy analysis.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 044047eae8

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Support JSON-escaped POSIX paths, exclude URL query routes, and stream unique index blobs through one Git process without expanding detection categories.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 85ea927596

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb
Comment thread public-source-release-audit/scripts/check_current_source.rb
Handle quoted bearer values, command-option homes, PuTTY headers, and named AWS secrets while replacing broad format promises with an explicit lexical contract.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ef0254ab7f

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Cover quoted canonical AWS names and Windows casing, and disable optional Git locks so split-index metadata stays untouched during verification.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5d766f9fab

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Require the exact Authorization key and a non-empty Windows username so generic map keys and profiles-directory prose remain public-safe.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b2c5e9f9f2

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

Comment thread public-source-release-audit/scripts/check_current_source.rb
Require separate review for UTF-16 index and worktree content instead of adding transcoding or allowing encoded supported credentials to pass.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7e819cca30

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

Comment thread public-source-release-audit/scripts/check_current_source.rb
Comment thread public-source-release-audit/scripts/check_current_source.rb
Apply the existing LFS fail-closed rule to worktree content and cover the exact canonical macOS root-home families without broadening the path contract.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8178d9b1ca

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Treat a closing markup delimiter as a bounded literal-path context and cover the supported plist/XML element form.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b423dab9fd

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb
Support UTF-8 profile components and a specific local file wrapper while explicitly excluding nested escaped-document decoding from the lexical audit.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6127a220f4

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Use an explicit source delimiter for terminal POSIX profile names so international names do not depend on ASCII word-boundary behavior.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 41c646c1da

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

Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb Outdated
Comment thread public-source-release-audit/scripts/check_current_source.rb
Use lstat for listed paths, clear inherited Git trace destinations, and align the terminal Windows markup boundary without expanding detection families.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9386cb99d3

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

Comment thread public-source-release-audit/scripts/check_current_source.rb
Track mode-160000 paths from the index and skip their initialized worktree directories while retaining path-name scanning and all other fail-closed checks.
@VladimirBrejcha

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: 5a6a4ad23a

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

@VladimirBrejcha
VladimirBrejcha merged commit b89100a into main Aug 22, 2026
1 check passed
@VladimirBrejcha
VladimirBrejcha deleted the codex/narrow-public-source-release-audit branch August 22, 2026 11:03
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