Skip to content

Conversation

@simonw
Copy link
Owner

@simonw simonw commented Nov 24, 2025

simonw and others added 7 commits November 24, 2025 08:44
ndjson.org has an expired domain.
Closes #659

The --functions option now accepts:
- File paths ending in .py (e.g., --functions my_funcs.py)
- Multiple invocations (e.g., --functions foo.py --functions 'def bar(): ...')
- Inline Python code (existing behavior)

Implementation follows the same pattern as llm's --functions flag
(simonw/llm@a880c123).

Changes:
- Added multiple=True to --functions Click option in query, bulk, and memory commands
- Modified _register_functions() to detect and read .py files
- Updated _maybe_register_functions() to iterate over multiple function sources
- Removed unused bytes/bytearray handling
- Added comprehensive tests for file paths and multiple invocations
- Updated documentation with examples

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Shorter help for --functions

---------

Co-authored-by: Claude <noreply@anthropic.com>
@simonw simonw mentioned this pull request Nov 24, 2025
Copy link
Owner Author

@simonw simonw left a comment

Choose a reason for hiding this comment

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

Need to make these changes.

@simonw
Copy link
Owner Author

simonw commented Nov 24, 2025

Had Claude Code apply those fixes: https://gistpreview.github.io/?f4c89636cc58fc7bf9820c06f2488b91

Apply changes from the review on this PR https://github.com/simonw/sqlite-utils/pull/689

@simonw simonw marked this pull request as ready for review November 24, 2025 18:24
@simonw simonw merged commit 07409df into 3.x Nov 24, 2025
75 checks passed
@simonw simonw deleted the 3.39-fixes branch November 24, 2025 18:44
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.

4 participants