@gotgenes/pi-permission-system 1.2.0 → 2.0.0

package/CHANGELOG.md

@@ -5,6 +5,41 @@ 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).
 
+## [2.0.0](https://github.com/gotgenes/pi-permission-system/compare/v1.2.1...v2.0.0) (2026-05-03)
+
+
+### ⚠ BREAKING CHANGES
+
+* the pi-permission-system:permission-request event channel is no longer emitted. No known consumers exist; the type was never exported. Re-adding with a proper public contract is tracked
+
+### Features
+
+* add /build-plan prompt template for non-TDD plans ([e98f13c](https://github.com/gotgenes/pi-permission-system/commit/e98f13c2ef32f51edda58ea065635bef31365baa))
+* delete permission-request event channel ([#20](https://github.com/gotgenes/pi-permission-system/issues/20)) ([6a41cfa](https://github.com/gotgenes/pi-permission-system/commit/6a41cfadc56e709d255538f63ff63d587e1b64f3))
+
+
+### Documentation
+
+* plan delete permission-request event channel ([#20](https://github.com/gotgenes/pi-permission-system/issues/20)) ([e202350](https://github.com/gotgenes/pi-permission-system/commit/e2023509f9ab849f5a1d8bc28a5705b2898b912b))
+* remove event channel from preserved-identity list ([#20](https://github.com/gotgenes/pi-permission-system/issues/20)) ([52299a2](https://github.com/gotgenes/pi-permission-system/commit/52299a27aeaed6e35849878fa224d8a78dcf0f6d))
+* **retro:** add retro notes for issue [#22](https://github.com/gotgenes/pi-permission-system/issues/22) ([55629fe](https://github.com/gotgenes/pi-permission-system/commit/55629fed16b6d73b6d7b02698227e2adab7acd3e))
+* update copyright in license ([b27994e](https://github.com/gotgenes/pi-permission-system/commit/b27994e7d67863ce8b28aa6bd680b60bc700d66d))
+
+## [1.2.1](https://github.com/gotgenes/pi-permission-system/compare/v1.2.0...v1.2.1) (2026-05-03)
+
+
+### Bug Fixes
+
+* **retro:** correct MD060 rule — column alignment, not separator spacing ([7f116b8](https://github.com/gotgenes/pi-permission-system/commit/7f116b884c03d9c346a640f306bca934b91214cd))
+
+
+### Documentation
+
+* plan relax on-disk identity rule ([#22](https://github.com/gotgenes/pi-permission-system/issues/22)) ([d886862](https://github.com/gotgenes/pi-permission-system/commit/d886862a3686746f48e618ab2f950c7b1109d804))
+* relax on-disk identity rule for config/log paths ([#22](https://github.com/gotgenes/pi-permission-system/issues/22)) ([352b103](https://github.com/gotgenes/pi-permission-system/commit/352b1038b1492b1f8222950edf4b6d093157128d))
+* **retro:** add retro notes for issue [#19](https://github.com/gotgenes/pi-permission-system/issues/19) ([1d4a4a6](https://github.com/gotgenes/pi-permission-system/commit/1d4a4a6959905f102dea08273e7b827d8111a583))
+* update README badges to match pi-autoformat style ([5c6ef1f](https://github.com/gotgenes/pi-permission-system/commit/5c6ef1fcfb2b6f5ee353765ebeeac7cdf09d2bbb))
+
 ## [1.2.0](https://github.com/gotgenes/pi-permission-system/compare/v1.1.0...v1.2.0) (2026-05-03)
 
 

package/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2026 MasuRii
+Copyright (c) 2026 MasuRii and Christopher D. Lasher
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/README.md

@@ -1,11 +1,12 @@
 # 🔐 @gotgenes/pi-permission-system
 
-[![Version](https://img.shields.io/badge/version-0.4.6-blue.svg)](package.json)
-[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
+[![npm version](https://img.shields.io/npm/v/@gotgenes/pi-permission-system?style=flat&logo=npm&logoColor=white)](https://www.npmjs.com/package/@gotgenes/pi-permission-system) [![CI](https://img.shields.io/github/actions/workflow/status/gotgenes/pi-permission-system/ci.yml?style=flat&logo=github&label=CI)](https://github.com/gotgenes/pi-permission-system/actions/workflows/ci.yml) [![License: MIT](https://img.shields.io/badge/License-MIT-blue.svg?style=flat)](https://opensource.org/licenses/MIT) [![TypeScript](https://img.shields.io/badge/TypeScript-6.x-3178C6?style=flat&logo=typescript&logoColor=white)](https://www.typescriptlang.org/) [![Pi Package](https://img.shields.io/badge/Pi-Package-6366F1?style=flat)](https://pi.mariozechner.at/)
 
 Permission enforcement extension for the Pi coding agent that provides centralized, deterministic permission gates for tool, bash, MCP, skill, and special operations.
 
-> **Fork notice:** This package is a friendly fork of [MasuRii/pi-permission-system](https://github.com/MasuRii/pi-permission-system), published to npm as `@gotgenes/pi-permission-system`. The extension's on-disk identity (config directory, log filenames, `/permission-system` slash command, and event channel names) is intentionally preserved so this fork and upstream share runtime state and remain drop-in interchangeable.
+> **Fork notice:** This package is a friendly fork of [MasuRii/pi-permission-system](https://github.com/MasuRii/pi-permission-system), published to npm as `@gotgenes/pi-permission-system`.
+> This fork diverges from upstream in config layout (#10).
+> The `/permission-system` slash command name is preserved; the config and log paths are not.
 
 ## Features
 

package/package.json

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

package/src/index.ts

@@ -79,26 +79,7 @@ const PERMISSION_FORWARDING_DIR = join(SESSIONS_DIR, "permission-forwarding");
 
 const ACTIVE_AGENT_TAG_REGEX = /<active_agent\s+name=["']([^"']+)["'][^>]*>/i;
 
-type PermissionRequestSource = "tool_call" | "skill_input" | "skill_read";
-type PermissionRequestState = "waiting" | "approved" | "denied";
-
-type PermissionRequestEvent = {
-  requestId: string;
-  source: PermissionRequestSource;
-  state: PermissionRequestState;
-  message: string;
-  toolCallId?: string;
-  toolName?: string;
-  skillName?: string;
-  path?: string;
-  command?: string;
-  target?: string;
-  toolInputPreview?: string;
-  agentName?: string | null;
-};
-
-const PERMISSION_REQUEST_EVENT_CHANNEL =
-  "pi-permission-system:permission-request";
+type PermissionReviewSource = "tool_call" | "skill_input" | "skill_read";
 const PATH_BEARING_TOOLS = new Set([
   "read",
   "write",
@@ -1347,24 +1328,11 @@ export default function piPermissionSystemExtension(pi: ExtensionAPI): void {
     return `${prefix}-${Date.now()}-${Math.random().toString(36).slice(2, 10)}-${process.pid}`;
   };
 
-  const emitPermissionRequestEvent = (event: PermissionRequestEvent): void => {
-    try {
-      pi.events.emit(PERMISSION_REQUEST_EVENT_CHANNEL, event);
-    } catch (error) {
-      writeDebugLog("permission_request.event_emit_failed", {
-        requestId: event.requestId,
-        source: event.source,
-        state: event.state,
-        error: formatUnknownErrorMessage(error),
-      });
-    }
-  };
-
   const reviewPermissionDecision = (
     event: string,
     details: {
       requestId: string;
-      source: PermissionRequestSource;
+      source: PermissionReviewSource;
       agentName: string | null;
       message: string;
       toolCallId?: string;
@@ -1399,7 +1367,7 @@ export default function piPermissionSystemExtension(pi: ExtensionAPI): void {
     ctx: ExtensionContext,
     details: {
       requestId: string;
-      source: PermissionRequestSource;
+      source: PermissionReviewSource;
       agentName: string | null;
       message: string;
       toolCallId?: string;
@@ -1413,38 +1381,10 @@ export default function piPermissionSystemExtension(pi: ExtensionAPI): void {
   ): Promise<PermissionPromptDecision> => {
     if (shouldAutoApprovePermissionState("ask", extensionConfig)) {
       reviewPermissionDecision("permission_request.auto_approved", details);
-      emitPermissionRequestEvent({
-        requestId: details.requestId,
-        source: details.source,
-        state: "approved",
-        message: details.message,
-        toolCallId: details.toolCallId,
-        toolName: details.toolName,
-        skillName: details.skillName,
-        path: details.path,
-        command: details.command,
-        target: details.target,
-        toolInputPreview: details.toolInputPreview,
-        agentName: details.agentName,
-      });
       return { approved: true, state: "approved" };
     }
 
     reviewPermissionDecision("permission_request.waiting", details);
-    emitPermissionRequestEvent({
-      requestId: details.requestId,
-      source: details.source,
-      state: "waiting",
-      message: details.message,
-      toolCallId: details.toolCallId,
-      toolName: details.toolName,
-      skillName: details.skillName,
-      path: details.path,
-      command: details.command,
-      target: details.target,
-      toolInputPreview: details.toolInputPreview,
-      agentName: details.agentName,
-    });
 
     const decision = await confirmPermission(ctx, details.message);
     reviewPermissionDecision(
@@ -1457,21 +1397,6 @@ export default function piPermissionSystemExtension(pi: ExtensionAPI): void {
         denialReason: decision.denialReason,
       },
     );
-    emitPermissionRequestEvent({
-      requestId: details.requestId,
-      source: details.source,
-      state: decision.approved ? "approved" : "denied",
-      message: details.message,
-      toolCallId: details.toolCallId,
-      toolName: details.toolName,
-      skillName: details.skillName,
-      path: details.path,
-      command: details.command,
-      target: details.target,
-      toolInputPreview: details.toolInputPreview,
-      agentName: details.agentName,
-    });
-
     return decision;
   };