Skip to content

Allow compatibility in skill frontmatter validation#376

Open
khanukov wants to merge 1 commit into
openai:mainfrom
khanukov:codex/allow-compatibility-frontmatter
Open

Allow compatibility in skill frontmatter validation#376
khanukov wants to merge 1 commit into
openai:mainfrom
khanukov:codex/allow-compatibility-frontmatter

Conversation

@khanukov
Copy link
Copy Markdown

@khanukov khanukov commented Apr 27, 2026

Summary

Fixes #187.

  • Allow compatibility in quick_validate.py frontmatter validation.
  • Validate compatibility as a non-empty string with the spec's 500 character limit.

Validation

  • Verified skills with valid compatibility values pass quick_validate.py.
  • Verified 500-character compatibility passes and 501-character compatibility fails.
  • Verified empty and non-string compatibility values fail.
  • Verified an unknown frontmatter field still fails validation.
  • Validated all existing skills: validated_skills=44.
  • Ran python3 -m compileall -q skills/.system skills/.curated.
  • Ran git diff --check.

@khanukov khanukov marked this pull request as ready for review April 27, 2026 22:51
@khanukov khanukov requested a review from a team April 27, 2026 22:51
@khanukov khanukov marked this pull request as draft April 27, 2026 22:56
@khanukov khanukov force-pushed the codex/allow-compatibility-frontmatter branch 3 times, most recently from a7e5cbc to b7559db Compare April 27, 2026 23:20
@khanukov khanukov force-pushed the codex/allow-compatibility-frontmatter branch from b7559db to c16d30e Compare April 27, 2026 23:22
@khanukov khanukov marked this pull request as ready for review April 27, 2026 23:26
@khanukov
Copy link
Copy Markdown
Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep it up!

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

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.

Allow "compatibility" property in frontmatter for consistency with Agent Skills Specification

1 participant