Merge pull request #2147 from PR0M3TH3AN/update-known-issues-lint-cleanup-18181985090645434

Update Known Issues and Cleanup Stale Lint Report

KNOWN_ISSUES.md

@@ -3,7 +3,14 @@
 ## Tests
 
 ### Unit Tests
-- `npm run test:unit` consistently times out in the CI environment due to resource constraints. Targeted testing (`test:dm:unit`, `test:dm:integration`) is recommended.
+- `npm run test:unit` may time out in resource-constrained environments (like CI) when running the full suite serially. CI workflows use sharding (`test:unit:shardX`) to mitigate this. Targeted testing (`test:dm:unit`, `test:dm:integration`) is recommended for local development focus. (Last checked: 2024-05-22)
+
+### Skipped Tests
+The following tests are skipped due to flakiness or environmental issues and should be investigated:
+
+- `tests/e2e/popover.spec.ts`: `keeps the bottom-right grid menu inside the viewport` (Visual regression layout issue)
+- `tests/moderation/video-card.test.mjs`: `VideoCard block action restores trusted mute hide state after override` (Logic issue with state restoration)
+- `tests/login-modal-controller.test.mjs`: `LoginModalController shows NIP-46 handshake panel` (JSDOM async activity leak)
 
 ### Visual Regression Tests (`test:visual`)
 - **Artifact Retention**: Visual tests are configured to retain screenshots, traces, and videos on failure to assist with debugging. These artifacts can be found in `artifacts/test-results/`.

LINT_ISSUES.md

@@ -1,20 +0,0 @@
-# Lint Issues Report
-
-## Summary
-Automated linting and formatting checks were run. No auto-fixable formatting issues were found. However, one linting error persists which requires manual intervention.
-
-## Issues
-
-### css/tailwind.source.css
-
-The `npm run lint:tokens` check failed with the following error:
-
-```
-css/tailwind.source.css:5757 → 767.98px
-  @media (max-width: 767.98px) {
-```
-
-**Recommendation:**
-This raw measurement (`767.98px`) should be moved into `css/tokens.css` as a design token, or the linter configuration (`scripts/check-design-tokens.mjs`) should be updated to whitelist this specific value if it represents a standard responsive breakpoint for mobile layout fixes.
-
-Note: This error currently prevents `npm run lint` from completing successfully, meaning subsequent checks (`lint:tailwind-brackets`, `lint:tailwind-colors`) are skipped in the standard run script. These subsequent checks were run manually and passed.