@pi-archimedes/core 2.11.2 → 2.12.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-archimedes/core",
-  "version": "2.11.2",
+  "version": "2.12.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/danielcherubini/pi-archimedes.git"
@@ -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 {};
   }