paperlint 3.2.0 → 4.0.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (312) hide show
  1. package/CLAUDE.md +2 -2
  2. package/dist/adapters/banal/failure.d.ts +1 -7
  3. package/dist/adapters/banal/failure.d.ts.map +1 -1
  4. package/dist/adapters/banal/failure.js +5 -7
  5. package/dist/adapters/banal/failure.js.map +1 -1
  6. package/dist/adapters/banal/index.d.ts.map +1 -1
  7. package/dist/adapters/banal/index.js +8 -11
  8. package/dist/adapters/banal/index.js.map +1 -1
  9. package/dist/adapters/banal/install.d.ts.map +1 -1
  10. package/dist/adapters/banal/install.js +4 -3
  11. package/dist/adapters/banal/install.js.map +1 -1
  12. package/dist/adapters/banal/invocation.d.ts +9 -4
  13. package/dist/adapters/banal/invocation.d.ts.map +1 -1
  14. package/dist/adapters/banal/invocation.js +13 -1
  15. package/dist/adapters/banal/invocation.js.map +1 -1
  16. package/dist/adapters/banal/locate.d.ts +12 -9
  17. package/dist/adapters/banal/locate.d.ts.map +1 -1
  18. package/dist/adapters/banal/locate.js +19 -12
  19. package/dist/adapters/banal/locate.js.map +1 -1
  20. package/dist/adapters/banal/run.d.ts +10 -1
  21. package/dist/adapters/banal/run.d.ts.map +1 -1
  22. package/dist/adapters/banal/run.js +8 -4
  23. package/dist/adapters/banal/run.js.map +1 -1
  24. package/dist/adapters/banal/settings.d.ts +1 -1
  25. package/dist/adapters/banal/settings.d.ts.map +1 -1
  26. package/dist/adapters/banal/settings.js +2 -1
  27. package/dist/adapters/banal/settings.js.map +1 -1
  28. package/dist/adapters/memory/index.d.ts.map +1 -1
  29. package/dist/adapters/memory/index.js +8 -2
  30. package/dist/adapters/memory/index.js.map +1 -1
  31. package/dist/adapters/node/process.io.d.ts +6 -3
  32. package/dist/adapters/node/process.io.d.ts.map +1 -1
  33. package/dist/adapters/node/process.io.js.map +1 -1
  34. package/dist/adapters/node/workspace.io.d.ts.map +1 -1
  35. package/dist/adapters/node/workspace.io.js +4 -2
  36. package/dist/adapters/node/workspace.io.js.map +1 -1
  37. package/dist/adapters/references/index.d.ts.map +1 -1
  38. package/dist/adapters/references/index.js +4 -2
  39. package/dist/adapters/references/index.js.map +1 -1
  40. package/dist/build-engine.d.ts +1 -1
  41. package/dist/build-engine.d.ts.map +1 -1
  42. package/dist/build-engine.js +8 -1
  43. package/dist/build-engine.js.map +1 -1
  44. package/dist/build.d.ts +2 -3
  45. package/dist/build.d.ts.map +1 -1
  46. package/dist/build.js +5 -6
  47. package/dist/build.js.map +1 -1
  48. package/dist/caller-path.d.ts +3 -0
  49. package/dist/caller-path.d.ts.map +1 -0
  50. package/dist/caller-path.js +13 -0
  51. package/dist/caller-path.js.map +1 -0
  52. package/dist/cli.d.ts +10 -17
  53. package/dist/cli.d.ts.map +1 -1
  54. package/dist/cli.js +62 -12
  55. package/dist/cli.js.map +1 -1
  56. package/dist/doctor.d.ts +12 -3
  57. package/dist/doctor.d.ts.map +1 -1
  58. package/dist/doctor.js +2 -1
  59. package/dist/doctor.js.map +1 -1
  60. package/dist/domain/paths.d.ts +8 -2
  61. package/dist/domain/paths.d.ts.map +1 -1
  62. package/dist/domain/paths.js +12 -1
  63. package/dist/domain/paths.js.map +1 -1
  64. package/dist/domain/ready.d.ts +24 -0
  65. package/dist/domain/ready.d.ts.map +1 -0
  66. package/dist/domain/ready.js +11 -0
  67. package/dist/domain/ready.js.map +1 -0
  68. package/dist/domain/record.d.ts +8 -0
  69. package/dist/domain/record.d.ts.map +1 -0
  70. package/dist/domain/record.js +8 -0
  71. package/dist/domain/record.js.map +1 -0
  72. package/dist/domain/sha256.d.ts.map +1 -1
  73. package/dist/domain/sha256.js +4 -2
  74. package/dist/domain/sha256.js.map +1 -1
  75. package/dist/engine.d.ts +23 -22
  76. package/dist/engine.d.ts.map +1 -1
  77. package/dist/engine.js.map +1 -1
  78. package/dist/eslint-rules/bib-reachable-entry.d.ts +1 -1
  79. package/dist/eslint-rules/bib-reachable-entry.js.map +1 -1
  80. package/dist/eslint-rules/install-path-literals.d.ts +2 -2
  81. package/dist/eslint-rules/install-path-literals.js.map +1 -1
  82. package/dist/eslint-rules/paper-research-question.d.ts +1 -1
  83. package/dist/eslint-rules/paper-research-question.js.map +1 -1
  84. package/dist/eslint-rules/paper-stages.d.ts +2 -2
  85. package/dist/eslint-rules/paper-stages.js.map +1 -1
  86. package/dist/eslint-rules/paper-typography.d.ts +6 -6
  87. package/dist/eslint-rules/paper-typography.js.map +1 -1
  88. package/dist/eslint-rules/pdf-last-page-balance.d.ts +1 -1
  89. package/dist/eslint-rules/pdf-last-page-balance.js.map +1 -1
  90. package/dist/eslint-rules/review-frontmatter.d.ts +2 -2
  91. package/dist/eslint-rules/review-frontmatter.js.map +1 -1
  92. package/dist/eslint-rules/sibling-frontmatter.d.ts +1 -1
  93. package/dist/eslint-rules/sibling-frontmatter.d.ts.map +1 -1
  94. package/dist/eslint-rules/temp-root-realpath.d.ts +1 -1
  95. package/dist/eslint-rules/temp-root-realpath.js.map +1 -1
  96. package/dist/eslint-rules/tex-build.d.ts +2 -2
  97. package/dist/eslint-rules/tex-build.js.map +1 -1
  98. package/dist/facts-file.d.ts.map +1 -1
  99. package/dist/facts-file.js +29 -26
  100. package/dist/facts-file.js.map +1 -1
  101. package/dist/hooks-settings.d.ts.map +1 -1
  102. package/dist/hooks-settings.js +17 -2
  103. package/dist/hooks-settings.js.map +1 -1
  104. package/dist/init.d.ts +3 -4
  105. package/dist/init.d.ts.map +1 -1
  106. package/dist/init.js +18 -8
  107. package/dist/init.js.map +1 -1
  108. package/dist/lib/paper-config.d.ts.map +1 -1
  109. package/dist/lib/paper-config.js +1 -5
  110. package/dist/lib/paper-config.js.map +1 -1
  111. package/dist/new-paper.d.ts.map +1 -1
  112. package/dist/new-paper.js +5 -1
  113. package/dist/new-paper.js.map +1 -1
  114. package/dist/paper-files.d.ts +4 -4
  115. package/dist/paper-files.d.ts.map +1 -1
  116. package/dist/paper-files.js +6 -6
  117. package/dist/paper-files.js.map +1 -1
  118. package/dist/paper-settings.js +3 -3
  119. package/dist/paper-settings.js.map +1 -1
  120. package/dist/pdf-facts.d.ts +51 -1
  121. package/dist/pdf-facts.d.ts.map +1 -1
  122. package/dist/pdf-facts.js +23 -15
  123. package/dist/pdf-facts.js.map +1 -1
  124. package/dist/pdf-geometry.d.ts.map +1 -1
  125. package/dist/pdf-geometry.js +2 -3
  126. package/dist/pdf-geometry.js.map +1 -1
  127. package/dist/ports/tool-installer.d.ts +2 -14
  128. package/dist/ports/tool-installer.d.ts.map +1 -1
  129. package/dist/presets.d.ts.map +1 -1
  130. package/dist/presets.js +16 -4
  131. package/dist/presets.js.map +1 -1
  132. package/dist/references.d.ts.map +1 -1
  133. package/dist/references.js +4 -4
  134. package/dist/references.js.map +1 -1
  135. package/dist/rules-config.d.ts +26 -1
  136. package/dist/rules-config.d.ts.map +1 -1
  137. package/dist/rules-config.js +9 -9
  138. package/dist/rules-config.js.map +1 -1
  139. package/dist/tex-requirements.d.ts +4 -0
  140. package/dist/tex-requirements.d.ts.map +1 -1
  141. package/dist/tex-requirements.js +14 -2
  142. package/dist/tex-requirements.js.map +1 -1
  143. package/dist/types.d.ts +5 -3
  144. package/dist/types.d.ts.map +1 -1
  145. package/dist/venue-rules.d.ts.map +1 -1
  146. package/dist/venue-rules.js +20 -7
  147. package/dist/venue-rules.js.map +1 -1
  148. package/docs/configuration.md +2 -2
  149. package/eslint-rules/bib-reachable-entry.ts +1 -1
  150. package/eslint-rules/install-path-literals.ts +2 -2
  151. package/eslint-rules/paper-research-question.ts +1 -1
  152. package/eslint-rules/paper-stages.ts +2 -2
  153. package/eslint-rules/paper-typography.ts +2 -2
  154. package/eslint-rules/pdf-last-page-balance.ts +1 -1
  155. package/eslint-rules/review-frontmatter.ts +1 -1
  156. package/eslint-rules/temp-root-realpath.ts +1 -1
  157. package/eslint-rules/tex-build.ts +2 -2
  158. package/eslint.config.mjs +7 -0
  159. package/fixtures/install-path-literals/clean.fixture.mjs +1 -1
  160. package/fixtures/install-path-literals/clean.md +2 -2
  161. package/fixtures/install-path-literals/defect.fixture.mjs +1 -1
  162. package/fixtures/install-path-literals/defect.md +2 -2
  163. package/hooks/paper-status-gates.sh +9 -19
  164. package/lib/paper-config.harness.mjs +1 -1
  165. package/lib/paper-config.ts +1 -5
  166. package/lib/skill-checks.mjs +24 -381
  167. package/lib/skill-checks.test.mjs +63 -115
  168. package/lib/skill-corpus.mjs +10 -251
  169. package/lib/skill-corpus.test.mjs +2 -13
  170. package/lib/skill-trigger-cases.mjs +1 -1
  171. package/package.json +1 -1
  172. package/scripts/coverage-config.test.ts +0 -1
  173. package/scripts/harness-api.frozen.json +0 -1
  174. package/skills/analyze-sibling-paper/SKILL.md +1 -39
  175. package/skills/analyze-sibling-paper/SKILL.md.spec.ts +0 -38
  176. package/skills/argument-arc/SKILL.md +2 -36
  177. package/skills/argument-arc/SKILL.md.spec.ts +1 -43
  178. package/skills/build-benchmark/SKILL.md +1 -41
  179. package/skills/build-benchmark/SKILL.md.spec.ts +0 -40
  180. package/skills/camera-ready/SKILL.md +1 -34
  181. package/skills/camera-ready/SKILL.md.spec.ts +0 -33
  182. package/skills/cold-read-diff/SKILL.md +1 -35
  183. package/skills/cold-read-diff/SKILL.md.spec.ts +0 -34
  184. package/skills/draft-paper/SKILL.md +2 -41
  185. package/skills/draft-paper/SKILL.md.spec.ts +1 -50
  186. package/skills/extend-paper/SKILL.md +2 -33
  187. package/skills/extend-paper/SKILL.md.spec.ts +1 -42
  188. package/skills/find-venue/SKILL.md +2 -38
  189. package/skills/find-venue/SKILL.md.spec.ts +1 -47
  190. package/skills/grade-paper-writing/SKILL.md +3 -38
  191. package/skills/grade-paper-writing/SKILL.md.spec.ts +2 -47
  192. package/skills/grade-paper-writing/prose-lint.mjs +2 -3
  193. package/skills/grade-paper-writing/prose-lint.test.ts +1 -1
  194. package/skills/harden-paper/SKILL.md +1 -39
  195. package/skills/harden-paper/SKILL.md.spec.ts +0 -38
  196. package/skills/map-prior-work/SKILL.md +1 -35
  197. package/skills/map-prior-work/SKILL.md.spec.ts +0 -34
  198. package/skills/osf-artifact-upload/osf-artifact-upload.harness.ts +3 -4
  199. package/skills/paper-adversarial-review/SKILL.md +2 -35
  200. package/skills/paper-adversarial-review/SKILL.md.spec.ts +1 -43
  201. package/skills/paper-pipeline/SKILL.md +2 -28
  202. package/skills/paper-pipeline/SKILL.md.spec.ts +1 -27
  203. package/skills/paper-pipeline/grade-paper-writing-ablation.eval.mjs +1 -1
  204. package/skills/paper-pipeline/paper-pipeline.harness.mjs +25 -40
  205. package/skills/paper-pipeline/references/acceptance-gate.md +1 -6
  206. package/skills/paper-pipeline/references/pipeline-status-template.md +8 -35
  207. package/skills/paper-pipeline/scripts/README.md +30 -225
  208. package/skills/paper-pipeline/scripts/artifact-coverage.harness.mjs +4 -8
  209. package/skills/paper-pipeline/scripts/consumer.harness.mjs +6 -104
  210. package/skills/paper-pipeline/scripts/consumer.mjs +23 -104
  211. package/skills/paper-pipeline/scripts/generated-code.harness.mjs +2 -3
  212. package/skills/paper-pipeline/scripts/population-map.harness.mjs +3 -3
  213. package/skills/paper-status/SKILL.md +2 -42
  214. package/skills/paper-status/SKILL.md.spec.ts +1 -41
  215. package/skills/pc-panel-review/SKILL.md +2 -46
  216. package/skills/pc-panel-review/SKILL.md.spec.ts +1 -45
  217. package/skills/plan-paper-timeline/SKILL.md +2 -36
  218. package/skills/plan-paper-timeline/SKILL.md.spec.ts +0 -36
  219. package/skills/render-paper/SKILL.md +1 -37
  220. package/skills/render-paper/SKILL.md.spec.ts +0 -36
  221. package/skills/research-ideate/SKILL.md +2 -39
  222. package/skills/research-ideate/SKILL.md.spec.ts +1 -47
  223. package/skills/study-accepted-papers/SKILL.md +1 -34
  224. package/skills/study-accepted-papers/SKILL.md.spec.ts +0 -33
  225. package/skills/submit-paper/SKILL.md +1 -33
  226. package/skills/submit-paper/SKILL.md.spec.ts +0 -32
  227. package/skills/sweep-design-space/SKILL.md +1 -38
  228. package/skills/sweep-design-space/SKILL.md.spec.ts +0 -37
  229. package/skills/tighten-paper/SKILL.md +2 -39
  230. package/skills/tighten-paper/SKILL.md.spec.ts +1 -47
  231. package/skills/tighten-paper/structure.mjs +4 -11
  232. package/skills/verify-citations/SKILL.md +2 -38
  233. package/skills/verify-citations/SKILL.md.spec.ts +1 -37
  234. package/src/adapters/banal/failure.ts +9 -10
  235. package/src/adapters/banal/index.test.ts +2 -2
  236. package/src/adapters/banal/index.ts +8 -10
  237. package/src/adapters/banal/install.ts +9 -4
  238. package/src/adapters/banal/invocation.test.ts +20 -8
  239. package/src/adapters/banal/invocation.ts +23 -8
  240. package/src/adapters/banal/locate.test.ts +2 -2
  241. package/src/adapters/banal/locate.ts +27 -21
  242. package/src/adapters/banal/run.test.ts +2 -2
  243. package/src/adapters/banal/run.ts +16 -10
  244. package/src/adapters/banal/settings.ts +2 -2
  245. package/src/adapters/curl/download.test.ts +4 -4
  246. package/src/adapters/memory/index.ts +4 -3
  247. package/src/adapters/node/files.test.ts +5 -3
  248. package/src/adapters/node/process.io.ts +9 -2
  249. package/src/adapters/node/process.test.ts +4 -4
  250. package/src/adapters/node/workspace.io.ts +9 -4
  251. package/src/adapters/references/index.ts +7 -2
  252. package/src/build-engine.test.ts +11 -6
  253. package/src/build-engine.ts +10 -5
  254. package/src/build.test.ts +10 -11
  255. package/src/build.ts +8 -9
  256. package/src/caller-path.ts +13 -0
  257. package/src/cli.harness.mjs +0 -2
  258. package/src/cli.test.ts +19 -8
  259. package/src/cli.ts +82 -25
  260. package/src/doctor.test.ts +1 -1
  261. package/src/doctor.ts +14 -5
  262. package/src/domain/paths.test.ts +15 -0
  263. package/src/domain/paths.ts +20 -1
  264. package/src/domain/ready.ts +31 -0
  265. package/src/domain/record.test.ts +20 -0
  266. package/src/domain/record.ts +12 -0
  267. package/src/domain/sha256.ts +5 -3
  268. package/src/engine.ts +30 -2
  269. package/src/facts-file.ts +43 -28
  270. package/src/hooks-settings.harness.ts +17 -0
  271. package/src/hooks-settings.ts +28 -4
  272. package/src/init.test.ts +25 -5
  273. package/src/init.ts +26 -12
  274. package/src/new-paper.harness.mjs +7 -23
  275. package/src/new-paper.ts +7 -3
  276. package/src/paper-files.test.ts +22 -19
  277. package/src/paper-files.ts +16 -27
  278. package/src/paper-settings.test.ts +2 -2
  279. package/src/paper-settings.ts +5 -5
  280. package/src/pdf-facts.test.ts +16 -5
  281. package/src/pdf-facts.ts +79 -22
  282. package/src/pdf-geometry.ts +5 -3
  283. package/src/ports/tool-installer.ts +2 -17
  284. package/src/presets.ts +28 -14
  285. package/src/references.test.ts +25 -19
  286. package/src/references.ts +4 -5
  287. package/src/rules-config.ts +40 -11
  288. package/src/tex-requirements.test.ts +21 -0
  289. package/src/tex-requirements.ts +29 -6
  290. package/src/toolchain.test.ts +9 -5
  291. package/src/toolchain.ts +2 -2
  292. package/src/types.ts +5 -3
  293. package/src/venue-rules.test.ts +8 -4
  294. package/src/venue-rules.ts +21 -11
  295. package/skills/paper-pipeline/scripts/announce.mjs +0 -67
  296. package/skills/paper-pipeline/scripts/announce.test.mjs +0 -102
  297. package/skills/paper-pipeline/scripts/fixtures/clean/PIPELINE-STATUS.md +0 -51
  298. package/skills/paper-pipeline/scripts/fixtures/dirty/PIPELINE-STATUS.md +0 -52
  299. package/skills/paper-pipeline/scripts/fixtures/dirty/paper.md +0 -6
  300. package/skills/paper-pipeline/scripts/fixtures/scorecard.ts +0 -100
  301. package/skills/paper-pipeline/scripts/ledger.cli.test.mjs +0 -302
  302. package/skills/paper-pipeline/scripts/ledger.mjs +0 -623
  303. package/skills/paper-pipeline/scripts/ledger.test.mjs +0 -254
  304. package/skills/paper-pipeline/scripts/pipeline-check.cli.test.mjs +0 -474
  305. package/skills/paper-pipeline/scripts/pipeline-check.harness.mjs +0 -328
  306. package/skills/paper-pipeline/scripts/pipeline-check.mjs +0 -750
  307. package/skills/paper-pipeline/scripts/pipeline-edges.mjs +0 -169
  308. package/skills/paper-pipeline/scripts/pipeline-edges.test.mjs +0 -73
  309. package/skills/paper-pipeline/scripts/run-mechanical.mjs +0 -648
  310. package/skills/paper-pipeline/scripts/run-mechanical.test.mjs +0 -468
  311. package/skills/paper-pipeline/scripts/status.mjs +0 -295
  312. package/skills/paper-pipeline/scripts/status.test.mjs +0 -232
@@ -7,11 +7,9 @@
7
7
  *
8
8
  * 1. where is the consumer's repository root, now that walking `..` from here lands in
9
9
  * `node_modules/` instead;
10
- * 2. where does the run ledger live, now that the directory next to this file is wiped by
11
- * `npm ci`;
12
- * 3. was this file executed, or merely imported — a question whose usual answer stops
10
+ * 2. was this file executed, or merely imported — a question whose usual answer stops
13
11
  * working the moment the file is reached through a symlink;
14
- * 4. by what path a SKILL'S PROSE names these scripts, so that a checker can tell an
12
+ * 3. by what path a SKILL'S PROSE names these scripts, so that a checker can tell an
15
13
  * instruction that runs them from one that runs something else. See the fourth-carrier
16
14
  * section at the bottom of this file.
17
15
  *
@@ -23,7 +21,7 @@
23
21
  * paper-pipeline/scripts
24
22
  *
25
23
  * so every command a skill's prose already names — `node .claude/skills/paper-pipeline/scripts/
26
- * ledger.mjs …` — keeps working unchanged. That is the whole reason the symlink exists: the
24
+ * <script>.mjs …` — keeps working unchanged. That is the whole reason the symlink exists: the
27
25
  * first consumer names those paths 306 times across 129 files, almost all of it in the PROSE of
