Merge branch 'codex/feedback-polish' into codex/daily-engagement

Author: gmval-g3

index.html

@@ -68,6 +68,7 @@
 }
 @media(hover:none),(pointer:coarse){
   #action-button{display:block}
+  #sound-button{right:max(104px,calc(104px + env(safe-area-inset-right)));bottom:max(28px,calc(28px + env(safe-area-inset-bottom)))}
 }
 @media(prefers-reduced-motion:reduce){
   #title-bar .dot,#hint{animation:none}
@@ -881,6 +882,7 @@ <h2 id="journal-title">SIGNAL LOG</h2>
 let clickMarker = null;
 
 canvas.addEventListener('click', (e) => {
+  feedback.unlock();
   canvas.focus({ preventScroll: true });
   const rect = canvas.getBoundingClientRect();
   const mx = e.clientX - rect.left, my = e.clientY - rect.top;
@@ -934,6 +936,7 @@ <h2 id="journal-title">SIGNAL LOG</h2>
 
 window.addEventListener('keydown', (e) => {
   if (!shouldHandleGameKey(e.target)) return;
+  feedback.unlock();
   const k = e.key.toLowerCase();
   if (['w','a','s','d','arrowup','arrowdown','arrowleft','arrowright',' ','enter'].includes(k)) e.preventDefault();
   keysDown[k] = true;
@@ -1099,6 +1102,7 @@ <h2 id="journal-title">SIGNAL LOG</h2>
 }
 
 actionButton.addEventListener('click', () => {
+  feedback.unlock();
   canvas.focus({ preventScroll: true });
   performAction();
 });

src/feedback.js

@@ -40,12 +40,24 @@
     let context = null;
     let muted = readMuted(storage);
 
+    function unlock() {
+      if (muted || !AudioContextClass) return false;
+      try {
+        context ||= new AudioContextClass();
+        if (context.state === 'suspended') {
+          const resume = context.resume();
+          if (resume && typeof resume.catch === 'function') resume.catch(() => {});
+        }
+        return true;
+      } catch {
+        return false;
+      }
+    }
+
     function play(name) {
       const cue = CUES[name];
-      if (muted || !cue || !AudioContextClass) return false;
+      if (!cue || !unlock()) return false;
       try {
-        context ||= new AudioContextClass();
-        if (context.state === 'suspended') context.resume();
         const start = context.currentTime;
         for (const [frequency, offset] of cue) {
           const oscillator = context.createOscillator();
@@ -77,6 +89,7 @@
     }
 
     return Object.freeze({
+      unlock,
       play,
       haptic,
       isMuted: () => muted,

tests/feedback.test.cjs

@@ -29,3 +29,21 @@ test('feedback is a safe no-op when browser APIs are unavailable', () => {
   assert.equal(controller.toggleMuted(), true);
   assert.equal(readMuted(storage), true);
 });
+
+test('audio unlock resumes a suspended context during input', () => {
+  let resumeCalls = 0;
+  class AudioContext {
+    constructor() {
+      this.state = 'suspended';
+    }
+
+    resume() {
+      resumeCalls++;
+      this.state = 'running';
+      return Promise.resolve();
+    }
+  }
+  const controller = createFeedbackController({ AudioContext }, memoryStorage());
+  assert.equal(controller.unlock(), true);
+  assert.equal(resumeCalls, 1);
+});