Remove "--parse" from CDS compile command

Removes an unintended change in CDS compile (to .cds.json) behavior
due to the (mis)use of the "--parse" command.

Fixes a regression in the expected query results in at least one case:

`javascript/frameworks/cap/src/sensitive-exposure/SensitiveExposure.ql`

Author: data-douser

extractors/cds/tools/src/cds/compiler/functions.ts

@@ -219,10 +219,18 @@ export function compileCdsToJson(
       'warn',
     ];
 
-    // For root CDS files in project-aware mode, add --parse flag to ensure complete model
-    if (projectBasedCompilation) {
-      compileArgs.push('--parse');
-    }
+    /**
+     * using the `--parse` flag changes how the data is shown in the generated .cds.json file,
+     * where annotations and other metadata will show up under the `extensions` property instead
+     * of the `definitions` property, as used in previous versions of the CDS extractor.
+     *
+     * DELETE this comment if the `--parse` flag is definitely not needed in the future.
+     *
+     * // For root CDS files in project-aware mode, add --parse flag to ensure complete model
+     *if (projectBasedCompilation) {
+     *  compileArgs.push('--parse');
+     *}
+     */
 
     const result = spawnSync(cdsCommand, compileArgs, spawnOptions);
 

extractors/cds/tools/test/src/cds/compiler.test.ts

@@ -72,78 +72,6 @@ describe('cdsCompiler', () => {
       (path.relative as jest.Mock).mockImplementation((_from, _to) => 'relative/path');
     });
 
-    it('should use --parse-csn flag for root CDS files in project-aware mode', () => {
-      const cdsFilePath = '/path/to/main.cds';
-      const sourceRoot = '/path/to';
-      const cdsCommand = 'cds';
-      const resolvedCdsPath = `/resolved/${cdsFilePath}`;
-      const expectedJsonPath = `${resolvedCdsPath}.json`;
-
-      // Create a test project map with main.cds as a root file
-      const projectDir = 'test-project';
-      const projectMap = new Map();
-      const importsMap = new Map();
-      importsMap.set('main.cds', [{ resolvedPath: 'imported-model.cds' }]);
-
-      projectMap.set(projectDir, {
-        projectDir,
-        cdsFiles: ['main.cds', 'imported-model.cds'],
-        imports: importsMap,
-      });
-
-      // Mock filesystem.fileExists to return true for CDS file
-      (filesystem.fileExists as jest.Mock).mockImplementation(path => path === resolvedCdsPath);
-
-      // Mock successful compilation
-      (childProcess.spawnSync as jest.Mock).mockReturnValue({
-        status: 0,
-        stderr: null,
-        error: null,
-      });
-
-      // Mock that the output JSON file exists
-      (filesystem.fileExists as jest.Mock).mockImplementation(
-        path => path === resolvedCdsPath || path === expectedJsonPath,
-      );
-
-      // Mock that the output is not a directory
-      (filesystem.dirExists as jest.Mock).mockReturnValue(false);
-
-      // Call the function with project information
-      const result = compileCdsToJson(
-        cdsFilePath,
-        sourceRoot,
-        cdsCommand,
-        undefined,
-        projectMap,
-        projectDir,
-      );
-
-      expect(result).toEqual({
-        success: true,
-        outputPath: expectedJsonPath,
-        compiledAsProject: true,
-      });
-
-      // Check that --parse flag was added for root file
-      expect(childProcess.spawnSync).toHaveBeenCalledWith(
-        cdsCommand,
-        [
-          'compile',
-          resolvedCdsPath,
-          '--to',
-          'json',
-          '--dest',
-          expectedJsonPath,
-          '--locations',
-          '--log-level',
-          'warn',
-          '--parse',
-        ],
-        expect.anything(),
-      );
-    });
-
     it('should successfully compile CDS file to JSON file', () => {
       const cdsFilePath = '/path/to/model.cds';
       const sourceRoot = '/path/to';

extractors/cds/tools/test/src/cds/compiler/functions.test.ts

@@ -234,10 +234,9 @@ describe('cds compiler functions', () => {
       expect(result.outputPath).toBe(cdsJsonOutPath);
       expect(result.compiledAsProject).toBe(true);
 
-      // Verify the --parse flag was added for project compilation
       expect(childProcess.spawnSync).toHaveBeenCalledWith(
         'cds',
-        expect.arrayContaining(['compile', resolvedCdsPath, '--to', 'json', '--parse']),
+        expect.arrayContaining(['compile', resolvedCdsPath, '--to', 'json']),
         expect.any(Object),
       );
     });