Repository navigation
Closed
Feature
2 / 22 of 2 issues completed
Copy link
Labels
priority: mediumReal value, no hard deadline; normal backlog flow.Real value, no hard deadline; normal backlog flow.status: readyTriaged, unblocked, and fully specified; eligible to pick up.Triaged, unblocked, and fully specified; eligible to pick up.work-class: structuralRefactors, migrations, contract changes; cross-cutting and hard to reverse.Refactors, migrations, contract changes; cross-cutting and hard to reverse.
Description
Refs: #7573
Agent Brief
Type: Feature
Summary: /corrections:codify absorbs retro's codify mode and /review:audit-enforceability; both are removed and every reference moves to codify. This is the parent's absorb-and-remove slice.
Current behavior:
plugins/corrections/skills/codify/SKILL.md"Boundary" says: "It does not run theretrocodify mode,/review:audit-enforceabilityorstyle; those keep their own counters until a later slice absorbs them." It also says it does not read review comments from pull requests, and its## Nextroutes finding-shaped checks to/review:audit-enforceability.plugins/review/skills/audit-enforceability/still exists. It maps each finding in a review findings file to the cheapest deterministic check.plugins/session-flow/skills/retro/still has acodify [reviews]mode (context/codify.md,scripts/fetch-review-comments.sh) that mines merged PR review comments.rg -n 'review:audit-enforceability|retro codify' plugins docs --glob '!**/CHANGELOG.md' --glob '!.changes/**'prints about 140 lines across about 60 files on main.
Desired behavior:
- Codify takes the two inputs it lacks, so the removals lose nothing: a review findings file (what audit-enforceability reads), and PR review comments as a counting source, including the author's own after merge (what retro
codify reviewsreads). plugins/review/skills/audit-enforceability/and retro'scodifymode are removed, with their probes, evals, schema keys, config entries and README rows.- Every reference moves to
/corrections:codify. Indocs/upstream/records and in corrections' own files, reword the reference to the current name rather than deleting the record.
Done when
-
rg -n 'review:audit-enforceability|retro codify' plugins docs --glob '!**/CHANGELOG.md' --glob '!.changes/**'prints nothing. -
plugins/review/skills/audit-enforceability/does not exist, and retro's argument hint has nocodifymode. - Codify has eval cases for a findings-file input and a PR-review-comment source, and each removed skill's or mode's eval cases are ported to codify with before/after results in the PR, showing no loss.
-
bash plugins/skill-quality/scripts/check-skill.sh --require-evals plugins/corrections/skillsexits 0, and so doescheck-skill.shforplugins/review/skillsandplugins/session-flow/skills. -
scripts/check-changelog-fragments.shpasses with fragments for corrections, review and session-flow.
Out of scope:
- audit-sessions
style(also absorbed per the parent): a separate slice. - Chat and meeting sources (wait for
communication).
Metadata
Metadata
Assignees
Labels
priority: mediumReal value, no hard deadline; normal backlog flow.Real value, no hard deadline; normal backlog flow.status: readyTriaged, unblocked, and fully specified; eligible to pick up.Triaged, unblocked, and fully specified; eligible to pick up.work-class: structuralRefactors, migrations, contract changes; cross-cutting and hard to reverse.Refactors, migrations, contract changes; cross-cutting and hard to reverse.
Type
Fields
No fields configured for Feature.
Activity
github-actions commentedon Oct 30, 2026
Proposed work class: work-class: structural
Basis: automated intake triage; the lanes App applied this class.
Size: Large
Complexity: Medium
Uncertainty: Knowable
Flags: none
Blocked by: none
Understood: extend /corrections:codify to take a review findings file and PR review comments (including the author's own after merge) as inputs. Then remove review's audit-enforceability skill and retro's codify mode, and repoint all roughly 60 files of references to codify. The done-when list is concrete and checkable, with a grep, skill-check scripts, ported eval cases and changelog fragments. No missing information. I found no open duplicate. The change touches three plugins and about 60 files, so a maintainer may want to split it into per-plugin PRs.
Automated triage, applied by the lanes App.
melodic-automation-lanes commentedon Oct 30, 2026
Not admitted to the issue lane yet: sub-issue #11129 is open.
melodic-automation-lanes commentedon Oct 30, 2026
Split at intake (owner Q93). This issue is Size Large, so its done-when is split into 2 sub-issues, each holding its task-list lines as written.
Automated split by the lanes App. Admission waits on this issue while a sub-issue is open; work-unblock closes it as completed once every sub-issue is closed by a merge or a listed human.
melodic-automation-lanes commentedon Nov 3, 2026
Every sub-issue of this split is closed as completed by a listed human or a merge (#11128, #11129), so the lanes App closes it as completed.