commit initial 09-12-2025 30

src/App.js

@@ -109,7 +109,7 @@ function App() {
     LANGUAGE_OPTIONS.find((l) => currentNode?.name?.endsWith(l.ext)) ||
     LANGUAGE_OPTIONS[0];
 
-  // ---------- Tree helpers ----------
+  // ---------- TREE helpers ----------
   const collectFolderPaths = (node, acc = []) => {
     if (!node) return acc;
     if (node.type === "folder") acc.push(node.path || "");
@@ -122,6 +122,7 @@ function App() {
     const filename = window.prompt("Filename (with extension):", "untitled.js");
     if (!filename) return;
 
+    // Determine default parent
     const selected = getNodeByPath(tree, activePath);
     let parentPath = "";
     if (selected?.type === "folder") parentPath = selected.path;
@@ -267,7 +268,7 @@ function App() {
     }
   };
 
-  // Initial run — NO browser prompt; rely on XTerm and accumStdin
+  // Initial run — DO NOT call run if code needs interactive input and we have no accumStdin.
   const handleRun = async () => {
     const node = getNodeByPath(tree, activePath);
     if (!node || node.type !== "file") {
@@ -280,27 +281,26 @@ function App() {
       return;
     }
 
-    // If code likely needs input and accumStdin empty, do NOT prompt — use XTerm
+    // If code likely needs input and we have no accumulated stdin, don't run — wait for user input in XTerm.
     const needs = codeNeedsInput(node.content, selectedLang);
-    let stdinToSend = accumStdin || stdin || "";
+    const haveAccum = !!(accumStdin && accumStdin.length > 0);
+    const haveLegacyStdin = !!(stdin && stdin.length > 0);
 
-    // If it needs input but accumStdin is empty, set awaitingInput true so user can type in XTerm
-    if (needs && !stdinToSend) {
-      // Inform user in terminal output and allow them to type in XTerm
+    if (needs && !haveAccum && !haveLegacyStdin) {
+      // Show helpful message in terminal and enable awaitingInput so user can type into XTerm.
       setOutput("[Interactive program detected — type input directly into the terminal]");
       setAwaitingInput(true);
-      // We still attempt a run with empty stdin to let the program show initial prompts if any.
-      // This may produce EOF errors for some programs; user can then type into the terminal to continue.
-    } else {
-      // reset terminal output for fresh run but keep accumulated stdin
-      resetTerminal(true);
-      setOutput(`[Running with stdin length=${stdinToSend ? stdinToSend.length : 0}]\n`);
+      return; // <-- important: do NOT run now
     }
 
+    // Otherwise run immediately with whichever stdin we have (accumStdin or legacy stdin)
+    const stdinToSend = accumStdin || stdin || "";
+
+    resetTerminal(true); // keep accumStdin
+    setOutput(`[Running with stdin length=${stdinToSend ? stdinToSend.length : 0}]\n`);
     setIsRunning(true);
     setProblems([]);
     try {
-      // Use local stdinToSend to avoid setState race
       const res = await runCode(node.content, selectedLang, stdinToSend);
       const out = res.output ?? "";
       setOutput(out);
@@ -313,7 +313,7 @@ function App() {
     } catch (err) {
       const e = String(err);
       setOutput(e);
-      // if it fails due to EOF, still allow user to type in XTerm
+      // Allow user to type into terminal after EOF errors
       setAwaitingInput(true);
     } finally {
       setIsRunning(false);