Skip to content

chore: stack upgrade#70

Merged
chaitanyapotti merged 10 commits intomasterfrom
chore/stack-upgrade
Mar 9, 2026
Merged

chore: stack upgrade#70
chaitanyapotti merged 10 commits intomasterfrom
chore/stack-upgrade

Conversation

@hieu-w
Copy link
Copy Markdown
Contributor

@hieu-w hieu-w commented Mar 5, 2026

Jira Link

Description

Stack upgrade: Node 24, latest deps, remove elliptic/buffer/bn.js.

  • Node & runtime: .nvmrc>=24.x; package.json engines node >= 22.x; Dockerfiles and CI workflows use Node 24.
  • Dependencies: Removed elliptic, bn.js, nodemon; added @toruslabs/metadata-helpers; upgraded Sentry to v10, eccrypto to v7, and other deps to latest.
  • Crypto: Replaced direct use of elliptic, Buffer, and bn.js with @toruslabs/metadata-helpers and native APIs in src/utils/index.ts (signature verification via coordsToPublicKey, secp256k1.verify, compact sig first 64 bytes) and src/routes/metadata.ts (nonce generation, batch size via utf8ToBytes). Tests use bytesToBigInt, encodeMessageBytes, and BigInt for timestamps.
  • Other: Redis get casts to string in socket/lock/metadata routes; ESLint import/no-extraneous-dependencies relaxed for test/config files.

How has this been tested?

  • npm run test:ci (Docker Compose + vitest): all 18 tests pass.

Screenshots (if appropriate)

N/A — backend only.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • My code follows the code style of this project. (run lint)
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Note

Medium Risk
Upgrades the runtime/tooling baseline (Node 22→24) for CI and production images, which can surface dependency/runtime incompatibilities at build or deploy time. Other changes are workflow/linters/ownership and are low risk.

Overview
Stack/tooling upgrade: CI workflows (any.yml, develop.yml, master.yml) now run on Node 24.x and use newer actions/checkout/actions/setup-node versions, and Docker images (Dockerfile, Dockerfile.dev) switch to node:24-alpine; .nvmrc is updated to >=24.x.

Repo hygiene: Adds .github/CODEOWNERS to enforce review ownership, removes nodemon.json, and updates ESLint to allow import/no-extraneous-dependencies for test/config paths.

Written by Cursor Bugbot for commit 753994e. This will update automatically on new commits. Configure here.

Comment thread .nvmrc
@hieu-w
Copy link
Copy Markdown
Contributor Author

hieu-w commented Mar 6, 2026

@hieu-w hieu-w requested a review from chaitanyapotti March 6, 2026 16:44
Comment thread .github/workflows/any.yml Outdated
Comment thread src/routes/metadata.ts Outdated
Copy link
Copy Markdown

@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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Comment thread .github/workflows/master.yml
@hieu-w hieu-w requested a review from chaitanyapotti March 9, 2026 09:15
@chaitanyapotti chaitanyapotti merged commit c7acc62 into master Mar 9, 2026
4 checks passed
@chaitanyapotti chaitanyapotti deleted the chore/stack-upgrade branch March 9, 2026 09:16
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