pi-model-profiles 0.3.2 → 0.3.4

package/CHANGELOG.md

@@ -7,6 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.4] - 2026-06-16
+
+### Fixed
+- Resolved symlinks before writing agent markdown to avoid overwriting the symlink target's parent instead of the linked file.
+- Used `parseCompleteNumericScalar` for frontmatter number parsing to handle trailing non-numeric characters correctly.
+
+## [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

package/package.json

@@ -1,66 +1,66 @@
-{
-  "name": "pi-model-profiles",
-  "version": "0.3.2",
-  "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",
-    "@earendil-works/pi-tui": "^0.74.0 || ^0.75.0"
-  },
-  "publishConfig": {
-    "access": "public"
-  }
-}
+{
+  "name": "pi-model-profiles",
+  "version": "0.3.4",
+  "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 || ^0.79.0",
+    "@earendil-works/pi-tui": "^0.74.0 || ^0.75.0 || ^0.77.0 || ^0.78.0 || ^0.79.0"
+  },
+  "publishConfig": {
+    "access": "public"
+  }
+}

package/src/agent-writer.ts

@@ -1,4 +1,4 @@
-import { existsSync, readdirSync, readFileSync } from "node:fs";
+import { existsSync, lstatSync, readdirSync, readFileSync, realpathSync } from "node:fs";
 import { basename, dirname, join, resolve } from "node:path";
 
 import { AGENTS_DIR } from "./constants.js";
@@ -33,6 +33,12 @@ interface SessionManagerLike {
 	getEntries(): readonly unknown[];
 }
 
+interface AgentDirectoryEntry {
+	name: string;
+	isFile(): boolean;
+	isSymbolicLink(): boolean;
+}
+
 type AgentScope = "user" | "project" | "both";
 
 export interface AgentSelectionOptions {
@@ -130,6 +136,22 @@ function isDirectory(path: string): boolean {
 	}
 }
 
+function isMarkdownAgentEntry(entry: AgentDirectoryEntry): boolean {
+	return entry.name.endsWith(".md") && (entry.isFile() || entry.isSymbolicLink());
+}
+
+function resolveAgentWritePath(filePath: string): string {
+	try {
+		return lstatSync(filePath).isSymbolicLink() ? realpathSync(filePath) : filePath;
+	} catch {
+		return filePath;
+	}
+}
+
+function writeAgentMarkdown(filePath: string, markdown: string): void {
+	writeFileAtomic(resolveAgentWritePath(filePath), markdown);
+}
+
 function findNearestProjectAgentDirs(cwd: string): string[] {
 	let currentDir = resolve(cwd);
 
@@ -187,15 +209,15 @@ export function scanAgentFiles(options: string | AgentSelectionOptions = AGENTS_
 	const agentsByName = new Map<string, AgentFileRecord>();
 
 	for (const agentsDir of sourceDirs) {
-		let entries: Array<{ name: string; isFile(): boolean }>;
+		let entries: AgentDirectoryEntry[];
 		try {
-			entries = readdirSync(agentsDir, { withFileTypes: true }) as Array<{ name: string; isFile(): boolean }>;
+			entries = readdirSync(agentsDir, { withFileTypes: true }) as AgentDirectoryEntry[];
 		} catch {
 			throw new ModelProfilesError(`Unable to read agents directory '${agentsDir}'.`, "AGENTS_DIR_UNAVAILABLE");
 		}
 
 		for (const entry of entries.sort((left, right) => left.name.localeCompare(right.name))) {
-			if (!entry.isFile() || !entry.name.endsWith(".md")) {
+			if (!isMarkdownAgentEntry(entry)) {
 				continue;
 			}
 
@@ -247,7 +269,7 @@ export function applyProfileToAgentRecord(agent: AgentFileRecord, fields: Profil
 	const normalizedFields = normalizeProfileFields(fields);
 	const markdown = readFileSync(agent.path, "utf-8");
 	const updatedMarkdown = updateMarkdownProfileFields(markdown, normalizedFields);
-	writeFileAtomic(agent.path, updatedMarkdown);
+	writeAgentMarkdown(agent.path, updatedMarkdown);
 	return {
 		updatedPath: agent.path,
 		fileName: agent.fileName,
@@ -314,7 +336,7 @@ export function applySavedProfile(profile: SavedProfile, options: string | Agent
 	}
 
 	for (const pending of pendingWrites) {
-		writeFileAtomic(pending.target.path, pending.updatedMarkdown);
+		writeAgentMarkdown(pending.target.path, pending.updatedMarkdown);
 	}
 
 	return {

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,16 +1,18 @@
-export class ModelProfilesError extends Error {
-	readonly code: string;
-
-	constructor(message: string, code = "MODEL_PROFILES_ERROR") {
-		super(message);
-		this.name = "ModelProfilesError";
-		this.code = code;
-	}
-}
-
-export function toErrorMessage(error: unknown): string {
-	if (error instanceof Error && error.message.trim()) {
-		return error.message;
-	}
-	return String(error);
-}
+export const COMMAND_NAME = "model-profiles";
+
+export class ModelProfilesError extends Error {
+	readonly code: string;
+
+	constructor(message: string, code = "MODEL_PROFILES_ERROR") {
+		super(message);
+		this.name = "ModelProfilesError";
+		this.code = code;
+	}
+}
+
+export function toErrorMessage(error: unknown): string {
+	if (error instanceof Error && error.message.trim()) {
+		return error.message;
+	}
+	return String(error);
+}