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.
- package/CLAUDE.md +2 -2
- package/dist/adapters/banal/failure.d.ts +1 -7
- package/dist/adapters/banal/failure.d.ts.map +1 -1
- package/dist/adapters/banal/failure.js +5 -7
- package/dist/adapters/banal/failure.js.map +1 -1
- package/dist/adapters/banal/index.d.ts.map +1 -1
- package/dist/adapters/banal/index.js +8 -11
- package/dist/adapters/banal/index.js.map +1 -1
- package/dist/adapters/banal/install.d.ts.map +1 -1
- package/dist/adapters/banal/install.js +4 -3
- package/dist/adapters/banal/install.js.map +1 -1
- package/dist/adapters/banal/invocation.d.ts +9 -4
- package/dist/adapters/banal/invocation.d.ts.map +1 -1
- package/dist/adapters/banal/invocation.js +13 -1
- package/dist/adapters/banal/invocation.js.map +1 -1
- package/dist/adapters/banal/locate.d.ts +12 -9
- package/dist/adapters/banal/locate.d.ts.map +1 -1
- package/dist/adapters/banal/locate.js +19 -12
- package/dist/adapters/banal/locate.js.map +1 -1
- package/dist/adapters/banal/run.d.ts +10 -1
- package/dist/adapters/banal/run.d.ts.map +1 -1
- package/dist/adapters/banal/run.js +8 -4
- package/dist/adapters/banal/run.js.map +1 -1
- package/dist/adapters/banal/settings.d.ts +1 -1
- package/dist/adapters/banal/settings.d.ts.map +1 -1
- package/dist/adapters/banal/settings.js +2 -1
- package/dist/adapters/banal/settings.js.map +1 -1
- package/dist/adapters/memory/index.d.ts.map +1 -1
- package/dist/adapters/memory/index.js +8 -2
- package/dist/adapters/memory/index.js.map +1 -1
- package/dist/adapters/node/process.io.d.ts +6 -3
- package/dist/adapters/node/process.io.d.ts.map +1 -1
- package/dist/adapters/node/process.io.js.map +1 -1
- package/dist/adapters/node/workspace.io.d.ts.map +1 -1
- package/dist/adapters/node/workspace.io.js +4 -2
- package/dist/adapters/node/workspace.io.js.map +1 -1
- package/dist/adapters/references/index.d.ts.map +1 -1
- package/dist/adapters/references/index.js +4 -2
- package/dist/adapters/references/index.js.map +1 -1
- package/dist/build-engine.d.ts +1 -1
- package/dist/build-engine.d.ts.map +1 -1
- package/dist/build-engine.js +8 -1
- package/dist/build-engine.js.map +1 -1
- package/dist/build.d.ts +2 -3
- package/dist/build.d.ts.map +1 -1
- package/dist/build.js +5 -6
- package/dist/build.js.map +1 -1
- package/dist/caller-path.d.ts +3 -0
- package/dist/caller-path.d.ts.map +1 -0
- package/dist/caller-path.js +13 -0
- package/dist/caller-path.js.map +1 -0
- package/dist/cli.d.ts +10 -17
- package/dist/cli.d.ts.map +1 -1
- package/dist/cli.js +62 -12
- package/dist/cli.js.map +1 -1
- package/dist/doctor.d.ts +12 -3
- package/dist/doctor.d.ts.map +1 -1
- package/dist/doctor.js +2 -1
- package/dist/doctor.js.map +1 -1
- package/dist/domain/paths.d.ts +8 -2
- package/dist/domain/paths.d.ts.map +1 -1
- package/dist/domain/paths.js +12 -1
- package/dist/domain/paths.js.map +1 -1
- package/dist/domain/ready.d.ts +24 -0
- package/dist/domain/ready.d.ts.map +1 -0
- package/dist/domain/ready.js +11 -0
- package/dist/domain/ready.js.map +1 -0
- package/dist/domain/record.d.ts +8 -0
- package/dist/domain/record.d.ts.map +1 -0
- package/dist/domain/record.js +8 -0
- package/dist/domain/record.js.map +1 -0
- package/dist/domain/sha256.d.ts.map +1 -1
- package/dist/domain/sha256.js +4 -2
- package/dist/domain/sha256.js.map +1 -1
- package/dist/engine.d.ts +23 -22
- package/dist/engine.d.ts.map +1 -1
- package/dist/engine.js.map +1 -1
- package/dist/eslint-rules/bib-reachable-entry.d.ts +1 -1
- package/dist/eslint-rules/bib-reachable-entry.js.map +1 -1
- package/dist/eslint-rules/install-path-literals.d.ts +2 -2
- package/dist/eslint-rules/install-path-literals.js.map +1 -1
- package/dist/eslint-rules/paper-research-question.d.ts +1 -1
- package/dist/eslint-rules/paper-research-question.js.map +1 -1
- package/dist/eslint-rules/paper-stages.d.ts +2 -2
- package/dist/eslint-rules/paper-stages.js.map +1 -1
- package/dist/eslint-rules/paper-typography.d.ts +6 -6
- package/dist/eslint-rules/paper-typography.js.map +1 -1
- package/dist/eslint-rules/pdf-last-page-balance.d.ts +1 -1
- package/dist/eslint-rules/pdf-last-page-balance.js.map +1 -1
- package/dist/eslint-rules/review-frontmatter.d.ts +2 -2
- package/dist/eslint-rules/review-frontmatter.js.map +1 -1
- package/dist/eslint-rules/sibling-frontmatter.d.ts +1 -1
- package/dist/eslint-rules/sibling-frontmatter.d.ts.map +1 -1
- package/dist/eslint-rules/temp-root-realpath.d.ts +1 -1
- package/dist/eslint-rules/temp-root-realpath.js.map +1 -1
- package/dist/eslint-rules/tex-build.d.ts +2 -2
- package/dist/eslint-rules/tex-build.js.map +1 -1
- package/dist/facts-file.d.ts.map +1 -1
- package/dist/facts-file.js +29 -26
- package/dist/facts-file.js.map +1 -1
- package/dist/hooks-settings.d.ts.map +1 -1
- package/dist/hooks-settings.js +17 -2
- package/dist/hooks-settings.js.map +1 -1
- package/dist/init.d.ts +3 -4
- package/dist/init.d.ts.map +1 -1
- package/dist/init.js +18 -8
- package/dist/init.js.map +1 -1
- package/dist/lib/paper-config.d.ts.map +1 -1
- package/dist/lib/paper-config.js +1 -5
- package/dist/lib/paper-config.js.map +1 -1
- package/dist/new-paper.d.ts.map +1 -1
- package/dist/new-paper.js +5 -1
- package/dist/new-paper.js.map +1 -1
- package/dist/paper-files.d.ts +4 -4
- package/dist/paper-files.d.ts.map +1 -1
- package/dist/paper-files.js +6 -6
- package/dist/paper-files.js.map +1 -1
- package/dist/paper-settings.js +3 -3
- package/dist/paper-settings.js.map +1 -1
- package/dist/pdf-facts.d.ts +51 -1
- package/dist/pdf-facts.d.ts.map +1 -1
- package/dist/pdf-facts.js +23 -15
- package/dist/pdf-facts.js.map +1 -1
- package/dist/pdf-geometry.d.ts.map +1 -1
- package/dist/pdf-geometry.js +2 -3
- package/dist/pdf-geometry.js.map +1 -1
- package/dist/ports/tool-installer.d.ts +2 -14
- package/dist/ports/tool-installer.d.ts.map +1 -1
- package/dist/presets.d.ts.map +1 -1
- package/dist/presets.js +16 -4
- package/dist/presets.js.map +1 -1
- package/dist/references.d.ts.map +1 -1
- package/dist/references.js +4 -4
- package/dist/references.js.map +1 -1
- package/dist/rules-config.d.ts +26 -1
- package/dist/rules-config.d.ts.map +1 -1
- package/dist/rules-config.js +9 -9
- package/dist/rules-config.js.map +1 -1
- package/dist/tex-requirements.d.ts +4 -0
- package/dist/tex-requirements.d.ts.map +1 -1
- package/dist/tex-requirements.js +14 -2
- package/dist/tex-requirements.js.map +1 -1
- package/dist/types.d.ts +5 -3
- package/dist/types.d.ts.map +1 -1
- package/dist/venue-rules.d.ts.map +1 -1
- package/dist/venue-rules.js +20 -7
- package/dist/venue-rules.js.map +1 -1
- package/docs/configuration.md +2 -2
- package/eslint-rules/bib-reachable-entry.ts +1 -1
- package/eslint-rules/install-path-literals.ts +2 -2
- package/eslint-rules/paper-research-question.ts +1 -1
- package/eslint-rules/paper-stages.ts +2 -2
- package/eslint-rules/paper-typography.ts +2 -2
- package/eslint-rules/pdf-last-page-balance.ts +1 -1
- package/eslint-rules/review-frontmatter.ts +1 -1
- package/eslint-rules/temp-root-realpath.ts +1 -1
- package/eslint-rules/tex-build.ts +2 -2
- package/eslint.config.mjs +7 -0
- package/fixtures/install-path-literals/clean.fixture.mjs +1 -1
- package/fixtures/install-path-literals/clean.md +2 -2
- package/fixtures/install-path-literals/defect.fixture.mjs +1 -1
- package/fixtures/install-path-literals/defect.md +2 -2
- package/hooks/paper-status-gates.sh +9 -19
- package/lib/paper-config.harness.mjs +1 -1
- package/lib/paper-config.ts +1 -5
- package/lib/skill-checks.mjs +24 -381
- package/lib/skill-checks.test.mjs +63 -115
- package/lib/skill-corpus.mjs +10 -251
- package/lib/skill-corpus.test.mjs +2 -13
- package/lib/skill-trigger-cases.mjs +1 -1
- package/package.json +1 -1
- package/scripts/coverage-config.test.ts +0 -1
- package/scripts/harness-api.frozen.json +0 -1
- package/skills/analyze-sibling-paper/SKILL.md +1 -39
- package/skills/analyze-sibling-paper/SKILL.md.spec.ts +0 -38
- package/skills/argument-arc/SKILL.md +2 -36
- package/skills/argument-arc/SKILL.md.spec.ts +1 -43
- package/skills/build-benchmark/SKILL.md +1 -41
- package/skills/build-benchmark/SKILL.md.spec.ts +0 -40
- package/skills/camera-ready/SKILL.md +1 -34
- package/skills/camera-ready/SKILL.md.spec.ts +0 -33
- package/skills/cold-read-diff/SKILL.md +1 -35
- package/skills/cold-read-diff/SKILL.md.spec.ts +0 -34
- package/skills/draft-paper/SKILL.md +2 -41
- package/skills/draft-paper/SKILL.md.spec.ts +1 -50
- package/skills/extend-paper/SKILL.md +2 -33
- package/skills/extend-paper/SKILL.md.spec.ts +1 -42
- package/skills/find-venue/SKILL.md +2 -38
- package/skills/find-venue/SKILL.md.spec.ts +1 -47
- package/skills/grade-paper-writing/SKILL.md +3 -38
- package/skills/grade-paper-writing/SKILL.md.spec.ts +2 -47
- package/skills/grade-paper-writing/prose-lint.mjs +2 -3
- package/skills/grade-paper-writing/prose-lint.test.ts +1 -1
- package/skills/harden-paper/SKILL.md +1 -39
- package/skills/harden-paper/SKILL.md.spec.ts +0 -38
- package/skills/map-prior-work/SKILL.md +1 -35
- package/skills/map-prior-work/SKILL.md.spec.ts +0 -34
- package/skills/osf-artifact-upload/osf-artifact-upload.harness.ts +3 -4
- package/skills/paper-adversarial-review/SKILL.md +2 -35
- package/skills/paper-adversarial-review/SKILL.md.spec.ts +1 -43
- package/skills/paper-pipeline/SKILL.md +2 -28
- package/skills/paper-pipeline/SKILL.md.spec.ts +1 -27
- package/skills/paper-pipeline/grade-paper-writing-ablation.eval.mjs +1 -1
- package/skills/paper-pipeline/paper-pipeline.harness.mjs +25 -40
- package/skills/paper-pipeline/references/acceptance-gate.md +1 -6
- package/skills/paper-pipeline/references/pipeline-status-template.md +8 -35
- package/skills/paper-pipeline/scripts/README.md +30 -225
- package/skills/paper-pipeline/scripts/artifact-coverage.harness.mjs +4 -8
- package/skills/paper-pipeline/scripts/consumer.harness.mjs +6 -104
- package/skills/paper-pipeline/scripts/consumer.mjs +23 -104
- package/skills/paper-pipeline/scripts/generated-code.harness.mjs +2 -3
- package/skills/paper-pipeline/scripts/population-map.harness.mjs +3 -3
- package/skills/paper-status/SKILL.md +2 -42
- package/skills/paper-status/SKILL.md.spec.ts +1 -41
- package/skills/pc-panel-review/SKILL.md +2 -46
- package/skills/pc-panel-review/SKILL.md.spec.ts +1 -45
- package/skills/plan-paper-timeline/SKILL.md +2 -36
- package/skills/plan-paper-timeline/SKILL.md.spec.ts +0 -36
- package/skills/render-paper/SKILL.md +1 -37
- package/skills/render-paper/SKILL.md.spec.ts +0 -36
- package/skills/research-ideate/SKILL.md +2 -39
- package/skills/research-ideate/SKILL.md.spec.ts +1 -47
- package/skills/study-accepted-papers/SKILL.md +1 -34
- package/skills/study-accepted-papers/SKILL.md.spec.ts +0 -33
- package/skills/submit-paper/SKILL.md +1 -33
- package/skills/submit-paper/SKILL.md.spec.ts +0 -32
- package/skills/sweep-design-space/SKILL.md +1 -38
- package/skills/sweep-design-space/SKILL.md.spec.ts +0 -37
- package/skills/tighten-paper/SKILL.md +2 -39
- package/skills/tighten-paper/SKILL.md.spec.ts +1 -47
- package/skills/tighten-paper/structure.mjs +4 -11
- package/skills/verify-citations/SKILL.md +2 -38
- package/skills/verify-citations/SKILL.md.spec.ts +1 -37
- package/src/adapters/banal/failure.ts +9 -10
- package/src/adapters/banal/index.test.ts +2 -2
- package/src/adapters/banal/index.ts +8 -10
- package/src/adapters/banal/install.ts +9 -4
- package/src/adapters/banal/invocation.test.ts +20 -8
- package/src/adapters/banal/invocation.ts +23 -8
- package/src/adapters/banal/locate.test.ts +2 -2
- package/src/adapters/banal/locate.ts +27 -21
- package/src/adapters/banal/run.test.ts +2 -2
- package/src/adapters/banal/run.ts +16 -10
- package/src/adapters/banal/settings.ts +2 -2
- package/src/adapters/curl/download.test.ts +4 -4
- package/src/adapters/memory/index.ts +4 -3
- package/src/adapters/node/files.test.ts +5 -3
- package/src/adapters/node/process.io.ts +9 -2
- package/src/adapters/node/process.test.ts +4 -4
- package/src/adapters/node/workspace.io.ts +9 -4
- package/src/adapters/references/index.ts +7 -2
- package/src/build-engine.test.ts +11 -6
- package/src/build-engine.ts +10 -5
- package/src/build.test.ts +10 -11
- package/src/build.ts +8 -9
- package/src/caller-path.ts +13 -0
- package/src/cli.harness.mjs +0 -2
- package/src/cli.test.ts +19 -8
- package/src/cli.ts +82 -25
- package/src/doctor.test.ts +1 -1
- package/src/doctor.ts +14 -5
- package/src/domain/paths.test.ts +15 -0
- package/src/domain/paths.ts +20 -1
- package/src/domain/ready.ts +31 -0
- package/src/domain/record.test.ts +20 -0
- package/src/domain/record.ts +12 -0
- package/src/domain/sha256.ts +5 -3
- package/src/engine.ts +30 -2
- package/src/facts-file.ts +43 -28
- package/src/hooks-settings.harness.ts +17 -0
- package/src/hooks-settings.ts +28 -4
- package/src/init.test.ts +25 -5
- package/src/init.ts +26 -12
- package/src/new-paper.harness.mjs +7 -23
- package/src/new-paper.ts +7 -3
- package/src/paper-files.test.ts +22 -19
- package/src/paper-files.ts +16 -27
- package/src/paper-settings.test.ts +2 -2
- package/src/paper-settings.ts +5 -5
- package/src/pdf-facts.test.ts +16 -5
- package/src/pdf-facts.ts +79 -22
- package/src/pdf-geometry.ts +5 -3
- package/src/ports/tool-installer.ts +2 -17
- package/src/presets.ts +28 -14
- package/src/references.test.ts +25 -19
- package/src/references.ts +4 -5
- package/src/rules-config.ts +40 -11
- package/src/tex-requirements.test.ts +21 -0
- package/src/tex-requirements.ts +29 -6
- package/src/toolchain.test.ts +9 -5
- package/src/toolchain.ts +2 -2
- package/src/types.ts +5 -3
- package/src/venue-rules.test.ts +8 -4
- package/src/venue-rules.ts +21 -11
- package/skills/paper-pipeline/scripts/announce.mjs +0 -67
- package/skills/paper-pipeline/scripts/announce.test.mjs +0 -102
- package/skills/paper-pipeline/scripts/fixtures/clean/PIPELINE-STATUS.md +0 -51
- package/skills/paper-pipeline/scripts/fixtures/dirty/PIPELINE-STATUS.md +0 -52
- package/skills/paper-pipeline/scripts/fixtures/dirty/paper.md +0 -6
- package/skills/paper-pipeline/scripts/fixtures/scorecard.ts +0 -100
- package/skills/paper-pipeline/scripts/ledger.cli.test.mjs +0 -302
- package/skills/paper-pipeline/scripts/ledger.mjs +0 -623
- package/skills/paper-pipeline/scripts/ledger.test.mjs +0 -254
- package/skills/paper-pipeline/scripts/pipeline-check.cli.test.mjs +0 -474
- package/skills/paper-pipeline/scripts/pipeline-check.harness.mjs +0 -328
- package/skills/paper-pipeline/scripts/pipeline-check.mjs +0 -750
- package/skills/paper-pipeline/scripts/pipeline-edges.mjs +0 -169
- package/skills/paper-pipeline/scripts/pipeline-edges.test.mjs +0 -73
- package/skills/paper-pipeline/scripts/run-mechanical.mjs +0 -648
- package/skills/paper-pipeline/scripts/run-mechanical.test.mjs +0 -468
- package/skills/paper-pipeline/scripts/status.mjs +0 -295
- 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.
|
|
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
|
-
*
|
|
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
|
-
*
|
|
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.
|
|
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
|
|
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
|
-
*
|
|
52
|
-
*
|
|
53
|
-
*
|
|
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
|
|
250
|
-
*
|
|
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
|
-
//
|
|
233
|
+
// WHERE A SKILL'S PROSE NAMES THESE SCRIPTS
|
|
301
234
|
//
|
|
302
|
-
// `papersRoot()` answers "where does the consumer keep its papers"
|
|
303
|
-
//
|
|
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
|
|
308
|
-
//
|
|
309
|
-
//
|
|
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
|
-
//
|
|
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
|
-
//
|
|
318
|
+
// WHICH WALL CLOCK THE CONSUMER SCHEDULES AGAINST
|
|
389
319
|
//
|
|
390
|
-
// `papersRoot()` answers "where are the papers", `
|
|
391
|
-
// `
|
|
392
|
-
//
|
|
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
|
-
//
|
|
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:
|
|
321
|
-
//
|
|
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
|
|
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,
|
|
111
|
-
//
|
|
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:
|
|
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
|
|
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
|
|
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:
|
|
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
|
|
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
|
|
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
|
|
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:
|
|
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
|