Skip to content

fix: export exists and prox_str from top-level paradedb package - #69

Merged
isaacvando merged 1 commit into
mainfrom
fix/export-exists-prox-str
Jun 15, 2026
Merged

fix: export exists and prox_str from top-level paradedb package#69
isaacvando merged 1 commit into
mainfrom
fix/export-exists-prox-str

Conversation

@philippemnoel

Copy link
Copy Markdown
Member

Bug

`exists` and `prox_str` are both implemented in `paradedb/sqlalchemy/search.py` but were missing from the re-export block and `all` in `paradedb/init.py`. This caused an `ImportError` for any user following the flat-import convention that every other function in the package supports:

```python
from paradedb import exists # ImportError
from paradedb import prox_str # ImportError
```

Fix

Added both symbols to the import statement and to `all` in `paradedb/init.py`, in alphabetical order consistent with the surrounding exports. No other changes.

Verification

  • Confirmed both symbols exist in `paradedb/sqlalchemy/search.py` (lines 227 and 258)
  • `python3 -c "import ast; ast.parse(open('paradedb/init.py').read())"` — passes
  • `ruff check paradedb/init.py` — all checks passed
  • `python3 -c "import paradedb; paradedb.exists; paradedb.prox_str"` — succeeds

Both `exists` and `prox_str` were implemented in
`paradedb/sqlalchemy/search.py` but were not re-exported in
`paradedb/__init__.py`, causing `ImportError` when users did
`from paradedb import exists` or `from paradedb import prox_str`.

Added both symbols to the import block and to `__all__`, in alphabetical
order consistent with the surrounding exports.

@isaacvando isaacvando left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🎉

@isaacvando
isaacvando merged commit ba0710d into main Jun 15, 2026
15 checks passed
@isaacvando
isaacvando deleted the fix/export-exists-prox-str branch June 15, 2026 14:42
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.

2 participants