Skip to content

RemixDesktop Chainlink CRE Websocket Server#7194

Open
ioedeveloper wants to merge 58 commits into
masterfrom
chainlink-cre
Open

RemixDesktop Chainlink CRE Websocket Server#7194
ioedeveloper wants to merge 58 commits into
masterfrom
chainlink-cre

Conversation

@ioedeveloper
Copy link
Copy Markdown
Collaborator

No description provided.

@ioedeveloper ioedeveloper requested a review from yann300 May 9, 2026 15:11
@netlify
Copy link
Copy Markdown

netlify Bot commented May 9, 2026

Deploy Preview for reliable-cocada-166884 ready!

Name Link
🔨 Latest commit acea887
🔍 Latest deploy log https://app.netlify.com/projects/reliable-cocada-166884/deploys/6a1de67e14190a000826fe22
😎 Deploy Preview https://deploy-preview-7194--reliable-cocada-166884.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

this.terminalApi.logHtml(html)
}

logGeneratedKey(data) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This could maybe be a little bit more generic...

@yann300 yann300 requested a review from hsy822 May 20, 2026 13:23
import { shell } from "electron";
import crypto from "crypto";
import { isE2ELocal } from "src/main";
import { isE2ELocal } from "../main";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please double check that change.

Copy link
Copy Markdown
Collaborator

@hsy822 hsy822 left a comment

Choose a reason for hiding this comment

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

Overall, I think the CRE inline /frontend idea makes sense. However, this branch is behind the latest QuickDapp lifecycle changes. After rebasing, are you planning to keep CRE as separate inline handling spread across the MCP handler, DappManager, preview, and deploy code? Or do you plan to implement it as a first-class target mode within the QuickDapp create/update/finalize/deploy pipeline?

@yann300 yann300 requested a review from hsy822 May 22, 2026 07:07
@hsy822
Copy link
Copy Markdown
Collaborator

hsy822 commented May 25, 2026

When clicking the Delete DApp button on the DApp detail page, there are cases where all files and folders are deleted except for the frontend folder inside default_workspace. As a result, the DApp still appears on the dashboard and is also shown in the preview. In some other cases, the files are not deleted at all, but the DApp no longer appears on the dashboard. Depending on the case, the DApp may not be deleted at all.

@hsy822
Copy link
Copy Markdown
Collaborator

hsy822 commented May 25, 2026

When clicking Back(to dashboard) from the DApp detail page, the screen should be captured and the captured preview should appear on the dashboard, but it does not.

@hsy822
Copy link
Copy Markdown
Collaborator

hsy822 commented May 25, 2026

  1. Create an inline DApp.
Screenshot 2026-05-25 at 10 31 59 AM
  1. When clicking Delete All DApps, the frontend folder and its files remain.
Screenshot 2026-05-25 at 10 32 36 AM
  1. If you refresh the page in that state, the folder and file contents change.
Screenshot 2026-05-25 at 10 32 43 AM

`- Fallback: config.title || 'My DApp'\n`

// Design rules are intentionally lower priority than build/runtime correctness.
const QUICKDAPP_DESIGN_RULES =
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why did you remove that?

export class FinalizeDAppGenerationHandler extends BaseToolHandler {
name = 'finalize_dapp_generation'
description = 'Finalize a DApp after ALL files have been written using write_file. This updates the config, notifies the UI, and opens the DApp preview. MUST be called after generate_dapp + write_file sequence is complete.'
description = 'Finalize a DApp after ALL files have been written using file_write. This updates the config, notifies the UI, and opens the DApp preview. MUST be called after generate_dapp + file_write sequence is complete.'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tis is actually write_file

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.

3 participants