Skip to content

Remove unstructured-ingest as a dependency#63

Merged
brndnblck merged 3 commits intomainfrom
potter/remove-unstructured-ingest
Feb 17, 2026
Merged

Remove unstructured-ingest as a dependency#63
brndnblck merged 3 commits intomainfrom
potter/remove-unstructured-ingest

Conversation

@potter-potter
Copy link
Contributor

@potter-potter potter-potter commented Nov 19, 2025

Note

Replaces unstructured-ingest with utic-public-types, updates imports and error types, adds local file_data_from_dict using Pydantic, and bumps version to 0.0.42.

  • Core/API:
    • Replace unstructured_ingest types/errors with utic_public_types in etl_uvicorn/api_generator.py and related modules.
    • Add local file_data_from_dict (Pydantic ValidationError-based) to parse FileData/BatchFileData.
  • Tests/Schema:
    • Update imports in tests and schema (file_data, errors) to utic_public_types.
  • Packaging:
    • Remove unstructured-ingest dependency; add utic-public-types in pyproject.toml.
  • Versioning/Changelog:
    • Bump to 0.0.42; add changelog entry for dependency removal.

Written by Cursor Bugbot for commit 3320a22. This will update automatically on new commits. Configure here.

]

[[package]]
name = "unstructured-ingest"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

adios friend

@potter-potter potter-potter changed the title initial commit Remove unstructured-ingest as a dependency Nov 19, 2025
class EtlApiException(Exception):
pass

def file_data_from_dict(data: dict) -> FileDataType:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think i'll just move this into utic_public_types instead.

return BatchFileData.model_validate(data)
except ValidationError:
logger.debug(f"{data} not valid for batch file data")
return FileData.model_validate(data)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this might also need try-return approach.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I should have explained this better. I'm gonna remove this from here and just move it into the utic-public-types in file_data where it really belongs. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats why i made the other PR.

@brndnblck brndnblck merged commit 9b81254 into main Feb 17, 2026
12 checks passed
@brndnblck brndnblck deleted the potter/remove-unstructured-ingest branch February 17, 2026 07:22
@brndnblck brndnblck restored the potter/remove-unstructured-ingest branch February 17, 2026 17:58
brndnblck pushed a commit that referenced this pull request Feb 17, 2026
brndnblck added a commit that referenced this pull request Feb 17, 2026
This reverts commit 9b81254.

Co-authored-by: Brandon Black <brndnblck>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants