Skip to content

Conversation

rudyyazdi
Copy link
Contributor

Summary

  • set up GitHub Actions workflow running npm ci and npm test

Testing

  • npm test (fails: No versions available for napi)

https://chatgpt.com/codex/tasks/task_e_686b5eb3eb1c8324938bb96e8779798e

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@rudyyazdi rudyyazdi merged commit ff27092 into main Jul 7, 2025
2 checks passed
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bug: Cross-Platform Build Script Fails

The build script in package.json hardcodes the Linux shared library extension (.so) and assumes Unix library naming conventions. This causes the cp command to fail on non-Linux platforms (macOS and Windows) because it cannot find the expected file. On macOS, Cargo produces .dylib files, and on Windows, it produces .dll files without the lib prefix. This prevents local builds and testing on these platforms.

package.json#L6-L7

"scripts": {
"build": "cargo build --release && cp target/release/libjson_stream_js.so index.node",

Fix in CursorFix in Web


Was this report helpful? Give feedback by reacting with 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant