pi-model-profiles 0.3.1 → 0.3.3

package/CHANGELOG.md

@@ -1,48 +1,59 @@
-# Changelog
-
-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).
-
-## [Unreleased]
-
-## [0.3.1] - 2026-05-22
-
-### Changed
-- Reworked debug logging to redact sensitive values and use asynchronous buffered file writes with safe shutdown.
-- Updated Pi peer dependencies and runtime imports to the `@earendil-works` scope.
-
-### Fixed
-- Improved debug log writer lifecycle handling so buffered events flush reliably without opening logs when debug is disabled.
-
-## [0.3.0] - 2026-04-30
-
-### Changed
-- Refined the model profiles modal layout with wider sizing, a single bordered grid, and clearer model table columns.
-- Updated the public README screenshot and usage details.
-- Bumped Pi peer dependency ranges to `^0.70.6`.
-
-## [0.2.0] - 2026-04-26
-
-### Added
-- Phase 6: Git & publishing preparation.
-- NPM package metadata, README, CHANGELOG, LICENSE, and package ignore rules.
-- Profile update, removal, persisted sorting, configuration, and file-gated debug logging.
-
-### Fixed
-- Confirmation prompts now accept typed input before update or removal actions run.
-- Sort menu keyboard handling now works consistently and closes without exiting the modal.
-- Profile update and removal command handlers now avoid duplicate scans and duplicate removal events.
-
-## [0.1.0] - 2026-04-25
-
-### Added
-- Initial extension structure
-- Core profile management functionality
-- Frontmatter parser implementation
-- Profile store with atomic writes
-- Import service for external profiles
-- Agent writer for profile application
-- Type definitions and error handling
-- Test suite for core functionality
+# Changelog
+
+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).
+
+## [Unreleased]
+
+## [0.3.3] - 2026-06-01
+
+### Changed
+- Lazy-loaded the model profiles command handler by extracting it into `src/command-handler.ts`, reducing startup work.
+- Widened Pi peer dependency compatibility to include Pi 0.77.x and 0.78.x.
+
+## [0.3.2] - 2026-05-26
+
+### Changed
+- Widened peer dependency ranges to `^0.74.0 || ^0.75.0`.
+
+## [0.3.1] - 2026-05-22
+
+### Changed
+- Reworked debug logging to redact sensitive values and use asynchronous buffered file writes with safe shutdown.
+- Updated Pi peer dependencies and runtime imports to the `@earendil-works` scope.
+
+### Fixed
+- Improved debug log writer lifecycle handling so buffered events flush reliably without opening logs when debug is disabled.
+
+## [0.3.0] - 2026-04-30
+
+### Changed
+- Refined the model profiles modal layout with wider sizing, a single bordered grid, and clearer model table columns.
+- Updated the public README screenshot and usage details.
+- Bumped Pi peer dependency ranges to `^0.70.6`.
+
+## [0.2.0] - 2026-04-26
+
+### Added
+- Phase 6: Git & publishing preparation.
+- NPM package metadata, README, CHANGELOG, LICENSE, and package ignore rules.
+- Profile update, removal, persisted sorting, configuration, and file-gated debug logging.
+
+### Fixed
+- Confirmation prompts now accept typed input before update or removal actions run.
+- Sort menu keyboard handling now works consistently and closes without exiting the modal.
+- Profile update and removal command handlers now avoid duplicate scans and duplicate removal events.
+
+## [0.1.0] - 2026-04-25
+
+### Added
+- Initial extension structure
+- Core profile management functionality
+- Frontmatter parser implementation
+- Profile store with atomic writes
+- Import service for external profiles
+- Agent writer for profile application
+- Type definitions and error handling
+- Test suite for core functionality

package/package.json

