fix: remove broken bech32 nsec/npub stub functions

DreamLab-AI Mega-Sprint · ruvnet · DreamLab-AI Mega-Sprint · commit 1072ba5dc3f4 · 2026-05-12T14:20:14.000Z
The privateKeyToNsec, nsecToPrivateKey, and publicKeyToNpub functions
emitted fake nsec_&lt;hex&gt; / npub_&lt;hex&gt; strings that are not valid NIP-19
bech32 encoding. These stubs could confuse users into thinking they
have a valid key backup, and any interoperability with other Nostr
clients would silently fail.

The functions are not imported or called anywhere in the codebase, so
they are removed entirely rather than replaced with throwing stubs.
Proper NIP-19 bech32 support can be added when needed via a bech32
library.

Co-Authored-By: claude-flow &lt;ruv@ruv.net&gt;
diff --git a/src/crypto.js b/src/crypto.js
@@ -161,35 +161,3 @@ export function isValidPublicKey (publicKey) {
   if (publicKey.length !== 64) return false;
   return /^[0-9a-fA-F]{64}$/.test(publicKey);
 }
-
-/**
- * Convert private key to nsec format (Bech32)
- * @param {string} privateKeyHex - 64-char hex private key
- * @returns {string} nsec1... format
- */
-export function privateKeyToNsec (privateKeyHex) {
-  // TODO: Implement bech32 encoding
-  // For now, return hex with nsec prefix as placeholder
-  return `nsec_${privateKeyHex}`;
-}
-
-/**
- * Convert nsec to private key hex
- * @param {string} nsec - nsec1... format
- * @returns {string} 64-char hex private key
- */
-export function nsecToPrivateKey (nsec) {
-  // TODO: Implement bech32 decoding
-  // For now, strip nsec_ prefix as placeholder
-  return nsec.replace(/^nsec_/, '');
-}
-
-/**
- * Convert public key to npub format (Bech32)
- * @param {string} publicKeyHex - 64-char hex public key
- * @returns {string} npub1... format
- */
-export function publicKeyToNpub (publicKeyHex) {
-  // TODO: Implement bech32 encoding
-  return `npub_${publicKeyHex}`;
-}
