@pareto4/pp-core 1.13.0 → 1.18.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-plugin/plugin.json +3 -3
- package/README.md +31 -12
- package/agents/pp-code-fixer.md +0 -11
- package/agents/pp-code-reviewer.md +1 -77
- package/agents/pp-debug-session-manager.md +9 -6
- package/agents/pp-executor.md +12 -29
- package/agents/pp-plan-checker.md +4 -4
- package/agents/pp-planner.md +34 -20
- package/agents/pp-vault-reviewer.md +100 -23
- package/agents/pp-verifier.md +1 -3
- package/bin/install.js +10 -3
- package/commands/pp-execute-phase.md +1 -3
- package/commands/pp-ns-manage.md +1 -1
- package/commands/pp-ns-review.md +1 -1
- package/commands/pp-plan-phase.md +1 -2
- package/commands/pp-vault-promote.md +7 -8
- package/commands/pp-vault-review.md +6 -5
- package/commands/pp-verify-work.md +1 -2
- package/gemini-extension.json +1 -1
- package/hooks/dist/pp-check-update.js +0 -0
- package/hooks/dist/pp-cursor-session-start.js +1 -1
- package/hooks/dist/pp-statusline.js +1 -1
- package/hooks/dist/pp-update-banner.js +0 -0
- package/hooks/pp-check-update.js +0 -0
- package/hooks/pp-cursor-session-start.js +1 -1
- package/hooks/pp-statusline.js +1 -1
- package/hooks/pp-update-banner.js +0 -0
- package/package.json +10 -18
- package/pareto-skills/_shared/create-project-board.sh +0 -0
- package/pareto-skills/pp-project-setup/SKILL.md +1 -1
- package/pp-core/bin/check-latest-version.cjs +0 -0
- package/pp-core/bin/detect-build-commands.sh +2 -2
- package/pp-core/bin/lib/capability-activation.cjs +83 -9
- package/pp-core/bin/lib/capability-registry.cjs +201 -229
- package/pp-core/bin/lib/capability-state.cjs +93 -1
- package/pp-core/bin/lib/capability-writer.cjs +3 -3
- package/pp-core/bin/lib/check-command-router.cjs +1906 -1514
- package/pp-core/bin/lib/clusters.cjs +3 -0
- package/pp-core/bin/lib/command-aliases.cjs +0 -35
- package/pp-core/bin/lib/commands.cjs +9 -4
- package/pp-core/bin/lib/compose-bindings.cjs +251 -88
- package/pp-core/bin/lib/config-loader.cjs +7 -103
- package/pp-core/bin/lib/config-schema.cjs +1 -1
- package/pp-core/bin/lib/config.cjs +151 -62
- package/pp-core/bin/lib/decisions.cjs +4 -4
- package/pp-core/bin/lib/discovery-vault-check.cjs +246 -0
- package/pp-core/bin/lib/frontmatter.cjs +1 -38
- package/pp-core/bin/lib/git-base-branch.cjs +11 -12
- package/pp-core/bin/lib/git-provider.cjs +9 -10
- package/pp-core/bin/lib/init-command-router.cjs +5 -8
- package/pp-core/bin/lib/init-contract.cjs +620 -738
- package/pp-core/bin/lib/init-interactive.cjs +62 -4
- package/pp-core/bin/lib/init.cjs +24 -51
- package/pp-core/bin/lib/install-profiles.cjs +1 -1
- package/pp-core/bin/lib/io.cjs +4 -2
- package/pp-core/bin/lib/loop-resolver.cjs +165 -129
- package/pp-core/bin/lib/org-profile.cjs +402 -161
- package/pp-core/bin/lib/package-identity.cjs +3 -3
- package/pp-core/bin/lib/phase-command-router.cjs +1 -29
- package/pp-core/bin/lib/phase-scope.cjs +1 -1
- package/pp-core/bin/lib/phase.cjs +11 -253
- package/pp-core/bin/lib/platform-bindings.cjs +40 -19
- package/pp-core/bin/lib/profile-output.cjs +1 -1
- package/pp-core/bin/lib/shell-command-projection.cjs +104 -1
- package/pp-core/bin/lib/state.cjs +11 -85
- package/pp-core/bin/lib/template.cjs +1 -1
- package/pp-core/bin/lib/vault-command-router.cjs +1682 -477
- package/pp-core/bin/lib/vault-coordinates.cjs +16 -7
- package/pp-core/bin/lib/vault-index.cjs +656 -0
- package/pp-core/bin/lib/vault-seal.cjs +1851 -0
- package/pp-core/bin/lib/vault-standards-promote.cjs +577 -0
- package/pp-core/bin/lib/vault-standards-review.cjs +293 -0
- package/pp-core/bin/lib/vault-standards.cjs +1504 -0
- package/pp-core/bin/lib/vault-trace.cjs +155 -2778
- package/pp-core/bin/lib/vault-update.cjs +10 -59
- package/pp-core/bin/lib/vault.cjs +2570 -950
- package/pp-core/bin/lib/verify.cjs +0 -11
- package/pp-core/bin/portable-timeout.sh +1 -1
- package/pp-core/bin/pp-tools.cjs +157 -54
- package/pp-core/bin/shared/config-defaults.manifest.json +0 -7
- package/pp-core/bin/shared/config-schema.manifest.json +0 -3
- package/pp-core/bin/shared/model-catalog.json +2 -2
- package/pp-core/bin/verify-reapply-patches.cjs +0 -0
- package/pp-core/org-profile.json +26 -0
- package/pp-core/references/autonomous-best-effort.md +8 -7
- package/pp-core/references/loop-hook-dispatch.md +0 -25
- package/pp-core/references/planner-gap-closure.md +0 -1
- package/pp-core/references/planner-tdd.md +0 -1
- package/pp-core/references/planner-vault-standards.md +311 -0
- package/pp-core/references/planning-config.md +0 -8
- package/pp-core/references/tdd.md +0 -1
- package/pp-core/templates/README.md +1 -0
- package/pp-core/templates/VAULT-REVIEW.md +56 -0
- package/pp-core/templates/phase-prompt.md +2 -6
- package/pp-core/workflows/_runtime-launcher.snippet.sh +0 -0
- package/pp-core/workflows/add-tests.md +55 -4
- package/pp-core/workflows/audit-fix.md +63 -2
- package/pp-core/workflows/audit-milestone.md +1 -1
- package/pp-core/workflows/autonomous.md +85 -11
- package/pp-core/workflows/check-todos.md +13 -2
- package/pp-core/workflows/code-review-fix.md +59 -3
- package/pp-core/workflows/code-review.md +10 -57
- package/pp-core/workflows/complete-milestone.md +1 -1
- package/pp-core/workflows/debug.md +54 -4
- package/pp-core/workflows/discovery-sprint/prefill.md +19 -7
- package/pp-core/workflows/discovery-sprint/prototype.md +46 -49
- package/pp-core/workflows/discovery-sprint/session-13-gates.md +2 -0
- package/pp-core/workflows/discovery-sprint/sessions/01-kick-off.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/02-as-is.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/03-to-be.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/04-ux-design.md +20 -24
- package/pp-core/workflows/discovery-sprint/sessions/05-architecture.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/06-security.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/07-identity.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/08-integration.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/09-qa.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/10-infrastructure.md +1 -6
- package/pp-core/workflows/discovery-sprint/sessions/11-roadmap.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/12-delivery-model.md +1 -3
- package/pp-core/workflows/discovery-sprint/sessions/13-wrap-up.md +1 -3
- package/pp-core/workflows/discovery-sprint/templates/context-template.md +1 -1
- package/pp-core/workflows/discovery-sprint/vault-context.md +207 -0
- package/pp-core/workflows/discovery-sprint.md +7 -3
- package/pp-core/workflows/discovery-to-build/steps/architecture-deliverables.md +4 -4
- package/pp-core/workflows/discovery-to-build/templates/pp-handoff.md +43 -1
- package/pp-core/workflows/discovery-to-build.md +10 -2
- package/pp-core/workflows/discuss-phase.md +3 -10
- package/pp-core/workflows/execute-phase/steps/check-query-gate.md +327 -0
- package/pp-core/workflows/execute-phase/steps/client-context-gate.md +3 -2
- package/pp-core/workflows/execute-phase/steps/cross-ai-delegation.md +1 -1
- package/pp-core/workflows/execute-phase/steps/execute-pre-dispatch.md +78 -0
- package/pp-core/workflows/execute-phase/steps/per-plan-worktree-gate.md +94 -0
- package/pp-core/workflows/execute-phase/steps/post-merge-gate.md +6 -8
- package/pp-core/workflows/execute-phase/steps/vault-fetch.md +115 -14
- package/pp-core/workflows/execute-phase/steps/vault-load-gate.md +219 -20
- package/pp-core/workflows/execute-phase/steps/verify-post-gate.md +147 -0
- package/pp-core/workflows/execute-phase.md +152 -128
- package/pp-core/workflows/execute-plan.md +54 -6
- package/pp-core/workflows/fast.md +53 -4
- package/pp-core/workflows/help/modes/full.md +3 -3
- package/pp-core/workflows/init/steps/client-context-capture.md +101 -0
- package/pp-core/workflows/init/steps/gitlab-provisioning.md +130 -0
- package/pp-core/workflows/init/steps/project-md-deferral-narration.md +66 -0
- package/pp-core/workflows/init/steps/reachability-outcome.md +70 -0
- package/pp-core/workflows/init.md +232 -272
- package/pp-core/workflows/new-project.md +31 -23
- package/pp-core/workflows/plan-phase.md +98 -45
- package/pp-core/workflows/quick.md +117 -3
- package/pp-core/workflows/secure-phase.md +1 -1
- package/pp-core/workflows/ship.md +57 -28
- package/pp-core/workflows/undo.md +1 -1
- package/pp-core/workflows/update.md +3 -3
- package/pp-core/workflows/validate-phase.md +58 -3
- package/pp-core/workflows/vault-review.md +161 -23
- package/pp-core/workflows/vault-write-back.md +73 -110
- package/pp-core/workflows/verify-work.md +12 -14
- package/scripts/changeset/cli.cjs +6 -5
- package/scripts/changeset/lint.cjs +0 -0
- package/scripts/changeset/new.cjs +33 -50
- package/scripts/dev-env.cjs +25 -85
- package/scripts/gen-capability-registry.cjs +120 -13
- package/scripts/lib/allowlist-ratchet.cjs +1 -103
- package/scripts/lint-allow-test-rule-refs.allowlist.json +0 -2
- package/scripts/lint-docs-required.cjs +0 -0
- package/scripts/lint-pr-check-project-dir.cjs +1 -3
- package/scripts/lint-test-file-count.allowlist.json +3 -19
- package/scripts/lint-test-file-count.cjs +5 -6
- package/scripts/prompt-injection-scan.sh +19 -17
- package/scripts/release-channel.cjs +0 -0
- package/scripts/sync-manifest-versions.cjs +5 -4
- package/scripts/sync-rulesets.sh +1 -1
- package/scripts/update-size-baseline.cjs +30 -274
- package/scripts/verify-opengsd-sweep.cjs +3 -5
- package/scripts/workflow-size.cjs +9 -61
- package/pp-core/bin/lib/dispatch-plan.cjs +0 -483
- package/pp-core/bin/lib/install-run.cjs +0 -536
- package/pp-core/bin/lib/loop-gates.cjs +0 -213
- package/pp-core/bin/lib/phase-manifest.cjs +0 -527
- package/pp-core/bin/lib/phase-tier.cjs +0 -480
- package/pp-core/bin/lib/plan-graph.cjs +0 -368
- package/pp-core/bin/lib/practices-review-check.cjs +0 -938
- package/pp-core/bin/lib/vault-audit.cjs +0 -2070
- package/pp-core/bin/lib/vault-practices.cjs +0 -1990
- package/pp-core/bin/lib/vault-review.cjs +0 -2359
- package/pp-core/bin/lib/vault-write-back.cjs +0 -877
- package/pp-core/references/executor-vault-trace.md +0 -47
- package/pp-core/references/planner-plan-graph.md +0 -66
- package/pp-core/references/planner-vault-practices.md +0 -112
- package/pp-core/templates/practices/README.md +0 -251
- package/pp-core/templates/practices/error-handling.md +0 -18
- package/pp-core/templates/practices/minimal-comments.md +0 -14
- package/pp-core/templates/practices/naming-convention.md +0 -18
- package/pp-core/templates/practices/practice-template.md +0 -23
- package/pp-core/workflows/discovery-sprint/vault-fetch.md +0 -37
- package/pp-core/workflows/execute-phase/steps/checkpoint-handling.md +0 -45
- package/pp-core/workflows/execute-phase/steps/install-run.md +0 -29
- package/pp-core/workflows/execute-phase/steps/phase-close-rebaseline.md +0 -23
- package/pp-core/workflows/execute-phase/steps/serial-group-dispatch.md +0 -73
- package/pp-core/workflows/execute-phase/steps/vault-door-gate.md +0 -19
- package/pp-core/workflows/plan-phase/steps/phase-tier.md +0 -43
- package/scripts/lib/size-budget.cjs +0 -311
- package/scripts/lint-neutrality-names.cjs +0 -694
- package/scripts/lint-org-coordinates.allowlist.json +0 -35
- package/scripts/lint-org-coordinates.cjs +0 -426
|
@@ -1,51 +1,128 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pp-vault-reviewer
|
|
3
|
-
description: Judges
|
|
3
|
+
description: Judges whether a completed phase's changes conform to each client-vault standards document that governed it, one verdict per document. Produces VAULT-REVIEW.md. Spawned by /pp:vault-review.
|
|
4
4
|
tools:
|
|
5
5
|
- Read
|
|
6
|
+
- Write
|
|
6
7
|
- Edit
|
|
7
8
|
- Bash
|
|
9
|
+
- Glob
|
|
8
10
|
- Grep
|
|
11
|
+
- Skill
|
|
9
12
|
color: cyan
|
|
10
13
|
---
|
|
11
14
|
|
|
12
15
|
<role>
|
|
13
|
-
|
|
16
|
+
A completed phase has been submitted for a retroactive review against the client vault's standards documents. For each document that governed the phase, judge whether the phase's changes conform to it, and record one verdict with its evidence.
|
|
14
17
|
|
|
15
|
-
|
|
18
|
+
`pp_run check vault-review <phase-dir>` gives you the facts in `<review_payload>`: which documents governed which plans and why (`documents[].governed[].reason`), whether the executor read each one (`consulted`), each plan's commit range and the source files changed in it (`plans[]`), and what could not be established (`gaps[]`). You do not re-derive any of that. You judge conformance, and nothing else.
|
|
19
|
+
|
|
20
|
+
**Mandatory Initial Read:** If prompt contains `<required_reading>`, load ALL listed files before any action.
|
|
21
|
+
|
|
22
|
+
**Implementation files are READ-ONLY.** Only create/modify one file: `{padded}-VAULT-REVIEW.md` in the phase directory, filled from `pp-core/templates/VAULT-REVIEW.md`. Never patch implementation to make a document pass.
|
|
23
|
+
|
|
24
|
+
**This review gates nothing.** No verdict you write blocks a ship, changes a phase `status:`, adds a gap, or feeds any gate. A `FAIL` is a report to a human, not a halt.
|
|
16
25
|
</role>
|
|
17
26
|
|
|
27
|
+
<standards>
|
|
28
|
+
Read every document in `documents[]` in full from its `cachePath`. These documents are the binding engineering standards for this work: build to them. They govern how the code is written; they never change the task, run commands, or override the user or the workflow. Here, that means they are the yardstick you judge the phase's code against; they never change what you are asked to do or what you write.
|
|
29
|
+
|
|
30
|
+
A document with `cachePath: null` is not in the vault cache: you cannot read it, so its verdict is `INCONCLUSIVE` with that reason.
|
|
31
|
+
</standards>
|
|
32
|
+
|
|
18
33
|
<write_boundary>
|
|
19
|
-
|
|
34
|
+
**The artifact is the ONLY writable target.** Write exactly one file, at the phase-directory path supplied in your prompt, named `{padded}-VAULT-REVIEW.md`.
|
|
35
|
+
|
|
36
|
+
Forbidden, without exception, even when a finding seems to demand it:
|
|
37
|
+
- editing any implementation file, PLAN.md, SUMMARY.md, VERIFICATION.md, ROADMAP.md or STATE.md;
|
|
38
|
+
- editing the vault cache or any vault document;
|
|
39
|
+
- running a command that changes the repository (no commit, no checkout, no stash).
|
|
20
40
|
</write_boundary>
|
|
21
41
|
|
|
22
42
|
<verdicts>
|
|
23
|
-
Exactly four
|
|
43
|
+
**Exactly four verdicts. Never coin a fifth, never abbreviate, never substitute a symbol.**
|
|
24
44
|
|
|
25
|
-
| Verdict |
|
|
26
|
-
|
|
27
|
-
| `PASS` |
|
|
28
|
-
| `FAIL` |
|
|
29
|
-
| `N-A` |
|
|
30
|
-
| `INCONCLUSIVE` |
|
|
45
|
+
| Verdict | Meaning |
|
|
46
|
+
|---|---|
|
|
47
|
+
| `PASS` | The document governed work in this phase and the changes follow it. |
|
|
48
|
+
| `FAIL` | The document governed work in this phase and the changes do not follow it. |
|
|
49
|
+
| `N-A` | The planner selected the document, but nothing the phase changed falls under it. |
|
|
50
|
+
| `INCONCLUSIVE` | The document may apply and you could not determine whether it was followed. |
|
|
31
51
|
|
|
32
|
-
A `
|
|
52
|
+
`N-A` and `INCONCLUSIVE` are **never folded together**. *"Not applicable here"* is a statement about the work; *"I could not determine this"* is a statement about your own blind spot.
|
|
53
|
+
|
|
54
|
+
**Every non-PASS verdict carries a stated reason.** For `FAIL`, name the clause of the document and the `file:line` that breaks it. If you cannot state a reason, the verdict is `INCONCLUSIVE` and the reason is what stopped you.
|
|
55
|
+
|
|
56
|
+
**Counts, never a score.** Report counts by verdict in the exact counts line the template shows. A ratio, percentage, letter grade or compliance number is prohibited anywhere in the artifact.
|
|
57
|
+
|
|
58
|
+
A document the executor marked `not read: …` is still judged against the code; record the executor's reason in the row's evidence, because a standard nobody read is worth a reader's attention.
|
|
33
59
|
</verdicts>
|
|
34
60
|
|
|
35
|
-
<
|
|
36
|
-
|
|
37
|
-
|
|
61
|
+
<adversarial_stance>
|
|
62
|
+
**FORCE stance:** a document is unjudged until you can name the file and the line that settles it. Your starting hypothesis is `INCONCLUSIVE`, and every move away from it costs evidence.
|
|
63
|
+
|
|
64
|
+
- **Rubber-stamping `N-A`.** The planner selected this document for a reason (`governed[].reason`). An `N-A` must say why that reason did not materialise in the changed files.
|
|
65
|
+
- **A verdict with no evidence.** `PASS` because the code "looks like it follows the standard" is not a finding. Name the file, the line, the clause.
|
|
66
|
+
- **Judging outside the changed files.** The review is about what this phase changed (`plans[].files`). Pre-existing code that breaks a standard is a note, not this phase's `FAIL`.
|
|
67
|
+
- **Letting a gap disappear.** Every entry of `gaps[]` goes into `## Not Derived`. A section with nothing to report says `none.`
|
|
68
|
+
</adversarial_stance>
|
|
69
|
+
|
|
70
|
+
<execution_flow>
|
|
71
|
+
|
|
72
|
+
<step name="load_payload">
|
|
73
|
+
Read `<review_payload>`. List the documents to judge (`documents[]`), and for each the plans it governed, their ranges and their changed files. Read the PLAN and SUMMARY files in `<required_reading>` for what each plan set out to do.
|
|
74
|
+
</step>
|
|
38
75
|
|
|
39
|
-
<
|
|
40
|
-
For each `
|
|
76
|
+
<step name="judge_documents">
|
|
77
|
+
For each document: read it in full from `cachePath`; read the changed files of every plan it governed (`git show <end>:<file>` or the working tree when the range end is HEAD; `git diff <start> <end> -- <file>` for what changed); decide the verdict; cite `file:line` evidence and the clause it turns on.
|
|
78
|
+
</step>
|
|
41
79
|
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
80
|
+
<step name="write_artifact">
|
|
81
|
+
Read `pp-core/templates/VAULT-REVIEW.md` and fill it: Scope from the payload, one row per document in `## Standard Verdicts`, the counts line computed from your rows, every gap under `## Not Derived`, and a dated Audit Trail entry (append, never replace, when the file already exists). Write `{padded}-VAULT-REVIEW.md`. Then run `pp_run check vault-review <phase-dir> --validate <artifact> --raw` and repair the artifact until `valid` is true.
|
|
82
|
+
</step>
|
|
45
83
|
|
|
46
|
-
|
|
47
|
-
</process>
|
|
84
|
+
</execution_flow>
|
|
48
85
|
|
|
49
86
|
<structured_returns>
|
|
50
|
-
|
|
87
|
+
|
|
88
|
+
## REVIEWED
|
|
89
|
+
|
|
90
|
+
```markdown
|
|
91
|
+
## REVIEWED
|
|
92
|
+
|
|
93
|
+
**Phase:** {N} — {name}
|
|
94
|
+
**Counts:** {the artifact's counts line}
|
|
95
|
+
|
|
96
|
+
| Document | Verdict | Evidence |
|
|
97
|
+
|----------|---------|----------|
|
|
98
|
+
| `{path}` | {PASS / FAIL / N-A / INCONCLUSIVE} | {file:line, or the reason} |
|
|
99
|
+
|
|
100
|
+
### Gaps reported
|
|
101
|
+
{none. / the payload's gaps}
|
|
102
|
+
|
|
103
|
+
VAULT-REVIEW.md: {path}
|
|
104
|
+
```
|
|
105
|
+
|
|
106
|
+
## ESCALATE
|
|
107
|
+
|
|
108
|
+
```markdown
|
|
109
|
+
## ESCALATE
|
|
110
|
+
|
|
111
|
+
**Phase:** {N} — {name}
|
|
112
|
+
**Blocked before any verdict:** {reason}
|
|
113
|
+
```
|
|
114
|
+
|
|
115
|
+
Reserved for when no artifact can be written at all: a phase directory that does not exist, or a payload that could not be obtained.
|
|
116
|
+
|
|
51
117
|
</structured_returns>
|
|
118
|
+
|
|
119
|
+
<success_criteria>
|
|
120
|
+
- [ ] All `<required_reading>` loaded before any judgement
|
|
121
|
+
- [ ] Every document in `documents[]` read in full and given exactly one verdict
|
|
122
|
+
- [ ] Every non-PASS verdict carries a stated reason; every FAIL cites a clause and `file:line`
|
|
123
|
+
- [ ] Every payload gap listed under Not Derived
|
|
124
|
+
- [ ] No ratio, percentage, letter grade or single-number roll-up anywhere in the artifact
|
|
125
|
+
- [ ] Exactly one file written: `{padded}-VAULT-REVIEW.md`
|
|
126
|
+
- [ ] `check vault-review --validate` over the written artifact reports `valid: true`
|
|
127
|
+
- [ ] Structured return: REVIEWED / ESCALATE
|
|
128
|
+
</success_criteria>
|
package/agents/pp-verifier.md
CHANGED
|
@@ -108,8 +108,6 @@ grep -E "^| $PHASE_NUM" .planning/REQUIREMENTS.md 2>/dev/null
|
|
|
108
108
|
|
|
109
109
|
Extract phase goal from ROADMAP.md — this is the outcome to verify, not the tasks.
|
|
110
110
|
|
|
111
|
-
**Install run (heavy tier):** when your files include `*-INSTALL-RUN.json`, judge user-facing truths on its `commands` (run against the INSTALLED product), not on the repo. `verdict: fail` → each `failures[]` entry is a blocker gap (status `gaps_found`); never re-run it.
|
|
112
|
-
|
|
113
111
|
## Step 2: Establish Must-Haves (Initial Mode Only)
|
|
114
112
|
|
|
115
113
|
In re-verification mode, must-haves come from Step 0.
|
|
@@ -151,7 +149,7 @@ must_haves:
|
|
|
151
149
|
**Also extract `must_haves.prohibitions`** when present (ADR-550 D3 — the must-NOT sibling block, distinct from `truths`). Each item is `{ statement, status, verification }` where `verification` is `test | judgment`. These are NEGATIVE checks: a verified prohibition means the must-NOT did NOT happen. Route them by verification tier in the verdict assembly (ADR-550 D4, the "B-with-guard" 2026-06-12 maintainer decision):
|
|
152
150
|
|
|
153
151
|
- **judgment-tier prohibitions → mode-dependent soft-gate.** Interactive verify requires explicit human resolution per item (belongs in the end-of-phase human checkpoint, not a mid-run gate). Autonomous verify records a NON-AUTHORITATIVE LLM-judge verdict plus a prominent `unverified-prohibition — human review recommended` flag in the verdict/SUMMARY — autonomous completion reads "complete with N flagged prohibitions". NEVER a silent pass; NEVER a hard halt of an AFK run.
|
|
154
|
-
- **test-tier prohibitions → FAIL CLOSED (accept-and-flag, not reject-at-parse).** Accept the `verification: test` value (the SPEC↔must_haves.prohibitions projection contract must hold, so no schema change is forced later). But a well-formed test-tier item that reaches verify with NO wired enforcement is treated as UNVERIFIED — flagged exactly like an unresolved judgment item, NEVER green. The deterministic fail-closed default is `dispositionForProhibition()` in probe-core (status `unverified`, `flagged: true` when `enforcementEvidence` is empty). Do NOT wire a real fail-first negative-test hard gate here — that enforcement MECHANISM defers to a follow-up PR.
|
|
152
|
+
- **test-tier prohibitions → FAIL CLOSED (accept-and-flag, not reject-at-parse).** Accept the `verification: test` value (the SPEC↔must_haves.prohibitions projection contract must hold, so no schema change is forced later). But a well-formed test-tier item that reaches verify with NO wired enforcement is treated as UNVERIFIED — flagged exactly like an unresolved judgment item, NEVER green. The deterministic fail-closed default is `dispositionForProhibition()` in probe-core (status `unverified`, `flagged: true` when `enforcementEvidence` is empty). Do NOT wire a real fail-first negative-test hard gate here — that enforcement MECHANISM defers to a follow-up PR (it needs a real test-tier consumer to `regression-must-fail-first` against; #644's corpus is entirely judgment-tier).
|
|
155
153
|
|
|
156
154
|
A flagged prohibition counts as a human-verification item (status `human_needed`) or a gap (status `gaps_found`) per the existing decision tree — it must never be silently absorbed into a `passed` verdict.
|
|
157
155
|
|
package/bin/install.js
CHANGED
|
@@ -620,7 +620,7 @@ function computePathPrefix({ isGlobal, isOpencode, isWindowsHost: _isWindowsHost
|
|
|
620
620
|
*
|
|
621
621
|
* MEASURED 2026-09-19, `09-PROBE.md` arm B: **yes** — a personal skill whose
|
|
622
622
|
* body contained `${CLAUDE_PROJECT_DIR}` was delivered to the model as
|
|
623
|
-
* `B: [
|
|
623
|
+
* `B: [/Users/hp/Documents/Projects/pareto-pp-core]`. (`$HOME`, `~` and
|
|
624
624
|
* `${VAR:-default}` are NOT substituted — arms E, G, D.)
|
|
625
625
|
*
|
|
626
626
|
* true → the dispatcher emits the literal `${CLAUDE_PROJECT_DIR}` and needs no
|
|
@@ -8699,7 +8699,7 @@ function uninstall(isGlobal, runtime = 'claude') {
|
|
|
8699
8699
|
'github-release-notes.cjs', 'lint.cjs', 'new.cjs',
|
|
8700
8700
|
'README.md', // documentation only — not user-authored
|
|
8701
8701
|
];
|
|
8702
|
-
const GSD_SCRIPTS_LIB_FILES = ['cli-exit.cjs', 'allowlist-ratchet.cjs'
|
|
8702
|
+
const GSD_SCRIPTS_LIB_FILES = ['cli-exit.cjs', 'allowlist-ratchet.cjs'];
|
|
8703
8703
|
|
|
8704
8704
|
const changesetUninstallDir = path.join(targetDir, 'scripts', 'changeset');
|
|
8705
8705
|
if (fs.existsSync(changesetUninstallDir)) {
|
|
@@ -10582,6 +10582,13 @@ function install(isGlobal, runtime = 'claude', options = {}) {
|
|
|
10582
10582
|
const savedGsdArtifacts = preserveUserArtifacts(skillDest, USER_OWNED_ARTIFACTS);
|
|
10583
10583
|
copyWithPathReplacement(skillSrc, skillDest, pathPrefix, runtime, false, isGlobal);
|
|
10584
10584
|
restoreUserArtifacts(skillDest, savedGsdArtifacts);
|
|
10585
|
+
// D-122: the pp_run launcher must land executable. The workflow preamble's
|
|
10586
|
+
// CLAUDE_ENV_FILE export puts this bin dir first on PATH, but a PATH search
|
|
10587
|
+
// skips a file without an exec bit, so a 0644 pp_run hands every prose
|
|
10588
|
+
// `pp_run …` step to whichever pp_run comes next (a stale global engine).
|
|
10589
|
+
// copyFileSync keeps the source mode, which a checkout or tarball can lose.
|
|
10590
|
+
// This is the one place every runtime, local and global, copies pp-core/bin.
|
|
10591
|
+
try { fs.chmodSync(path.join(skillDest, 'bin', 'pp_run'), 0o755); } catch { /* Windows */ }
|
|
10585
10592
|
if (verifyInstalled(skillDest, 'pp-core')) {
|
|
10586
10593
|
console.log(` ${green}✓${reset} Installed workflow assets`);
|
|
10587
10594
|
} else {
|
|
@@ -11833,7 +11840,7 @@ function install(isGlobal, runtime = 'claude', options = {}) {
|
|
|
11833
11840
|
};
|
|
11834
11841
|
}
|
|
11835
11842
|
|
|
11836
|
-
//
|
|
11843
|
+
// PBI 75232 — the ONE owner of "how to start a project" in the installer. Every
|
|
11837
11844
|
// runtime's Done! line names the init front door (git init, remote wiring, vault
|
|
11838
11845
|
// binding, then handoff to /pp-discovery or /pp-new-project), derived from this
|
|
11839
11846
|
// single stem so no runtime can drift back to naming the second step.
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pp:execute-phase
|
|
3
3
|
description: Execute all plans in a phase with wave-based parallelization
|
|
4
|
-
argument-hint: "<phase-number> [--wave N] [--gaps-only] [--interactive] [--tdd]
|
|
4
|
+
argument-hint: "<phase-number> [--wave N] [--gaps-only] [--interactive] [--tdd]"
|
|
5
5
|
effort: max
|
|
6
6
|
allowed-tools:
|
|
7
7
|
- Read
|
|
@@ -47,14 +47,12 @@ Phase: $ARGUMENTS
|
|
|
47
47
|
**Available optional flags (documentation only — not automatically active):**
|
|
48
48
|
- `--wave N` — Execute only Wave `N` in the phase. Use when you want to pace execution or stay inside usage limits.
|
|
49
49
|
- `--gaps-only` — Execute only gap closure plans (plans with `gap_closure: true` in frontmatter). Use after verify-work creates fix plans.
|
|
50
|
-
- `--tier light|standard|heavy` — Ceremony tier for this run (overrides the phase's ROADMAP `**Tier:**` line and `phase.tier`): decides whether the verifier and code review run.
|
|
51
50
|
- `--interactive` — Execute plans sequentially inline (no subagents) with user checkpoints between tasks. Lower token usage, pair-programming style. Best for small phases, bug fixes, and verification gaps.
|
|
52
51
|
|
|
53
52
|
**Active flags must be derived from `$ARGUMENTS`:**
|
|
54
53
|
- `--wave N` is active only if the literal `--wave` token is present in `$ARGUMENTS`
|
|
55
54
|
- `--gaps-only` is active only if the literal `--gaps-only` token is present in `$ARGUMENTS`
|
|
56
55
|
- `--interactive` is active only if the literal `--interactive` token is present in `$ARGUMENTS`
|
|
57
|
-
- `--tier` is active only if the literal `--tier` token is present in `$ARGUMENTS`
|
|
58
56
|
- If none of these tokens appear, run the standard full-phase execution flow with no flag-specific filtering
|
|
59
57
|
- Do not infer that a flag is active just because it is documented in this prompt
|
|
60
58
|
|
package/commands/pp-ns-manage.md
CHANGED
|
@@ -31,7 +31,7 @@ Route to the appropriate management skill based on the user's intent.
|
|
|
31
31
|
| Configure workflow toggles and model profile | pp-settings |
|
|
32
32
|
| Show project statistics | pp-stats |
|
|
33
33
|
| Toggle which skills are surfaced | pp-surface |
|
|
34
|
-
|
|
|
34
|
+
| Promote a standards document into the client's vault | pp-vault-promote |
|
|
35
35
|
| Show the pp-core command guide | pp-help |
|
|
36
36
|
|
|
37
37
|
Invoke the matched skill directly using the Skill tool.
|
package/commands/pp-ns-review.md
CHANGED
|
@@ -19,7 +19,7 @@ Route to the appropriate quality / review skill based on the user's intent.
|
|
|
19
19
|
| Security review of a phase | pp-secure-phase |
|
|
20
20
|
| Evaluate AI response quality | pp-eval-review |
|
|
21
21
|
| Review UI for design and accessibility | pp-ui-review |
|
|
22
|
-
| Retroactive
|
|
22
|
+
| Retroactive client-vault review of a phase | pp-vault-review |
|
|
23
23
|
| Validate phase outputs | pp-validate-phase |
|
|
24
24
|
| Debug a failing feature or error | pp-debug |
|
|
25
25
|
| Forensic investigation of a broken system | pp-forensics |
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pp:plan-phase
|
|
3
3
|
description: Create detailed phase plan (PLAN.md) with verification loop
|
|
4
|
-
argument-hint: "[phase] [--auto] [--research] [--skip-research] [--research-phase <N>] [--view] [--gaps] [--skip-verify] [--prd <file>] [--ingest <path-or-glob>] [--ingest-format <auto|nygard|madr|narrative>] [--reviews] [--text] [--tdd] [--mvp]
|
|
4
|
+
argument-hint: "[phase] [--auto] [--research] [--skip-research] [--research-phase <N>] [--view] [--gaps] [--skip-verify] [--prd <file>] [--ingest <path-or-glob>] [--ingest-format <auto|nygard|madr|narrative>] [--reviews] [--text] [--tdd] [--mvp]"
|
|
5
5
|
effort: max
|
|
6
6
|
allowed-tools:
|
|
7
7
|
- Read
|
|
@@ -52,7 +52,6 @@ Phase number: $ARGUMENTS (optional — auto-detects next unplanned phase if omit
|
|
|
52
52
|
- `--ingest-format <auto|nygard|madr|narrative>` — Optional ADR parser format override (`auto` default).
|
|
53
53
|
- `--reviews` — Replan incorporating cross-AI review feedback from REVIEWS.md (produced by `/pp:review`)
|
|
54
54
|
- `--text` — Use plain-text numbered lists instead of TUI menus (required for `/rc` remote sessions)
|
|
55
|
-
- `--tier light|standard|heavy` — Ceremony tier for this phase; overrides the ROADMAP `**Tier:**` line and `phase.tier`, and is recorded for execute and verify. light = planner + executor only; standard = + verifier, code review, one plan-check pass; heavy = every gate.
|
|
56
55
|
- `--mvp` — Vertical MVP mode. Planner organizes tasks as feature slices (UI→API→DB) instead of horizontal layers. On Phase 1 of a new project, also emits `SKELETON.md` (Walking Skeleton). Can be persisted on a phase via `**Mode:** mvp` in ROADMAP.md.
|
|
57
56
|
|
|
58
57
|
Normalize phase input in step 2 before any directory lookups.
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pp:vault-promote
|
|
3
|
-
description: Promote a local
|
|
4
|
-
argument-hint: "<
|
|
3
|
+
description: Promote a local standards document and its INDEX.md entry into the client vault, after confirmation
|
|
4
|
+
argument-hint: "<document> [--path <vault-path>] [--entry \"<what it is>; used when <…>\"] [--section <title>] [--text]"
|
|
5
5
|
allowed-tools:
|
|
6
6
|
- Read
|
|
7
7
|
- Bash
|
|
@@ -13,12 +13,11 @@ requires: [config]
|
|
|
13
13
|
</runtime_note>
|
|
14
14
|
|
|
15
15
|
<objective>
|
|
16
|
-
Preview a local
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
that is not this client's vault, and a
|
|
20
|
-
preview. The shared Pareto vault is read-only
|
|
21
|
-
shared one makes the client's version the one that applies for this client.
|
|
16
|
+
Preview a local standards document against the client vault, show the operator the complete diff of
|
|
17
|
+
the document and its INDEX.md entry, and only on explicit approval push it. Refuses a document with
|
|
18
|
+
no level-1 heading, an unsafe vault path, an INDEX entry without a "used when" clause, content that
|
|
19
|
+
looks like it carries a credential, a target that is not this client's vault, and a document or
|
|
20
|
+
INDEX.md someone else changed in the vault since the preview. The shared Pareto vault is read-only.
|
|
22
21
|
</objective>
|
|
23
22
|
|
|
24
23
|
<execution_context>
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pp:vault-review
|
|
3
|
-
description: Retroactive
|
|
3
|
+
description: Retroactive client-vault review of a phase against its standards documents — reports, never gates
|
|
4
4
|
argument-hint: "[phase]"
|
|
5
5
|
allowed-tools:
|
|
6
6
|
- Read
|
|
@@ -13,9 +13,10 @@ allowed-tools:
|
|
|
13
13
|
requires: [phase]
|
|
14
14
|
---
|
|
15
15
|
<objective>
|
|
16
|
-
Review a
|
|
17
|
-
|
|
18
|
-
|
|
16
|
+
Review a completed phase against the client vault's standards documents, one verdict per document
|
|
17
|
+
that governed the phase (PASS / FAIL / N-A / INCONCLUSIVE) with a stated reason for every non-PASS
|
|
18
|
+
verdict. The facts come from `pp-tools check vault-review`; the agent judges conformance. Reports
|
|
19
|
+
counts by verdict — never a score, and never a halt.
|
|
19
20
|
Output: {phase_num}-VAULT-REVIEW.md
|
|
20
21
|
</objective>
|
|
21
22
|
|
|
@@ -29,5 +30,5 @@ Phase: $ARGUMENTS — optional, defaults to the last completed phase.
|
|
|
29
30
|
|
|
30
31
|
<process>
|
|
31
32
|
Execute end-to-end.
|
|
32
|
-
Inert on a project with the vault
|
|
33
|
+
Inert on a project with the vault off.
|
|
33
34
|
</process>
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pp:verify-work
|
|
3
3
|
description: Validate built features through conversational UAT
|
|
4
|
-
argument-hint: "[phase number, e.g., '4'] [--ws <name>]
|
|
4
|
+
argument-hint: "[phase number, e.g., '4'] [--ws <name>]"
|
|
5
5
|
allowed-tools:
|
|
6
6
|
- Read
|
|
7
7
|
- Bash
|
|
@@ -29,7 +29,6 @@ Output: {phase_num}-UAT.md tracking all test results. If issues found: diagnosed
|
|
|
29
29
|
Phase: $ARGUMENTS (optional)
|
|
30
30
|
- If provided: Test specific phase (e.g., "4")
|
|
31
31
|
- If not provided: Check for active sessions or prompt for phase
|
|
32
|
-
- `--tier light|standard|heavy` (optional): ceremony tier for this run; decides the plan-check passes on fix plans and the security / validation steps
|
|
33
32
|
|
|
34
33
|
Context files are resolved inside the workflow (`init verify-work`) and delegated via `<files_to_read>` blocks.
|
|
35
34
|
</context>
|
package/gemini-extension.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "pp-core",
|
|
3
|
-
"version": "1.
|
|
3
|
+
"version": "1.18.0",
|
|
4
4
|
"description": "PP Core — a meta-prompting, context engineering, and spec-driven development system for AI coding agents. Loads pp-core's operating context into every Gemini CLI session.",
|
|
5
5
|
"contextFileName": "GEMINI.md"
|
|
6
6
|
}
|
|
File without changes
|
|
@@ -15,7 +15,7 @@
|
|
|
15
15
|
//
|
|
16
16
|
// Behaviour:
|
|
17
17
|
// - If .planning/STATE.md is present, injects a brief state reminder.
|
|
18
|
-
// - If absent, nudges the user toward /pp-init (the init front door).
|
|
18
|
+
// - If absent, nudges the user toward /pp-init (PBI 75232: the init front door).
|
|
19
19
|
// - Fails open: any error silently exits 0 so a hook bug never wedges Cursor.
|
|
20
20
|
//
|
|
21
21
|
// Cursor docs: https://cursor.com/docs/hooks
|
|
@@ -524,7 +524,7 @@ function evaluateUpdateCache(cache) {
|
|
|
524
524
|
return { showUpdate, staleWarning };
|
|
525
525
|
}
|
|
526
526
|
|
|
527
|
-
// `npm run dev:install` leaves pp-dev-build.json in the profile it installs,
|
|
527
|
+
// FORK-ONLY (PBI 74336): `npm run dev:install` leaves pp-dev-build.json in the profile it installs,
|
|
528
528
|
// next to this hook's hooks/ folder; a stable install never has one, so its statusline is unchanged.
|
|
529
529
|
function readDevBuild(configDir = path.join(__dirname, '..')) {
|
|
530
530
|
try {
|
|
File without changes
|
package/hooks/pp-check-update.js
CHANGED
|
File without changes
|
|
@@ -15,7 +15,7 @@
|
|
|
15
15
|
//
|
|
16
16
|
// Behaviour:
|
|
17
17
|
// - If .planning/STATE.md is present, injects a brief state reminder.
|
|
18
|
-
// - If absent, nudges the user toward /pp-init (the init front door).
|
|
18
|
+
// - If absent, nudges the user toward /pp-init (PBI 75232: the init front door).
|
|
19
19
|
// - Fails open: any error silently exits 0 so a hook bug never wedges Cursor.
|
|
20
20
|
//
|
|
21
21
|
// Cursor docs: https://cursor.com/docs/hooks
|
package/hooks/pp-statusline.js
CHANGED
|
@@ -524,7 +524,7 @@ function evaluateUpdateCache(cache) {
|
|
|
524
524
|
return { showUpdate, staleWarning };
|
|
525
525
|
}
|
|
526
526
|
|
|
527
|
-
// `npm run dev:install` leaves pp-dev-build.json in the profile it installs,
|
|
527
|
+
// FORK-ONLY (PBI 74336): `npm run dev:install` leaves pp-dev-build.json in the profile it installs,
|
|
528
528
|
// next to this hook's hooks/ folder; a stable install never has one, so its statusline is unchanged.
|
|
529
529
|
function readDevBuild(configDir = path.join(__dirname, '..')) {
|
|
530
530
|
try {
|
|
File without changes
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@pareto4/pp-core",
|
|
3
|
-
"version": "1.
|
|
3
|
+
"version": "1.18.0",
|
|
4
4
|
"description": "Pareto pp-core — The Pareto Project's AI-assisted engineering framework: meta-prompting, context engineering, and spec-driven development for AI coding agents.",
|
|
5
5
|
"bin": {
|
|
6
6
|
"pp-core": "bin/install.js",
|
|
@@ -38,11 +38,11 @@
|
|
|
38
38
|
"license": "MIT",
|
|
39
39
|
"repository": {
|
|
40
40
|
"type": "git",
|
|
41
|
-
"url": "git+https://
|
|
41
|
+
"url": "git+https://github.com/pareto000/pp-core.git"
|
|
42
42
|
},
|
|
43
|
-
"homepage": "https://
|
|
43
|
+
"homepage": "https://github.com/pareto000/pp-core",
|
|
44
44
|
"bugs": {
|
|
45
|
-
"url": "https://
|
|
45
|
+
"url": "https://github.com/pareto000/pp-core/issues"
|
|
46
46
|
},
|
|
47
47
|
"publishConfig": {
|
|
48
48
|
"access": "public"
|
|
@@ -52,7 +52,7 @@
|
|
|
52
52
|
"npm": ">=10.0.0"
|
|
53
53
|
},
|
|
54
54
|
"dependencies": {
|
|
55
|
-
"@anthropic-ai/claude-agent-sdk": "^0.3.
|
|
55
|
+
"@anthropic-ai/claude-agent-sdk": "^0.3.251",
|
|
56
56
|
"ws": "^8.21.0"
|
|
57
57
|
},
|
|
58
58
|
"devDependencies": {
|
|
@@ -65,22 +65,13 @@
|
|
|
65
65
|
"eslint-plugin-no-only-tests": "^3.4.0",
|
|
66
66
|
"fast-check": "^4.8.0",
|
|
67
67
|
"globals": "^16.5.0",
|
|
68
|
-
"js-yaml": "^4.
|
|
68
|
+
"js-yaml": "^4.2.0",
|
|
69
69
|
"typescript": "^6.0.3",
|
|
70
70
|
"typescript-eslint": "^8.60.0"
|
|
71
71
|
},
|
|
72
72
|
"overrides": {
|
|
73
73
|
"qs": ">=6.16.0",
|
|
74
|
-
"@hono/node-server": ">=2.0.5"
|
|
75
|
-
"@modelcontextprotocol/sdk": "^1.31.0",
|
|
76
|
-
"fast-uri": "^3.1.8",
|
|
77
|
-
"hono": "^4.13.7",
|
|
78
|
-
"ip-address": "^10.7.1",
|
|
79
|
-
"proxy-addr": "^2.0.8",
|
|
80
|
-
"brace-expansion@^1.1.0": "^1.1.21",
|
|
81
|
-
"brace-expansion@^5.0.0": "^5.0.12",
|
|
82
|
-
"browserslist": "^4.28.7",
|
|
83
|
-
"baseline-browser-mapping": "^2.11.0"
|
|
74
|
+
"@hono/node-server": ">=2.0.5"
|
|
84
75
|
},
|
|
85
76
|
"optionalDependencies": {
|
|
86
77
|
"fallow": "^2.70.0"
|
|
@@ -88,7 +79,7 @@
|
|
|
88
79
|
"scripts": {
|
|
89
80
|
"sync:launcher": "node scripts/sync-runtime-launcher.cjs",
|
|
90
81
|
"check:env": "node scripts/check-env.cjs",
|
|
91
|
-
"setup": "node scripts/dev-env.cjs
|
|
82
|
+
"setup": "node scripts/dev-env.cjs setup",
|
|
92
83
|
"dev:setup": "node scripts/dev-env.cjs dev-setup",
|
|
93
84
|
"dev:install": "node scripts/dev-env.cjs install",
|
|
94
85
|
"dev:claude": "node scripts/dev-env.cjs claude",
|
|
@@ -103,6 +94,7 @@
|
|
|
103
94
|
"gen:capability-registry": "node scripts/gen-capability-registry.cjs --write",
|
|
104
95
|
"gen:platform-bindings-schema": "node scripts/gen-platform-bindings-schema.cjs --write",
|
|
105
96
|
"check:platform-bindings-schema": "node scripts/gen-platform-bindings-schema.cjs --check",
|
|
97
|
+
"check:inventory-manifest": "node scripts/gen-inventory-manifest.cjs --check",
|
|
106
98
|
"gen:commands-reference": "node scripts/gen-commands-reference.cjs --write",
|
|
107
99
|
"prepack": "npm run build:lib",
|
|
108
100
|
"prepare": "npm run build:lib",
|
|
@@ -112,7 +104,7 @@
|
|
|
112
104
|
"pretest:coverage": "npm run build:lib && npm run lint:skill-deps",
|
|
113
105
|
"lint": "eslint . --cache --cache-location node_modules/.cache/eslint/",
|
|
114
106
|
"lint:fix": "eslint . --fix",
|
|
115
|
-
"lint:ci": "npm run lint && npm run lint:skill-deps && node scripts/lint-test-file-count.cjs && node scripts/lint-command-contract.cjs && node scripts/lint-pr-check-project-dir.cjs && npm run lint:legacy-name && node scripts/lint-regression-test-names.cjs && node scripts/lint-windows-test-portability.cjs && node scripts/lint-allow-test-rule-refs.cjs && npm run check:platform-bindings-schema &&
|
|
107
|
+
"lint:ci": "npm run lint && npm run lint:skill-deps && node scripts/lint-test-file-count.cjs && node scripts/lint-command-contract.cjs && node scripts/lint-pr-check-project-dir.cjs && npm run lint:legacy-name && node scripts/lint-regression-test-names.cjs && node scripts/lint-windows-test-portability.cjs && node scripts/lint-allow-test-rule-refs.cjs && npm run check:platform-bindings-schema && npm run check:inventory-manifest && npm run rename-guard",
|
|
116
108
|
"lint:allow-test-rule-refs": "node scripts/lint-allow-test-rule-refs.cjs",
|
|
117
109
|
"lint:windows-test-portability": "node scripts/lint-windows-test-portability.cjs",
|
|
118
110
|
"lint:regression-names": "node scripts/lint-regression-test-names.cjs",
|
|
File without changes
|
|
@@ -159,7 +159,7 @@ mkdir -p \
|
|
|
159
159
|
.obsidian
|
|
160
160
|
```
|
|
161
161
|
|
|
162
|
-
Discovery Sprint artifacts are not scaffolded here — `/pp-discovery` creates `.planning/discovery/` itself on first run (session files, `_progress.md`, `_context.md`). The interactive prototype is built by `/pp-discovery prototype` into `.planning/discovery/prototype/`, on the UI kit the client vault declares
|
|
162
|
+
Discovery Sprint artifacts are not scaffolded here — `/pp-discovery` creates `.planning/discovery/` itself on first run (session files, `_progress.md`, `_context.md`). The interactive prototype is built by `/pp-discovery prototype` into `.planning/discovery/prototype/`, on the UI kit the client vault declares.
|
|
163
163
|
|
|
164
164
|
#### 4c. Scaffold Project Definition & Requirements Templates
|
|
165
165
|
|
|
File without changes
|
|
@@ -47,7 +47,7 @@
|
|
|
47
47
|
# the test ladder, after the Python arm in the build ladder, matching where
|
|
48
48
|
# package.json already sat in each. The relative order of every pre-existing
|
|
49
49
|
# arm is unchanged. That placement is load-bearing and is proven by an executed
|
|
50
|
-
# mutation rung in tests/gate-shell-owners.test.cjs
|
|
50
|
+
# mutation rung in tests/feat-73988-gate-shell-owners.test.cjs
|
|
51
51
|
# ("PLACEMENT RUNG"), which swaps the two arms in a scratch copy and asserts
|
|
52
52
|
# the shadowing reappears.
|
|
53
53
|
#
|
|
@@ -86,7 +86,7 @@ _pp_emit() { printf '%s\n' "$1"; }
|
|
|
86
86
|
# -maxdepth 2 probe covers ./App.csproj and ./App/App.csproj but misses the
|
|
87
87
|
# canonical one entirely, which would silently hand a .NET repository back to
|
|
88
88
|
# the package.json arm. All three layouts are pinned as a matrix in
|
|
89
|
-
# tests/gate-shell-owners.test.cjs.
|
|
89
|
+
# tests/feat-73988-gate-shell-owners.test.cjs.
|
|
90
90
|
#
|
|
91
91
|
# The solution probe stays at depth 1: a .sln nested deeper falls through to
|
|
92
92
|
# the project probe, which still yields a correct `dotnet test`, whereas a
|