v0.1.6

Author: Qard

.github/workflows/CI.yml

@@ -281,9 +281,6 @@ jobs:
           architecture: ${{ matrix.settings.architecture }}
       - name: Install dependencies
         run: pnpm install
-      - name: Fix soname
-        run: pnpm run build:fix
-        shell: bash
       - uses: actions/download-artifact@v4
         with:
           name: bindings-${{ matrix.settings.target }}
@@ -379,6 +376,7 @@ jobs:
 
             corepack disable
             npm i -gf pnpm
+            node fix-python-soname.js
             pnpm install --prefer-offline
             pnpm test
 

fix-python-soname.js

@@ -21,76 +21,43 @@ const arch = os.arch()
 // Function to detect if this is a development install vs dependency install
 function isDevInstall() {
   const env = process.env
-  
+
   // Method 1: Check if INIT_CWD and PWD are the same (local dev install)
   if (env.INIT_CWD && env.PWD) {
     if (env.INIT_CWD === env.PWD || env.INIT_CWD.indexOf(env.PWD) === 0) {
       return true
     }
   }
-  
+
   // Method 2: Check for .git folder existence (dev environment)
   if (fs.existsSync(path.join(__dirname, '.git'))) {
     return true
   }
-  
+
   // Method 3: Check if we're in production mode
   if (env.NODE_ENV === 'production' || env.npm_config_production) {
     return false
   }
-  
+
   return false
 }
 
-// Function to find the correct .node file for this platform
-function findNodeFile() {
-  // Only run on Linux - other platforms don't need soname fixing
-  if (platform !== 'linux') return
-
-  // Map Node.js arch to napi-rs target
-  const archMap = {
-    'x64': 'x86_64-unknown-linux-gnu',
-    'arm64': 'aarch64-unknown-linux-gnu',
-  }
-
-  const target = archMap[arch]
-  if (!target) return
-
-  // Try to find the .node file with various naming patterns
-  const possiblePaths = [
-    // Specific platform builds
-    path.join(__dirname, `python-node.${target}.node`),
-    path.join(__dirname, `index.${target}.node`),
-    path.join(__dirname, `npm/${target}/python-node.${target}.node`),
-    // Generic .node files (common during testing)
-    path.join(__dirname, 'python-node.node'),
-    path.join(__dirname, 'index.node'),
-    // Look for any .node file in current directory
-    ...fs.readdirSync(__dirname)
-      .filter(f => f.endsWith('.node') && !f.includes('node_modules'))
-      .map(f => path.join(__dirname, f))
-  ]
-
-  for (const nodePath of possiblePaths) {
-    if (fs.existsSync(nodePath)) {
-      return nodePath
-    }
-  }
-
-  // Return undefined if no .node file found - this is expected during development
-  return undefined
+// Only patch soname on Linux
+if (platform !== 'linux') {
+  console.log(`No need to fix soname on platform: ${platform}`)
+  process.exit(0)
 }
 
 // Get the node file path
-const nodeFilePath = findNodeFile()
-if (!nodeFilePath) {
+const nodeFilePath = path.join(__dirname, `python-node.linux-${arch}-gnu.node`)
+if (!fs.existsSync(nodeFilePath)) {
   if (isDevInstall()) {
     // No .node file found during dev install - this is expected, skip silently
-    console.log('No .node file found during development install, skipping soname fix')
+    console.log(`${nodeFilePath} not found during development install, skipping soname fix`)
     process.exit(0)
   } else {
     // No .node file found when installed as dependency - this is an error
-    console.error('Error: Could not find *.node file to fix soname')
+    console.error(`Error: Could not find "${nodeFilePath}" to fix soname`)
     process.exit(1)
   }
 }

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@platformatic/python-node",
   "description": "Run ASGI-compatible Python apps in Node.js",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "main": "index.js",
   "types": "index.d.ts",
   "napi": {
@@ -30,10 +30,8 @@
     "artifacts": "napi artifacts",
     "build": "npm run build:debug -- --release",
     "build:debug": "napi build --platform --features napi-support",
-    "build:fix": "node fix-python-soname.js",
     "build:wasm": "rustup target add wasm32-wasip1 && cargo build --package fix-python-soname --target wasm32-wasip1 --release && cp target/wasm32-wasip1/release/fix-python-soname.wasm .",
     "prepublishOnly": "npm run build:wasm && napi prepublish -t npm",
-    "postinstall": "npm run build:fix",
     "lint": "oxlint",
     "test": "node --test test/**.test.mjs",
     "universal": "napi universal",