Skip to content

Fix skill repo review findings - #12

Merged
songxuchen merged 1 commit into
mainfrom
codex/fix-review-findings
Aug 16, 2026
Merged

Fix skill repo review findings#12
songxuchen merged 1 commit into
mainfrom
codex/fix-review-findings

Conversation

@songxuchen

Copy link
Copy Markdown
Contributor

What changed

  • Parse skill frontmatter with PyYAML in tests/validate.py and install PyYAML in CI.
  • Update README single-skill install command to use --skill.
  • Update eval docs to reflect Claude Code 2.1.232 behavior: plugin eval is documented but still early-access gated at runtime.
  • Clarify auto-dubbing CLI timeout guidance for normal terminals vs agent host shells.

Validation

  • python3 tests/validate.py
  • YAML parse of workflow, eval cases, and every */SKILL.md frontmatter
  • claude plugin validate --strict .claude-plugin/plugin.json
  • claude plugin validate --strict .claude-plugin/marketplace.json
  • Markdown local-link check
  • git diff --check

Note: a smoke run of claude plugin eval . --case routing-video-scoring --runs 1 --no-publish ... still exits with plugin eval is currently in early access, which is now documented.

@sapient-app

sapient-app Bot commented Aug 16, 2026

Copy link
Copy Markdown

Lightsage docs evals

Waiting for the staging docs URL before running evals.

Lightsage will start the selected PR evals automatically when GitHub reports a successful docs deployment for this PR. This usually happens within 15 minutes.

Commit: efa40df
Status: waiting for staging docs URL

@songxuchen
songxuchen merged commit 3157330 into main Aug 16, 2026
1 check passed
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