[pull] main from czlonkowski:main - #104
Merged
Merged
Conversation
* feat: add run and cancel actions to n8n_evaluations (#936) n8n 2.32.0 shipped the Public API endpoints for triggering and cancelling evaluation test runs, which unblocks the write half of the tool: run triggers POST /workflows/{id}/test-runs, cancel hits the /cancel subroute, both behind the new testRun:create/testRun:cancel API-key scopes. Error mapping distinguishes quota exhaustion (402), missing scope or unlicensed instance (403), missing evaluation trigger or already-finished run (409), and pre-2.32 instances - which answer run with 405 because the route exists with GET only. The gate re-reads the instance version on 404/405 instead of trusting the client cache. The tool loses readOnlyHint; run and cancel register as destructive operations, and the read-only deployment recipes block them. Conceived by Romuald Członkowski - www.aiadvisors.pl/en Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix: note the workflow:execute requirement in the n8n_evaluations tool description Copilot review round one: the tool doc and the 403 guidance both name the key owner's workflow:execute project scope for run/cancel, but the model-facing tool description omitted it. Conceived by Romuald Członkowski - www.aiadvisors.pl/en Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…#966) * docs: sync skills pack to n8n-skills v1.27.0 (evaluations run/cancel) The bundled skills pack still described n8n_evaluations as read-only and 2.30-only. Sync from n8n-skills v1.27.0 (commit d6d7772) updates the router one-liner, the tools-expert API list, and the WORKFLOW_GUIDE section with the run/cancel actions shipped in 2.67.0: per-action version gating, testRun scope + workflow:execute requirements, 402/403/405/409 semantics, and a side-effect warning before starting a run. Conceived by Romuald Członkowski - www.aiadvisors.pl/en Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * docs: correct cancel response and pre-2.32 status codes (n8n-skills v1.27.1) Copilot review findings: the cancel example now shows the actual return ({id, status: "cancelled"} plus the async-wind-down note), and the gotcha no longer claims cancel answers 405 pre-2.32 — that route does not exist at all and yields 404; only run's route exists GET-only. Conceived by Romuald Członkowski - www.aiadvisors.pl/en Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * docs: complete the 403 causes for n8n_evaluations (n8n-skills v1.27.2) Copilot round-two comments: the 403 guidance omitted the unlicensed-plan cause the tool's own error hints list, and the version phrasing was ambiguous with per-action minimums. All three causes and the per-action key requirements are now explicit in the router one-liner and the WORKFLOW_GUIDE intro and gotchas. Conceived by Romuald Członkowski - www.aiadvisors.pl/en Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * test: accept n8n's full execution-status vocabulary in live-instance assertions The live test instance now holds a canceled execution (left by the n8n_evaluations cancel smoke test), and the two integration assertions allowed only success/error/running/waiting — failing CI on unrelated changes. Accept every status n8n's API can return: canceled, crashed, new, and unknown included. Conceived by Romuald Członkowski - www.aiadvisors.pl/en Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * docs: round-three review polish (n8n-skills v1.27.3) 403 guidance says 'can mean' and includes the workflow-access cause, the tools-expert entry spells out per-action key-creation minimums, and the changelog disambiguates n8n-mcp 2.67.0 from an n8n version. Conceived by Romuald Członkowski - www.aiadvisors.pl/en Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )