llm: accept a directory argument, recursing for .md/.txt#209
Merged
Conversation
A directory passed to `assembly llm` now expands (via rglob, path-sorted) into the .md/.txt files it holds, each read under its own header just like explicit file arguments. An empty directory is a usage error raised before auth/network — the empty-guard the justfile's `transcripts/**/*.md(.N)` glob used to carry now that the CLI can. This lets `assembly llm "…" transcripts/` replace the glob-and-guard a caller would otherwise hand-roll. Split the file/directory-context tests out of test_llm_command.py into test_llm_files.py to stay under the 500-line gate. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LwoV9EXHTakfpw9E5hp89G
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A directory passed to
assembly llmnow expands (via rglob, path-sorted)into the .md/.txt files it holds, each read under its own header just like
explicit file arguments. An empty directory is a usage error raised before
auth/network — the empty-guard the justfile's
transcripts/**/*.md(.N)glob used to carry now that the CLI can.
This lets
assembly llm "…" transcripts/replace the glob-and-guard acaller would otherwise hand-roll. Split the file/directory-context tests
out of test_llm_command.py into test_llm_files.py to stay under the
500-line gate.
Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01LwoV9EXHTakfpw9E5hp89G