@jakobm/peacekeeper 0.14.0 → 0.15.0

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
@@ -240,6 +244,9 @@ does not grant trust automatically.
 ## Check project conventions locally
 
 Run `peacekeeper check-project` to check the local project configuration.
+All projects must declare nonblank `author` and `repository` values in
+`package.json`. npm object forms are supported with nonblank `author.name`
+and `repository.url` strings.
 Run `peacekeeper check-project --explain` for the complete expected
 architecture, requirements, configuration examples, and verification limits.
 The command prints the bundled [formal requirements](scripts/check-project-requirements.txt),

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,8 +1,9 @@
 {
   "name": "@jakobm/peacekeeper",
-  "version": "0.14.0",
+  "version": "0.15.0",
   "description": "Shared repository test, Git worktree, and Codex hook commands",
   "license": "MIT",
+  "author": "Jakob Mattsson",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/jakobmattsson/peacekeeper.git"

package/scripts/check-project-requirements.txt

@@ -39,6 +39,13 @@ Project configuration
     }
   projectChecks and both its fields are mandatory in an existing configuration.
 
+Package metadata: all projects, including private packages
+  package.json must declare author and repository.
+  author must be a nonblank string or an object with a nonblank string name.
+  repository must be a nonblank string (including npm repository shorthand)
+  or an object with a nonblank string url. Whitespace-only values fail.
+  These checks validate presence, not repository reachability or author identity.
+
 Toolchain: package.json and all Node-based CI jobs
   Pin exact versions in package.json with:
     volta pin node@<exact-version> npm@<exact-version>

package/scripts/check-project.js

@@ -40,6 +40,14 @@ function checkProject(cwd = process.cwd()) {
     try { action(); } catch (error) { errors.push(`[${category}] ${error.message}`); }
     finally { category = previous; }
   };
+  check('package', () => {
+    const nonblank = value => typeof value === 'string' && value.trim().length > 0;
+    requireThat(nonblank(manifest.author) || nonblank(manifest.author?.name),
+      'package.json: author must be a nonblank string or an object with a nonblank name.');
+    requireThat(nonblank(manifest.repository) || nonblank(manifest.repository?.url),
+      'package.json: repository must be a nonblank string or an object with a nonblank url.');
+  });
+
   const workflows = new Map();
   check('CI', () => {
     const directory = path.join(root, '.github/workflows');
@@ -227,7 +235,7 @@ function main() {
   if (errors.length) {
     for (const error of errors) console.error(`FAIL ${error}`);
     process.exitCode = 1;
-  } else console.log(`Project checks passed: tools, hooks, CI${published ? ', publication' : ' (private package; publication skipped)'}.`);
+  } else console.log(`Project checks passed: package, tools, hooks, CI${published ? ', publication' : ' (private package; publication skipped)'}.`);
 }
 if (require.main === module) {
   try { main(); } catch (error) { console.error(`FAIL ${error.message}`); process.exitCode = 1; }

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`);
 }