refactor: move skill flag param at end

Author: damyanpetev

packages/cli/lib/commands/ai-config.ts

@@ -29,7 +29,7 @@ export function configureSkills(agents: AIAgentTarget[]): void {
 	}
 }
 
-export async function configure(agents?: AIAgentTarget[], skills = true, assistants?: AiCodingAssistant[]): Promise<void> {
+export async function configure(agents?: AIAgentTarget[], assistants?: AiCodingAssistant[], skills = true): Promise<void> {
 	if (!agents?.length) {
 		agents = await promptForAgents();
 	}
@@ -143,7 +143,7 @@ const command: CommandModule = {
 			Util.log("No AI configuration selected. Skipping.");
 			return;
 		}
-		await configure(agents, true, assistants);
+		await configure(agents, assistants);
 	}
 };
 

packages/cli/lib/commands/new.ts

@@ -169,7 +169,7 @@ const command: NewCommandType = {
 		const rawAgents = argv.agents as string[] | undefined;
 		const filteredAgents = rawAgents?.filter(a => a !== "none") as AIAgentTarget[] | undefined;
 		if (rawAgents == null || rawAgents.indexOf("none") === -1 || filteredAgents?.length) {
-			await configure(filteredAgents, true, argv.assistants as AiCodingAssistant[] | undefined);
+			await configure(filteredAgents, argv.assistants as AiCodingAssistant[] | undefined);
 		}
 		process.chdir("..");
 

spec/unit/new-spec.ts

@@ -408,23 +408,23 @@ describe("Unit - New command", () => {
 
 			await newCmd.handler({ name: "Test", framework: "jq", agents: ["claude", "cursor"], _: ["new"], $0: "new" });
 
-			expect(configureSpy).toHaveBeenCalledWith(["claude", "cursor"], true, undefined);
+			expect(configureSpy).toHaveBeenCalledWith(["claude", "cursor"], undefined);
 		});
 
 		it("calls configure with undefined when --agents is not provided", async () => {
 			createProjectMocks();
 
 			await newCmd.handler({ name: "Test", framework: "jq", _: ["new"], $0: "new" });
 
-			expect(configureSpy).toHaveBeenCalledWith(undefined, true, undefined);
+			expect(configureSpy).toHaveBeenCalledWith(undefined, undefined);
 		});
 
 		it("calls configure with single agent", async () => {
 			createProjectMocks();
 
 			await newCmd.handler({ name: "Test", framework: "jq", agents: ["generic"], _: ["new"], $0: "new" });
 
-			expect(configureSpy).toHaveBeenCalledWith(["generic"], true, undefined);
+			expect(configureSpy).toHaveBeenCalledWith(["generic"], undefined);
 		});
 
 		it("calls configure before package install", async () => {
@@ -469,7 +469,7 @@ describe("Unit - New command", () => {
 			await newCmd.handler({ name: "Test", framework: "jq", skipInstall: true, agents: ["claude"], _: ["new"], $0: "new" });
 
 			expect(PackageManager.installPackages).not.toHaveBeenCalled();
-			expect(configureSpy).toHaveBeenCalledWith(["claude"], true, undefined);
+			expect(configureSpy).toHaveBeenCalledWith(["claude"], undefined);
 		});
 
 		it("does not call configure when project creation fails (bad name)", async () => {