@getcoherent/cli 0.6.53 → 0.6.54

package/dist/index.js

@@ -1830,9 +1830,9 @@ export function cn(...inputs: ClassValue[]) {
   if (!existsSync3(componentsUiPath)) mkdirSync3(componentsUiPath, { recursive: true });
 }
 async function ensureRegistryComponents(config2, projectPath) {
-  const provider = getComponentProvider();
+  const provider2 = getComponentProvider();
   const baseComponents = ["button", "card", "input", "label", "switch"];
-  await provider.installBatch(baseComponents, projectPath);
+  await provider2.installBatch(baseComponents, projectPath);
   const generator = new ComponentGenerator(config2);
   const uiDir = join3(projectPath, "components", "ui");
   if (!existsSync3(uiDir)) mkdirSync3(uiDir, { recursive: true });
@@ -2110,8 +2110,8 @@ import {
 
 // src/agents/modifier.ts
 import chalk4 from "chalk";
-async function parseModification(message, context, provider = "auto", options) {
-  const ai = await createAIProvider(provider);
+async function parseModification(message, context, provider2 = "auto", options) {
+  const ai = await createAIProvider(provider2);
   if (options?.planOnly) {
     const prompt2 = buildPlanOnlyPrompt(message, context.config);
     const raw2 = await ai.parseModification(prompt2);
@@ -3781,11 +3781,11 @@ async function updateManifestSafe(projectRoot, fn) {
   ]).catch(() => {
   });
 }
-async function splitGeneratePages(spinner, message, modCtx, provider, parseOpts) {
+async function splitGeneratePages(spinner, message, modCtx, provider2, parseOpts) {
   let pageNames = [];
   spinner.start("Phase 1/6 \u2014 Planning pages...");
   try {
-    const planResult = await parseModification(message, modCtx, provider, { ...parseOpts, planOnly: true });
+    const planResult = await parseModification(message, modCtx, provider2, { ...parseOpts, planOnly: true });
     const pageReqs = planResult.requests.filter((r) => r.type === "add-page");
     pageNames = pageReqs.map((r) => {
       const c = r.changes;
@@ -3845,7 +3845,7 @@ async function splitGeneratePages(spinner, message, modCtx, provider, parseOpts)
   if (parseOpts.projectRoot) {
     spinner.start("Phase 2/6 \u2014 Generating architecture plan...");
     try {
-      const ai = await createAIProvider(provider ?? "auto");
+      const ai = await createAIProvider(provider2 ?? "auto");
       const cachedPlan = loadPlan(parseOpts.projectRoot);
       let planWarnings = [];
       if (cachedPlan) {
@@ -3907,7 +3907,7 @@ async function splitGeneratePages(spinner, message, modCtx, provider, parseOpts)
     spinner.start(`Phase 3/6 \u2014 Generating ${homePage.name} page (sets design direction)...`);
     try {
       const anchorPrompt = buildAnchorPagePrompt(homePage, message, allPagesList, allRoutes, plan);
-      const homeResult = await parseModification(anchorPrompt, modCtx, provider, parseOpts);
+      const homeResult = await parseModification(anchorPrompt, modCtx, provider2, parseOpts);
       const codePage = homeResult.requests.find((r) => r.type === "add-page");
       if (codePage) {
         homeRequest = codePage;
@@ -3946,7 +3946,7 @@ async function splitGeneratePages(spinner, message, modCtx, provider, parseOpts)
           plan,
           styleContext,
           projectRoot,
-          await createAIProvider(provider ?? "auto")
+          await createAIProvider(provider2 ?? "auto")
         );
         if (generated.length > 0) {
           const updatedManifest = await loadManifest3(projectRoot);
@@ -3965,7 +3965,7 @@ async function splitGeneratePages(spinner, message, modCtx, provider, parseOpts)
       if (!shouldSkip) {
         spinner.start("Phase 4.5/6 \u2014 Extracting shared components (legacy)...");
         try {
-          const extraction = await extractSharedComponents(homePageCode, projectRoot, provider ?? "auto");
+          const extraction = await extractSharedComponents(homePageCode, projectRoot, provider2 ?? "auto");
           parseOpts.sharedComponentsSummary = extraction.summary;
           if (extraction.components.length > 0) {
             const names = extraction.components.map((c) => c.name).join(", ");
@@ -4070,7 +4070,7 @@ ${existingAppPageCode}
         styleContext
       ].filter(Boolean).join("\n\n");
       try {
-        const result = await parseModification(prompt, modCtx, provider, parseOpts);
+        const result = await parseModification(prompt, modCtx, provider2, parseOpts);
         phase5Done++;
         spinner.text = `Phase 5/6 \u2014 ${phase5Done}/${remainingPages.length} pages generated...`;
         const codePage = result.requests.find((r) => r.type === "add-page");
@@ -4087,7 +4087,7 @@ ${existingAppPageCode}
               const missedNames = verification.missed.map((m) => m.component);
               console.log(chalk6.yellow(`  \u26A0 Missed reuse in "${name}": ${missedNames.join(", ")} \u2014 patching...`));
               try {
-                const ai = await createAIProvider(provider);
+                const ai = await createAIProvider(provider2);
                 if (ai.editPageCode) {
                   const patchLines = verification.missed.map((m) => {
                     const importPath = m.importPath || `@/components/shared/${m.component.replace(/([A-Z])/g, (_, c, i) => (i ? "-" : "") + c.toLowerCase()).replace(/^-/, "")}`;
@@ -4153,7 +4153,7 @@ Keep all existing functionality. Only add imports and replace inline duplicates.
           retryPageType,
           retryTieredNote
         );
-        const retryResult = await parseModification(lightweightPrompt, modCtx, provider, {
+        const retryResult = await parseModification(lightweightPrompt, modCtx, provider2, {
           ...parseOpts,
           lightweight: true
         });
@@ -4219,14 +4219,14 @@ async function extractSharedComponents(homePageCode, projectRoot, aiProvider) {
     return true;
   });
   const results = [];
-  const provider = getComponentProvider();
+  const provider2 = getComponentProvider();
   for (const item of filtered) {
     try {
       let { code: fixedCode } = await autoFixCode(item.code);
       fixedCode = fixedCode.replace(/export default function (\w+)/g, "export function $1");
       const shadcnImports = [...fixedCode.matchAll(/from\s+["']@\/components\/ui\/(.+?)["']/g)];
       for (const match of shadcnImports) {
-        await provider.installComponent(match[1], projectRoot);
+        await provider2.installComponent(match[1], projectRoot);
       }
       const result = await generateSharedComponent2(projectRoot, {
         name: item.name,
@@ -4915,9 +4915,9 @@ async function applyModification(request, dsm, cm, pm, projectRoot, aiProvider,
     }
     case "add-component": {
       const componentData = request.changes;
-      const provider = getComponentProvider();
-      if (componentData.source === "shadcn" && provider.has(componentData.id)) {
-        const result2 = await provider.installComponent(componentData.id, projectRoot);
+      const provider2 = getComponentProvider();
+      if (componentData.source === "shadcn" && provider2.has(componentData.id)) {
+        const result2 = await provider2.installComponent(componentData.id, projectRoot);
         if (result2.success && result2.componentDef) {
           const mergedData = {
             ...result2.componentDef,
@@ -5628,8 +5628,8 @@ async function interactiveChat(options, chatCommandFn) {
   await dsm.load();
   const cm = new ComponentManager3(config2);
   const validProviders = ["claude", "openai", "auto"];
-  const provider = (options.provider || "auto").toLowerCase();
-  if (!validProviders.includes(provider)) {
+  const provider2 = (options.provider || "auto").toLowerCase();
+  if (!validProviders.includes(provider2)) {
     console.error(chalk9.red(`
 \u274C Invalid provider: ${options.provider}`));
     process.exit(1);
@@ -5774,7 +5774,7 @@ async function interactiveChat(options, chatCommandFn) {
       resolvedInput = componentMatch[2];
     }
     try {
-      await chatCommandFn(resolvedInput, { provider, interactive: false, _throwOnError: true, ...extraOpts });
+      await chatCommandFn(resolvedInput, { provider: provider2, interactive: false, _throwOnError: true, ...extraOpts });
       await dsm.load();
     } catch (err) {
       if (!err._printed) {
@@ -5825,11 +5825,11 @@ async function chatCommand(message, options) {
     bail("Migration in progress");
   }
   const validProviders = ["claude", "openai", "auto"];
-  const provider = (options.provider || "auto").toLowerCase();
+  const provider2 = (options.provider || "auto").toLowerCase();
   let releaseLock;
   try {
     releaseLock = await acquireProjectLock(projectRoot);
-    if (!validProviders.includes(provider)) {
+    if (!validProviders.includes(provider2)) {
       spinner.fail("Invalid provider");
       console.error(chalk10.red(`
 \u274C Invalid provider: ${options.provider}`));
@@ -5871,7 +5871,7 @@ async function chatCommand(message, options) {
       const { generateSharedComponent: generateSharedComponent6 } = await import("@getcoherent/core");
       const { autoFixCode: autoFixCode2 } = await import("./quality-validator-IM6YFKLI.js");
       const { extractPropsInterface, extractDependencies } = await import("./component-extractor-VYJLT5NR.js");
-      const aiProvider = await createAIProvider2(provider ?? "auto");
+      const aiProvider = await createAIProvider2(provider2 ?? "auto");
       const prompt = `Generate a React component called "${componentName}". Description: ${message}.
 Use shadcn/ui components and Tailwind CSS semantic tokens. Export the component as a named export.
 Include a TypeScript props interface.
@@ -5996,12 +5996,13 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
     const SPLIT_THRESHOLD = 4;
     const parseOpts = { sharedComponentsSummary, projectRoot };
     const modCtx = { config: dsm.getConfig(), componentManager: cm };
-    const multiPageHint = /\b(pages?|sections?)\s*[:]\s*\w/i.test(message) || (message.match(
+    const explicitPageTarget = !!(options.page || options.component);
+    const multiPageHint = !explicitPageTarget && (/\b(pages?|sections?)\s*[:]\s*\w/i.test(message) || (message.match(
       /\b(?:registration|about|catal|account|contact|pricing|dashboard|settings|login|sign.?up|blog|portfolio|features)\b/gi
-    ) || []).length >= SPLIT_THRESHOLD;
+    ) || []).length >= SPLIT_THRESHOLD);
     if (multiPageHint) {
       try {
-        const splitResult = await splitGeneratePages(spinner, message, modCtx, provider, parseOpts);
+        const splitResult = await splitGeneratePages(spinner, message, modCtx, provider2, parseOpts);
         requests = splitResult.requests;
         if (splitResult.plan && projectRoot) {
           savePlan(projectRoot, splitResult.plan);
@@ -6011,7 +6012,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
       } catch {
         spinner.warn("Split generation encountered an issue \u2014 trying page-by-page...");
         try {
-          const planResult = await parseModification(message, modCtx, provider, { ...parseOpts, planOnly: true });
+          const planResult = await parseModification(message, modCtx, provider2, { ...parseOpts, planOnly: true });
           const pageReqs = planResult.requests.filter((r) => r.type === "add-page");
           requests = [];
           for (let i = 0; i < pageReqs.length; i++) {
@@ -6023,7 +6024,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
               const single = await parseModification(
                 `Create ONE page called "${pageName}" at route "${pageRoute}". Context: ${message}. Generate complete pageCode for this single page only.`,
                 modCtx,
-                provider,
+                provider2,
                 parseOpts
               );
               const codePage = single.requests.find((r) => r.type === "add-page");
@@ -6061,7 +6062,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
       } catch {
       }
       try {
-        const result = await parseModification(message, modCtx, provider, { ...parseOpts, reusePlanDirective });
+        const result = await parseModification(message, modCtx, provider2, { ...parseOpts, reusePlanDirective });
         requests = result.requests;
         uxRecommendations = result.uxRecommendations;
         const pagesWithoutCode = requests.filter(
@@ -6077,7 +6078,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
               const single = await parseModification(
                 `Create a page called "${pageName}" at route "${page.route || "/" + (page.id || pageName.toLowerCase())}". ${message}. Generate complete pageCode for this ONE page only.`,
                 modCtx,
-                provider,
+                provider2,
                 parseOpts
               );
               const codePage = single.requests.find((r) => r.type === "add-page");
@@ -6095,7 +6096,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
         if (isTruncated || isJsonError) {
           spinner.warn("Response too large \u2014 splitting into smaller requests...");
           try {
-            const splitResult = await splitGeneratePages(spinner, message, modCtx, provider, parseOpts);
+            const splitResult = await splitGeneratePages(spinner, message, modCtx, provider2, parseOpts);
             requests = splitResult.requests;
             if (splitResult.plan && projectRoot) {
               savePlan(projectRoot, splitResult.plan);
@@ -6229,15 +6230,15 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
     if (missingComponents.length > 0) {
       spinner.stop();
       console.log(chalk10.cyan("\n\u{1F50D} Pre-flight check: Installing missing components...\n"));
-      const provider2 = getComponentProvider();
+      const provider3 = getComponentProvider();
       for (const componentId of missingComponents) {
         if (DEBUG4) {
           console.log(chalk10.gray(`    [DEBUG] Trying to install: ${componentId}`));
-          console.log(chalk10.gray(`    [DEBUG] provider.has(${componentId}): ${provider2.has(componentId)}`));
+          console.log(chalk10.gray(`    [DEBUG] provider.has(${componentId}): ${provider3.has(componentId)}`));
         }
-        if (provider2.has(componentId)) {
+        if (provider3.has(componentId)) {
           try {
-            const result = await provider2.installComponent(componentId, projectRoot);
+            const result = await provider3.installComponent(componentId, projectRoot);
             if (DEBUG4) console.log(chalk10.gray(`    [DEBUG] installComponent result: ${result.success}`));
             if (result.success && result.componentDef) {
               if (!cm.read(componentId)) {
@@ -6322,7 +6323,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
     spinner.start("Applying modifications...");
     const results = [];
     for (const request of normalizedRequests) {
-      const result = await applyModification(request, dsm, cm, pm, projectRoot, provider, message);
+      const result = await applyModification(request, dsm, cm, pm, projectRoot, provider2, message);
       results.push(result);
     }
     for (const request of normalizedRequests) {
@@ -6413,7 +6414,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
             const { requests: linkedRequests } = await parseModification(
               `add ${pageName} page at route ${linkedRoute}`,
               { config: dsm.getConfig(), componentManager: cm },
-              provider
+              provider2
             );
             let anySuccess = false;
             for (const raw of linkedRequests.map((r) => applyDefaults(r))) {
@@ -6422,7 +6423,7 @@ Return JSON: { "requests": [{ "type": "add-page", "changes": { "name": "${compon
                 scaffoldSpinner.warn(`  Skipped scaffold: ${linkedReq.error}`);
                 continue;
               }
-              const linkedResult = await applyModification(linkedReq, dsm, cm, pm, projectRoot, provider);
+              const linkedResult = await applyModification(linkedReq, dsm, cm, pm, projectRoot, provider2);
               if (linkedResult.success) {
                 results.push(linkedResult);
                 normalizedRequests.push(linkedReq);
@@ -6710,7 +6711,7 @@ ${uxRecommendations}
         console.log(chalk10.white('   coherent chat "add account page"'));
         console.log(chalk10.white('   coherent chat "add settings page"'));
       } else if (error.message.includes("API key not found") || error.message.includes("ANTHROPIC_API_KEY") || error.message.includes("OPENAI_API_KEY")) {
-        const isOpenAI = error.message.includes("OpenAI") || typeof provider !== "undefined" && provider === "openai";
+        const isOpenAI = error.message.includes("OpenAI") || typeof provider2 !== "undefined" && provider2 === "openai";
         const providerName = isOpenAI ? "OpenAI" : "Anthropic Claude";
         const envVar = isOpenAI ? "OPENAI_API_KEY" : "ANTHROPIC_API_KEY";
         const url = isOpenAI ? "https://platform.openai.com" : "https://console.anthropic.com";
@@ -7381,13 +7382,13 @@ async function fixMissingComponentExports(projectRoot) {
   } catch {
   }
   const generator = new ComponentGenerator2(config2 || { components: [], pages: [], tokens: {} });
-  const provider = getComponentProvider();
+  const provider2 = getComponentProvider();
   for (const [componentId, needed] of neededExports) {
     const componentFile = join9(uiDir, `${componentId}.tsx`);
     if (!existsSync11(componentFile)) {
-      if (provider.has(componentId)) {
+      if (provider2.has(componentId)) {
         try {
-          const result = await provider.installComponent(componentId, projectRoot);
+          const result = await provider2.installComponent(componentId, projectRoot);
           if (result.success) {
             console.log(chalk11.dim(`   \u2714 Installed missing ${componentId}.tsx`));
           }
@@ -7420,9 +7421,9 @@ async function fixMissingComponentExports(projectRoot) {
     }
     const missing = [...needed].filter((n) => !existingExports.has(n));
     if (missing.length === 0) continue;
-    if (provider.has(componentId)) {
+    if (provider2.has(componentId)) {
       try {
-        const result = await provider.installComponent(componentId, projectRoot, { force: true });
+        const result = await provider2.installComponent(componentId, projectRoot, { force: true });
         if (result.success) {
           console.log(chalk11.dim(`   \u2714 Reinstalled ${componentId}.tsx (added missing exports: ${missing.join(", ")})`));
         }
@@ -7519,8 +7520,8 @@ function extractShadcnComponentFromModuleNotFound(msg) {
   return m?.[1] ?? null;
 }
 async function autoInstallShadcnComponent(componentId, projectRoot) {
-  const provider = getComponentProvider();
-  const result = await provider.installComponent(componentId, projectRoot);
+  const provider2 = getComponentProvider();
+  const result = await provider2.installComponent(componentId, projectRoot);
   return result.success;
 }
 var DEFAULT_PORT = 3e3;
@@ -8309,8 +8310,8 @@ async function fixCommand(opts = {}) {
         if (!existsSync13(filePath)) missingFiles.push(id);
       }
     }
-    const provider = getComponentProvider();
-    const toInstall = [.../* @__PURE__ */ new Set([...missingComponents, ...missingFiles])].filter((id) => provider.has(id));
+    const provider2 = getComponentProvider();
+    const toInstall = [.../* @__PURE__ */ new Set([...missingComponents, ...missingFiles])].filter((id) => provider2.has(id));
     if (toInstall.length > 0) {
       if (dryRun) {
         fixes.push(`Would install components: ${toInstall.join(", ")}`);
@@ -8319,7 +8320,7 @@ async function fixCommand(opts = {}) {
         let installed = 0;
         for (const componentId of toInstall) {
           try {
-            const result = await provider.installComponent(componentId, projectRoot);
+            const result = await provider2.installComponent(componentId, projectRoot);
             if (!result.success) continue;
             if (result.componentDef && !cm.read(componentId)) {
               const regResult = await cm.register(result.componentDef);
@@ -8493,6 +8494,15 @@ async function fixCommand(opts = {}) {
       const hasSidebar = plan.groups.some((g) => g.layout === "sidebar" || g.layout === "both");
       const sidebarPath = resolve8(projectRoot, "components", "shared", "sidebar.tsx");
       if (hasSidebar && !existsSync13(sidebarPath) && !dryRun) {
+        const sidebarUiPath = resolve8(projectRoot, "components", "ui", "sidebar.tsx");
+        if (!existsSync13(sidebarUiPath) && provider.has("sidebar")) {
+          try {
+            await provider.installComponent("sidebar", projectRoot);
+            console.log(chalk15.green("  \u2714 Auto-installed Sidebar UI component"));
+          } catch {
+            console.log(chalk15.yellow("  \u26A0 Could not install Sidebar UI component"));
+          }
+        }
         if (!dsm) {
           dsm = new DesignSystemManager8(project.configPath);
           await dsm.load();
@@ -10645,8 +10655,8 @@ async function migrateAction(options) {
     console.log(chalk28.yellow("No components/ui directory found. Nothing to migrate."));
     return;
   }
-  const provider = getComponentProvider();
-  const managedIds = new Set(provider.listNames());
+  const provider2 = getComponentProvider();
+  const managedIds = new Set(provider2.listNames());
   const files = readdirSync9(uiDir).filter((f) => f.endsWith(".tsx"));
   const migratable = files.map((f) => f.replace(".tsx", "")).filter((id) => managedIds.has(id));
   if (migratable.length === 0) {
@@ -10670,7 +10680,7 @@ Found ${migratable.length} component(s) to migrate:`));
       const filePath = join16(uiDir, `${id}.tsx`);
       if (existsSync19(filePath)) rmSync6(filePath);
     }
-    const results = await provider.installBatch(migratable, projectRoot, { force: true });
+    const results = await provider2.installBatch(migratable, projectRoot, { force: true });
     let migrated = 0;
     for (const [id, result] of results) {
       if (result.success) {

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.6.53",
+  "version": "0.6.54",
   "description": "CLI interface for Coherent Design Method",
   "type": "module",
   "main": "./dist/index.js",