fix review-mode IIFE syntax error + add keyboard-shortcut e2e tests

0afc54c451f9e15f688cf124317d74d352b94ad1 · 2026-05-11 21:23:07 -0700 · SteveStudio2

- server.js line 1237: \' inside a nested template literal was emitting
  a bare single-quote, producing url(''...'') in the rendered script and
  crashing the entire review-mode IIFE (Unexpected string). Fixed by
  dropping the inner url() quotes (bare url(path) is valid CSS).
- tests/keyboard-shortcuts.spec.js: 6 serial Playwright tests covering
  K/keep+why-chip, R/reject+why-chip, ArrowRight/Left nav, 1-9 slider,
  ?-keymap toast + mouseover focus, P→pair-modal, and the modal-open
  guard that suppresses K/1-9 while pair-modal is open. Uses card slots
  nth(4-10) to avoid colliding with review.spec.js (nth 0-3).
- All 11 tests (6 new + 5 existing) pass in 3.2m.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Files touched

M server.js

Diff

commit 0afc54c451f9e15f688cf124317d74d352b94ad1
Author: SteveStudio2 <stevestudio2@SteveStacStudio.lan>
Date:   Mon May 11 21:23:07 2026 -0700

    fix review-mode IIFE syntax error + add keyboard-shortcut e2e tests
    
    - server.js line 1237: \' inside a nested template literal was emitting
      a bare single-quote, producing url(''...'') in the rendered script and
      crashing the entire review-mode IIFE (Unexpected string). Fixed by
      dropping the inner url() quotes (bare url(path) is valid CSS).
    - tests/keyboard-shortcuts.spec.js: 6 serial Playwright tests covering
      K/keep+why-chip, R/reject+why-chip, ArrowRight/Left nav, 1-9 slider,
      ?-keymap toast + mouseover focus, P→pair-modal, and the modal-open
      guard that suppresses K/1-9 while pair-modal is open. Uses card slots
      nth(4-10) to avoid colliding with review.spec.js (nth 0-3).
    - All 11 tests (6 new + 5 existing) pass in 3.2m.
    
    Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 server.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server.js b/server.js
index 452c8a5..68e7d7a 100644
--- a/server.js
+++ b/server.js
@@ -1234,7 +1234,7 @@ ${isAdmin ? `
       const pinBtn = '<button class="pair-pin" type="button" data-pin-key="' + escapeAttr(s.pin_key) + '" title="' + pinTitle + '">★</button>';
       if (s.type === 'design') {
         const thumb = s.thumb_url
-          ? '<div class="pair-thumb" style="background-image:url(\''+s.thumb_url+'\')"></div>'
+          ? '<div class="pair-thumb" style="background-image:url('+s.thumb_url+')"></div>'
           : '<div class="pair-thumb material">◧</div>';
         return '<div class="pair-card' + pinned + '" data-pin-key="' + escapeAttr(s.pin_key) + '">' +
                '<a class="pair-link" href="/design/' + s.design_id + '" target="_blank">' + thumb +

← 7ffd43b wallco.ai admin: /admin/leads viewer — table of all wallco_l · back to Wallco Ai · remove debug-kb spec scaffolding files (left from investigat 880142e →