pi-model-profiles 0.3.2 → 0.3.4

package/src/profile-sort-service.ts

@@ -1,105 +1,105 @@
-import { multiProfilesDebugLogger } from "./debug-logger.js";
-import { loadMultiProfilesConfig, saveMultiProfilesConfig } from "./config.js";
-import type { ProfileSortResult, ProfilesFile, SavedProfile } from "./types.js";
-import type { ProfileSortOrder } from "./types.js";
-
-/**
- * Compare two profile names using locale-aware string comparison.
- */
-function compareByName(left: SavedProfile, right: SavedProfile): number {
-	return left.name.localeCompare(right.name);
-}
-
-/**
- * Compare two profiles by their createdAt timestamp.
- */
-function compareByDate(left: SavedProfile, right: SavedProfile): number {
-	const leftDate = new Date(left.createdAt).getTime();
-	const rightDate = new Date(right.createdAt).getTime();
-	return leftDate - rightDate;
-}
-
-/**
- * Sort profiles based on the specified order.
- * Returns a new array (immutable pattern), preserving the original data.profiles.
- * 
- * @param data - The profiles file containing the profiles array
- * @param order - The sort order to apply
- * @returns ProfileSortResult with sorted profiles and applied sort order
- */
-export function sortProfiles(data: ProfilesFile, order: ProfileSortOrder): ProfileSortResult {
-	const comparator = getSortComparator(order);
-	const sortedProfiles = [...data.profiles].sort(comparator);
-
-	return {
-		sortedProfiles,
-		sortOrder: order,
-	};
-}
-
-/**
- * Get the comparator function for a given sort order.
- */
-function getSortComparator(order: ProfileSortOrder): (left: SavedProfile, right: SavedProfile) => number {
-	switch (order) {
-		case "name-asc":
-			return compareByName;
-		case "name-desc":
-			return (left, right) => compareByName(right, left);
-		case "date-asc":
-			return compareByDate;
-		case "date-desc":
-			return (left, right) => compareByDate(right, left);
-	}
-}
-
-/**
- * Get the current sort order from config.
- */
-export function getCurrentSortOrder(): ProfileSortOrder {
-	const result = loadMultiProfilesConfig();
-	return result.config.sorting.defaultSort;
-}
-
-/**
- * Persist the sort order to config.
- */
-export function persistSortOrder(order: ProfileSortOrder): void {
-	const result = loadMultiProfilesConfig();
-	const config = result.config;
-	config.sorting.defaultSort = order;
-	saveMultiProfilesConfig(config);
-
-	multiProfilesDebugLogger.log("config", {
-		event: "sort_order_persisted",
-		order,
-	});
-}
-
-/**
- * Get display label for a sort order.
- */
-export function getSortOrderLabel(order: ProfileSortOrder): string {
-	switch (order) {
-		case "name-asc":
-			return "Name (A-Z)";
-		case "name-desc":
-			return "Name (Z-A)";
-		case "date-asc":
-			return "Date (Oldest)";
-		case "date-desc":
-			return "Date (Newest)";
-	}
-}
-
-/**
- * Get all available sort orders with their labels.
- */
-export function getAvailableSortOrders(): Array<{ order: ProfileSortOrder; label: string }> {
-	return [
-		{ order: "name-asc", label: "Name (A-Z)" },
-		{ order: "name-desc", label: "Name (Z-A)" },
-		{ order: "date-asc", label: "Date (Oldest)" },
-		{ order: "date-desc", label: "Date (Newest)" },
-	];
-}
+import { multiProfilesDebugLogger } from "./debug-logger.js";
+import { loadMultiProfilesConfig, saveMultiProfilesConfig } from "./config.js";
+import type { ProfileSortResult, ProfilesFile, SavedProfile } from "./types.js";
+import type { ProfileSortOrder } from "./types.js";
+
+/**
+ * Compare two profile names using locale-aware string comparison.
+ */
+function compareByName(left: SavedProfile, right: SavedProfile): number {
+	return left.name.localeCompare(right.name);
+}
+
+/**
+ * Compare two profiles by their createdAt timestamp.
+ */
+function compareByDate(left: SavedProfile, right: SavedProfile): number {
+	const leftDate = new Date(left.createdAt).getTime();
+	const rightDate = new Date(right.createdAt).getTime();
+	return leftDate - rightDate;
+}
+
+/**
+ * Sort profiles based on the specified order.
+ * Returns a new array (immutable pattern), preserving the original data.profiles.
+ * 
+ * @param data - The profiles file containing the profiles array
+ * @param order - The sort order to apply
+ * @returns ProfileSortResult with sorted profiles and applied sort order
+ */
+export function sortProfiles(data: ProfilesFile, order: ProfileSortOrder): ProfileSortResult {
+	const comparator = getSortComparator(order);
+	const sortedProfiles = [...data.profiles].sort(comparator);
+
+	return {
+		sortedProfiles,
+		sortOrder: order,
+	};
+}
+
+/**
+ * Get the comparator function for a given sort order.
+ */
+function getSortComparator(order: ProfileSortOrder): (left: SavedProfile, right: SavedProfile) => number {
+	switch (order) {
+		case "name-asc":
+			return compareByName;
+		case "name-desc":
+			return (left, right) => compareByName(right, left);
+		case "date-asc":
+			return compareByDate;
+		case "date-desc":
+			return (left, right) => compareByDate(right, left);
+	}
+}
+
+/**
+ * Get the current sort order from config.
+ */
+export function getCurrentSortOrder(): ProfileSortOrder {
+	const result = loadMultiProfilesConfig();
+	return result.config.sorting.defaultSort;
+}
+
+/**
+ * Persist the sort order to config.
+ */
+export function persistSortOrder(order: ProfileSortOrder): void {
+	const result = loadMultiProfilesConfig();
+	const config = result.config;
+	config.sorting.defaultSort = order;
+	saveMultiProfilesConfig(config);
+
+	multiProfilesDebugLogger.log("config", {
+		event: "sort_order_persisted",
+		order,
+	});
+}
+
+/**
+ * Get display label for a sort order.
+ */
+export function getSortOrderLabel(order: ProfileSortOrder): string {
+	switch (order) {
+		case "name-asc":
+			return "Name (A-Z)";
+		case "name-desc":
+			return "Name (Z-A)";
+		case "date-asc":
+			return "Date (Oldest)";
+		case "date-desc":
+			return "Date (Newest)";
+	}
+}
+
+/**
+ * Get all available sort orders with their labels.
+ */
+export function getAvailableSortOrders(): Array<{ order: ProfileSortOrder; label: string }> {
+	return [
+		{ order: "name-asc", label: "Name (A-Z)" },
+		{ order: "name-desc", label: "Name (Z-A)" },
+		{ order: "date-asc", label: "Date (Oldest)" },
+		{ order: "date-desc", label: "Date (Newest)" },
+	];
+}

package/src/profile-update-service.ts

@@ -1,134 +1,134 @@
-import { multiProfilesDebugLogger } from "./debug-logger.js";
-import { ModelProfilesError } from "./errors.js";
-import { captureAgentSnapshots, type AgentSelectionOptions } from "./agent-writer.js";
-import type { ProfilesFile, ProfileUpdateResult, SavedProfile } from "./types.js";
-
-/**
- * Error code for profile not found during update.
- */
-export const PROFILE_NOT_FOUND_CODE = "PROFILE_NOT_FOUND";
-
-export interface ProfileUpdateDataResult {
-	data: ProfilesFile;
-	result: ProfileUpdateResult;
-}
-
-interface ProfileUpdatePlan {
-	updatedData: ProfilesFile;
-	updatedProfile: SavedProfile;
-	previousProfile: SavedProfile;
-	result: ProfileUpdateResult;
-}
-
-function createProfileNotFoundError(profileId: string): ModelProfilesError {
-	return new ModelProfilesError(
-		`Profile '${profileId}' not found. It may have been removed already.`,
-		PROFILE_NOT_FOUND_CODE,
-	);
-}
-
-function buildUpdatePlan(
-	data: ProfilesFile,
-	profileId: string,
-	agentOptions: AgentSelectionOptions,
-): ProfileUpdatePlan {
-	const profileIndex = data.profiles.findIndex((profile) => profile.id === profileId);
-	const profile = data.profiles[profileIndex];
-
-	if (profileIndex === -1 || !profile) {
-		multiProfilesDebugLogger.log("profile-update", {
-			event: "update_failed",
-			profileId,
-			reason: "profile_not_found",
-			profileCount: data.profiles.length,
-		});
-		throw createProfileNotFoundError(profileId);
-	}
-
-	const snapshot = captureAgentSnapshots(agentOptions);
-	const updatedProfile: SavedProfile = {
-		id: profile.id,
-		name: profile.name,
-		agents: snapshot.agents,
-		createdAt: profile.createdAt,
-		updatedAt: new Date().toISOString(),
-	};
-	const updatedProfiles = [
-		...data.profiles.slice(0, profileIndex),
-		updatedProfile,
-		...data.profiles.slice(profileIndex + 1),
-	];
-
-	return {
-		updatedData: {
-			version: data.version,
-			importedAt: data.importedAt,
-			profiles: updatedProfiles,
-		},
-		updatedProfile,
-		previousProfile: profile,
-		result: {
-			updatedProfileId: profile.id,
-			updatedProfileName: profile.name,
-			updatedAgents: snapshot.agents.length,
-			warnings: snapshot.warnings,
-		},
-	};
-}
-
-function logProfileUpdated(plan: ProfileUpdatePlan): void {
-	multiProfilesDebugLogger.log("profile-update", {
-		event: "profile_updated",
-		profileId: plan.updatedProfile.id,
-		profileName: plan.updatedProfile.name,
-		beforeAgentCount: plan.previousProfile.agents.length,
-		afterAgentCount: plan.updatedProfile.agents.length,
-		warnings: plan.result.warnings,
-	});
-}
-
-/**
- * Update a profile with the current agent state.
- *
- * Captures the current agent snapshots and replaces the profile's agents array.
- * Preserves profile.id, profile.name, and profile.createdAt.
- * Updates profile.updatedAt timestamp to current time.
- * Logs update event with before/after agent counts via debug logger.
- *
- * @param data - The profiles file data
- * @param profileId - The ID of the profile to update
- * @param agentOptions - Options for capturing agent snapshots
- * @returns ProfileUpdateResult with updated profile info and agent counts
- * @throws ModelProfilesError with code PROFILE_NOT_FOUND if profile doesn't exist
- */
-export function updateProfile(
-	data: ProfilesFile,
-	profileId: string,
-	agentOptions: AgentSelectionOptions,
-): ProfileUpdateResult {
-	const plan = buildUpdatePlan(data, profileId, agentOptions);
-	logProfileUpdated(plan);
-	return plan.result;
-}
-
-/**
- * Update a profile and return the updated profiles file with update metadata.
- *
- * @param data - The profiles file data
- * @param profileId - The ID of the profile to update
- * @param agentOptions - Options for capturing agent snapshots
- * @returns New ProfilesFile with the profile updated and update metadata
- * @throws ModelProfilesError with code PROFILE_NOT_FOUND if profile doesn't exist
- */
-export function updateProfileAndReturn(
-	data: ProfilesFile,
-	profileId: string,
-	agentOptions: AgentSelectionOptions,
-): ProfileUpdateDataResult {
-	const plan = buildUpdatePlan(data, profileId, agentOptions);
-	logProfileUpdated(plan);
-	return {
-		data: plan.updatedData,
-		result: plan.result,
-	};
-}
+import { multiProfilesDebugLogger } from "./debug-logger.js";
+import { ModelProfilesError } from "./errors.js";
+import { captureAgentSnapshots, type AgentSelectionOptions } from "./agent-writer.js";
+import type { ProfilesFile, ProfileUpdateResult, SavedProfile } from "./types.js";
+
+/**
+ * Error code for profile not found during update.
+ */
+export const PROFILE_NOT_FOUND_CODE = "PROFILE_NOT_FOUND";
+
+export interface ProfileUpdateDataResult {
+	data: ProfilesFile;
+	result: ProfileUpdateResult;
+}
+
+interface ProfileUpdatePlan {
+	updatedData: ProfilesFile;
+	updatedProfile: SavedProfile;
+	previousProfile: SavedProfile;
+	result: ProfileUpdateResult;
+}
+
+function createProfileNotFoundError(profileId: string): ModelProfilesError {
+	return new ModelProfilesError(
+		`Profile '${profileId}' not found. It may have been removed already.`,
+		PROFILE_NOT_FOUND_CODE,
+	);
+}
+
+function buildUpdatePlan(
+	data: ProfilesFile,
+	profileId: string,
+	agentOptions: AgentSelectionOptions,
+): ProfileUpdatePlan {
+	const profileIndex = data.profiles.findIndex((profile) => profile.id === profileId);
+	const profile = data.profiles[profileIndex];
+
+	if (profileIndex === -1 || !profile) {
+		multiProfilesDebugLogger.log("profile-update", {
+			event: "update_failed",
+			profileId,
+			reason: "profile_not_found",
+			profileCount: data.profiles.length,
+		});
+		throw createProfileNotFoundError(profileId);
+	}
+
+	const snapshot = captureAgentSnapshots(agentOptions);
+	const updatedProfile: SavedProfile = {
+		id: profile.id,
+		name: profile.name,
+		agents: snapshot.agents,
+		createdAt: profile.createdAt,
+		updatedAt: new Date().toISOString(),
+	};
+	const updatedProfiles = [
+		...data.profiles.slice(0, profileIndex),
+		updatedProfile,
+		...data.profiles.slice(profileIndex + 1),
+	];
+
+	return {
+		updatedData: {
+			version: data.version,
+			importedAt: data.importedAt,
+			profiles: updatedProfiles,
+		},
+		updatedProfile,
+		previousProfile: profile,
+		result: {
+			updatedProfileId: profile.id,
+			updatedProfileName: profile.name,
+			updatedAgents: snapshot.agents.length,
+			warnings: snapshot.warnings,
+		},
+	};
+}
+
+function logProfileUpdated(plan: ProfileUpdatePlan): void {
+	multiProfilesDebugLogger.log("profile-update", {
+		event: "profile_updated",
+		profileId: plan.updatedProfile.id,
+		profileName: plan.updatedProfile.name,
+		beforeAgentCount: plan.previousProfile.agents.length,
+		afterAgentCount: plan.updatedProfile.agents.length,
+		warnings: plan.result.warnings,
+	});
+}
+
+/**
+ * Update a profile with the current agent state.
+ *
+ * Captures the current agent snapshots and replaces the profile's agents array.
+ * Preserves profile.id, profile.name, and profile.createdAt.
+ * Updates profile.updatedAt timestamp to current time.
+ * Logs update event with before/after agent counts via debug logger.
+ *
+ * @param data - The profiles file data
+ * @param profileId - The ID of the profile to update
+ * @param agentOptions - Options for capturing agent snapshots
+ * @returns ProfileUpdateResult with updated profile info and agent counts
+ * @throws ModelProfilesError with code PROFILE_NOT_FOUND if profile doesn't exist
+ */
+export function updateProfile(
+	data: ProfilesFile,
+	profileId: string,
+	agentOptions: AgentSelectionOptions,
+): ProfileUpdateResult {
+	const plan = buildUpdatePlan(data, profileId, agentOptions);
+	logProfileUpdated(plan);
+	return plan.result;
+}
+
+/**
+ * Update a profile and return the updated profiles file with update metadata.
+ *
+ * @param data - The profiles file data
+ * @param profileId - The ID of the profile to update
+ * @param agentOptions - Options for capturing agent snapshots
+ * @returns New ProfilesFile with the profile updated and update metadata
+ * @throws ModelProfilesError with code PROFILE_NOT_FOUND if profile doesn't exist
+ */
+export function updateProfileAndReturn(
+	data: ProfilesFile,
+	profileId: string,
+	agentOptions: AgentSelectionOptions,
+): ProfileUpdateDataResult {
+	const plan = buildUpdatePlan(data, profileId, agentOptions);
+	logProfileUpdated(plan);
+	return {
+		data: plan.updatedData,
+		result: plan.result,
+	};
+}