paqad-ai 1.65.0 → 1.66.0

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # paqad-ai
 
+## 1.66.0
+
+### Minor Changes
+
+- 87c8153: Wire the idle planning skills into the enforced stage contract (#359): four fully-built-but-never-invoked planning helpers now run when relevant, and only when relevant.
+
+  - The enforced feature-development **planning stage** gains three lane-gated instruction blocks, sourced from one shared set of constants so the default policy object and the rendered `feature-development.yaml` cannot drift: an all-lanes reuse reflex (consult the Existing surface + `index query`, follow the solution-architect Reuse / Extend / Build-new procedure), a graduated/full block that runs `diff-minimizer` before the plan compiles and `existing-doc-checker` before any new doc, and a graduated/full multi-module block that runs `cross-module-impact-scanner`.
+  - **Lane honesty:** a fast-lane change pays exactly one sentence; no skill procedure loads for it. A new `reusePlanningInstructions(lane, { multiModule })` assembler is the single source of that lane-gating logic.
+  - The #357 plan schema now accepts an optional per-step `classification` (`ac-satisfying` | `necessary-setup` | `scaffolding` | `over-build`) so a diff-minimizer verdict can ride in the compiled `plan.json`; a plan without it stays valid.
+  - No new hook and no new runtime gate — the skills feed fields the schema already checks, and the wired-vs-sidelined map in the onboarded-project overview is updated to keep it honest.
+
 ## 1.65.0
 
 ### Minor Changes

package/dist/cli/index.js

@@ -8154,6 +8154,16 @@ var init_paths2 = __esm({
 import { existsSync as existsSync15, readFileSync as readFileSync14 } from "fs";
 import { join as join30 } from "path";
 import YAML6 from "yaml";
+function reusePlanningInstructions(lane, options = {}) {
+  const instructions = [REUSE_PLANNING_INSTRUCTION];
+  if (lane === "graduated" || lane === "full") {
+    instructions.push(DIFF_MINIMIZER_PLANNING_INSTRUCTION);
+    if (options.multiModule === true) {
+      instructions.push(CROSS_MODULE_IMPACT_PLANNING_INSTRUCTION);
+    }
+  }
+  return instructions;
+}
 function featureDevelopmentPolicyPath(projectRoot) {
   return join30(projectRoot, PATHS.WORKFLOWS_DIR, "feature-development.yaml");
 }
@@ -8194,7 +8204,11 @@ function defaultFeatureDevelopmentPolicy() {
         instructions: [
           "Review the canonical module and instruction docs before planning the change.",
           "Keep planning scoped to the requested feature and the current repository state.",
-          "Rules-as-scripts (issue #89): invoke rule-script-reconciler at planning entry. RS-* drift (rules edited without script regen, manual map edits, failing fixtures) surfaces via the Decision Pause Contract. Resolve per escalation.rule_scripts_stale before planning continues."
+          "Rules-as-scripts (issue #89): invoke rule-script-reconciler at planning entry. RS-* drift (rules edited without script regen, manual map edits, failing fixtures) surfaces via the Decision Pause Contract. Resolve per escalation.rule_scripts_stale before planning continues.",
+          // Issue #359 — the reuse-first block (all lanes) + the graduated/full skill blocks.
+          // The full+multiModule call materialises every lane-gated line; the inline lane
+          // conditions in each string keep a fast-lane read to the first sentence only.
+          ...reusePlanningInstructions("full", { multiModule: true })
         ],
         required_inputs: ["active request", "canonical docs"],
         strictness: {
@@ -8580,6 +8594,9 @@ stages:
       - Review the canonical module and instruction docs before planning the change.
       - Keep planning scoped to the requested feature and the current repository state.
       - "Rules-as-scripts (issue #89): invoke rule-script-reconciler at planning entry. RS-* drift (rules edited without script regen, manual map edits, failing fixtures) surfaces via the Decision Pause Contract. Resolve per escalation.rule_scripts_stale before planning continues."
+      - "${REUSE_PLANNING_INSTRUCTION}"
+      - "${DIFF_MINIMIZER_PLANNING_INSTRUCTION}"
+      - "${CROSS_MODULE_IMPACT_PLANNING_INSTRUCTION}"
     required_inputs:
       - active request
       - canonical docs
@@ -8704,7 +8721,7 @@ stages:
       - CI gate result
 `;
 }
-var STAGE_ORDER, REQUIRED_TRUE_STRICTNESS, SPEC_REVIEW_ENFORCEMENT_INSTRUCTION;
+var STAGE_ORDER, REQUIRED_TRUE_STRICTNESS, SPEC_REVIEW_ENFORCEMENT_INSTRUCTION, REUSE_PLANNING_INSTRUCTION, DIFF_MINIMIZER_PLANNING_INSTRUCTION, CROSS_MODULE_IMPACT_PLANNING_INSTRUCTION;
 var init_feature_development_policy = __esm({
   "src/pipeline/feature-development-policy.ts"() {
     "use strict";
@@ -8730,6 +8747,9 @@ var init_feature_development_policy = __esm({
       documentation_sync: { require_canonical_sync: true }
     };
     SPEC_REVIEW_ENFORCEMENT_INSTRUCTION = "Spec-review enforcement (issue #401): SCRIPT-ENFORCED. `paqad-ai spec freeze` runs the spec-quality review itself and refuses to freeze while a critical defect is open, so never run `compliance review` by hand to satisfy the clause above. Major and minor findings never block. The defect summary is folded into the bundle's specification.json and no .paqad/compliance/<slug>/spec-review.json is written by this workflow. Author the spec markdown inside the project: a path resolving outside the project root is rejected.";
+    REUSE_PLANNING_INSTRUCTION = "Reuse before you build: consult the Existing surface section (and `npx paqad-ai index query <name>`) for existing helpers; follow the solution-architect procedure \u2014 Reuse / Extend / Build-new, and record a rejection rationale for every Build-new.";
+    DIFF_MINIMIZER_PLANNING_INSTRUCTION = "On graduated and full lanes: run the diff-minimizer skill over the implementation sequence before compiling the plan; drop steps it classifies over-build or record why they stay. Run existing-doc-checker before creating any new doc or registry artifact.";
+    CROSS_MODULE_IMPACT_PLANNING_INSTRUCTION = "On graduated and full lanes touching more than one module: run cross-module-impact-scanner; surface breaking / silent-shift rows as Decision Packets (existing categories).";
   }
 });
 
@@ -30092,7 +30112,13 @@ var PLAN_SCHEMA = {
         properties: {
           id: { type: "string", minLength: 1 },
           description: { type: "string", minLength: 1 },
-          module: { type: "string", minLength: 1 }
+          module: { type: "string", minLength: 1 },
+          // Issue #359 — the diff-minimizer verdict for the step. Optional and additive so a
+          // plan.json compiled before this change stays valid (INV-3); when present it must
+          // be one of the skill's four classifications.
+          classification: {
+            enum: ["ac-satisfying", "necessary-setup", "scaffolding", "over-build"]
+          }
         }
       }
     },
@@ -32812,7 +32838,7 @@ init_cancelled_error();
 init_events();
 
 // src/index.ts
-var VERSION = "1.65.0";
+var VERSION = "1.66.0";
 
 // src/cli/commands/audit.ts
 init_esm_shims();