feat(rules): add code comment placement rule

Merged
SH
3mo ago
+42-7

Adds a compact code-comments rule to rules/, propagated to all agent instruction files via ruler.

  • Comments carry only non-obvious WHY (hidden constraints, workarounds, surprising behavior)
  • No WHAT comments, change history, or task ID references - those belong in git log, the PR, or the issue
  • Uncertainty goes in the PR body, not comment padding
  • Docs/README: current behavior snapshot only

Based on the placement test: not "is this information useful?" but "is the code the right place for it?"

šŸ¤– Generated with Claude Code


Summary by cubic

Adds a clear code comment rule: write only non‑obvious WHY; keep WHAT, history, and task IDs out of source. Also switches skills tasks to the project‑local CLI and bumps skills to ^1.5.22 for consistent installs.

  • New Features

    • Added rules/code-comments.md with concise guidance; propagated via @intellectronica/ruler.
  • Dependencies

    • Upgraded skills to ^1.5.22; updated bun.lock (includes tar and related deps).
    • Makefile: use local $(SKILLS_CLI) instead of bun x; run bun install --frozen-lockfile --minimum-release-age 0 before skills tasks.
    • Added UPGRADE_LOG.md with verification steps.