Skip to content

feat(planning): fail a plan whose file:line citation does not exist, and stamp the commit it was checked at #10037

Description

@kyle-sexton

Refs: #6924

What to do

W5 of #6917. Add a script that reads a PLAN.md, resolves every path:line citation against a named commit, and fails on any path or line that does not exist; wire it into /planning:plan's outcome gate, and have plans record the commit the check ran at.

Done when

  • The script's test exits 0 and includes a case where a plan citing a missing file, and one citing a line past the end of a file, each exit non-zero, plus a stay-quiet case for a valid citation.
  • git grep -n on plugins/planning/skills/plan/ shows the outcome gate calling the script and the plan template carrying a checked-at commit line.
  • bash plugins/skill-quality/scripts/check-skill.sh plugins/planning/skills exits 0.
  • ci-status is green on the PR, and the PR body carries Refs: #6917.

Split by the owner's HITL session (2026-10-24, owner decisions C12 and Q93-Q95). Intake triage sets its fields.

Activity

  1. added theissue type on Oct 25, 2026
  2. added
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Oct 25, 2026
  3. github-actions commented on Oct 25, 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: add a script that reads a PLAN.md, resolves each path:line citation against a named commit, and fails on a missing file or a line past the end of the file. It would be wired into /planning:plan's outcome gate, and the plan template would gain a checked-at commit line. The done-when items are concrete and checkable. I found no likely duplicates among the open issues.

    Automated triage, applied by the lanes App.

  4. 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 Oct 25, 2026
  5. changed the issue type fromtoon Oct 25, 2026
  6. added
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    and removed on Oct 25, 2026
  7. changed the issue type fromtoon Oct 25, 2026
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

    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.priority: mediumReal value, no hard deadline; normal backlog flow.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.

    Type

    Fields

    No fields configured for Task.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions