Skip to content

Broaden global skill consistency audits#133

Merged
RomanOstr merged 1 commit into
mainfrom
release/local
Jul 24, 2026
Merged

Broaden global skill consistency audits#133
RomanOstr merged 1 commit into
mainfrom
release/local

Conversation

@RomanOstr

Copy link
Copy Markdown
Contributor

Make skills-consistency-review target one manifest-backed installed skill regardless of name, add selected-skill runtime validation, and align the global automation contract and documentation. Checks passed: targeted runtime tests, full source validation, focused mypy, and Markdown lint.

@RomanOstr
RomanOstr merged commit b0783c3 into main Jul 24, 2026
3 checks passed
@RomanOstr
RomanOstr deleted the release/local branch July 24, 2026 02:22

@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: 862e31992b

ℹ️ 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 on lines +127 to +128
- Run `--mode skill --skill <skill-name> --repo-root <repo-root>` so common and
profile-specific source checks execute only for the selected skill.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve repo-level deterministic checks

When the global automation is the only routine audit, narrowing this step to --mode skill means each per-installed-skill invocation returns after check_selected_skills; the source validator's repo-wide checks for stale README rows, unknown $skill references in portable repo text, and other check_source_governance_consistency invariants only run on the full path. After a rename/removal or stale cross-repo reference outside the selected skill folder, no single-skill invocation covers the dropped repository-level deterministic contract even though the action still claims contract/doc/source consistency, so keep one full source validation per source repo or explicitly move those invariants into selected validation.

AGENTS.md reference: AGENTS.md:L81-L83

Useful? React with 👍 / 👎.

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