28
26
  * skills, which no refactor can rewrite.
29
27
  *
@@ -35,12 +33,11 @@
35
33
  * process.cwd() /…/<the consumer repository root>
36
34
  *
37
35
  * Node resolves the entry point to its realpath (`--preserve-symlinks-main` is off by default)
38
- * but leaves `argv[1]` as typed. Three consequences, each one a function here:
36
+ * but leaves `argv[1]` as typed. Two consequences, each one a function here:
39
37
  *
40
38
  * 🔴 `import.meta.url === \`file://${process.argv[1]}\`` — the main guard nine scripts in this
41
39
  * directory used — is `false` through the symlink. NOT an error: the process exits 0 having
42
- * done nothing. `ledger.mjs record …` would silently record nothing, `announce.mjs` would
43
- * silently announce nothing, and every caller would read success. `isMain()` compares
40
+ * done nothing, and every caller reads success. `isMain()` compares
44
41
  * against the REALPATH of `argv[1]` instead, which is true through both routes.
45
42
  *
46
43
  * 🔴 `resolve(import.meta.dirname, "..", "..", "..", "..")` — the repository root, correct
@@ -48,30 +45,9 @@
48
45
  * by `process.cwd()`, which is the consumer root because every one of those 306 invocations
49
46
  * is `node .claude/skills/…` typed from the repository root. `consumerRoot()`.
