fix: Fix issue with icon themes not loading for preview nodes

[Joffcom]

Summary

INodeTypeDescription supports iconUrl which works for local paths but for community node previews we were always expecting just one url, This PR adds support for the themes while also keeping the option to have a single iconUrl.

For testing set ENVIRONMENT=staging to use the staging Strapi instance then search for Binalyze or Air

Light Mode

Dark Mode

---

This change will also require a workflow update for publishing community nodes to make sure we set the below

iconUrl: {
  light: 'xx',
  dark: 'yy'
}

The current setting below does still work so we can merge and release before we make any changes.

iconUrl: 'xx'

---

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/NODE-3396/get-dark-and-light-icons-rendering-for-cnoc

Review / Merge checklist

• PR title and summary are descriptive. (conventions)
• Docs updated or follow-up ticket created.
• Tests included.
• PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

[cubic-dev-ai]

cubic analysis

No issues found across 2 files. Review in cubic

[bundlemon]

BundleMon

Files added (2)

Status	Path	Size	Limits
✅	WASM Dependencies tree-sitter-bash.wasm	+181.26KB	-
✅	WASM Dependencies tree-sitter.wasm	+74.47KB	-

Total files change +255.73KB

Groups added (2)

Status	Path	Size	Limits
✅	**/*.js	+5.34MB	-
✅	**/*.css	+187.85KB	-

Final result: ✅

View report in BundleMon website ➡️

---

Current branch size history

[michael-radency]

LGTM

[currents-bot]

✅

E2E Tests: n8n tests passed after 4m 39s

🟢 508 · 🔴 0 · ⚪️ 0

View Run Details

Run Details

• Project: n8n

• Groups: 1

• Framework: Currents

• Run Status: Passed

• Commit: 0035f05

• Spec files: 105

• Overall tests: 508

• Duration: 4m 39s

• Parallelization: 1

---

This message was posted automatically by currents.dev | Integration Settings

[janober]

Got released with [email protected]