mindsystem-cc 4.2.1 → 4.3.1

package/mindsystem/workflows/transition.md

@@ -97,7 +97,6 @@ Update the file:
 
 - Mark current phase: `[x] Complete`
 - Add completion date
-- Update plan count to final (e.g., "3/3 plans complete")
 - Update Progress table
 - Keep next phase as `[ ] Not started`
 
@@ -112,11 +111,11 @@ Update the file:
 
 ## Progress
 
-| Phase             | Plans Complete | Status      | Completed  |
-| ----------------- | -------------- | ----------- | ---------- |
-| 1. Foundation     | 3/3            | Complete    | 2025-01-15 |
-| 2. Authentication | 0/2            | Not started | -          |
-| 3. Core Features  | 0/1            | Not started | -          |
+| Phase             | Status      | Completed  |
+| ----------------- | ----------- | ---------- |
+| 1. Foundation     | Complete    | 2025-01-15 |
+| 2. Authentication | Not started | -          |
+| 3. Core Features  | Not started | -          |
 ```
 
 </step>
@@ -442,7 +441,6 @@ Respect user judgment — they know if work matters.
 
 **If marking complete with incomplete plans:**
 
-- Update ROADMAP: "2/3 plans complete" (not "3/3")
 - Note in transition message which plans were skipped
 
 </partial_completion>
@@ -453,7 +451,7 @@ Transition is complete when:
 
 - [ ] Current phase plan summaries verified (all exist or user chose to skip)
 - [ ] Any stale handoffs deleted
-- [ ] ROADMAP.md updated with completion status and plan count
+- [ ] ROADMAP.md updated with completion status
 - [ ] PROJECT.md evolved (requirements, decisions, description if needed)
 - [ ] STATE.md updated (position, project reference, context, session)
 - [ ] Progress table updated

package/mindsystem/workflows/verify-work.md

@@ -6,6 +6,7 @@ Complete verify-and-fix session: by session end, everything verified, issues fix
 
 <execution_context>
 <!-- mock-patterns.md loaded on demand for transient_state mocks (see generate_mocks step) -->
+<!-- knowledge files loaded on first fix attempt (see investigate_issue step) -->
 </execution_context>
 
 <process>
@@ -74,7 +75,8 @@ Provide a phase number to start testing (e.g., /ms:verify-work 4)
 **Find SUMMARY.md files for the phase:**
 
 ```bash
