Skip to content

refactor(js): consolidate public-key-only identity creation#49

Merged
sachaservan merged 1 commit intomainfrom
refactor/codesmell
Jan 29, 2026
Merged

refactor(js): consolidate public-key-only identity creation#49
sachaservan merged 1 commit intomainfrom
refactor/codesmell

Conversation

@sachaservan
Copy link
Member

@sachaservan sachaservan commented Jan 29, 2026

Summary by cubic

Consolid BFS public-key-only identity creation into a single helper to NB remove duplicate logic and keep behavior consistent. No functional changes; still uses a placeholder private key when only a public key is provided.

  • Refactors
    • Added private static Identity.fromPublicKeyBytes(...) using the default cipher suite and a placeholder private key.
    • Updated existing factory methods to delegate to the new helper, centralizing validation and deserialization.

Written for commit 4e4b4ee. Summary will update on new commits.

Copy link

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

Choose a reason for hiding this comment

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

No issues found across 1 file

@sachaservan sachaservan merged commit d603cf6 into main Jan 29, 2026
4 checks passed
@sachaservan sachaservan deleted the refactor/codesmell branch January 29, 2026 01:41
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.

1 participant