chore: daily nip-research-agent run

Co-authored-by: PR0M3TH3AN <[email protected]>

NIP_INVENTORY.md

@@ -5,12 +5,13 @@
 | **NIP-01** | Basic Protocol Flow | `js/nostr/client.js`, `docs/nips/01.md` | Compliant | Base implementation verified via `client.js`. |
 | **NIP-04** | Encrypted Direct Message | `js/nostr/dmDecryptWorker.js`, `js/nostr/client.js` | Compliant | Verified by `tests/nostr-specs/nip04-nip44.test.mjs`. |
 | **NIP-07** | `window.nostr` Capability | `js/nostr/nip07Permissions.js`, `js/nostr/adapters/nip07Adapter.js` | Compliant | Retry logic and timeouts verified in `nip07_compliance.test.mjs`. |
-| **NIP-09** | Event Deletion | `js/nostr/client.js`, `docs/nips/09.md` | Partial | `deleteAllVersions` implements Kind 5 logic. |
+| **NIP-09** | Event Deletion | `js/nostr/client.js`, `js/nostrEventSchemas.js` | Compliant | `deleteAllVersions` calls `_hardDeleteVersions` which uses Kind 5. |
 | **NIP-10** | Text Notes & Threads | `js/nostr/commentEvents.js`, `docs/nips/10.md` | Compliant | `buildCommentEvent` implements NIP-10 threading markers (E, P, K tags). |
 | **NIP-17** | Private Direct Messages | `js/nostr/client.js`, `js/ui/profileModal/ProfileDirectMessageRenderer.js` | Compliant | Explicit support for Gift Wraps (Kind 1059) and Seals (Kind 13). |
 | **NIP-19** | bech32-encoded entities | `js/utils/nostrHelpers.js`, `js/nostr/nip71.js` | Compliant | Uses `nostr-tools` for correct encoding/decoding. |
-| **NIP-21** | `nostr:` URI scheme | `docs/nips/21.md` | Unknown | Usage in content parsing verified in `js/utils/nostrHelpers.js`. |
+| **NIP-21** | `nostr:` URI scheme | `js/app/routerCoordinator.js`, `js/utils/nostrHelpers.js` | Partial | Router handles `nostr:` prefix but only for hex/npub. `nprofile`/`nevent` not supported in helper. |
 | **NIP-33** | Addressable Events | `js/nostr/client.js` | Compliant | `d` tag usage verified in `video_note_compliance.test.mjs`. |
+| **NIP-42** | Authentication of clients to relays | `js/nostr/client.js` | Non-compliant | No `AUTH` command handling found in client or pool. |
 | **NIP-44** | Encrypted Payloads (Versioned) | `js/nostr/dmDecryptWorker.js`, `js/nostr/client.js` | Compliant | Verified by `tests/nostr-specs/nip04-nip44.test.mjs`. |
 | **NIP-46** | Nostr Remote Signing | `js/nostr/nip46Client.js` | Compliant | Full client implementation for remote signing. |
 | **NIP-51** | Lists | `js/userBlocks.js`, `js/subscriptions.js` | Compliant | Mute list (Kind 10000) and Subscription list (Kind 30000) logic aligns with spec. |
@@ -33,5 +34,4 @@
 | **Kind 30078** | App Data | `js/nostr/client.js`, `js/nostr/videoPayloadBuilder.js` | Compliant | Verified by `tests/nostr-specs/kind30078.test.mjs`. |
 | **Kind 30079** | Watch History | `js/nostr/watchHistory.js`, `config/instance-config.js` | Compliant | Parameterized replaceable list (custom kind). Implements bucketing and encryption. |
 | **NIP-25** | Reactions | `js/nostr/reactionEvents.js`, `js/nostrEventSchemas.js` | Compliant | Implements Kind 7 reaction events. |
-| **NIP-42** | Authentication of clients to relays | `js/nostr/client.js` | Non-compliant | No `AUTH` command handling found. |
 | **NIP-47** | Wallet Connect | `js/payments/nwcClient.js` | Compliant | Implements NWC client for zaps. |

context/CONTEXT_2026-02-21_23-39-00_nip-research-agent.md

@@ -0,0 +1,19 @@
+# Context: NIP Research Agent Run (2026-02-21)
+
+**Agent:** bitvid-nip-research-agent
+**Date:** 2026-02-21
+**Goal:** Research and verify compliance for NIP-09 (Deletion), NIP-21 (URI Scheme), and NIP-42 (Auth). Update NIP Inventory and generate report.
+
+**Scope:**
+- Verify `js/nostr/client.js` for NIP-09 and NIP-42 compliance.
+- Verify `js/utils/nostrHelpers.js` for NIP-21 compliance.
+- Fetch missing NIP specs (09, 21, 42).
+- Update `NIP_INVENTORY.md`.
+- Generate `nip-report-2026-02-21.md`.
+
+**Definition of Done:**
+- Context/Todo/Decisions/TestLog files created.
+- NIP specs downloaded to `artifacts/nips/`.
+- NIP-09, NIP-21, NIP-42 status updated in `NIP_INVENTORY.md` based on code analysis.
+- `nip-report-2026-02-21.md` created/updated.
+- `npm run lint` passes.

decisions/DECISIONS_2026-02-21_23-39-00_nip-research-agent.md

@@ -0,0 +1,5 @@
+# Decisions: NIP Research Agent Run (2026-02-21)
+
+- **Assumption:** NIP-42 is non-compliant as per initial inventory. Verification will focus on confirming absence of `AUTH` handler.
+- **Assumption:** NIP-21 is handled in `js/utils/nostrHelpers.js` or similar utility files.
+- **Assumption:** NIP-09 deletion logic resides in `js/nostr/client.js` or `js/nostr/adapters/`.

docs/agents/task-logs/daily/2026-02-21_23-45-10_nip-research-agent_completed.md

@@ -0,0 +1,14 @@
+# NIP Research Agent Completed
+
+**Date:** 2026-02-21
+**Agent:** nip-research-agent
+**Status:** Success
+
+## Summary
+- Verified NIP-09 (Compliant), NIP-21 (Partial), NIP-42 (Non-compliant).
+- Created `artifacts/nips/{09,21,42}.md`.
+- Created `scripts/verify-nip21.mjs`.
+- Updated `NIP_INVENTORY.md`.
+- Generated `nip-report-2026-02-21.md`.
+- Tests passed (shard 1).
+- Lint passed.

nip-report-2026-02-21.md

@@ -1,18 +1,44 @@
-# NIP Compliance Report (2026-02-21)
+# NIP Research Report — 2026-02-21
 
-## Executive Summary
+**Agent:** bitvid-nip-research-agent
-This run focused on investigating **NIP-42 (Relay Authentication)**. Analysis of the codebase confirmed that the client currently lacks support for the `AUTH` command and the associated Kind 22242 event signing flow. The NIP-42 specification was fetched and saved to `artifacts/nips/42.md`.
+**Focus:** NIP-09 (Deletion), NIP-21 (URI Scheme), NIP-42 (Auth).
 
-## Changes
+## 1. Inventory Updates
-- **NIP-42**: Status updated from `Unknown` to `Non-compliant`.
-- **Artifacts**: Fetched `artifacts/nips/42.md`.
 
-## Compliance Status (Snapshot)
+| NIP | Previous Status | New Status | Findings |
-- **Compliant**: NIP-01, NIP-04, NIP-07, NIP-10, NIP-17, NIP-19, NIP-33, NIP-44, NIP-46, NIP-51, NIP-56, NIP-57, NIP-59, NIP-65, NIP-78, NIP-94, NIP-98, NIP-25, NIP-47.
+|---|---|---|---|
-- **Partial**: NIP-09 (Event Deletion - incomplete Kind 5), NIP-71 (Video Events - mixed Kind 30078/21/22), NIP-96 (HTTP File Storage).
+| **NIP-09** | Partial | **Compliant** | `js/nostr/client.js` implements `deleteAllVersions` which performs both soft delete (tombstone) and hard delete (Kind 5). `js/nostrEventSchemas.js` defines correct Kind 5 schema. |
-- **Non-compliant**: NIP-42 (Auth).
+| **NIP-21** | Unknown | **Partial** | `js/app/routerCoordinator.js` strips `nostr:` prefix but delegates to `js/utils/nostrHelpers.js` which only handles `npub1...` and hex. URIs like `nostr:nprofile1...` or `nostr:nevent1...` fail. |
-- **Unknown**: NIP-21 (URI Scheme).
+| **NIP-42** | Non-compliant | **Non-compliant** | Confirmed total absence of `AUTH` message handling in `js/nostr/client.js` and `ConnectionManager`. |
 
