@geraldmaron/construct 1.0.8 → 1.0.10

package/lib/hooks/_lib/specialist-agent.mjs

@@ -0,0 +1,36 @@
+/**
+ * lib/hooks/_lib/specialist-agent.mjs — Detect whether a Construct specialist
+ * sub-agent is currently active.
+ *
+ * A specialist is "active" when one of:
+ *   - process.env.CONSTRUCT_AGENT_ID is set (explicit fence dispatch)
+ *   - ~/.cx/last-agent-<id>.json or ~/.cx/last-agent.json carries a
+ *     timestamp inside the 10-minute fence window (mirrors guard-bash.mjs)
+ *
+ * Differentiates "subagent driving the session, hold the line" from "main
+ * session (likely a human), warn but allow".
+ */
+
+import { readFileSync, existsSync } from 'fs';
+import { join } from 'path';
+import { homedir } from 'os';
+
+const FENCE_WINDOW_MS = 10 * 60 * 1000;
+
+export function isSpecialistAgentActive({ now = Date.now, home = homedir, env = process.env } = {}) {
+  if (env.CONSTRUCT_AGENT_ID) return true;
+  try {
+    const cxDir = join(home(), '.cx');
+    const id = String(env.CONSTRUCT_AGENT_ID || '').replace(/^cx-/, '');
+    const candidates = [];
+    if (id) candidates.push(join(cxDir, `last-agent-${id.replace(/[^a-z0-9._-]/gi, '_')}.json`));
+    candidates.push(join(cxDir, 'last-agent.json'));
+    for (const path of candidates) {
+      if (!existsSync(path)) continue;
+      const data = JSON.parse(readFileSync(path, 'utf8'));
+      const lastTs = data?.ts ? Date.parse(data.ts) : 0;
+      if (lastTs && (now() - lastTs) < FENCE_WINDOW_MS) return true;
+    }
+  } catch { /* tracker missing or unreadable — treat as no agent active */ }
+  return false;
+}

package/lib/hooks/pre-push-gate.mjs

@@ -9,6 +9,12 @@
  *                           template policy before the PR is opened
  *   gh pr edit              same body lint when --body / --body-file is changed
  *
+ * Template-policy severity is conditional: a hard block (exit 2) when a
+ * specialist sub-agent is active (CONSTRUCT_AGENT_ID set, or the cx-tracker
+ * shows a fresh dispatch in the last 10 minutes); a warning otherwise so a
+ * human driving the session can override editorial nits without ceremony.
+ * CI still enforces the same rules on the resulting PR.
+ *
  * Silent and instant when there's nothing to run.
  *
  * Bypasses (env vars):
@@ -24,6 +30,7 @@ import { execSync, spawn, spawnSync } from 'child_process';
 import { dirname, join, resolve } from 'path';
 import { tmpdir } from 'os';
 import { logHookFailure } from './_lib/log.mjs';
+import { isSpecialistAgentActive } from './_lib/specialist-agent.mjs';
 import { emitRoleEvent } from '../roles/hook-emit.mjs';
 
 let input = {};
@@ -72,16 +79,21 @@ if ((isGhPrCreate || isGhPrEdit) && process.env.CONSTRUCT_SKIP_PR_LINT !== '1')
         });
         const out = (result.stdout?.toString() || '') + (result.stderr?.toString() || '');
         if (result.status !== 0) {
-          process.stderr.write(`[pre-push-gate] PR body fails template policy:\n${out}\n`);
+          const agentActive = isSpecialistAgentActive();
+          const severity = agentActive ? 'fail' : 'warn';
+          const banner = agentActive
+            ? '[pre-push-gate] PR body fails template policy (specialist agent active — blocking):'
+            : '[pre-push-gate] PR body fails template policy (no specialist agent active — warning only):';
+          process.stderr.write(`${banner}\n${out}\n`);
           process.stderr.write(`Override: CONSTRUCT_SKIP_PR_LINT=1 <command>\n`);
           emitRoleEvent({
-            type: 'push_gate.fail',
-            summary: 'PR body fails template policy',
+            type: agentActive ? 'push_gate.fail' : 'push_gate.warn',
+            summary: `PR body fails template policy (${severity})`,
             hookInput: input,
-            context: { command: isGhPrCreate ? 'gh pr create' : 'gh pr edit', detail: out.slice(0, 800) },
+            context: { command: isGhPrCreate ? 'gh pr create' : 'gh pr edit', detail: out.slice(0, 800), agentActive },
           });
           if (tmpFile) try { rmSync(dirname(tmpFile), { recursive: true, force: true }); } catch {}
-          process.exit(2);
+          if (agentActive) process.exit(2);
         }
       } finally {
         if (tmpFile) try { rmSync(dirname(tmpFile), { recursive: true, force: true }); } catch {}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geraldmaron/construct",
-  "version": "1.0.8",
+  "version": "1.0.10",
   "type": "module",
   "description": "Construct — agent orchestration layer for OpenCode, Claude Code, and other coding surfaces",
   "bin": {

package/scripts/sync-specialists.mjs

@@ -138,8 +138,9 @@ if (validationErrors.length > 0) {
 
 const DRY_RUN = process.argv.includes("--dry-run");
 const COMPRESS_PERSONAS = process.argv.includes("--compress-personas");
-const lockPath = path.join(root, ".cx", "sync.lock");
-const stagingDir = path.join(root, ".cx", "sync-staging");
+const projectDir = process.argv.includes("--project") ? process.cwd() : null;
+const lockPath = path.join(projectDir || root, ".cx", "sync.lock");
+const stagingDir = path.join(projectDir || root, ".cx", "sync-staging");
 
 /** Acquire an exclusive lockfile. Aborts if already held by a live process. */
 function acquireLock() {
@@ -192,7 +193,7 @@ function writeFile(file, content) {
   }
 
   // Two-phase: write to staging, commit later.
-  const rel = path.relative(root, file);
+  const rel = path.relative(projectDir || root, file);
   const stagingPath = path.join(stagingDir, rel);
   mkdirp(path.dirname(stagingPath));
   fs.writeFileSync(stagingPath, stamped);
@@ -1175,7 +1176,6 @@ function syncSkills() {
 
 // --- Main ---
 
-const projectDir = process.argv.includes("--project") ? process.cwd() : null;
 const entries = buildEntries();
 
 if (COMPRESS_PERSONAS) {