Fix Practice Speaking: strip SOLA_NEXT tags from voice transcript (v1.0.21)

The transcript display was showing raw [SOLA_NEXT]...[/SOLA_NEXT] tags because
stripSolaTags() was called on individual streaming tokens — the full tag spans
many tokens so the regex never matched. Fixed by accumulating tokens in a
displayBuffer and only emitting text up to any [SOLA_NEXT] marker.

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

Author: caswelltom

amd/build/voice.min.js

@@ -74,6 +74,10 @@ define(['local_ai_course_assistant/sse_client'], function(SSE) {
     var sseController = null;
     /** @type {string} SSE token accumulation buffer for sentence extraction */
     var sentenceBuffer = '';
+    /** @type {string} Accumulates tokens to find and hide the SOLA_NEXT block from transcript */
+    var displayBuffer = '';
+    /** @type {number} How many chars of displayBuffer have already been emitted to transcript */
+    var displayEmitted = 0;
     /** @type {number|null} Timer to restart recognition after TTS ends */
     var restartTimer = null;
     /** @type {string} Derived TTS proxy URL */
@@ -380,6 +384,8 @@ define(['local_ai_course_assistant/sse_client'], function(SSE) {
             onTranscriptCb('user', text);
         }
         sentenceBuffer = '';
+        displayBuffer = '';
+        displayEmitted = 0;
 
         var postBody = {
             sesskey:  cfg.sessKey  || '',
@@ -399,11 +405,15 @@ define(['local_ai_course_assistant/sse_client'], function(SSE) {
                 if (!connected) {
                     return;
                 }
-                // Show streaming text in voice transcript area (strips SOLA_NEXT inline).
-                var cleanToken = stripSolaTags(token);
-                if (cleanToken && onTranscriptCb) {
-                    onTranscriptCb('assistant', cleanToken);
+                // Accumulate for display. Only emit text that precedes [SOLA_NEXT] —
+                // the tag spans many tokens so single-token regex never matches.
+                displayBuffer += token;
+                var solaStart = displayBuffer.indexOf('[SOLA_NEXT]');
+                var visible = solaStart === -1 ? displayBuffer : displayBuffer.slice(0, solaStart);
+                if (visible.length > displayEmitted && onTranscriptCb) {
+                    onTranscriptCb('assistant', visible.slice(displayEmitted));
                 }
+                displayEmitted = visible.length;
                 sentenceBuffer += token;
                 maybeSendSentence();
             },

amd/src/voice.js

@@ -231,5 +231,12 @@ function xmldb_local_ai_course_assistant_upgrade($oldversion) {
         upgrade_plugin_savepoint(true, 2026030804, 'local', 'ai_course_assistant');
     }
 
+    if ($oldversion < 2026030805) {
+        // v1.0.21: Fix Practice Speaking transcript showing raw [SOLA_NEXT]...[/SOLA_NEXT]
+        // tags. Use display accumulator in voice.js so tag is detected across token boundaries.
+        // No schema changes needed.
+        upgrade_plugin_savepoint(true, 2026030805, 'local', 'ai_course_assistant');
+    }
+
     return true;
 }

db/upgrade.php

@@ -25,7 +25,7 @@
 defined('MOODLE_INTERNAL') || die();
 
 $plugin->component = 'local_ai_course_assistant';
-$plugin->version = 2026030804;
+$plugin->version = 2026030805;
 $plugin->requires = 2024100700; // Moodle 4.5+.
 $plugin->maturity = MATURITY_STABLE;
-$plugin->release = '1.0.20';
+$plugin->release = '1.0.21';