pi-profiles-manager 1.1.1 → 1.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-profiles-manager",
-	"version": "1.1.1",
+	"version": "1.1.4",
 	"description": "Interactive SDD model profile management built natively for the Pi Coding Agent.",
 	"main": "index.ts",
 	"type": "module",

package/src/commands.test.ts

@@ -70,16 +70,27 @@ describe("registerCommands", () => {
 		);
 	});
 
-	it("registers /profiles:sync alias command", () => {
+	it("registers slash-command aliases for every profile action", () => {
 		const pi = mockPi();
 		const mgr = new ProfileManager(pi, mockCtx());
 		mgr.setConfig(configWithProfiles([]));
 
 		registerCommands(pi, mgr, vi.fn() as any, vi.fn() as any);
-		expect(pi.registerCommand).toHaveBeenCalledWith(
-			"profiles:sync",
-			expect.any(Object),
-		);
+
+		for (const action of [
+			"list",
+			"status",
+			"sync",
+			"use",
+			"save",
+			"next",
+			"off",
+		]) {
+			expect(pi.registerCommand).toHaveBeenCalledWith(
+				`profiles:${action}`,
+				expect.any(Object),
+			);
+		}
 	});
 
 	it("completes actions first and profile names after use or save", () => {
@@ -267,23 +278,28 @@ describe("registerCommands", () => {
 		expect(openTui).not.toHaveBeenCalled();
 	});
 
-	it("profiles:sync alias command delegates directly to sync without opening TUI", async () => {
+	it("profiles action aliases delegate through the existing action handling", async () => {
 		const pi = mockPi();
 		const ctx = mockCtx();
 		const mgr = new ProfileManager(pi, ctx);
+		const save = vi.fn(async () => {});
 		const sync = vi.fn(async () => {});
 		const openTui = vi.fn(async () => {});
-		mgr.setConfig(configWithProfiles([]));
+		mgr.setConfig(configWithProfiles(["alpha"]));
 
-		registerCommands(pi, mgr, vi.fn() as any, vi.fn() as any, openTui, sync);
+		registerCommands(pi, mgr, save as any, vi.fn() as any, openTui, sync);
 
-		const callArgs = (pi.registerCommand as any).mock.calls.find(
-			(c: any[]) => c[0] === "profiles:sync",
-		);
-		const handler = callArgs[1].handler;
-		await handler("", ctx);
+		const handlerFor = (command: string) =>
+			(pi.registerCommand as any).mock.calls.find(
+				(c: any[]) => c[0] === command,
+			)[1].handler;
+		await handlerFor("profiles:save")("snapshot", ctx);
+		await handlerFor("profiles:sync")("", ctx);
+		await handlerFor("profiles:list")("", ctx);
 
+		expect(save).toHaveBeenCalledWith(ctx, "snapshot");
 		expect(sync).toHaveBeenCalledWith(ctx);
+		expect(ctx.ui.notify).toHaveBeenCalledWith("alpha");
 		expect(openTui).not.toHaveBeenCalled();
 	});
 });

package/src/commands.ts

@@ -30,19 +30,52 @@ export function registerCommands(
 	openTui: OpenTuiFn = async () => {},
 	sync: SyncFn = async () => {},
 ) {
-	pi.registerCommand("profiles:sync", {
-		description: "Sync PiProfiles with managed agents folder",
-		async handler(_args: string, ctx: ExtensionCommandContext) {
-			try {
+	const handleAction = async (args: string, ctx: ExtensionCommandContext) => {
+		manager.setContext(ctx);
+		const { verb, name } = parseCommand(args);
+
+		try {
+			if (verb === "list") {
+				return ctx.ui.notify(manager.names().join("\n") || "No profiles");
+			}
+			if (verb === "status") {
+				const active = manager.state.get(ctx.sessionManager.getSessionId());
+				return ctx.ui.notify(active?.profile ?? "none");
+			}
+			if (verb === "sync") {
 				await sync(ctx);
-			} catch (error: unknown) {
-				ctx.ui.notify(
-					error instanceof Error ? error.message : String(error),
-					"error",
-				);
+				return;
 			}
-		},
-	});
+			if (verb === "use") {
+				if (!name) {
+					return ctx.ui.notify("Usage: /profiles use <name>", "error");
+				}
+				await manager.use(name);
+				return;
+			}
+			if (verb === "save") {
+				if (!name) {
+					return ctx.ui.notify("Usage: /profiles save <name>", "error");
+				}
+				await _save(ctx, name);
+				return;
+			}
+			if (verb === "next") {
+				await manager.next();
+				return;
+			}
+			if (verb === "off") {
+				await manager.off();
+				return;
+			}
+			return openTui(ctx as ContextLike);
+		} catch (error: unknown) {
+			ctx.ui.notify(
+				error instanceof Error ? error.message : String(error),
+				"error",
+			);
+		}
+	};
 
 	pi.registerCommand("profiles", {
 		description: "Manage SDD model profiles",
@@ -65,51 +98,14 @@ export function registerCommands(
 				.filter((name) => name.startsWith(namePrefix))
 				.map((value) => ({ value, label: value }));
 		},
-		async handler(args: string, ctx: ExtensionCommandContext) {
-			manager.setContext(ctx);
-			const { verb, name } = parseCommand(args);
-
-			try {
-				if (verb === "list") {
-					return ctx.ui.notify(manager.names().join("\n") || "No profiles");
-				}
-				if (verb === "status") {
-					const active = manager.state.get(ctx.sessionManager.getSessionId());
-					return ctx.ui.notify(active?.profile ?? "none");
-				}
-				if (verb === "sync") {
-					await sync(ctx);
-					return;
-				}
-				if (verb === "use") {
-					if (!name) {
-						return ctx.ui.notify("Usage: /profiles use <name>", "error");
-					}
-					await manager.use(name);
-					return;
-				}
-				if (verb === "save") {
-					if (!name) {
-						return ctx.ui.notify("Usage: /profiles save <name>", "error");
-					}
-					await _save(ctx, name);
-					return;
-				}
-				if (verb === "next") {
-					await manager.next();
-					return;
-				}
-				if (verb === "off") {
-					await manager.off();
-					return;
-				}
-				return openTui(ctx as ContextLike);
-			} catch (error: unknown) {
-				ctx.ui.notify(
-					error instanceof Error ? error.message : String(error),
-					"error",
-				);
-			}
-		},
+		handler: handleAction,
 	});
+
+	for (const action of COMMAND_ACTIONS) {
+		pi.registerCommand(`profiles:${action}`, {
+			description: `PiProfiles: ${action}`,
+			handler: (args: string, ctx: ExtensionCommandContext) =>
+				handleAction(`${action} ${args}`.trim(), ctx),
+		});
+	}
 }