50
47
  *
51
- * 🔴 `join(import.meta.dirname, "runs.jsonl")` — the ledger's default home — now points inside
52
- * `node_modules`, which `npm ci` deletes. Appending there is SILENT DATA LOSS: the write
53
- * succeeds, the rows are real, and they vanish at the next install with no error anywhere.
54
- * `ledgerPath()` refuses to return that path at all.
55
- *
56
- * ── THE LEDGER, THREE RUNGS ─────────────────────────────────────────────────
57
- * 1. `PIPELINE_LEDGER` in the environment — always wins. This is what test harnesses set to
58
- * keep fixture rows out of real history, so it must outrank a declaration on disk.
59
- * 2. `{ "ledger": "…" }` in the CONSUMER's root `paperlint.json`, read
60
- * from `process.cwd()`, resolved relative to it.
61
- * 3. `runs.jsonl` beside this file — ONLY when this file is not inside `node_modules`, i.e.
62
- * when the package is being developed in its own checkout. Inside `node_modules` with
63
- * nothing declared, this THROWS.
64
- *
65
- * 🔴 WHY RUNG 3 THROWS RATHER THAN GUESSING. Every other candidate default is worse in the same
66
- * direction. Writing beside the file loses data at the next `npm ci`. Writing to
67
- * `<cwd>/runs.jsonl` puts an append-only journal at the top of someone's repository without
68
- * asking. Writing nowhere makes `record()` a no-op, which is the stored "nothing was wrong" this
69
- * ledger exists to abolish. A throw is the only one of the four that cannot be mistaken for
70
- * having worked — and it carries the cure, naming the key and the file to put it in.
71
- *
72
- * ⚠️ THE EXAMPLES HERE ARE DELIBERATELY GENERIC (`docs/pipeline-runs.jsonl`). The first
73
- * consumer's own directory names are absent by the same rule `papers.harness.mjs` part IV
74
- * enforces for `papers.mjs`: a package that spells out one user's private tree has one user.
48
+ * ⚠️ THE EXAMPLES HERE ARE DELIBERATELY GENERIC. The first consumer's own directory names are
49
+ * absent by the same rule `papers.harness.mjs` part IV enforces for `papers.mjs`: a package that
50
+ * spells out one user's private tree has one user.
75
51
  */