@@ -1,66 +1,66 @@
-{
-  "name": "pi-model-profiles",
-  "version": "0.3.1",
-  "private": false,
-  "description": "Pi extension for saving, importing, and applying agent model frontmatter profiles.",
-  "type": "module",
-  "main": "./index.ts",
-  "exports": {
-    ".": "./index.ts"
-  },
-  "files": [
-    "index.ts",
-    "src",
-    "README.md",
-    "CHANGELOG.md",
-    "config/config.example.json",
-    "LICENSE"
-  ],
-  "scripts": {
-    "build": "npx --yes -p typescript@5.9.2 tsc -p tsconfig.json",
-    "lint": "npm run build",
-    "test:clean": "node -e \"require('node:fs').rmSync('.test-dist', { recursive: true, force: true })\"",
-    "prepublishOnly": "npm run test",
-    "pretest": "npm run test:clean",
-    "test": "npx --yes -p typescript@5.9.2 tsc --strict --skipLibCheck --module nodenext --moduleResolution nodenext --target ES2022 --outDir .test-dist src/types-shims.d.ts src/errors.ts src/types.ts src/constants.ts src/atomic-write.ts src/profile-fields.ts src/frontmatter-parser.ts src/profile-store.ts src/agent-writer.ts src/import-service.ts src/config.ts src/debug-logger.ts test/frontmatter-parser.test.ts test/import-service.test.ts test/agent-writer.test.ts test/profile-store.test.ts test/debug-logger.test.ts && node --test .test-dist/test/frontmatter-parser.test.js .test-dist/test/import-service.test.js .test-dist/test/agent-writer.test.js .test-dist/test/profile-store.test.js .test-dist/test/debug-logger.test.js",
-    "posttest": "npm run test:clean",
-    "check": "npm run build && npm run test",
-    "package:dry-run": "npm pack --dry-run"
-  },
-  "keywords": [
-    "pi-package",
-    "pi",
-    "pi-extension",
-    "pi-coding-agent",
-    "coding-agent",
-    "model-profiles",
-    "frontmatter",
-    "agent-configuration",
-    "profiles"
-  ],
-  "author": "MasuRii",
-  "license": "MIT",
-  "homepage": "https://github.com/MasuRii/pi-model-profiles#readme",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/MasuRii/pi-model-profiles.git"
-  },
-  "bugs": {
-    "url": "https://github.com/MasuRii/pi-model-profiles/issues"
-  },
-  "engines": {
-    "node": ">=20"
-  },
-  "pi": {
-    "extensions": [
-      "./index.ts"
-    ]
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.75.4",
-    "@earendil-works/pi-tui": "^0.75.4"
-  },
-  "publishConfig": {
-    "access": "public"
-  }
-}
+{
+  "name": "pi-model-profiles",
+  "version": "0.3.3",
+  "private": false,
+  "description": "Pi extension for saving, importing, and applying agent model frontmatter profiles.",
+  "type": "module",
+  "main": "./index.ts",
+  "exports": {
+    ".": "./index.ts"
+  },
+  "files": [
+    "index.ts",
+    "src",
+    "README.md",
+    "CHANGELOG.md",
+    "config/config.example.json",
+    "LICENSE"
+  ],
+  "scripts": {
+    "build": "npx --yes -p typescript@5.9.2 tsc -p tsconfig.json",
+    "lint": "npm run build",
+    "test:clean": "node -e \"require('node:fs').rmSync('.test-dist', { recursive: true, force: true })\"",
+    "prepublishOnly": "npm run test",
+    "pretest": "npm run test:clean",
+    "test": "npx --yes -p typescript@5.9.2 tsc --strict --skipLibCheck --module nodenext --moduleResolution nodenext --target ES2022 --outDir .test-dist src/types-shims.d.ts src/errors.ts src/types.ts src/constants.ts src/atomic-write.ts src/profile-fields.ts src/frontmatter-parser.ts src/profile-store.ts src/agent-writer.ts src/import-service.ts src/config.ts src/debug-logger.ts test/frontmatter-parser.test.ts test/import-service.test.ts test/agent-writer.test.ts test/profile-store.test.ts test/debug-logger.test.ts && node --test .test-dist/test/frontmatter-parser.test.js .test-dist/test/import-service.test.js .test-dist/test/agent-writer.test.js .test-dist/test/profile-store.test.js .test-dist/test/debug-logger.test.js",
+    "posttest": "npm run test:clean",
+    "check": "npm run build && npm run test",
+    "package:dry-run": "npm pack --dry-run"
+  },
+  "keywords": [
+    "pi-package",
+    "pi",
+    "pi-extension",
+    "pi-coding-agent",
+    "coding-agent",
+    "model-profiles",
+    "frontmatter",
+    "agent-configuration",
+    "profiles"
+  ],
+  "author": "MasuRii",
+  "license": "MIT",
+  "homepage": "https://github.com/MasuRii/pi-model-profiles#readme",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/MasuRii/pi-model-profiles.git"
+  },
+  "bugs": {
+    "url": "https://github.com/MasuRii/pi-model-profiles/issues"
+  },
+  "engines": {
+    "node": ">=20"
+  },
+  "pi": {
+    "extensions": [
+      "./index.ts"
+    ]
+  },
+  "peerDependencies": {
+    "@earendil-works/pi-coding-agent": "^0.74.0 || ^0.75.0 || ^0.77.0 || ^0.78.0",
+    "@earendil-works/pi-tui": "^0.74.0 || ^0.75.0 || ^0.77.0 || ^0.78.0"
+  },
+  "publishConfig": {
+    "access": "public"
+  }
+}

