Repository navigation
Describe rook 0.1.11 in the agent skill - #52
Merged
Merged
Conversation
Rook-Skill-Coverage: [{"id":"sync-orphans","action":"edit","reason":"Added a note to scenarios.md that sync stops before uploading when a scenario names a feature the agent lacks, lists the fix, and must succeed before an upstream run.","paths":["skill-installer/skills/references/scenarios.md"]},{"id":"registered-session","action":"edit","reason":"Added an errors bullet saying such commands refuse before running or charging when the session is unregistered, and should be reported as no run and no spend.","paths":["skill-installer/skills/references/headless-contract.md"]},{"id":"session-transcript","action":"edit","reason":"Extended the home-state paragraph to say transcript.jsonl records raw input and output of every role call, including refused, failed and interrupted calls.","paths":["skill-installer/skills/references/headless-contract.md"]},{"id":"client-headers","action":"no-edit","reason":"Request headers and debug client display are internal transport details. The skill neither uses nor describes them for driving or interpreting runs.","paths":[]},{"id":"owasp-mapping","action":"edit","reason":"Added to scenarios.md that red-team scenarios carry OWASP LLM, ASI and MITRE ATLAS references, and that generate writes and names unrunnable scenarios with a reason instead of dropping them.","paths":["skill-installer/skills/references/scenarios.md"]},{"id":"redteam-reports","action":"edit","reason":"Added to verdicts.md that a grounded partial compromise counts as Fail and that reports keep refusals, interrupted runs and coverage gaps for the summary.","paths":["skill-installer/skills/references/verdicts.md"]},{"id":"cli-text","action":"no-edit","reason":"SKILL.md and ci.md already teach sync before run, and troubleshooting.md already routes the update remedy through rook update. The guide and help wording changes alter no command, flag or result the skill relies on.","paths":["skill-installer/skills/SKILL.md","skill-installer/skills/references/ci.md","skill-installer/skills/references/troubleshooting.md"]},{"id":"logout-note","action":"no-edit","reason":"The skill never runs rook logout. ci.md already explains that LT_USERNAME and LT_ACCESS_KEY sign commands in ahead of any stored sign-in.","paths":["skill-installer/skills/references/ci.md"]},{"id":"consent-logo","action":"no-edit","reason":"The logo on the consent page and the one-time OAuth client re-registration are cosmetic. They do not change the login commands or remedies the skill documents.","paths":[]},{"id":"boot-banner","action":"no-edit","reason":"The TUI boot banner mascot is purely visual. The skill drives the CLI headlessly and does not describe the banner.","paths":[]}]
lt-user-access-token-stage
Bot
requested review from
gauravBansal06 and
vipulkv
as code owners
October 12, 2026 19:32
…am hooks - generate keeps unrunnable scenarios with executable: false and a skip_reason prefix; coverage-gaps.yaml lists patterns with no scenario. - Red-team patterns that need fixture seeding, a collect hook or a fresh session are skipped until the profile has that hook. - run takes at most two scenarios per attack pattern unless --only. - report --json can carry coverage_gaps, redteam_coverage and summary_claims; an older stored Partial reads as Unable to Verify. - "In 0.1.6" notes that still hold now say "Since 0.1.6". Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
|
I checked the bot's prose against the published I added
Checks against rook 0.1.11:
The branch is 2 commits behind Needs one approval. After the merge, |
gauravBansal06
approved these changes
Oct 13, 2026
This was referenced Oct 13, 2026
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
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.
Regenerated the Rook agent skill for the
0.1.11release.0.1.11(npm@testmuai/rook@0.1.11, releasev0.1.11)claude-opus-5-5skill-installer/skills/references/scenarios.md skill-installer/skills/references/verdicts.md skill-installer/skills/references/headless-contract.mdStated rationale: Documented the release's sync feature check, the registered-session refusal, full session transcripts, OWASP and MITRE scenario references with named unrunnable scenarios, and the report changes for refusals, coverage gaps and partial compromises, while leaving cosmetic, header and text-only changes undocumented.
Behavior accounting (model claims; verify against the diff)
Generated from the published CHANGELOG section, GitHub release notes and help output of the released
binary installed from public npm — no private source was read. The mirrors were
regenerated with
scripts/sync-skill-mirrors.sh, and the skill mirrorand flag harnesses passed against
rook 0.1.11before this PR opened.The contract harness runs in this PR's public CI, away from the private
checkout and the token that opened the PR.
Another release PR can open concurrently; compare this branch with current main before merging.
Read the prose before merging. The harnesses check format, mirror parity and
that every documented flag exists; none of them checks whether a sentence is true.