Skip to content

Add Code node to web frontend code language detection#2221

Draft
Copilot wants to merge 3 commits intofeat/ts-backend-migrationfrom
copilot/uncomment-code-node-and-dependency
Draft

Add Code node to web frontend code language detection#2221
Copilot wants to merge 3 commits intofeat/ts-backend-migrationfrom
copilot/uncomment-code-node-and-dependency

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

The Code node (nodetool.code.Code) and isolated-vm dependency are already present and active in the codebase — fully implemented, exported, registered, and tested (131 tests). The only gap was the web frontend's syntax highlighting.

  • Add nodetool.code.Code to determineCodeLanguage() in StringProperty.tsx so the Code node property editor gets JavaScript syntax highlighting, matching the existing ExecuteJavaScript handling
if (
  nodeType === "nodetool.code.ExecuteJavaScript" ||
  nodeType === "nodetool.code.Code"
) {
  return "javascript";
}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: georgi <19498+georgi@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 19, 2026

Deploying nodetool-staging with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2bd8dc8
Status:🚫  Build failed.

View logs

Co-authored-by: georgi <19498+georgi@users.noreply.github.com>
Copilot AI changed the title [WIP] Uncomment Code node and isolated-vm dependency Add Code node to web frontend code language detection Mar 19, 2026
Copilot AI requested a review from georgi March 19, 2026 12:01
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 19, 2026

Deploying nodetool with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2bd8dc8
Status:🚫  Build failed.

View logs

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