Skip to content

Add Node Rust native extension detectors - #121

Merged
andrew merged 2 commits into
mainfrom
add-node-rust-detectors
Jul 31, 2026
Merged

Add Node Rust native extension detectors#121
andrew merged 2 commits into
mainfrom
add-node-rust-detectors

Conversation

@andrew

@andrew andrew commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Adds napi-rs and Neon native-extension definitions for Node projects. Detection covers their scoped npm CLI packages, Cargo.toml configuration, and napi-rs build scripts, with fixtures for positive signals and near misses.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds new Node “native_extension” tool definitions for Rust-based addons (napi-rs and Neon) to the knowledge base, along with test fixtures and detector tests to ensure these projects are classified with high confidence.

Changes:

  • Added KB definitions for napi-rs and Neon (detection signals + suggested commands + taxonomy).
  • Added testdata/ fixtures for representative Neon and napi-rs projects.
  • Extended detector tests to validate positive signals and a “near miss” scenario.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
testdata/neon-project/src/lib.rs Adds minimal Rust source fixture for Neon project detection.
testdata/neon-project/package.json Adds Neon fixture package.json including @neon-rs/cli and build scripts.
testdata/neon-project/Cargo.toml Adds Neon fixture Cargo.toml with cdylib + neon dependency.
testdata/napi-rs-project/src/lib.rs Adds minimal Rust source fixture for napi-rs project detection.
testdata/napi-rs-project/package.json Adds napi-rs fixture package.json including @napi-rs/cli and build script.
testdata/napi-rs-project/Cargo.toml Adds napi-rs fixture Cargo.toml with cdylib + napi crates/build dependency.
knowledge/node/neon.toml Introduces Neon detector definition and metadata under native_extension.
knowledge/node/napi-rs.toml Introduces napi-rs detector definition and metadata under native_extension.
detect/detect_test.go Adds tests validating both project fixtures and individual detection signals/near-misses.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread knowledge/node/napi-rs.toml Outdated
@andrew
andrew force-pushed the add-node-rust-detectors branch from 456cc28 to 16d95f8 Compare July 31, 2026 15:53
@andrew
andrew merged commit 29ec2b7 into main Jul 31, 2026
8 checks passed
@andrew
andrew deleted the add-node-rust-detectors branch July 31, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants