@rpamis/comet 0.2.1 → 0.2.2

package/assets/skills/comet/scripts/comet-yaml-validate.sh

@@ -1,132 +1,132 @@
-#!/bin/bash
-# Comet YAML Schema Validator — validates .comet.yaml structure
-# Usage: comet-yaml-validate.sh <change-name>
-# Exit 0 = valid, exit 1 = errors found (printed to stderr)
-
-set -euo pipefail
-
-red()   { echo -e "\033[31m$1\033[0m" >&2; }
-green() { echo -e "\033[32m$1\033[0m" >&2; }
-warn()  { echo -e "\033[33m$1\033[0m" >&2; }
-
-# Input validation - prevent path traversal
-validate_change_name() {
-  local name="$1"
-  # Reject empty names
-  if [ -z "$name" ]; then
-    red "ERROR: Change name cannot be empty" >&2
-    exit 1
-  fi
-  # Only allow alphanumeric, hyphens, and underscores
-  if [[ ! "$name" =~ ^[a-zA-Z0-9_-]+$ ]]; then
-    red "ERROR: Invalid change name: '$name'" >&2
-    red "Valid characters: a-z, A-Z, 0-9, -, _" >&2
-    exit 1
-  fi
-  # Reject path traversal attempts
-  if [[ "$name" =~ \.\. ]]; then
-    red "ERROR: Change name cannot contain '..' (path traversal not allowed)" >&2
-    exit 1
-  fi
-}
-
-validate_change_name "$1"
-
-CHANGE="$1"
-YAML="openspec/changes/$CHANGE/.comet.yaml"
-
-ERRORS=0
-WARNINGS=0
-
-# Helper: get value of a top-level field (handles null, empty, quoted)
-field_value() {
-  grep "^${1}:" "$YAML" 2>/dev/null | sed "s/^${1}: *//" | tr -d '"' | tr -d "'" || true
-}
-
-fail()  { red "  FAIL: $1"; ERRORS=$((ERRORS + 1)); }
-warn_msg() { warn "  WARN: $1"; WARNINGS=$((WARNINGS + 1)); }
-
-echo "[VALIDATE] $YAML" >&2
-
-# --- Required fields ---
-REQUIRED_FIELDS="workflow phase design_doc plan build_mode isolation verify_mode verify_result verified_at archived"
-for field in $REQUIRED_FIELDS; do
-  if ! grep -q "^${field}:" "$YAML" 2>/dev/null; then
-    fail "missing required field '$field'"
-  fi
-done
-
-# --- Enum validation ---
-validate_enum() {
-  local field="$1" value="$2"
-  shift 2
-  local valid_values="$*"
-
-  # null or empty is always acceptable
-  if [ -z "$value" ] || [ "$value" = "null" ]; then
-    return 0
-  fi
-
-  for v in $valid_values; do
-    if [ "$value" = "$v" ]; then
-      return 0
-    fi
-  done
-  fail "$field='$value' is not valid. Expected: $valid_values"
-}
-
-workflow=$(field_value "workflow")
-phase=$(field_value "phase")
-build_mode=$(field_value "build_mode")
-isolation=$(field_value "isolation")
-verify_mode=$(field_value "verify_mode")
-verify_result=$(field_value "verify_result")
-archived=$(field_value "archived")
-design_doc=$(field_value "design_doc")
-plan=$(field_value "plan")
-
-validate_enum "workflow"      "$workflow"      "full hotfix tweak"
-validate_enum "phase"         "$phase"          "design build verify archive"
-validate_enum "build_mode"    "$build_mode"     "subagent-driven-development executing-plans direct"
-validate_enum "isolation"     "$isolation"      "branch worktree"
-validate_enum "verify_mode"   "$verify_mode"    "light full"
-validate_enum "verify_result" "$verify_result"  "pending pass fail"
-validate_enum "archived"      "$archived"       "true false"
-
-# --- Path validation ---
-
-if [ -n "$design_doc" ] && [ "$design_doc" != "null" ]; then
-  if [ ! -f "$design_doc" ]; then
-    fail "design_doc='$design_doc' does not exist on disk"
-  fi
-fi
-
-if [ -n "$plan" ] && [ "$plan" != "null" ]; then
-  if [ ! -f "$plan" ]; then
-    fail "plan='$plan' does not exist on disk"
-  fi
-fi
-
-# --- Unknown keys check ---
-KNOWN_KEYS="workflow phase design_doc plan build_mode isolation verify_mode verify_result verified_at archived"
-while IFS=: read -r key _; do
-  key="${key// /}"
-  [ -z "$key" ] && continue
-  found=0
-  for known in $KNOWN_KEYS; do
-    [ "$key" = "$known" ] && found=1 && break
-  done
-  if [ "$found" -eq 0 ]; then
-    warn_msg "unknown field '$key' found"
-  fi
-done < "$YAML"
-
-# --- Summary ---
-echo "" >&2
-if [ "$ERRORS" -gt 0 ]; then
-  red "$ERRORS error(s), $WARNINGS warning(s) — validation FAILED"
-  exit 1
-else
-  green "0 errors, $WARNINGS warning(s) — validation PASSED"
-  exit 0
-fi
+#!/bin/bash
+# Comet YAML Schema Validator — validates .comet.yaml structure
+# Usage: comet-yaml-validate.sh <change-name>
+# Exit 0 = valid, exit 1 = errors found (printed to stderr)
+
+set -euo pipefail
+
+red()   { echo -e "\033[31m$1\033[0m" >&2; }
+green() { echo -e "\033[32m$1\033[0m" >&2; }
+warn()  { echo -e "\033[33m$1\033[0m" >&2; }
+
+# Input validation - prevent path traversal
+validate_change_name() {
+  local name="$1"
+  # Reject empty names
+  if [ -z "$name" ]; then
+    red "ERROR: Change name cannot be empty" >&2
+    exit 1
+  fi
+  # Only allow alphanumeric, hyphens, and underscores
+  if [[ ! "$name" =~ ^[a-zA-Z0-9_-]+$ ]]; then
+    red "ERROR: Invalid change name: '$name'" >&2
+    red "Valid characters: a-z, A-Z, 0-9, -, _" >&2
+    exit 1
+  fi
+  # Reject path traversal attempts
+  if [[ "$name" =~ \.\. ]]; then
+    red "ERROR: Change name cannot contain '..' (path traversal not allowed)" >&2
+    exit 1
+  fi
+}
+
+validate_change_name "$1"
+
+CHANGE="$1"
+YAML="openspec/changes/$CHANGE/.comet.yaml"
+
+ERRORS=0
+WARNINGS=0
+
+# Helper: get value of a top-level field (handles null, empty, quoted)
+field_value() {
+  grep "^${1}:" "$YAML" 2>/dev/null | sed "s/^${1}: *//" | tr -d '"' | tr -d "'" || true
+}
+
+fail()  { red "  FAIL: $1"; ERRORS=$((ERRORS + 1)); }
+warn_msg() { warn "  WARN: $1"; WARNINGS=$((WARNINGS + 1)); }
+
+echo "[VALIDATE] $YAML" >&2
+
+# --- Required fields ---
+REQUIRED_FIELDS="workflow phase design_doc plan build_mode isolation verify_mode verify_result verified_at archived"
+for field in $REQUIRED_FIELDS; do
+  if ! grep -q "^${field}:" "$YAML" 2>/dev/null; then
+    fail "missing required field '$field'"
+  fi
+done
+
+# --- Enum validation ---
+validate_enum() {
+  local field="$1" value="$2"
+  shift 2
+  local valid_values="$*"
+
+  # null or empty is always acceptable
+  if [ -z "$value" ] || [ "$value" = "null" ]; then
+    return 0
+  fi
+
+  for v in $valid_values; do
+    if [ "$value" = "$v" ]; then
+      return 0
+    fi
+  done
+  fail "$field='$value' is not valid. Expected: $valid_values"
+}
+
+workflow=$(field_value "workflow")
+phase=$(field_value "phase")
+build_mode=$(field_value "build_mode")
+isolation=$(field_value "isolation")
+verify_mode=$(field_value "verify_mode")
+verify_result=$(field_value "verify_result")
+archived=$(field_value "archived")
+design_doc=$(field_value "design_doc")
+plan=$(field_value "plan")
+
+validate_enum "workflow"      "$workflow"      "full hotfix tweak"
+validate_enum "phase"         "$phase"          "design build verify archive"
+validate_enum "build_mode"    "$build_mode"     "subagent-driven-development executing-plans direct"
+validate_enum "isolation"     "$isolation"      "branch worktree"
+validate_enum "verify_mode"   "$verify_mode"    "light full"
+validate_enum "verify_result" "$verify_result"  "pending pass fail"
+validate_enum "archived"      "$archived"       "true false"
+
+# --- Path validation ---
+
+if [ -n "$design_doc" ] && [ "$design_doc" != "null" ]; then
+  if [ ! -f "$design_doc" ]; then
+    fail "design_doc='$design_doc' does not exist on disk"
+  fi
+fi
+
+if [ -n "$plan" ] && [ "$plan" != "null" ]; then
+  if [ ! -f "$plan" ]; then
+    fail "plan='$plan' does not exist on disk"
+  fi
+fi
+
+# --- Unknown keys check ---
+KNOWN_KEYS="workflow phase design_doc plan build_mode isolation verify_mode verify_result verified_at archived"
+while IFS=: read -r key _; do
+  key="${key// /}"
+  [ -z "$key" ] && continue
+  found=0
+  for known in $KNOWN_KEYS; do
+    [ "$key" = "$known" ] && found=1 && break
+  done
+  if [ "$found" -eq 0 ]; then
+    warn_msg "unknown field '$key' found"
+  fi
+done < "$YAML"
+
+# --- Summary ---
+echo "" >&2
+if [ "$ERRORS" -gt 0 ]; then
+  red "$ERRORS error(s), $WARNINGS warning(s) — validation FAILED"
+  exit 1
+else
+  green "0 errors, $WARNINGS warning(s) — validation PASSED"
+  exit 0
+fi

package/assets/skills/comet-archive/SKILL.md

@@ -1,63 +1,63 @@
----
-name: comet-archive
-description: "Comet Phase 5: Archive. Invoke with /comet-archive. Sync delta spec to main spec, archive change."
----
-
-# Comet Phase 5: Archive (Archive)
-
-## Prerequisites
-
-- Verification passed (Phase 4 complete)
-- Branch handled
-- `verify_result: pass` in `openspec/changes/<name>/.comet.yaml`
-
-## Steps
-
-### 0. Entry State Verification (Entry Check)
-
-Execute entry verification:
-
-```bash
-COMET_STATE="${COMET_STATE:-$(find . -path '*/comet/scripts/comet-state.sh' -type f -print -quit)}"
-bash "$COMET_STATE" check <name> archive
-```
-
-Proceed to Step 1 after verification passes. The script outputs specific failure reasons when verification fails.
-
-### 1. Execute Archive
-
-Run the archive script to automatically complete all steps:
-
-```bash
-COMET_ARCHIVE="${COMET_ARCHIVE:-$(find . -path '*/comet/scripts/comet-archive.sh' -type f -print -quit)}"
-bash "$COMET_ARCHIVE" "<change-name>"
-```
-
-The script automatically executes:
-1. Entry state validation (phase=archive, verify_result=pass, archived=false)
-2. Delta spec sync to main spec (overwrite)
-3. Design doc frontmatter annotation (archived-with, status)
-4. Plan frontmatter annotation (archived-with)
-5. Move change to archive directory
-6. Update archived: true
-
-If script returns non-zero exit code, report error and stop.
-If script returns zero exit code, archive is complete.
-
-Use `--dry-run` flag to preview without executing.
-
-### 2. Lifecycle Closed Loop
-
-Spec lifecycle completes here:
-```
-brainstorming → delta spec → implementation → verification → main spec overwrite → design doc annotation → archive
-```
-
-## Exit Conditions
-
-- Archive script executed successfully (exit code 0)
-- **Phase guard**: Run `bash $COMET_GUARD <change-name> archive`, confirm archive complete after all PASS
-
-## Complete
-
-Comet workflow complete. To start new work, invoke `/comet` or `/comet-open`.
+---
+name: comet-archive
+description: "Comet Phase 5: Archive. Invoke with /comet-archive. Sync delta spec to main spec, archive change."
+---
+
+# Comet Phase 5: Archive (Archive)
+
+## Prerequisites
+
+- Verification passed (Phase 4 complete)
+- Branch handled
+- `verify_result: pass` in `openspec/changes/<name>/.comet.yaml`
+
+## Steps
+
+### 0. Entry State Verification (Entry Check)
+
+Execute entry verification:
+
+```bash
+COMET_STATE="${COMET_STATE:-$(find . -path '*/comet/scripts/comet-state.sh' -type f -print -quit)}"
+bash "$COMET_STATE" check <name> archive
+```
+
+Proceed to Step 1 after verification passes. The script outputs specific failure reasons when verification fails.
+
+### 1. Execute Archive
+
+Run the archive script to automatically complete all steps:
+
+```bash
+COMET_ARCHIVE="${COMET_ARCHIVE:-$(find . -path '*/comet/scripts/comet-archive.sh' -type f -print -quit)}"
+bash "$COMET_ARCHIVE" "<change-name>"
+```
+
+The script automatically executes:
+1. Entry state validation (phase=archive, verify_result=pass, archived=false)
+2. Delta spec sync to main spec (overwrite)
+3. Design doc frontmatter annotation (archived-with, status)
+4. Plan frontmatter annotation (archived-with)
+5. Move change to archive directory
+6. Update archived: true
+
+If script returns non-zero exit code, report error and stop.
+If script returns zero exit code, archive is complete.
+
+Use `--dry-run` flag to preview without executing.
+
+### 2. Lifecycle Closed Loop
+
+Spec lifecycle completes here:
+```
+brainstorming → delta spec → implementation → verification → main spec overwrite → design doc annotation → archive
+```
+
+## Exit Conditions
+
+- Archive script executed successfully (exit code 0)
+- **Phase guard**: Run `bash $COMET_GUARD <change-name> archive`, confirm archive complete after all PASS
+
+## Complete
+
+Comet workflow complete. To start new work, invoke `/comet` or `/comet-open`.