@pi-archimedes/core 2.11.2 → 2.13.0

package/README.md

@@ -34,6 +34,6 @@ After installing Pi, choose one installation command above, then `cd` into your
 
 ## Part of the suite
 
-In [pi-archimedes](https://github.com/danielcherubini/pi-archimedes), core is always registered — it isn't one of the `/plugins` toggles — and it underpins what the other components share: the channel vocabulary that carries subagent todos to the task board and subagent questions to the ask UI, and the bridge that reaches the Archimedes Client. The diff renderer is standalone and does not depend on core's chrome — it only shares the suite when it loads.
+In [pi-archimedes](https://github.com/archimedes-codes/pi-archimedes), core is always registered — it isn't one of the `/plugins` toggles — and it underpins what the other components share: the channel vocabulary that carries subagent todos to the task board and subagent questions to the ask UI, and the bridge that reaches the Archimedes Client. The diff renderer is standalone and does not depend on core's chrome — it only shares the suite when it loads.
 
-← [Back to pi-archimedes](https://github.com/danielcherubini/pi-archimedes)
+← [Back to pi-archimedes](https://github.com/archimedes-codes/pi-archimedes)

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@pi-archimedes/core",
-  "version": "2.11.2",
+  "version": "2.13.0",
   "repository": {
     "type": "git",
-    "url": "https://github.com/danielcherubini/pi-archimedes.git"
+    "url": "https://github.com/archimedes-codes/pi-archimedes.git"
   },
   "type": "module",
   "keywords": [
@@ -36,8 +36,8 @@
     "@earendil-works/pi-tui": ">=0.85.0"
   },
   "devDependencies": {
-    "@earendil-works/pi-coding-agent": "1.2.0",
-    "@earendil-works/pi-tui": "1.2.0",
+    "@earendil-works/pi-coding-agent": "1.4.0",
+    "@earendil-works/pi-tui": "1.4.0",
     "typescript": "^6.0.0"
   },
   "pi": {

package/src/settings-io.test.ts

@@ -199,6 +199,44 @@ describe("loadConfig", () => {
     const result = loadConfig("test.ns", { foo: "bar", count: 42 });
     expect(result).toEqual({ foo: "bar", count: 42 });
   });
+
+  it("handles leading UTF-8 BOM in settings.json", () => {
+    const settingsPath = join(tempDir, "settings.json");
+    fs.writeFileSync(
+      settingsPath,
+      "\uFEFF" + JSON.stringify({ "test.ns": { foo: "overridden" } }),
+      "utf-8",
+    );
+    const result = loadConfig("test.ns", { foo: "bar", count: 42 });
+    expect(result).toEqual({ foo: "overridden", count: 42 });
+  });
+
+  it("prevents clobbering sibling namespaces when settings.json has UTF-8 BOM", () => {
+    const settingsPath = join(tempDir, "settings.json");
+    fs.writeFileSync(
+      settingsPath,
+      "\uFEFF" + JSON.stringify({ "test.ns": { foo: "overridden" } }),
+      "utf-8",
+    );
+    loadConfig("test.ns", { foo: "bar", count: 42 });
+    saveConfig("other.ns", { val: 123 });
+    const raw = fs.readFileSync(settingsPath, "utf-8");
+    expect(raw.startsWith("\uFEFF")).toBe(false);
+    const data = JSON.parse(raw);
+    expect(data["test.ns"]).toEqual({ foo: "overridden" });
+    expect(data["other.ns"]).toEqual({ val: 123 });
+  });
+
+  it("readEffectiveSettings returns parsed object from BOM-prefixed settings.json (file fallback)", () => {
+    const settingsPath = join(tempDir, "settings.json");
+    fs.writeFileSync(
+      settingsPath,
+      "\uFEFF" + JSON.stringify({ "test.ns": { foo: "overridden" } }),
+      "utf-8",
+    );
+    const result = readEffectiveSettings();
+    expect(result).toEqual({ "test.ns": { foo: "overridden" } });
+  });
 });
 
 describe("saveConfig", () => {

package/src/settings-io.ts

@@ -22,8 +22,9 @@ function readRawSettings(): string | null {
 /** Parse a raw settings string, returning empty object on null/corrupt input. */
 function parseSettings(raw: string | null): Record<string, unknown> {
   if (raw === null) return {};
+  const sanitized = raw.startsWith("\uFEFF") ? raw.slice(1) : raw;
   try {
-    return JSON.parse(raw) as Record<string, unknown>;
+    return JSON.parse(sanitized) as Record<string, unknown>;
   } catch {
     return {};
   }