Skip to content

Conversation

schrothbn
Copy link
Contributor

Summary

This PR corrects the output format of the call workflow tool for the execution via the engine. This should fix the output logs when using this tool.
Also adds tests for the execute-method.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/AI-1597/ai-agent-logs-dont-show-output-of-subworkflow-as-a-tool-making-it-hard

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)

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Oct 21, 2025
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@schrothbn schrothbn marked this pull request as ready for review October 21, 2025 12:16
@schrothbn schrothbn requested review from a team and burivuhster and removed request for a team October 21, 2025 12:16
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.

1 issue found across 2 files

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="packages/@n8n/nodes-langchain/nodes/tools/ToolWorkflow/ToolWorkflow.node.test.ts">

<violation number="1" location="packages/@n8n/nodes-langchain/nodes/tools/ToolWorkflow/ToolWorkflow.node.test.ts:84">
These tests spy on WorkflowToolService.prototype.createTool in multiple test cases without restoring the original method, so the second jest.spyOn call throws and the suite fails. Add jest.restoreAllMocks() (or reuse a shared spy) before re-spying.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant