feat(claude,cursor): fold ponytail into deslop, wire both into code-reviewer - #226
Merged
Merged
Conversation
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
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
Contributor
⏭️ Comment Ratio: SkippedOnly 7 lines added, below the minimum of 50 for this check to apply.
Comment ratio 0% · Limit 15% 4 files analyzed
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 |
Contributor
|
Size Change: 0 B Total Size: 273 B ℹ️ View Unchanged
|
commit: |
…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
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
5 tasks
…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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 Changes
Fold the
ponytailskill and its/ponytailcommand intodeslop, so one skill and one/deslopcommand run three checks in a single pass, across every tool this repo ships (Claude Code, Cursor, Codex, Gemini CLI, OpenCode). Also wiredeslop,jsdoc,humanizer, anddocumentationinto thecode-reviewersubagent's review./deslopruns the reuse-first decision ladder (over-engineering: unneeded deps, wrappers, config), the AI style-tell checklist (needless comments, defensive checks,anycasts), and thehumanizerpattern list over any changed README, doc, comment block, or markdown, so one run covers both code and prose.AskUserQuestion) in a client that has one, a lettered list per theuser-questionsrule otherwise.deslopused to apply its fixes directly; it now asks first, matching howponytailalready worked.code-reviewerfoldsdeslop's ladder and style tells,jsdoc's conventions,humanizer's pattern list, anddocumentation'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,humanizerskills, all six READMEs,AGENTS.md's skill list), regeneratesGEMINI.md, and bumps.codex-plugin/plugin.jsonandgemini-extension.json(0.11.0 → 0.12.0) since the change touches.agents/skills/.🧪 How to test
node --experimental-strip-types scripts/agentFiles.tsfrom the repo root.agent files in sync (3 command sets)with noGEMINI.md is stalewarning./deslopon 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.code-revieweron the same branch and confirm it reports findings by the five categories above, in order, then asks which to hand off.✅ Checklist
pnpm format && pnpm lint:fix && pnpm typecheck && pnpm test.🚀 Release Impact
Breaking change:
/ponytailno longer exists. Run/deslopin its place — it now covers over-engineering, code style tells, and prose humanizing.🤖 Generated with Claude Code
https://claude.ai/code/session_01T8T7uGR5JcXVN3S5zi3nHN