add validate option logic

Author: ren-yamanashi

packages/aws-cdk/lib/commands/init/init.ts

@@ -91,6 +91,8 @@ export interface CliInitOptions {
  * Initialize a CDK package in the current directory
  */
 export async function cliInit(options: CliInitOptions) {
+  await ensureValidCliInitOptions(options, options.ioHelper);
+
   const ioHelper = options.ioHelper;
   const canUseNetwork = options.canUseNetwork ?? true;
   const generateOnly = options.generateOnly ?? false;
@@ -128,6 +130,15 @@ export async function cliInit(options: CliInitOptions) {
   );
 }
 
+/**
+ * Validate CLI init options and handle invalid or incompatible option combinations
+ */
+async function ensureValidCliInitOptions(options: CliInitOptions, ioHelper: IoHelper) {
+  if (options.packageManager && !['javascript', 'typescript'].includes(options.language ?? '')) {
+    await ioHelper.defaults.warn(`--package-manager option is only applicable for JavaScript and TypeScript projects. Ignoring the provided value: ${options.packageManager}`);
+  }
+}
+
 /**
  * Load a local custom template from file system path
  * @param fromPath - Path to the local template directory or multi-template repository

packages/aws-cdk/test/commands/init.test.ts

@@ -1423,6 +1423,61 @@ describe('constructs version', () => {
       expect(await fs.pathExists(path.join(workDir, 'requirements.txt'))).toBeTruthy();
     });
   });
+
+  describe('validate CLI init options', () => {
+    test.each([
+      'python',
+      'java',
+      'go',
+      'csharp',
+      'fsharp',
+    ])('warns when package-manager option is specified for non-JS language=%s', async (language) => {
+      await withTempDir(async (workDir) => {
+        const warnSpy = jest.spyOn(ioHelper.defaults, 'warn');
+
+        await cliInit({
+          ioHelper,
+          type: 'app',
+          language,
+          packageManager: 'npm',
+          canUseNetwork: false,
+          generateOnly: true,
+          workDir,
+        });
+
+        expect(warnSpy).toHaveBeenCalledWith(
+          expect.stringContaining('--package-manager option is only applicable for JavaScript and TypeScript projects'),
+        );
+
+        warnSpy.mockRestore();
+      });
+    });
+
+    test.each([
+      'typescript',
+      'javascript',
+    ])('does not warn when package-manager option is specified for language=%s', async (language) => {
+      await withTempDir(async (workDir) => {
+        const warnSpy = jest.spyOn(ioHelper.defaults, 'warn');
+
+        await cliInit({
+          ioHelper,
+          type: 'app',
+          language,
+          packageManager: 'npm',
+          canUseNetwork: false,
+          generateOnly: true,
+          workDir,
+        });
+
+        expect(warnSpy).not.toHaveBeenCalledWith(
+          expect.stringContaining('--package-manager option is only applicable for JavaScript and TypeScript projects'),
+        );
+
+        warnSpy.mockRestore();
+      });
+    });
+  })
 });
 
 test('when no version number is present (e.g., local development), the v2 templates are chosen by default', async () => {