-## Next Steps
+## 2. Compliance Details
-- **NIP-42 (Auth)**: Implement `AUTH` command handler in `ConnectionManager` or `SimplePool` wrapper.
+
-- **NIP-21**: Verify `nostr:` URI scheme handling.
+### NIP-09 (Event Deletion)
+- **Status:** Compliant.
+- **Evidence:** `NostClient.deleteAllVersions` creates Kind 5 events with proper `e` and `a` tags referencing all historical versions. It also performs local tombstoning.
+- **Action:** Updated status in inventory.
+
+### NIP-21 (`nostr:` URI Scheme)
+- **Status:** Partial.
+- **Evidence:**
+  - `routerCoordinator.js` handles `nostr:` prefix.
+  - `nostrHelpers.normalizeHexPubkey` strictly checks for `startsWith('npub1')`.
+  - Verification script `scripts/verify-nip21.mjs` confirmed failure for `nprofile`.
+- **Gap:** Navigation via `nostr:nprofile...` or `nostr:naddr...` is not supported.
+- **Recommendation:** Update `normalizeHexPubkey` to support `nprofile` decoding via `nostr-tools`.
+
+### NIP-42 (Relay Auth)
+- **Status:** Non-compliant.
+- **Evidence:** `grep "AUTH" js/` yielded no protocol handler results. `SimplePool` usage does not enable auth.
+- **Impact:** Cannot connect to relays requiring authentication (e.g. paid relays).
+- **Recommendation:** Implement `AUTH` challenge handler in `ConnectionManager`.
+
+## 3. Artifacts Created
+- `artifacts/nips/09.md` (Fetched)
+- `artifacts/nips/21.md` (Fetched)
+- `artifacts/nips/42.md` (Fetched)
+- `scripts/verify-nip21.mjs` (Reproduction script)
+
+## 4. Next Steps
+- **P1:** Fix NIP-21 support in `js/utils/nostrHelpers.js` to handle `nprofile` and `naddr`.
+- **P2:** Implement NIP-42 Auth flow.

scripts/verify-nip21.mjs

@@ -0,0 +1,35 @@
+
+import { normalizeHexPubkey } from '../js/utils/nostrHelpers.js';
+
+// Mock window.NostrTools
+global.window = {
+  NostrTools: {
+    nip19: {
+      decode: (str) => {
+        if (str.startsWith('npub1')) return { type: 'npub', data: 'hexpubkey' };
+        if (str.startsWith('nprofile1')) return { type: 'nprofile', data: { pubkey: 'hexpubkey' } };
+        throw new Error('Invalid');
+      },
+      npubEncode: (hex) => 'npub1' + hex
+    }
+  }
+};
+
+console.log('Testing normalizeHexPubkey with various inputs...');
+
+const hex = '0000000000000000000000000000000000000000000000000000000000000001';
+const npub = 'npub1test';
+const nprofile = 'nprofile1test';
+
+const resHex = normalizeHexPubkey(hex);
+console.log(`Hex input: ${resHex === hex ? 'PASS' : 'FAIL'} (${resHex})`);
+
+const resNpub = normalizeHexPubkey(npub);
+console.log(`Npub input: ${resNpub === 'hexpubkey' ? 'PASS' : 'FAIL'} (${resNpub})`);
+
+const resNprofile = normalizeHexPubkey(nprofile);
+console.log(`Nprofile input: ${resNprofile === 'hexpubkey' ? 'PASS' : 'FAIL'} (${resNprofile})`);
+
+if (resNprofile !== 'hexpubkey') {
+    console.log('Confirmed: nprofile is NOT handled by normalizeHexPubkey.');
+}

test_logs/TEST_LOG_2026-02-21_23-39-00_nip-research-agent.md

@@ -0,0 +1,17 @@
+# Test Log: NIP Research Agent Run (2026-02-21)
+
+## Environment
+- Agent: bitvid-nip-research-agent
+- Date: 2026-02-21
+- Node Version: v22+
+
+## Log
+- Acquired lock.
+- Created context files.
+- Fetched NIP-09, NIP-21, NIP-42 specs.
+- Analyzed `js/nostr/client.js` and `js/nostrEventSchemas.js`.
+- Verified NIP-21 partial support using `scripts/verify-nip21.mjs`:
+  - `nostr:<hex>` works (via prefix stripping in router).
+  - `nostr:npub1...` works.
+  - `nostr:nprofile1...` fails (not handled in `nostrHelpers.js`).
+- Ran `npm run test:unit:shard1`: Passed.

todo/TODO_2026-02-21_23-39-00_nip-research-agent.md

@@ -0,0 +1,21 @@
+# Todo: NIP Research Agent Run (2026-02-21)
+
+- [x] Create Context/Todo/Decisions/TestLog files.
+- [x] Explore codebase:
+    - [x] Read `js/nostr/client.js`.
+    - [x] List `js/utils/`.
+    - [x] Read `js/utils/nostrHelpers.js`.
+    - [x] Grep for `nostr:` and `AUTH`.
+- [x] Fetch Specs:
+    - [x] Download NIP-09.
+    - [x] Download NIP-21.
+    - [x] Download NIP-42.
+- [x] Analyze & Verify:
+    - [x] Verify NIP-21 logic (`scripts/verify-nip21.mjs`).
+    - [x] Verify NIP-09 logic (`deleteAllVersions`).
+    - [x] Verify NIP-42 logic (`AUTH`).
+- [x] Update `NIP_INVENTORY.md`.
+- [x] Generate `nip-report-2026-02-21.md`.
+- [x] Run `npm test` (shard 1 passed).
+- [ ] Run `npm run lint`.
+- [ ] Create completion log.