Skip to content

Conversation

@cstuncsik
Copy link
Contributor

@cstuncsik cstuncsik commented Jul 28, 2025

Summary

Replaces misleading "No path back to node" error with helpful "Execute node for preview" message when referencing nodes without execution data.
Also fixes AI/tool node path detection regression and adds cycle detection for node traversal.

Related Linear tickets, Github issues, and Community forum posts

PAY-3168

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)

… improve error handling in workflow-data-proxy
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic analysis

No issues found across 2 files. Review in cubic

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Jul 28, 2025
@cstuncsik cstuncsik changed the title fix(editor): Replace misleading "No path back to node" error with helpful execution message fix(core): Replace misleading "No path back to node" error with helpful execution message Jul 28, 2025
@cstuncsik cstuncsik requested a review from ivov July 29, 2025 13:41
Copy link
Member

@ivov ivov left a comment

Choose a reason for hiding this comment

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

Change looks clean, tested runners working.

@currents-bot
Copy link

currents-bot bot commented Jul 29, 2025

E2E Tests: n8n tests passed after 5m 39.7s

🟢 508 · 🔴 0 · ⚪️ 0

View Run Details

Run Details

  • Project: n8n

  • Groups: 1

  • Framework: Currents

  • Run Status: Passed

  • Commit: 895d350

  • Spec files: 105

  • Overall tests: 508

  • Duration: 5m 39.7s

  • Parallelization: 1


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

@cstuncsik cstuncsik merged commit 164800f into master Jul 29, 2025
34 checks passed
@cstuncsik cstuncsik deleted the pay-3168-no-path-back-to-node-bug-when-manual-prod-executions-clearly branch July 29, 2025 15:22
@CharlieKolb CharlieKolb added the release/backport Changes that need to be backported to older releases. label Aug 1, 2025
CharlieKolb pushed a commit that referenced this pull request Aug 1, 2025
@github-actions github-actions bot mentioned this pull request Aug 1, 2025
@janober
Copy link
Member

janober commented Aug 1, 2025

Got released with n8n@1.105.1

cstuncsik added a commit that referenced this pull request Aug 1, 2025
despairblue pushed a commit that referenced this pull request Aug 4, 2025
@github-actions github-actions bot mentioned this pull request Aug 4, 2025
r00gm added a commit that referenced this pull request Aug 15, 2025
…th helpful execution message (#17759)"

This reverts commit 164800f.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team release/backport Changes that need to be backported to older releases. Released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants