prizmkit 1.1.47 → 1.1.49

package/bundled/skills/feature-planner/scripts/validate-and-generate.py

@@ -6,12 +6,14 @@ for the dev-pipeline system.
 Commands:
   validate    Validate an existing .prizmkit/plans/feature-list.json
   template    Generate a blank template .prizmkit/plans/feature-list.json
+  generate    Validate a draft JSON and generate final feature-list.json with defaults
   summary     Print a summary table of features from a .prizmkit/plans/feature-list.json
   grade       Generate grading results from eval runs (for npm run skill:review)
 
 Usage:
   python3 validate-and-generate.py validate --input .prizmkit/plans/feature-list.json [--output validated.json] [--mode new|incremental]
   python3 validate-and-generate.py template --output .prizmkit/plans/feature-list.json
+  python3 validate-and-generate.py generate --input draft.json --output .prizmkit/plans/feature-list.json [--mode new|incremental]
   python3 validate-and-generate.py summary --input .prizmkit/plans/feature-list.json [--format markdown|json]
   python3 validate-and-generate.py grade --workspace /.codebuddy/skill-evals/feature-planner-workspace --iteration iteration-1
 
@@ -716,6 +718,66 @@ def cmd_template(args):
     return 0
 
 
+def cmd_generate(args):
+    """Handle the 'generate' command.
+
+    Loads a draft JSON (produced by AI), fills in defaults, validates,
+    and writes the final feature-list.json.
+    """
+    if not args.input:
+        _err("--input is required for the generate command")
+        return 2
+    if not args.output:
+        _err("--output is required for the generate command")
+        return 2
+
+    # Load draft (supports stdin via '-')
+    if args.input == "-":
+        try:
+            data = json.load(sys.stdin)
+        except json.JSONDecodeError as exc:
+            _err("Invalid JSON from stdin: {}".format(exc))
+            return 2
+    else:
+        data, load_err = _load_json(args.input)
+        if load_err:
+            _err(load_err)
+            return 2
+
+    # Fill in defaults
+    now = datetime.now(timezone.utc).strftime("%Y-%m-%dT%H:%M:%SZ")
+    data.setdefault("$schema", SCHEMA_VERSION)
+    data.setdefault("created_at", now)
+    data.setdefault("created_by", "feature-planner")
+
+    # Ensure project_name from app_name if needed
+    if "project_name" not in data and "app_name" in data:
+        data["project_name"] = data["app_name"]
+
+    # Set default status for features without one
+    for feature in data.get("features", []):
+        feature.setdefault("status", "pending")
+
+    # Validate
+    mode = getattr(args, "mode", "new") or "new"
+    result = validate_feature_list(data, planning_mode=mode)
+
+    # Output validation result
+    print(json.dumps(result, indent=2, ensure_ascii=False))
+
+    if result["valid"]:
+        _write_json(args.output, data)
+        _info("Generated feature-list written to {}".format(args.output))
+        return 0
+    else:
+        _err("Validation failed with {} error(s)".format(len(result["errors"])))
+        for e in result["errors"]:
+            _err("  " + e)
+        for w in result.get("warnings", []):
+            _warn("  " + w)
+        return 1
+
+
 def cmd_summary(args):
     """Handle the 'summary' command."""
     if not args.input:
@@ -857,6 +919,7 @@ def main():
             "  %(prog)s validate --input .prizmkit/plans/feature-list.json --mode incremental\n"
             "  %(prog)s validate --input .prizmkit/plans/feature-list.json --output validated.json\n"
             "  %(prog)s template --output .prizmkit/plans/feature-list.json\n"
+            "  %(prog)s generate --input draft.json --output .prizmkit/plans/feature-list.json\n"
             "  %(prog)s summary --input .prizmkit/plans/feature-list.json\n"
             "  %(prog)s summary --input .prizmkit/plans/feature-list.json --format json\n"
         ),
@@ -891,6 +954,24 @@ def main():
         "--output", required=True, help="Path to write template file"
     )
 
