Fix native CI failures

Three issues surfaced on the first CI run:

- WASM CI (ci.yml) broke because `native` was added to pnpm-workspace.yaml
  but the committed pnpm-lock.yaml has no native importer, so the
  frozen-lockfile install failed. The native package is standalone (npm +
  make, with unpublished platform optionalDependencies) and doesn't belong
  in the pnpm WASM monorepo. Remove it from the workspace.
- glibc Linux jobs ran the Alpine `apk add` step and failed: the matrix
  derived musl from `!!v.libc`, but glibc platforms now carry
  `libc: ["glibc"]`. Detect musl via `(v.libc||[]).includes('musl')`.
- musl builds wrote to prebuilds/linux-x64 (no suffix) because the
  Makefile's `ldd --version` parsing didn't detect musl in node:20-alpine,
  while the JS loader correctly looked in linux-x64-musl. Detect musl via
  the presence of /lib/ld-musl-*, make PLATFORM_KEY overridable, and pin it
  from the CI matrix (make build PLATFORM_KEY=<platform>).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: benasher44

.github/workflows/native-build.yml

@@ -23,7 +23,7 @@ jobs:
               platform: key,
               runner: v.runner,
               container: v.container || '',
-              musl: !!v.libc,
+              musl: (v.libc || []).includes('musl'),
             }));
             console.log(JSON.stringify(matrix));
           ")
@@ -57,7 +57,7 @@ jobs:
 
       - name: Build native addon
         working-directory: native
-        run: make build
+        run: make build PLATFORM_KEY=${{ matrix.platform }}
 
       - name: Build TypeScript
         working-directory: native

.github/workflows/native-ci.yml

@@ -35,7 +35,7 @@ jobs:
               platform: key,
               runner: v.runner,
               container: v.container || '',
-              musl: !!v.libc,
+              musl: (v.libc || []).includes('musl'),
             }));
             console.log(JSON.stringify(matrix));
           ")

native/Makefile

@@ -22,14 +22,17 @@ else
 NODE_ARCH := $(ARCH)
 endif
 
-MUSL := $(shell ldd --version 2>&1 | grep -c musl 2>/dev/null || echo 0)
-ifeq ($(MUSL),1)
+# Detect musl libc (Alpine) by the presence of its dynamic loader.
+# More reliable than parsing `ldd --version`, which varies across distros.
+ifneq ($(wildcard /lib/ld-musl-*),)
 PLATFORM_SUFFIX := -musl
 else
 PLATFORM_SUFFIX :=
 endif
 
-PLATFORM_KEY := $(PLATFORM)-$(NODE_ARCH)$(PLATFORM_SUFFIX)
+# PLATFORM_KEY can be overridden (e.g. `make build PLATFORM_KEY=linux-x64-musl`)
+# so CI can pin the output dir to the known target instead of relying on detection.
+PLATFORM_KEY ?= $(PLATFORM)-$(NODE_ARCH)$(PLATFORM_SUFFIX)
 CACHE_DIR := .cache/$(PLATFORM_KEY)
 LIBPG_QUERY_DIR := $(CACHE_DIR)/libpg_query
 

pnpm-workspace.yaml

@@ -1,7 +1,6 @@
 packages:
   - 'parser'
   - 'full'
-  - 'native'
   - 'versions/18'
   - 'versions/17' 
   - 'versions/16'