Skip to content

Conversation

@coodos
Copy link
Contributor

@coodos coodos commented Dec 15, 2025

Description of change

downgrade dependency for multiformats as base58 export doesn't work somehow for newer versions

Issue Number

Type of change

  • Chore (refactoring, build scripts or anything else that isn't user-facing)

How the change has been tested

Change checklist

  • I have ensured that the CI Checks pass locally
  • I have removed any unnecessary logic
  • My code is well documented
  • I have signed my commits
  • My code follows the pattern of the application
  • I have self reviewed my code

Summary by CodeRabbit

  • Chores
    • Updated dependency version constraint to use a fixed version instead of a flexible range, improving stability and consistency across deployments.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 15, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The multiformats dependency in the signature-validator package is pinned from a caret range constraint (^13.3.2) to an exact version (13.3.2), restricting future installations to a single specified version rather than allowing minor or patch updates.

Changes

Cohort / File(s) Summary
Dependency version pinning
infrastructure/signature-validator/package.json
Changed multiformats from ^13.3.2 to 13.3.2 — exact version constraint replacing semver range

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • hotfix: signature validotr dep #613: Pins the multiformats dependency to an exact version while also updating code to consume it via static import—both changes directly address the same dependency.

Poem

🐰 A version locked tight, no more drift in sight,
The rabbit pins true, keeping builds just right!
No caret to roam, dependencies stay home,
In exact versions safe, no more need to roam.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/downgrade-multiformats-dep

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 41741b5 and 2579783.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • infrastructure/signature-validator/package.json (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coodos coodos merged commit 8a5bf70 into main Dec 15, 2025
3 of 4 checks passed
@coodos coodos deleted the chore/downgrade-multiformats-dep branch December 15, 2025 11:07
@coodos coodos restored the chore/downgrade-multiformats-dep branch December 15, 2025 11:13
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