Skip to content

feat: review the agent definitions against community practice and add the wrap-up skill - #20

Merged
archae0pteryx merged 9 commits into
mainfrom
feat/agent-review-builder-reviewer
Oct 28, 2026
Merged

archae0pteryx merged 9 commits into
mainfrom
feat/agent-review-builder-reviewer

Conversation

@archae0pteryx

@archae0pteryx archae0pteryx commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • builder: smallest-diff rule, narrower reading of an ambiguous brief, and a Noticed report section; orchestrate carries Noticed into its Not done.
  • reviewer: pre-existing problems are at most a NOTE, hidden failures are a check, and the unused Skill tool is gone.
  • hall-monitor: a fast rules-only pass over the diff at low effort, with maxTurns.
  • planner: each lane names the existing code it follows as path:line, and the orchestrate brief carries it.
  • comment-janitor: verifies a comment's claims against the code; runs on Sonnet.
  • maxTurns and omitClaudeMd on the narrow agents (grunt, git-janitor-investigator, hall-monitor).
  • New wrap-up skill with scripts/wrap_up.py (tested) doing the end-of-work checklist; git-janitor is now its Haiku background runner and no longer needs killall-skills:git-janitor.
  • tasks/todo.md tracks the review. Plugin version 2.5.0 to 2.6.0.

Checks

build_agents.py --check, test_install.py, test_wrap_up.py, and git diff --check pass.

@archae0pteryx archae0pteryx changed the title feat: keep builder diffs minimal, report what it noticed, and have reviewer check hidden failures feat: review the agent definitions against community practice and add the wrap-up skill Oct 28, 2026
@archae0pteryx
archae0pteryx merged commit c102b0d into main Oct 28, 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