Skip to content

refactor(code-metrics): audit router wave 3 of 5 (coverage) #12621

Closed as not planned
Closed as not planned

Description

@kyle-sexton

Refs #12491

Blocked by #12620.

Outcome

Wave 3 of 5 in the #12491 sized wave plan. It moves the rest of audit-coverage into the code-metrics:audit router: the body becomes actions/coverage.md and the scripts go under scripts/. It removes the leaf directory and repoints every audit-coverage skill reference in this wave, with no alias. The report skill value and the dispatch.sh id keep audit-coverage.

  • Measured: 130,605 bytes of git diff --no-renames at b92833d99a, after waves 1 and 2.

Done when

  • Before the move, the skill's evals/evals.json cases are converted with convert-evals-json.py (plugins/evals/skills/plugin-eval/SKILL.md:436), and before.json is captured on the pre-move tree.
  • /evals:plugin-eval --baseline before.json --margin 5 passes at non-inferiority.
  • git diff --no-renames origin/main of the PR is under 262144 bytes.
  • ls -d plugins/code-metrics/skills/audit-* | wc -l is 2.
  • scripts/validate-plugins.sh passes, and a merge probe is posted per D82/Q186.

Related

Activity

  1. added
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Nov 3, 2026
  2. github-actions commented on Nov 3, 2026

    @github-actions
    Contributor

    Proposed work class: work-class: structural
    Basis: automated intake triage; the lanes App applied this class.

    Size: Medium
    Complexity: Medium
    Uncertainty: Known
    Flags: none
    Blocked by: #12491

    Understood: wave 3 of 5 in the #12491 plan. It moves the audit-coverage skill body and scripts into the code-metrics:audit router, removes the leaf directory, and repoints references. The audit-coverage report value and dispatch id stay unchanged. The scope and done-when checks are exact. The work is blocked until the wave 2 work under #12491 is done. I found no duplicates among the open issues. The sibling wave issues are separate steps of the same plan.

    Automated triage, applied by the lanes App.

  3. added
    priority: mediumReal value, no hard deadline; normal backlog flow.
    and removed
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Nov 3, 2026
  4. added
    status: readyTriaged, unblocked, and fully specified; eligible to pick up.
    work-class: structuralRefactors, migrations, contract changes; cross-cutting and hard to reverse.
    on Nov 3, 2026
  5. melodic-automation-lanes commented on Nov 3, 2026

    @melodic-automation-lanes
    Contributor

    Not admitted to the issue lane yet: blocker #12491 is not closed by a listed human or a merge.

  6. added
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    on Nov 3, 2026
  7. 1 remaining item

  8. github-actions commented on Nov 3, 2026

    @github-actions
    Contributor
    No description provided.
  9. github-actions commented on Nov 3, 2026

    @github-actions
    Contributor
    No description provided.
  10. melodic-automation-lanes commented on Nov 3, 2026

    @melodic-automation-lanes
    Contributor

    The work-resolve-issue lane stopped on this issue (stopped): the run stopped for a reason it gave no code for, which names no intent, design, security or cost finding, so the lane runs it again only if the issue has had no retry, for any stop, since a listed human last added agent-ready, and otherwise stops.
    It keeps agent-ready, and work-unblock starts it again (retry 1 of 1).

    In the run's words:

    Prerequisite waves not on the default branch: wave 3 moves audit-coverage into the code-metrics:audit router, but plugins/code-metrics/skills/audit/ does not exist at 9e00972. The router is created by wave 1 (#12619), and this issue is blocked by wave 2 (#12620), which is still an open issue with no linked branch or pull request. Run this again after #12619 and #12620 merge.

    What the run says it did not do:

    • All done-when items (eval conversion and baseline, plugin-eval non-inferiority, diff size, leaf count, validate-plugins and merge probe): not started because the prerequisite router from waves 1 and 2 is absent
    • Repository checks (check-changelog-fragments, check-changed-skills, check-affected) were not run because the working tree has no changes

    Run: https://github.com/melodic-software/claude-code-plugins/actions/runs/41698521453

  11. added
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    and removed
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    on Nov 3, 2026
  12. github-actions commented on Nov 3, 2026

    @github-actions
    Contributor
    No description provided.
  13. github-actions commented on Nov 3, 2026

    @github-actions
    Contributor
    No description provided.
  14. added
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    and removed
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    on Nov 3, 2026
  15. melodic-automation-lanes commented on Nov 3, 2026

    @melodic-automation-lanes
    Contributor

    The work-resolve-issue lane stopped partway through this issue and needs a person.

    Reason: design

    Why, in the run's own words:

    design-change: wave 3 needs the code-metrics:audit router, and that router is not on the default branch. plugins/code-metrics/skills/audit/ does not exist at 0ec3c9a. Wave 1 (#12619), which creates the router, is open, and its PR #12641 was closed without merging. Wave 2 (#12620), which this issue says blocks it, is open with no branch or PR, and join.py and test_join.py are still under skills/audit-coverage/scripts/. If this run built the router, it would be choosing wave 1's primary design here, and the combined diff would go over the per-wave 262144-byte budget. A person needs to land waves 1 and 2, or re-plan the sequence, before this issue can run.

    What the run says it did not do:

    • All five done-when items (eval conversion and before.json baseline, plugin-eval non-inferiority, diff byte budget, audit-* leaf count of 2, validate-plugins.sh and merge probe): not started, because waves 1 and 2 are unbuilt
    • Repository checks (check-changelog-fragments, check-changed-skills, check-affected) not run: the run stopped before any edit, so there is no change to check

    Run: https://github.com/melodic-software/claude-code-plugins/actions/runs/41702609643

    To retry, remove needs-human and add agent-ready again.

  16. removed
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    on Nov 3, 2026
  17. kyle-sexton commented on Nov 3, 2026

    @kyle-sexton
    MemberAuthor

    Closed not planned: the owner re-scoped #12491 to option 3 (keep the leaves, distinct trimmed descriptions) and dropped the router merge and its waves: #12491 (comment). The remaining work is #12491 itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority: mediumReal value, no hard deadline; normal backlog flow.work-class: structuralRefactors, migrations, contract changes; cross-cutting and hard to reverse.

    Type

    Fields

    Work status

    agent-ready

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions