fix: pass documents as keyword argument in EmbeddingBasedDocumentSplitter

[Amnah199]

Related Issues

• fixes https://linear.app/deepset/issue/BUI-422/code-component-embeddingbaseddocumentsplitter-cannot-be-connected-to

EmbeddingBasedDocumentSplitter._calculate_embeddings calls document_embedder.run with a positional argument. This fails when the embedder's run() method only accepts **kwargs
(e.g. deepset Cloud's Code component wrapper) raising:

TypeError: Code.run() takes 1 positional argument but 2 were given.

Note: There is an example pipeline with logs in the linear ticket.

Proposed Changes:

Changed both call sites to use keyword arguments.

How did you test it?

Notes for the reviewer

Checklist

• I have read the contributors guidelines and the code of conduct.
• I have updated the related issue with new insights and changes.
• I have added unit tests and updated the docstrings.
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
• I have documented my code.
• I have added a release note file, following the contributors guidelines.
• I have run pre-commit hooks and fixed any issue.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
haystack-docs	Ignored		Apr 1, 2026 10:36pm

[sjrl]

@Amnah199 I think this is technically a fine change, but just as a heads up the protocol for DocumentEmbedder does indicate that the embedder should accept at least one positional argument called documents. So it would probably also be good to update the embedder causing the issue to change documents to a positional argument.

[Amnah199]

The issue was addressed in https://github.com/deepset-ai/deepset-cloud-custom-nodes/pull/604.
Closing this PR because the update is not needed.