fix(cli): ensure --no-provider skips hosting config prompt

closes #234

Author: RonithManikonda

packages/cli/src/commands/create/generate-hosting-config.ts

@@ -7,13 +7,13 @@ import { runTask } from 'src/utils/tasks.js';
 import cloudflareConfigRaw from './hosting-config/_headers.txt?raw';
 import netlifyConfigRaw from './hosting-config/netlify_toml.txt?raw';
 import vercelConfigRaw from './hosting-config/vercel.json?raw';
-import { DEFAULT_VALUES, readFlag } from './options.js';
+import { DEFAULT_VALUES, readFlag, type CreateOptions } from './options.js';
 
-export async function generateHostingConfig(dest: string, flags: { dryRun: boolean; provider?: string }) {
-  let provider = readFlag(flags, 'provider' as any);
+export async function generateHostingConfig(dest: string, flags: CreateOptions) {
+  let provider = readFlag(flags, 'provider');
 
   if (provider === undefined) {
-    provider = await prompts.select({
+    provider = (await prompts.select({
       message: 'Select hosting providers for automatic configuration:',
       options: [
         { value: 'Vercel', label: 'Vercel' },
@@ -22,36 +22,39 @@ export async function generateHostingConfig(dest: string, flags: { dryRun: boole
         { value: 'skip', label: 'Skip hosting configuration' },
       ],
       initialValue: DEFAULT_VALUES.provider,
-    });
+    })) as string;
   }
 
   if (provider === 'skip') {
     prompts.log.message(
-      `${chalk.blue('hosting provider config [skip]')} You can configure hosting provider settings manually later. For more information see https://tutorialkit.dev/guides/deployment/#headers-configuration`,
+      `${chalk.blue('hosting provider config [skip]')} You can configure hosting provider settings manually later.`
     );
-    return;
+    return provider;
   }
 
   prompts.log.info(`${chalk.blue('Hosting Configuration')} Setting up configuration for ${provider}`);
 
   const resolvedDest = path.resolve(dest);
-
   if (!fs.existsSync(resolvedDest)) {
     fs.mkdirSync(resolvedDest, { recursive: true });
   }
 
-  let config;
-  let filename;
+  let config: string | undefined;
+  let filename: string | undefined;
 
-  if (provider.includes('Vercel')) {
-    config = typeof vercelConfigRaw === 'string' ? vercelConfigRaw : JSON.stringify(vercelConfigRaw, null, 2);
-    filename = 'vercel.json';
-  } else if (provider.includes('Netlify')) {
-    config = netlifyConfigRaw;
-    filename = 'netlify.toml';
-  } else if (provider.includes('Cloudflare')) {
-    config = cloudflareConfigRaw;
-    filename = '_headers';
+  switch (provider) {
+    case 'Vercel':
+      config = typeof vercelConfigRaw === 'string' ? vercelConfigRaw : JSON.stringify(vercelConfigRaw, null, 2);
+      filename = 'vercel.json';
+      break;
+    case 'Netlify':
+      config = netlifyConfigRaw;
+      filename = 'netlify.toml';
+      break;
+    case 'Cloudflare':
+      config = cloudflareConfigRaw;
+      filename = '_headers';
+      break;
   }
 
   if (config && filename) {
@@ -62,9 +65,11 @@ export async function generateHostingConfig(dest: string, flags: { dryRun: boole
       task: async () => {
         const filepath = path.join(resolvedDest, filename);
         fs.writeFileSync(filepath, config);
-
         return `Added ${filepath}`;
       },
     });
   }
+
+  return provider;
 }
+

packages/cli/src/commands/create/index.ts

@@ -148,18 +148,7 @@ async function _createTutorial(flags: CreateOptions): Promise<undefined> {
 
   await copyTemplate(resolvedDest, flags);
 
-  const provider = await prompts.select({
-    message: 'Select hosting providers for automatic configuration:',
-    options: [
-      { value: 'Vercel', label: 'Vercel' },
-      { value: 'Netlify', label: 'Netlify' },
-      { value: 'Cloudflare', label: 'Cloudflare' },
-      { value: 'skip', label: 'Skip hosting configuration' },
-    ],
-    initialValue: 'Vercel',
-  });
-
-  await generateHostingConfig(resolvedDest, { dryRun: flags.dryRun, provider: String(provider) });
+  const provider = await generateHostingConfig(resolvedDest, flags);
 
   updatePackageJson(resolvedDest, tutorialName, flags, String(provider));
 

packages/cli/src/commands/create/options.ts

@@ -12,6 +12,7 @@ export interface CreateOptions {
   defaults?: boolean;
   packageManager?: string;
   force?: boolean;
+  provider?: string;
 }
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));