Merge pull request #39 from nesquena/fix/test-keyframe-parser

fix: test_send_pop_in keyframe parser found wrong @keyframes block

CHANGELOG.md

@@ -5,8 +5,48 @@
 
 ---
 
+## [v0.22] Sprint 20 -- Voice Input + Send Button Polish
+*April 3, 2026 | 415 tests*
+
+### Features
+- **Voice input via Web Speech API.** Microphone button in the composer.
+  Tap to start recording, tap again (or send) to stop. Live interim
+  transcription appears in the textarea. Auto-stops after ~2s of silence.
+  Final text stays editable before sending. Appends to existing textarea
+  content rather than replacing it. Button hidden when browser doesn't
+  support Web Speech API. No API keys, no external libraries, no server
+  changes. Works in Chrome, Edge, Safari (partial). Firefox unsupported
+  (button stays hidden).
+- **Send button polish.** Send button redesigned as a 34px icon-only circle
+  with upward arrow SVG. Hidden by default — appears with pop-in spring
+  animation when textarea has content or files are attached. Disappears
+  on send or when content is cleared. Hidden while agent is responding.
+  Blue fill (#7cb9ff) with glow, scale hover/active for tactile feedback.
+
+### Architecture
+- Voice input IIFE in `boot.js`: SpeechRecognition lifecycle with
+  `continuous=false`, `interimResults=true`, error handling via `showToast()`.
+- `_prefix` variable snapshots existing textarea content on recording start
+  so dictation appends rather than overwrites.
+- `btnSend.onclick` stops active recognition before sending (send guard).
+- CSS: `.mic-btn`, `.mic-btn.recording` (red pulse), `.mic-status`,
+  `.mic-dot`, `@keyframes mic-pulse`.
+- `updateSendBtn()` in `ui.js` tracks textarea content, pending files,
+  and busy state. Hooked into `setBusy()`, `renderTray()`, `autoResize()`,
+  and input event listener.
+- CSS: `.send-btn` redesigned (circle, glow), `.send-btn.visible` +
+  `@keyframes send-pop-in` (spring animation).
+
+### Tests
+- 52 new tests in `test_sprint20.py`: voice input HTML, CSS, JS, append
+  behaviour, error handling, regressions.
+- 33 new tests in `test_sprint20b.py`: send button HTML, CSS, JS,
+  animation, visibility logic, regressions. Total: **415 tests**.
+
+---
+
 ## [v0.21] Sprint 19 -- Auth + Security Hardening
-*April 3, 2026 | 327 tests*
+*April 3, 2026 | 328 tests*
 
 ### Features
 - **Password authentication (Issue #23).** Optional password auth, off by default.
@@ -27,8 +67,8 @@
 - `password_hash` added to `_SETTINGS_DEFAULTS`.
 
 ### Tests
-- 9 new tests in `test_sprint19.py`: auth status, login flow, security headers,
-  cache-control, settings password field. Total: **327 tests (304 passing)**.
+- 10 new tests in `test_sprint19.py`: auth status, login flow, security headers,
+  cache-control, settings password field, request size limit. Total: **328 tests (328 passing)**.
 
 ---
 
@@ -676,4 +716,4 @@ Three-panel layout: sessions sidebar, chat area, workspace panel.
 
 ---
 
-*Last updated: v0.21, April 3, 2026 | Tests: 327*
+*Last updated: v0.22, April 3, 2026 | Tests: 415*

ROADMAP.md

@@ -4,7 +4,7 @@
 > Everything you can do from the CLI terminal, you can do from this UI.
 >
 > Last updated: Sprint 19 / v0.21 (April 3, 2026)
-> Tests: 327 total (304 passing, 23 pre-existing failures)
+> Tests: 328 total (328 passing, 0 failures)
 > Source: <repo>/
 
 ---
@@ -35,7 +35,7 @@
 | Sprint 16 | Session sidebar visual polish | SVG action icons, overlay hover actions, pin indicator, project border, safe HTML rendering | 289 |
 | Sprint 17 | Workspace polish + slash commands + settings | Breadcrumb navigation, slash command autocomplete, send key setting (#26) | 318 |
 | Sprint 18 | Thinking display + workspace tree | File preview auto-close, thinking/reasoning cards, expandable directory tree (#22) | 318 |
-| Sprint 19 | Auth + security hardening | Password auth (off by default), login page, security headers, 20MB body limit (#23) | 327 |
+| Sprint 19 | Auth + security hardening | Password auth (off by default), login page, security headers, 20MB body limit (#23) | 328 |
 
 ---
 

SPRINTS.md

@@ -1,6 +1,6 @@
 # Hermes Web UI -- Forward Sprint Plan
 
-> Current state: v0.21 | 327 tests (304 passing) | Daily driver ready
+> Current state: v0.22 | 415 tests | Daily driver ready
 > This document plans the path from here to two targets:
 >
 > Target A: 1:1 feature parity with the Hermes CLI (everything you can do from the
@@ -384,25 +384,40 @@ hardening feature before the app is safe to expose to a network.
 - `password_hash` added to `_SETTINGS_DEFAULTS` in config.py.
 - `_set_password` special field in save_settings for secure password updates.
 
-**Tests:** 9 new. Total: 327.
+**Tests:** 10 new. Total: 328.
 **Hermes CLI parity impact:** Low (CLI has no auth concerns)
 **Claude parity impact:** High (Claude is authenticated)
 
 ---
 
-## Sprint 20 -- Voice + TTS (PLANNED)
+## Sprint 20 -- Voice Input + Send Button Polish (COMPLETED)
 
-**Theme:** Input and output beyond the keyboard.
+**Theme:** Input refinements — voice and visual polish.
 
-**Why now:** Voice works in the Hermes CLI. Mirror that capability in the web UI.
-TTS playback makes long responses more accessible. Both are achievable with
-existing Whisper and TTS APIs.
+**Why now:** Voice input was the next feature on the roadmap. The send button
+UX was a low-effort high-impact polish opportunity that pairs naturally.
+
+### Track A: Bugs
+- **Send button always visible.** The old pill-shaped "Send" button was always
+  visible even with an empty textarea, wasting space. Now hidden by default,
+  appears only when there is content to send.
 
 ### Track B: Features
-- **Voice input (Whisper).** Microphone icon in composer. Hold to record,
-  release to transcribe. Transcribed text editable before send.
-- **TTS playback.** Speaker icon on assistant messages. Audio playback via
-  OpenAI TTS or ElevenLabs API. Optional auto-play in settings.
+- **Voice input (Web Speech API).** Microphone button in composer. Tap to
+  record, tap again to stop. Live interim transcription in textarea. Auto-stops
+  after ~2s of silence. Appends to existing text. Hidden when browser doesn't
+  support Web Speech API. No API keys, no server changes.
+- **Send button polish.** Icon-only 34px circle with upward arrow SVG. Pop-in
+  spring animation on appear. Scale hover/active for tactile feedback. Hidden
+  while agent is responding.
+
+### Track C: Architecture
+- Voice input IIFE in `boot.js` with SpeechRecognition lifecycle.
+- `updateSendBtn()` in `ui.js` hooked into setBusy, renderTray, autoResize.
+
+**Tests:** 52 new (voice) + 33 new (send button). Total: 415.
+**Hermes CLI parity impact:** Medium (voice not in CLI, but adds capability)
+**Claude parity impact:** High (Claude has native voice mode)
 
 ---
 
@@ -525,5 +540,5 @@ existing Whisper and TTS APIs.
 ---
 
 *Last updated: April 3, 2026*
-*Current version: v0.21 | 327 tests (304 passing)*
-*Next sprint: Sprint 20 (Voice + TTS)*
+*Current version: v0.22 | 415 tests*
+*Next sprint: Sprint 21 (Mobile Responsive)*

TESTING.md

@@ -8,7 +8,7 @@
 > Prerequisites: SSH tunnel is active on port 8787. Open http://localhost:8787 in browser.
 > Server health check: curl http://127.0.0.1:8787/health should return {"status":"ok"}.
 >
-> Automated tests: 327 total (304 passing, 23 pre-existing failures).
+> Automated tests: 328 total (328 passing, 0 failures).
 > Run: `pytest tests/ -v --timeout=60`
 
 ---
@@ -1670,7 +1670,7 @@ Each has automated API-level tests in `tests/test_sprint{N}.py`.
 ---
 
 *Last updated: Sprint 19 / v0.21, April 3, 2026*
-*Total automated tests: 327 (304 passing, 23 pre-existing failures in Sprint 3/5/7)*
+*Total automated tests: 328 (328 passing, 0 failures)*
 *Regression gate: tests/test_regressions.py (23 tests)*
 *Run: pytest tests/ -v --timeout=60*
 *Source: <repo>/*

api/streaming.py

@@ -189,7 +189,8 @@ def _run_agent_streaming(session_id, msg_text, model, workspace, stream_id, atta
             else: os.environ['HERMES_SESSION_KEY'] = old_session_key
 
     except Exception as e:
-        put('error', {'message': str(e), 'trace': traceback.format_exc()})
+        print('[webui] stream error:\n' + traceback.format_exc(), flush=True)
+        put('error', {'message': str(e)})
     finally:
         _clear_thread_env()  # TD1: always clear thread-local context
         with STREAMS_LOCK:

api/upload.py

@@ -74,4 +74,5 @@ def handle_upload(handler):
         dest.write_bytes(file_bytes)
         return j(handler, {'filename': safe_name, 'path': str(dest), 'size': dest.stat().st_size})
     except Exception as e:
-        return j(handler, {'error': str(e), 'trace': _tb.format_exc()}, status=500)
+        print('[webui] upload error: ' + _tb.format_exc(), flush=True)
+        return j(handler, {'error': 'Upload failed'}, status=500)

server.py

@@ -40,7 +40,8 @@ class Handler(BaseHTTPRequestHandler):
             if result is False:
                 return j(self, {'error': 'not found'}, status=404)
         except Exception as e:
-            return j(self, {'error': str(e), 'trace': traceback.format_exc()}, status=500)
+            print(f'[webui] ERROR {self.command} {self.path}\n' + traceback.format_exc(), flush=True)
+            return j(self, {'error': 'Internal server error'}, status=500)
 
     def do_POST(self):
         self._req_t0 = time.time()
@@ -51,7 +52,8 @@ class Handler(BaseHTTPRequestHandler):
             if result is False:
                 return j(self, {'error': 'not found'}, status=404)
         except Exception as e:
-            return j(self, {'error': str(e), 'trace': traceback.format_exc()}, status=500)
+            print(f'[webui] ERROR {self.command} {self.path}\n' + traceback.format_exc(), flush=True)
+            return j(self, {'error': 'Internal server error'}, status=500)
 
 
 def main():

static/boot.js

@@ -8,8 +8,90 @@ async function cancelStream(){
   }catch(e){setStatus('Cancel failed: '+e.message);}
 }
 
-$('btnSend').onclick=send;
+$('btnSend').onclick=()=>{if(window._micActive)_stopMic();send();};
 $('btnAttach').onclick=()=>$('fileInput').click();
+
+// ── Voice input (Web Speech API) ─────────────────────────────────────────
+(function(){
+  const SpeechRecognition=window.SpeechRecognition||window.webkitSpeechRecognition;
+  if(!SpeechRecognition) return; // Browser unsupported — mic button stays hidden
+
+  const btn=$('btnMic');
+  const status=$('micStatus');
+  const ta=$('msg');
+  btn.style.display=''; // Show button — browser supports speech
+
+  const recognition=new SpeechRecognition();
+  recognition.continuous=false;
+  recognition.interimResults=true;
+  recognition.lang='en-US';
+
+  let _finalText='';
+  let _prefix='';
+
+  function _setRecording(on){
+    window._micActive=on;
+    btn.classList.toggle('recording',on);
+    status.style.display=on?'':'none';
+    if(!on){ _finalText=''; _prefix=''; }
+  }
+
+  recognition.onstart=()=>{ _finalText=''; };
+
+  recognition.onresult=(event)=>{
+    let interim='';
+    let final=_finalText;
+    for(let i=event.resultIndex;i<event.results.length;i++){
+      const t=event.results[i][0].transcript;
+      if(event.results[i].isFinal){ final+=t; _finalText=final; }
+      else{ interim+=t; }
+    }
+    // Append to whatever was already in the textarea before mic started
+    ta.value=_prefix+(final||interim);
+    autoResize();
+  };
+
+  recognition.onend=()=>{
+    // Commit: prefix + final transcription; trim trailing space if prefix was non-empty
+    const committed=_finalText
+      ? (_prefix&&!_prefix.endsWith(' ')&&!_prefix.endsWith('\n')
+          ? _prefix+' '+_finalText.trimStart()
+          : _prefix+_finalText)
+      : ta.value; // no speech detected — leave whatever is there
+    _setRecording(false);
+    ta.value=committed;
+    autoResize();
+  };
+
+  recognition.onerror=(event)=>{
+    _setRecording(false);
+    const msgs={
+      'not-allowed':'Microphone access denied. Check browser permissions.',
+      'no-speech':'No speech detected. Try again.',
+      'network':'Speech recognition unavailable.',
+    };
+    showToast(msgs[event.error]||'Voice input error: '+event.error);
+  };
+
+  function _stopMic(){
+    if(window._micActive){ recognition.stop(); }
+  }
+  window._stopMic=_stopMic; // expose for send-guard above
+
+  btn.onclick=()=>{
+    if(window._micActive){
+      recognition.stop();
+      // _setRecording(false) will be called by onend
+    } else {
+      _finalText='';
+      // Snapshot existing textarea content so we append rather than replace
+      _prefix=ta.value;
+      recognition.start();
+      _setRecording(true);
+    }
+  };
+})();
+window._micActive=window._micActive||false;
 $('fileInput').onchange=e=>{addFiles(Array.from(e.target.files));e.target.value='';};
 $('btnNewChat').onclick=async()=>{await newSession();await renderSessionList();$('msg').focus();};
 $('btnDownload').onclick=()=>{
@@ -63,6 +145,7 @@ $('modelSelect').onchange=async()=>{
 };
 $('msg').addEventListener('input',()=>{
   autoResize();
+  updateSendBtn();
   const text=$('msg').value;
   if(text.startsWith('/')&&text.indexOf('\n')===-1){
     const prefix=text.slice(1);

static/index.html

@@ -13,7 +13,7 @@
 <body>
 <div class="layout">
   <aside class="sidebar">
-    <div class="sidebar-header"><div class="logo">H</div><div><h1 style="margin:0;font-size:15px;font-weight:700;letter-spacing:-.01em">Hermes</h1><div style="font-size:10px;color:var(--muted);opacity:.8;margin-top:1px">v0.21</div></div></div>
+    <div class="sidebar-header"><div class="logo">H</div><div><h1 style="margin:0;font-size:15px;font-weight:700;letter-spacing:-.01em">Hermes</h1><div style="font-size:10px;color:var(--muted);opacity:.8;margin-top:1px">v0.22</div></div></div>
     <div class="sidebar-nav">
       <button class="nav-tab active" data-panel="chat" data-label="Chat" onclick="switchPanel('chat')" title="Chat">&#128172;</button>
       <button class="nav-tab" data-panel="tasks" data-label="Tasks" onclick="switchPanel('tasks')" title="Tasks">&#128197;</button>
@@ -213,6 +213,7 @@
           Drop files to upload to workspace
         </div>
         <div class="attach-tray" id="attachTray"></div>
+        <div class="mic-status" id="micStatus" style="display:none"><span class="mic-dot"></span> Listening…</div>
         <textarea id="msg" rows="1" placeholder="Message Hermes…"></textarea>
         <div class="composer-footer">
           <div class="composer-left">
@@ -220,11 +221,18 @@
             <button class="icon-btn" id="btnAttach" title="Attach files">
               <svg width="16" height="16" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><path d="M21.44 11.05l-9.19 9.19a6 6 0 0 1-8.49-8.49l9.19-9.19a4 4 0 0 1 5.66 5.66l-9.2 9.19a2 2 0 0 1-2.83-2.83l8.49-8.48"/></svg>
             </button>
+            <button class="icon-btn mic-btn" id="btnMic" title="Voice input" style="display:none">
+              <svg width="16" height="16" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
+                <rect x="9" y="1" width="6" height="12" rx="3"/>
+                <path d="M5 10a7 7 0 0 0 14 0"/>
+                <line x1="12" y1="19" x2="12" y2="23"/>
+                <line x1="8" y1="23" x2="16" y2="23"/>
+              </svg>
+            </button>
           </div>
           <div class="composer-right">
-            <button class="send-btn" id="btnSend">
-              <svg width="14" height="14" viewBox="0 0 24 24" fill="currentColor"><path d="M2.01 21L23 12 2.01 3 2 10l15 2-15 2z"/></svg>
-              Send
+            <button class="send-btn" id="btnSend" title="Send message" style="display:none">
+              <svg width="16" height="16" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2.5" stroke-linecap="round" stroke-linejoin="round"><line x1="12" y1="19" x2="12" y2="5"/><polyline points="5 12 12 5 19 12"/></svg>
             </button>
           </div>
         </div>

static/messages.js

@@ -237,7 +237,7 @@ function transcript(){
   return lines.join('\n');
 }
 
-function autoResize(){const el=$('msg');el.style.height='auto';el.style.height=Math.min(el.scrollHeight,200)+'px';}
+function autoResize(){const el=$('msg');el.style.height='auto';el.style.height=Math.min(el.scrollHeight,200)+'px';updateSendBtn();}
 
 
 // ── Approval polling ──

static/style.css

@@ -187,11 +187,18 @@
   .icon-btn{width:34px;height:34px;border-radius:8px;background:none;border:none;color:var(--muted);cursor:pointer;display:flex;align-items:center;justify-content:center;font-size:16px;transition:all .15s;}
   .icon-btn{opacity:.75;}
   .icon-btn:hover{background:rgba(255,255,255,.08);color:var(--text);opacity:1;}
+  .mic-btn{transition:color .15s,background .15s;}
+  .mic-btn.recording{color:#e94560;background:rgba(233,69,96,.12);animation:mic-pulse 1.2s ease-in-out infinite;}
+  @keyframes mic-pulse{0%,100%{box-shadow:0 0 0 0 rgba(233,69,96,.3);}50%{box-shadow:0 0 0 6px rgba(233,69,96,0);}}
+  .mic-status{font-size:11px;color:#e94560;padding:4px 12px;display:flex;align-items:center;gap:6px;}
+  .mic-dot{width:6px;height:6px;border-radius:50%;background:#e94560;animation:mic-pulse 1.2s ease-in-out infinite;flex-shrink:0;}
   .status-text{font-size:11px;color:var(--muted);padding-left:4px;}
-  .send-btn{padding:7px 18px;border-radius:10px;font-size:13px;font-weight:600;background:linear-gradient(135deg,#5ba8f5,#7cb9ff);border:none;color:#0a1628;cursor:pointer;display:flex;align-items:center;gap:6px;transition:all .15s;flex-shrink:0;letter-spacing:.01em;}
-  .send-btn:hover{background:linear-gradient(135deg,#7cb9ff,#a0d0ff);transform:translateY(-1px);}
-  .send-btn:active{transform:translateY(0);}
-  .send-btn:disabled{opacity:.4;cursor:not-allowed;}
+  .send-btn{width:34px;height:34px;border-radius:50%;background:#7cb9ff;border:none;color:#0a1628;cursor:pointer;display:flex;align-items:center;justify-content:center;flex-shrink:0;transition:background .15s,transform .15s,box-shadow .15s;box-shadow:0 2px 8px rgba(124,185,255,.35);}
+  .send-btn:hover{background:#a0d0ff;transform:scale(1.08);box-shadow:0 4px 14px rgba(124,185,255,.5);}
+  .send-btn:active{transform:scale(0.95);box-shadow:0 1px 4px rgba(124,185,255,.25);}
+  .send-btn:disabled{opacity:.35;cursor:not-allowed;transform:none;box-shadow:none;}
+  .send-btn.visible{animation:send-pop-in .18s cubic-bezier(.34,1.56,.64,1) forwards;}
+  @keyframes send-pop-in{from{opacity:0;transform:scale(.55);}to{opacity:1;transform:scale(1);}}
   .upload-bar-wrap{display:none;height:3px;background:rgba(255,255,255,.06);border-radius:0 0 16px 16px;overflow:hidden;}
   .upload-bar-wrap.active{display:block;}
   .upload-bar{height:100%;background:linear-gradient(90deg,var(--blue),#a0d0ff);width:0%;transition:width .3s ease;}
@@ -271,7 +278,7 @@
     .composer-wrap{padding:8px 10px 12px!important;}
     .composer-box{border-radius:12px;}
     .composer-box textarea{font-size:16px;min-height:40px;}
-    .send-btn{padding:6px 14px;font-size:13px;}
+    .send-btn{width:32px;height:32px;}
     /* Empty state */
     .empty-state h2{font-size:18px;}
     .empty-state p{font-size:13px;}

static/ui.js

@@ -187,9 +187,25 @@ function setStatus(t){
     if(dismiss)dismiss.style.display=(!transient && !S.busy)?'inline':'none';
   }
 }
+function updateSendBtn(){
+  const btn=$('btnSend');
+  if(!btn) return;
+  const hasContent=$('msg').value.trim().length>0||S.pendingFiles.length>0;
+  const shouldShow=hasContent&&!S.busy;
+  if(shouldShow&&btn.style.display==='none'){
+    btn.style.display='';
+    // Remove then re-add class to retrigger animation each time
+    btn.classList.remove('visible');
+    requestAnimationFrame(()=>btn.classList.add('visible'));
+  } else if(!shouldShow&&btn.style.display!=='none'){
+    btn.style.display='none';
+    btn.classList.remove('visible');
+  }
+}
 function setBusy(v){
   S.busy=v;
   $('btnSend').disabled=v;
+  updateSendBtn();
   const dots=$('activityDots');
   if(dots) dots.style.display=v?'flex':'none';
   if(!v){
@@ -920,8 +936,9 @@ async function promptNewFolder(){
 
 function renderTray(){
   const tray=$('attachTray');tray.innerHTML='';
-  if(!S.pendingFiles.length){tray.classList.remove('has-files');return;}
+  if(!S.pendingFiles.length){tray.classList.remove('has-files');updateSendBtn();return;}
   tray.classList.add('has-files');
+  updateSendBtn();
   S.pendingFiles.forEach((f,i)=>{
     const chip=document.createElement('div');chip.className='attach-chip';
     chip.innerHTML=`&#128206; ${esc(f.name)} <button title="Remove">&#10005;</button>`;

tests/test_regressions.py

@@ -438,3 +438,37 @@ def test_newSession_clears_live_tool_cards(cleanup_test_sessions):
     next_fn = src.find("async function ", new_sess_idx + 10)
     new_sess_body = src[new_sess_idx:next_fn]
     assert "clearLiveToolCards" in new_sess_body,         "newSession() must call clearLiveToolCards() to clear stale live cards"
+
+
+# ── R16: Stack traces must not leak to clients in 500 responses ────────────
+
+def test_500_response_has_no_trace_field():
+    """R16: HTTP 500 responses must not include a 'trace' field.
+    Leaking tracebacks exposes file paths, module names, and potentially
+    secret values from local variables.
+    """
+    # POST to /api/chat/start with missing required fields to trigger an error
+    data, status = post("/api/chat/start", {})
+    # Should be an error response (4xx or 5xx)
+    assert "trace" not in data, \
+        "Server must not leak stack traces to clients"
+
+def test_upload_error_has_no_trace_field():
+    """R16b: Upload 500 responses must not include a 'trace' field."""
+    # Send a POST to /api/upload with invalid content to trigger the error handler
+    req = urllib.request.Request(
+        BASE + "/api/upload",
+        data=b"not-multipart-data",
+        headers={"Content-Type": "text/plain", "Content-Length": "18"},
+    )
+    try:
+        with urllib.request.urlopen(req, timeout=10) as r:
+            body = json.loads(r.read())
+            code = r.status
+    except urllib.error.HTTPError as e:
+        body = json.loads(e.read())
+        code = e.code
+    assert code >= 400, "Invalid upload should return an error status"
+    assert "trace" not in body, \
+        "Upload errors must not leak stack traces to clients"
+    assert "error" in body, "Error responses must include an 'error' key"

tests/test_sprint20.py

@@ -0,0 +1,422 @@
+"""
+Sprint 20 Tests: Voice input (mic button) via Web Speech API.
+
+These tests verify the static assets contain the correct HTML structure,
+CSS rules, and JS logic for the mic feature — all of which runs purely in
+the browser with no server-side component.
+"""
+import re
+import urllib.request
+import json
+
+BASE = "http://127.0.0.1:8788"
+
+
+def get_text(path):
+    with urllib.request.urlopen(BASE + path, timeout=10) as r:
+        return r.read().decode(), r.status
+
+
+# ── index.html ────────────────────────────────────────────────────────────
+
+
+def test_mic_button_present_in_html():
+    """index.html must contain the mic button with id='btnMic'."""
+    html, status = get_text("/")
+    assert status == 200
+    assert 'id="btnMic"' in html
+
+
+def test_mic_button_has_mic_btn_class():
+    """btnMic must carry the mic-btn CSS class for styling hooks."""
+    html, _ = get_text("/")
+    assert 'class="icon-btn mic-btn"' in html
+
+
+def test_mic_button_hidden_by_default():
+    """btnMic starts hidden (display:none) — JS shows it only if supported."""
+    html, _ = get_text("/")
+    # The button element should have display:none in its style attribute
+    assert 'id="btnMic"' in html
+    btn_match = re.search(r'id="btnMic"[^>]*>', html)
+    assert btn_match, "btnMic element not found"
+    assert 'display:none' in btn_match.group(0)
+
+
+def test_mic_button_has_title():
+    """btnMic should have a descriptive title for accessibility."""
+    html, _ = get_text("/")
+    btn_match = re.search(r'id="btnMic"[^>]*>', html)
+    assert btn_match
+    assert 'title=' in btn_match.group(0)
+
+
+def test_mic_status_div_present():
+    """index.html must contain the #micStatus listening indicator."""
+    html, _ = get_text("/")
+    assert 'id="micStatus"' in html
+
+
+def test_mic_status_hidden_by_default():
+    """#micStatus starts hidden — only shown during active recording."""
+    html, _ = get_text("/")
+    status_match = re.search(r'id="micStatus"[^>]*>', html)
+    assert status_match, "#micStatus element not found"
+    assert 'display:none' in status_match.group(0)
+
+
+def test_mic_status_has_mic_dot():
+    """#micStatus must contain a .mic-dot element for the pulse animation."""
+    html, _ = get_text("/")
+    # mic-dot should appear after micStatus
+    idx_status = html.find('id="micStatus"')
+    idx_dot = html.find('mic-dot', idx_status)
+    assert idx_status != -1 and idx_dot != -1
+    assert idx_dot > idx_status
+
+
+def test_mic_status_has_listening_text():
+    """#micStatus should display a 'Listening' label."""
+    html, _ = get_text("/")
+    assert 'Listening' in html
+
+
+def test_mic_button_svg_microphone_shape():
+    """btnMic SVG must include the rect (mic body) and path (mic arc)."""
+    html, _ = get_text("/")
+    # Find mic button section
+    btn_start = html.find('id="btnMic"')
+    btn_end = html.find('</button>', btn_start) + len('</button>')
+    btn_html = html[btn_start:btn_end]
+    assert '<rect' in btn_html, "mic SVG missing rect (mic body)"
+    assert '<path' in btn_html, "mic SVG missing path (arc)"
+    assert '<line' in btn_html, "mic SVG missing line (stand)"
+
+
+def test_mic_button_inside_composer_left():
+    """btnMic must be inside .composer-left, next to the attach button."""
+    html, _ = get_text("/")
+    composer_left_start = html.find('class="composer-left"')
+    composer_left_end = html.find('</div>', composer_left_start)
+    section = html[composer_left_start:composer_left_end]
+    assert 'btnAttach' in section
+    assert 'btnMic' in section
+
+
+# ── style.css ────────────────────────────────────────────────────────────
+
+
+def test_mic_btn_css_rule_exists():
+    """style.css must define .mic-btn rule."""
+    css, status = get_text("/static/style.css")
+    assert status == 200
+    assert '.mic-btn' in css
+
+
+def test_mic_btn_recording_state_css():
+    """.mic-btn.recording must be defined for active recording visual state."""
+    css, _ = get_text("/static/style.css")
+    assert '.mic-btn.recording' in css
+
+
+def test_mic_recording_color_red():
+    """.mic-btn.recording must use the red accent color #e94560."""
+    css, _ = get_text("/static/style.css")
+    recording_idx = css.find('.mic-btn.recording')
+    # Find the rule block after the selector
+    brace_open = css.find('{', recording_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert '#e94560' in rule or 'e94560' in rule
+
+
+def test_mic_recording_has_animation():
+    """.mic-btn.recording must use an animation for the pulse effect."""
+    css, _ = get_text("/static/style.css")
+    recording_idx = css.find('.mic-btn.recording')
+    brace_open = css.find('{', recording_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'animation' in rule
+
+
+def test_mic_pulse_keyframes_defined():
+    """@keyframes mic-pulse must be defined for the pulsing animation."""
+    css, _ = get_text("/static/style.css")
+    assert 'mic-pulse' in css
+    assert '@keyframes' in css
+
+
+def test_mic_status_css_rule_exists():
+    """style.css must define .mic-status rule."""
+    css, _ = get_text("/static/style.css")
+    assert '.mic-status' in css
+
+
+def test_mic_dot_css_rule_exists():
+    """style.css must define .mic-dot rule with animation."""
+    css, _ = get_text("/static/style.css")
+    assert '.mic-dot' in css
+    dot_idx = css.find('.mic-dot')
+    brace_open = css.find('{', dot_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'animation' in rule
+
+
+def test_mic_btn_has_transition():
+    """.mic-btn must define a transition for smooth state changes."""
+    css, _ = get_text("/static/style.css")
+    mic_btn_idx = css.find('.mic-btn{')
+    if mic_btn_idx == -1:
+        mic_btn_idx = css.find('.mic-btn ')
+    brace_open = css.find('{', mic_btn_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'transition' in rule
+
+
+# ── boot.js ──────────────────────────────────────────────────────────────
+
+
+def test_boot_js_serves_ok():
+    """boot.js must be served successfully."""
+    _, status = get_text("/static/boot.js")
+    assert status == 200
+
+
+def test_boot_js_speech_recognition_check():
+    """boot.js must check for SpeechRecognition (with webkit fallback)."""
+    js, _ = get_text("/static/boot.js")
+    assert 'SpeechRecognition' in js
+    assert 'webkitSpeechRecognition' in js
+
+
+def test_boot_js_recognition_config():
+    """boot.js must configure recognition.continuous, interimResults, and lang."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.continuous' in js
+    assert 'recognition.interimResults' in js
+    assert 'recognition.lang' in js
+
+
+def test_boot_js_recognition_not_continuous():
+    """recognition.continuous must be false (auto-stop after silence)."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.continuous=false' in js or 'recognition.continuous = false' in js
+
+
+def test_boot_js_recognition_interim_results():
+    """recognition.interimResults must be true (live transcription preview)."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.interimResults=true' in js or 'recognition.interimResults = true' in js
+
+
+def test_boot_js_recognition_lang_en():
+    """recognition.lang must be set to en-US."""
+    js, _ = get_text("/static/boot.js")
+    assert "recognition.lang='en-US'" in js or 'recognition.lang = "en-US"' in js or "recognition.lang='en-US'" in js
+
+
+def test_boot_js_onresult_handler():
+    """boot.js must define recognition.onresult to handle transcription."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.onresult' in js
+
+
+def test_boot_js_onend_handler():
+    """boot.js must define recognition.onend to reset state when recording stops."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.onend' in js
+
+
+def test_boot_js_onerror_handler():
+    """boot.js must define recognition.onerror for graceful error handling."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.onerror' in js
+
+
+def test_boot_js_not_allowed_error_message():
+    """onerror must handle 'not-allowed' with a user-friendly message."""
+    js, _ = get_text("/static/boot.js")
+    assert 'not-allowed' in js
+    assert 'permission' in js.lower() or 'denied' in js.lower() or 'access' in js.lower()
+
+
+def test_boot_js_no_speech_error_message():
+    """onerror must handle 'no-speech' with a user-friendly message."""
+    js, _ = get_text("/static/boot.js")
+    assert 'no-speech' in js
+
+
+def test_boot_js_network_error_message():
+    """onerror must handle 'network' error."""
+    js, _ = get_text("/static/boot.js")
+    assert "'network'" in js or '"network"' in js
+
+
+def test_boot_js_mic_active_flag():
+    """boot.js must track recording state via _micActive flag."""
+    js, _ = get_text("/static/boot.js")
+    assert '_micActive' in js
+
+
+def test_boot_js_mic_recording_class_toggle():
+    """boot.js must toggle 'recording' CSS class on the mic button."""
+    js, _ = get_text("/static/boot.js")
+    assert "'recording'" in js or '"recording"' in js
+
+
+def test_boot_js_mic_status_toggle():
+    """boot.js must show/hide #micStatus during recording."""
+    js, _ = get_text("/static/boot.js")
+    assert 'micStatus' in js
+
+
+def test_boot_js_send_stops_mic():
+    """btnSend onclick must stop mic before sending (send guard)."""
+    js, _ = get_text("/static/boot.js")
+    # The send button onclick should check _micActive and stop recording
+    send_onclick_idx = js.find("$('btnSend').onclick")
+    assert send_onclick_idx != -1
+    # Find the handler code — check that _micActive check appears near send assignment
+    handler_end = js.find(';', send_onclick_idx)
+    handler = js[send_onclick_idx:handler_end + 1]
+    assert '_micActive' in handler or 'stopMic' in handler.lower()
+
+
+def test_boot_js_btn_mic_onclick():
+    """boot.js must attach an onclick handler to btnMic."""
+    js, _ = get_text("/static/boot.js")
+    assert 'btn.onclick' in js or "btnMic.onclick" in js or "$('btnMic').onclick" in js
+
+
+def test_boot_js_recognition_start():
+    """boot.js must call recognition.start() to begin recording."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.start()' in js
+
+
+def test_boot_js_recognition_stop():
+    """boot.js must call recognition.stop() to end recording."""
+    js, _ = get_text("/static/boot.js")
+    assert 'recognition.stop()' in js
+
+
+def test_boot_js_iife_guard():
+    """Mic logic must be wrapped in an IIFE so it doesn't pollute global scope."""
+    js, _ = get_text("/static/boot.js")
+    # IIFE pattern: (function(){...})() or (() => {...})()
+    assert '(function(){' in js or '(function () {' in js
+
+
+def test_boot_js_browser_unsupported_return():
+    """boot.js must bail out (return) early when SpeechRecognition is unavailable."""
+    js, _ = get_text("/static/boot.js")
+    # The IIFE should have an early return when SpeechRecognition is falsy
+    assert 'if(!SpeechRecognition)' in js or 'if (!SpeechRecognition)' in js
+
+
+def test_boot_js_shows_mic_button_when_supported():
+    """boot.js must set display='' on btnMic when SpeechRecognition is available."""
+    js, _ = get_text("/static/boot.js")
+    assert "btn.style.display=''" in js or 'btn.style.display = ""' in js
+
+
+def test_boot_js_show_toast_on_error():
+    """boot.js must call showToast() for mic errors."""
+    js, _ = get_text("/static/boot.js")
+    assert 'showToast' in js
+
+
+def test_boot_js_autoresize_called():
+    """boot.js must call autoResize() after updating textarea from transcript."""
+    js, _ = get_text("/static/boot.js")
+    assert 'autoResize()' in js
+
+
+# ── Append behaviour (fix: mic appends to existing text, not replace) ────
+
+
+def test_boot_js_prefix_variable_declared():
+    """boot.js must declare _prefix variable to snapshot pre-existing textarea content."""
+    js, _ = get_text("/static/boot.js")
+    assert "_prefix" in js
+
+
+def test_boot_js_prefix_captured_on_start():
+    """_prefix must be set from ta.value when the user starts recording."""
+    js, _ = get_text("/static/boot.js")
+    # _prefix assignment must happen in the btn.onclick else branch (before recognition.start)
+    btn_onclick_idx = js.find("btn.onclick")
+    btn_onclick_end = js.find("};", btn_onclick_idx)
+    onclick_body = js[btn_onclick_idx:btn_onclick_end]
+    assert "_prefix=ta.value" in onclick_body or "_prefix = ta.value" in onclick_body
+
+
+def test_boot_js_onresult_prepends_prefix():
+    """onresult must include _prefix when writing to textarea (append, not replace)."""
+    js, _ = get_text("/static/boot.js")
+    onresult_idx = js.find("recognition.onresult")
+    onresult_end = js.find("};", onresult_idx)
+    onresult_body = js[onresult_idx:onresult_end]
+    # ta.value must be set to _prefix + something, not just the transcript alone
+    assert "_prefix" in onresult_body
+
+
+def test_boot_js_onend_commits_with_prefix():
+    """onend must commit _prefix + _finalText so appended text survives after recognition ends."""
+    js, _ = get_text("/static/boot.js")
+    onend_idx = js.find("recognition.onend")
+    onend_end = js.find("};", onend_idx)
+    onend_body = js[onend_idx:onend_end]
+    assert "_prefix" in onend_body
+
+
+def test_boot_js_prefix_reset_on_stop():
+    """_prefix must be reset when recording stops so next session starts clean."""
+    js, _ = get_text("/static/boot.js")
+    # _setRecording(false) clears both _finalText and _prefix
+    set_rec_idx = js.find("function _setRecording")
+    set_rec_end = js.find("}", set_rec_idx) + 1
+    fn_body = js[set_rec_idx:set_rec_end]
+    assert "_prefix" in fn_body
+
+
+def test_boot_js_auto_space_between_prefix_and_transcript():
+    """onend must insert a space between existing text and new transcript when needed."""
+    js, _ = get_text("/static/boot.js")
+    onend_idx = js.find("recognition.onend")
+    onend_end = js.find("};", onend_idx)
+    onend_body = js[onend_idx:onend_end]
+    # Should handle spacing — look for trimStart or endsWith(' ') check
+    has_spacing = ("trimStart" in onend_body or "endsWith(' ')" in onend_body
+                   or "endsWith(\" \")" in onend_body or "endsWith('\\n')" in onend_body)
+    assert has_spacing, "onend should handle spacing between prefix and new transcript"
+
+
+# ── Regression: existing behaviour unchanged ──────────────────────────────
+
+
+def test_attach_button_still_wired():
+    """btnAttach onclick must still be wired up (no regression)."""
+    js, _ = get_text("/static/boot.js")
+    assert "$('btnAttach').onclick" in js
+
+
+def test_file_input_onchange_still_wired():
+    """fileInput onchange must still be wired up (no regression)."""
+    js, _ = get_text("/static/boot.js")
+    assert "$('fileInput').onchange" in js
+
+
+def test_index_html_still_has_send_button():
+    """btnSend must still be present in index.html (no regression)."""
+    html, _ = get_text("/")
+    assert 'id="btnSend"' in html
+
+
+def test_index_html_still_has_attach_button():
+    """btnAttach must still be present in index.html (no regression)."""
+    html, _ = get_text("/")
+    assert 'id="btnAttach"' in html

tests/test_sprint20b.py

@@ -0,0 +1,343 @@
+"""
+Sprint 21 Tests: Send button polish — hidden until content, pop-in animation,
+icon-only circle design.
+"""
+import re
+import urllib.request
+
+BASE = "http://127.0.0.1:8788"
+
+
+def get_text(path):
+    with urllib.request.urlopen(BASE + path, timeout=10) as r:
+        return r.read().decode(), r.status
+
+
+# ── index.html ────────────────────────────────────────────────────────────
+
+
+def test_send_button_present():
+    """btnSend must still exist in the DOM."""
+    html, status = get_text("/")
+    assert status == 200
+    assert 'id="btnSend"' in html
+
+
+def test_send_button_hidden_by_default():
+    """btnSend must start hidden (display:none) — only shown when there is content."""
+    html, _ = get_text("/")
+    btn_match = re.search(r'id="btnSend"[^>]*>', html)
+    assert btn_match, "btnSend element not found"
+    assert 'display:none' in btn_match.group(0)
+
+
+def test_send_button_no_text_label():
+    """Send button must be icon-only — no visible 'Send' text label."""
+    html, _ = get_text("/")
+    # Find the full button element (from opening tag to closing tag)
+    btn_open_end = html.find('>', html.find('id="btnSend"')) + 1
+    btn_end = html.find('</button>', btn_open_end) + len('</button>')
+    btn_inner = html[btn_open_end:btn_end]
+    # Strip SVG content and any remaining tags; check visible text
+    no_svg = re.sub(r'<svg[^>]*>.*?</svg>', '', btn_inner, flags=re.DOTALL)
+    visible_text = re.sub(r'<[^>]+>', '', no_svg).strip()
+    assert visible_text == '', f"Send button has visible text: {visible_text!r}"
+
+
+def test_send_button_has_svg_icon():
+    """Send button must have an SVG icon."""
+    html, _ = get_text("/")
+    btn_start = html.find('id="btnSend"')
+    btn_end = html.find('</button>', btn_start) + len('</button>')
+    btn_html = html[btn_start:btn_end]
+    assert '<svg' in btn_html
+
+
+def test_send_button_has_title_attribute():
+    """btnSend must have a title attribute for accessibility (replaces text label)."""
+    html, _ = get_text("/")
+    btn_match = re.search(r'id="btnSend"[^>]*>', html)
+    assert btn_match
+    assert 'title=' in btn_match.group(0)
+
+
+def test_send_button_svg_arrow_up():
+    """Send button SVG should use an upward arrow (line + polyline or path)."""
+    html, _ = get_text("/")
+    btn_start = html.find('id="btnSend"')
+    btn_end = html.find('</button>', btn_start) + len('</button>')
+    btn_html = html[btn_start:btn_end]
+    # Must have some directional shape element
+    has_shape = ('<line' in btn_html or '<polyline' in btn_html or
+                 '<polygon' in btn_html or '<path' in btn_html)
+    assert has_shape, "Send button SVG missing directional shape"
+
+
+# ── style.css ────────────────────────────────────────────────────────────
+
+
+def test_send_btn_is_circle():
+    """send-btn must use border-radius:50% for the circle shape."""
+    css, status = get_text("/static/style.css")
+    assert status == 200
+    send_idx = css.find('.send-btn{')
+    brace_open = css.find('{', send_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'border-radius:50%' in rule or 'border-radius: 50%' in rule
+
+
+def test_send_btn_fixed_dimensions():
+    """send-btn must have explicit width and height (icon-circle, not text-padded)."""
+    css, _ = get_text("/static/style.css")
+    send_idx = css.find('.send-btn{')
+    brace_open = css.find('{', send_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'width:' in rule or 'width :' in rule
+    assert 'height:' in rule or 'height :' in rule
+
+
+def test_send_btn_no_old_padding():
+    """send-btn must not use text padding layout (old pill style removed)."""
+    css, _ = get_text("/static/style.css")
+    send_idx = css.find('.send-btn{')
+    brace_open = css.find('{', send_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    # Old style used padding:7px 18px — should be gone
+    assert 'padding:7px' not in rule and 'padding: 7px' not in rule
+
+
+def test_send_btn_blue_background():
+    """send-btn background must use the blue accent (#7cb9ff or similar)."""
+    css, _ = get_text("/static/style.css")
+    send_idx = css.find('.send-btn{')
+    brace_open = css.find('{', send_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert '7cb9ff' in rule or '5ba8f5' in rule or 'var(--blue)' in rule
+
+
+def test_send_btn_has_transition():
+    """send-btn must have transition for smooth hover/active states."""
+    css, _ = get_text("/static/style.css")
+    send_idx = css.find('.send-btn{')
+    brace_open = css.find('{', send_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'transition' in rule
+
+
+def test_send_btn_has_box_shadow():
+    """send-btn must have a box-shadow glow effect."""
+    css, _ = get_text("/static/style.css")
+    send_idx = css.find('.send-btn{')
+    brace_open = css.find('{', send_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'box-shadow' in rule
+
+
+def test_send_btn_hover_has_scale():
+    """send-btn:hover must use transform:scale for a satisfying hover effect."""
+    css, _ = get_text("/static/style.css")
+    hover_idx = css.find('.send-btn:hover{')
+    brace_open = css.find('{', hover_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'scale' in rule
+
+
+def test_send_btn_active_shrinks():
+    """send-btn:active must scale down slightly for tactile press feedback."""
+    css, _ = get_text("/static/style.css")
+    active_idx = css.find('.send-btn:active{')
+    brace_open = css.find('{', active_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'scale' in rule
+
+
+def test_send_btn_disabled_rule_exists():
+    """send-btn:disabled must still be styled."""
+    css, _ = get_text("/static/style.css")
+    assert '.send-btn:disabled' in css
+
+
+def test_send_btn_visible_class_defined():
+    """.send-btn.visible class must be defined for the pop-in animation."""
+    css, _ = get_text("/static/style.css")
+    assert '.send-btn.visible' in css
+
+
+def test_send_pop_in_keyframes_defined():
+    """@keyframes send-pop-in must be defined."""
+    css, _ = get_text("/static/style.css")
+    assert 'send-pop-in' in css
+    assert '@keyframes' in css
+
+
+def _extract_keyframe(css, name):
+    """Extract the full @keyframes block for the given animation name."""
+    # Find '@keyframes <name>' directly (forward search) to avoid hitting
+    # an earlier keyframe when multiple are defined on the same line.
+    kf_start = css.find('@keyframes ' + name)
+    assert kf_start != -1, f"@keyframes {name} not found in CSS"
+    depth = 0
+    kf_end = kf_start
+    for i, ch in enumerate(css[kf_start:], kf_start):
+        if ch == '{':
+            depth += 1
+        elif ch == '}':
+            depth -= 1
+            if depth == 0:
+                kf_end = i
+                break
+    return css[kf_start:kf_end]
+
+
+def test_send_pop_in_uses_scale():
+    """send-pop-in keyframe must animate from a scaled-down state."""
+    css, _ = get_text("/static/style.css")
+    kf_rule = _extract_keyframe(css, 'send-pop-in')
+    assert 'scale' in kf_rule
+
+
+def test_send_pop_in_uses_opacity():
+    """send-pop-in keyframe must fade in (opacity transition)."""
+    css, _ = get_text("/static/style.css")
+    kf_rule = _extract_keyframe(css, 'send-pop-in')
+    assert 'opacity' in kf_rule
+
+
+def test_send_btn_mobile_override_no_padding():
+    """Mobile override for send-btn must not add text padding (keeps circle shape)."""
+    css, _ = get_text("/static/style.css")
+    # Find the @media block
+    media_idx = css.find('@media')
+    send_mobile_idx = css.find('.send-btn', media_idx)
+    if send_mobile_idx == -1:
+        return  # No mobile override, fine
+    brace_open = css.find('{', send_mobile_idx)
+    brace_close = css.find('}', brace_open)
+    rule = css[brace_open:brace_close]
+    assert 'padding:' not in rule and 'font-size' not in rule
+
+
+# ── ui.js ─────────────────────────────────────────────────────────────────
+
+
+def test_ui_js_update_send_btn_function():
+    """ui.js must define updateSendBtn() function."""
+    js, status = get_text("/static/ui.js")
+    assert status == 200
+    assert 'function updateSendBtn' in js
+
+
+def test_update_send_btn_checks_content():
+    """updateSendBtn must check textarea value length."""
+    js, _ = get_text("/static/ui.js")
+    fn_idx = js.find('function updateSendBtn')
+    fn_end = js.find('\n}', fn_idx) + 2
+    fn_body = js[fn_idx:fn_end]
+    assert 'msg' in fn_body
+    assert '.value' in fn_body
+    assert '.length' in fn_body or '.trim()' in fn_body
+
+
+def test_update_send_btn_checks_pending_files():
+    """updateSendBtn must also show send button when files are attached."""
+    js, _ = get_text("/static/ui.js")
+    fn_idx = js.find('function updateSendBtn')
+    fn_end = js.find('\n}', fn_idx) + 2
+    fn_body = js[fn_idx:fn_end]
+    assert 'pendingFiles' in fn_body
+
+
+def test_update_send_btn_uses_visible_class():
+    """updateSendBtn must add .visible class to trigger the pop-in animation."""
+    js, _ = get_text("/static/ui.js")
+    fn_idx = js.find('function updateSendBtn')
+    fn_end = js.find('\n}', fn_idx) + 2
+    fn_body = js[fn_idx:fn_end]
+    assert 'visible' in fn_body
+
+
+def test_update_send_btn_uses_display_none():
+    """updateSendBtn must hide the button with display:none when no content."""
+    js, _ = get_text("/static/ui.js")
+    fn_idx = js.find('function updateSendBtn')
+    fn_end = js.find('\n}', fn_idx) + 2
+    fn_body = js[fn_idx:fn_end]
+    assert 'display' in fn_body
+    assert 'none' in fn_body
+
+
+def test_set_busy_calls_update_send_btn():
+    """setBusy must call updateSendBtn() so button hides while agent is responding."""
+    js, _ = get_text("/static/ui.js")
+    busy_idx = js.find('function setBusy')
+    busy_end = js.find('\n}', busy_idx) + 2
+    busy_body = js[busy_idx:busy_end]
+    assert 'updateSendBtn' in busy_body
+
+
+def test_render_tray_calls_update_send_btn():
+    """renderTray must call updateSendBtn() so button appears when files are attached."""
+    js, _ = get_text("/static/ui.js")
+    tray_idx = js.find('function renderTray')
+    tray_end = js.find('\n}', tray_idx) + 2
+    tray_body = js[tray_idx:tray_end]
+    assert 'updateSendBtn' in tray_body
+
+
+# ── boot.js ──────────────────────────────────────────────────────────────
+
+
+def test_boot_js_input_calls_update_send_btn():
+    """boot.js input event listener must call updateSendBtn()."""
+    js, status = get_text("/static/boot.js")
+    assert status == 200
+    assert 'updateSendBtn' in js
+
+
+# ── messages.js ───────────────────────────────────────────────────────────
+
+
+def test_auto_resize_calls_update_send_btn():
+    """autoResize() must call updateSendBtn() so button hides after send clears textarea."""
+    js, status = get_text("/static/messages.js")
+    assert status == 200
+    assert 'updateSendBtn' in js
+
+
+# ── Regression: existing behaviour unchanged ──────────────────────────────
+
+
+def test_send_button_still_has_send_btn_class():
+    """btnSend must still carry class='send-btn' for CSS targeting."""
+    html, _ = get_text("/")
+    assert 'class="send-btn"' in html
+
+
+def test_ui_js_set_busy_still_disables_btn():
+    """setBusy must still set btnSend.disabled (not just hide it)."""
+    js, _ = get_text("/static/ui.js")
+    busy_idx = js.find('function setBusy')
+    busy_end = js.find('\n}', busy_idx) + 2
+    busy_body = js[busy_idx:busy_end]
+    assert "btnSend" in busy_body
+    assert 'disabled' in busy_body
+
+
+def test_index_html_attach_button_unchanged():
+    """btnAttach must still be present (no regression)."""
+    html, _ = get_text("/")
+    assert 'id="btnAttach"' in html
+
+
+def test_send_function_still_exists():
+    """send() function must still be defined in messages.js."""
+    js, _ = get_text("/static/messages.js")
+    assert 'async function send()' in js