Skip to content

fix: replace deleted tyler-smith/go-bip39 with fbsobreira/go-bip39#170

Merged
fbsobreira merged 2 commits intomasterfrom
fix/replace-deleted-bip39-dependency
Dec 3, 2025
Merged

fix: replace deleted tyler-smith/go-bip39 with fbsobreira/go-bip39#170
fbsobreira merged 2 commits intomasterfrom
fix/replace-deleted-bip39-dependency

Conversation

@fbsobreira
Copy link
Owner

@fbsobreira fbsobreira commented Dec 2, 2025

Description

The upstream repository github.com/tyler-smith/go-bip39 has been deleted and is no longer accessible. This PR migrates the dependency to github.com/fbsobreira/go-bip39, a maintained fork created from a cached copy of the original library.

Additionally, this PR fixes bugs in the pre-commit hook that were causing false positives during commits.

Fixes #169

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Ticket

N/A

How Has This Been Tested?

  • Manual tests (please describe below)

  • Verified go mod tidy runs without errors

  • Verified imports resolve correctly

  • Tested mnemonic generation and derivation functions

  • Confirmed pre-commit hook works correctly with staged go.mod/go.sum files

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

Changes

  • cmd/subcommands/keys.go: Updated import from tyler-smith/go-bip39 to fbsobreira/go-bip39
  • pkg/keys/mnemonic.go: Updated import from tyler-smith/go-bip39 to fbsobreira/go-bip39
  • pkg/mnemonic/mnemonic.go: Updated import from tyler-smith/go-bip39 to fbsobreira/go-bip39
  • go.mod/go.sum: Replaced dependency and updated golang.org/x/* packages
  • .githooks/pre-commit: Fixed bug where hook incorrectly failed on staged go.mod/go.sum files; fixed linter to run per-package instead of on mixed files

  The upstream repository github.com/tyler-smith/go-bip39 has been deleted.
  This change migrates to github.com/fbsobreira/go-bip39 v1.2.0, a maintained fork created from a cached copy of the original library.

  Changes:
  - Replace go-bip39 import in cmd/subcommands/keys.go
  - Replace go-bip39 import in pkg/keys/mnemonic.go
  - Replace go-bip39 import in pkg/mnemonic/mnemonic.go
  - Update go.mod and go.sum with new dependency
  - Update golang.org/x dependencies to latest versions
Copy link
Collaborator

@cryptoguyinza cryptoguyinza left a comment

Choose a reason for hiding this comment

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

Looks good! Lets move forward.

@fbsobreira fbsobreira merged commit 56139a4 into master Dec 3, 2025
3 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.

'https://github.com/tyler-smith/go-bip39/' not found

2 participants