godpowers 3.13.0 → 3.13.1

package/lib/route-quality-sync.js

@@ -10,6 +10,7 @@ const fs = require('fs');
 const path = require('path');
 
 const { parseSimpleYaml } = require('./intent');
+const { read, write } = require('./sync-fs');
 
 const LOG_PATH = '.godpowers/surface/ROUTE-QUALITY-SYNC.md';
 const CONTEXTUAL_NEXT_VALUES = new Set([
@@ -100,18 +101,6 @@ const TIER_GATE_COMMANDS = new Set([
   '/god-harden'
 ]);
 
-function read(projectRoot, relPath) {
-  const file = path.join(projectRoot, relPath);
-  if (!fs.existsSync(file)) return '';
-  return fs.readFileSync(file, 'utf8');
-}
-
-function write(projectRoot, relPath, content) {
-  const file = path.join(projectRoot, relPath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, content);
-}
-
 function listFiles(projectRoot, relDir, pattern) {
   const dir = path.join(projectRoot, relDir);
   if (!fs.existsSync(dir)) return [];

package/lib/state.js

@@ -53,10 +53,26 @@ const SUBSTEP_LABELS = {
  * @property {number} ordinal One-based step position.
  */
 
+// Canonical project-relative location of the state file. Other modules that
+// need to name state.json (gates, dispatch findings, audits) import this rather
+// than re-typing the literal (ARC-002).
+const STATE_FILE = '.godpowers/state.json';
+
 function statePath(projectRoot) {
   return path.join(projectRoot, '.godpowers', 'state.json');
 }
 
+// A typed error so callers (e.g. the CLI dispatcher) can detect corrupt state
+// by `err.code === 'CORRUPT_STATE'` instead of matching the message prose.
+function corruptStateError(file, cause) {
+  const err = new Error(
+    `Corrupt state file at ${file}: ${cause.message}. ` +
+    `Fix the JSON or remove the file to let Godpowers reinitialize it.`
+  );
+  err.code = 'CORRUPT_STATE';
+  return err;
+}
+
 function tierNumber(tierKey) {
   const match = String(tierKey).match(/^tier-(\d+)$/);
   return match ? Number(match[1]) : Number.MAX_SAFE_INTEGER;
@@ -88,10 +104,7 @@ function read(projectRoot) {
   try {
     return JSON.parse(raw);
   } catch (e) {
-    throw new Error(
-      `Corrupt state file at ${file}: ${e.message}. ` +
-      `Fix the JSON or remove the file to let Godpowers reinitialize it.`
-    );
+    throw corruptStateError(file, e);
   }
 }
 
@@ -144,10 +157,7 @@ async function readAsync(projectRoot) {
   try {
     return JSON.parse(raw);
   } catch (e) {
-    throw new Error(
-      `Corrupt state file at ${file}: ${e.message}. ` +
-      `Fix the JSON or remove the file to let Godpowers reinitialize it.`
-    );
+    throw corruptStateError(file, e);
   }
 }
 
@@ -421,6 +431,7 @@ module.exports = {
   updateSubStepAsync,
   hashFile,
   detectDrift,
+  STATE_FILE,
   statePath,
   isInitialized,
   isInitializedState,

package/lib/sync-fs.js

@@ -0,0 +1,37 @@
+/**
+ * Shared filesystem helpers for the lib/*-sync.js family.
+ *
+ * Every sync module used to redefine its own byte-identical read/write/exists/
+ * readJson against a project root (ARC-001). They now share these so a change
+ * to path handling or read semantics lives in one place. Module-specific log
+ * writers (appendLog) stay per-module because their headers and formats differ.
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+function read(projectRoot, relPath) {
+  const file = path.join(projectRoot, relPath);
+  if (!fs.existsSync(file)) return '';
+  return fs.readFileSync(file, 'utf8');
+}
+
+function write(projectRoot, relPath, content) {
+  const file = path.join(projectRoot, relPath);
+  fs.mkdirSync(path.dirname(file), { recursive: true });
+  fs.writeFileSync(file, content);
+}
+
+function exists(projectRoot, relPath) {
+  return fs.existsSync(path.join(projectRoot, relPath));
+}
+
+function readJson(projectRoot, relPath) {
+  try {
+    return JSON.parse(read(projectRoot, relPath));
+  } catch (err) {
+    return null;
+  }
+}
+
+module.exports = { read, write, exists, readJson };

package/lib/text-util.js

@@ -0,0 +1,19 @@
+/**
+ * Small shared string helpers (QUAL-002).
+ *
+ * slugify is the canonical home for the "lowercase, collapse non-alphanumerics
+ * to '-', strip edge '-', truncate to 40 chars" contract. lib/evidence.js keeps
+ * its own copy on purpose: that module is vendored from the upstream engine and
+ * its helpers are provenance-tracked, so it must not import first-party code.
+ */
+
+function slugify(text, fallback = '') {
+  const slug = String(text == null ? '' : text)
+    .toLowerCase()
+    .replace(/[^a-z0-9]+/g, '-')
+    .replace(/^-+|-+$/g, '')
+    .slice(0, 40);
+  return slug || fallback;
+}
+
+module.exports = { slugify };

package/lib/workflow-helper-groups.js

@@ -6,6 +6,10 @@
  */
 
 const HELPER_GROUPS = {
+  'context-bootstrap': [
+    'pillars-detect',
+    'pillars-init'
+  ],
   'standard-closeout': [
     'repo-doc-sync',
     'repo-surface-sync',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "godpowers",
-  "version": "3.13.0",
+  "version": "3.13.1",
   "description": "AI-powered development system: 120 slash commands and 40 specialist agents that take a project from raw idea to hardened production. Runs inside Claude Code, Codex, Cursor, Windsurf, Gemini, and 10+ other AI coding tools.",
   "bin": {
     "godpowers": "./bin/install.js"
@@ -24,7 +24,7 @@
     "test:e2e": "node tests/integration/full-arc.test.js",
     "test:mcp": "npm --workspace @godpowers/mcp test",
     "coverage": "c8 --reporter=text --reporter=lcov node scripts/run-tests.js",
-    "coverage:lib": "c8 --include=lib/**/*.js --check-coverage --lines 90 --reporter=text node scripts/run-tests.js",
+    "coverage:lib": "c8 --include=lib/**/*.js --check-coverage --lines 90 --branches 75 --reporter=text node scripts/run-tests.js",
     "test:audit": "npm audit --omit=dev && git diff --check && npm run test:surface",
     "pack:check": "node scripts/check-package-contents.js",
     "pack:mcp:check": "npm --workspace @godpowers/mcp run pack:check",

package/references/orchestration/GOD-ORCHESTRATOR-RUNBOOK.md

@@ -115,6 +115,13 @@ This converts existing Godpowers artifacts into managed source references in
 the relevant pillar files, so old projects are Pillar-ized as part of being
 Godpower-ized.
 
+In the greenfield `full-arc` workflow this start-of-arc step is surfaced as the
+tier-0 `context` job, whose `context-bootstrap` helper group expands to
+`pillars-detect` (`lib/pillars.detect`) and `pillars-init` (`lib/pillars.init`).
+The job uses `god-orchestrator` as a local runtime call, not a `god-context-writer`
+spawn, so it changes nothing about the behavior described above; it only makes the
+init visible in `/god-mode --plan` alongside the closeout `pillars-sync-plan`.
+
 Before each major command, compute the task-specific Pillars load set with
 `lib/pillars.computeLoadSet(projectRoot, taskText)`. Load `agents/context.md`
 and `agents/repo.md` first, then the routed primary pillars and their direct

package/workflows/full-arc.yaml

@@ -3,6 +3,15 @@
 # the agents in the order this file specifies. To preview without running,
 # use `/god-mode --workflow=<name> --plan`.
 #
+# Native Pillars context (AGENTS.md + agents/context.md + agents/repo.md) is
+# woven through this arc by the orchestrator's local runtime, not a specialist
+# agent. It is now visible at both ends: the tier-0 `context` preamble runs
+# context-bootstrap (lib/pillars.detect then lib/pillars.init) so a greenfield
+# project is Pillar-ized before planning, and the `final-sync` standard-closeout
+# runs pillars-sync-plan (lib/pillars.planArtifactSync) so the pillars reflect
+# the artifacts the arc produced. Both are local runtime calls; see
+# references/orchestration/GOD-ORCHESTRATOR-RUNBOOK.md "Native Pillars context".
+#
 apiVersion: godpowers/v1
 kind: Workflow
 metadata:
@@ -15,8 +24,17 @@ metadata:
 on: [/god-mode]
 
 jobs:
+  context:
+    tier: 0
+    uses: god-orchestrator@^1.0.0
+    local-helper-groups:
+      - context-bootstrap
+    with:
+      action: pillars-bootstrap
+
   prd:
     tier: 1
+    needs: context
     uses: god-pm@^1.0.0
     with:
       template: PRD.md