Skip to content

Fix CLI framework integ && add mcp && update audio endpoint behavoir #6534

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
yann300 wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Fix CLI framework integ && add mcp && update audio endpoint behavoir #6534

yann300 wants to merge 23 commits into from

Conversation

@yann300
Copy link
Contributor

@yann300 yann300 commented Nov 10, 2025 •
edited
Loading

Clicking compile from the udapp plugin, will initiate a compilation in foundry.
to test it:

  • run desktop.
  • switch to a foundry project.
  • click on compile.
  • the terminal should display messages from foundry.

Other update:

  • improve how remix listen on compilation result from hardhat/foundry (e.g only emit compilationFinished for the current file.
  • Solidity plugin UI: selecting compile with hardhat/foundry will no longer trigger a Remix compilation.
  • Fix solidity scan with desktop.
  • Add mcp endpoint for foundry/hardhat.
  • When transcribing, append the text in the chat box.

@yann300 yann300 requested a review from bunsenstraat November 10, 2025 20:52
@netlify
Copy link

netlify bot commented Nov 10, 2025 •
edited
Loading

✅ Deploy Preview for reliable-cocada-166884 ready!

Name Link
🔨 Latest commit 1f5ce71
🔍 Latest deploy log https://app.netlify.com/projects/reliable-cocada-166884/deploys/6939317e26ea3b0008568cf3
😎 Deploy Preview https://deploy-preview-6534--reliable-cocada-166884.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Aniket-Engg
Aniket-Engg reviewed Nov 21, 2025
View reviewed changes
Copy link
Collaborator

@Aniket-Engg Aniket-Engg left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It shows logs from foundry compilation.

Although while loading a foundry project, it shows both:

Foundry plugin activated
Hardhat plugin activated

clicking on Editor compile button doesn't run foundry compilation even if checkbox is selected in compiler plugin. Compiling using keyboard shortcut runs foundry compilation too

Other points: Remix creates two config files: remix.config.json & remix-compiler.config.js . Then there is a foundry.toml file already in the project. I think foundry considers foundry.toml file to pick configuration.

This can be confusing for the users and should be simplified, may be in other PR.

@yann300 yann300 requested a review from Aniket-Engg December 1, 2025 19:25
@yann300 yann300 mentioned this pull request Dec 4, 2025
Closed
ci-bot added 23 commits December 10, 2025 09:25
foundry compile
bdadf93
remove logs
08d9b78
update plugin manager postion and fix layout
462d3e8
fix layout
0486822
let the framework to use default settings
05d5b42
fix logging and do not compile with remix when external compiler are set
06fb6e2
remove listening..
bd972cf
fix emitting new compiled contracts
0e3fa5a
update label && fix foundry comp
6c6a958
fix solidity scan
61de784
remove code
c79eec0
fix hardhat comp
2f01c86
listen on build done
98a53bc
add listening to hardhat change
0e23b4f
disable remix compiler if external framework
2e1b292
fix solidity scan call in desktop
51db687
rm log
30102b6
rm logs
60ff39b
check file exist before emitting
8c47eb7
log and fix ui
3ec2e8b
foundry hardhat mcp handler
c45bb49
when transcribing, append to the chat box
b8ae798
remove log & cleanup
1f5ce71
@yann300 yann300 changed the title Foundry compile Fix CLI framework integ && add mcp && update audio endpoint behavoir Dec 10, 2025
@nx-cloud
Copy link

nx-cloud bot commented Dec 10, 2025 •
edited
Loading

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.

View your CI Pipeline Execution ↗ for commit 1f5ce71

Command Status Duration Result
nx lint remix-core-plugin ❌ Failed 2s View ↗
nx build plugin_api --configuration=production ... ✅ Succeeded <1s View ↗
nx lint solidity-compiler ✅ Succeeded <1s View ↗
nx lint remix-ws-templates ✅ Succeeded <1s View ↗
nx lint remix-url-resolver ✅ Succeeded <1s View ↗
nx lint contract-verification ✅ Succeeded <1s View ↗
nx run-many --target=test --projects=remix-anal... ✅ Succeeded 1m 43s View ↗
nx build remix-ide --cloud --verbose ✅ Succeeded 2m 13s View ↗
Additional runs (1) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2025-12-10 11:38:29 UTC

@ci-pr-comment
Copy link

ci-pr-comment bot commented Dec 10, 2025 •
edited
Loading

❌ E2E failures detected (workflow: web)

Last run: Wed, Dec 10, 2025, 11:55 AM UTC

View HTML report

Top failing tests (4/4):

  • Should execute `readFile` api from file manager external api (fileManager_api_group2.test)
  • run Remixd tests (remixd_group1.test)
  • Should load using compiler from link passed in remix URL (url_group3.test)
  • Import Github folder from URL params (url_group4.test)

Report generated by CI; artifacts are retained per CircleCI retention settings.

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

Reviewers

@bunsenstraat bunsenstraat Awaiting requested review from bunsenstraat

@Aniket-Engg Aniket-Engg Awaiting requested review from Aniket-Engg

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yann300 @Aniket-Engg