phronesis 1.0.0 → 1.1.0

package/src/hooks.js

@@ -47,39 +47,15 @@ export async function runHooksDueCheck({ session, dir, actor, surface } = {}) {
   const resolvedActor = actor || process.env.PHRONESIS_ACTOR || 'agent';
   const resolvedSurface = surface || 'cli';
   try {
-    const { due } = await runDueCheck({ installRoot, session, actor: resolvedActor, surface: resolvedSurface });
-    if (!due.length) {
-      console.log('No decisions due for review.');
-    } else {
-      console.log(`Phronesis due-check: ${due.length} decision(s) due for review.`);
-      for (const d of due) {
-        const parts = d.decision_path.split('/');
-        const domain = parts[1];
-        const name = parts[parts.length - 1].replace(/\.md$/, '');
-        const overdue = d.days_overdue === 0 ? 'due today' : `${d.days_overdue} day(s) overdue`;
-        console.log(`\n  ${d.decision_path} — ${overdue} (review_date ${d.review_date})`);
-        console.log(
-          `    record the outcome: phronesis act record_outcome --domain ${domain} --session ${session} --json '{"target":"decisions/${name}","outcome_status":"<${ENUMS.outcome_status.join('|')}>","lesson":"<what you learned>"}'`,
-        );
-      }
-    }
-    // Schedule half (change 0042): a malformed schedule.json is a SEPARATE failure domain —
-    // it degrades to zero due entries here (validate/doctor own the fault), never breaking the
-    // decision due-check above.
-    const { due: scheduleDue } = await runScheduleDueCheck({ installRoot, session, actor: resolvedActor, surface: resolvedSurface });
-    if (scheduleDue.length) {
-      console.log(`\nPhronesis schedule: ${scheduleDue.length} item(s) due.`);
-      for (const s of scheduleDue) {
-        const overdue = s.days_overdue === 0 ? 'due now' : `${s.days_overdue} day(s) overdue`;
-        console.log(`\n  ${s.schedule_id} — ${s.kind}:${s.target} — ${overdue}`);
-        if (s.kind === 'cli_verb' || s.kind === 'connector_pull') {
-          console.log(`    run now: phronesis due run ${s.schedule_id}`);
-        }
-        console.log(`    mark done: phronesis due complete ${s.schedule_id}`);
-      }
-    }
+    const result = await collectHooksDueCheck({
+      installRoot,
+      session,
+      actor: resolvedActor,
+      surface: resolvedSurface,
+    });
+    console.log(result.output);
     process.exitCode = 0;
-    return due;
+    return result.due;
   } catch (err) {
     console.error(`hooks due-check: ${err.message}`);
     process.exitCode = 1;
@@ -87,6 +63,45 @@ export async function runHooksDueCheck({ session, dir, actor, surface } = {}) {
   }
 }
 
+// Composable due-check primitive for the resolved `phronesis hook session-start` interface.
+// It keeps the authoritative core emitters and the long-standing operator-facing bytes in one
+// place: the legacy `hooks due-check` command prints `output`, while the hook dispatcher either
+// prints it raw (Claude Code) or places it in Codex's additionalContext envelope.
+export async function collectHooksDueCheck({ installRoot, session, actor = 'agent', surface = 'cli' } = {}) {
+  const { due } = await runDueCheck({ installRoot, session, actor, surface });
+  // Schedule is a separate failure domain: the core primitive already degrades a malformed
+  // schedule registry to an empty list, leaving the decision due-check intact.
+  const { due: scheduleDue } = await runScheduleDueCheck({ installRoot, session, actor, surface });
+  const lines = [];
+  if (!due.length) {
+    lines.push('No decisions due for review.');
+  } else {
+    lines.push(`Phronesis due-check: ${due.length} decision(s) due for review.`);
+    for (const d of due) {
+      const parts = d.decision_path.split('/');
+      const domain = parts[1];
+      const name = parts[parts.length - 1].replace(/\.md$/, '');
+      const overdue = d.days_overdue === 0 ? 'due today' : `${d.days_overdue} day(s) overdue`;
+      lines.push(`\n  ${d.decision_path} — ${overdue} (review_date ${d.review_date})`);
+      lines.push(
+        `    record the outcome: phronesis act record_outcome --domain ${domain} --session ${session} --json '{"target":"decisions/${name}","outcome_status":"<${ENUMS.outcome_status.join('|')}>","lesson":"<what you learned>"}'`,
+      );
+    }
+  }
+  if (scheduleDue.length) {
+    lines.push(`\nPhronesis schedule: ${scheduleDue.length} item(s) due.`);
+    for (const s of scheduleDue) {
+      const overdue = s.days_overdue === 0 ? 'due now' : `${s.days_overdue} day(s) overdue`;
+      lines.push(`\n  ${s.schedule_id} — ${s.kind}:${s.target} — ${overdue}`);
+      if (s.kind === 'cli_verb' || s.kind === 'connector_pull') {
+        lines.push(`    run now: phronesis due run ${s.schedule_id}`);
+      }
+      lines.push(`    mark done: phronesis due complete ${s.schedule_id}`);
+    }
+  }
+  return { due, scheduleDue, output: lines.join('\n') };
+}
+
 // Sweep status: has this session emitted its capture-sweep marker (`session.end` with a
 // `sweep` payload — `sweep.candidates: 0` is the honest negative)? Prints `swept` or
 // `pending`; exits 0 when swept, 1 when pending — the scaffolded Stop-hook script keys
@@ -127,7 +142,7 @@ function parseHookPayload(label, payload) {
 
 // The operator-facing copy for the J1 zero-capture warning (core-0027). The POLICY (when
 // to warn) is @phronesis/core's meshZeroWarning; this is presentation only.
-function meshZeroWarningMessage(w) {
+export function meshZeroWarningMessage(w) {
   return (
     `Phronesis: the heavy skill "${w.skill_name}" completed with 0 mesh enrichments ` +
     `(no links_updated across ${w.action_types_committed} committed action(s)). A substantive ` +

package/src/init.js

@@ -3,9 +3,16 @@ import fse from 'fs-extra';
 import chalk from 'chalk';
 import slugify from 'slugify';
 import { createInstallation } from './scaffold.js';
-import { confirmOverwrite, promptProfile } from './prompts.js';
+import {
+  confirmOverwrite,
+  confirmPathAppend,
+  confirmPersistence,
+  PERSISTENCE_PROMPT,
+  promptProfile,
+} from './prompts.js';
 import { resolveProfile } from './profile.js';
 import { DEFAULT_DOMAIN } from './layout.js';
+import { runLauncherInstall } from './launcher.js';
 
 const DEFAULT_DIR = 'phronesis-installation';
 
@@ -23,7 +30,20 @@ async function isPhronesisInstall(dir) {
   return fse.pathExists(join(dir, '.phronesis', 'registry.json'));
 }
 
-export async function init({ dir, force = false, domain, name, role, company, workingStyle } = {}) {
+export async function init({
+  dir,
+  force = false,
+  domain,
+  name,
+  role,
+  company,
+  workingStyle,
+  persist,
+  isTTY = Boolean(process.stdin.isTTY),
+  promptPersistence = confirmPersistence,
+  promptPath = confirmPathAppend,
+  installLauncher = runLauncherInstall,
+} = {}) {
   const targetDir = resolve(process.cwd(), dir || DEFAULT_DIR);
   const domainSlug = slugify(domain || DEFAULT_DOMAIN, { lower: true, strict: true }) || DEFAULT_DOMAIN;
   if (domain && domainSlug !== domain) {
@@ -61,7 +81,7 @@ export async function init({ dir, force = false, domain, name, role, company, wo
       }
       // A prior Phronesis install — replacement is in-bounds, but still confirm.
       if (!force) {
-        if (process.stdin.isTTY) {
+        if (isTTY) {
           const ok = await confirmOverwrite(targetDir);
           if (!ok) {
             console.log('Setup cancelled.');
@@ -90,7 +110,7 @@ export async function init({ dir, force = false, domain, name, role, company, wo
   try {
     profile = await resolveProfile({
       flags: { name, role, company, workingStyle },
-      isTTY: Boolean(process.stdin.isTTY),
+      isTTY,
       prompt: promptProfile,
     });
   } catch (err) {
@@ -120,6 +140,19 @@ export async function init({ dir, force = false, domain, name, role, company, wo
   }
 
   printSuccess(result);
+  let shouldPersist = persist === true;
+  if (persist === undefined && isTTY) {
+    shouldPersist = await promptPersistence({ message: PERSISTENCE_PROMPT });
+  }
+  if (shouldPersist) {
+    try {
+      await installLauncher({ isTTY, promptPath });
+    } catch (err) {
+      console.error(`Command installation failed: ${err.message}`);
+      process.exitCode = 1;
+      return { ...result, persistence: { ok: false, error: err.message } };
+    }
+  }
   return result;
 }