@andares/pi-toolkits 0.4.2 → 0.4.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andares/pi-toolkits",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "Integrated toolkit extension for pi — ask mode (read-only Q&A), model favorites, prompt stash, third-party code review, and a dedicated compaction model",
   "type": "module",
   "license": "MIT",

package/src/features/favorites/favorites.test.ts

@@ -23,7 +23,15 @@ import {
 	ModelSelectorComponent,
 	type Theme,
 } from "@earendil-works/pi-coding-agent";
-import { afterAll, beforeEach, beforeAll, describe, expect, it } from "vitest";
+import {
+	afterAll,
+	beforeEach,
+	beforeAll,
+	describe,
+	expect,
+	it,
+	vi,
+} from "vitest";
 import {
 	FavoritesStore,
 	getFavoritesStore,
@@ -102,14 +110,17 @@ describe("FavoritesStore", () => {
 	it("falls back to empty + defaults on a corrupted file", () => {
 		const file = freshStoreFile();
 		// Intentionally write invalid JSON — the READ side must tolerate it.
+		writeFileSync(file, "{ not json", "utf8");
+		// Suppress the designed degradation log; the spy asserts it fired.
+		const warn = vi.spyOn(console, "warn").mockImplementation(() => {});
 		try {
-			writeFileSync(file, "{ not json", "utf8");
-		} catch {
-			// A failed write still leaves the store to hit the read-error path.
+			const store = new FavoritesStore(file);
+			expect(store.count()).toBe(0);
+			expect(store.getConfig().cycleOnlyFavorites).toBe(true);
+			expect(warn).toHaveBeenCalled();
+		} finally {
+			warn.mockRestore();
 		}
-		const store = new FavoritesStore(file);
-		expect(store.count()).toBe(0);
-		expect(store.getConfig().cycleOnlyFavorites).toBe(true);
 	});
 
 	it("loads cycleOnlyFavorites config from disk", () => {

package/src/features/review/review.test.ts

@@ -204,13 +204,15 @@ describe("ReviewStore", () => {
 
 	it("falls back to unset on a corrupted file", () => {
 		const file = freshStoreFile();
+		writeFileSync(file, "{ not json", "utf8");
+		// Suppress the designed degradation log; the spy asserts it fired.
+		const warn = vi.spyOn(console, "warn").mockImplementation(() => {});
 		try {
-			writeFileSync(file, "{ not json", "utf8");
-		} catch {
-			// A failed write still leaves the store to hit the read-error path.
+			expect(new ReviewStore(file).getModel()).toBeUndefined();
+			expect(warn).toHaveBeenCalled();
+		} finally {
+			warn.mockRestore();
 		}
-		const store = new ReviewStore(file);
-		expect(store.getModel()).toBeUndefined();
 	});
 
 	it("ignores a stored value without a slash", () => {