refactor: Add native Python runner runtime env var

[ivov]

https://linear.app/n8n/issue/CAT-1290

[cubic-dev-ai]

1 issue found across 5 files

_{React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.}

[bundlemon]

BundleMon

Unchanged files (2)

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

No change in files bundle size

Groups updated (2)

Status	Path	Size	Limits
✅	**/*.js	5.73MB (+370.51KB +6.74%)	-
✅	**/*.css	195.45KB (+4.2KB +2.19%)	-

Final result: ✅

View report in BundleMon website ➡️

---

Current branch size history | Target branch size history

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...ontend/editor-ui/src/components/ParameterInput.vue	0.00%	3 Missing ⚠️

📢 Thoughts on this report? Let us know!

[Cadiac]

Tested and confirmed that this works. A curious user might toggle that on regardless of UI hiding the option, but that shouldn't really matter (even on cloud) as it simply won't work, right?

[currents-bot]

✅

E2E Tests: n8n tests passed after 5m 31.9s

🟢 550 · 🔴 0 · ⚪️ 0

View Run Details

Run Details

• Project: n8n

• Groups: 2

• Framework: Playwright

• Run Status: Passed

• Commit: 66027d4

• Spec files: 112

• Overall tests: 550

• Duration: 5m 31.9s

• Parallelization: 3

Groups

GroupId	Results	Spec Files Progress
ui	🟢 153 · 🔴 0 · ⚪️ 0	39 / 39
No name	🟢 397 · 🔴 0 · ⚪️ 0	73 / 73

---

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

[ivov]

Tested and confirmed that this works. A curious user might toggle that on regardless of UI hiding the option, but that shouldn't really matter (even on cloud) as it simply won't work, right?

If a self-hoster enables it without a runner, the broker will wait indefinitely for a runner to take the task. Cloud is not a concern as we control the environment. Historically undocumented env vars haven't been an issue.

[ivov]

@Cadiac Once more please, I missed that e2e relies on action

[Cadiac]

Tested and confirmed that this works. A curious user might toggle that on regardless of UI hiding the option, but that shouldn't really matter (even on cloud) as it simply won't work, right?

If a self-hoster enables it without a runner, the broker will wait indefinitely for a runner to take the task. Cloud is not a concern as we control the environment. Historically undocumented env vars haven't been an issue.

Yeah, I'm mostly concerned about someone bypassing the check on the UI (pasting in a workflow with the python task runner enabled perhaps?) and executing their WF on cloud, would that break their instance? I'd hope only the WF execution would fail.

But at that point that's kind of the fault of the user, maybe not so once we start making the new runner more public

[ivov]

Tested and confirmed that this works. A curious user might toggle that on regardless of UI hiding the option, but that shouldn't really matter (even on cloud) as it simply won't work, right?

If a self-hoster enables it without a runner, the broker will wait indefinitely for a runner to take the task. Cloud is not a concern as we control the environment. Historically undocumented env vars haven't been an issue.

Yeah, I'm mostly concerned about someone bypassing the check on the UI (pasting in a workflow with the python task runner enabled perhaps?) and executing their WF on cloud, would that break their instance? I'd hope only the WF execution would fail.

But at that point that's kind of the fault of the user, maybe not so once we start making the new runner more public

Turns out even in that case they'll hit this error and won't hang.