Release v1.1.5 - Remove debug and signing logs

- Disable debug() logging function in background scripts
- Remove backgroundLogNip07Action calls for NIP-07 operations
- Remove backgroundLogPermissionStored calls for permission events
- Clean up unused imports and result variables
- Simplify switch statement returns in processNip07Request

Files modified:
- package.json (version bump)
- projects/chrome/src/background-common.ts
- projects/chrome/src/background.ts
- projects/firefox/src/background-common.ts
- projects/firefox/src/background.ts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

package.json

@@ -1,12 +1,12 @@
 {
   "name": "plebeian-signer",
-  "version": "v1.1.3",
+  "version": "v1.1.5",
   "custom": {
     "chrome": {
-      "version": "v1.1.3"
+      "version": "v1.1.5"
     },
     "firefox": {
-      "version": "v1.1.3"
+      "version": "v1.1.5"
     }
   },
   "scripts": {

projects/chrome/public/manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "name": "Plebeian Signer - Nostr Identity Manager & Signer",
   "description": "Manage and switch between multiple identities while interacting with Nostr apps",
-  "version": "1.1.3",
+  "version": "1.1.5",
   "homepage_url": "https://github.com/PlebeianApp/plebeian-signer",
   "options_page": "options.html",
   "permissions": [

projects/chrome/src/app/components/edit-identity/ncryptsec/ncryptsec.component.html

@@ -6,16 +6,26 @@
   <span>Get ncryptsec</span>
 </div>
 
-<p class="description">
-  Enter a password to encrypt your private key. The resulting ncryptsec can be
-  used to securely backup or transfer your key.
-</p>
+<!-- QR Code (shown after generation) -->
+@if (ncryptsec) {
+<div class="qr-container">
+  <button
+    type="button"
+    class="qr-button"
+    title="Copy to clipboard"
+    (click)="copyToClipboard(ncryptsec); toast.show('Copied to clipboard')"
+  >
+    <img [src]="ncryptsecQr" alt="ncryptsec QR code" class="qr-code" />
+  </button>
+</div>
+}
 
 <!-- PASSWORD INPUT -->
 <div class="password-section">
   <label for="ncryptsecPasswordInput">Password</label>
   <div class="input-group sam-mt-h">
     <input
+      #passwordInput
       id="ncryptsecPasswordInput"
       type="password"
       class="form-control"
@@ -41,35 +51,10 @@
   }
 </button>
 
-<!-- NCRYPTSEC OUTPUT -->
-@if (ncryptsec) {
-<div class="result-section">
-  <!-- QR Code -->
-  <div class="qr-container">
-    <img [src]="ncryptsecQr" alt="ncryptsec QR code" class="qr-code" />
-  </div>
 
-  <!-- ncryptsec text -->
-  <div class="ncryptsec-container">
-    <input
-      type="text"
-      class="form-control ncryptsec-output"
-      [value]="ncryptsec"
-      readonly
-      (click)="copyToClipboard(ncryptsec); toast.show('Copied to clipboard')"
-      title="Click to copy"
-    />
-    <button
-      class="btn btn-outline-secondary"
-      type="button"
-      (click)="copyToClipboard(ncryptsec); toast.show('Copied to clipboard')"
-    >
-      <i class="bi bi-copy"></i> Copy
-    </button>
-  </div>
-
-  <p class="hint">Tap the text or button to copy to clipboard</p>
-</div>
-}
+<p class="description">
+  Enter a password to encrypt your private key. The resulting ncryptsec can be
+  used to securely backup or transfer your key.
+</p>
 
 <lib-toast #toast [bottom]="16"></lib-toast>

projects/chrome/src/app/components/edit-identity/ncryptsec/ncryptsec.component.scss

@@ -38,52 +38,33 @@
   margin-bottom: var(--size);
 }
 
-.result-section {
-  display: flex;
-  flex-direction: column;
-  align-items: center;
-  gap: var(--size);
-  margin-top: var(--size);
-}
-
 .qr-container {
-  background: white;
-  padding: var(--size);
-  border-radius: 8px;
   display: flex;
   justify-content: center;
   align-items: center;
+  margin-bottom: var(--size);
+}
+
+.qr-button {
+  background: white;
+  padding: var(--size);
+  border-radius: 8px;
+  border: none;
+  cursor: pointer;
+  transition: transform 0.15s ease, box-shadow 0.15s ease;
+
+  &:hover {
+    transform: scale(1.02);
+    box-shadow: 0 4px 12px rgba(0, 0, 0, 0.15);
+  }
+
+  &:active {
+    transform: scale(0.98);
+  }
 }
 
 .qr-code {
   width: 250px;
   height: 250px;
-}
-
-.ncryptsec-container {
-  width: 100%;
-  display: flex;
-  flex-direction: column;
-  gap: var(--size-h);
-
-  .ncryptsec-output {
-    font-family: monospace;
-    font-size: 0.75rem;
-    word-break: break-all;
-    cursor: pointer;
-
-    &:hover {
-      background-color: var(--bs-gray-100);
-    }
-  }
-
-  button {
-    width: 100%;
-  }
-}
-
-.hint {
-  color: var(--text-muted);
-  font-size: 0.8rem;
-  text-align: center;
+  display: block;
 }

projects/chrome/src/app/components/edit-identity/ncryptsec/ncryptsec.component.ts

@@ -1,4 +1,11 @@
-import { Component, inject, OnInit } from '@angular/core';
+import {
+  AfterViewInit,
+  Component,
+  ElementRef,
+  inject,
+  OnInit,
+  ViewChild,
+} from '@angular/core';
 import { ActivatedRoute } from '@angular/router';
 import {
   IconButtonComponent,
@@ -16,7 +23,12 @@ import * as QRCode from 'qrcode';
   templateUrl: './ncryptsec.component.html',
   styleUrl: './ncryptsec.component.scss',
 })
