fix(@angular/cli): correctly parse scoped packages in yarn classic list output

The `parseYarnClassicDependencies` function incorrectly parsed scoped packages (e.g., `@angular/[email protected]`) by splitting at the first `@`, resulting in an empty name. This commit updates the logic to split at the last `@`, ensuring scoped package names are correctly identified.

Author: clydin

packages/angular/cli/src/package-managers/parsers.ts

@@ -155,8 +155,10 @@ export function parseYarnClassicDependencies(
   for (const json of parseJsonLines(stdout, logger)) {
     if (json.type === 'tree' && json.data?.trees) {
       for (const info of json.data.trees) {
-        const name = info.name.split('@')[0];
-        const version = info.name.split('@').pop();
+        const lastAtIndex = info.name.lastIndexOf('@');
+        const name = info.name.slice(0, lastAtIndex);
+        const version = info.name.slice(lastAtIndex + 1);
+
         dependencies.set(name, {
           name,
           version,

packages/angular/cli/src/package-managers/parsers_spec.ts

@@ -9,6 +9,7 @@
 import {
   parseBunDependencies,
   parseNpmLikeError,
+  parseYarnClassicDependencies,
   parseYarnClassicError,
   parseYarnModernDependencies,
 } from './parsers';
@@ -115,6 +116,38 @@ describe('parsers', () => {
     });
   });
 
+  describe('parseYarnClassicDependencies', () => {
+    it('should parse yarn classic list output', () => {
+      const stdout = JSON.stringify({
+        type: 'tree',
+        data: {
+          trees: [{ name: '[email protected]', children: [] }],
+        },
+      });
+
+      const deps = parseYarnClassicDependencies(stdout);
+      expect(deps.size).toBe(1);
+      expect(deps.get('rxjs')).toEqual({ name: 'rxjs', version: '7.8.2' });
+    });
+
+    it('should handle scoped packages', () => {
+      const stdout = JSON.stringify({
+        type: 'tree',
+        data: {
+          trees: [{ name: '@angular/[email protected]', children: [] }],
+        },
+      });
+
+      const deps = parseYarnClassicDependencies(stdout);
+      expect(deps.size).toBe(1);
+      expect(deps.get('@angular/core')).toEqual({ name: '@angular/core', version: '18.0.0' });
+    });
+
+    it('should return empty map for empty stdout', () => {
+      expect(parseYarnClassicDependencies('').size).toBe(0);
+    });
+  });
+
   describe('parseBunDependencies', () => {
     it('should parse bun pm ls output', () => {
       const stdout = `