chore: remove some choices from create-fuels CLI

.changeset/quiet-toes-chew.md

@@ -0,0 +1,5 @@
+---
+"create-fuels": patch
+---
+
+chore: remove some choices from `create-fuels` CLI

.github/workflows/pr-release.yaml

@@ -8,7 +8,7 @@ jobs:
     name: "Release PR to npm"
     runs-on: ubuntu-latest
     # comment out if:false to enable release PR to npm
-    if: false
+    # if: false
     permissions: write-all
     steps:
       - name: Checkout

packages/create-fuels/src/cli.ts

@@ -10,7 +10,7 @@ import { join } from 'path';
 import { tryInstallFuelUp } from './lib';
 import { getPackageManager } from './lib/getPackageManager';
 import type { ProgramOptions } from './lib/setupProgram';
-import { promptForProgramsToInclude, promptForProjectPath } from './prompts';
+import { promptForProjectPath } from './prompts';
 import { error, log } from './utils/logger';
 
 export { setupProgram } from './lib/setupProgram';
@@ -50,20 +50,18 @@ export const runScaffoldCli = async ({
   program,
   args = process.argv,
   shouldInstallDeps = false,
-  forceDisablePrompts = false,
 }: {
   program: Command;
   args: string[];
   shouldInstallDeps?: boolean;
-  forceDisablePrompts?: boolean;
 }) => {
   program.parse(args);
 
   let projectPath = program.args[0] ?? (await promptForProjectPath());
 
   const opts = program.opts<ProgramOptions>();
   const verboseEnabled = opts.verbose ?? false;
-  const packageManager = await getPackageManager(opts);
+  const packageManager = getPackageManager(opts);
 
   if (!process.env.VITEST) {
     await tryInstallFuelUp(verboseEnabled);
@@ -102,9 +100,11 @@ export const runScaffoldCli = async ({
   if (hasAnyCliProgramsToInclude) {
     programsToInclude = cliProgramsToInclude;
   } else {
-    programsToInclude = await promptForProgramsToInclude({
-      forceDisablePrompts,
-    });
+    programsToInclude = {
+      contract: true,
+      predicate: true,
+      script: true,
+    };
   }
 
   while (!programsToInclude.contract && !programsToInclude.predicate && !programsToInclude.script) {
@@ -114,10 +114,6 @@ export const runScaffoldCli = async ({
     if (process.env.VITEST) {
       throw new Error();
     }
-
-    programsToInclude = await promptForProgramsToInclude({
-      forceDisablePrompts,
-    });
   }
 
   const fileCopySpinner = ora({

packages/create-fuels/src/lib/getPackageManager.test.ts

@@ -1,18 +1,13 @@
 import { mockLogger } from '../../test/utils/mockLogger';
-import * as promptsMod from '../prompts';
 
 import type { PackageManager } from './getPackageManager';
 import { availablePackageManagers, getPackageManager, packageMangers } from './getPackageManager';
 
-const mockAllDeps = (opts: { packageManager?: PackageManager } = {}) => {
+const mockAllDeps = () => {
   const { warn } = mockLogger();
-  const promptForPackageManager = vi
-    .spyOn(promptsMod, 'promptForPackageManager')
-    .mockResolvedValue(opts.packageManager);
 
   return {
     warn,
-    promptForPackageManager,
   };
 };
 
@@ -35,20 +30,20 @@ const runScenarios: [PackageManager, string][] = [
 describe('getPackageManager', () => {
   it.each(availablePackageManagers)(
     `should get the correct package manager for %s`,
-    async (packageManager: PackageManager) => {
+    (packageManager: PackageManager) => {
       const expectedPackageManager = packageMangers[packageManager];
       const opts = { [packageManager]: true };
 
-      const result = await getPackageManager(opts);
+      const result = getPackageManager(opts);
 
       expect(result).toEqual(expectedPackageManager);
     }
   );
 
   it.each(installScenarios)(
     'should have the correct install commands',
-    async (packageManager, expectedInstallCommand) => {
-      const command = await getPackageManager({ [packageManager]: true });
+    (packageManager, expectedInstallCommand) => {
+      const command = getPackageManager({ [packageManager]: true });
 
       const install = command.install;
 
@@ -58,50 +53,33 @@ describe('getPackageManager', () => {
 
   it.each(runScenarios)(
     'should have the correct run commands',
-    async (packageManager, expectedRunCommand) => {
-      const command = await getPackageManager({ [packageManager]: true });
+    (packageManager, expectedRunCommand) => {
+      const command = getPackageManager({ [packageManager]: true });
 
       const run = command.run(runCommand);
 
       expect(run).toEqual(expectedRunCommand);
     }
   );
 
-  it('should warn the user if more than one package manager selected', async () => {
-    const { warn, promptForPackageManager } = mockAllDeps();
+  it('should warn the user if more than one package manager selected', () => {
+    const { warn } = mockAllDeps();
     const opts = { pnpm: true, npm: true };
 
-    await getPackageManager(opts);
+    getPackageManager(opts);
 
-    expect(promptForPackageManager).toBeCalled();
     expect(warn).toBeCalledWith('More than one package manager was selected.');
   });
 
-  it('should allow inputting of a package manager via prompt', async () => {
+  it('should default to npm if no package manager is selected', () => {
     const packageManager = 'npm';
     const expectedPackageManager = packageMangers[packageManager];
-    const { warn, promptForPackageManager } = mockAllDeps({
-      packageManager,
-    });
+    const { warn } = mockAllDeps();
     const opts = {};
 
-    const result = await getPackageManager(opts);
+    const result = getPackageManager(opts);
 
     expect(warn).not.toBeCalled();
-    expect(promptForPackageManager).toBeCalled();
-    expect(result).toEqual(expectedPackageManager);
-  });
-
-  it('should default to pnpm if no package manager is selected', async () => {
-    const packageManager = 'pnpm';
-    const expectedPackageManager = packageMangers[packageManager];
-    const { warn, promptForPackageManager } = mockAllDeps();
-    const opts = {};
-
-    const result = await getPackageManager(opts);
-
-    expect(warn).not.toBeCalled();
-    expect(promptForPackageManager).toBeCalled();
     expect(result).toEqual(expectedPackageManager);
   });
 });

packages/create-fuels/src/lib/getPackageManager.ts

@@ -1,4 +1,3 @@
-import { promptForPackageManager } from '../prompts';
 import { warn } from '../utils/logger';
 
 import type { ProgramOptions } from './setupProgram';
@@ -26,7 +25,7 @@ export const packageMangers = {
   },
 } as const;
 
-export const getPackageManager = async (opts: ProgramOptions) => {
+export const getPackageManager = (opts: ProgramOptions) => {
   const packageMangerOpts = {
     pnpm: opts.pnpm,
     npm: opts.npm,
@@ -41,12 +40,9 @@ export const getPackageManager = async (opts: ProgramOptions) => {
   if (cliChosenPackageManagerSelected.length > 1) {
     warn('More than one package manager was selected.');
   }
-  if (cliChosenPackageManagerSelected.length !== 1) {
-    packageManager = await promptForPackageManager();
-  }
 
   if (!packageManager) {
-    packageManager = 'pnpm';
+    packageManager = 'npm'; // default to npm if the user has not specified a package manager (eg. --pnpm, --bun)
   }
   return packageMangers[packageManager];
 };

packages/create-fuels/src/lib/setupProgram.ts

@@ -19,9 +19,9 @@ export const setupProgram = () => {
     .option('-c, --contract', 'Include contract program')
     .option('-p, --predicate', 'Include predicate program')
     .option('-s, --script', 'Include script program')
-    .option('--pnpm', 'Use pnpm as the package manager')
-    .option('--npm', 'Use npm as the package manager')
-    .option('--bun', 'Use bun as the package manager')
+    .option('--pnpm', 'Use pnpm to install dependencies')
+    .option('--npm', 'Use npm to install dependencies')
+    .option('--bun', 'Use bun to install dependencies')
     .option('--verbose', 'Enable verbose logging')
     .addHelpCommand()
     .showHelpAfterError(true);

packages/create-fuels/src/prompts/index.ts

@@ -1,4 +1,2 @@
 export * from './promptFuelUpInstall';
 export * from './promptForProjectPath';
-export * from './promptForPackageManager';
-export * from './promptForProgramsToInclude';