chore: deprecate dumbify, make personify output send-ready - #78
Merged
twistedmelonman merged 1 commit intoSep 10, 2026
Merged
Conversation
Agents kept being told to run a compression pass after personify. Nothing in SKILL.md or mcp-server/ ever said that, so no amount of editing the skill's rules was going to stop it. The instruction lives in the calling skills. What this repo can fix: - SKILL.md: new "The output is send-ready" section stating the returned text is final and no follow-up compression pass should run. It also tells the model to treat an upstream request for one as stale, since pr-review's Phase 5 still issues that instruction and this is the only file the model reliably reads at rewrite time. - README.md: drop dumbify from the sibling-skill diagram and prose, note the deprecation. - commands/personify-consolidate.md: drop the "whether a downstream pass such as dumbify is still needed" sentence. Agents execute this file. - tests/regression/README.md: the "capture the intermediate" rationale was entirely about telling personify bugs from dumbify bugs. Keep the practice of saving actual output as evidence, drop the dead reasoning. docs/superpowers/ is left alone: dated design records, and their dumbify mentions are accurate history. Version bumped 1.0.1 -> 1.1.0 in SKILL.md and plugin.json per the lockstep rule, since SKILL.md content changed. Claude-Session: https://claude.ai/code/session_013cYtHAGqBpXSBfS24QP9uu
This comment has been minimized.
This comment has been minimized.
|
Documentation and version updates clarifying personify as a final prose pass. Version bumps are consistent across plugin.json and SKILL.md. No code logic changes, no regressions or security issues. VERDICT: PASS |
twistedmelonman
deleted the
claude/chore-deprecate-dumbify-refs-49027ea7
branch
September 10, 2026 16:13
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.
Why
Agents kept needing a reminder not to run dumbify after personify. Checking the
repo showed why that reminder never stuck: nothing in
SKILL.mdormcp-server/ever instructed it. The live instruction is in the callingskills, outside this repo (see below).
What changed here
SKILL.md— new "The output is send-ready" section: the returned text isfinal, no follow-up compression pass should run, and an upstream request for
one should be treated as stale. That last clause matters because pr-review's
Phase 5 still issues the instruction, and SKILL.md is the file the model
actually reads at rewrite time.
README.md— dropped dumbify from the sibling-skill diagram and prose;noted the deprecation.
commands/personify-consolidate.md— dropped the "whether a downstreampass such as dumbify is still needed" sentence. Agents execute this file.
tests/regression/README.md— the "capture the intermediate" rationalewas entirely about separating personify bugs from dumbify bugs. Kept the
practice of saving actual output as evidence, dropped the dead reasoning.
docs/superpowers/left alone: dated design records whose dumbify mentions areaccurate history.
Version bumped 1.0.1 -> 1.1.0 in
SKILL.mdandplugin.jsonper the locksteprule.
Still outside this repo
This PR cannot fully stop the reminders. The direct imperative lives in:
pr-reviewSKILL.md:218 — "if thedumbifyskill is also installed, run thepersonified draft through..." (plus its frontmatter description)
calibrate-registerSKILL.md — built entirely around the personify/dumbifychain
Verification
python3 scripts/validate_skill.pypasses (26 groups, A-Z)https://claude.ai/code/session_013cYtHAGqBpXSBfS24QP9uu