package/src/command-handler.ts

@@ -0,0 +1,137 @@
+import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+
+import {
+	INITIAL_PROFILE_NAME,
+	PROFILE_NAME_SUFFIX,
+	PROFILE_STORE_PATH,
+} from "./constants.js";
+import { applySavedProfile, captureAgentSnapshots, detectActiveAgentName } from "./agent-writer.js";
+import { toErrorMessage } from "./errors.js";
+import { loadAndPrepareProfiles } from "./import-service.js";
+import { updateProfileAndReturn } from "./profile-update-service.js";
+import {
+	appendProfile,
+	createProfile,
+	findProfileById,
+	renameStoredProfile,
+	resolveUniqueProfileName,
+	saveProfilesFile,
+} from "./profile-store.js";
+import { removeProfileAndUpdate } from "./profile-removal-service.js";
+import { openProfilesModal, type ProfileModalResult } from "./profile-modal.js";
+import type { AppliedProfileOutcome, ProfilesFile } from "./types.js";
+
+function buildCurrentProfileName(activeAgentName: string | null, data: ProfilesFile): string {
+	const baseName = activeAgentName ? `${activeAgentName} ${PROFILE_NAME_SUFFIX}` : INITIAL_PROFILE_NAME;
+	return resolveUniqueProfileName(baseName, data.profiles);
+}
+
+function notifyWarnings(ctx: ExtensionCommandContext, warnings: readonly string[]): void {
+	if (warnings.length === 0) {
+		return;
+	}
+	ctx.ui.notify(warnings.join(" "), "warning");
+}
+
+function summarizeApplyOutcome(outcome: AppliedProfileOutcome): string {
+	const appliedCount = outcome.appliedAgents.length;
+	const missingCount = outcome.missingAgents.length;
+	const appliedLabel = `${appliedCount} agent file${appliedCount === 1 ? "" : "s"}`;
+	if (missingCount === 0) {
+		return appliedLabel;
+	}
+	return `${appliedLabel}; ${missingCount} missing`;
+}
+
+async function reloadAfterApply(ctx: ExtensionCommandContext, outcome: AppliedProfileOutcome): Promise<void> {
+	const summary = summarizeApplyOutcome(outcome);
+	ctx.ui.notify(`Profile '${outcome.profileName}' applied across ${summary}. Reloading…`, "info");
+
+	try {
+		await ctx.reload();
+	} catch (error) {
+		ctx.ui.notify(
+			`Profile '${outcome.profileName}' applied across ${summary}, but automatic reload failed: ${toErrorMessage(error)}. Run /reload.`,
+			"error",
+		);
+	}
+}
+
+export async function handleModelProfilesCommand(ctx: ExtensionCommandContext): Promise<void> {
+	const agentOptions = { cwd: ctx.cwd, scope: "both" as const };
+	const prepared = loadAndPrepareProfiles(PROFILE_STORE_PATH, agentOptions);
+	notifyWarnings(ctx, prepared.warnings);
+
+	let data = prepared.data;
+	const activeAgentName = detectActiveAgentName(ctx.sessionManager, ctx.getSystemPrompt());
+
+	const result: ProfileModalResult = await openProfilesModal(ctx, data, activeAgentName, {
+		renameProfile: async (profileId, nextName) => {
+			data = renameStoredProfile(data, profileId, nextName);
+			saveProfilesFile(data, PROFILE_STORE_PATH);
+			return {
+				data,
+				message: `Renamed saved snapshot to '${nextName.trim()}'.`,
+				selectedProfileId: profileId,
+			};
+		},
+		addCurrentProfile: async () => {
+			const snapshot = captureAgentSnapshots(agentOptions);
+			notifyWarnings(ctx, snapshot.warnings);
+			const profile = createProfile(buildCurrentProfileName(activeAgentName, data), snapshot.agents);
+			data = appendProfile(data, profile);
+			saveProfilesFile(data, PROFILE_STORE_PATH);
+			return {
+				data,
+				message: `Saved current agents snapshot (${snapshot.agents.length} agents).`,
+				selectedProfileId: profile.id,
+			};
+		},
+		applyProfile: async (profileId) => {
+			const profile = findProfileById(data, profileId);
+			if (!profile) {
+				throw new Error(`Saved profile '${profileId}' was not found.`);
+			}
+			const applied = applySavedProfile(profile, agentOptions);
+			notifyWarnings(ctx, applied.warnings);
+			return {
+				...applied,
+				profileName: profile.name,
+			};
+		},
+		removeProfile: async (profileId) => {
+			const profile = findProfileById(data, profileId);
+			if (!profile) {
+				throw new Error(`Saved profile '${profileId}' was not found.`);
+			}
+			const removal = removeProfileAndUpdate(data, profileId);
+			data = removal.data;
+			saveProfilesFile(data, PROFILE_STORE_PATH);
+			return {
+				data,
+				message: `Removed saved snapshot '${removal.result.removedProfileName}' (${removal.result.remainingCount} snapshots remaining).`,
+				selectedProfileId: data.profiles[0]?.id,
+			};
+		},
+		updateProfile: async (profileId) => {
+			const profile = findProfileById(data, profileId);
+			if (!profile) {
+				throw new Error(`Saved profile '${profileId}' was not found.`);
+			}
+			const update = updateProfileAndReturn(data, profileId, agentOptions);
+			notifyWarnings(ctx, update.result.warnings);
+			data = update.data;
+			saveProfilesFile(data, PROFILE_STORE_PATH);
+			return {
+				data,
+				message: `Updated '${profile.name}' with current agent state (${update.result.updatedAgents} agents).`,
+				selectedProfileId: profileId,
+			};
+		},
+	});
+
+	if (result.type === "applied") {
+		await reloadAfterApply(ctx, result.outcome);
+		return;
+	}
+}

