Skip to content

docs(skill): require minimal repository-native security fixes - #583

Draft
tdullien1 wants to merge 1 commit into
mainfrom
codex/minimal-native-security-patches
Draft

docs(skill): require minimal repository-native security fixes#583
tdullien1 wants to merge 1 commit into
mainfrom
codex/minimal-native-security-patches

Conversation

@tdullien1

Copy link
Copy Markdown
Collaborator

Summary

Make automatically generated security patches follow local coding conventions and remain narrowly focused on the vulnerability being fixed.

Changes

  • Identify applicable organization-wide and project-specific style guides before editing.
  • Match nearby naming, types, control flow, formatting, and error-handling conventions.
  • Preserve existing APIs, helper signatures, assertions, and legitimate behavior unless a change is required for security closure.
  • Avoid unrelated refactoring, cleanup, renaming, and formatting changes.
  • Review the complete candidate patch against its original base while preserving pre-existing user changes.

Testing

  • Focused CLI skill-command suite: bun test --timeout 30000 tests-ts/cli-skills.test.ts — 25 passed.
  • Bundled skill structural validation — passed.
  • Prettier check for the changed skill — passed.
  • git diff --check — passed.

Risk and rollout

Instruction-only update to one bundled skill; no runtime APIs, dependencies, or release configuration change. Existing requirements for complete security closure and preserving user-owned changes remain intact.

Public disclosure review

  • No customer, partner, prospect, or user identities, data, or identifying details are included.
  • No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
  • I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 20, 2026

@mldangelo-oai mldangelo-oai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great! I will trust that this performs better than it did before. Would be cool to eval but not necessary

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.

2 participants