@jakobm/peacekeeper 0.14.0 → 0.14.1

package/README.md

@@ -57,7 +57,9 @@ values cause failure without changing the file. Repeating the command on a
 complete valid file leaves it unchanged.
 
 All CLI commands validate configuration before executing, except help,
-`check-project --explain`, and `update-config`. An incomplete existing file fails
+`check-project --explain`, and `update-config`. Checkpoint hooks instead validate
+the event worktree configuration when the event requires instructions or a check.
+An incomplete existing file fails
 with missing field paths and a repair hint. Updates introducing new fields
 therefore require `peacekeeper update-config` before ordinary commands can run.
 
@@ -92,6 +94,8 @@ Complete example:
 
 `loggingEnabled` has the bundled default `true`; set it to `false` to disable invocation logs and the background cleanup worker's log file. Existing logs are kept. The setting is read from the primary checkout's configuration, including for linked worktrees. SessionStart and Stop logs include the process and event working directories, event type, primary-checkout classification, and hook response or error. Only selected event fields are logged, never the full incoming payload. These logs require the Peacekeeper script to start; a missing Node.js executable or package must be diagnosed in Codex's own logs.
 
+Checkpoint hook failures write the cause to stderr and exit with code 2 instead of reporting success. For `Stop`, this requests continued work under the Codex hook protocol. Invocation logging is advisory: a logging failure is reported on stderr, but the checkpoint instructions or check still run and retain their result. Behavior configuration comes from the event's worktree, while log settings come from the primary checkout. A logging warning alone does not change the exit code.
+
 To publish checkpoint branches automatically, set `git.checkpointPushRemotes` (fragment):
 
 ```json

package/bin/peacekeeper.js

@@ -45,7 +45,8 @@ function main() {
   }
   const informational = (args.length === 1 && ['--help', '-h'].includes(args[0]))
     || (command === 'check-project' && args.length === 1 && args[0] === '--explain');
-  if (command !== 'update-config' && !informational) require('../scripts/project-config').projectConfig();
+  const checkpointHook = ['checkpoint-codex-session-start', 'checkpoint-codex-stop'].includes(command);
+  if (command !== 'update-config' && !checkpointHook && !informational) require('../scripts/project-config').projectConfig();
   const script = path.resolve(__dirname, '..', 'scripts', `${command}.js`);
   const result = spawnSync(process.execPath, [script, ...args], {
     cwd: process.cwd(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jakobm/peacekeeper",
-  "version": "0.14.0",
+  "version": "0.14.1",
   "description": "Shared repository test, Git worktree, and Codex hook commands",
   "license": "MIT",
   "repository": {

package/scripts/log-script-invocation.js

@@ -78,8 +78,18 @@ function runLoggedCheckpointHook(buildResponse, errorPrefix) {
   let logPath;
   let event;
   let response;
+  function log(action) {
+    try {
+      action();
+    } catch (error) {
+      process.stderr.write(`${errorPrefix}: logging failed: ${error.message}\n`);
+    }
+  }
+  log(() => { logPath = logScriptInvocation(); });
+  function record(value) {
+    if (logPath) log(() => fs.appendFileSync(logPath, `${JSON.stringify(value)}\n`));
+  }
   try {
-    logPath = logScriptInvocation();
     const input = fs.readFileSync(0, 'utf8').trim();
     event = input === '' ? null : JSON.parse(input);
     const details = {
@@ -88,16 +98,20 @@ function runLoggedCheckpointHook(buildResponse, errorPrefix) {
       source: event?.source,
       stopHookActive: event?.stop_hook_active,
     };
-    if (logPath) fs.appendFileSync(logPath, `${JSON.stringify({ event: details })}\n`);
+    record({ event: details });
     if (typeof event?.cwd === 'string') {
       const primaryWorktree = isPrimaryWorktree(event.cwd);
-      if (logPath) fs.appendFileSync(logPath, `${JSON.stringify({ primaryWorktree })}\n`);
+      record({ primaryWorktree });
     }
     response = buildResponse(event);
   } catch (error) {
-    response = { systemMessage: `${errorPrefix}: ${error.message}` };
+    const message = `${errorPrefix}: ${error.message}`;
+    process.stderr.write(`${message}\n`);
+    process.exitCode = 2;
+    record({ error: message });
+    return;
   }
-  if (logPath) fs.appendFileSync(logPath, `${JSON.stringify({ response })}\n`);
+  record({ response });
   process.stdout.write(`${JSON.stringify(response)}\n`);
 }