@gotgenes/pi-permission-system 3.0.1 → 3.0.2

package/CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.0.2](https://github.com/gotgenes/pi-permission-system/compare/v3.0.1...v3.0.2) (2026-05-03)
+
+
+### Documentation
+
+* plan align test mock-cleanup and node:* default-export rules ([#35](https://github.com/gotgenes/pi-permission-system/issues/35)) ([480aa02](https://github.com/gotgenes/pi-permission-system/commit/480aa02183009a6693a6699948563345871f198d))
+* **retro:** add retro notes for issue [#21](https://github.com/gotgenes/pi-permission-system/issues/21) ([c7aae09](https://github.com/gotgenes/pi-permission-system/commit/c7aae099a48e58506195d843de797a1ae45b723a))
+
 ## [3.0.1](https://github.com/gotgenes/pi-permission-system/compare/v3.0.0...v3.0.1) (2026-05-03)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotgenes/pi-permission-system",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "description": "Permission enforcement extension for the Pi coding agent.",
   "type": "module",
   "main": "./index.ts",

package/tests/bash-filter.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, test, vi } from "vitest";
+import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
 
 import type { PermissionState } from "../src/types.js";
 
@@ -20,10 +20,15 @@ import {
   findCompiledWildcardMatch,
 } from "../src/wildcard-matcher.js";
 
+const mockedCompilePatterns = vi.mocked(compileWildcardPatterns);
 const mockedFindMatch = vi.mocked(findCompiledWildcardMatch);
 
+beforeEach(() => {
+  mockedCompilePatterns.mockClear();
+  mockedFindMatch.mockReset();
+});
+
 afterEach(() => {
-  vi.clearAllMocks();
   vi.restoreAllMocks();
 });
 

package/tests/external-directory.test.ts

@@ -2,9 +2,13 @@ import { join } from "node:path";
 import { afterEach, describe, expect, test, vi } from "vitest";
 
 // Mock node:os so tilde-expansion is deterministic across platforms.
-vi.mock("node:os", () => ({
-  homedir: vi.fn(() => "/mock/home"),
-}));
+vi.mock("node:os", () => {
+  const homedir = vi.fn(() => "/mock/home");
+  return {
+    homedir,
+    default: { homedir },
+  };
+});
 
 import {
   formatExternalDirectoryAskPrompt,

package/tests/permission-prompts.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, test, vi } from "vitest";
+import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
 
 // Mock tool-input-preview collaborator before importing the module under test.
 vi.mock("../src/tool-input-preview.js", () => ({
@@ -22,8 +22,11 @@ import type { PermissionCheckResult } from "../src/types.js";
 
 const mockedFormatToolInput = vi.mocked(formatToolInputForPrompt);
 
+beforeEach(() => {
+  mockedFormatToolInput.mockReset();
+});
+
 afterEach(() => {
-  vi.clearAllMocks();
   vi.restoreAllMocks();
 });
 

package/tests/tool-input-preview.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, test, vi } from "vitest";
+import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
 
 // Mock logging collaborator before importing the module under test.
 vi.mock("../src/logging.js", () => ({
@@ -29,8 +29,11 @@ import type { PermissionCheckResult } from "../src/types.js";
 
 const mockedStringify = vi.mocked(safeJsonStringify);
 
+beforeEach(() => {
+  mockedStringify.mockReset();
+});
+
 afterEach(() => {
-  vi.clearAllMocks();
   vi.restoreAllMocks();
 });