pi-session-cleanup 1.1.2 → 1.1.4

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [Unreleased]
+
+## [1.1.4] - 2026-06-16
+
+### Fixed
+- Added round-trip date validation for parsed calendar timestamps so invalid dates (e.g., month 13 or day 32) no longer produce false-positive sort keys.
+
+## 1.1.3 - 2026-06-01
+
+- Deferred command module loading for `/session-cleanup` and `/nix` handlers while preserving inline completions.
+- Widened Pi coding-agent and Pi TUI peer dependency ranges to include `^0.77.0 || ^0.78.0`.
+
 ## 1.1.2 - 2026-05-26
 
 - Widened peer dependency ranges to `^0.74.0 || ^0.75.0`.

package/package.json

@@ -1,66 +1,66 @@
-{
-  "name": "pi-session-cleanup",
-  "version": "1.1.2",
-  "description": "Pi extension for interactive batch session cleanup and safe deletion.",
-  "type": "module",
-  "main": "./index.ts",
-  "exports": {
-    ".": "./index.ts"
-  },
-  "files": [
-    "index.ts",
-    "src",
-    "config/config.example.json",
-    "README.md",
-    "CHANGELOG.md",
-    "LICENSE"
-  ],
-  "scripts": {
-    "build": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.json --noCheck",
-    "lint": "npm run build",
-    "test:clean": "node -e \"require('node:fs').rmSync('.test-dist', { recursive: true, force: true })\"",
-    "pretest": "npm run test:clean",
-    "test": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.test.json && node --test .test-dist/test/*.test.js",
-    "posttest": "npm run test:clean",
-    "check": "npm run lint && npm run test",
-    "package:dry-run": "npm pack --dry-run"
-  },
-  "keywords": [
-    "pi-package",
-    "pi",
-    "pi-extension",
-    "session",
-    "cleanup",
-    "delete",
-    "pi-coding-agent",
-    "pi-tui",
-    "session-management",
-    "tui",
-    "safe-delete"
-  ],
-  "author": "MasuRii",
-  "license": "MIT",
-  "engines": {
-    "node": ">=20"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "pi": {
-    "extensions": [
-      "./index.ts"
-    ]
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.74.0 || ^0.75.0",
-    "@earendil-works/pi-tui": "^0.74.0 || ^0.75.0"
-  },
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/MasuRii/pi-session-cleanup.git"
-  },
-  "bugs": {
-    "url": "https://github.com/MasuRii/pi-session-cleanup/issues"
-  },
-  "homepage": "https://github.com/MasuRii/pi-session-cleanup#readme"
-}
+{
+  "name": "pi-session-cleanup",
+  "version": "1.1.4",
+  "description": "Pi extension for interactive batch session cleanup and safe deletion.",
+  "type": "module",
+  "main": "./index.ts",
+  "exports": {
+    ".": "./index.ts"
+  },
+  "files": [
+    "index.ts",
+    "src",
+    "config/config.example.json",
+    "README.md",
+    "CHANGELOG.md",
+    "LICENSE"
+  ],
+  "scripts": {
+    "build": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.json --noCheck",
+    "lint": "npm run build",
+    "test:clean": "node -e \"require('node:fs').rmSync('.test-dist', { recursive: true, force: true })\"",
+    "pretest": "npm run test:clean",
+    "test": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.test.json && node --test .test-dist/test/*.test.js",
+    "posttest": "npm run test:clean",
+    "check": "npm run lint && npm run test",
+    "package:dry-run": "npm pack --dry-run"
+  },
+  "keywords": [
+    "pi-package",
+    "pi",
+    "pi-extension",
+    "session",
+    "cleanup",
+    "delete",
+    "pi-coding-agent",
+    "pi-tui",
+    "session-management",
+    "tui",
+    "safe-delete"
+  ],
+  "author": "MasuRii",
+  "license": "MIT",
+  "engines": {
+    "node": ">=20"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "pi": {
+    "extensions": [
+      "./index.ts"
+    ]
+  },
+  "peerDependencies": {
+    "@earendil-works/pi-coding-agent": "^0.74.0 || ^0.75.0 || ^0.78.0 || ^0.79.0",
+    "@earendil-works/pi-tui": "^0.74.0 || ^0.75.0 || ^0.78.0 || ^0.79.0"
+  },
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/MasuRii/pi-session-cleanup.git"
+  },
+  "bugs": {
+    "url": "https://github.com/MasuRii/pi-session-cleanup/issues"
+  },
+  "homepage": "https://github.com/MasuRii/pi-session-cleanup#readme"
+}

package/src/index.ts

@@ -1,16 +1,104 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 import { SESSION_CLEANUP_COMMAND, SESSION_NIX_COMMAND } from "./constants.js";
-import {
-  getSessionCleanupArgumentCompletions,
-  handleSessionCleanupCommand,
-} from "./session-cleanup-command.js";
-import {
-  getSessionNixArgumentCompletions,
-  handleSessionNixCommand,
-} from "./session-nix-command.js";
 import { flushScheduledSessionDeletionForQuit } from "./session-quit-shutdown.js";
 
+type SessionCleanupCommandModule = typeof import("./session-cleanup-command.js");
+type SessionNixCommandModule = typeof import("./session-nix-command.js");
+type CommandCompletion = {
+  value: string;
+  label: string;
+  description?: string;
+};
+
+const SESSION_CLEANUP_ARGUMENT_COMPLETIONS = [
+  {
+    value: "current",
+    label: "current",
+    description: "List sessions from the current working directory only",
+  },
+  {
+    value: "all",
+    label: "all",
+    description: "List sessions across every working directory",
+  },
+  {
+    value: "help",
+    label: "help",
+    description: "Show usage",
+  },
+] as const satisfies readonly CommandCompletion[];
+
+const SESSION_NIX_ARGUMENT_COMPLETIONS = [
+  {
+    value: "quit",
+    label: "quit",
+    description: "Delete the current session and quit Pi immediately",
+  },
+  {
+    value: "agent",
+    label: "agent",
+    description: "Start a fresh session with a selected target agent",
+  },
+  {
+    value: "help",
+    label: "help",
+    description: "Show usage",
+  },
+] as const satisfies readonly CommandCompletion[];
+
+let sessionCleanupCommandModule: SessionCleanupCommandModule | undefined;
+let sessionCleanupCommandModulePromise: Promise<SessionCleanupCommandModule> | undefined;
+let sessionNixCommandModule: SessionNixCommandModule | undefined;
+let sessionNixCommandModulePromise: Promise<SessionNixCommandModule> | undefined;
+
+function loadSessionCleanupCommandModule(): Promise<SessionCleanupCommandModule> {
+  if (sessionCleanupCommandModule) {
+    return Promise.resolve(sessionCleanupCommandModule);
+  }
+
+  sessionCleanupCommandModulePromise ??= import("./session-cleanup-command.js").then(
+    (module) => {
+      sessionCleanupCommandModule = module;
+      return module;
+    },
+  );
+  return sessionCleanupCommandModulePromise;
+}
+
+function loadSessionNixCommandModule(): Promise<SessionNixCommandModule> {
+  if (sessionNixCommandModule) {
+    return Promise.resolve(sessionNixCommandModule);
+  }
+
+  sessionNixCommandModulePromise ??= import("./session-nix-command.js").then(
+    (module) => {
+      sessionNixCommandModule = module;
+      return module;
+    },
+  );
+  return sessionNixCommandModulePromise;
+}
+
+function getMatchedCompletions(
+  argumentPrefix: string,
+  completions: readonly CommandCompletion[],
+): CommandCompletion[] | null {
+  const normalizedPrefix = argumentPrefix.trim().toLowerCase();
+  if (!normalizedPrefix) {
+    return completions.map((completion) => ({ ...completion }));
+  }
+
+  const matched = completions.filter((completion) =>
+    completion.value.startsWith(normalizedPrefix),
+  );
+  if (matched.length === 0) {
+    return null;
+  }
+
+  return matched.map((completion) => ({ ...completion }));
+}
+
 export default function sessionCleanupExtension(pi: ExtensionAPI): void {
   pi.on("session_shutdown", async (_event, ctx) => {
     await flushScheduledSessionDeletionForQuit(ctx);
@@ -19,8 +107,10 @@ export default function sessionCleanupExtension(pi: ExtensionAPI): void {
   pi.registerCommand(SESSION_CLEANUP_COMMAND, {
     description:
       "Batch-select previous sessions and delete them with confirmation.",
-    getArgumentCompletions: getSessionCleanupArgumentCompletions,
+    getArgumentCompletions: (argumentPrefix) =>
+      getMatchedCompletions(argumentPrefix, SESSION_CLEANUP_ARGUMENT_COMPLETIONS),
     handler: async (args, ctx) => {
+      const { handleSessionCleanupCommand } = await loadSessionCleanupCommandModule();
       await handleSessionCleanupCommand(args, ctx);
     },
   });
@@ -28,8 +118,10 @@ export default function sessionCleanupExtension(pi: ExtensionAPI): void {
   pi.registerCommand(SESSION_NIX_COMMAND, {
     description:
       "Start a fresh session, switch to a target agent, or delete the current session and quit Pi.",
-    getArgumentCompletions: getSessionNixArgumentCompletions,
+    getArgumentCompletions: (argumentPrefix) =>
+      getMatchedCompletions(argumentPrefix, SESSION_NIX_ARGUMENT_COMPLETIONS),
     handler: async (args, ctx) => {
+      const { handleSessionNixCommand } = await loadSessionNixCommandModule();
       await handleSessionNixCommand(args, ctx);
     },
   });

package/src/session-sort.ts

@@ -69,7 +69,23 @@ function parseCalendarCandidate(path: string): number | null {
   }
 
   const utcTimestamp = Date.UTC(year, month - 1, day, hour, minute, second);
-  return Number.isFinite(utcTimestamp) ? utcTimestamp : null;
+  if (!Number.isFinite(utcTimestamp)) {
+    return null;
+  }
+
+  const parsed = new Date(utcTimestamp);
+  if (
+    parsed.getUTCFullYear() !== year ||
+    parsed.getUTCMonth() !== month - 1 ||
+    parsed.getUTCDate() !== day ||
+    parsed.getUTCHours() !== hour ||
+    parsed.getUTCMinutes() !== minute ||
+    parsed.getUTCSeconds() !== second
+  ) {
+    return null;
+  }
+
+  return utcTimestamp;
 }
 
 function timestampFromPath(sessionPath: string): number {