nova-spec 1.0.2 → 1.0.4

package/novaspec/guardrails/review-checks.sh

@@ -0,0 +1,115 @@
+#!/usr/bin/env bash
+# Guardrail: deterministic pre-review checks.
+#
+# Usage: bash novaspec/guardrails/review-checks.sh <ticket-id> [base-branch]
+#
+# Runs (in order):
+#   1. diff is non-empty (committed + working tree)
+#   2. every "Files to touch" entry from tasks.md appears in the diff
+#   3. lint clean (if `npm run lint` / `pnpm lint` / `yarn lint` exists)
+#   4. tests pass (if `npm test` / `pnpm test` / `yarn test` exists)
+#
+# Exits 0 if every applicable check passes, 1 if any blocking check fails,
+# 2 on usage error. Skipped checks (e.g. no lint script) are reported but
+# don't fail.
+
+set -uo pipefail
+
+if [ -z "${1:-}" ]; then
+  echo "Usage: $0 <ticket-id> [base-branch]" >&2
+  exit 2
+fi
+
+ticket="$1"
+base="${2:-main}"
+fail=0
+
+# 1. Diff non-empty
+diff_total="$(git diff "$base"...HEAD 2>/dev/null; git diff HEAD 2>/dev/null)"
+if [ -z "$diff_total" ]; then
+  echo "✗ Empty diff: no committed or staged changes against $base."
+  fail=1
+else
+  echo "✓ Diff is non-empty."
+fi
+
+# 2. Files to touch
+tasks="context/changes/active/${ticket}/tasks.md"
+if [ -f "$tasks" ]; then
+  # Pull every path under a "Files to touch" section. We look for lines that
+  # look like list items containing a path-ish token, but only between that
+  # heading and the next heading.
+  declared="$(awk '
+    /^#+ +Files to touch/i, /^#+ +/ {
+      if (NR > 1 && /^#+ +/ && !/^#+ +Files to touch/i) next_section=1
+      if (next_section) next
+      if (match($0, /[`"]?([\.a-zA-Z0-9_\/-]+\.[a-zA-Z0-9]+)[`"]?/, m)) print m[1]
+    }
+  ' "$tasks" | sort -u)"
+
+  if [ -n "$declared" ]; then
+    missing=""
+    while IFS= read -r f; do
+      [ -z "$f" ] && continue
+      if ! git diff "$base"...HEAD --name-only | grep -qxF "$f" \
+         && ! git diff HEAD --name-only | grep -qxF "$f"; then
+        missing+="   - $f"$'\n'
+      fi
+    done <<< "$declared"
+
+    if [ -n "$missing" ]; then
+      echo "✗ Files declared in tasks.md but missing from diff:"
+      printf '%s' "$missing"
+      fail=1
+    else
+      echo "✓ All declared files present in diff."
+    fi
+  else
+    echo "ℹ︎ tasks.md has no 'Files to touch' section — skipping declared-files check."
+  fi
+else
+  echo "ℹ︎ No tasks.md (quick-fix path) — skipping declared-files check."
+fi
+
+# 3 + 4. Lint and test
+detect_pm() {
+  if [ -f pnpm-lock.yaml ]; then echo pnpm
+  elif [ -f yarn.lock ]; then echo yarn
+  elif [ -f package-lock.json ] || [ -f package.json ]; then echo npm
+  else echo ''
+  fi
+}
+
+has_script() {
+  local pm="$1" script="$2"
+  if [ ! -f package.json ]; then return 1; fi
+  node -e "process.exit(((require('./package.json').scripts||{})['$script'])?0:1)" 2>/dev/null
+}
+
+pm="$(detect_pm)"
+
+if [ -n "$pm" ] && has_script "$pm" lint; then
+  echo "Running $pm run lint…"
+  if "$pm" run lint --silent >/dev/null 2>&1; then
+    echo "✓ Lint clean."
+  else
+    echo "✗ Lint failed. Run \`$pm run lint\` and fix before review."
+    fail=1
+  fi
+else
+  echo "ℹ︎ No lint script — skipping."
+fi
+
+if [ -n "$pm" ] && has_script "$pm" test; then
+  echo "Running $pm test…"
+  if "$pm" test --silent >/dev/null 2>&1; then
+    echo "✓ Tests pass."
+  else
+    echo "✗ Tests failed. Run \`$pm test\` and fix before review."
+    fail=1
+  fi
+else
+  echo "ℹ︎ No test script — skipping."
+fi
+
+[ "$fail" -eq 0 ]

package/novaspec/templates/ticket-summary.md

@@ -0,0 +1,14 @@
+## Ticket: <TICKET-ID> — "<title>"
+
+Classification    : <quick-fix | feature | architecture> (<estimated effort>)
+Affected services : <svc-1> ✓, <svc-2> ✓
+Branch created    : <type>/<TICKET>-<slug> (from <base>)
+
+Loaded context:
+  Services      : <names with ✓ if `context/services/<name>.md` exists, ✗ otherwise>
+  Decisions read: <list of decision filenames, or "none">
+  Gotchas read  : <list, or "none">
+  Gaps          : <list of missing context, or "none">
+  Questions     : <open questions for the user, or "none">
+
+Next step: <next command, e.g. `/nova-spec` or `/nova-build`>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nova-spec",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Spec-Driven Development framework for Claude Code and OpenCode",
   "bin": {
     "nova-spec": "bin/nova-spec.js"
@@ -8,11 +8,18 @@
   "files": [
     "bin/",
     "lib/",
-    "novaspec/",
-    "AGENTS.md"
+    "novaspec/agents/",
+    "novaspec/commands/",
+    "novaspec/guardrails/",
+    "novaspec/skills/",
+    "novaspec/templates/",
+    "novaspec/config.example.yml",
+    "AGENTS.md",
+    "INSTALL.md",
+    "PHILOSOPHY.md"
   ],
   "scripts": {
-    "test": "echo \"No automated tests — use smoke test in CLAUDE.md\""
+    "test": "node test/smoke.test.js"
   },
   "dependencies": {
     "@inquirer/prompts": "^7.0.0"

package/novaspec/.nova-manifest.json

@@ -1,30 +0,0 @@
-{
-  "version": "1.0.1",
-  "generated_at": "2026-05-10T07:46:23.881Z",
-  "hashes": {
-    "commands/nova-build": "5d9c103168f97dd60e4efbe1f9a277a7",
-    "commands/nova-diff": "529389d8d7d187c70de708610101009c",
-    "commands/nova-plan": "44709e96a49e86d808f3cad94c0bdc86",
-    "commands/nova-review": "aa2e4e91fe5e774ae7f374c7f3bf8882",
-    "commands/nova-spec": "726194e235d566dfbb81e785a6437b46",
-    "commands/nova-start": "d031e1958cf769864f566e9d768b1552",
-    "commands/nova-status": "cf0b7db18d4290479c35ed3b87fecc56",
-    "commands/nova-sync": "4364efbb594eeddaff7deba8d7e2a947",
-    "commands/nova-wrap": "846dfee6faaba65d80c082bb90e9b306",
-    "skills/close-requirement": {
-      "SKILL.md": "43d5f32320a635b15601a21ef5a0bf93"
-    },
-    "skills/jira-integration": {
-      "SKILL.md": "513f8b116be4bed955a8e3631bbf250c"
-    },
-    "skills/update-service-context": {
-      "SKILL.md": "b0788661cc61e2c6acb4bead24f48017"
-    },
-    "skills/write-decision": {
-      "SKILL.md": "20ea0ac9f19ec17a72d562f811c4a61e"
-    },
-    "agents/context-loader": "42a4cf797452bd76c720bd8f36e261f4",
-    "agents/nova-review-agent": "5658926243d4ce01cfc3b79ee8e4ad93"
-  },
-  "outdated_customs": []
-}

package/novaspec/config.yml

@@ -1,23 +0,0 @@
-# nova-spec — project configuration
-# This file is gitignored — do not push it to the repo.
-
-branch:
-  pattern: "{type}/{ticket}-{slug}"
-  types:
-    bugfix: bugfix
-    hotfix: hotfix
-    feature: feature
-    documentation: docs
-    refactor: refactor
-    chore: chore
-    architecture: arch
-  ticket_case: upper
-  base: main
-
-jira:
-  skill: "jira-integration"
-  url: https://your-workspace.atlassian.net
-  project: PROJ
-  email: 
-  token: ${JIRA_API_TOKEN}
-  done_transition_id: "41"