Fix pruneSourceNativeArtifacts to detect vendor platform containers generically

The previous prebuilds-name-only approach missed vendor/audio-capture/ which
uses the same platform-subdir layout but without a 'prebuilds' dir name.

New heuristic: a directory is a 'platform container' when ALL its direct
subdirectories have compound platform-arch names (e.g. darwin-arm64,
linux-x64, arm64-linux, win32-x64) AND there are at least 2 subdirs
(prevents false-positives on single-platform optional deps).

looksLikeNativePlatformDir() requires both a platform component AND an arch
component in the name, so bare 'x64' or 'arm64' don't match, and neither
does '@swc/core-win32-x64-msvc' (starts with 'core-' not a platform token).
This safely handles the @swc/ scope case that the earlier implementation
accidentally broke.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: newbe36524

packages/code-server/scripts/build-artifacts.mjs

@@ -163,45 +163,56 @@ export async function pruneSourceNativeArtifacts(sourceRoot = codeServerRoot) {
   let removedAny = false
 
   async function walkSource(dirPath) {
-    let entries
+    let dirents
     try {
-      entries = await readdir(dirPath)
+      dirents = await readdir(dirPath, { withFileTypes: true })
     } catch {
       return
     }
 
-    for (const entry of entries) {
-      const entryPath = path.join(dirPath, entry)
+    const subDirNames = dirents.filter((d) => d.isDirectory()).map((d) => d.name)
 
-      // Skip vscode-reh-web output directories at lib/ level
-      if (dirPath === libDir && entry.startsWith("vscode-reh-web-")) {
-        continue
-      }
+    // Skip vscode-reh-web output directories at lib/ level — those are handled separately
+    const toVisit = subDirNames.filter(
+      (name) => !(dirPath === libDir && name.startsWith("vscode-reh-web-")),
+    )
 
-      const entryStat = await stat(entryPath).catch(() => null)
-      if (!entryStat?.isDirectory()) continue
-
-      if (entry === "prebuilds") {
-        // Prune non-Windows platform subdirs from this prebuilds directory
-        const platformDirs = await readdir(entryPath).catch(() => [])
-        for (const platformDir of platformDirs) {
-          if (!shouldKeepWindowsNativeArtifact(platformDir)) {
-            await rm(path.join(entryPath, platformDir), { recursive: true, force: true })
-            removedAny = true
-          }
+    // A directory is a "platform container" when ALL its subdirectories have compound
+    // platform-arch names (e.g. darwin-arm64, linux-x64, win32-x64, arm64-linux).
+    // Require at least 2 to avoid false-positives on single-platform optional deps.
+    if (toVisit.length >= 2 && toVisit.every(looksLikeNativePlatformDir)) {
+      for (const name of toVisit) {
+        if (!shouldKeepWindowsNativeArtifact(name)) {
+          await rm(path.join(dirPath, name), { recursive: true, force: true })
+          removedAny = true
         }
-        // Don't recurse into prebuilds
-        continue
       }
+      return
+    }
 
-      await walkSource(entryPath)
+    for (const name of toVisit) {
+      await walkSource(path.join(dirPath, name))
     }
   }
 
   await walkSource(sourceRoot)
   return removedAny
 }
 
+// Returns true for compound platform-arch directory names used by prebuild-install,
+// node-pre-gyp, and similar tools (e.g. darwin-arm64, linux-x64, arm64-linux, win32-x64).
+// Requires both a platform and an arch component to avoid false-positives on names like
+// 'x64' or 'arm64' which are used by packages that are NOT native prebuilds containers.
+export function looksLikeNativePlatformDir(name) {
+  return (
+    /^(darwin|linux|android|freebsd|openbsd|sunos|win32|windows)[-_](arm64|arm|x64|ia32|x86|s390x|ppc64|riscv64|loong64)/.test(
+      name,
+    ) ||
+    /^(arm64|arm|x64|ia32|x86)[-_](linux|darwin|win32|windows|musl)/.test(name)
+  )
+}
+
+
 export async function pruneWindowsNativeArtifacts(
   runtimeRoot = path.join(codeServerRoot, `lib/vscode-reh-web-win32-${upstreamArch}`),
 ) {

packages/code-server/scripts/build-artifacts.test.mjs

@@ -6,6 +6,7 @@ import path from "node:path"
 
 import {
   copyPackageTemplates,
+  looksLikeNativePlatformDir,
   patchBuildVscodeScript,
   pruneSourceNativeArtifacts,
   pruneWindowsNativeArtifacts,
@@ -228,3 +229,74 @@ test("pruneSourceNativeArtifacts removes non-Windows prebuilds from source tree
   await access(path.join(outputPrebuilds, "win32-x64", "computer.node"))
 })
 
+test("pruneSourceNativeArtifacts handles vendor/audio-capture style platform containers", async () => {
+  const sourceRoot = await mkdtemp(path.join(os.tmpdir(), "code-server-source-prune-vendor-"))
+  const libDir = path.join(sourceRoot, "lib")
+
+  // Simulate @anthropic-ai/claude-agent-sdk/vendor/audio-capture platform dirs
+  const audioCaptureDir = path.join(
+    sourceRoot,
+    "lib",
+    "vscode",
+    "extensions",
+    "copilot",
+    "node_modules",
+    "@anthropic-ai",
+    "claude-agent-sdk",
+    "vendor",
+    "audio-capture",
+  )
+  await mkdir(path.join(audioCaptureDir, "arm64-linux"), { recursive: true })
+  await mkdir(path.join(audioCaptureDir, "darwin-arm64"), { recursive: true })
+  await mkdir(path.join(audioCaptureDir, "win32-x64"), { recursive: true })
+  for (const plat of ["arm64-linux", "darwin-arm64", "win32-x64"]) {
+    await writeFile(path.join(audioCaptureDir, plat, "audio-capture.node"), `${plat}\n`)
+  }
+
+  // Simulate @swc/ scope — should NOT be treated as a platform container
+  const swcDir = path.join(
+    sourceRoot,
+    "lib",
+    "vscode",
+    "extensions",
+    "copilot",
+    "node_modules",
+    "@swc",
+  )
+  await mkdir(path.join(swcDir, "core"), { recursive: true })
+  await mkdir(path.join(swcDir, "helpers"), { recursive: true })
+  await mkdir(path.join(swcDir, "core-win32-x64-msvc"), { recursive: true })
+  await writeFile(path.join(swcDir, "core-win32-x64-msvc", "swc.win32.node"), "native\n")
+  await writeFile(path.join(swcDir, "helpers", "index.js"), "// helpers\n")
+
+  const changed = await pruneSourceNativeArtifacts(sourceRoot)
+
+  assert.equal(changed, true)
+
+  // audio-capture: Windows kept, non-Windows removed
+  await access(path.join(audioCaptureDir, "win32-x64", "audio-capture.node"))
+  await assert.rejects(access(path.join(audioCaptureDir, "arm64-linux", "audio-capture.node")))
+  await assert.rejects(access(path.join(audioCaptureDir, "darwin-arm64", "audio-capture.node")))
+
+  // @swc scope: untouched — core, helpers, and core-win32-x64-msvc all preserved
+  await access(path.join(swcDir, "helpers", "index.js"))
+  await access(path.join(swcDir, "core-win32-x64-msvc", "swc.win32.node"))
+})
+
+test("looksLikeNativePlatformDir identifies compound platform-arch names", () => {
+  assert.equal(looksLikeNativePlatformDir("darwin-arm64"), true)
+  assert.equal(looksLikeNativePlatformDir("darwin-x64"), true)
+  assert.equal(looksLikeNativePlatformDir("linux-x64"), true)
+  assert.equal(looksLikeNativePlatformDir("win32-x64"), true)
+  assert.equal(looksLikeNativePlatformDir("arm64-linux"), true)
+  assert.equal(looksLikeNativePlatformDir("windows-arm64"), true)
+
+  // Single-component names must NOT match
+  assert.equal(looksLikeNativePlatformDir("x64"), false)
+  assert.equal(looksLikeNativePlatformDir("arm64"), false)
+  assert.equal(looksLikeNativePlatformDir("core"), false)
+  assert.equal(looksLikeNativePlatformDir("helpers"), false)
+  assert.equal(looksLikeNativePlatformDir("Release"), false)
+  assert.equal(looksLikeNativePlatformDir("core-win32-x64-msvc"), false)
+})
+