Conversation
… management and dependency handling This commit adds Rust build artifacts to .gitignore, updates the import path in next-env.d.ts for better type resolution, and introduces new scripts in package.json for cleaning Rust build artifacts and pre-build processes. Additionally, it includes a new dependency '@fasttify/liquid-forge-native' in next.config.ts and updates the files section in the liquid-forge-native package.json to include '*.node'.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on February 27
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| "type-check": "tsc --noEmit", | ||
| "type-check:fast": "tsc --noEmit --skipLibCheck", | ||
| "clean:rust": "rm -rf packages/liquid-forge-native/target packages/liquid-forge-native/src packages/liquid-forge-native/benches packages/liquid-forge-native/.cargo", | ||
| "prebuild": "pnpm run clean:rust", |
There was a problem hiding this comment.
Prebuild script deletes source code instead of artifacts
High Severity
The clean:rust script removes packages/liquid-forge-native/src, benches, and .cargo directories which contain actual Rust source code and configuration, not build artifacts. The src/ directory contains the main library implementation (lib.rs, filters/text.rs, filters/html.rs with hundreds of lines of code). Since prebuild runs clean:rust before every pnpm build, this would delete all source code and break the build. Only target/ (the build output directory) is appropriate to clean.
1 participant
Note
Introduces native module support and Rust build hygiene across the repo.
@fasttify/liquid-forge-nativetoserverExternalPackagesinnext.config.ts.gitignoreclean:rust,prebuild, andcheck-build-sizescripts in rootpackage.json*.nodebinaries inpackages/liquid-forge-native/package.jsonfilesnext-env.d.tsto reference./.next/types/routes.d.tsWritten by Cursor Bugbot for commit 84794df. This will update automatically on new commits. Configure here.