@code-fixer-23/pi-session-manager 1.0.5 → 1.1.0

package/CHANGELOG.md

@@ -1,3 +1,29 @@
+## 1.1.0 (2026-08-23)
+
+### 🚀 Features
+
+- **pi-session-manager:** persist config through extension settings ([933f395](https://github.com/louiss0/pi-packages/commit/933f395))
+
+### 🩹 Fixes
+
+- **workspace:** pin typescript to ~6.0.3 across packages ([e11938a](https://github.com/louiss0/pi-packages/commit/e11938a))
+- **pi-session-manager:** align test mocks with strict compiler flags ([fba5182](https://github.com/louiss0/pi-packages/commit/fba5182))
+
+### ❤️ Thank You
+
+- louiss0 @louiss0
+
+## 1.0.6 (2026-07-14)
+
+### 🩹 Fixes
+
+- **pi-session-manager:** display README asset ([4628004](https://github.com/louiss0/pi-packages/commit/4628004))
+- **pi-session-manager:** respect empty title cancellation ([37729af](https://github.com/louiss0/pi-packages/commit/37729af))
+
+### ❤️ Thank You
+
+- louiss0 @louiss0
+
 ## 1.0.5 (2026-07-08)
 
 ### 🩹 Fixes

package/README.md

@@ -1,76 +1,99 @@
-# pi-session-manager
-
-[!session-manager](https://raw.githubusercontent.com/louiss0/pi-packages/main/pi-session-manager/assets/Pi-Session-Manager-Big.png)
-
-
-[![npm version](https://img.shields.io/npm/v/@code-fixer-23/pi-session-manager.svg)](https://www.npmjs.com/package/@code-fixer-23/pi-session-manager)
-[![downloads](https://img.shields.io/npm/dm/@code-fixer-23/pi-session-manager.svg)](https://www.npmjs.com/package/@code-fixer-23/pi-session-manager)
-[![license](https://img.shields.io/npm/l/@code-fixer-23/pi-session-manager.svg)](./LICENSE)
-[![CI](https://github.com/louiss0/pi-packages/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/louiss0/pi-packages/actions/workflows/ci.yml)
-
-
-`pi-session-manager` adds local session cleanup and series management workflows to PI. It helps keep session history organized by pruning stale sessions stored on your machine, naming related sessions consistently, and carrying series context forward between sessions. The package plugs into PI's extension lifecycle so it can react when a session starts, restore persisted series metadata, and expose commands that coordinate session creation, continuation, and cleanup.
-
-## pi-session-manager
-
-This package ships a single PI extension in `extensions/index.ts`, supported by a small test suite and a helper script for scaffolding additional extension entry points.
-
-### Behavior overview
-
-The extension centers on two workflows:
-
-1. **Session cleanup** — it can remove inactive local sessions automatically on startup or manually through commands.
-2. **Session series management** — it records related sessions under a shared series name, preserves that context across session restarts, and lets you continue or extend a series later.
-
-On session start, the extension first restores any persisted series data from the temp directory so PI can resume the correct session name and custom entry. When the session starts normally or after a reload, it also loads the package config, checks the cleanup rules, and removes stale local sessions if they exceed the configured age limit.
-
-If you want a session gone, delete it through the package commands instead of editing the config by hand. The config is an internal coordination file, not a user-edit surface.
-
-> Warning: do not edit `pi-session-manager.config.json` directly. If you need to remove a series, use `session:series delete` and let the package update its own state.
-
-### Commands
-
-#### `session:clean:inactive`
-Deletes local sessions that have been inactive longer than the configured day limit. This command uses the package config stored in the agent directory, so the cleanup window matches the same rule used during startup cleanup. Use this when you want PI to resume with a clean local session set without manually touching files.
-
-#### `session:clean:older-than <duration>`
-Deletes local sessions older than a specific duration. The argument accepts either full units like `5days`, `2weeks`, `12hours` or shorthand forms like `5d`, `2w`, `12h`. This is useful when you want to remove sessions using an ad hoc retention threshold instead of the saved day limit.
-
-#### `session:delete-last <count>`
-Deletes the most recent N local sessions. The argument is an integer from 1 to 10, which makes it a quick recovery tool for removing only the newest session files when you want to clean up by hand.
-
-#### `session:series <action>`
-Coordinates the session-series lifecycle. The available actions are:
-
-- `create` — create a new series and a first session inside it, then persist the resulting session metadata so PI can restore the series context later.
-- `new` — add a new session to an existing series and keep the series record updated.
-- `continue` — inspect the currently active session entry, recover its series, and start the next session in that same chain.
-- `delete` — remove a series and all local session files whose names start with that series prefix. This is the safest way to remove a series; avoid editing the config file directly.
-
-### Features
-
-#### Startup lifecycle coordination
-The extension listens to PI session startup and reload events to restore persisted session-series data before the rest of the session flow continues. That lets PI reopen the correct session name and custom entry even after a new process starts.
-
-#### Persistent session-series state
-When a series session is created, the extension writes a small temp file in the OS temp directory. On the next session start, it consumes that file, applies the stored session name and entry, and then removes the temp file so the state only applies once.
-
-#### Configured cleanup rules
-The package stores its config in the PI agent directory as `pi-session-manager.config.json`. That config controls the automatic inactivity cleanup threshold and remembers the series/title structure per working directory.
-
-Treat this file as internal state. Do not edit it manually unless you are debugging the package itself. If you need to remove a series, use `session:series delete` instead of changing the config file.
-
-The package is designed to work with local session files only, so it should never be treated as a remote or shared-session cleanup tool.
-
-### Developer notes
-
-- Package metadata lives in `package.json` and identifies this as a PI extension package.
-- The implementation is in `extensions/index.ts`.
-- Tests live alongside the extension in `extensions/index.test.ts`.
-- `scripts/create-extension.ts` is a small helper for scaffolding a new extension file under `extensions/`.
-
-### Assets in this repository
-
-- `extensions/` — extension implementation and tests
-- `scripts/` — helper script for creating new extension files
-- No `prompts/`, `skills/`, or `themes/` directories are present in this package
+# pi-session-manager
+
+![Session Manager](https://raw.githubusercontent.com/louiss0/pi-packages/main/pi-session-manager/assets/Pi-Session_Manager-Big.png)
+
+
+[![npm version](https://img.shields.io/npm/v/@code-fixer-23/pi-session-manager.svg)](https://www.npmjs.com/package/@code-fixer-23/pi-session-manager)
+[![downloads](https://img.shields.io/npm/dm/@code-fixer-23/pi-session-manager.svg)](https://www.npmjs.com/package/@code-fixer-23/pi-session-manager)
+[![license](https://img.shields.io/npm/l/@code-fixer-23/pi-session-manager.svg)](./LICENSE)
+[![CI](https://github.com/louiss0/pi-packages/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/louiss0/pi-packages/actions/workflows/ci.yml)
+
+
+`pi-session-manager` adds local session cleanup and series management workflows to PI. It helps keep session history organized by pruning stale sessions stored on your machine, naming related sessions consistently, and carrying series context forward between sessions. The package plugs into PI's extension lifecycle so it can react when a session starts, restore persisted series metadata, and expose commands that coordinate session creation, continuation, and cleanup.
+
+## pi-session-manager
+
+This package ships a single PI extension in `extensions/index.ts`, supported by a small test suite and a helper script for scaffolding additional extension entry points.
+
+### Behavior overview
+
+The extension centers on two workflows:
+
+1. **Session cleanup** — it can remove inactive local sessions automatically on startup or manually through commands.
+2. **Session series management** — it records related sessions under a shared series name, preserves that context across session restarts, and lets you continue or extend a series later.
+
+On session start, the extension first restores any persisted series data from the temp directory so PI can resume the correct session name and custom entry. When the session starts normally or after a reload, it also loads the package config, checks the cleanup rules, and removes stale local sessions if they exceed the configured age limit.
+
+If you want a session gone, delete it through the package commands instead of editing the config by hand. The config is an internal coordination file, not a user-edit surface.
+
+> Warning: do not edit the extension's stored settings directly. If you need to remove a series, use `session-manager:series delete` and let the package update its own state.
+
+### Commands
+
+All commands live under the `session-manager` namespace. Every cleanup command that works on the current project has an `:all` twin that works on **every** session in the sessions area (`~/.pi/agent/sessions/`), no matter which project it belongs to. After any deletion, you are shown the list of deleted sessions grouped and sorted by project name.
+
+#### `session-manager:clean:inactive`
+Deletes local sessions that have been inactive longer than the configured day limit. This command reads the day limit from the shared extension settings, so the cleanup window matches the same rule used during startup cleanup. Use this when you want PI to resume with a clean local session set without manually touching files.
+
+#### `session-manager:clean:inactive:all`
+The same inactivity rule as `session-manager:clean:inactive`, but applied to every session stored in the sessions area across all projects. This is the tool for pruning outdated sessions from projects you no longer work in.
+
+#### `session-manager:clean:older-than <duration>`
+Deletes local sessions older than a specific duration. The argument accepts either full units like `5days`, `2weeks`, `12hours` or shorthand forms like `5d`, `2w`, `12h`. This is useful when you want to remove sessions using an ad hoc retention threshold instead of the saved day limit.
+
+#### `session-manager:clean:older-than:all <duration>`
+The same duration rule as `session-manager:clean:older-than`, but applied to all sessions in the sessions area across every project.
+
+#### `session-manager:delete-last <count>`
+Deletes the most recent N local sessions. The argument is an integer from 1 to 10, which makes it a quick recovery tool for removing only the newest session files when you want to clean up by hand.
+
+#### `session-manager:delete-last:all <count>`
+Deletes the most recent N sessions across every project in the sessions area. Like the local variant, the count is an integer from 1 to 10. A completion hint labels each option as "last N in every project" so it is easy to tell apart from the project-scoped command.
+
+#### `session-manager:series <action>`
+Coordinates the session-series lifecycle. The available actions are:
+
+- `create` — create a new series and a first session inside it, then persist the resulting session metadata so PI can restore the series context later.
+- `new` — add a new session to an existing series and keep the series record updated.
+- `continue` — inspect the currently active session entry, recover its series, and start the next session in that same chain.
+- `delete` — remove a series and all local session files whose names start with that series prefix. This is the safest way to remove a series; avoid editing the config file directly.
+
+### Deleted session report
+Every manual cleanup command (local and `:all`) finishes by presenting what was removed. Sessions are sorted by project name — derived from the deepest folder segment of each session's working directory — then by modified time inside each project, and rendered as a grouped listing:
+
+```
+Removed 3 session(s):
+project-a
+  Refactor Auth--Create JWT Token
+project-b
+  Fix Memory Leak--Patch Heap Snapshot
+  UI Migration--Move Button Component
+```
+
+### Features
+
+#### Startup lifecycle coordination
+The extension listens to PI session startup and reload events to restore persisted session-series data before the rest of the session flow continues. That lets PI reopen the correct session name and custom entry even after a new process starts.
+
+#### Persistent session-series state
+When a series session is created, the extension writes a small temp file in the OS temp directory. On the next session start, it consumes that file, applies the stored session name and entry, and then removes the temp file so the state only applies once.
+
+#### Configured cleanup rules
+Settings are persisted through [@juanibiapina/pi-extension-settings](https://www.npmjs.com/package/@juanibiapina/pi-extension-settings). The `Session Deletion Day Limit` setting controls the automatic inactivity cleanup threshold and can be changed from PI with the `/extension-settings` command (default: 3 days). The series/title structure per working directory is remembered in the same store under the `pi-session-manager` extension name.
+
+Treat these stored values as internal state. Do not edit them manually unless you are debugging the package itself. If you need to remove a series, use `session-manager:series delete` instead of changing the stored settings.
+
+The package is designed to work with local session files only, so it should never be treated as a remote or shared-session cleanup tool.
+
+### Developer notes
+
+- Package metadata lives in `package.json` and identifies this as a PI extension package.
+- The implementation is in `extensions/index.ts`.
+- Tests live alongside the extension in `extensions/index.test.ts`.
+- `scripts/create-extension.ts` is a small helper for scaffolding a new extension file under `extensions/`.
+
+### Assets in this repository
+
+- `extensions/` — extension implementation and tests
+- `scripts/` — helper script for creating new extension files
+- No `prompts/`, `skills/`, or `themes/` directories are present in this package

package/extensions/index.test.ts

@@ -11,6 +11,10 @@ import {
   handleSessionCleanOlderThan,
   handleSessionDeleteLast,
   handleSessionSeries,
+  formatDeletedSessionsListing,
+  getSessionProjectName,
+  sortSessionsByProjectName,
+  presentDeletedSessions,
   getSessionEntryWithSeries,
   getSessionSeriesDataTempPath,
   persistSessionSeriesData,
@@ -226,6 +230,131 @@ class SessionManagerConfiguratorMock implements $SessionManagerConfigurator {
   }
 }
 
+describe("deleted session reporting", () => {
+  type SessionOverrides = {
+    [K in keyof SessionInfo]?: SessionInfo[K] | undefined;
+  };
+
+  const buildSession = (overrides: SessionOverrides = {}): SessionInfo =>
+    ({
+      path: "/path/to/session",
+      id: "session-id",
+      cwd: "/user/work/project-a",
+      name: "Session A",
+      parentSessionPath: "/path/to/parent",
+      created: new Date(0),
+      modified: new Date(0),
+      messageCount: 1,
+      firstMessage: "Hello from session A",
+      allMessagesText: "Full history for session A",
+      ...overrides,
+    }) as SessionInfo;
+
+  it("derives the project name from the deepest cwd segment", () => {
+    const posixSession = buildSession({ cwd: "/user/work/project-a" });
+    const windowsSession = buildSession({
+      cwd: "C:\\Users\\bvlou\\projects\\project-b",
+    });
+
+    expect(getSessionProjectName(posixSession)).toBe("project-a");
+    expect(getSessionProjectName(windowsSession)).toBe("project-b");
+  });
+
+  it("sorts sessions by project name before modified time", () => {
+    const projectBSession = buildSession({
+      id: "project-b-session",
+      name: "Project B Session",
+      cwd: "/user/work/project-b",
+      modified: new Date(1000),
+    });
+    const projectAOldestSession = buildSession({
+      id: "project-a-oldest",
+      name: "Project A Oldest",
+      cwd: "/user/work/project-a",
+      modified: new Date(1000),
+    });
+    const projectARecentSession = buildSession({
+      id: "project-a-recent",
+      name: "Project A Recent",
+      cwd: "/user/work/project-a",
+      modified: new Date(5000),
+    });
+
+    expect(
+      sortSessionsByProjectName([
+        projectBSession,
+        projectARecentSession,
+        projectAOldestSession,
+      ]),
+    ).toEqual([projectAOldestSession, projectARecentSession, projectBSession]);
+  });
+
+  it("formats deleted sessions grouped under sorted project names", () => {
+    const projectBSession = buildSession({
+      name: "Project B Session",
+      cwd: "/user/work/project-b",
+    });
+    const projectASession = buildSession({
+      name: "Project A Session",
+      cwd: "/user/work/project-a",
+    });
+
+    expect(
+      formatDeletedSessionsListing([projectBSession, projectASession]),
+    ).toBe(
+      [
+        "Removed 2 session(s):",
+        "project-a",
+        "  Project A Session",
+        "project-b",
+        "  Project B Session",
+      ].join("\n"),
+    );
+  });
+
+  it("falls back to first message then id when a session has no name", () => {
+    const unnamedSession = buildSession({ name: undefined });
+
+    expect(formatDeletedSessionsListing([unnamedSession])).toBe(
+      ["Removed 1 session(s):", "project-a", "  Hello from session A"].join(
+        "\n",
+      ),
+    );
+  });
+
+  it("notifies nothing was deleted when the deletion list is empty", () => {
+    const context = {
+      ui: {
+        notify: vi.fn<ExtensionUIContext["notify"]>(),
+      },
+    } satisfies MockExtenstionCommandContext;
+
+    presentDeletedSessions([], castToExtensionContext(context));
+
+    expect(context.ui.notify).toHaveBeenCalledWith(
+      "No sessions matched, so nothing was deleted.",
+      "info",
+    );
+  });
+
+  it("presents the formatted listing when sessions were deleted", () => {
+    const context = {
+      ui: {
+        notify: vi.fn<ExtensionUIContext["notify"]>(),
+      },
+    } satisfies MockExtenstionCommandContext;
+
+    const deletedSession = buildSession();
+
+    presentDeletedSessions([deletedSession], castToExtensionContext(context));
+
+    expect(context.ui.notify).toHaveBeenCalledWith(
+      formatDeletedSessionsListing([deletedSession]),
+      "info",
+    );
+  });
+});
+
 // THis is written like this so that I can use the outline to find tests
 const test = it
   .extend("sessions", () => {
@@ -449,7 +578,7 @@ describe("handleSessionCleanOlderThan", () => {
     );
 
     expect(context.ui.notify).toHaveBeenCalledWith(
-      `Deleteing sessions that are from ${durationRecord.integer} ${durationRecord.unit} ago`,
+      `Deleting sessions that are from ${durationRecord.integer} ${durationRecord.unit} ago`,
     );
 
     expect(
@@ -551,8 +680,8 @@ describe("handleSessionSeries", () => {
           getSessionEntryWithSeries() {
             return undefined;
           },
-          removeSessionFiles() {
-            return;
+          removeSessionFiles(sessions) {
+            return sessions;
           },
         },
         castToExtensionContext(context),
@@ -609,8 +738,43 @@ describe("handleSessionSeries", () => {
           getSessionEntryWithSeries() {
             return undefined;
           },
-          removeSessionFiles() {
-            return;
+          removeSessionFiles(sessions) {
+            return sessions;
+          },
+        },
+        castToExtensionContext(context),
+      );
+
+      expect(context.ui.input).toHaveBeenCalledTimes(1);
+      expect(context.newSession).not.toHaveBeenCalled();
+    });
+
+    it("stops creating a session series when the series input is empty", async () => {
+      const context = {
+        cwd: "/pi-packages",
+        newSession: vi.fn<ExtensionCommandContext["newSession"]>(),
+        ui: {
+          notify: vi.fn<ExtensionUIContext["notify"]>(),
+          input: vi
+            .fn<ExtensionUIContext["input"]>()
+            .mockResolvedValueOnce("")
+            .mockResolvedValueOnce("Should not be prompted"),
+        },
+      } satisfies MockExtenstionCommandContext;
+
+      await handleSessionSeries(
+        "create",
+        {
+          sessionManagerConfigurator: new SessionManagerConfiguratorMock(),
+          sessionFilter: new MockSessionFilter(
+            [],
+            new MockPastTimestampCalculator(),
+          ),
+          getSessionEntryWithSeries() {
+            return undefined;
+          },
+          removeSessionFiles(sessions) {
+            return sessions;
           },
         },
         castToExtensionContext(context),
@@ -641,6 +805,7 @@ describe("handleSessionSeries", () => {
           notify: vi.fn<ExtensionUIContext["notify"]>(),
           input: vi
             .fn<ExtensionUIContext["input"]>()
+            .mockResolvedValueOnce(" ")
             .mockResolvedValueOnce("  Implement Auth  ")
             .mockResolvedValueOnce("  Implement Billing  ")
             .mockResolvedValueOnce("  Create JWT Token  "),
@@ -675,8 +840,8 @@ describe("handleSessionSeries", () => {
           getSessionEntryWithSeries() {
             return undefined;
           },
-          removeSessionFiles() {
-            return;
+          removeSessionFiles(sessions) {
+            return sessions;
           },
         },
         castToExtensionContext(context),
@@ -694,7 +859,7 @@ describe("handleSessionSeries", () => {
         "warning",
       );
 
-      expect(context.ui.input).toHaveBeenCalledTimes(3);
+      expect(context.ui.input).toHaveBeenCalledTimes(4);
 
       expect(appendSessionSeriesBasedOnCwdSpy).toHaveBeenCalledWith(
         sessionCtx.cwd,
@@ -900,8 +1065,8 @@ describe("handleSessionSeries", () => {
         getSessionEntryWithSeries() {
           return undefined;
         },
-        removeSessionFiles() {
-          return;
+        removeSessionFiles(sessions) {
+          return sessions;
         },
       },
       castToExtensionContext(context),
@@ -980,8 +1145,8 @@ describe("handleSessionSeries", () => {
         getSessionEntryWithSeries() {
           return undefined;
         },
-        removeSessionFiles() {
-          return;
+        removeSessionFiles(sessions) {
+          return sessions;
         },
       },
       castToExtensionContext(context),
@@ -1050,8 +1215,8 @@ describe("handleSessionSeries", () => {
         getSessionEntryWithSeries() {
           return undefined;
         },
-        removeSessionFiles() {
-          return;
+        removeSessionFiles(sessions) {
+          return sessions;
         },
       },
       castToExtensionContext(context),
@@ -1117,8 +1282,8 @@ describe("handleSessionSeries", () => {
           } as SessionSeriesEntry;
         },
         sessionManagerConfigurator: new SessionManagerConfiguratorMock(),
-        removeSessionFiles() {
-          return;
+        removeSessionFiles(sessions) {
+          return sessions;
         },
       },
       castToExtensionContext(context),
@@ -1209,8 +1374,8 @@ describe("handleSessionSeries", () => {
         getSessionEntryWithSeries,
         sessionManagerConfigurator,
 
-        removeSessionFiles() {
-          return;
+        removeSessionFiles(sessions) {
+          return sessions;
         },
       },
       castToExtensionContext(context),