76
52
  import {
77
53
  existsSync,
@@ -246,68 +222,22 @@ function readlinkOr(entry, readlink) {
246
222
  * True when `dir` lies inside a `node_modules` directory.
247
223
  *
248
224
  * Segment-wise, not `includes("node_modules")`: a repository legitimately named
249
- * `my-node_modules-inspector` is not an installed package, and a substring test would refuse to
250
- * write its ledger. Cheap to get right, and the failure it prevents is a throw in someone's face.
225
+ * `my-node_modules-inspector` is not an installed package, and a substring test would refuse it.
226
+ * Cheap to get right, and the failure it prevents is a throw in someone's face.
251
227
  */
252
228
  export function insideNodeModules(dir) {
253
229
  return dir.split(sep).includes("node_modules");
254
230
  }
255
231
 
256
- /**
257
- * Where the run ledger lives. Three rungs; see the docblock.
258
- *
259
- * @param hereDir the directory of the module that owns the ledger — pass `import.meta.dirname`.
260
- * @returns an absolute path. Never a path inside `node_modules`.
261
- */
262
- export function ledgerPath(
263
- hereDir,
264
- { env = process.env, cwd = process.cwd() } = {},
265
- ) {
266
- // Rung 1 — the environment. Deliberately first: harnesses redirect the ledger BEFORE importing
267
- // it, and a declaration on disk that could override that would put fixture rows in real history.
268
- if (env.PIPELINE_LEDGER) return resolve(env.PIPELINE_LEDGER);
269
-
270
- // Rung 2 — the consumer's declaration.
271
- const root = consumerRoot({ env, cwd });
272
- const declared = settingsOf(consumerRoot({ env, cwd }))?.ledger;
273
- // 🔴 `declared === undefined`, NOT `declared ?? default` — the same distinction `papersRoot()`
274
- // makes and for the same reason: `"ledger": null` is a keystroke, not an absence, and reading
275
- // it as "nothing was declared" would silently pick a different file than the one asked for.
276
- if (declared !== undefined) {
277
- if (typeof declared !== "string" || declared.length === 0)
278
- throw new TypeError(
279
- `${CONFIG_KEY}: "ledger" must be a non-empty string, got ${JSON.stringify(declared)}`,
280
- );
281
- return resolve(root, declared);
282
- }
283
-
284
- // Rung 3 — beside this file, but only in the package's own checkout.
285
- if (insideNodeModules(hereDir))
286
- throw new Error(
287
- `${CONFIG_KEY}: no ledger location is declared, and the default (a file beside this ` +
288
- `module) is inside node_modules, which \`npm ci\` deletes.\n` +
289
- `Declare where the ledger lives, in ${join(root, CONFIG_FILE)}:\n` +
290
- ` { "ledger": "docs/pipeline-runs.jsonl" }\n` +
291
- `or set PIPELINE_LEDGER for a single run.\n` +
292
- `This is thrown rather than defaulted on purpose: appending to a path under ` +
293
- `node_modules succeeds, so the rows would look recorded right up until the next ` +
294
- `install removed them, with no error at any point.`,
295
- );
296
- return join(hereDir, "runs.jsonl");
297
- }
298
-
299
232
  // ═════════════════════════════════════════════════════════════════════════════════════════════
300
- // THE FOURTH CARRIER — WHERE A SKILL'S PROSE NAMES THESE SCRIPTS
233
+ // WHERE A SKILL'S PROSE NAMES THESE SCRIPTS
301
234
  //
302
- // `papersRoot()` answers "where does the consumer keep its papers"; `ledgerPath()` answers
303
- // "where does the run journal live"; `citeChecks` (read in `run-mechanical.mjs`) answers "where
304
- // are the consumer's own citation checkers". This answers the fourth: BY WHAT PATH does a skill
305
- // instruct the model to run `announce.mjs` and `ledger.mjs`.
235
+ // `papersRoot()` answers "where does the consumer keep its papers". This answers: BY WHAT PATH
236
+ // does a skill instruct the model to run this package's scripts.
306
237
  //
307
- // 🔴 IT IS A DIFFERENT KIND OF PATH FROM THE OTHER THREE, and the difference decides the API.
308
- // The other three are read by CODE and may be absolute. This one is compared against text a
309
- // human wrote inside a SKILL.md — `node .claude/skills/paper-pipeline/scripts/ledger.mjs record
310
- // …` — so it must stay relative to the consumer root and spelled with `/`, exactly as the prose
238
+ // 🔴 IT IS A DIFFERENT KIND OF PATH, and the difference decides the API. `papersRoot()` is read by
239
+ // CODE and may be absolute. This one is compared against text a human wrote inside a SKILL.md —
240
+ // `node .claude/skills/paper-pipeline/scripts/<script>.mjs …` — so it must stay relative to the consumer root and spelled with `/`, exactly as the prose
311
241
  // spells it. Returning an absolute path here would make every `startsWith` test false and every
312
242
  // check that depends on it pass over an empty set.
313
243
  //
@@ -346,7 +276,7 @@ export function scriptsRoot({ env = process.env, cwd = process.cwd() } = {}) {
346
276
  const root = consumerRoot({ env, cwd });
347
277
  const declared = settingsOf(consumerRoot({ env, cwd }))?.scripts;
348
278
  // 🔴 `declared === undefined`, NOT `declared ?? DEFAULT` — the same distinction `papersRoot()`
349
- // and `ledgerPath()` make, for the same reason: `"scripts": null` is a keystroke, not an
279
+ // makes, for the same reason: `"scripts": null` is a keystroke, not an
350
280
  // absence, and silently substituting the default for it hides a typo behind a working run.
351
281
  const rel = declared === undefined ? DEFAULT_SCRIPTS_ROOT : declared;
352
282
  if (typeof rel !== "string" || rel.length === 0)
@@ -385,12 +315,11 @@ export function scriptsRoot({ env = process.env, cwd = process.cwd() } = {}) {
385
315
  }
386
316
 
387
317
  // ═════════════════════════════════════════════════════════════════════════════════════════════
388
- // THE SIXTH CARRIER — WHICH WALL CLOCK THE CONSUMER SCHEDULES AGAINST
318
+ // WHICH WALL CLOCK THE CONSUMER SCHEDULES AGAINST
389
319
  //
390
- // `papersRoot()` answers "where are the papers", `ledgerPath()` "where is the journal",
391
- // `citeChecks` "where are the consumer's citation checkers", `scriptsRoot()` "by what path does
392
- // prose name these scripts", `triggerCases` "where are the consumer's own trigger cases". This
393
- // answers the sixth: IN WHAT TIME ZONE does a deadline anchor get written.
320
+ // `papersRoot()` answers "where are the papers", `scriptsRoot()` "by what path does prose name
321
+ // these scripts", `triggerCases` "where are the consumer's own trigger cases". This answers: IN
322
+ // WHAT TIME ZONE does a deadline anchor get written.
394
323
  //
395
324
  // 🔴 WHY IT IS A DECLARATION AND NOT A CONSTANT. A deadline is the one quantity in this pipeline
396
325
  // that is meaningless without a zone, and the zone belongs to the PERSON, not to the pipeline —
@@ -446,7 +375,7 @@ export function consumerTimezone({
446
375
  }
447
376
 
448
377
  // ═════════════════════════════════════════════════════════════════════════════════════════════
449
- // THE SEVENTH CARRIER — WHOM THE SCHOLARLY APIs SHOULD CONTACT
378
+ // WHOM THE SCHOLARLY APIs SHOULD CONTACT
450
379
  //
451
380
  // Crossref and friends run a "polite pool": send a `mailto:` in the User-Agent and you get the
452
381
  // faster, more reliable tier, plus a warning by email instead of a silent block if your usage
@@ -485,13 +414,7 @@ export function consumerContactEmail({
485
414
  return declared;
486
415
  }
487
416
 
488
- /**
489
- * The individual script paths, derived from one root.
490
- *
491
- * 🔴 THE NAMES LIVE HERE, NOT IN THE CONSUMER — the same half that keeps `paperFiles()` honest.
492
- * `announce.mjs` and `ledger.mjs` are this package's contract; a consumer that spelled them out
493
- * itself would keep its copy of the list in step by hand, and a hand-kept list rots silently.
494
- */
417
+ /** The prefix every prose path to this package's scripts starts with, derived from one root. */
495
418
  export function pipelineScripts(root) {
496
419
  // 🔴 ONE TRAILING SLASH, NOT THE ONE THAT WAS TYPED. `scriptsRoot()` returns the declaration
497
420
  // verbatim — it must, the value is compared against prose — so `"scripts": "tools/pipeline/"`
@@ -504,9 +427,5 @@ export function pipelineScripts(root) {
504
427
  return {
505
428
  /** What every prose path under this root starts with. Trailing slash, for `startsWith`. */
506
429
  prefix: `${base}/`,
507
- /** Announces that a gate has started. */
508
- announce: `${base}/announce.mjs`,
509
- /** Appends the verdict row. */
510
- ledger: `${base}/ledger.mjs`,
511
430
  };
512
431
  }
@@ -317,9 +317,8 @@ function fixture({ scripts = {}, allow = null, bundle = {}, at = "repro" }) {
317
317
  }
318
318
 
319
319
  // ── 11. the ledger prints on a CLEAN run, and the header states the count ───────────────────────
320
- // The count matters beyond tidiness: `run-mechanical.mjs` reads this check in `flags` mode, where
321
- // it prefers a stated count and otherwise COUNTS OUTPUT LINES. The ledger is ~20 lines that are not
322
- // findings, so without the stated count a clean paper would be recorded as twenty findings.
320
+ // The count matters beyond tidiness: the ledger is ~20 lines that are not findings, so a caller
321
+ // counting output lines would read a clean paper as twenty findings.
323
322
  {
324
323
  const r = run(
325
324
  fixture({
@@ -11,7 +11,7 @@
11
11
  * The findings themselves are planted in `population-map.test.mjs` (vitest; until #52 a
12
12
  * self-test script this harness spawned). This file keeps what only the command shows: finding
13
13
  * paper.md and repro/populations.tsv, and `--flags-only` staying silent on a clean paper — the
14
- * wiring the CI step and `run-mechanical.mjs` actually invoke.
14
+ * wiring the CI step actually invokes.
15
15
  */
16
16
  import assert from "node:assert/strict";
17
17
  import { execFileSync } from "node:child_process";
@@ -107,8 +107,8 @@ const REG = [
107
107
  }
108
108
 
109
109
  // ── 4. KNOWN GAP, asserted so it stays visible ─────────────────────────────────────────
110
- // 🔴 A paper with NO repro/populations.tsv exits 0 in silence, and `run-mechanical.mjs` records
111
- // that silence as PASS for draft-paper. So a paper that never declared its populations scores
110
+ // 🔴 A paper with NO repro/populations.tsv exits 0 in silence, which reads as a pass for
111
+ // draft-paper. So a paper that never declared its populations scores
112
112
  // identically to one that declared them and tied every one. Absence is invisible again — the same
113
113
  // shape artifact-coverage.mjs was written to close for the released bundle.
114
114
  //
@@ -4,21 +4,10 @@ description: Answer "what's the status of the paper?" in one pass — MEASURE wh
4
4
  allowed-tools: [Bash, Read, Grep, Glob]
5
5
  ---
6
6
 
7
- <!-- vigiles:sha256:51bc50b2485782e4 compiled from skills/paper-status/SKILL.md.spec.ts -->
7
+ <!-- vigiles:sha256:cdf5e1fcefdb916d compiled from skills/paper-status/SKILL.md.spec.ts -->
8
8
 
9
9
  # paper-status — state, measured before it is narrated
10
10
 
11
- ## Run me
12
-
13
- 🔴 FIRST, before any other step:
14
-
15
- ```
16
- node .claude/skills/paper-pipeline/scripts/announce.mjs paper-status <paper-dir>
17
- ```
18
-
19
- An advisory pass cannot be observed failing — silence is both its error state and its normal
20
- state — so starting is an event, and events get written down.
21
-
22
11
  The author asks "what's the status there?" constantly, and the answer keeps coming from the wrong place: my
23
12
  memory of the session, which drifts, or a 200-line scorecard he cannot skim. Both fail the same way —
24
13
  **they narrate state instead of measuring it.**
@@ -49,18 +38,13 @@ So on any turn that touched a paper, the reply ends with **three lines, unprompt
49
38
  Three failures make this a rule rather than a preference, all in one session:
50
39
  - I finished a page-fit cut and stopped, so he asked what the status was;
51
40
  - I left artifact hosting on his side of the split for a day and a half while the credential to clear
52
- it sat in the environment (see `credential-available` in `pipeline-check.mjs`);
41
+ it sat in the environment;
53
42
  - three consecutive panels returned Weak Accept and the ceiling they named was never the next task,
54
43
  because nothing turned "here is the score" into "here is what to do about it".
55
44
 
56
45
  **Never end a paper turn on a completed sub-task alone.** "Fits in 8 pages" is a measurement, not a
57
46
  status; the status is what that measurement makes possible next.
58
47
 
59
- **Enforcement leg.** `pipeline-check.mjs` prints its banner **unconditionally**, including when it has
60
- zero findings — deadline countdown, verdict row, and `➡️ NEXT`. It runs from `paper-status-gates.sh`
61
- on every edit to a paper source, so the state arrives without anyone pulling it. If that banner and
62
- this section ever disagree, the banner is a measurement and this is prose: believe the banner.
63
-
64
48
  ## How to run it
65
49
 
66
50
  ### 1. Measure (no reading yet)
@@ -90,8 +74,6 @@ gate's verdict **does not describe the current paper** — say so explicitly, an
90
74
  An accept-probability from a superseded draft is not evidence about this one; carrying it forward is
91
75
  the single most common way a status report lies.
92
76
 
93
- If the project ships a checker (`.claude/skills/paper-pipeline/scripts/pipeline-check.mjs`), run it.
94
-
95
77
  ### 3. Read the judgement rows — from BOTH checklists, they answer different questions
96
78
 
97
79
  The project already carries the checklists this skill should report against. Do not invent a new list
@@ -126,28 +108,6 @@ Short. He is asking because he wants to decide what to do next, not to read a re
126
108
  5. **What moved since he last asked**, three lines maximum.
127
109
  6. **The deadline, in his timezone**, and what is on the critical path to it.
128
110
 
129
- ## Record the verdict
130
-
131
- 🔴 LAST step, once the deliverable exists:
132
-
133
- ```
134
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record paper-status <paper-dir> FINDING <count> <report-path>
135
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record paper-status <paper-dir> ABSTAINED <reason> "<one line>"
136
- ```
137
-
138
- This skill reports state and does not judge the paper, so what it records is about the **status run**:
139
- **FINDING** — `<count>` blockers printed, `<report-path>` the status output.
140
- **ABSTAINED** — `no-witness`: the status run measured everything it can and printed zero blockers.
141
- `crashed`: the build did not run, so nothing was measured.
142
-
143
- 🔴 `crashed` is the one outcome that must never be narrated as a status. This skill exists because
144
- state was being narrated instead of measured; a status report built on a build that did not happen
145
- is that same failure with a fresh timestamp on it. Under the old vocabulary this was `ERROR`, which
146
- sat in the same column as a verdict about the paper — it does not any more.
147
-
148
- 🔴 **There is no PASS**, so "zero blockers" cannot be stored. Print it, and let the reader draw the
149
- conclusion from an empty blocker list they can see.
150
-
151
111
  ## Rules
152
112
 
153
113
  - **Never report a gate as run because I intended to run it.** Predicting a background agent's result,
@@ -14,17 +14,6 @@ export default experimental_skill({
14
14
  body: `
15
15
  # paper-status — state, measured before it is narrated
16
16
 
17
- ## Run me
18
-
19
- 🔴 FIRST, before any other step:
20
-
21
- \`\`\`
22
- node .claude/skills/paper-pipeline/scripts/announce.mjs paper-status <paper-dir>
23
- \`\`\`
24
-
25
- An advisory pass cannot be observed failing — silence is both its error state and its normal
26
- state — so starting is an event, and events get written down.
27
-
28
17
  The author asks "what's the status there?" constantly, and the answer keeps coming from the wrong place: my
29
18
  memory of the session, which drifts, or a 200-line scorecard he cannot skim. Both fail the same way —
30
19
  **they narrate state instead of measuring it.**
@@ -55,18 +44,13 @@ So on any turn that touched a paper, the reply ends with **three lines, unprompt
55
44
  Three failures make this a rule rather than a preference, all in one session:
56
45
  - I finished a page-fit cut and stopped, so he asked what the status was;
57
46
  - I left artifact hosting on his side of the split for a day and a half while the credential to clear
58
- it sat in the environment (see \`credential-available\` in \`pipeline-check.mjs\`);
47
+ it sat in the environment;
59
48
  - three consecutive panels returned Weak Accept and the ceiling they named was never the next task,
60
49
  because nothing turned "here is the score" into "here is what to do about it".
61
50
 
62
51
  **Never end a paper turn on a completed sub-task alone.** "Fits in 8 pages" is a measurement, not a
63
52
  status; the status is what that measurement makes possible next.
64
53
 
65
- **Enforcement leg.** \`pipeline-check.mjs\` prints its banner **unconditionally**, including when it has
66
- zero findings — deadline countdown, verdict row, and \`➡️ NEXT\`. It runs from \`paper-status-gates.sh\`
67
- on every edit to a paper source, so the state arrives without anyone pulling it. If that banner and
68
- this section ever disagree, the banner is a measurement and this is prose: believe the banner.
69
-
70
54
  ## How to run it
71
55
 
72
56
  ### 1. Measure (no reading yet)
@@ -96,8 +80,6 @@ gate's verdict **does not describe the current paper** — say so explicitly, an
96
80
  An accept-probability from a superseded draft is not evidence about this one; carrying it forward is
97
81
  the single most common way a status report lies.
98
82
 
99
- If the project ships a checker (\`.claude/skills/paper-pipeline/scripts/pipeline-check.mjs\`), run it.
100
-
101
83
  ### 3. Read the judgement rows — from BOTH checklists, they answer different questions
102
84
 
103
85
  The project already carries the checklists this skill should report against. Do not invent a new list
@@ -132,28 +114,6 @@ Short. He is asking because he wants to decide what to do next, not to read a re
132
114
  5. **What moved since he last asked**, three lines maximum.
133
115
  6. **The deadline, in his timezone**, and what is on the critical path to it.
134
116
 
135
- ## Record the verdict
136
-
137
- 🔴 LAST step, once the deliverable exists:
138
-
139
- \`\`\`
140
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record paper-status <paper-dir> FINDING <count> <report-path>
141
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record paper-status <paper-dir> ABSTAINED <reason> "<one line>"
142
- \`\`\`
143
-
144
- This skill reports state and does not judge the paper, so what it records is about the **status run**:
145
- **FINDING** — \`<count>\` blockers printed, \`<report-path>\` the status output.
146
- **ABSTAINED** — \`no-witness\`: the status run measured everything it can and printed zero blockers.
147
- \`crashed\`: the build did not run, so nothing was measured.
148
-
149
- 🔴 \`crashed\` is the one outcome that must never be narrated as a status. This skill exists because
150
- state was being narrated instead of measured; a status report built on a build that did not happen
151
- is that same failure with a fresh timestamp on it. Under the old vocabulary this was \`ERROR\`, which
152
- sat in the same column as a verdict about the paper — it does not any more.
153
-
154
- 🔴 **There is no PASS**, so "zero blockers" cannot be stored. Print it, and let the reader draw the
155
- conclusion from an empty blocker list they can see.
156
-
157
117
  ## Rules
158
118
 
159
119
  - **Never report a gate as run because I intended to run it.** Predicting a background agent's result,
@@ -4,7 +4,7 @@ description: Use when asking "what would the program committee decide?" / "simul
4
4
  allowed-tools: [Read, Write, Grep, Glob, Bash, WebSearch, WebFetch, Agent, Skill]
5
5
  ---
6
6
 
7
- <!-- vigiles:sha256:9622701a349ea249 compiled from skills/pc-panel-review/SKILL.md.spec.ts -->
7
+ <!-- vigiles:sha256:b9feb099347291ff compiled from skills/pc-panel-review/SKILL.md.spec.ts -->
8
8
 
9
9
  # pc-panel-review — model the whole PC, not one reviewer
10
10
 
@@ -15,17 +15,6 @@ allowed-tools: [Read, Write, Grep, Glob, Bash, WebSearch, WebFetch, Agent, Skill
15
15
  > single-lens spot-check (formerly the standalone `venue-review-sim` skill). Reach for Venue-fit mode or
16
16
  > `paper-adversarial-review` for a fast spot-check; run the full panel (this) for the decision.
17
17
 
18
- ## Run me
19
-
20
- 🔴 FIRST, before any other step:
21
-
22
- ```
23
- node .claude/skills/paper-pipeline/scripts/announce.mjs pc-panel-review <paper-dir>
24
- ```
25
-
26
- An advisory pass cannot be observed failing — silence is both its error state and its normal
27
- state — so starting is an event, and events get written down.
28
-
29
18
  The Venue-fit mode gives ONE reviewer's fit score; `paper-adversarial-review` gives ONE defect hunt.
30
19
  A real decision is made by **2–4 reviewers with different priorities + a meta-review** that weighs
31
20
  consensus over any single voice. This skill runs that. The payoff over a single review: a **must-fix
@@ -61,8 +50,7 @@ prose-only read can.
61
50
  (proven: ~88% accept on a paper the human reader found exhausting). These two artifacts are the
62
51
  panel's only legitimate source for the readability/structure signal.
63
52
  *(Scorecard: this gate is row **`panel`**, and it declares **`structure`** (tighten-paper) + **`writing`**
64
- (grade-paper-writing) as its required inputs — `pipeline-check.mjs` reports `panel` marked done while
65
- either input is missing or newer than it.)*
53
+ (grade-paper-writing) as its required inputs.)*
66
54
  3. **Pick 3 (or 4) reviewer lenses** that a real PC for THIS venue would field. Menu — choose by fit:
67
55
  - **Methods/stats skeptic** — sample size, construct validity, multiple comparisons, overclaims,
68
56
  whether the CIs support the claims.
@@ -145,38 +133,6 @@ the claim falsifiable: *TEXT* invites "then write it", and *IMMOVABLE* invites "
145
133
  🔴 **A deadline is a constraint on how much you rework, never an argument for the current form.** If
146
134
  the author invokes it against a TEXT prong, that is the failure this section exists to catch.
147
135
 
148
- **Enforced** by `../paper-pipeline/scripts/pipeline-check.mjs` → `ceiling-unplanned`: a ceiling recorded
149
- in the scorecard whose prongs carry none of the three labels is a finding.
150
-
151
- ## Record the verdict
152
-
153
- 🔴 LAST step, once the deliverable exists:
154
-
155
- ```
156
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record pc-panel-review <paper-dir> FINDING <count> <report-path>
157
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record pc-panel-review <paper-dir> ABSTAINED <reason> "<one line>"
158
- ```
159
-
160
- **FINDING** — `<count>` is the number of **consensus** must-fixes (flagged by ≥2 independent
161
- reviewers, not the union of everything anyone said), `<report-path>` is the meta-review. Add
162
- `--blocking` when the chair's decision is reject.
163
- **ABSTAINED** — `blocked`: the panel never ran because its required inputs (the persona stall
164
- inventory, the structural verdict) do not exist. `no-witness`: the panel ran and reached no
165
- consensus must-fix.
166
-
167
- 🔴 **THIS IS THE SKILL THE DELETION OF `PASS` WAS WRITTEN FOR.** Five model reviewers from one
168
- vendor agreeing was being recorded as an acquittal — and correlated agreement between instances of
169
- one model is not independent evidence of anything. There is now no constructor that can say it.
170
- "Accept with no must-fix" is `ABSTAINED no-witness`: the panel produced no finding, which is a fact
171
- about the panel and not a fact about the paper.
172
-
173
- 🔴 **The `blocked` row is the one that matters most here.** A blocked panel and a panel nobody
174
- launched read identically in prose, and on 2026-08-04 a status line said the panel was running when
175
- it had never been launched at all. Record the block; do not leave the check silent.
176
-
177
- Venue-fit mode records under the same skill name with the fit verdict and `<count>` 0 — it is one
178
- reviewer's spot-check, and its row should not be mistaken for the panel's decision.
179
-
180
136
  ## Rules
181
137
  - **Independence is non-negotiable** — never let reviewers see each other's reviews before the
182
138
  meta-review; shared context collapses the panel into one voice.
@@ -31,17 +31,6 @@ export default experimental_skill({
31
31
  > single-lens spot-check (formerly the standalone \`venue-review-sim\` skill). Reach for Venue-fit mode or
32
32
  > \`paper-adversarial-review\` for a fast spot-check; run the full panel (this) for the decision.
33
33
 
34
- ## Run me
35
-
36
- 🔴 FIRST, before any other step:
37
-
38
- \`\`\`
39
- node .claude/skills/paper-pipeline/scripts/announce.mjs pc-panel-review <paper-dir>
40
- \`\`\`
41
-
42
- An advisory pass cannot be observed failing — silence is both its error state and its normal
43
- state — so starting is an event, and events get written down.
44
-
45
34
  The Venue-fit mode gives ONE reviewer's fit score; \`paper-adversarial-review\` gives ONE defect hunt.
46
35
  A real decision is made by **2–4 reviewers with different priorities + a meta-review** that weighs
47
36
  consensus over any single voice. This skill runs that. The payoff over a single review: a **must-fix
@@ -77,8 +66,7 @@ prose-only read can.
77
66
  (proven: ~88% accept on a paper the human reader found exhausting). These two artifacts are the
78
67
  panel's only legitimate source for the readability/structure signal.
79
68
  *(Scorecard: this gate is row **\`panel\`**, and it declares **\`structure\`** (tighten-paper) + **\`writing\`**
80
- (grade-paper-writing) as its required inputs — \`pipeline-check.mjs\` reports \`panel\` marked done while
81
- either input is missing or newer than it.)*
69
+ (grade-paper-writing) as its required inputs.)*
82
70
  3. **Pick 3 (or 4) reviewer lenses** that a real PC for THIS venue would field. Menu — choose by fit:
83
71
  - **Methods/stats skeptic** — sample size, construct validity, multiple comparisons, overclaims,
84
72
  whether the CIs support the claims.
@@ -161,38 +149,6 @@ the claim falsifiable: *TEXT* invites "then write it", and *IMMOVABLE* invites "
161
149
  🔴 **A deadline is a constraint on how much you rework, never an argument for the current form.** If
162
150
  the author invokes it against a TEXT prong, that is the failure this section exists to catch.
163
151
 
164
- **Enforced** by \`../paper-pipeline/scripts/pipeline-check.mjs\` → \`ceiling-unplanned\`: a ceiling recorded
165
- in the scorecard whose prongs carry none of the three labels is a finding.
166
-
167
- ## Record the verdict
168
-
169
- 🔴 LAST step, once the deliverable exists:
170
-
171
- \`\`\`
172
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record pc-panel-review <paper-dir> FINDING <count> <report-path>
173
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record pc-panel-review <paper-dir> ABSTAINED <reason> "<one line>"
174
- \`\`\`
175
-
176
- **FINDING** — \`<count>\` is the number of **consensus** must-fixes (flagged by ≥2 independent
177
- reviewers, not the union of everything anyone said), \`<report-path>\` is the meta-review. Add
178
- \`--blocking\` when the chair's decision is reject.
179
- **ABSTAINED** — \`blocked\`: the panel never ran because its required inputs (the persona stall
180
- inventory, the structural verdict) do not exist. \`no-witness\`: the panel ran and reached no
181
- consensus must-fix.
182
-
183
- 🔴 **THIS IS THE SKILL THE DELETION OF \`PASS\` WAS WRITTEN FOR.** Five model reviewers from one
184
- vendor agreeing was being recorded as an acquittal — and correlated agreement between instances of
185
- one model is not independent evidence of anything. There is now no constructor that can say it.
186
- "Accept with no must-fix" is \`ABSTAINED no-witness\`: the panel produced no finding, which is a fact
187
- about the panel and not a fact about the paper.
188
-
189
- 🔴 **The \`blocked\` row is the one that matters most here.** A blocked panel and a panel nobody
190
- launched read identically in prose, and on 2026-08-04 a status line said the panel was running when
191
- it had never been launched at all. Record the block; do not leave the check silent.
192
-
193
- Venue-fit mode records under the same skill name with the fit verdict and \`<count>\` 0 — it is one
194
- reviewer's spot-check, and its row should not be mistaken for the panel's decision.
195
-
196
152
  ## Rules
197
153
  - **Independence is non-negotiable** — never let reviewers see each other's reviews before the
198
154
  meta-review; shared context collapses the panel into one voice.
@@ -1,24 +1,13 @@
1
1
  ---
2
2
  name: plan-paper-timeline
3
3
  description: Turn a venue's CFP dates into a scheduled, buffered plan on the Google Calendar — build the timeline backwards from the AoE submission deadline (harden + panel-review + artifact + anonymized host all land BEFORE it), create calendar events in the author's own time zone for submit-day / deadline / camera-ready, and apply the FILING-SPACING rule so multiple papers don't cluster in the month before a planned external filing. Use right after find-venue hands you real deadlines and before you start drafting. Compose with find-venue (deadline source) and submit-paper.
4
- allowed-tools: [Read, Write, Grep, Glob, mcp__Google_Calendar__list_events, mcp__Google_Calendar__search_events, mcp__Google_Calendar__create_event, mcp__Google_Calendar__update_event, Bash(node .claude/skills/paper-pipeline/scripts/announce.mjs:*), Bash(node .claude/skills/paper-pipeline/scripts/ledger.mjs:*)]
4
+ allowed-tools: [Read, Write, Grep, Glob, mcp__Google_Calendar__list_events, mcp__Google_Calendar__search_events, mcp__Google_Calendar__create_event, mcp__Google_Calendar__update_event]
5
5
  ---
6
6
 
7
- <!-- vigiles:sha256:faf0c175f1a0c5e5 compiled from skills/plan-paper-timeline/SKILL.md.spec.ts -->
7
+ <!-- vigiles:sha256:0f95e8225033cd72 compiled from skills/plan-paper-timeline/SKILL.md.spec.ts -->
8
8
 
9
9
  # plan-paper-timeline — CFP dates → a buffered, filing-aware schedule
10
10
 
11
- ## Run me
12
-
13
- 🔴 FIRST, before any other step:
14
-
15
- ```
16
- node .claude/skills/paper-pipeline/scripts/announce.mjs plan-paper-timeline <paper-dir>
17
- ```
18
-
19
- An advisory pass cannot be observed failing — silence is both its error state and its normal
20
- state — so starting is an event, and events get written down.
21
-
22
11
  A deadline is not a plan. This skill converts the four dates every CFP gives you — **submission (AoE),
23
12
  notification, camera-ready, event** — into calendar events with the buffers that actually keep a
24
13
  submission from slipping, and sequences *multiple* papers so the cadence reads as sustained acclaim,
@@ -58,7 +47,6 @@ institutional email has no expedite route** — no support queue jump, no "I'm o
58
47
  That is why `access` is scheduled in the first week rather than the last: it is the one item where the
59
48
  only remedy for being late is having started earlier.
60
49
 
61
- `pipeline-check.mjs` reports an `access` row still `☐` inside the moderation window as a finding, and
62
50
  `submit-paper` is hard-blocked on `access` being green.
63
51
 
64
52
  ## 1. Build the timeline backwards from the AoE deadline
@@ -142,28 +130,6 @@ When scheduling paper N against papers already on the calendar and a known/likel
142
130
  second, stronger publication lands in a later month on purpose.
143
131
  - Prefer a paper with an earlier, safely-past deadline over one that would pile onto the filing month.
144
132
 
145
- ## 5. Record the verdict
146
-
147
- 🔴 LAST step, once the deliverable exists:
148
-
149
- ```
150
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record plan-paper-timeline <paper-dir> FINDING <count> <report-path>
151
- node .claude/skills/paper-pipeline/scripts/ledger.mjs record plan-paper-timeline <paper-dir> ABSTAINED <reason> "<one line>"
152
- ```
153
-
154
- **FINDING** — `<count>` slots have no buffer, or collide with another paper's pre-filing window;
155
- `<report-path>` is the plan. Add `--blocking` when the deadline cannot be met from here.
156
- **ABSTAINED** — `no-witness`: the backwards plan is on the calendar, `access` is green and no slot
157
- is tight. `input-missing`: `find-venue` has produced no dated deadline to plan against.
158
-
159
- This is the one skill whose blocking finding is a fact about the calendar rather than about the
160
- paper: portal moderation runs up to two weeks and an author with no institutional email has no
161
- expedite route, so one recorded today is still true tomorrow no matter how good the science gets.
162
-
163
- 🔴 **There is no PASS**, so "the plan is on the calendar" is not storable here. Put the event ids in
164
- the `ABSTAINED` note — markdown is the source of truth and the calendar is its projection, and a row
165
- with neither is a claim about a calendar nobody opened.
166
-
167
133
  ## Scorecard rows this skill owns
168
134
  Two, and they fail differently — see `paper-pipeline/references/pipeline-status-template.md`:
169
135
  - **`access`** — *can you physically submit?* (§0). Account · profile ACTIVE since `<ISO>` · portal