@gotgenes/pi-permission-system 10.8.0 → 10.9.0

package/CHANGELOG.md

@@ -5,6 +5,13 @@ 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).
 
+## [10.9.0](https://github.com/gotgenes/pi-packages/compare/pi-permission-system-v10.8.0...pi-permission-system-v10.9.0) (2026-06-10)
+
+
+### Features
+
+* add ToolInputFormatterRegistrar write-side interface ([#366](https://github.com/gotgenes/pi-packages/issues/366)) ([e000eb0](https://github.com/gotgenes/pi-packages/commit/e000eb02a507c06e241b5a35cac5334e06dca1e2))
+
 ## [10.8.0](https://github.com/gotgenes/pi-packages/compare/pi-permission-system-v10.7.2...pi-permission-system-v10.8.0) (2026-06-10)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotgenes/pi-permission-system",
-  "version": "10.8.0",
+  "version": "10.9.0",
   "description": "Permission enforcement extension for the Pi coding agent.",
   "type": "module",
   "exports": {

package/src/permissions-service.ts

@@ -1,10 +1,10 @@
 import { buildInputForSurface } from "./input-normalizer";
-import type { PermissionManager } from "./permission-manager";
+import type { ScopedPermissionManager } from "./permission-manager";
 import type { PermissionsService } from "./service";
 import type { SessionRules } from "./session-rules";
 import type {
   ToolInputFormatter,
-  ToolInputFormatterRegistry,
+  ToolInputFormatterRegistrar,
 } from "./tool-input-formatter-registry";
 
 /**
@@ -16,9 +16,9 @@ import type {
  */
 export class LocalPermissionsService implements PermissionsService {
   constructor(
-    private readonly permissionManager: PermissionManager,
-    private readonly sessionRules: SessionRules,
-    private readonly formatterRegistry: ToolInputFormatterRegistry,
+    private readonly permissionManager: ScopedPermissionManager,
+    private readonly sessionRules: Pick<SessionRules, "getRuleset">,
+    private readonly formatterRegistry: ToolInputFormatterRegistrar,
   ) {}
 
   checkPermission(

package/src/tool-input-formatter-registry.ts

@@ -19,6 +19,14 @@ export interface ToolInputFormatterLookup {
   get(toolName: string): ToolInputFormatter | undefined;
 }
 
+/**
+ * Registration side of the formatter registry (ISP — exposes only the
+ * write surface, mirroring the read-only {@link ToolInputFormatterLookup}).
+ */
+export interface ToolInputFormatterRegistrar {
+  register(toolName: string, formatter: ToolInputFormatter): () => void;
+}
+
 /**
  * Persistent registry mapping tool names to custom preview formatters.
  *
@@ -26,7 +34,9 @@ export interface ToolInputFormatterLookup {
  * per-tool-call `ToolPreviewFormatter` construction cycle.
  * Exposed to sibling extensions via `PermissionsService.registerToolInputFormatter`.
  */
-export class ToolInputFormatterRegistry implements ToolInputFormatterLookup {
+export class ToolInputFormatterRegistry
+  implements ToolInputFormatterLookup, ToolInputFormatterRegistrar
+{
   private readonly formatters = new Map<string, ToolInputFormatter>();
 
   /**

package/test/permissions-service.test.ts

@@ -1,14 +1,15 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
-import type { PermissionManager } from "#src/permission-manager";
+import type { ScopedPermissionManager } from "#src/permission-manager";
 import { LocalPermissionsService } from "#src/permissions-service";
 import type { Ruleset } from "#src/rule";
 import type { SessionRules } from "#src/session-rules";
 import type {
   ToolInputFormatter,
-  ToolInputFormatterRegistry,
+  ToolInputFormatterRegistrar,
 } from "#src/tool-input-formatter-registry";
 
 import { makeCheckResult } from "#test/helpers/handler-fixtures";
+import { makeFakePermissionManager } from "#test/helpers/session-fixtures";
 
 // ── input-normalizer stub ──────────────────────────────────────────────────
 
@@ -22,38 +23,29 @@ vi.mock("#src/input-normalizer", () => ({
 
 // ── helpers ────────────────────────────────────────────────────────────────
 
-function makePermissionManager(): PermissionManager {
-  return {
-    checkPermission: vi
-      .fn<PermissionManager["checkPermission"]>()
-      .mockReturnValue(makeCheckResult()),
-    getToolPermission: vi
-      .fn<PermissionManager["getToolPermission"]>()
-      .mockReturnValue("allow"),
-  } as unknown as PermissionManager;
-}
-
-function makeSessionRules(rules: Ruleset = []): SessionRules {
+function makeSessionRules(
+  rules: Ruleset = [],
+): Pick<SessionRules, "getRuleset"> {
   return {
     getRuleset: vi.fn<SessionRules["getRuleset"]>().mockReturnValue(rules),
-  } as unknown as SessionRules;
+  };
 }
 
-function makeFormatterRegistry(): ToolInputFormatterRegistry {
+function makeFormatterRegistry(): ToolInputFormatterRegistrar {
   return {
     register: vi
-      .fn<ToolInputFormatterRegistry["register"]>()
+      .fn<ToolInputFormatterRegistrar["register"]>()
       .mockReturnValue(vi.fn()),
-  } as unknown as ToolInputFormatterRegistry;
+  };
 }
 
 function makeService(overrides?: {
-  permissionManager?: PermissionManager;
-  sessionRules?: SessionRules;
-  formatterRegistry?: ToolInputFormatterRegistry;
+  permissionManager?: ScopedPermissionManager;
+  sessionRules?: Pick<SessionRules, "getRuleset">;
+  formatterRegistry?: ToolInputFormatterRegistrar;
 }) {
   const permissionManager =
-    overrides?.permissionManager ?? makePermissionManager();
+    overrides?.permissionManager ?? makeFakePermissionManager();
   const sessionRules = overrides?.sessionRules ?? makeSessionRules();
   const formatterRegistry =
     overrides?.formatterRegistry ?? makeFormatterRegistry();