Skip to content

Concise skills rollout: visualization (2 skills, 2 model-invoked) #6595

Description

@kyle-sexton

Part of #6476. Rewrite visualization's skills to the skill criteria in one draft PR, following the per-plugin runbook (/playbooks:skill-authoring, reference/rewrite-a-plugin.md). The runbook and the rewrite protocol it cites decide what ships; this issue holds only what is specific to the plugin.

The plugin today

As of origin/main on 2026-10-24:

  • Skills: present, visualize (model-invoked).
  • Outcome eval case directories under the plugin: 0. Probe files under probes/: 0.

Done when

  • /skill-quality:check plugins/visualization on the PR head reports no FAIL that the run on main does not.
  • /evals:validate passes on plugins/visualization's eval suite at the PR head.
  • Each model-invoked skill has a probe file: plugins/visualization/probes/present.json, plugins/visualization/probes/visualize.json.
  • The PR body holds a per-skill decision table, description and body decided separately, with the before and after numbers behind each row.
  • The PR body names every skill or case left unmeasured, with the reason.
  • The plugin's version or changelog entry is in the PR, per the plugin's release mode.

Eval spend and review

  • Eval passes within prepaid API credit are pre-approved (owner decision Q46, 2026-10-24), at the --max-cost-usd 5 per-pass ceiling set on #6476 on 2026-10-18.
  • No person gates the PR: the owner's 2026-10-24 decision (C12) leaves only an intent mismatch during the build or a security issue at review as stops.

Generated by Claude Code

Activity

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

    @github-actions
    Contributor

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

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

    Understood: rewrite the two model-invoked skills in the visualization plugin (present, visualize) to the skill criteria in one draft PR, following the per-plugin runbook. The done-when list is explicit: skill-quality check, eval validation, a probe file per skill, a decision table in the PR body, unmeasured items named, and a version or changelog entry. The earlier hold was released by the owner's 2026-10-24 comment. I found no open questions and no likely duplicates.

    Automated triage, applied by the lanes App.

  3. added theissue type on Oct 12, 2026
  4. added
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    and removed
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Oct 12, 2026
  5. kyle-sexton commented on Oct 19, 2026

    @kyle-sexton
    MemberAuthor

    This was generated by AI during triage.

    Held: per the owner decision on #6476 (2026-10-18), this plugin is in the rollout middle group, which the owner decides after batch 2 reports its numbers. It waits for that decision.

  6. removed
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    on Oct 24, 2026
  7. kyle-sexton commented on Oct 24, 2026

    @kyle-sexton
    MemberAuthor

    Released by the owner's HITL session (2026-10-24, C12): it waited only on a removed gate. Intake triage re-runs on it to set its fields, then admission takes it.

  8. github-actions commented on Oct 25, 2026

    @github-actions
    Contributor

    The triage sweep dispatched intake triage for this issue (attempt 1 of 2).

  9. added
    status: readyTriaged, unblocked, and fully specified; eligible to pick up.
    work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    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 Oct 25, 2026
  10. melodic-automation-lanes commented on Oct 25, 2026

    @melodic-automation-lanes
    Contributor

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

    Why, in the run's own words:

    The rewrite protocol (/playbooks:skill-authoring skill-criteria.md ## Measurement step 6, and reference/rewrite-a-plugin.md steps 1, 2, 5, 6, 8) ships only parts that were measured: it needs /skill-quality:check before and after, /evals:validate, paid /evals:plugin-eval passes on each target model, /skill-quality:check measure-invocation, and /docs-hygiene:compress compare. This CI lane loads only the work-items plugin, and its Bash runs only the lane wrappers (no claude CLI, no bash on repo scripts), so none of those can run here. Rewriting either skill without them would ship unmeasured text, which the protocol forbids, and the PR's per-skill decision table needs numbers this lane cannot produce. Done instead: step 3, the frozen probe sets for both model-invoked skills, written blind by fresh agents from the before descriptions and committed before any rewrite. A run that can execute the eval tooling (an attended session, or a lane with the evals and skill-quality plugins and the claude CLI) should continue from this branch.

    What the run says it did not do:

    • Skill rewrite of present and visualize (runbook step 4): not started, because the protocol ships only measured parts and measurement cannot run in this lane
    • /skill-quality:check plugins/visualization before and after: the skill-quality plugin is not loaded and its script cannot run under this lane's Bash allowlist
    • /evals:validate on the visualization eval suite, and the case-design, calibration and smoke pass (runbook step 2): the evals plugin is not loaded here
    • Paid eval passes before and after on each target model, plus the measure-invocation held-out trigger run (runbook step 5): no claude CLI or eval tooling in this lane
    • /docs-hygiene:compress compare meaning diff with two labelers (runbook step 6): plugin not loaded
    • Per-skill decision table with before and after numbers, and the list of unmeasured skills or cases, in the PR body: no numbers exist yet
    • bash plugins/skill-quality/scripts/measure-invocation.sh validate plugins/visualization/probes: cannot run here (Bash allows only the lane wrappers); CI runs it for every plugins/*/probes directory, left to the PR's CI
    • new-changelog-fragment wrapper refused a detached HEAD, so the fragment was written by hand in the same format; check-changelog-fragments accepts it

    Partial work is on the branch work-resolve-issue/issue-6595-40190516272-1. A retry continues from it while it is not behind the default branch, and otherwise starts fresh from the default branch; delete the branch first to always start fresh.

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

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

  11. kyle-sexton commented on Oct 27, 2026

    @kyle-sexton
    MemberAuthor

    This was generated by AI.

    Parked for capacity, not for the owner. This rollout's remaining work is the measured eval passes of the batch-1 protocol, which the issue lane cannot run: it cannot start claude or claude plugin eval (see the lane's escalation on #6585, https://github.com/melodic-software/claude-code-plugins/actions/runs/40186335064). It waits on the plugin eval dispatch work in #9704, now a blocked-by edge, so admission holds it until #9704 closes.

    Under the owner's 2026-10-26 stop rule, a person is needed only for a change in intent or vision, an implementation finding that changes the intent or primary design, or a high or critical security issue; host, attended and eval runs are agent work. So needs-human comes off, and the issue moves to host-run with status: confirmed, which keeps the issue lane from taking it when #9704 closes: #9704 adds an eval dispatch workflow, not eval capability for the issue lane.

  12. added
    host-runNeeds an unattended run on a host; the host runner takes it, not a person or the issue lane.
    status: confirmedA maintainer confirmed the item is real and in scope; ready to be worked.
    and removed
    status: readyTriaged, unblocked, and fully specified; eligible to pick up.
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    on Oct 27, 2026
  13. self-assigned this
    on Oct 31, 2026
  14. kyle-sexton commented on Oct 31, 2026

    @kyle-sexton
    MemberAuthor
    No description provided.
  15. kyle-sexton commented on Oct 31, 2026

    @kyle-sexton
    MemberAuthor

    Result: landed in #11623 (merged 2026-10-31T16:47:23Z).

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

Metadata

Metadata

Assignees

Labels

host-runNeeds an unattended run on a host; the host runner takes it, not a person or the issue lane.priority: mediumReal value, no hard deadline; normal backlog flow.status: confirmedA maintainer confirmed the item is real and in scope; ready to be worked.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.

Type

Fields

No fields configured for Task.

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions