Fix detection of native and ignoreNative option

Author: tryone144

action.yml

@@ -5,6 +5,10 @@ inputs:
     required: false
     default: ""
     description: "Packages to ignore, a JSON object or array"
+  ignoreNative:
+    required: false
+    default: false
+    description: "Ignore implicit build step for node-gyp bindings"
 runs:
   using: "node24"
   main: "dist/index.js"

dist/index.js

@@ -1,6 +1,6 @@
 {
   "name": "npm-no-scripts",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Verify NPM dependencies don't have lifecycle scripts",
   "homepage": "https://github.com/cryptool-org/npm-no-scripts#readme",
   "bugs": {

package-lock.json

@@ -4,12 +4,13 @@ import { depth as traverse } from "treeverse"
 
 import { analyze } from "./analyze"
 import { shouldIgnoreFinding, shouldIgnorePackage } from "./ignore"
-import { getIgnoreList } from "./input"
+import { getIgnoreList, getIgnoreNative } from "./input"
 
 async function run() {
   try {
     // Get list of packages to ignore from the input
     const ignoreList = getIgnoreList()
+    const shouldIgnoreNative = getIgnoreNative()
 
     // Walk the node_modules/ folder to build a list of all installed dependencies
     core.debug("Load installed packages in node_modules/")
@@ -31,6 +32,7 @@ async function run() {
     // Aggregate all findings (and respect ignored packages)
     const ignored = new Set()
     const errors = []
+
     for (const [name, info] of results.entries()) {
       if (shouldIgnorePackage(ignoreList, info.node.target)) {
         ignored.add(name)
@@ -41,6 +43,11 @@ async function run() {
           ignored.add(name)
           continue
         }
+        if (f.type === "NATIVE_BINDINGS" && shouldIgnoreNative) {
+          ignored.add(name)
+          continue
+        }
+
         const version = info.node.target.version
         if (f.type === "NATIVE_BINDINGS") {
           errors.push(

package.json

@@ -60,7 +60,13 @@ function validateIgnoreOptions(item, opts) {
 
   if (native === true || native === "true" || native === 1) {
     options.native = true
-  } else if (native === false || native === "false" || native === 0) {
+  } else if (
+    native === null ||
+    native === undefined ||
+    native === false ||
+    native === "false" ||
+    native === 0
+  ) {
     // these will also be false, as is the default
   } else if (typeof native !== "boolean") {
     core.warning(`Invalid format for '${item}.native': Must be 'boolean'`)
@@ -102,3 +108,12 @@ export function getIgnoreList() {
 
   return ignores
 }
+
+export function getIgnoreNative() {
+  try {
+    return core.getBooleanInput("ignoreNative", { required: false })
+  } catch (err) {
+    core.warning(err)
+    return false
+  }
+}