Skip to content

[codex] Split meta skill CLI and review gates - #1

Merged
yaojingang merged 509 commits into
mainfrom
codex/meta-skill-architecture-followup
Jun 17, 2026
Merged

[codex] Split meta skill CLI and review gates#1
yaojingang merged 509 commits into
mainfrom
codex/meta-skill-architecture-followup

Conversation

@yaojingang

Copy link
Copy Markdown
Owner

Summary

  • Split the Yao CLI argparse surface into scripts/yao_cli_parser.py while preserving the existing scripts/yao.py command handlers and public CLI behavior.
  • Split Review Studio gate evaluation into scripts/review_studio_gates.py, leaving render_review_studio.py focused on report assembly and HTML rendering.
  • Updated trust/package/registry/Review Studio artifacts so the new internal modules are declared, audited, and included in the generated archive.
  • Fixed the Review Studio test setup so install simulation evidence generated during tests no longer overwrites the root report with a temporary package path.

Validation

  • python3 tests/verify_review_studio.py
  • make ci-test -> 56/56 targets passed
  • git diff --check
  • Archive consistency check confirmed both new internal modules are present in dist/yao-meta-skill.zip and Review Studio remains ready with score 100.

Notes

The unrelated untracked local file reports/yao-deploy-bridge-skill-research-plan.md was intentionally excluded from this commit.

@yaojingang
yaojingang force-pushed the codex/meta-skill-architecture-followup branch from 99fa202 to f42bed2 Compare June 13, 2026 12:00
@yaojingang
yaojingang marked this pull request as ready for review June 17, 2026 10:42
@yaojingang
yaojingang merged commit 31ce04c into main Jun 17, 2026
2 checks passed
@yaojingang
yaojingang deleted the codex/meta-skill-architecture-followup branch June 17, 2026 10:43
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