Skip to content

Remove unused imports from rbf-service.ts#523

Merged
jamespepper81 merged 1 commit into
devfrom
claude/cleanup-rbf-service-JKAcn
Jan 28, 2026
Merged

Remove unused imports from rbf-service.ts#523
jamespepper81 merged 1 commit into
devfrom
claude/cleanup-rbf-service-JKAcn

Conversation

@jamespepper81
Copy link
Copy Markdown
Contributor

Description

Clean up unused imports in the RBF service module. This PR removes four unused function imports from ecc-utils.ts that are no longer referenced in the codebase, along with an obsolete comment.

Type of Change

  • 🐛 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)
  • 📚 Documentation update
  • 🎨 UI/UX improvement
  • ♻️ Code refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🔒 Security improvement
  • 🧪 Test coverage improvement
  • 🔧 Build/CI improvement

Changes Made

  • Removed unused import: validateECCLibrary
  • Removed unused import: deriveAddressIndexFromAddress
  • Removed unused import: findNextUnusedAddressIndex
  • Removed obsolete comment referencing validateECCLibrary
  • Kept necessary imports: validateECCLibraryFull, estimateTransactionSize, deriveAddressIndexAndChainFromAddress, and generateCancellationAddress

Testing Performed

Automated Testing

  • Linting passes
  • No compilation errors
  • All existing tests pass

Checklist

  • My code follows the project's code style guidelines
  • I have performed a self-review of my own code
  • My changes generate no new warnings or errors

Additional Notes

This is a straightforward cleanup to improve code maintainability by removing dead imports that are not used anywhere in the rbf-service.ts module.

https://claude.ai/code/session_016Cs3ckjXRRHT6eX6JYLmXq

Remove validateECCLibrary, deriveAddressIndexFromAddress, and
findNextUnusedAddressIndex imports that were not used in this file.

https://claude.ai/code/session_016Cs3ckjXRRHT6eX6JYLmXq
@jamespepper81 jamespepper81 merged commit 3f91a2d into dev Jan 28, 2026
3 checks passed
@jamespepper81 jamespepper81 deleted the claude/cleanup-rbf-service-JKAcn branch January 28, 2026 08:30
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