fix(app-test-cases): correct decisions-log phase tag (3-commcare, not 6-qa-and-training)

.claude-plugin/marketplace.json

@@ -6,13 +6,13 @@
     "url": "https://github.com/jjackson"
   },
   "metadata": {
-    "version": "0.13.327"
+    "version": "0.13.328"
   },
   "plugins": [
     {
       "name": "ace",
       "source": "./",
-      "version": "0.13.327",
+      "version": "0.13.328",
       "description": "AI Connect Engine — orchestrates the CRISPR-Connect lifecycle from idea through app building, Connect setup, LLO management, and closeout"
     }
   ]

.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "ace",
-  "version": "0.13.327",
+  "version": "0.13.328",
   "description": "AI Connect Engine — orchestrates the CRISPR-Connect lifecycle from idea through app building, Connect setup, LLO management, and closeout",
   "author": {
     "name": "Jonathan Jackson",

VERSION

@@ -1 +1 @@
-0.13.327
+0.13.328

agents/commcare-setup.md

@@ -281,8 +281,7 @@ pre-flight (real failure mode from turmeric run 20260509-0455). See
 - Reads: pdd-to-app-journeys.md, both app summaries, Nova blueprints
 - Writes:
   - app-test-cases.yaml + recipes/J*.yaml under app-test-cases/
-  - Appended `test-scenario-count`, `test-archetype-coverage` rows in `decisions.yaml` (merge-only; bar criterion per `skills/idea-to-pdd/SKILL.md § Decisions Log Convention`).
-    Note: `app-test-cases/SKILL.md` records these rows with `phase: 6-qa-and-training` even though dispatch happens here in Phase 3. That's an upstream `phase:` mismatch in the SKILL.md (see issue #399 follow-up); don't paper over it on the writer side.
+  - Appended `test-scenario-count`, `test-archetype-coverage` rows in `decisions.yaml` (merge-only; rows use `phase: 3-commcare` matching this dispatch site; bar criterion per `skills/idea-to-pdd/SKILL.md § Decisions Log Convention`).
 - Halts on missing inputs or recipe-validation failure
 
 Phase 6 shallow runs the smoke recipes; /ace:qa-deep runs them all.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ace",
-  "version": "0.13.327",
+  "version": "0.13.328",
   "description": "AI Connect Engine - orchestrator for building Connect Opps using AI",
   "type": "module",
   "scripts": {

skills/app-test-cases/SKILL.md

@@ -526,7 +526,7 @@ qualify under the bar for this phase — a working template, not a
 required set. The skill applies the bar criterion and emits whatever
 rows meet it; the catalog is a teaching device that improves over time.
 
-### Common load-bearing decisions for Phase 6
+### Common load-bearing decisions for Phase 3
 
 | ID | Question | Map to surface |
 |---|---|---|
@@ -538,13 +538,20 @@ The orchestrator's Phase Write-Back Verifier (`agents/ace-orchestrator.md`
 contract; the renderer (`skills/decisions-render`) regenerates the gdoc
 at end of every phase.
 
-Each row this skill writes uses `phase: 6-qa-and-training` and
-`skill: app-test-cases`.
+Each row this skill writes uses `phase: 3-commcare` and
+`skill: app-test-cases`. The convention is the phase the skill is
+dispatched in, not the phase its outputs are consumed in — this skill is
+dispatched from `agents/commcare-setup.md § Step 2.6` even though
+`app-test-cases.yaml` + per-journey recipes are consumed by Phase 6's
+`app-screenshot-capture`. Aligns with the artifact manifest, which
+already maps the producer to `3-commcare/` (see
+`lib/artifact-manifest.ts`).
 
 ## Change log
 
 | Date | Change | Author |
 |------|--------|--------|
 | 2026-05-04 | Initial version. Phase 3 producer for app-test-cases.yaml; binds pdd-to-app-journeys.md to Nova-built structure with Maestro recipe stubs. Successor to qa-plan (retired in same release). | ACE team |
 | 2026-05-08 | Add `## Decisions Log` section: 2 anchor rows (test-scenario-count, test-archetype-coverage) + bar-criterion reference. Pairs with decisions-log PR #4 (Phase 3-10 writes). | ACE team (decisions-log PR #4) |
+| 2026-05-22 | Fix `phase:` tag in Decisions Log footer: was `6-qa-and-training` (the consuming phase), now `3-commcare` (the dispatching phase, matching the artifact manifest's existing `3-commcare/` path mapping). Follow-up to issue #399. | ACE team |
 | 2026-05-12 | Add Step 3.4 — recipe-wide `mobile_resolve_selectors` gate. Halts `[BLOCKER]` on any unresolved logical selector before recipes are written to Drive. Shifts left a class of Phase 6 blockers (leep + turmeric runs both hit this in early May) to Phase 3, where Nova form/field context is still in-scope. Follows PR #249's `connect-2.62.0.yaml` calibration. | ACE team |