Skip to content

[codex] Implement SkillOps command group - #1

Merged
ponponusa merged 3 commits into
mainfrom
codex/skillops-implemantation
Jul 4, 2026
Merged

[codex] Implement SkillOps command group#1
ponponusa merged 3 commits into
mainfrom
codex/skillops-implemantation

Conversation

@ponponusa

Copy link
Copy Markdown
Owner

Summary

  • Add the nested skills command group for SkillOps inventory, static checks, reports, sync, routes, eval workspace planning, and explicit Codex eval runner support.
  • Implement dependency-free Agent Skills frontmatter and eval manifest validation with deterministic registry/report generation.
  • Update README, Japanese README, extension guides, changelog, and tests for the new workflow.

Validation

  • PYTHONPYCACHEPREFIX=/tmp/agent-context-pycache scripts/run_checks.sh
  • Adversarial phase reviews for PR1 and PR2-PR5 converged with no remaining findings before commit.

@ponponusa
ponponusa marked this pull request as ready for review July 4, 2026 05:41
@ponponusa
ponponusa requested a review from Copilot July 4, 2026 05:41

Copilot AI 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.

Pull request overview

Implements a new nested skills command group (“SkillOps”) in scripts/agent_context.py to inventory/validate repository-local Agent Skills under .agents/skills/, generate deterministic registry and health reports, sync compact skill routes into .agents/routing.md, and scaffold eval workspaces / run explicit Codex evals. This is integrated into top-level check() and documented across READMEs, extension guides, and the changelog, with expanded unit test coverage.

Changes:

  • Add SkillOps core implementation: skill discovery, dependency-free frontmatter parsing, local reference + eval manifest validation, registry/report generation, routes sync, and eval workspace/runner helpers.
  • Integrate skill validation and route checking into the existing check() flow and add CLI subcommands under skills.
  • Update documentation and add extensive tests covering success and failure cases (including symlink and unsafe-path handling).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tests/test_agent_context.py Adds end-to-end and unit tests for SkillOps inventory/check/sync/routes/workspace/Codex-runner behavior.
scripts/agent_context.py Implements SkillOps command group, validation/parsing logic, deterministic outputs, and check() integration.
references/extension-guide.md Documents new SkillOps outputs and extension points.
references/extension-guide.ja.md Japanese documentation update for SkillOps commands and helpers.
README.md Adds SkillOps usage documentation and CLI examples.
README.ja.md Japanese README update for SkillOps usage and safety notes.
CHANGELOG.md Records new skills command group and capabilities.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/agent_context.py Outdated
Comment thread scripts/agent_context.py Outdated
Comment thread scripts/agent_context.py
Comment thread scripts/agent_context.py Outdated
Comment thread scripts/agent_context.py

@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: cf567f9b46

ℹ️ 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 scripts/agent_context.py Outdated
Comment thread scripts/agent_context.py Outdated

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comment thread scripts/agent_context.py Outdated
Comment thread scripts/agent_context.py Outdated
Comment thread README.md Outdated
@ponponusa
ponponusa merged commit a6d41e7 into main Jul 4, 2026
6 checks passed
@ponponusa
ponponusa deleted the codex/skillops-implemantation branch July 4, 2026 06:19
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.

2 participants