@lastboy/pai 0.1.0 → 0.2.0

package/README.md

@@ -110,6 +110,13 @@ pai import mine.json
 
 Invalid or unsupported files fail with a clear message and a non-zero exit code.
 
+**Cross-platform.** A store exported on macOS or Linux imports on Windows and
+back. Stores contain no filesystem paths, so nothing is machine-specific, and
+import accepts the encodings Windows tooling produces: UTF-8, UTF-8 with a BOM
+(Notepad, `Set-Content`) and UTF-16 with a BOM (PowerShell 5.1's `>` redirect).
+Line endings are normalized, so the same rule written on Windows and on
+macOS/Linux merges as one instead of duplicating.
+
 ### `pai experiment …`
 
 Experimental commands. They may change or disappear, and they are the only

package/dist/adapters/claude/guidelines.js

@@ -17,7 +17,8 @@ const BULLET = /^\s*[-*]\s+(.+?)\s*$/;
 export function parseGuidelines(markdown) {
     const guidelines = [];
     let category = 'General';
-    for (const line of markdown.split('\n')) {
+    // A BOM (Windows editors) would otherwise hide the first heading.
+    for (const line of markdown.replace(/^/, '').split('\n')) {
         const heading = HEADING.exec(line);
         if (heading?.[1]) {
             category = heading[1];

package/dist/cli/program.js

@@ -16,6 +16,7 @@ import { findGuidelineFiles, parseGuidelines } from '../adapters/claude/guidelin
 import { filterGuidelineGroups } from '../core/guidelines.js';
 import { emptyStore, mergeStores, parseRuleStore, serializeRuleStore, } from '../core/rule-store.js';
 import { globalStorePath, projectStorePath, readStore, writeStore, } from '../persistence/rule-store-files.js';
+import { decodeTextFile } from '../persistence/text-file.js';
 import { parseSelection, renderGuidelines, renderReview, renderSessionList } from './render.js';
 async function ask(question) {
     const rl = createInterface({ input: process.stdin, output: process.stdout });
@@ -48,6 +49,10 @@ async function chooseSession(out) {
     out('');
     return sessions[index];
 }
+/** Node's fs errors are readable already; anything else gets stringified. */
+function describeError(error) {
+    return error instanceof Error ? error.message : String(error);
+}
 // Same relative depth from src/cli and dist/cli.
 function packageVersion() {
     const pkg = JSON.parse(readFileSync(join(import.meta.dirname, '..', '..', 'package.json'), 'utf8'));
@@ -74,7 +79,7 @@ export function createProgram(out) {
             process.exitCode = 1;
             return;
         }
-        const transcript = await readFile(chosen.path, 'utf8');
+        const transcript = decodeTextFile(await readFile(chosen.path));
         const session = parseTranscript(transcript, chosen.id);
         for (const line of renderReview(buildReview(session))) {
             out(line);
@@ -91,7 +96,7 @@ export function createProgram(out) {
         const groups = await Promise.all(files.map(async (file) => ({
             scope: file.scope,
             path: file.path,
-            guidelines: parseGuidelines(await readFile(file.path, 'utf8')),
+            guidelines: parseGuidelines(decodeTextFile(await readFile(file.path))),
         })));
         // --global and --project together = no scope filter (same as neither).
         const scope = options.global && !options.project
@@ -118,19 +123,25 @@ export function createProgram(out) {
             process.exitCode = 1;
             return;
         }
-        const global = readStore(globalStorePath());
-        const project = readStore(projectStorePath(process.cwd()));
-        const combined = {
-            ...emptyStore(),
-            rules: [...global.rules, ...project.rules].filter((rule) => options.scope === undefined || rule.scope === options.scope),
-        };
-        const json = serializeRuleStore(combined);
-        if (options.out === undefined) {
-            out(json.trimEnd());
-            return;
+        try {
+            const global = readStore(globalStorePath());
+            const project = readStore(projectStorePath(process.cwd()));
+            const combined = {
+                ...emptyStore(),
+                rules: [...global.rules, ...project.rules].filter((rule) => options.scope === undefined || rule.scope === options.scope),
+            };
+            const json = serializeRuleStore(combined);
+            if (options.out === undefined) {
+                out(json.trimEnd());
+                return;
+            }
+            writeFileSync(options.out, json, 'utf8');
+            out(`Exported ${combined.rules.length} rule(s) to ${options.out}`);
+        }
+        catch (error) {
+            out(`Export failed: ${describeError(error)}`);
+            process.exitCode = 1;
         }
-        writeFileSync(options.out, json, 'utf8');
-        out(`Exported ${combined.rules.length} rule(s) to ${options.out}`);
     });
     program
         .command('import')
@@ -140,10 +151,10 @@ export function createProgram(out) {
         .action(async (file, options) => {
         let incoming;
         try {
-            incoming = parseRuleStore(await readFile(file, 'utf8'));
+            incoming = parseRuleStore(decodeTextFile(await readFile(file)));
         }
         catch (error) {
-            out(error instanceof Error ? error.message : String(error));
+            out(`Could not read ${file}: ${describeError(error)}`);
             process.exitCode = 1;
             return;
         }
@@ -151,14 +162,22 @@ export function createProgram(out) {
             { scope: 'global', path: globalStorePath() },
             { scope: 'project', path: projectStorePath(process.cwd()) },
         ];
-        for (const target of targets) {
-            const rules = incoming.rules.filter((rule) => rule.scope === target.scope);
-            if (rules.length === 0)
-                continue;
-            const result = mergeStores(readStore(target.path), { ...emptyStore(), rules });
-            if (!options.dryRun)
-                writeStore(target.path, result.store);
-            out(`${target.scope}: ${result.added} added, ${result.merged} updated with new evidence, ${result.store.rules.length} total → ${target.path}`);
+        try {
+            for (const target of targets) {
+                const rules = incoming.rules.filter((rule) => rule.scope === target.scope);
+                if (rules.length === 0)
+                    continue;
+                const result = mergeStores(readStore(target.path), { ...emptyStore(), rules });
+                if (!options.dryRun)
+                    writeStore(target.path, result.store);
+                out(`${target.scope}: ${result.added} added, ${result.merged} updated with new evidence, ${result.store.rules.length} total → ${target.path}`);
+            }
+        }
+        catch (error) {
+            out(`Import failed: ${describeError(error)}`);
+            out('Nothing was changed for the scope that failed.');
+            process.exitCode = 1;
+            return;
         }
         if (incoming.rules.length === 0)
             out('Nothing to import — the file contains no rules.');
@@ -179,7 +198,7 @@ export function createProgram(out) {
             process.exitCode = 1;
             return;
         }
-        const transcript = await readFile(chosen.path, 'utf8');
+        const transcript = decodeTextFile(await readFile(chosen.path));
         const events = parseEvents(transcript);
         const candidates = events
             .map((event, index) => ({ event, index }))
@@ -265,7 +284,7 @@ export function createProgram(out) {
             process.exitCode = 1;
             return;
         }
-        const transcript = await readFile(chosen.path, 'utf8');
+        const transcript = decodeTextFile(await readFile(chosen.path));
         const messages = parseEvents(transcript).filter((e) => e.kind === 'user');
         const limit = Math.max(1, Number(options.limit) || 10);
         const sample = messages.slice(-limit);

package/dist/persistence/rule-store-files.js

@@ -1,7 +1,8 @@
-import { mkdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { mkdirSync, renameSync, unlinkSync, writeFileSync } from 'node:fs';
 import { homedir } from 'node:os';
 import { dirname, join } from 'node:path';
 import { emptyStore, parseRuleStore, serializeRuleStore } from '../core/rule-store.js';
+import { readTextFile } from './text-file.js';
 // PAI writes only to its own agent-neutral `.pai/` directories.
 export function globalStorePath(home = homedir()) {
     return join(home, '.pai', 'rules.json');
@@ -9,17 +10,40 @@ export function globalStorePath(home = homedir()) {
 export function projectStorePath(cwd) {
     return join(cwd, '.pai', 'rules.json');
 }
+/** A missing store is empty; a corrupt one is an error naming the file. */
 export function readStore(path) {
     let contents;
     try {
-        contents = readFileSync(path, 'utf8');
+        contents = readTextFile(path);
     }
     catch {
         return emptyStore();
     }
-    return parseRuleStore(contents);
+    try {
+        return parseRuleStore(contents);
+    }
+    catch (error) {
+        throw new Error(`${error instanceof Error ? error.message : String(error)} — ${path}`);
+    }
 }
+/**
+ * Written via a temporary file and renamed into place, so an interrupted
+ * write cannot leave a half-written store behind.
+ */
 export function writeStore(path, store) {
     mkdirSync(dirname(path), { recursive: true });
-    writeFileSync(path, serializeRuleStore(store), 'utf8');
+    const temporary = `${path}.tmp`;
+    try {
+        writeFileSync(temporary, serializeRuleStore(store), 'utf8');
+        renameSync(temporary, path);
+    }
+    catch (error) {
+        try {
+            unlinkSync(temporary);
+        }
+        catch {
+            // Nothing to clean up.
+        }
+        throw error;
+    }
 }

package/dist/persistence/text-file.js

@@ -0,0 +1,23 @@
+import { readFileSync } from 'node:fs';
+// Files exported on one platform are routinely imported on another. Windows
+// tooling adds byte-order marks and PowerShell 5.1's `>` redirect writes
+// UTF-16LE, both of which break a plain UTF-8 read.
+export function decodeTextFile(bytes) {
+    if (bytes.length >= 3 && bytes[0] === 0xef && bytes[1] === 0xbb && bytes[2] === 0xbf) {
+        return bytes.subarray(3).toString('utf8');
+    }
+    if (bytes.length >= 2 && bytes[0] === 0xff && bytes[1] === 0xfe) {
+        return bytes.subarray(2).toString('utf16le');
+    }
+    if (bytes.length >= 2 && bytes[0] === 0xfe && bytes[1] === 0xff) {
+        const body = bytes.subarray(2);
+        if (body.length % 2 !== 0)
+            return body.toString('utf8');
+        // swap16 mutates, so work on a copy of the caller's buffer.
+        return Buffer.from(body).swap16().toString('utf16le');
+    }
+    return bytes.toString('utf8');
+}
+export function readTextFile(path) {
+    return decodeTextFile(readFileSync(path));
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lastboy/pai",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "PAI — Personal AI Supervisor: a local-first CLI that observes how you work with AI coding agents and turns it into knowledge you own",
   "keywords": [
     "cli",