fix(cli): make the package.json and angular.json edits no-op when unchanged

ensureDevScript and angularProxyEdit re-serialized the whole file even when
nothing needed adding, which reformatted the user's package.json/angular.json
and made the edit op non-idempotent. Both now return the source untouched when
the target value is already present, so the file-writer skips the file. Also
reference vite.config.* / nuxt.config.* in the patcher summary messages.

Author: mridang

apps/cli/src/lib/orca/patchers/rule/angular/angular-json.ts

@@ -60,11 +60,19 @@ export function angularProxyEdit(opts: {
       });
     }
     const options = isObject(serve.options) ? (serve.options as Record<string, unknown>) : {};
+    let changed = false;
     if (options.proxyConfig === undefined) {
       options.proxyConfig = opts.proxyConfig;
+      changed = true;
     }
     if (opts.port !== undefined && options.port === undefined) {
       options.port = opts.port;
+      changed = true;
+    }
+    // Nothing to add — return the source untouched so the edit op skips it
+    // instead of reformatting (re-indenting) the user's angular.json.
+    if (!changed) {
+      return source;
     }
     serve.options = options;
     // Preserve the user's key order (JSON, no comments) instead of sorting.

apps/cli/src/lib/orca/patchers/rule/angular/index.ts

@@ -17,11 +17,13 @@ const SDK_DEPENDENCY = "@zitadel/sdk-angular";
  */
 function ensureDevScript(source: string | undefined): string {
   const pkg: Record<string, unknown> = source ? parseJsonObject(source, "package.json") : {};
-  const scripts: Record<string, unknown> = isObject(pkg.scripts) ? { ...pkg.scripts } : {};
-  if (scripts.dev === undefined) {
-    scripts.dev = "ng serve";
+  const scripts = isObject(pkg.scripts) ? pkg.scripts : undefined;
+  // Leave an existing dev script untouched, returning the source unchanged so
+  // the edit op skips the file instead of reformatting the user's package.json.
+  if (source !== undefined && scripts?.dev !== undefined) {
+    return source;
   }
-  pkg.scripts = scripts;
+  pkg.scripts = { ...(scripts ?? {}), dev: "ng serve" };
   return `${stableStringify(pkg)}\n`;
 }
 

apps/cli/src/lib/orca/patchers/rule/nuxt/index.ts

@@ -22,7 +22,7 @@ const NUXT_CONFIG_PATHS = configCandidates("nuxt.config");
 /**
  * Rule-based patcher for a Nuxt app. Like Next.js, Nuxt proxies the backend and
  * verifies the session through server middleware — here the `@zitadel/sdk-nuxt`
- * module, registered via a non-destructive `nuxt.config.ts` edit. Contributes
+ * module, registered via a non-destructive `nuxt.config.*` edit. Contributes
  * the login/register/profile pages (the raw `<zitadel-login>`/`<zitadel-logout>`
  * elements), the client/server plugins, the `app.vue` router, and the SDK dep.
  */
@@ -89,7 +89,7 @@ export class NuxtPatcher extends AbstractRulePatcher {
     return {
       title: "Nuxt integration",
       detail:
-        "Wrote login/register/profile pages + plugins and registered @zitadel/sdk-nuxt in nuxt.config.ts.",
+        "Wrote login/register/profile pages + plugins and registered @zitadel/sdk-nuxt in nuxt.config.*.",
     };
   }
 }

apps/cli/src/lib/orca/patchers/rule/react/index.ts

@@ -63,7 +63,7 @@ export class ReactPatcher extends AbstractRulePatcher implements ViteSupport {
     return {
       title: "React (Vite) integration",
       detail:
-        "Wrote src/App.tsx auth entry and merged the /__nextgen dev proxy into vite.config.ts.",
+        "Wrote src/App.tsx auth entry and merged the /__nextgen dev proxy into vite.config.*.",
     };
   }
 }

apps/cli/src/lib/orca/patchers/rule/vue/index.ts

@@ -58,7 +58,7 @@ export class VuePatcher extends AbstractRulePatcher implements ViteSupport {
     return {
       title: "Vue (Vite) integration",
       detail:
-        "Wrote src/App.vue auth entry and merged the /__nextgen dev proxy into vite.config.ts.",
+        "Wrote src/App.vue auth entry and merged the /__nextgen dev proxy into vite.config.*.",
     };
   }
 }

apps/cli/tests/unit/lib/orca/patchers/rule/angular/angular-json.test.ts

@@ -26,6 +26,11 @@ describe("angularProxyEdit", () => {
     expect(JSON.parse(out).projects.app.architect.serve.options.proxyConfig).toBe("mine.cjs");
   });
 
+  it("returns the source untouched when proxyConfig and port are already set", () => {
+    const once = edit(ng({ architect: { serve: { options: {} } } }));
+    expect(edit(once)).toBe(once);
+  });
+
   it("resolves the project named by defaultProject", () => {
     const out = edit(
       JSON.stringify({