Skip to content

skill-quality + planning: run the skill gate in CI over changed skills; injection-block lint decision; interview lint fixes #1153

Description

@kyle-sexton

Parent

Source: handoff-inbox item 20260723-090435-melodic-session-plugins-audit (F3, LOW), producer SW2030 (Windows/PowerShell).

What to build

2 of 3 marketplace skills the producer gated with skill-quality:check fail the marketplace's own static gate — evidence the gate is not run in this repo's CI. Producer results: planning:interview FAIL (MD041 first-line H2; MD013 ×8 clustered in the dynamic ## Pre-computed context injection block; MD060 table pipe; 264-line hub; no "Use when:" phrasing warning), source-control:setup FAIL (already fixed via #1140/PR #1146), re-anchor:sweep-all-disciplines PASS clean.

NOTE before "fixing" lint: this repo's .markdownlint-cli2.jsonc disables MD013 and MD041 repo-side — the producer ran the gate from a consumer context without that config. First verify which findings actually apply under the repo's own lint config (the #1140 lesson), and whether the gate applies repo lint config when run in CI.

Ship:

  1. Run skill-quality:check in this repo's CI over changed skills (changed-file-scoped, not full-tree, to keep CI fast) so shipped skills cannot regress below the marketplace's own bar.
  2. Decide deliberately whether dynamic-injection blocks (declared shell: blocks) are lint-exempt, and encode that decision in the gate itself instead of leaving them as permanent per-consumer failures. Record the decision.
  3. Fix whatever planning:interview findings remain valid under the repo's lint config + the injection-block decision (e.g. MD060 table pipe, hub length soft-cap consideration, "Use when:" phrasing).

Acceptance criteria

  • CI job runs the skill-quality gate over skills changed in the PR; red on gate errors
  • Injection-block lint-exemption decision made, encoded in the gate (skill-quality plugin), and documented
  • planning:interview passes the gate under the repo's lint config after the decision is applied
  • No "fixes" applied for rules the repo's .markdownlint-cli2.jsonc disables
  • Version bumps for every touched plugin per semver weight

Blocked by

None — can start immediately (serialize after the skill-quality resolution fix to avoid same-file conflicts).

Activity

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

Metadata

Metadata

Assignees

Labels

priority: lowNice-to-have, cosmetic, or speculative; opportunistic.status: readyTriaged, unblocked, and fully specified; eligible to pick up.

Type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions