@boyingliu01/xp-gate 0.20.0 → 0.20.2

package/principles/boy-scout.ts

@@ -785,12 +785,44 @@ function reportBaselineUpdate(
   console.log(`   preserved:${preserved.length > 0 ? ` ${list(preserved)}` : ' (none)'}`);
 }
 
+/**
+ * Record budget entries for files the gate meets for the first time, then save
+ * and report the write. Split out of `runEnforcement` to keep both under the
+ * long-function threshold; also carries the #452 REQ-2 visibility contract:
+ * the report distinguishes "created" from "updated" and names every entry the
+ * auto-init introduced, never just a count.
+ */
+async function autoInitMissingEntries(
+  baseline: Record<string, BaselineEntry>,
+  missingBaselineEntries: string[],
+  currentWarnings: Record<string, number>,
+  baselinePath: string,
+  baselineExisted: boolean,
+): Promise<void> {
+  recordAutoInitializedEntries(baseline, missingBaselineEntries, currentWarnings);
+  await saveBaseline(baselinePath, baseline);
+  const action = baselineExisted ? 'updated' : 'created';
+  const noun = missingBaselineEntries.length === 1 ? 'entry' : 'entries';
+  console.log(`ℹ️  Baseline ${action}: ${baselinePath}`);
+  console.log(`   auto-initialized ${noun}: ${missingBaselineEntries.join(', ')}`);
+}
+
 async function runEnforcement(newFiles: string[], modifiedFiles: string[], baselinePath: string): Promise<EnforcementResult> {
   const allFiles = [...newFiles.filter(f => f.trim()), ...modifiedFiles.filter(f => f.trim())];
   const currentWarnings = await analyzeWarningsForFiles(allFiles);
-  
+
+  // Distinguishing "created" from "updated" is part of the auto-init contract
+  // (#452 REQ-2): a silently rewritten baseline is how unrelated entries used
+  // to vanish without anyone noticing.
+  let baselineExisted = false;
+  try {
+    await fs.access(baselinePath);
+    baselineExisted = true;
+  } catch {
+    baselineExisted = false;
+  }
   const baseline = await loadBaseline(baselinePath);
-  
+
   // Check for missing baseline entries for modified files
   const missingBaselineEntries: string[] = [];
   for (const file of modifiedFiles) {
@@ -798,13 +830,11 @@ async function runEnforcement(newFiles: string[], modifiedFiles: string[], basel
       missingBaselineEntries.push(file);
     }
   }
-  
+
   // Files being modified for the first time get their current count recorded, so
   // the next modification is compared against a real budget instead of nothing.
   if (missingBaselineEntries.length > 0) {
-    recordAutoInitializedEntries(baseline, missingBaselineEntries, currentWarnings);
-    await saveBaseline(baselinePath, baseline);
-    console.log(`ℹ️  Auto-initialized baseline for ${missingBaselineEntries.length} files`);
+    await autoInitMissingEntries(baseline, missingBaselineEntries, currentWarnings, baselinePath, baselineExisted);
   }
   
   const deltaResults: DeltaResult[] = [];

package/scripts/sync-package-content.js

@@ -419,7 +419,7 @@ function checkAdapterDrift(srcRootPath, mirrorRootPath) {
   if (mismatched.length > 0) {
     console.error(`[drift-check] ERROR: ${mismatched.length} adapter file(s) have drifted (SHA-256 mismatch):`);
     for (const f of mismatched) console.error(`  - ${f}`);
-    console.error('[drift-check] Fix: Run `node src/npm-package/scripts/sync-package-content.js` (or `npm pack`) to resync.');
+    console.error('[drift-check] Fix: Run `node src/npm-package/scripts/sync-package-content.js --fix` to resync (plain invocation is a check-only gate for CI).');
   }
 
   if (srcOnly.length > 0 || mirrorOnly.length > 0 || mismatched.length > 0) {
@@ -449,11 +449,18 @@ function checkDocsDrift(preCommitPath, prePushPath, readmePath, agentsPath) {
 function main() {
   console.error(`[sync] repo root: ${REPO_ROOT}`);
   console.error(`[sync] package root: ${PKG_ROOT}`);
-  if (checkDocsDrift() === false) {
-    process.exit(1);
-  }
-  if (checkAdapterDrift() === false) {
-    process.exit(1);
+  // Drift gates run BEFORE the sync on purpose: CI's Mirror Parity job relies
+  // on a drifted commit failing here rather than being silently healed by the
+  // copy loop. Local repair therefore needs the explicit `--fix` escape hatch
+  // (the pre-sync "Fix: Run …" advice used to deadlock against this gate).
+  const fixMode = process.argv.includes('--fix');
+  if (!fixMode) {
+    if (checkDocsDrift() === false) {
+      process.exit(1);
+    }
+    if (checkAdapterDrift() === false) {
+      process.exit(1);
+    }
   }
   const skills = syncSkills();
   const plugins = syncPlugins();

package/skills/admin-template-guidelines/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: admin-template-guidelines
+version: 1.0.0
 description: "6 maintainability rules for AI-generated admin interfaces (Fastify + Nunjucks + HTMX + Alpine.js). Prevents route bloat, test inconsistency, Nunjucks traps, repeated data logic, auth blind spots, and HTMX/Alpine confusion."
 maturity: alpha
 ---

package/skills/batch-grill-me/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: batch-grill-me
+version: 1.0.0
 description: A relentless interview that asks every frontier question at once, round by round.
 disable-model-invocation: true
 ---

package/skills/clipboard-vision/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: clipboard-vision
+version: 1.0.0
 description: Use when the user explicitly asks to analyze, describe, OCR, or inspect a single image currently stored in the system clipboard.
 ---
 

package/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-10-07
-**Commit:** ff55f35b
-**Branch:** fix/batch-defects-20261007
-**Version:** 0.20.0.0
+**Generated:** 2026-10-12
+**Commit:** unknown
+**Branch:** unknown
+**Version:** 0.20.2.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/skills/domain-modeling/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: domain-modeling
+version: 1.0.0
 description: Build and sharpen a project's domain model. Use when the user wants to pin down domain terminology or a ubiquitous language, record an architectural decision, or when another skill needs to maintain the domain model.
 ---
 

package/skills/grill-with-docs/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: grill-with-docs
+version: 1.0.0
 description: A relentless interview to sharpen a plan or design, which also creates docs (ADR's and glossary) as we go.
 disable-model-invocation: true
 ---

package/skills/grilling/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: grilling
+version: 1.0.0
 description: Grill the user relentlessly about a plan, decision, or idea. Use when the user wants to stress-test their thinking, or uses any 'grill' trigger phrases.
 ---
 

package/skills/improve-codebase-architecture/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: improve-codebase-architecture
+version: 1.0.0
 description: Find deepening opportunities in a codebase, informed by domain language in CONTEXT.md and decisions in ADRs. Use when user wants to improve architecture, find refactoring opportunities, consolidate tightly-coupled modules, or make a codebase more AI-navigable. Run periodically (weekly or after surges of development).
 ---
 

package/skills/ralph-loop/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: ralph-loop
+version: 1.0.0
 description: Use when executing Sprint-Flow Phase 2 BUILD, processing one REQ, building the next requirement, iterating with clean context, or requiring full regression per requirement.
 maturity: stable
 ---

package/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-10-07
-**Commit:** ff55f35b
-**Branch:** fix/batch-defects-20261007
-**Version:** 0.20.0.0
+**Generated:** 2026-10-12
+**Commit:** unknown
+**Branch:** unknown
+**Version:** 0.20.2.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/skills/test-driven-development/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: test-driven-development
+version: 1.0.0
 description: Use when implementing any feature or bugfix, before writing implementation code
 ---
 

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

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-10-07
-**Commit:** ff55f35b
-**Branch:** fix/batch-defects-20261007
-**Version:** 0.20.0.0
+**Generated:** 2026-10-12
+**Commit:** unknown
+**Branch:** unknown
+**Version:** 0.20.2.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

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

@@ -1,5 +1,6 @@
 ---
 name: test-specification-alignment
+version: 1.0.0
 description: "Use when asked to run tests, verify tests, align tests with specification.yaml, before BUILD verification, or before release/ship."
 ---
 

package/skills/to-issues/SKILL.md

@@ -1,5 +1,6 @@
 ---
 name: to-issues
+version: 1.0.0
 description: Break a plan, spec, or PRD into independently-grabbable issues using tracer-bullet vertical slices. Use when user wants to convert a plan into issues, create implementation tickets, or break down work into vertical slices. Integrates with XP-Gate sprint-flow Phase 1.
 ---