-export class NcryptsecComponent extends NavComponent implements OnInit {
+export class NcryptsecComponent
+  extends NavComponent
+  implements OnInit, AfterViewInit
+{
+  @ViewChild('passwordInput') passwordInput!: ElementRef<HTMLInputElement>;
+
   privkeyHex = '';
   ncryptsecPassword = '';
   ncryptsec = '';
@@ -35,6 +47,10 @@ export class NcryptsecComponent extends NavComponent implements OnInit {
     this.#initialize(identityId);
   }
 
+  ngAfterViewInit(): void {
+    this.passwordInput.nativeElement.focus();
+  }
+
   async generateNcryptsec() {
     if (!this.privkeyHex || !this.ncryptsecPassword) {
       return;

projects/chrome/src/background-common.ts

@@ -40,10 +40,13 @@ export interface UnlockResponseMessage {
   error?: string;
 }
 
-export const debug = function (message: any) {
-  const dateString = new Date().toISOString();
-  console.log(`[Plebeian Signer - ${dateString}]: ${JSON.stringify(message)}`);
-};
+// Debug logging disabled - uncomment for development
+// export const debug = function (message: any) {
+//   const dateString = new Date().toISOString();
+//   console.log(`[Plebeian Signer - ${dateString}]: ${JSON.stringify(message)}`);
+// };
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-empty-function
+export const debug = function (_message: any) {};
 
 export type PromptResponse =
   | 'reject'

projects/chrome/src/background.ts

@@ -1,7 +1,5 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import {
-  backgroundLogNip07Action,
-  backgroundLogPermissionStored,
   NostrHelper,
   NwcClient,
   NwcConnection_DECRYPTED,
@@ -441,7 +439,6 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
           policy,
           req.params?.kind
         );
-        await backgroundLogPermissionStored(req.host, req.method, policy, req.params?.kind);
       } else if (response === 'approve-all') {
         // P2: Store permission for ALL kinds/uses of this method from this host
         await storePermission(
@@ -452,8 +449,6 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
           'allow',
           undefined // undefined kind = allow all kinds for signEvent
         );
-        await backgroundLogPermissionStored(req.host, req.method, 'allow', undefined);
-        debug(`Stored approve-all permission for ${req.method} from ${req.host}`);
       } else if (response === 'reject-all') {
         // P2: Store deny permission for ALL uses of this method from this host
         await storePermission(
@@ -464,15 +459,9 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
           'deny',
           undefined
         );
-        await backgroundLogPermissionStored(req.host, req.method, 'deny', undefined);
-        debug(`Stored reject-all permission for ${req.method} from ${req.host}`);
       }
 
       if (['reject', 'reject-once', 'reject-all'].includes(response)) {
-        await backgroundLogNip07Action(req.method, req.host, false, false, {
-          kind: req.params?.kind,
-          peerPubkey: req.params?.peerPubkey,
-        });
         throw new Error('Permission denied');
       }
     } else {
@@ -481,71 +470,47 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
   }
 
   const relays: Relays = {};
-  let result: any;
 
   switch (req.method) {
     case 'getPublicKey':
-      result = NostrHelper.pubkeyFromPrivkey(currentIdentity.privkey);
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove);
-      return result;
+      return NostrHelper.pubkeyFromPrivkey(currentIdentity.privkey);
 
     case 'signEvent':
-      result = signEvent(req.params, currentIdentity.privkey);
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        kind: req.params?.kind,
-      });
-      return result;
+      return signEvent(req.params, currentIdentity.privkey);
 
     case 'getRelays':
       browserSessionData.relays.forEach((x) => {
         relays[x.url] = { read: x.read, write: x.write };
       });
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove);
       return relays;
 
     case 'nip04.encrypt':
