Skip to content

polish: apply /cc-review P4/P5 fixes to new skills and rules#381

Merged
philoserf merged 1 commit intomainfrom
polish/cc-review-fixes
Apr 11, 2026
Merged

polish: apply /cc-review P4/P5 fixes to new skills and rules#381
philoserf merged 1 commit intomainfrom
polish/cc-review-fixes

Conversation

@philoserf
Copy link
Copy Markdown
Owner

Summary

Four small polish fixes surfaced by /cc-review on the recent discipline skills and rules. No behavior changes — readability, consistency, and self-containment.

  • brainstorming — standalone invocation prefixed with slash for consistency with /use-discipline
  • writing-plans — date-prefix path convention mirrored from description into "Before writing" body so it's discoverable when the body is read standalone
  • parallelism — agent-cap rationale inlined; drops the self-reference to ~/.claude/CLAUDE.md since this file is itself loaded from ~/.claude/rules/
  • use-discipline — blank line added before "This skill does not orchestrate review" so it no longer visually attaches to the session-review bullet

Test plan

  • prettier-clean
  • /cc-review surfaced these as P4/P5 items
  • Cross-references (skill-creator:skill-creator, session-review, review-fix, vc-ship, rules/testing.md) still resolve

- brainstorming: prefix standalone invocation with slash for consistency
- writing-plans: mirror date-prefix path convention into body
- parallelism: inline agent-cap rationale, drop CLAUDE.md self-reference
- use-discipline: unstick "does not orchestrate review" with blank line

Co-Authored-By: Claude Code <noreply@anthropic.com>
@philoserf philoserf merged commit 1e7ead2 into main Apr 11, 2026
2 checks passed
@philoserf philoserf deleted the polish/cc-review-fixes branch April 11, 2026 13:48
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