browser-assistant 2.8.4 → 2.8.5

package/extension/automation-policy.js

@@ -379,6 +379,25 @@
             const unique = [...new Set(candidates)];
             return unique.length === 1 ? unique[0] : null;
         },
+        getPracticeSkip(doc, question) {
+            const anchor = question.form || question.container;
+            if (!anchor)
+                return null;
+            const skips = [...doc.querySelectorAll('button,[role="button"]')].filter(button => button.isConnected && button.getClientRects().length && !button.disabled
+                && button.getAttribute('aria-disabled') !== 'true' && !button.closest('#assistant-root')
+                && /^skip(?: question)?$/i.test((button.innerText || button.textContent || button.getAttribute('aria-label') || '').trim()));
+            if (skips.length !== 1)
+                return null;
+            for (let scope = anchor; scope && scope !== doc.body; scope = scope.parentElement) {
+                if (!scope.contains(skips[0]))
+                    continue;
+                const submits = [...scope.querySelectorAll('button,[role="button"],input[type="submit"]')].filter(button => button.getClientRects().length && !button.closest('#assistant-root')
+                    && /^submit(?: answer)?$/i.test((button.innerText || button.textContent || button.value || button.getAttribute('aria-label') || '').trim()));
+                if (submits.length === 1)
+                    return skips[0];
+            }
+            return null;
+        },
         getBlockingReason(doc) {
             const expired = [...doc.querySelectorAll('p,span,div,h1,h2,h3,h4')].some(node => node.getClientRects().length && !node.closest('#assistant-root') && !node.children.length
                 && /^time['’]s up!\s*you cannot attempt this assessment now\.?$/i.test((node.innerText || node.textContent || '').trim()));

package/extension/content.js

@@ -415,6 +415,7 @@
         const currentQuestion = automation.batch?.title || (automation.running && automation.codingQuestionText)
             || (authorized ? getAuthorizedQuestion(context)?.text : '');
         const pageName = authorized ? context.target.id : window.location.hostname;
+        const deferred = automation.deferredQuestions?.size ?? 0;
         if (status) {
             status.textContent = authorized
                 ? `Build ${extensionVersion} · ${context.target.id}: ${automation.state} · Completed: ${automation.completed}${automation.error ? ` · ${automation.error}` : ''}`
@@ -431,14 +432,15 @@
         if (question)
             question.textContent = currentQuestion || '—';
         if (progress)
-            progress.textContent = `${automation.completed} / —`;
+            progress.textContent = deferred ? `${automation.completed} completed · ${deferred} deferred` : `${automation.completed} / —`;
         if (latest) {
+            const reviewing = ['WAITING_FOR_REVIEW', 'DEFERRING_QUESTION', 'REVIEW_NEEDED'].includes(automation.state);
             latest.textContent = automation.error
-                ? `⚠ ${automation.error}`
+                ? reviewing ? automation.error : `⚠ ${automation.error}`
                 : authorized
                     ? `Current status: ${automation.state.replace(/_/g, ' ').toLowerCase()}.`
                     : 'Automation is available only on an authorized QA page.';
-            latest.classList.toggle('has-error', Boolean(automation.error));
+            latest.classList.toggle('has-error', Boolean(automation.error) && !reviewing);
         }
         if (start)
             start.disabled = !authorized || automation.running;
@@ -477,6 +479,8 @@
         automation.paused = false;
         automation.pendingNavigation = false;
         automation.state = reason;
+        if (reason === 'REVIEW_NEEDED')
+            automation.error = 'Review deferred questions on the portal.';
         updateAutomationUI();
     }
     async function waitWhilePaused(runId) {
@@ -1178,6 +1182,73 @@
                 stopAutomation('STOPPED');
         }
     }
+    async function continueAfterUnverifiedQuestion(question, runId, reason, allowSkip = true) {
+        if (!(await waitWhilePaused(runId)))
+            return false;
+        const initial = getAutomationContext();
+        const adapter = getAutomationAdapter(initial);
+        if (!adapter || initial.target.id !== automation.targetId) {
+            stopAutomation('UNAUTHORIZED');
+            return false;
+        }
+        const fresh = getAuthorizedQuestion(initial);
+        const sameQuestion = fresh?.signature === question.signature && hasSameQuestionOptions(question, fresh);
+        const practice = initial.target.id === 'ccbp-learning-practice'
+            && /^\/(?:default-question|primitive-coding(?:-review)?)\/practice\//.test(location.pathname);
+        const hasDraft = fresh?.input?.value?.trim() || fresh?.options.some(option => option.isSelected?.() || option.input?.checked);
+        if (allowSkip && practice && sameQuestion && !hasDraft && question.type !== 'ordering'
+            && !adapter.isSaving?.(document) && !adapter.getSubmittedAdvance?.(document)
+            && automationPolicy.canExecute('click', initial) && automationPolicy.canExecute('validate', initial)) {
+            const skip = adapter.getPracticeSkip?.(document, fresh);
+            if (skip) {
+                automation.state = 'DEFERRING_QUESTION';
+                automation.error = reason;
+                automation.pendingNavigation = true;
+                updateAutomationUI();
+                skip.click();
+                const transition = await waitForPracticeTransition(question.signature, runId);
+                if (!automation.running || automation.runId !== runId)
+                    return false;
+                automation.pendingNavigation = false;
+                if (transition === 'next' || transition === 'complete') {
+                    automation.deferredQuestions.add(question.signature);
+                    automation.error = '';
+                    return true;
+                }
+            }
+        }
+        automation.state = 'WAITING_FOR_REVIEW';
+        automation.error = `${reason} Use the page controls to review and move to the next question. Automation will continue here.`;
+        updateAutomationUI();
+        while (await waitWhilePaused(runId)) {
+            if (automation.state !== 'WAITING_FOR_REVIEW') {
+                automation.state = 'WAITING_FOR_REVIEW';
+                updateAutomationUI();
+            }
+            const context = getAutomationContext();
+            const currentAdapter = getAutomationAdapter(context);
+            if (context.mode !== 'FULL' || context.target.id !== initial.target.id || currentAdapter?.id !== adapter.id) {
+                stopAutomation('UNAUTHORIZED');
+                return false;
+            }
+            const blocked = currentAdapter.getBlockingReason?.(document);
+            if (blocked) {
+                automation.error = blocked;
+                stopAutomation('PAUSED');
+                return false;
+            }
+            if (!currentAdapter.isSaving?.(document)) {
+                const current = getAuthorizedQuestion(context);
+                if (currentAdapter.getSubmittedAdvance?.(document) || currentAdapter.isComplete(document)
+                    || (current && current.signature !== question.signature && !hasSameQuestionOptions(question, current))) {
+                    automation.error = '';
+                    return true;
+                }
+            }
+            await new Promise(resolve => setTimeout(resolve, 250));
+        }
+        return false;
+    }
     async function startPracticeAutomation() {
         if (window.BrowserAssistantJournal?.matches()) {
             await window.BrowserAssistantJournal.start();
@@ -1230,6 +1301,7 @@
         automation.paused = false;
         automation.error = '';
         automation.completed = 0;
+        automation.deferredQuestions = new Set();
         automation.targetId = initialContext.target.id;
         automation.pendingNavigation = false;
         automation.currentSignature = null;
@@ -1286,10 +1358,12 @@
                 const transition = await waitForPracticeTransition(previousSignature, runId);
                 if (!automation.running || automation.runId !== runId)
                     break;
-                if (transition === 'next' || transition === 'complete')
+                if (transition === 'next' || transition === 'complete') {
                     automation.completed += 1;
+                    automation.deferredQuestions.delete(previousSignature);
+                }
                 if (transition === 'complete') {
-                    stopAutomation('COMPLETED');
+                    stopAutomation(automation.deferredQuestions.size ? 'REVIEW_NEEDED' : 'COMPLETED');
                     break;
                 }
                 if (transition !== 'next') {
@@ -1300,6 +1374,10 @@
                 continue;
             }
             if (currentAdapter.isComplete(document)) {
+                if (automation.deferredQuestions.size) {
+                    stopAutomation('REVIEW_NEEDED');
+                    break;
+                }
                 const nextPractice = currentAdapter.getNextPractice?.(document);
                 if (!nextPractice) {
                     stopAutomation('COMPLETED');
@@ -1341,6 +1419,12 @@
                 await runCodingAutomation(question, runId);
                 break;
             }
+            if (automation.deferredQuestions.has(question.signature)) {
+                automation.currentSignature = question.signature;
+                if (await continueAfterUnverifiedQuestion(question, runId, 'This question was deferred earlier.', false))
+                    continue;
+                break;
+            }
             if (automation.processed.has(question.signature)) {
                 automation.error = 'Repeated question detected.';
                 stopAutomation('PAUSED');
@@ -1403,7 +1487,12 @@
                 if (!automation.running || automation.runId !== runId)
                     break;
                 automation.error = error.message || 'Local AI server unavailable.';
-                stopAutomation(['answer_unconfirmed', 'vision_unconfirmed'].includes(error.code) ? 'PAUSED' : 'ERROR');
+                if (['answer_unconfirmed', 'vision_unconfirmed'].includes(error.code)) {
+                    if (await continueAfterUnverifiedQuestion(question, runId, automation.error))
+                        continue;
+                }
+                else
+                    stopAutomation('ERROR');
                 break;
             }
             if (!automation.running || automation.runId !== runId || !response)
@@ -1474,6 +1563,11 @@
             // transient code-block markup.
             if (!answerIndexes.length || !current || !hasSameQuestionOptions(question, current)) {
                 automation.error = !answerIndexes.length ? 'The verified answer did not match the displayed options exactly.' : !current ? 'Question controls disappeared while solving.' : 'Question text or options changed while solving. Please retry.';
+                if (!answerIndexes.length && current?.signature === question.signature && hasSameQuestionOptions(question, current)) {
+                    if (await continueAfterUnverifiedQuestion(question, runId, automation.error))
+                        continue;
+                    break;
+                }
                 stopAutomation('PAUSED');
                 break;
             }
@@ -1590,7 +1684,7 @@
             }
             if (transition === 'complete') {
                 automation.completed += 1;
-                stopAutomation('COMPLETED');
+                stopAutomation(automation.deferredQuestions.size ? 'REVIEW_NEEDED' : 'COMPLETED');
                 break;
             }
             if (transition !== 'next') {

package/extension/manifest.json

@@ -1,7 +1,7 @@
 {
     "manifest_version": 3,
     "name": "Browser Assistant",
-    "version": "2.8.4",
+    "version": "2.8.5",
     "description": "Scans webpages for MCQs and coding problems. Powered by Groq (cloud) and Ollama (local fallback).",
     "permissions": [
         "activeTab",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browser-assistant",
-  "version": "2.8.4",
+  "version": "2.8.5",
   "description": "Chrome extension and local Python backend for exploring webpage questions and coding exercises",
   "license": "MIT",
   "repository": {

package/server/main.py

@@ -190,7 +190,7 @@ async def lifespan(_app: FastAPI):
             client.close()
 
 
-app = FastAPI(title="Browser Assistant API", version="2.8.4", lifespan=lifespan)
+app = FastAPI(title="Browser Assistant API", version="2.8.5", lifespan=lifespan)
 install_local_guards(app)
 
 @app.exception_handler(HTTPException)