-PHASE_DIR=$(ls -d .planning/phases/${PHASE_ARG}* 2>/dev/null | head -1)
+ms-tools find-phase "${PHASE_ARG}"
+# Extract PHASE_DIR from the `dir` field of the JSON output
 ls "$PHASE_DIR"/*-SUMMARY.md 2>/dev/null
 ```
 
@@ -341,6 +343,19 @@ Progress auto-recalculates on every `uat-update` call. No manual progress recalc
 <step name="investigate_issue">
 **Investigate reported issue:**
 
+**0. Load knowledge (first issue only):**
+
+If this is the first issue requiring investigation in this session, load knowledge files now. On subsequent issues, knowledge is already in context — skip to step 1.
+
+```bash
+ms-tools config-get subsystems
+grep "^subsystem:" "$PHASE_DIR"/*-SUMMARY.md 2>/dev/null
+```
+
+Match SUMMARY.md subsystem values against config subsystems. Read matching `.planning/knowledge/{subsystem}.md` files (1-3 most relevant).
+
+This knowledge informs investigation (where to look, known pitfalls) and any subsequent fixes.
+
 **1. Lightweight investigation (2-3 tool calls):**
 
 ```
@@ -350,24 +365,21 @@ Progress auto-recalculates on every `uat-update` call. No manual progress recalc
 3. Check git log for recent changes to relevant files
 ```
 
-**2. Determine if cause is found:**
+**2. Report finding and act:**
 
-**If cause found AND fix is simple (single file, straightforward change):**
-- Propose fix in plain language:
-  ```
-  Found the issue. In `ErrorBanner.tsx` line 42, the error message is
-  hardcoded to "Something went wrong" instead of using the actual error
-  from the API response.
+State the root cause and the fix in plain language before applying. This lets the user capture the lesson in persistent instructions or catch a wrong direction before reviewing the diff. No confirmation needed — go straight to fixing.
 
-  I'll change it to use `error.message` from props.
+Format:
+```
+{Root cause — what the code does wrong and why, 1-2 sentences}
 
-  Apply this fix?
-  ```
-- Present options: [Yes / Let me see the code first / Different approach]
-- If approved: Go to `apply_fix`
+Fix: {what the fix changes, 1-2 sentences}
+```
 
-**If cause found BUT fix is complex (multiple files, architectural):**
-- Report finding
+**If fix is simple (single file, straightforward change):**
+- Go to `apply_fix` immediately
+
+**If fix is complex (multiple files, architectural):**
 - Spawn ms-verify-fixer subagent (go to `escalate_to_fixer`)
 
 **If cause NOT found after 2-3 checks:**
@@ -440,6 +452,10 @@ You are a Mindsystem verify-fixer. Investigate this issue, find the root cause,
 
 {lightweight_investigation_results}
 
+## Knowledge Context
+
+{loaded_knowledge_content or "No knowledge files loaded for this session."}
+
 ## Your task
 
 1. Investigate to find root cause

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mindsystem-cc",
-  "version": "4.2.1",
+  "version": "4.3.1",
   "description": "The engineer's meta-prompting system for Claude Code.",
   "bin": {
     "mindsystem-cc": "bin/install.js"

package/scripts/ms-tools.py

@@ -1198,6 +1198,7 @@ def cmd_doctor_scan(args: argparse.Namespace) -> None:
 
     # ---- CHECK 11: Browser Verification ----
     print("=== Browser Verification ===")
+    is_web, web_signal = False, "not checked"
     bv_enabled = True
     bv_config = config.get("browser_verification", {})
     if isinstance(bv_config, dict):
@@ -1233,6 +1234,28 @@ def cmd_doctor_scan(args: argparse.Namespace) -> None:
                 record("PASS", "Browser Verification")
     print()
 
+    # ---- CHECK 12: Screenshot Optimization ----
+    print("=== Screenshot Optimization ===")
+    if not bv_enabled:
+        print("Status: SKIP")
+        print("Browser verification disabled")
+        record("SKIP", "Screenshot Optimization")
+    elif not is_web:
+        print("Status: SKIP")
+        print(f"Not a web project ({web_signal})")
+        record("SKIP", "Screenshot Optimization")
+    else:
+        if shutil.which("cwebp"):
+            print("Status: PASS")
+            print("cwebp available — screenshots will be converted to WebP")
+            record("PASS", "Screenshot Optimization")
+        else:
+            print("Status: WARN")
+            print("cwebp not found — browser screenshots will remain as PNG (larger files)")
+            print("Install: brew install webp | apt install webp | choco install webp")
+            record("WARN", "Screenshot Optimization")
+    print()
+
     # ---- SUMMARY ----
     total = pass_count + warn_count + fail_count + skip_count
     print("=== Summary ===")
@@ -3126,7 +3149,7 @@ def _load_uat(args_phase: str) -> tuple[Path, "UATFile"]:
     if phase_dir is None:
         print(f"Error: Phase directory not found for {phase}", file=sys.stderr)
         sys.exit(1)
-    uat_path = phase_dir / f"{phase_dir.name}-UAT.md"
+    uat_path = phase_dir / f"{phase}-UAT.md"
     if not uat_path.is_file():
         print(f"Error: UAT file not found: {uat_path}", file=sys.stderr)
         sys.exit(1)
@@ -3166,7 +3189,7 @@ def cmd_uat_init(args: argparse.Namespace) -> None:
     phase_name = phase_dir.name
     uat = UATFile.from_init_json(data, phase_name)
 
-    out_path = phase_dir / f"{phase_name}-UAT.md"
+    out_path = phase_dir / f"{phase}-UAT.md"
     out_path.write_text(uat.serialize(), encoding="utf-8")
 
     n_tests = len(uat.tests)
@@ -3196,7 +3219,7 @@ def cmd_uat_update(args: argparse.Namespace) -> None:
         print(f"Error: Phase directory not found for {phase}", file=sys.stderr)
         sys.exit(1)
 
-    uat_path = phase_dir / f"{phase_dir.name}-UAT.md"
+    uat_path = phase_dir / f"{phase}-UAT.md"
     if not uat_path.is_file():
         print(f"Error: UAT file not found: {uat_path}", file=sys.stderr)
         sys.exit(1)
@@ -3264,7 +3287,7 @@ def cmd_uat_status(args: argparse.Namespace) -> None:
         print(f"Error: Phase directory not found for {phase}", file=sys.stderr)
         sys.exit(1)
 
-    uat_path = phase_dir / f"{phase_dir.name}-UAT.md"
+    uat_path = phase_dir / f"{phase}-UAT.md"
     if not uat_path.is_file():
         print(f"Error: UAT file not found: {uat_path}", file=sys.stderr)
         sys.exit(1)