package/src/errors.ts

@@ -1,3 +1,5 @@
+export const COMMAND_NAME = "model-profiles";
+
 export class ModelProfilesError extends Error {
 	readonly code: string;
 

package/src/import-service.ts

@@ -1,60 +1,60 @@
-import { AGENTS_DIR, INITIAL_PROFILE_NAME, PROFILE_STORE_PATH } from "./constants.js";
-import { captureAgentSnapshots, type AgentSelectionOptions } from "./agent-writer.js";
-import { appendProfile, createProfile, loadProfilesFile, resolveUniqueProfileName, saveProfilesFile } from "./profile-store.js";
-import { loadMultiProfilesConfig } from "./config.js";
-import { multiProfilesDebugLogger } from "./debug-logger.js";
-import type { ImportProfilesResult, ProfilesFile } from "./types.js";
-
-export function ensureProfilesImported(
-	data: ProfilesFile,
-	agentOptions: string | AgentSelectionOptions = AGENTS_DIR,
-): ImportProfilesResult {
-	if (data.importedAt) {
-		return {
-			data,
-			imported: false,
-			importedCount: 0,
-			warnings: [],
-		};
-	}
-
-	const snapshot = captureAgentSnapshots(agentOptions);
-	const timestamp = new Date().toISOString();
-	const profile = createProfile(resolveUniqueProfileName(INITIAL_PROFILE_NAME, data.profiles), snapshot.agents, { timestamp });
-
-	return {
-		data: {
-			...appendProfile(data, profile),
-			importedAt: timestamp,
-		},
-		imported: true,
-		importedCount: 1,
-		warnings: snapshot.warnings,
-	};
-}
-
-export function loadAndPrepareProfiles(
-	storePath = PROFILE_STORE_PATH,
-	agentOptions: string | AgentSelectionOptions = AGENTS_DIR,
-): ImportProfilesResult {
-	const configLoad = loadMultiProfilesConfig();
-
-	multiProfilesDebugLogger.log("extension.initialized", {
-		configCreated: configLoad.created,
-		timestamp: new Date().toISOString(),
-		profilesVersion: 2,
-	});
-
-	const loaded = loadProfilesFile(storePath, agentOptions);
-	const prepared = ensureProfilesImported(loaded.data, agentOptions);
-
-	if (loaded.needsSave || prepared.imported) {
-		saveProfilesFile(prepared.data, storePath);
-	}
-
-	const warnings = [configLoad.warning, loaded.warning, ...prepared.warnings].filter((message): message is string => Boolean(message));
-	return {
-		...prepared,
-		warnings,
-	};
-}
+import { AGENTS_DIR, INITIAL_PROFILE_NAME, PROFILE_STORE_PATH } from "./constants.js";
+import { captureAgentSnapshots, type AgentSelectionOptions } from "./agent-writer.js";
+import { appendProfile, createProfile, loadProfilesFile, resolveUniqueProfileName, saveProfilesFile } from "./profile-store.js";
+import { loadMultiProfilesConfig } from "./config.js";
+import { multiProfilesDebugLogger } from "./debug-logger.js";
+import type { ImportProfilesResult, ProfilesFile } from "./types.js";
+
+export function ensureProfilesImported(
+	data: ProfilesFile,
+	agentOptions: string | AgentSelectionOptions = AGENTS_DIR,
+): ImportProfilesResult {
+	if (data.importedAt) {
+		return {
+			data,
+			imported: false,
+			importedCount: 0,
+			warnings: [],
+		};
+	}
+
+	const snapshot = captureAgentSnapshots(agentOptions);
+	const timestamp = new Date().toISOString();
+	const profile = createProfile(resolveUniqueProfileName(INITIAL_PROFILE_NAME, data.profiles), snapshot.agents, { timestamp });
+
+	return {
+		data: {
+			...appendProfile(data, profile),
+			importedAt: timestamp,
+		},
+		imported: true,
+		importedCount: 1,
+		warnings: snapshot.warnings,
+	};
+}
+
+export function loadAndPrepareProfiles(
+	storePath = PROFILE_STORE_PATH,
+	agentOptions: string | AgentSelectionOptions = AGENTS_DIR,
+): ImportProfilesResult {
+	const configLoad = loadMultiProfilesConfig();
+
+	multiProfilesDebugLogger.log("extension.initialized", {
+		configCreated: configLoad.created,
+		timestamp: new Date().toISOString(),
+		profilesVersion: 2,
+	});
+
+	const loaded = loadProfilesFile(storePath, agentOptions);
+	const prepared = ensureProfilesImported(loaded.data, agentOptions);
+
+	if (loaded.needsSave || prepared.imported) {
+		saveProfilesFile(prepared.data, storePath);
+	}
+
+	const warnings = [configLoad.warning, loaded.warning, ...prepared.warnings].filter((message): message is string => Boolean(message));
+	return {
+		...prepared,
+		warnings,
+	};
+}