feat(cli): packaged agent SKILL.md, oddish skill command, review --wait - #1278
Closed
kyle-compute wants to merge 9 commits into
Closed
feat(cli): packaged agent SKILL.md, oddish skill command, review --wait#1278kyle-compute wants to merge 9 commits into
kyle-compute wants to merge 9 commits into
Conversation
- Package assets/skills/oddish/SKILL.md in the wheel: a browse.sh-shaped agent guide (auth, atomic nop/oracle sweeps, monitoring, trigger QA then review --wait --json, pull evidence, exit codes, gotchas). - New local-only 'oddish skill' command prints the packaged guide, '--install [--dir]' copies it into a detected agent skills dir, '--path' prints its location. - 'oddish review --wait [--wait-timeout N]' polls zero-limit review pages until no QA run is active, then renders/emits as usual; read-only and warns + emits current state on timeout. - Docs: DOCS.md agent-skill section + review --wait, README command list, AGENTS.md maintenance rule now includes the packaged SKILL.md.
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Oddish previewCommit:
Vercel deployment URL: https://oddish-6wqpljvi6.oddish.app Plan:
This comment is updated by the PR Preview workflow. |
Collaborator
Author
|
Closing: branch is based on cli-improvements, so against staging (== main) the diff includes all of that branch's commits. Will retarget. |
Collaborator
Author
|
Intended to be a 300 line pr, codex sent in pr based upon previous experimental changes. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Ships a prebuilt agent guide inside the oddish package so an agent with
pip install oddishcan self-bootstrap CLI usage without fetching the repo.oddish/src/oddish/assets/skills/oddish/SKILL.md— packaged into the wheel (verified viauv build+unzip -l). Structure follows the browse.sh skill shape: setup/auth rules, core invariants (frozen task version per comparison, nop/oracle baseline gating, task-scoped QA, read-only review), five workflows with exact--jsoncommands, thereview --jsonoutput contract, exit codes, gotchas.oddish skill— new local-only command (no API key): prints the packaged SKILL.md to stdout,--install [--dir]copies it into a detected agent skills dir (./.claude/skills,~/.claude/skills,~/.kimi-code/skills,~/.agents/skills),--pathprints its location. Help text disambiguates from the probe skills library.oddish review --wait [--wait-timeout N]— polls zero-limit review pages until no QA run is active, then renders/emits as usual. Read-only (tests assert GET-only); on timeout warns on stderr and emits the current document with its liveqa.status.Enables the agent QA recipe:
Tests / verification
tests/test_cli_skill.py(frontmatter, print/install/path) + two newtest_cli_review.pycases (wait-until-settle, timeout) — 14 passed.test_cli_status.py+test_cli_backfill_analysis.pysanity — 9 passed.ruff checkclean on touched files.oddish/assets/skills/oddish/SKILL.md.Docs
DOCS.md: new "Agent skill" section,review --waitdocumented, command list updated.oddish/README.md: command list.AGENTS.md: CLI-surface maintenance rule now includes the packaged SKILL.md.Merge-order note
Based on
cli-improvements(82464100) becauseoddish reviewonly exists there. Sincestaging==mainright now, this PR's diff includes thecli-improvementscommits until that branch merges.