Skip to content

fix: remove stale State entry from haystack.dataclasses lazy import structure#11621

Open
Kropiunig wants to merge 2 commits into
deepset-ai:mainfrom
Kropiunig:fix/remove-stale-state-lazy-import
Open

fix: remove stale State entry from haystack.dataclasses lazy import structure#11621
Kropiunig wants to merge 2 commits into
deepset-ai:mainfrom
Kropiunig:fix/remove-stale-state-lazy-import

Conversation

@Kropiunig

Copy link
Copy Markdown

PR #9578 removed the state submodule (haystack/dataclasses/state.py) and its TYPE_CHECKING import, but left the "state": ["State"] entry in the _import_structure dict that drives the lazy importer. As a result State still appears in dir(haystack.dataclasses), yet from haystack.dataclasses import State raises ModuleNotFoundError: No module named 'haystack.dataclasses.state'. This removes the stale entry so the public namespace matches reality (the real State now lives in haystack.components.agents).

@Kropiunig Kropiunig requested a review from a team as a code owner June 14, 2026 05:14
@Kropiunig Kropiunig requested review from sjrl and removed request for a team June 14, 2026 05:14
@vercel

vercel Bot commented Jun 14, 2026

Copy link
Copy Markdown

@Kropiunig is attempting to deploy a commit to the deepset Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant

CLAassistant commented Jun 14, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ sjrl
❌ Kropiunig
You have signed the CLA already but the status is still pending? Let us recheck it.

@sjrl

sjrl commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Hey @Kropiunig thanks for the contribution! Please sign the CLA agreement #11621 (comment) otherwise we can't accept your contribution.

@sjrl sjrl self-assigned this Jun 15, 2026
@sjrl sjrl added the ignore-for-release-notes PRs with this flag won't be included in the release notes. label Jun 15, 2026
@sjrl

sjrl commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Hey @Kropiunig just checking in if you saw my previous comment

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

Labels

ignore-for-release-notes PRs with this flag won't be included in the release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants