Skip to content

fix: accept string paths in corpus extraction - #1389

Open
gaoflow wants to merge 2 commits into
Graphify-Labs:v8from
gaoflow:fix-1386-string-path-corpus
Open

fix: accept string paths in corpus extraction#1389
gaoflow wants to merge 2 commits into
Graphify-Labs:v8from
gaoflow:fix-1386-string-path-corpus

Conversation

@gaoflow

@gaoflow gaoflow commented Jun 19, 2026

Copy link
Copy Markdown

Summary

  • normalize string file paths to Path at the extract_corpus_parallel boundary before slicing and token packing
  • keep internal extraction unit annotations aligned with existing Path | FileSlice behavior
  • add a regression test for list[str] input

Fixes #1386

Testing

  • uv run pytest tests/test_chunking.py -q
  • uv run pytest tests/ -q
  • uv run ruff check graphify/llm.py tests/test_chunking.py
  • git diff --check
  • uv run graphify update .

AI assistance was used under my direction.

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.

extract_corpus_parallel raises AttributeError: 'str' object has no attribute 'parent' when passed string paths

1 participant