+    # -- generate --
+    p_generate = subparsers.add_parser(
+        "generate",
+        help="Validate a draft and generate final feature-list.json with defaults filled in",
+    )
+    p_generate.add_argument(
+        "--input", required=True, help="Path to draft JSON (or '-' for stdin)"
+    )
+    p_generate.add_argument(
+        "--output", required=True, help="Path to write final feature-list.json"
+    )
+    p_generate.add_argument(
+        "--mode",
+        choices=["new", "incremental"],
+        default="new",
+        help="Validation mode (default: new)",
+    )
+
     # -- summary --
     p_summary = subparsers.add_parser(
         "summary",
@@ -931,6 +1012,7 @@ def main():
     dispatch = {
         "validate": cmd_validate,
         "template": cmd_template,
+        "generate": cmd_generate,
         "summary": cmd_summary,
         "grade": cmd_grade,
     }

package/bundled/skills/refactor-pipeline-launcher/SKILL.md

@@ -130,23 +130,30 @@ Detect user intent from their message, then follow the corresponding workflow:
 
    Wait for user confirmation before proceeding.
 
-5. **Ask execution mode** (first user decision):
+5. **Ask execution mode** (first user decision — SEPARATE `AskUserQuestion` call):
 
-   Present the three modes and ask the user to choose:
-   - **(1) Foreground** (recommended) — pipeline runs in the current session via `run-refactor.sh run`. Visible output and direct error feedback.
-   - **(2) Background daemon** — pipeline runs fully detached via `launch-refactor-daemon.sh`. Survives AI CLI session closure.
-   - **(3) Manual** — display the final assembled commands only. Do not execute anything. User runs them on their own.
+   ⛔ **This MUST be its own standalone `AskUserQuestion` call.** Do NOT combine execution mode with step 6 config questions. The execution mode question is asked ALONE, user responds, THEN you proceed to step 6.
 
-6. **Ask configuration options** ⚠️ MANDATORY INTERACTIVE STEP — applies to ALL execution modes (Foreground, Background, AND Manual). You MUST ask the user to configure options and WAIT for their response BEFORE proceeding to step 7. Do NOT skip this step or merge it with step 7.
+   Use `AskUserQuestion` with exactly 1 question:
 
-   ⛔ **HARD STOP**: You MUST call `AskUserQuestion` with the questions below and WAIT for the user's response. You MUST NOT:
+   **Question 1 — Execution mode** (multiSelect: false):
+   - Foreground (Recommended) — pipeline runs in the current session via `run-refactor.sh run`. Visible output and direct error feedback.
+   - Background daemon — pipeline runs fully detached via `launch-refactor-daemon.sh`. Survives AI CLI session closure.
+   - Manual — display the final assembled commands only. Do not execute anything. User runs them on their own.
+
+   ⚠️ STOP HERE and wait for user response before continuing to step 6.
+
+6. **Ask configuration options** ⚠️ MANDATORY INTERACTIVE STEP — applies to ALL execution modes (Foreground, Background, AND Manual). This is a SEPARATE `AskUserQuestion` call from step 5. You MUST ask the user to configure options and WAIT for their response BEFORE proceeding to step 7.
+
+   ⛔ **HARD STOP**: You MUST call `AskUserQuestion` with the 4 questions below and WAIT for the user's response. You MUST NOT:
+   - Combine step 5 and step 6 into one `AskUserQuestion` call (this is the most common violation — execution mode MUST be asked separately in step 5)
    - Skip this step and jump to step 7
    - Merge step 6 and step 7 into one response
    - Assume default values and show the command without asking
    - Show the command as text and ask "ready?" without presenting the options
-   If you find yourself writing the final command before the user has answered these questions, STOP — you are violating this rule.
+   If you find yourself writing the final command before the user has answered these 4 questions, STOP — you are violating this rule.
 
-   Use `AskUserQuestion` to present the following configuration choices. Each question is a separate selectable option:
+   Use `AskUserQuestion` to present ALL 4 configuration choices (the full 4-question budget goes to config, NOT shared with execution mode):
 
    **Question 1 — Verbose logging** (multiSelect: false):
    - On (default) — Detailed AI session logs including tool calls and subagent activity

package/bundled/skills/refactor-planner/SKILL.md

@@ -293,12 +293,14 @@ If issues found, discuss with user and resolve before proceeding.
 
 **Goal**: Produce `.prizmkit/plans/refactor-list.json` and validate it.
 
-1. Generate `.prizmkit/plans/refactor-list.json` at `.prizmkit/plans/`
-2. Run validation:
+**IMPORTANT: Do NOT hand-write the final JSON file.** Instead:
+1. Write a draft JSON to `.prizmkit/plans/refactor-list.draft.json` with all collected refactor data.
+2. Call the generate script to validate and produce the final file:
    ```bash
-   python3 ${SKILL_DIR}/scripts/validate-and-generate-refactor.py validate --input .prizmkit/plans/refactor-list.json
+   python3 ${SKILL_DIR}/scripts/validate-and-generate-refactor.py generate --input .prizmkit/plans/refactor-list.draft.json --output .prizmkit/plans/refactor-list.json
    ```
-3. If validation fails -> fix issues and re-validate (max 3 attempts)
+   The script fills in defaults (`$schema`, `created_at`, `created_by`), validates all fields, and writes the final file only if validation passes.
+3. If validation fails -> fix the draft and retry (max 3 attempts)
 4. If validation passes -> present final summary
 
 **CHECKPOINT CP-RP-6**: `.prizmkit/plans/refactor-list.json` generated and validated.
@@ -380,9 +382,12 @@ For simple refactoring with minimal scope:
    
    **NEVER proceed without explicit user selection via `AskUserQuestion`. Do NOT render options as plain text — the user must be able to click/select.**
 3. Draft items (title + type + scope + description + acceptance_criteria + behavior_preservation + dependencies)
-4. Run validation script immediately
+4. Write draft to `.prizmkit/plans/refactor-list.draft.json`, then call the generate script:
+   ```bash
+   python3 ${SKILL_DIR}/scripts/validate-and-generate-refactor.py generate --input .prizmkit/plans/refactor-list.draft.json --output .prizmkit/plans/refactor-list.json
+   ```
 5. If valid -> summarize and recommend next step
-6. If invalid -> apply fixes, re-validate (max 2 attempts, then escalate to full workflow)
+6. If invalid -> apply fixes to the draft, re-run generate (max 2 attempts, then escalate to full workflow)
 
 ### When NOT to Use Fast Path
 - More than 2 refactor items

package/bundled/skills/refactor-planner/scripts/validate-and-generate-refactor.py

@@ -6,11 +6,13 @@ for the dev-pipeline system.
 Commands:
   validate    Validate an existing .prizmkit/plans/refactor-list.json
   template    Generate a blank template .prizmkit/plans/refactor-list.json
+  generate    Validate a draft JSON and generate final refactor-list.json with defaults
   summary     Print a summary table of refactors from a .prizmkit/plans/refactor-list.json
 
 Usage:
   python3 validate-and-generate-refactor.py validate --input .prizmkit/plans/refactor-list.json [--output validated.json]
   python3 validate-and-generate-refactor.py template --output .prizmkit/plans/refactor-list.json
+  python3 validate-and-generate-refactor.py generate --input draft.json --output .prizmkit/plans/refactor-list.json
   python3 validate-and-generate-refactor.py summary --input .prizmkit/plans/refactor-list.json [--format markdown|json]
 
 Python 3.6+ required. No external dependencies.
@@ -22,6 +24,7 @@ import json
 import os
 import re
 import sys
+from datetime import datetime, timezone
 
 # ---------------------------------------------------------------------------
 # Constants
@@ -687,6 +690,61 @@ def cmd_template(args):
     return 0
 
 
+def cmd_generate(args):
+    """Handle the 'generate' command.
+
+    Loads a draft JSON (produced by AI), fills in defaults, validates,
+    and writes the final refactor-list.json.
+    """
+    if not args.input:
+        _err("--input is required for the generate command")
+        return 2
+    if not args.output:
+        _err("--output is required for the generate command")
+        return 2
+
+    # Load draft (supports stdin via '-')
+    if args.input == "-":
+        try:
+            data = json.load(sys.stdin)
+        except json.JSONDecodeError as exc:
+            _err("Invalid JSON from stdin: {}".format(exc))
+            return 2
+    else:
+        data, load_err = _load_json(args.input)
+        if load_err:
+            _err(load_err)
+            return 2
+
+    # Fill in defaults
+    now = datetime.now(timezone.utc).strftime("%Y-%m-%dT%H:%M:%SZ")
+    data.setdefault("$schema", SCHEMA_VERSION)
+    data.setdefault("created_at", now)
+    data.setdefault("created_by", "refactor-planner")
+
+    # Set default status for refactors without one
+    for refactor in data.get("refactors", []):
+        refactor.setdefault("status", "pending")
+
+    # Validate
+    result = validate_refactor_list(data)
+
+    # Output validation result
+    print(json.dumps(result, indent=2, ensure_ascii=False))
+
+    if result["valid"]:
+        _write_json(args.output, data)
+        _info("Generated refactor-list written to {}".format(args.output))
+        return 0
+    else:
+        _err("Validation failed with {} error(s)".format(len(result["errors"])))
+        for e in result["errors"]:
+            _err("  " + e)
+        for w in result.get("warnings", []):
+            _warn("  " + w)
+        return 1
+
+
 def cmd_summary(args):
     """Handle the 'summary' command."""
     if not args.input:
@@ -719,6 +777,7 @@ def main():
             "  %(prog)s validate --input .prizmkit/plans/refactor-list.json\n"
             "  %(prog)s validate --input .prizmkit/plans/refactor-list.json --output validated.json\n"
             "  %(prog)s template --output .prizmkit/plans/refactor-list.json\n"
+            "  %(prog)s generate --input draft.json --output .prizmkit/plans/refactor-list.json\n"
             "  %(prog)s summary --input .prizmkit/plans/refactor-list.json\n"
             "  %(prog)s summary --input .prizmkit/plans/refactor-list.json --format json\n"
         ),
@@ -747,6 +806,18 @@ def main():
         "--output", required=True, help="Path to write template file"
     )
 
+    # -- generate --
+    p_generate = subparsers.add_parser(
+        "generate",
+        help="Validate a draft and generate final refactor-list.json with defaults",
+    )
+    p_generate.add_argument(
+        "--input", required=True, help="Path to draft JSON (or '-' for stdin)"
+    )
+    p_generate.add_argument(
+        "--output", required=True, help="Path to write final refactor-list.json"
+    )
+
     # -- summary --
     p_summary = subparsers.add_parser(
         "summary",
@@ -771,6 +842,7 @@ def main():
     dispatch = {
         "validate": cmd_validate,
         "template": cmd_template,
+        "generate": cmd_generate,
         "summary": cmd_summary,
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prizmkit",
-  "version": "1.1.47",
+  "version": "1.1.49",
   "description": "Create a new PrizmKit-powered project with clean initialization — no framework dev files, just what you need.",
   "type": "module",
   "bin": {