backlog.py: plan field N/A blocks work-backlog-item Step 2

Fact-Check

Date: 2026-02-27 Claims checked: 4

Summary: VERIFIED: 4 | REFUTED: 0 | INCONCLUSIVE: 0

RT-ICA

Goal: Prevent work-backlog-item Step 2 from treating "N/A" plan values as valid plan paths.

Decision: APPROVED Missing: None — all conditions AVAILABLE or DERIVABLE

Groomed (2026-02-27)

Reproducibility

1. Run uv run .claude/skills/backlog/scripts/backlog.py list --format json
2. Filter results for items with status: done or status: resolved
3. Note items with plan: "N/A" in JSON output
4. Invoke /work-backlog-item with title of one of these items
5. Observe Step 2 checks if plan field is truthy (SKILL.md line 199-205)
6. Confirm false-positive: "This item already has a plan at N/A"

Output / Evidence

• backlog.py line 260: item["**Plan**"] = str(meta.get("plan") or fm.get("plan") or "")
  • Converts plan: N/A to string "N/A", which is truthy in Python
• backlog.py line 1003: _update_item_metadata(..., {"metadata": {"status": "done", ... "plan": plan}})
  • Explicitly sets plan: N/A when closing items
• work-backlog-item/SKILL.md lines 199-205: checks if plan field is truthy without filtering out "N/A"
• Current barrier: 22 items with plan: N/A all have status: done, so they're filtered at JSON stage; latent bug surfaces if a completed item gets reopened or a non-completed item receives plan: N/A

Priority

7/10 — Prevents work-backlog-item from processing items marked done (current safety by accident), but exposes code smell: N/A should normalize to empty, not stringify to truthy value. Low immediate risk (all affected items filtered), moderate debt (confuses workflow state).

Impact

• Blocks: Items with plan: N/A cannot be reopened and re-planned without manual frontmatter edit
• Bottleneck: Design confusion between "completed" (status: done) and "no plan needed" (plan: N/A); both conflate in Step 2 logic

Scope

Two viable fixes (both produce same outcome):

Option A: Normalize in backlog.py parsing (recommended)

• Line 260: Change str(meta.get("plan") or fm.get("plan") or "") to filter N/A to empty
• Applies globally; all downstream code sees clean data
• Single source of truth

Option B: Guard in work-backlog-item Step 2

• Lines 199-205: Add condition if plan and plan != "N/A" before stopping
• Localized; other skills/agents using backlog.py still receive N/A strings

Output / Evidence

Expected Behavior: When work-backlog-item Step 2 extracts plan field, it should treat "N/A" as "no plan available" (equivalent to empty string), not as a valid plan path.

Acceptance Criteria:

1. backlog.py list --format json on items with plan: N/A returns plan: "" (empty string)
2. /work-backlog-item with a reopened item does NOT report "plan at N/A"
3. Existing 22 files with plan: N/A are unaffected (status: done)
4. New items closing without a plan serialize to plan: "" in JSON

Dependencies

• Depends on: None
• Blocks: None (safe to do independently)

Research

No external research needed. Bug is fully characterized from codebase analysis.

Skills

• /work-backlog-item — consumer of the plan field
• /backlog — manages per-item files and JSON serialization

Agents

None required for fix implementation.

Prior Work

• backlog.py lines 260, 1003 — plan field serialization and close logic
• work-backlog-item/SKILL.md lines 199-205 — Step 2 plan guard

Files

• .claude/skills/backlog/scripts/backlog.py (lines 260, 1003)
• .claude/skills/work-backlog-item/SKILL.md (lines 199-205)

Decision

APPROVED — Small fix, no blockers. Recommend Option A (normalize at parse time).

Effort

Small — 1-line string filter or 1-line guard condition. ~15 minutes implementation + validation.