Skip to content

fix(parser): warn when a task has no **Files:** section at all - #63

Merged
bacsystem merged 2 commits into
developfrom
fix/files-section-missing-warning
Jul 22, 2026
Merged

fix(parser): warn when a task has no **Files:** section at all#63
bacsystem merged 2 commits into
developfrom
fix/files-section-missing-warning

Conversation

@bacsystem

Copy link
Copy Markdown
Owner

Summary

Twin of the 0.6.22 **Interfaces:** fix, same root cause, other section: parseFiles still used extractSection, which flattens "section missing" and "section present but empty" to ''. A header typo (**File:**, **Archivos:**) silently removed the task from file-based serialization — two tasks sharing a file could run in parallel and hit an avoidable merge conflict, with a misleading symptom (looks like a Consumes/Produces problem, not a Files one).

Type of change

  • fix — bug fix

Main changes

  • parseFiles now uses extractOptionalSection (same helper parseInterfaces already uses) and warns when the **Files:** section is missing entirely.
  • extractSection is now unused (both parseFiles and parseInterfaces use the optional variant) and has been removed.
  • skills/plan/SKILL.md: extended the 0.6.22 rule to cover both **Files:** and **Interfaces:** headers.
  • Verified plan-parser.js is not inlined into the generated workflow (npm run build + git diff --exit-code workflows/parallel-plan-executor.js — no change).

Version

  • Bump: patch (diagnostics-only, no API or graph change)
  • New version: v0.6.23

Checklist

  • Code reviewed (bugs, conventions, simplifications)
  • CHANGELOG.md updated
  • Version updated in the relevant file(s) (package.json, .claude-plugin/plugin.json, .cursor-plugin/plugin.json, gemini-extension.json)
  • README updated (if applicable) — no user-facing behavior/API change
  • Tests pass (163/163, npm test), TDD (RED confirmed before GREEN)

Twin of the 0.6.22 **Interfaces:** fix, same root cause: extractSection
flattened "missing" and "empty" to ''. A header typo (**File:**) silently
removed the task from file-based serialization — overlapping tasks ran in
parallel and hit avoidable merge conflicts, with a misleading symptom
(the missing dependency looked like a Consumes/Produces problem).
parseFiles now uses extractOptionalSection and warns. extractSection is
now unused (both parseFiles and parseInterfaces use the optional variant)
and has been removed.
@bacsystem
bacsystem merged commit f6943bb into develop Jul 22, 2026
1 check failed
@bacsystem
bacsystem deleted the fix/files-section-missing-warning branch July 22, 2026 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant