Merge pull request TheBushidoCollective#17 from TheBushidoCollective/hampton/fix-status-persistence

fix(status): commit unit/intent status changes to git

Author: jwaldrip

plugin/skills/advance/SKILL.md

@@ -70,6 +70,9 @@ INTENT_DIR=".ai-dlc/${INTENT_SLUG}"
 CURRENT_UNIT=$(echo "$ITERATION_JSON" | han parse json currentUnit -r --default "")
 if [ -n "$CURRENT_UNIT" ] && [ -f "$INTENT_DIR/${CURRENT_UNIT}.md" ]; then
   update_unit_status "$INTENT_DIR/${CURRENT_UNIT}.md" "completed"
+  # Commit the status change so it persists across sessions
+  git add "$INTENT_DIR/${CURRENT_UNIT}.md"
+  git commit -m "status: mark ${CURRENT_UNIT} as completed"
 fi
 ```
 
@@ -217,9 +220,19 @@ SKIP_INTEGRATOR=false
     // See Step 2e below
     return runIntegration();
   }
-  // Integration passed - Mark intent as done
+  // Integration passed or skipped - Mark intent as done
   state.status = "complete";
   // han keep save iteration.json '<updated JSON>'
+```
+
+```bash
+# Update intent.md frontmatter status so it persists in git (not just ephemeral han keep)
+han parse yaml-set status "complete" < "$INTENT_DIR/intent.md" > "$INTENT_DIR/intent.md.tmp" && mv "$INTENT_DIR/intent.md.tmp" "$INTENT_DIR/intent.md"
+git add "$INTENT_DIR/intent.md"
+git commit -m "status: mark intent ${INTENT_SLUG} as complete"
+```
+
+```javascript
   // Output completion summary (see Step 5)
   return completionSummary;
 }
@@ -312,6 +325,12 @@ Task({
 ```bash
 STATE=$(echo "$STATE" | han parse json --set "integratorComplete=true" --set "status=complete")
 han keep save iteration.json "$STATE"
+
+# Update intent.md frontmatter status so it persists in git (not just ephemeral han keep)
+han parse yaml-set status "complete" < "$INTENT_DIR/intent.md" > "$INTENT_DIR/intent.md.tmp" && mv "$INTENT_DIR/intent.md.tmp" "$INTENT_DIR/intent.md"
+git add "$INTENT_DIR/intent.md"
+git commit -m "status: mark intent ${INTENT_SLUG} as complete"
+
 # Proceed to Step 5 (completion summary)
 ```
 

plugin/skills/construct/SKILL.md

@@ -435,6 +435,9 @@ fi
 
 ```bash
 update_unit_status "$UNIT_FILE" "in_progress"
+# Commit the status change so it persists across sessions
+git add "$UNIT_FILE"
+git commit -m "status: mark $(basename "$UNIT_FILE" .md) as in_progress"
 ```
 
 3. **Resolve per-unit workflow** — read the unit's `workflow:` frontmatter field. If present, resolve it to a hat sequence. If absent, check for discipline-based defaults before falling back to the intent-level workflow:
@@ -644,7 +647,12 @@ Task({
 
 **If no next hat** (last hat in workflow -- unit complete):
 
-a. Mark unit as completed: `update_unit_status "$UNIT_FILE" "completed"`
+a. Mark unit as completed and commit:
+```bash
+update_unit_status "$UNIT_FILE" "completed"
+git add "$UNIT_FILE"
+git commit -m "status: mark $(basename "$UNIT_FILE" .md) as completed"
+```
 b. Remove unit from `unitStates`
 c. Merge or PR based on effective change strategy:
 
@@ -1068,6 +1076,9 @@ source "${CLAUDE_PLUGIN_ROOT}/lib/dag.sh"
 # Update unit status to in_progress in the intent worktree
 # UNIT_FILE points to the file in .ai-dlc/{intent-slug}/
 update_unit_status "$UNIT_FILE" "in_progress"
+# Commit the status change so it persists across sessions
+git add "$UNIT_FILE"
+git commit -m "status: mark $(basename "$UNIT_FILE" .md) as in_progress"
 ```
 
 **Track current unit in iteration state** so `/advance` knows which unit to mark completed: