Skip to content

feat: add comment-sicko agent and generalize base branch resolution - #243

Merged
Cain96 merged 2 commits into
mainfrom
feat/comment-sicko-and-base-branch-resolution
Aug 31, 2026
Merged

feat: add comment-sicko agent and generalize base branch resolution#243
Cain96 merged 2 commits into
mainfrom
feat/comment-sicko-and-base-branch-resolution

Conversation

@Cain96

@Cain96 Cain96 commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a comment-sicko subagent and /no-comments command that hunt narration, banners, commented-out code, and unjustified lint/type suppressions, then land root-cause fixes.
  • Generalize base-branch resolution in /deslop, /thermo-nuclear-code-quality-review, and the git-workflow skill so it no longer hardcodes main.

Changes

  • dot_config/claude/agents/comment-sicko.md: new agent definition for hunting and flagging comment/suppression debt.
  • dot_config/claude/commands/no-comments.md: new command that orchestrates comment-sicko, adjudicates findings, and applies fixes.
  • dot_config/claude/commands/deslop.md, dot_config/claude/commands/thermo-nuclear-code-quality-review.md, dot_config/claude/skills/git-workflow/SKILL.md: resolve the base branch via origin/HEAD / gh / git remote show instead of assuming main.

Test Plan

  • Reviewed diff manually; no executable code paths, config-only changes to Claude command/agent/skill markdown.

Cain96 added 2 commits August 31, 2026 10:30
Repos with a non-main default branch broke /deslop, the thermo-nuclear
review command, and the git-workflow skill's rebase snippet. Resolve
via origin/HEAD, gh, or git remote show before falling back to main.
comment-sicko hunts narration, banners, commented-out code, and
unjustified lint/type suppressions in a diff, flagging root-cause
fixes as MUST KILL without touching application code. no-comments
orchestrates it: spawn, adjudicate findings, fix trivial cases, land
root-cause fixes for the rest, and offer to encode legitimate
constraint comments into types/tests/lint.
@Cain96 Cain96 self-assigned this Aug 31, 2026
@Cain96
Cain96 enabled auto-merge August 31, 2026 01:31
@Cain96
Cain96 merged commit 0d9fcd5 into main Aug 31, 2026
3 checks passed
@Cain96
Cain96 deleted the feat/comment-sicko-and-base-branch-resolution branch August 31, 2026 01:33
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