Skip to content

docs: align repository security and contributor guidance - #4962

Merged
seratch merged 1 commit into
mainfrom
codex/security-guidance
Sep 11, 2026
Merged

seratch merged 1 commit into
mainfrom
codex/security-guidance

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

The repository's security policy only linked the general disclosure policy, and it had no local contributor guide. Expand SECURITY.md, add CONTRIBUTING.md, and add an agent security checklist in AGENTS.md to cover repository scope, private sanitized reporting, secrets, dependencies, CI, and publishing.

The guidance distinguishes policy requirements from verified operational controls and preserves existing development and release procedures. This change is limited to the three documentation files.

Test plan

  • Verify private-reporting, redaction, dependency-cooldown, untrusted-CI, release-approval, publishing-evidence, and scan-status scenarios against the shared security policy and repository configuration.
  • Check all 37 local Markdown links and anchors, resolve the root security policy for the SDK, and run git diff --check.
  • Complete two consecutive clean adversarial review rounds with two fresh, independent reviewers per round, including security review of the reporting and trust-boundary guidance.
  • SDK runtime tests, type checks, and MkDocs builds are not applicable to this repository-guidance-only change.

Issue number

None.

Checks

  • Confirmed no new runtime tests are needed for this guidance-only change.
  • I've run .agents/skills/code-change-verification/scripts/run.sh — not applicable to repository-meta-only changes under AGENTS.md.
  • I've confirmed all applicable local verification steps pass.
  • Completed independent adversarial review before submitting this PR.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

Codex Review Summary

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

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-11T22:23:25.696858Z 3828825 PR opened
🔒 Security Review ✅ Completed 2026-09-11T22:24:26.429899Z 3828825 PR opened
ℹ️ 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.

@seratch seratch added dependencies documentation Improvements or additions to documentation and removed dependencies labels Sep 11, 2026
@seratch
seratch merged commit faf10fd into main Sep 11, 2026
19 checks passed
@seratch
seratch deleted the codex/security-guidance branch September 11, 2026 22:55

@HAYDEN-OAI HAYDEN-OAI 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.

Retrospective review of 3828825 (this PR merged while the requested review was pending). No substantive introduced issues found across the complete three-file diff.

Checked the private-reporting route, package and repository scope, SDK-owned trust boundaries, contributor/agent guidance, and release-policy wording against the existing release procedure and shared SDK security requirements. The new text preserves repository-specific protections and distinguishes required controls from verified configuration; it does not claim that documentation establishes compliance or clears findings.

Static review only. Tests, lint, CI status, coverage, and mergeability were not run or assessed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants