Skip to content

fix: remove deprecated stellar-sdk package and update imports#73

Merged
daiwikmh merged 1 commit into
Stellar-Tools:mainfrom
jayantvaibhavspj:fix/remove-deprecated-stellar-sdk
Apr 30, 2026
Merged

fix: remove deprecated stellar-sdk package and update imports#73
daiwikmh merged 1 commit into
Stellar-Tools:mainfrom
jayantvaibhavspj:fix/remove-deprecated-stellar-sdk

Conversation

@jayantvaibhavspj

@jayantvaibhavspj jayantvaibhavspj commented Apr 29, 2026

Copy link
Copy Markdown

Problem

The project had duplicate and conflicting Stellar SDK dependencies:

  • stellar-sdk (deprecated as of v13.3.0) - producing npm warnings
  • @stellar/stellar-sdk (current recommended) - used throughout codebase

This created inconsistency: tools/stellar.ts imported from the deprecated package while the rest of the codebase used @stellar/stellar-sdk.

Solution

✅ Removed deprecated stellar-sdk dependency from package.json
✅ Updated import in tools/stellar.ts to use @stellar/stellar-sdk
✅ Ensured 100% import consistency across the codebase

Changes

  • package.json: Removed "stellar-sdk": "^13.3.0" dependency
  • tools/stellar.ts: Changed import from stellar-sdk to @stellar/stellar-sdk

Testing

✅ All 59 unit & integration tests pass
✅ TypeScript compilation successful
✅ No breaking changes
✅ npm install no longer produces deprecation warnings

Impact

  • Quality: Eliminates deprecation warnings
  • Consistency: Single source of truth for Stellar SDK across codebase
  • Maintenance: Easier to track and update dependencies
  • Best Practice: Aligns with official Stellar SDK migration recommendations

Summary by cubic

Removed deprecated stellar-sdk and standardized imports to @stellar/stellar-sdk to remove npm warnings and keep the SDK usage consistent. No functional changes.

  • Dependencies
    • Removed stellar-sdk from package.json
    • Updated tools/stellar.ts to import @stellar/stellar-sdk

Written for commit ed427b3. Summary will update on new commits. Review in cubic

- Replace deprecated 'stellar-sdk' package with '@stellar/stellar-sdk' in package.json dependencies
- Update import in tools/stellar.ts from 'stellar-sdk' to '@stellar/stellar-sdk'
- All existing tests pass with this change
- Addresses npm deprecation warning for stellar-sdk v13.3.0
- Ensures consistency with the rest of the codebase which already uses @stellar/stellar-sdk

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 3 files

@daiwikmh daiwikmh merged commit 71394bc into Stellar-Tools:main Apr 30, 2026
2 of 5 checks passed
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