-      result = await nip04Encrypt(
+      return await nip04Encrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.plaintext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     case 'nip44.encrypt':
-      result = await nip44Encrypt(
+      return await nip44Encrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.plaintext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     case 'nip04.decrypt':
-      result = await nip04Decrypt(
+      return await nip04Decrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.ciphertext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     case 'nip44.decrypt':
-      result = await nip44Decrypt(
+      return await nip44Decrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.ciphertext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     default:
       throw new Error(`Not supported request method '${req.method}'.`);
@@ -625,7 +590,6 @@ async function processWeblnRequest(req: BackgroundRequestMessage): Promise<any>
         method,
         policy
       );
-      await backgroundLogPermissionStored(req.host, method, policy);
     } else if (response === 'approve-all' && method !== 'webln.sendPayment' && method !== 'webln.keysend') {
       // P2: Store permission for all uses of this WebLN method
       await storePermission(
@@ -635,8 +599,6 @@ async function processWeblnRequest(req: BackgroundRequestMessage): Promise<any>
         method,
         'allow'
       );
-      await backgroundLogPermissionStored(req.host, method, 'allow');
-      debug(`Stored approve-all permission for ${method} from ${req.host}`);
     }
 
     if (['reject', 'reject-once', 'reject-all'].includes(response)) {

projects/firefox/public/manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "name": "Plebeian Signer",
   "description": "Nostr Identity Manager & Signer",
-  "version": "1.1.3",
+  "version": "1.1.5",
   "homepage_url": "https://github.com/PlebeianApp/plebeian-signer",
   "options_page": "options.html",
   "permissions": [

projects/firefox/src/app/components/edit-identity/ncryptsec/ncryptsec.component.html

@@ -6,16 +6,26 @@
   <span>Get ncryptsec</span>
 </div>
 
-<p class="description">
-  Enter a password to encrypt your private key. The resulting ncryptsec can be
-  used to securely backup or transfer your key.
-</p>
+<!-- QR Code (shown after generation) -->
+@if (ncryptsec) {
+<div class="qr-container">
+  <button
+    type="button"
+    class="qr-button"
+    title="Copy to clipboard"
+    (click)="copyToClipboard(ncryptsec); toast.show('Copied to clipboard')"
+  >
+    <img [src]="ncryptsecQr" alt="ncryptsec QR code" class="qr-code" />
+  </button>
+</div>
+}
 
 <!-- PASSWORD INPUT -->
 <div class="password-section">
   <label for="ncryptsecPasswordInput">Password</label>
   <div class="input-group sam-mt-h">
     <input
+      #passwordInput
       id="ncryptsecPasswordInput"
       type="password"
       class="form-control"
@@ -41,35 +51,10 @@
   }
 </button>
 
-<!-- NCRYPTSEC OUTPUT -->
-@if (ncryptsec) {
-<div class="result-section">
-  <!-- QR Code -->
-  <div class="qr-container">
-    <img [src]="ncryptsecQr" alt="ncryptsec QR code" class="qr-code" />
-  </div>
 
-  <!-- ncryptsec text -->
-  <div class="ncryptsec-container">
-    <input
-      type="text"
-      class="form-control ncryptsec-output"
-      [value]="ncryptsec"
-      readonly
-      (click)="copyToClipboard(ncryptsec); toast.show('Copied to clipboard')"
-      title="Click to copy"
-    />
-    <button
-      class="btn btn-outline-secondary"
-      type="button"
-      (click)="copyToClipboard(ncryptsec); toast.show('Copied to clipboard')"
-    >
-      <i class="bi bi-copy"></i> Copy
-    </button>
-  </div>
-
-  <p class="hint">Tap the text or button to copy to clipboard</p>
-</div>
-}
+<p class="description">
+  Enter a password to encrypt your private key. The resulting ncryptsec can be
+  used to securely backup or transfer your key.
+</p>
 
 <lib-toast #toast [bottom]="16"></lib-toast>

projects/firefox/src/app/components/edit-identity/ncryptsec/ncryptsec.component.scss

@@ -38,52 +38,33 @@
   margin-bottom: var(--size);
 }
 
-.result-section {
-  display: flex;
-  flex-direction: column;
-  align-items: center;
-  gap: var(--size);
-  margin-top: var(--size);
-}
-
 .qr-container {
-  background: white;
-  padding: var(--size);
-  border-radius: 8px;
   display: flex;
   justify-content: center;
   align-items: center;
+  margin-bottom: var(--size);
+}
+
+.qr-button {
+  background: white;
+  padding: var(--size);
+  border-radius: 8px;
+  border: none;
+  cursor: pointer;
+  transition: transform 0.15s ease, box-shadow 0.15s ease;
+
+  &:hover {
+    transform: scale(1.02);
+    box-shadow: 0 4px 12px rgba(0, 0, 0, 0.15);
+  }
+
+  &:active {
+    transform: scale(0.98);
+  }
 }
 
 .qr-code {
   width: 250px;
   height: 250px;
-}
-
-.ncryptsec-container {
-  width: 100%;
-  display: flex;
-  flex-direction: column;
-  gap: var(--size-h);
-
-  .ncryptsec-output {
-    font-family: monospace;
-    font-size: 0.75rem;
-    word-break: break-all;
-    cursor: pointer;
-
-    &:hover {
-      background-color: var(--bs-gray-100);
-    }
-  }
-
-  button {
-    width: 100%;
-  }
-}
-
-.hint {
-  color: var(--text-muted);
-  font-size: 0.8rem;
-  text-align: center;
+  display: block;
 }

projects/firefox/src/app/components/edit-identity/ncryptsec/ncryptsec.component.ts

@@ -1,4 +1,11 @@
-import { Component, inject, OnInit } from '@angular/core';
+import {
+  AfterViewInit,
+  Component,
+  ElementRef,
+  inject,
+  OnInit,
+  ViewChild,
+} from '@angular/core';
 import { ActivatedRoute } from '@angular/router';
 import {
   IconButtonComponent,
@@ -16,7 +23,12 @@ import * as QRCode from 'qrcode';
   templateUrl: './ncryptsec.component.html',
   styleUrl: './ncryptsec.component.scss',
 })
-export class NcryptsecComponent extends NavComponent implements OnInit {
+export class NcryptsecComponent
+  extends NavComponent
+  implements OnInit, AfterViewInit
+{
+  @ViewChild('passwordInput') passwordInput!: ElementRef<HTMLInputElement>;
+
   privkeyHex = '';
   ncryptsecPassword = '';
   ncryptsec = '';
@@ -35,6 +47,10 @@ export class NcryptsecComponent extends NavComponent implements OnInit {
     this.#initialize(identityId);
   }
 
+  ngAfterViewInit(): void {
+    this.passwordInput.nativeElement.focus();
+  }
+
   async generateNcryptsec() {
     if (!this.privkeyHex || !this.ncryptsecPassword) {
       return;

projects/firefox/src/background-common.ts

@@ -41,10 +41,13 @@ export interface UnlockResponseMessage {
   error?: string;
 }
 
-export const debug = function (message: any) {
-  const dateString = new Date().toISOString();
-  console.log(`[Plebeian Signer - ${dateString}]: ${JSON.stringify(message)}`);
-};
+// Debug logging disabled - uncomment for development
+// export const debug = function (message: any) {
+//   const dateString = new Date().toISOString();
+//   console.log(`[Plebeian Signer - ${dateString}]: ${JSON.stringify(message)}`);
+// };
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-empty-function
+export const debug = function (_message: any) {};
 
 export type PromptResponse =
   | 'reject'

projects/firefox/src/background.ts

@@ -1,7 +1,5 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import {
-  backgroundLogNip07Action,
-  backgroundLogPermissionStored,
   NostrHelper,
   NwcClient,
   NwcConnection_DECRYPTED,
@@ -441,7 +439,6 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
           policy,
           req.params?.kind
         );
-        await backgroundLogPermissionStored(req.host, req.method, policy, req.params?.kind);
       } else if (response === 'approve-all') {
         // P2: Store permission for ALL kinds/uses of this method from this host
         await storePermission(
@@ -452,8 +449,6 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
           'allow',
           undefined // undefined kind = allow all kinds for signEvent
         );
-        await backgroundLogPermissionStored(req.host, req.method, 'allow', undefined);
-        debug(`Stored approve-all permission for ${req.method} from ${req.host}`);
       } else if (response === 'reject-all') {
         // P2: Store deny permission for ALL uses of this method from this host
         await storePermission(
@@ -464,15 +459,9 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
           'deny',
           undefined
         );
-        await backgroundLogPermissionStored(req.host, req.method, 'deny', undefined);
-        debug(`Stored reject-all permission for ${req.method} from ${req.host}`);
       }
 
       if (['reject', 'reject-once', 'reject-all'].includes(response)) {
-        await backgroundLogNip07Action(req.method, req.host, false, false, {
-          kind: req.params?.kind,
-          peerPubkey: req.params?.peerPubkey,
-        });
         throw new Error('Permission denied');
       }
     } else {
@@ -481,71 +470,47 @@ async function processNip07Request(req: BackgroundRequestMessage): Promise<any>
   }
 
   const relays: Relays = {};
-  let result: any;
 
   switch (req.method) {
     case 'getPublicKey':
-      result = NostrHelper.pubkeyFromPrivkey(currentIdentity.privkey);
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove);
-      return result;
+      return NostrHelper.pubkeyFromPrivkey(currentIdentity.privkey);
 
     case 'signEvent':
-      result = signEvent(req.params, currentIdentity.privkey);
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        kind: req.params?.kind,
-      });
-      return result;
+      return signEvent(req.params, currentIdentity.privkey);
 
     case 'getRelays':
       browserSessionData.relays.forEach((x) => {
         relays[x.url] = { read: x.read, write: x.write };
       });
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove);
       return relays;
 
     case 'nip04.encrypt':
