feat: fork bcryptjs as @opensourceframework/bcryptjs#110
feat: fork bcryptjs as @opensourceframework/bcryptjs#110riceharvest wants to merge 3 commits intomainfrom
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
8ec8ff2 to
eed8395
Compare
CI Feedback 🧐A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
- Adds new package under packages/bcryptjs - Preserves original API - Includes minimal smoke test Fixes #108
c35ada4 to
75b1b59
Compare
The pnpm-lock.yaml was pinning turbo-linux-64@1.13.4 (Turbo v1) while turbo@2.8.15+ was specified in devDependencies. This caused all CI jobs to fail with 'Unknown key tasks' since Turbo v1 uses 'pipeline' instead of 'tasks' in turbo.json. Regenerated lockfile with pnpm store prune + force install.
Forks the unmaintained
bcryptjspackage into@opensourceframework/bcryptjsto provide a maintained drop-in replacement.## ChangesTesting:
node test/bcrypt.test.js– hashing works.Fixes #108