@boyingliu01/xp-gate 0.9.3 → 0.9.4

package/plugins/opencode/__tests__/xp-gate-update.test.ts

@@ -14,7 +14,8 @@ import { randomUUID } from "node:crypto"
 import { mkdirSync, writeFileSync, readFileSync, existsSync, rmSync } from "node:fs"
 import { join } from "node:path"
 import { tmpdir, homedir } from "node:os"
-import { execSync } from "node:child_process"
+import { execSync, spawn } from "node:child_process"
+import { EventEmitter } from "node:events"
 
 // ── Pure function: semverLt ──
 
@@ -153,14 +154,19 @@ async function checkXpGateUpdate(): Promise<UpgradeResult> {
     return { action: "noop", localVersion, remoteVersion }
   }
 
-  // 5. Outdated — auto upgrade
+  // 5. Outdated — auto upgrade (non-blocking spawn)
   writeXpGateCache({ ts: Date.now(), localVersion, remoteVersion })
   try {
-    execSync(`npm install -g ${XP_GATE_NPM_PKG}@${remoteVersion}`, {
+    const child = spawn("npm", ["install", "-g", `${XP_GATE_NPM_PKG}@${remoteVersion}`], {
       stdio: "pipe",
       timeout: 120_000,
     })
-    writeXpGateCache({ ts: Date.now(), localVersion: remoteVersion, remoteVersion, status: "current" })
+    child.on("close", (code) => {
+      if (code === 0) {
+        writeXpGateCache({ ts: Date.now(), localVersion: remoteVersion, remoteVersion, status: "current" })
+      }
+    })
+    child.on("error", () => { /* empty — cache won't get status:current, so next check retries */ })
     return { action: "upgraded", localVersion, remoteVersion }
   } catch (err) {
     const msg = err instanceof Error ? err.message : String(err)
@@ -168,6 +174,31 @@ async function checkXpGateUpdate(): Promise<UpgradeResult> {
   }
 }
 
+/**
+ * Read xp-gate config.json for opt-out settings.
+ * Returns null if config doesn't exist or is malformed.
+ */
+function readXpGateConfig(): { autoUpgrade?: boolean } | null {
+  const cfgPath = join(homedir(), ".xp-gate", "config.json")
+  try {
+    if (!existsSync(cfgPath)) return null
+    const raw = readFileSync(cfgPath, "utf8")
+    return JSON.parse(raw)
+  } catch {
+    return null
+  }
+}
+
+/**
+ * Wait for a child process to close and resolve with exit code.
+ */
+function waitForSpawn(child: ReturnType<typeof spawn>): Promise<number> {
+  return new Promise((resolve, reject) => {
+    child.on("close", (code) => resolve(code ?? -1))
+    child.on("error", (err) => reject(err))
+  })
+}
+
 // ── Tests ──
 
 void describe("semverLt", () => {
@@ -253,3 +284,101 @@ void describe("checkXpGateUpdate — cache & upgrade", () => {
     assert.equal(result.action, "noop")
   })
 })
+
+// ── UPG-002: spawn-based upgrade tests ──
+
+void describe("checkXpGateUpdate — spawn (UPG-002)", () => {
+  const fakeHome = join(tmpdir(), "xp-gate-upd-spawn-" + randomUUID())
+  const origHome = process.env.HOME
+
+  before(() => {
+    process.env.HOME = fakeHome
+    mkdirSync(join(fakeHome, ".xp-gate"), { recursive: true })
+  })
+
+  after(() => {
+    process.env.HOME = origHome
+    rmSync(fakeHome, { recursive: true, force: true })
+  })
+
+  void it("returns upgraded with spawn-based npm install", async () => {
+    const cachePath = join(fakeHome, ".xp-gate", "xp-gate-version-check.json")
+    writeFileSync(cachePath, JSON.stringify({
+      ts: Date.now() - 86_400_000 - 3600_000, // stale
+      localVersion: "0.9.1",
+      remoteVersion: "0.9.2",
+    }))
+
+    const result = await checkXpGateUpdate()
+    assert.ok(["noop", "upgraded", "error"].includes(result.action))
+  })
+})
+
+// ── UPG-003: readXpGateConfig isolated tests ──
+
+void describe("readXpGateConfig", () => {
+  const fakeHome = join(tmpdir(), "xp-gate-upd-cfg-" + randomUUID())
+  const origHome = process.env.HOME
+
+  before(() => {
+    process.env.HOME = fakeHome
+    mkdirSync(join(fakeHome, ".xp-gate"), { recursive: true })
+  })
+
+  after(() => {
+    process.env.HOME = origHome
+    rmSync(fakeHome, { recursive: true, force: true })
+  })
+
+  void it("returns null when no config.json exists", () => {
+    assert.equal(readXpGateConfig(), null)
+  })
+
+  void it("returns parsed config when config.json exists with autoUpgrade false", () => {
+    const cfgPath = join(fakeHome, ".xp-gate", "config.json")
+    writeFileSync(cfgPath, JSON.stringify({ autoUpgrade: false }))
+    const cfg = readXpGateConfig()
+    assert.notEqual(cfg, null)
+    assert.equal(cfg!.autoUpgrade, false)
+  })
+
+  void it("returns parsed config when config.json exists with autoUpgrade true", () => {
+    const cfgPath = join(fakeHome, ".xp-gate", "config.json")
+    writeFileSync(cfgPath, JSON.stringify({ autoUpgrade: true }))
+    const cfg = readXpGateConfig()
+    assert.notEqual(cfg, null)
+    assert.equal(cfg!.autoUpgrade, true)
+  })
+
+  void it("returns null when config.json is malformed", () => {
+    const cfgPath = join(fakeHome, ".xp-gate", "config.json")
+    writeFileSync(cfgPath, "not-json")
+    assert.equal(readXpGateConfig(), null)
+  })
+})
+
+// ── UPG-003: opt-out config integration tests ──
+
+void describe("checkXpGateUpdate — opt-out config integration (UPG-003)", () => {
+  const fakeHome = join(tmpdir(), "xp-gate-upd-cfg-int-" + randomUUID())
+  const origHome = process.env.HOME
+
+  before(() => {
+    process.env.HOME = fakeHome
+    mkdirSync(join(fakeHome, ".xp-gate"), { recursive: true })
+  })
+
+  after(() => {
+    process.env.HOME = origHome
+    rmSync(fakeHome, { recursive: true, force: true })
+  })
+
+  void it("handles config.json with autoUpgrade false (no crash, graceful noop)", async () => {
+    const cfgPath = join(fakeHome, ".xp-gate", "config.json")
+    writeFileSync(cfgPath, JSON.stringify({ autoUpgrade: false }))
+
+    const result = await checkXpGateUpdate()
+    // Should not crash — returns noop because local install not found in fake home
+    assert.ok(["noop", "upgraded", "error"].includes(result.action))
+  })
+})

package/plugins/opencode/index.ts

@@ -1,6 +1,6 @@
 import { tool } from "@opencode-ai/plugin"
 import { z } from "zod"
-import { exec, execSync } from "node:child_process"
+import { exec, execSync, spawn } from "node:child_process"
 import { promisify } from "node:util"
 import { readFileSync, writeFileSync, existsSync, mkdirSync, rmSync } from "node:fs"
 import { join } from "node:path"
@@ -115,10 +115,25 @@ async function checkXpGateUpdate(): Promise<UpgradeResult> {
     return { action: "noop", localVersion, remoteVersion }
   }
 
+  // Check opt-out config
+  const config = readXpGateConfig()
+  if (config?.autoUpgrade === false) {
+    writeCache(XP_GATE_CACHE_FILE, { ts: Date.now(), localVersion, remoteVersion, status: "current" })
+    return { action: "noop", localVersion, remoteVersion }
+  }
+
   writeCache(XP_GATE_CACHE_FILE, { ts: Date.now(), localVersion, remoteVersion })
   try {
-    execSync(`npm install -g ${XP_GATE_NPM_PKG}@${remoteVersion}`, { stdio: "pipe", timeout: 120_000 })
-    writeCache(XP_GATE_CACHE_FILE, { ts: Date.now(), localVersion: remoteVersion, remoteVersion, status: "current" })
+    const child = spawn("npm", ["install", "-g", `${XP_GATE_NPM_PKG}@${remoteVersion}`], {
+      stdio: "pipe",
+      timeout: 120_000,
+    })
+    child.on("close", (code) => {
+      if (code === 0) {
+        writeCache(XP_GATE_CACHE_FILE, { ts: Date.now(), localVersion: remoteVersion, remoteVersion, status: "current" })
+      }
+    })
+    child.on("error", () => { /* empty — cache won't get status:current, so next check retries */ })
     return { action: "upgraded", localVersion, remoteVersion }
   } catch (err) {
     const msg = err instanceof Error ? err.message : String(err)
@@ -126,6 +141,16 @@ async function checkXpGateUpdate(): Promise<UpgradeResult> {
   }
 }
 
+function readXpGateConfig(): { autoUpgrade?: boolean } | null {
+  const cfgPath = join(homedir(), ".xp-gate", "config.json")
+  try {
+    if (!existsSync(cfgPath)) return null
+    return JSON.parse(readFileSync(cfgPath, "utf8"))
+  } catch {
+    return null
+  }
+}
+
 // ── OpenCode plugin version check (notification only) ──
 
 async function checkPluginUpdate(pluginDir: string): Promise<void> {

package/plugins/opencode/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/opencode-plugin",
-  "version": "0.9.3",
+  "version": "0.9.4",
   "type": "module",
   "main": "index.ts",
   "description": "XP-Gate quality gates + AI workflow skills + Sprint Flow TUI sidebar for OpenCode",

package/plugins/opencode/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/plugins/opencode/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 **11-phase** development pipeline: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → USER ACCEPTANCE → FEEDBACK → SHIP → LAND → CLEANUP. Phase 2 default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE in Phase 1: design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/opencode/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/opencode/tui-plugin.ts

@@ -15,6 +15,7 @@
 import { existsSync, readFileSync } from "node:fs"
 import { join } from "node:path"
 import type { TuiPlugin, TuiSlotPlugin, TuiSlotProps } from "@opencode-ai/plugin/tui"
+import { PHASE_NAMES, PHASE_ORDER, getLatestTimestamp, isStale } from "../../src/npm-package/lib/shared-phase-constants.js"
 
 // ── Sprint state schema ──
 
@@ -44,24 +45,6 @@ interface SprintState {
   phase_history?: SprintPhaseHistory[]
 }
 
-// ── Constants ──
-
-const PHASE_NAMES: Record<string, string> = {
-  "-1": "ISOLATE",
-  "-0.5": "AUTO-ESTIMATE",
-  "0": "THINK",
-  "1": "PLAN",
-  "2": "BUILD",
-  "3": "REVIEW",
-  "4": "USER ACCEPT",
-  "5": "FEEDBACK",
-  "6": "SHIP",
-  "7": "LAND",
-  "8": "CLEANUP",
-}
-
-const PHASE_ORDER = ["-1", "-0.5", "0", "1", "2", "3", "4", "5", "6", "7", "8"]
-
 // ── Helpers ──
 
 function readSprintState(dir: string): SprintState | null {
@@ -74,26 +57,6 @@ function readSprintState(dir: string): SprintState | null {
   }
 }
 
-function isStale(state: SprintState): boolean {
-  if (!state || !state.started_at) return false
-  const started = new Date(state.started_at).getTime()
-  if (isNaN(started)) return false
-  let latest = started
-  if (Array.isArray(state.phase_history)) {
-    for (const ph of state.phase_history) {
-      if (ph.completed_at) {
-        const t = new Date(ph.completed_at).getTime()
-        if (!isNaN(t) && t > latest) latest = t
-      }
-      if (ph.started_at) {
-        const t = new Date(ph.started_at).getTime()
-        if (!isNaN(t) && t > latest) latest = t
-      }
-    }
-  }
-  return Date.now() - latest > 3_600_000
-}
-
 function statusSymbol(status: string | undefined, key: string, currentPhase: string | number | undefined): string {
   if (status === "completed") return "✓"
   if (status === "in_progress") return "→"
@@ -109,57 +72,63 @@ function renderPhaseLine(key: string, history: SprintPhaseHistory | undefined, c
     .replace(/\s+$/, "")
 }
 
-function renderSprintSidebar(state: SprintState): string {
-  if (!state || !state.task_description) return ""
-
-  const lines: string[] = []
-  const metrics = state.metrics || {}
-  const currentPhase = state.phase
-
-  // Build lookup
-  const historyByPhase: Record<string, SprintPhaseHistory> = {}
+function buildPhaseLookup(state: SprintState): Record<string, SprintPhaseHistory> {
+  const lookup: Record<string, SprintPhaseHistory> = {}
   if (Array.isArray(state.phase_history)) {
     for (const ph of state.phase_history) {
-      historyByPhase[String(ph.phase)] = ph
+      lookup[String(ph.phase)] = ph
     }
   }
+  return lookup
+}
 
-  // Title
-  lines.push(`SPRINT: ${state.task_description}`)
+function buildMetricsLine(metrics: SprintState["metrics"]): string | null {
+  const parts: string[] = []
+  if (metrics?.tests_passed != null) parts.push(`tests:${metrics.tests_passed}`)
+  if (metrics?.coverage_pct != null) parts.push(`cov:${metrics.coverage_pct}%`)
+  return parts.length > 0 ? parts.join(" ") : null
+}
 
-  // Metrics
-  const metricParts: string[] = []
-  if (metrics.tests_passed != null) {
-    metricParts.push(`tests:${metrics.tests_passed}`)
-  }
-  if (metrics.coverage_pct != null) {
-    metricParts.push(`cov:${metrics.coverage_pct}%`)
-  }
-  if (metricParts.length > 0) {
-    lines.push(metricParts.join(" "))
-  }
+function buildStaleWarning(state: SprintState): string | null {
+  if (!state || !state.started_at) return null
+  return isStale(state) ? "⚠ idle >1h" : null
+}
 
-  // Stale warning
-  if (isStale(state)) {
-    lines.push("⚠ idle >1h")
-  }
+function renderBuildReqs(history: SprintPhaseHistory): string[] {
+  if (!history.reqs) return []
+  return Object.entries(history.reqs)
+    .filter(([, r]) => r.name)
+    .map(([id, r]) => `  ${statusSymbol(r.status, id, undefined)} ${r.name}`)
+}
 
-  // Phase progress
+function renderPhaseLines(
+  historyByPhase: Record<string, SprintPhaseHistory>,
+  currentPhase: string | number | undefined,
+): string[] {
+  const lines: string[] = []
   for (const key of PHASE_ORDER) {
     const history = historyByPhase[key]
-    // Only show phases with activity or current
     if (!history && String(currentPhase) !== key) continue
-    const line = renderPhaseLine(key, history, currentPhase)
-    lines.push(line)
-
-    // REQ-level progress for BUILD phase
+    lines.push(renderPhaseLine(key, history, currentPhase))
     if (key === "2" && history?.reqs) {
-      const reqNames = Object.entries(history.reqs)
-        .filter(([, r]) => r.name)
-        .map(([id, r]) => `  ${statusSymbol(r.status, id, undefined)} ${r.name}`)
-      if (reqNames.length > 0) lines.push(...reqNames)
+      lines.push(...renderBuildReqs(history))
     }
   }
+  return lines
+}
+
+function renderSprintSidebar(state: SprintState): string {
+  if (!state || !state.task_description) return ""
+
+  const lines: string[] = [`SPRINT: ${state.task_description}`]
+  const metricsLine = buildMetricsLine(state.metrics)
+  if (metricsLine) lines.push(metricsLine)
+
+  const staleWarning = buildStaleWarning(state)
+  if (staleWarning) lines.push(staleWarning)
+
+  const historyByPhase = buildPhaseLookup(state)
+  lines.push(...renderPhaseLines(historyByPhase, state.phase))
 
   return lines.join("\n")
 }

package/plugins/qoder/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/plugins/qoder/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 **11-phase** development pipeline: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → USER ACCEPTANCE → FEEDBACK → SHIP → LAND → CLEANUP. Phase 2 default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE in Phase 1: design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/qoder/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/principles/AGENTS.md

@@ -1,9 +1,9 @@
 # PRINCIPLES CHECKER MODULE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Clean Code & SOLID principles checker — **Gate 4** of pre-commit. 14 rules × 9 language adapters, SARIF 2.1.0 output. Houses the **Boy Scout Rule** enforcement engine (Gate 6) and warning-baseline storage.

package/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 **11-phase** development pipeline: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → USER ACCEPTANCE → FEEDBACK → SHIP → LAND → CLEANUP. Phase 2 default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE in Phase 1: design must pass Delphi review (≥90% consensus) before any coding.

package/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.