-      result = await nip04Encrypt(
+      return await nip04Encrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.plaintext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     case 'nip44.encrypt':
-      result = await nip44Encrypt(
+      return await nip44Encrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.plaintext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     case 'nip04.decrypt':
-      result = await nip04Decrypt(
+      return await nip04Decrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.ciphertext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     case 'nip44.decrypt':
-      result = await nip44Decrypt(
+      return await nip44Decrypt(
         currentIdentity.privkey,
         req.params.peerPubkey,
         req.params.ciphertext
       );
-      await backgroundLogNip07Action(req.method, req.host, true, recklessApprove, {
-        peerPubkey: req.params.peerPubkey,
-      });
-      return result;
 
     default:
       throw new Error(`Not supported request method '${req.method}'.`);
@@ -625,7 +590,6 @@ async function processWeblnRequest(req: BackgroundRequestMessage): Promise<any>
         method,
         policy
       );
-      await backgroundLogPermissionStored(req.host, method, policy);
     } else if (response === 'approve-all' && method !== 'webln.sendPayment' && method !== 'webln.keysend') {
       // P2: Store permission for all uses of this WebLN method
       await storePermission(
@@ -635,8 +599,6 @@ async function processWeblnRequest(req: BackgroundRequestMessage): Promise<any>
         method,
         'allow'
       );
-      await backgroundLogPermissionStored(req.host, method, 'allow');
-      debug(`Stored approve-all permission for ${method} from ${req.host}`);
     }
 
     if (['reject', 'reject-once', 'reject-all'].includes(response)) {