Skip to content

Added “Run in Sandbox” & “Analyze Code” Functionality 🚀#78

Open
Chhavi-16 wants to merge 6 commits intoSushma-1706:mainfrom
Chhavi-16:pr-47
Open

Added “Run in Sandbox” & “Analyze Code” Functionality 🚀#78
Chhavi-16 wants to merge 6 commits intoSushma-1706:mainfrom
Chhavi-16:pr-47

Conversation

@Chhavi-16
Copy link
Copy Markdown
Contributor

Description:

🖥️ Run in Sandbox: Execute user code safely and get stdout, stderr, and exit codes.

🧠 Analyze & Explain Code: Send code to the AI backend and display structured explanations.

⏳ Added loading states and error handling for smooth user experience.

🌐 Supports multiple languages: JavaScript, Python, Java, C, C++.

Impact:

💡 Users can now run and analyze code directly from the editor.

🔧 Prepares the editor for future features like visualization & step-by-step execution.

Testing:

✅ Manual tests with sample code for each language.

✅ Verified correct output, error handling, and mode-specific analysis.

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 6, 2025

@Chhavi-16 is attempting to deploy a commit to the sushma-1706's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Chhavi-16
Copy link
Copy Markdown
Contributor Author

@Sushma-1706 check it once

@Sushma-1706
Copy link
Copy Markdown
Owner

Hi @Chhavi-16 👋

Thanks for your contribution! I checked out the branch and tried running the project. I ran into a few issues that need fixing before we can merge:

Duplicate dependencies in package.json

Some packages like @radix-ui/react-dialog, @radix-ui/react-progress, @radix-ui/react-tabs, etc. are listed twice with different versions.

Please keep only the latest version for each and remove duplicates.

Build errors in Dashboard.tsx

ActiveComponent is declared twice → remove the duplicate.

Some mismatched tags (

... and

...).

There are stray parentheses/extra } causing JSX errors.

A JSX comment seems to have an extra ) before it.

Because of these issues, npm run dev fails with compilation errors. Could you please clean these up and test locally before updating the PR? That way the app will build and run correctly when merged 🚀

Thanks again for the work — once these fixes are made, it’ll be good to merge! 🙌

@Sushma-1706
Copy link
Copy Markdown
Owner

can you share the screenshot of the working of Buttons

@Chhavi-16
Copy link
Copy Markdown
Contributor Author

ok

@Chhavi-16
Copy link
Copy Markdown
Contributor Author

i want to say that first you have to click on file index.js and open with integrated terminal then run node index.js then after that you have to pen terminal of codeviz folder and npm run dev
Screenshot 2025-09-07 162550
you can check it

@Chhavi-16
Copy link
Copy Markdown
Contributor Author

@Sushma-1706 did you check it , tell me if there is still any issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants