Skip to content

feat(claude,cursor): fold ponytail into deslop, wire both into code-reviewer - #226

Merged
stijnvanhulle merged 7 commits into
mainfrom
claude/eloquent-maxwell-1eg8x2
Sep 18, 2026
Merged

stijnvanhulle merged 7 commits into
mainfrom
claude/eloquent-maxwell-1eg8x2

Conversation

@stijnvanhulle

@stijnvanhulle stijnvanhulle commented Sep 18, 2026

Copy link
Copy Markdown
Owner

🎯 Changes

Fold the ponytail skill and its /ponytail command into deslop, so one skill and one /deslop command run three checks in a single pass, across every tool this repo ships (Claude Code, Cursor, Codex, Gemini CLI, OpenCode). Also wire deslop, jsdoc, humanizer, and documentation into the code-reviewer subagent's review.

  • /deslop runs the reuse-first decision ladder (over-engineering: unneeded deps, wrappers, config), the AI style-tell checklist (needless comments, defensive checks, any casts), and the humanizer pattern list over any changed README, doc, comment block, or markdown, so one run covers both code and prose.
  • Lists every finding — file, what it violates, proposed fix — and stops before editing anything.
  • Confirms with the user before applying: a native picker (AskUserQuestion) in a client that has one, a lettered list per the user-questions rule otherwise. deslop used to apply its fixes directly; it now asks first, matching how ponytail already worked.
  • code-reviewer folds deslop's ladder and style tells, jsdoc's conventions, humanizer's pattern list, and documentation's style guide into its findings, in that order, then asks which to hand off for fixing. It stays read-only: it reports by these skills' criteria rather than running their apply steps.

Also updates every cross-reference (pr, humanizer skills, all six READMEs, AGENTS.md's skill list), regenerates GEMINI.md, and bumps .codex-plugin/plugin.json and gemini-extension.json (0.11.0 → 0.12.0) since the change touches .agents/skills/.

🧪 How to test

  • Step 1: Run node --experimental-strip-types scripts/agentFiles.ts from the repo root.
  • Step 2: Confirm it prints agent files in sync (3 command sets) with no GEMINI.md is stale warning.
  • Step 3: In Claude Code, type /deslop on a branch with a diff that changes both code and a README, and confirm it lists code and prose findings and asks before editing anything.
  • Step 4: Invoke code-reviewer on the same branch and confirm it reports findings by the five categories above, in order, then asks which to hand off.

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I ran the full pre-PR check locally: pnpm format && pnpm lint:fix && pnpm typecheck && pnpm test.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is for the docs (no release).
  • This change is breaking, and the body says what breaks and what a user has to do.

Breaking change: /ponytail no longer exists. Run /deslop in its place — it now covers over-engineering, code style tells, and prose humanizing.

🤖 Generated with Claude Code

https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN

Runs the reuse-first ladder and the AI style-tell checklist in one pass,
reports every finding, and applies only what the user confirms through
the native picker (or a lettered list where none exists), matching how
ponytail already asked before editing.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN
@codesandbox

codesandbox Bot commented Sep 18, 2026

Copy link
Copy Markdown

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@github-actions

github-actions Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

⏭️ Comment Ratio: Skipped

Only 7 lines added, below the minimum of 50 for this check to apply.

Code Comments
Added +7 0
Removed −16 0
Net −9 0

Comment ratio 0%  ·  Limit 15%

4 files analyzed
File Language Code Δ Comments Δ Ratio
tools/gemini/commands/deslop.toml TOML +7 0 0%
.codex-plugin/plugin.json JSON 0 0
gemini-extension.json JSON 0 0
tools/gemini/commands/ponytail.toml TOML −16 0

The ratio is the share of comment lines among all lines added. Lines are counted per changed file before and after the change; positive deltas are summed. Comparing bf5d39f…44eda88, counted with tokei 12.1.2.

@github-actions

Copy link
Copy Markdown
Contributor

Size Change: 0 B

Total Size: 273 B

ℹ️ View Unchanged
Filename Size
packages/core/dist/index.js 112 B
packages/demo/dist/index.js 161 B

compressed-size-action

@pkg-pr-new

pkg-pr-new Bot commented Sep 18, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/stijnvanhulle/template/@stijnvanhulle/template-core@226
npm i https://pkg.pr.new/stijnvanhulle/template/@stijnvanhulle/template-demo@226

commit: 50bd115

…deslop

Keep the existing deslop name instead of introducing trim, since it already
covers the code-slop half of the merge and needs no new name to learn.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN
.claude/worktrees/ holds a background agent's isolated git worktree,
a local scratch checkout that never belongs in version control.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN
@stijnvanhulle stijnvanhulle changed the title feat(claude,cursor): merge deslop and ponytail into one trim skill feat(claude,cursor): fold ponytail into the deslop skill Sep 18, 2026
Fold prose review into deslop instead of pointing to humanizer as a
separate step, so one /deslop run covers over-engineering, code style
tells, and AI writing tells in any changed README, doc, or comment.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN
… into code-reviewer

Have code-reviewer check for over-engineering, code and prose AI tells,
and JSDoc conventions in one pass, ordered substance before style before
prose, then ask which findings to hand off for fixing. It stays
read-only: it reports by these skills' criteria rather than running
their apply steps.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN
@stijnvanhulle stijnvanhulle changed the title feat(claude,cursor): fold ponytail into the deslop skill feat(claude,cursor): fold ponytail into deslop, wire both into code-reviewer Sep 18, 2026
claude and others added 2 commits September 18, 2026 10:03
…eded

Broaden the user-questions rule past fixed-choice forks: any time a task
is blocked on information only the user can supply, use the native
picker with best-guess options and Other, instead of a plain question.
Secrets and credentials are the one exception and stay plain.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN
Split the jsdoc skill's example gallery into references/examples.md so the
default load only carries the tag tables and guidelines, shortened its
frontmatter description (always in context in every skill listing), and
tightened repetitive prose in the pr skill's steps without dropping any
instruction.


Claude-Session: https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN

Co-authored-by: Claude <noreply@anthropic.com>
@stijnvanhulle
stijnvanhulle merged commit e0639c2 into main Sep 18, 2026
9 checks passed
@github-actions
github-actions Bot deleted the claude/eloquent-maxwell-1eg8x2 branch September 18, 2026 10:29
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.

2 participants