browser-assistant 2.8.22 → 2.8.23

package/extension/background.js

@@ -286,10 +286,10 @@ chrome.runtime.onMessage.addListener((request, sender, sendResponse) => {
                     return { ok: false, error: 'The portal submit command and IDE workspace do not match.' };
                 if (request.action === 'runOnlineIde' && (!ide.active || typeof request.code !== 'string'
                     || !request.code.trim() || new TextEncoder().encode(request.code).length > 12000)) {
-                    return { ok: false, error: 'The matching IDE must be active and the source must fit the terminal input limit.' };
+                    return { ok: false, error: 'The matching IDE must be active and the app.js source must be at most 12 KB.' };
                 }
                 const results = await chrome.scripting.executeScript({ target: { tabId: ide.id, allFrames: true }, world: 'MAIN',
-                    func: browserAssistantOnlineIdeTerminal,
+                    func: browserAssistantOnlineIde,
                     args: [request.action, workspace, questionId, request.token, request.action === 'runOnlineIde' ? request.code : null] });
                 const frameResults = results.map(entry => entry.result).filter(Boolean);
                 return frameResults.length === 1 ? frameResults[0] : { ok: false, retry: true, error: 'The IDE workspace is still opening.' };

package/extension/content.js

@@ -1211,6 +1211,8 @@ function mountJournalCountdown(signal) {
                         if (result?.done) {
                             if (result.passed)
                                 automation.completed = 1;
+                            else if (result.saveUnconfirmed)
+                                automation.error = 'Code was pasted into app.js, but the saved file did not match. Check app.js and save before running it.';
                             else
                                 automation.error = result.submitted ? 'Grading finished. Review the test results in the IDE.' : 'The IDE command finished without a confirmed submission. Check the terminal.';
                             stopAutomation(result.passed ? 'COMPLETED' : 'CHECK_IDE_RESULT');

package/extension/editor-bridge.js

@@ -88,9 +88,9 @@ function browserAssistantEditor(operation, selector, expected, value) {
         return { ok: false, error: error.message };
     }
 }
-// Runs in the portal IDE frame only. The terminal command is built here from
-// fixed operations and encoded source, never from model-generated shell text.
-async function browserAssistantOnlineIdeTerminal(action, workspace, questionId, token, source) {
+// Run the IDE's normal paste and autosave flow. The terminal only verifies and
+// grades the saved file; generated source is never passed to the shell.
+async function browserAssistantOnlineIde(action, workspace, questionId, token, source) {
     if (location.origin !== 'https://ide.ccbp.in' || location.pathname.replace(/\/$/, '') !== `/playground/${workspace}`)
         return null;
     const visible = node => node.getClientRects().length > 0;
@@ -101,6 +101,7 @@ async function browserAssistantOnlineIdeTerminal(action, workspace, questionId,
     const rowContainer = input.closest('.xterm')?.querySelector('.xterm-rows');
     const rows = () => [...(rowContainer?.children || [])].map(row => row.textContent.trim()).filter(Boolean);
     const finishMarker = `BA_RESULT_${token}`;
+    const saveMarker = `BA_APP_JS_NOT_SAVED_${token}`;
     const reportPrefix = `https://learning.ccbp.in/ide-coding-submission/${questionId}/`;
     const reports = lines => {
         const text = lines.join('');
@@ -119,7 +120,7 @@ async function browserAssistantOnlineIdeTerminal(action, workspace, questionId,
         const report = index >= 0 && reports(lines.slice(index)).find(id => !previous.reports.includes(id));
         const passed = result === `${finishMarker}:0` && Boolean(report) && index > 0
             && lines.slice(Math.max(0, index - 3), index).includes('All test cases passed');
-        return { ok: true, done: true, passed, submitted: Boolean(report), url: report ? reportPrefix + report : null };
+        return { ok: true, done: true, passed, submitted: Boolean(report), saveUnconfirmed: lines.includes(saveMarker), url: report ? reportPrefix + report : null };
     }
     if (previous?.token === token)
         return { ok: false, error: 'This IDE run was already started. Check the terminal.' };
@@ -146,35 +147,60 @@ async function browserAssistantOnlineIdeTerminal(action, workspace, questionId,
         return { ok: false, retry: true, error: 'Waiting for the workspace app.js file in Explorer.' };
     files[0].click();
     await new Promise(resolve => setTimeout(resolve, 200));
-    const editors = [...document.querySelectorAll('.monaco-editor[data-uri]')].filter(visible);
-    const editor = editors.length === 1 && editors[0];
-    if (!editor || !editor.getAttribute('data-uri').endsWith(`/${prompt[2]}/app.js`)) {
+    const findEditor = () => {
+        const editors = [...document.querySelectorAll('.monaco-editor[data-uri]')].filter(visible);
+        const node = editors.length === 1 && editors[0];
+        const uri = node && node.getAttribute('data-uri');
+        return uri && uri.startsWith('file:') && uri.endsWith(`/${prompt[2]}/app.js`) ? node : null;
+    };
+    const editor = findEditor();
+    if (!editor) {
         return { ok: false, retry: true, error: 'Waiting for the app.js editor.' };
     }
+    const editorInput = editor.querySelector('textarea.inputarea');
     const renderedCode = editor.querySelector('.view-lines');
-    if (!renderedCode || renderedCode.textContent.trim())
-        return { ok: false, error: 'app.js already contains code. Review the prepared result before replacing it.' };
-    const bytes = new TextEncoder().encode(source);
-    const encoded = btoa(Array.from(bytes, byte => String.fromCharCode(byte)).join(''));
+    if (!editorInput || !renderedCode?.querySelector('.view-line'))
+        return { ok: false, retry: true, error: 'Waiting for the app.js editor input.' };
+    const model = window.monaco?.editor?.getEditors?.().find(instance => instance.getDomNode() === editor)?.getModel();
+    const isEmpty = () => {
+        if (model)
+            return model.getValue() === '';
+        // The portal bundles Monaco without a global editor API. Require the
+        // visible first and only row to be blank; never select or replace text.
+        const lineNumbers = editor.querySelectorAll('.margin-view-overlays .line-numbers');
+        return lineNumbers.length === 1 && lineNumbers[0].textContent.trim() === '1'
+            && renderedCode.querySelectorAll('.view-line').length === 1 && !renderedCode.textContent.trim();
+    };
+    if (!isEmpty())
+        return { ok: false, error: 'app.js is not empty or its contents could not be checked. Review the code in the editor.' };
+    const code = source.replace(/\r\n?/g, '\n');
+    const bytes = new TextEncoder().encode(code);
     const digest = Array.from(new Uint8Array(await crypto.subtle.digest('SHA-256', bytes)), byte => byte.toString(16).padStart(2, '0')).join('');
-    const writer = [
-        'const fs = require("node:fs")',
-        'const [encoded, digest, folder] = process.argv.slice(1)',
-        'const code = Buffer.from(encoded, "base64")',
-        'if (require("node:crypto").createHash("sha256").update(code).digest("hex") !== digest) throw Error("Incomplete source received")',
-        'if (!process.cwd().endsWith("/nodejs/coding-practices/" + folder)) throw Error("Unexpected project directory")',
-        'const stat = fs.lstatSync("app.js")',
-        'if (!stat.isFile() || fs.readFileSync("app.js", "utf8").trim()) throw Error("app.js is not an empty regular file")',
-        'const check = require("node:child_process").spawnSync(process.execPath, ["--check"], {input: code})',
-        'if (check.status !== 0) throw Error(String(check.stderr || "JavaScript syntax check failed"))',
-        'fs.writeFileSync("app.js", code)',
-        'if (!fs.readFileSync("app.js").equals(code)) throw Error("Could not verify the saved file")',
-    ].join('; ');
-    // All arguments are restricted to letters, digits, or base64 by construction.
+    if (document.visibilityState !== 'visible' || findEditor() !== editor || !isEmpty()) {
+        return { ok: false, error: 'The app.js editor changed before pasting.' };
+    }
+    editorInput.focus();
+    if (document.activeElement !== editorInput || editorInput.readOnly || editorInput.disabled) {
+        return { ok: false, error: 'The app.js editor is not available for editing.' };
+    }
+    const pasteData = new DataTransfer();
+    pasteData.setData('text/plain', code);
+    editorInput.dispatchEvent(new ClipboardEvent('paste', { clipboardData: pasteData, bubbles: true, cancelable: true, composed: true }));
+    const pasteDeadline = Date.now() + 1500;
+    const didPaste = () => model ? model.getValue() === code : Boolean(renderedCode.textContent.trim());
+    while (!didPaste() && Date.now() < pasteDeadline && document.activeElement === editorInput && findEditor() === editor) {
+        await new Promise(resolve => setTimeout(resolve, 50));
+    }
+    if (!didPaste() || findEditor() !== editor || document.activeElement !== editorInput || document.visibilityState !== 'visible') {
+        return { ok: false, error: 'The IDE did not confirm the paste into app.js. Check the editor before trying again.' };
+    }
+    // Check exact saved bytes before grading; allow time for the IDE's autosave.
+    // All interpolated values have restricted alphabets.
+    const saved = `printf '%s  app.js\\n' '${digest}' | sha256sum --check --status`;
     const echoMarker = `BA_COMMAND_${token}`;
-    const command = `node -e '${writer}' '${encoded}' '${digest}' '${prompt[2]}' && ccbp submit ${workspace}; printf '\\n${finishMarker}:%s\\n' "$?" #${echoMarker}`;
+    const command = `( for ba_try in 1 2 3 4 5 6 7 8 9 10; do ${saved} && break; sleep 0.5; done; if ${saved}; then node --check app.js && ccbp submit ${workspace}; else printf '%s\\n' ${saveMarker}; false; fi ); printf '\\n${finishMarker}:%s\\n' "$?" #${echoMarker}`;
     if (rows().at(-1) !== before.at(-1))
-        return { ok: false, error: 'The IDE terminal changed before submission.' };
+        return { ok: false, error: 'Code is in app.js, but the IDE terminal changed before grading.' };
     input.focus();
     if (document.activeElement !== input)
         return { ok: false, error: 'The IDE terminal could not be focused.' };

package/extension/manifest.json

@@ -1,7 +1,7 @@
 {
     "manifest_version": 3,
     "name": "Browser Assistant",
-    "version": "2.8.22",
+    "version": "2.8.23",
     "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.22",
+  "version": "2.8.23",
   "description": "Chrome extension and local Python backend for exploring webpage questions and coding exercises",
   "license": "MIT",
   "repository": {

package/server/main.py

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