- `handleVisibility` only scheduled intervals but didn't trigger an immediate refresh when returning to the tab.
- This caused `tests/unit/services/exploreDataService.test.mjs` to fail (assertion `0 !== 1`).
- Added explicit calls to `refreshWatchHistoryTagCounts` and `refreshTagIdf` when `document.hidden` becomes false.
Co-authored-by: PR0M3TH3AN <[email protected]>
- Ran `npm audit` and `npm outdated`.
- Generated `artifacts/npm-audit.json`, `artifacts/npm-outdated.json`, and `artifacts/deps-report.md`.
- Identified minor vulnerabilities in `ajv` (via `serve`) and outdated protocol/tools packages.
- Created `test_logs` and completion log.
Co-authored-by: PR0M3TH3AN <[email protected]>
Extracts all hashtag preference related logic into a new controller
`js/ui/profileModal/ProfileHashtagController.js` to reduce the size of `js/ui/profileModalController.js` and improve maintainability.
- Moves `normalizeHashtagTag`, `formatHashtagTag`, `getResolvedHashtagPreferences`, `setHashtagStatus`, `refreshHashtagBackgroundStatus`, `populateHashtagPreferences`, `renderHashtagList`, `sanitizeHashtagList`, `createHashtagListItem`, `persistHashtagPreferences`, `handleAddHashtagPreference`, `handleRemoveHashtagPreference`, `handleHashtagPreferencesChange`, `describeHashtagPreferencesError`.
- Updates `ProfileModalController` to delegate to `ProfileHashtagController`.
- Adds delegate getters/methods in `ProfileModalController` to maintain backward compatibility for tests.
- Reduces `js/ui/profileModalController.js` by ~600 lines.
- Updates file size baseline in `scripts/check-file-size.mjs`.
Passes `npm run lint` and `npm run test:unit`.
Co-authored-by: PR0M3TH3AN <[email protected]>
Audited the onboarding process and confirmed all steps pass.
Added .npmrc with engine-strict=true to enforce Node.js >=22 requirement.
Updated CONTRIBUTING.md and README.md to explicitly mention the Node version requirement and its enforcement.
Added artifacts/ONBOARDING_REPORT.md with the audit report.
Co-authored-by: PR0M3TH3AN <[email protected]>
- Created `issues/todo-hashtag-preferences-unit-test.md` to document a flaky unit test in `tests/hashtag-preferences.test.mjs`.
- Generated `artifacts/todos.txt` with the scan results.
Co-authored-by: PR0M3TH3AN <[email protected]>
Implements a load testing script for Nostr relays that simulates multiple clients, generates mixed event traffic (View/Video), and measures performance metrics including latency, throughput, and CPU usage. It also includes strict safety guardrails to prevent accidental load testing against public relays.
Signed-off-by: Jules <[email protected]>
Co-authored-by: PR0M3TH3AN <[email protected]>
- Updated playwright and @playwright/test to 1.58.2.
- Updated dependency tracking issues for nostr-tools, stylelint, and esbuild.
- Included dependency scan artifacts in description.
Co-authored-by: PR0M3TH3AN <[email protected]>
- Implement CPU usage tracking in load-test.mjs
- Run load test with 1000 clients for 60s
- Generate load-report.json with identified bottlenecks (crypto)
Co-authored-by: PR0M3TH3AN <[email protected]>
- Added `scripts/agent/telemetry-aggregator.mjs` to aggregate error logs while stripping PII.
- Updated `AGENTS.md` with telemetry policy.
- Cleaned up `package.json` dev dependencies.
- Fixed `tests/nostr-login-permissions.test.mjs` by updating `js/nostr/client.js` to request all encryption permissions during login.
- Fixed `tests/hashtag-preferences.test.mjs` by clearing `nostrClient.extensionPermissionCache` in test setup to prevent state leakage.
Co-authored-by: PR0M3TH3AN <[email protected]>
Updated `loginWithExtension` to request `DEFAULT_NIP07_PERMISSION_METHODS` instead of `DEFAULT_NIP07_CORE_METHODS`. This ensures encryption permissions (NIP-04/44) are requested upfront and cached, preventing permission errors or unexpected popups during subsequent operations like DM decryption or preference loading.
Also includes artifacts from TODO triage:
- `artifacts/todos.txt`: Scanned TODOs list.
- `artifacts/issue_moderation_tests.md`: Issue report for moderation test TODOs.
Co-authored-by: PR0M3TH3AN <[email protected]>
- Excluded `*.map` and `*.min.js` files from the `todos.txt` generation command to prevent massive minified lines from crashing CI scripts.
- Mocked `nostrClient.readRelays` in `tests/app-batch-fetch-profiles.test.mjs` to fix assertion errors caused by duplicate network calls.
- Regenerated `artifacts/todos.txt` with the clean output.
Co-authored-by: PR0M3TH3AN <[email protected]>
Scanned the codebase for TODO/FIXME markers.
Generated artifacts/todos.txt.
Verified that identified TODOs in tests/visual/moderation.spec.ts are tracked in issues/todo-moderation-tests.md.
No new issues or PRs required as no other actionable TODOs were found.
Co-authored-by: PR0M3TH3AN <[email protected]>
Removed unused exports and functions identified as dead code.
- Removed unused exports from `js/utils/storage.js`, `js/utils/storagePointer.js`, `js/utils/videoTags.js`, `js/ui/moderationCopy.js`.
- Removed unused function `updateTagPillStrip` from `js/ui/components/tagPillList.js` and associated tests.
- Updated `tests/utils/video-tags.test.mjs` to remove test for private function.
- Added `artifacts/depcheck.json` showing dependency analysis.
Co-authored-by: PR0M3TH3AN <[email protected]>
- Generate `artifacts/npm-audit.json` and `artifacts/npm-outdated.json`.
- Document required major upgrades in `issues/upgrade-*.md`.
- Fix ReferenceError in `tests/nostr-login-permissions.test.mjs` exposed during verification.
- Note: `cross-env` upgrade reverted as it is a major version bump requiring separate review.
- Note: `nostr-tools` upgrade skipped per security policy.
Co-authored-by: PR0M3TH3AN <[email protected]>
