make-pdf: "MAKE-PDF SETUP (run BEFORE any make-pdf command)" is buried 13 sections deep
- Dominant language
- TypeScript
- Stars
- 133k
- Forks
- 19.9k
- Avg merge
- 18h 46m
- Merged PRs (30d)
- 26
Description
The `make-pdf/SKILL.md` workflow places the section that explicitly instructs the user to run a setup check **before any make-pdf command** 13 sections after the document's preamble:
```
Line 25: ## Preamble (run first)
Line 105: ## Plan Mode Safe Operations
Line 109: ## Skill Invocation During Plan Mode
Line 216: ## Skill routing
Line 274: ## Artifacts Sync (skill start)
Line 400: ## Model-Specific Behavioral Patch (claude)
Line 418: ## Voice
Line 426: ## Completion Status Protocol
Line 436: ## Operational Self-Improvement
Line 446: ## Telemetry (run last)
Line 475: ## Plan Status Footer
Line 492: ## MAKE-PDF SETUP (run this check BEFORE any make-pdf command) ← here
Line 528: ## Core patterns
```
`## MAKE-PDF SETUP` appears **after** `## Telemetry (run last)` and `## Plan Status Footer`, both of which read like document-closing sections. An agent processing the SKILL.md sequentially has no way to know that a "must-run-first" check is buried at the bottom; it will execute the make-pdf workflow without the SETUP precondition.
### Suggested fix
Move `## MAKE-PDF SETUP (run this check BEFORE any make-pdf command)` to **immediately after** `## Preamble (run first)`. Both sections are preconditions; they belong together at the top of the document.
Optional second pass: rename `## Preamble (run first)` → `## Preamble (always-on environment setup)` and `## MAKE-PDF SETUP` → `## Step 0: make-pdf preflight check` so the ordering is explicit.
Contributor guide
Assessment
This issue has not been assessed yet.