feat(runtime): mark code-server as optional component

Mark code-server as optional component (required: false) to align with omniroute. This allows code-server to be excluded from default bundled runtime installs and only installed when explicitly requested.

Changes:

- Add required: false to code-server component in manifest

- Update integration tests to verify code-server is not installed by default

- Update unit tests to verify code-server optional behavior

- Update optional install test to include code-server

Co-Authored-By: Hagicode <noreply@hagicode.com>
Signed-off-by: newbe36524 <newbe36524@qq.com>

Author: newbe36524

runtime/manifest.yaml

@@ -50,6 +50,7 @@ components:
 
 - name: "code-server"
   type: "bundled-runtime"
+  required: false
   source: "desktop-vendored-runtime"
   version: "4.117.0"
   bundledInstallMode: "archive-7z-only"

scripts/integration-runtime-key-path.mjs

@@ -253,32 +253,14 @@ try {
       ".hagicode-runtime.json"
     );
 
-    assertFile(codeServerArchive);
+    assertMissingPath(codeServerArchive);
     assertMissingPath(omnirouteArchive);
     assertMissingPath(omnirouteBin);
     assertMissingPath(codeServerBin);
     assertMissingPath(omnirouteCurrentRoot);
     assertMissingPath(codeServerCurrentRoot);
     assertMissingPath(omnirouteMarkerPath);
-
-    const codeServerMarker = JSON.parse(
-      fs.readFileSync(codeServerMarkerPath, "utf8")
-    );
-    assertEqual(
-      codeServerMarker.bundledInstallMode,
-      "archive-7z-only",
-      "code-server archive-only marker mode"
-    );
-    assertEqual(
-      codeServerMarker.archivePath,
-      codeServerArchive,
-      "code-server archive marker path"
-    );
-    assertEqual(
-      codeServerMarker.wrapperPath,
-      null,
-      "code-server archive-only wrapper marker"
-    );
+    assertMissingPath(codeServerMarkerPath);
 
     runtimeInstallLines.push(
       `- Managed root: ${managedRoot}`,
@@ -287,9 +269,8 @@ try {
       `- Managed npm prefix reserved path: ${path.join(managedRoot, "program", "npm")}`,
       `- Installed components: ${componentNames.join(", ")}`,
       `- Omniroute optional archive not installed by default: ${omnirouteArchive}`,
-      `- Code-server archive: ${codeServerArchive}`,
-      `- Code-server vendored asset: ${codeServerMarker.vendoredAssetName}`,
-      "- Verified default bundled runtime install keeps required code-server as archive-7z-only and leaves optional omniroute uninstalled"
+      `- Code-server optional archive not installed by default: ${codeServerArchive}`,
+      "- Verified default bundled runtime install leaves optional omniroute and code-server uninstalled"
     );
   });
 
@@ -341,14 +322,14 @@ try {
         "--runtime-root",
         managedRoot,
         "--components",
-        "omniroute"
+        "omniroute,code-server"
       ],
       repoRoot
     );
     assertIncludes(
       optionalInstallOutput,
       "Runtime install complete.",
-      "optional omniroute archive install output"
+      "optional bundled runtime archive install output"
     );
 
     await prepareBundledServiceConfigs(managedRoot);

src/__tests__/runtime-manager.test.ts

@@ -135,6 +135,7 @@ components:
     )
     expect(manifest.componentMap.get("omniroute")?.required).toBe(false)
     expect(manifest.componentMap.get("omniroute")?.bundledInstallMode).toBe("archive-7z-only")
+    expect(manifest.componentMap.get("code-server")?.required).toBe(false)
     expect(manifest.componentMap.get("code-server")?.bundledInstallMode).toBe("archive-7z-only")
   })
 
@@ -149,10 +150,10 @@ components:
     expect(plan.plan.map((item) => item.componentName)).toEqual([
       "node",
       "dotnet",
-      "server",
-      "code-server"
+      "server"
     ])
     expect(plan.plan.map((item) => item.componentName)).not.toContain("omniroute")
+    expect(plan.plan.map((item) => item.componentName)).not.toContain("code-server")
   })
 
   it("includes optional components when explicitly requested", async () => {
@@ -161,11 +162,15 @@ components:
     })
     const state = createInitialRuntimeState(manifest, resolveRuntimePaths(manifest))
 
-    const plan = planRuntimeLifecycle("install", manifest, state, {
+    const omniRoutePlan = planRuntimeLifecycle("install", manifest, state, {
       components: ["omniroute"]
     })
+    const codeServerPlan = planRuntimeLifecycle("install", manifest, state, {
+      components: ["code-server"]
+    })
 
-    expect(plan.plan.map((item) => item.componentName)).toEqual(["node", "omniroute"])
+    expect(omniRoutePlan.plan.map((item) => item.componentName)).toEqual(["node", "omniroute"])
+    expect(codeServerPlan.plan.map((item) => item.componentName)).toEqual(["node", "code-server"])
   })
 
   it("rejects bundledInstallMode on non-bundled runtime components", async () => {
@@ -490,7 +495,7 @@ components:
       const state = createInitialRuntimeState(manifest, paths)
       const now = new Date().toISOString()
 
-      for (const componentName of ["node", "dotnet", "server", "code-server"] as const) {
+      for (const componentName of ["node", "dotnet", "server"] as const) {
         const component = manifest.componentMap.get(componentName)
         if (!component) {
           throw new Error(`Missing fixture component ${componentName}`)
@@ -518,10 +523,13 @@ components:
 
       const report = await queryRuntimeState({ runtimeRoot })
       const omniroute = report.components.find((component) => component.name === "omniroute")
+      const codeServer = report.components.find((component) => component.name === "code-server")
 
       expect(report.ready).toBe(true)
       expect(omniroute?.required).toBe(false)
       expect(omniroute?.status).toBe("not-installed")
+      expect(codeServer?.required).toBe(false)
+      expect(codeServer?.status).toBe("not-installed")
     } finally {
       await rm(runtimeRoot, { recursive: true, force: true })
     }