still depends hard on llm

[daniel-baumann]

Hi,

thank you very much for working on this. Unfortunately, litecli 1.17.1 still fails with llm not installed:

$ litecli
Traceback (most recent call last):
  File "/usr/bin/litecli", line 5, in <module>
    from litecli.main import cli
  File "/usr/lib/python3/dist-packages/litecli/main.py", line 38, in <module>
    from .completion_refresher import CompletionRefresher
  File "/usr/lib/python3/dist-packages/litecli/completion_refresher.py", line 7, in <module>
    from .packages.special.main import COMMANDS
  File "/usr/lib/python3/dist-packages/litecli/packages/special/__init__.py", line 22, in <module>
    from . import llm
ImportError: cannot import name 'llm' from partially initialized module 'litecli.packages.special' (most likely due to a circular import) (/usr/lib/python3/dist-packages/litecli/packages/special/__init__.py)

Regards,
Daniel

[amjith]

What is the output of litecli --version?

[daniel-baumann]

$ litecli --version
Traceback (most recent call last):
  File "/usr/bin/litecli", line 5, in <module>
    from litecli.main import cli
  File "/usr/lib/python3/dist-packages/litecli/main.py", line 38, in <module>
    from .completion_refresher import CompletionRefresher
  File "/usr/lib/python3/dist-packages/litecli/completion_refresher.py", line 7, in <module>
    from .packages.special.main import COMMANDS
  File "/usr/lib/python3/dist-packages/litecli/packages/special/__init__.py", line 22, in <module>
    from . import llm
ImportError: cannot import name 'llm' from partially initialized module 'litecli.packages.special' (most likely due to a circular import) (/usr/lib/python3/dist-packages/litecli/packages/special/__init__.py)
$

I'm building 1.17.1 in order to upload it in Debian (I'm the Debian maintainer for litecli), I've uploaded it with patching out llm but it would be nice, to have it fixed properly.

Regards,
Daniel

[amjith]

@daniel-baumann Thank you for maintaining the DEB package. 🙏

I can't seem to reproduce this. Here's what I get when I install it in a fresh new virtualenv (tempenv-7a3c258845c38):

(tempenv-7a3c258845c38)
~
❯ uv pip freeze
Using Python 3.12.9 environment at: .virtualenvs/tempenv-7a3c258845c38
cli-helpers==2.9.0
click==8.3.1
configobj==5.0.9
litecli==1.17.1
pip==25.3
prompt-toolkit==3.0.52
pygments==2.19.2
sqlparse==0.5.5
tabulate==0.9.0
wcwidth==0.5.3
(tempenv-7a3c258845c38)
~
❯ litecli --version
litecli, version 1.17.1
(tempenv-7a3c258845c38)
~
❯ litecli :memory:
LiteCli: 1.17.1 (SQLite: 3.47.1)
GitHub: https://github.com/dbcli/litecli
:memory:> select "hello"
+---------+
| "hello" |
+---------+
| hello   |
+---------+
1 row in set
Time: 0.001s
:memory:>

[jkerhin]

I am not a package maintainer so I could be missing something, but this is what I think is going on:

1. A previous release of litecli had a dependency on llm, and llm is not packaged for Debian
2. A Debian bug was filed noting that the Debian release of litecli wasn't working due to the dependency noted in 1.
3. Daniel updated the Debian release to patch out the litecli/packages/special/llm.py file from the Debian release link
4. The 1.17.1 release of litecli did remove the dependency on llm from pyproject.toml, moving it to the [ai] extras
5. Daniel's Debian build process is still removing the litecli/packages/special/llm.py file, which is a "no-op" on a system without llm installed. When that file is removed in v1.17.1. I can get the exact error that Daniel is seeing

From a fresh Debian Trixie vm:

root@scw-nervous-mestorf:~/workspace/clean# ./venv/bin/pip freeze
cli_helpers==2.9.0
click==8.3.1
configobj==5.0.9
litecli==1.17.1
prompt_toolkit==3.0.52
Pygments==2.19.2
sqlparse==0.5.5
tabulate==0.9.0
wcwidth==0.5.3
root@scw-nervous-mestorf:~/workspace/clean# ./venv/bin/litecli --help
Usage: litecli [OPTIONS] [DATABASE]

  A SQLite terminal client with auto-completion and syntax highlighting.

  Examples:
    - litecli lite_database

Options:
  -V, --version           Show the version and exit.
  -D, --database TEXT     Database to use.
  -R, --prompt TEXT       Prompt format (Default: "\d> ").
  -l, --logfile FILENAME  Log every query and its results to a file.
  --liteclirc FILE        Location of liteclirc file.
  --auto-vertical-output  Automatically switch to vertical output mode if the
                          result is wider than the terminal width.
  -t, --table             Display batch output in table format.
  --csv                   Display batch output in CSV format.
  --warn / --no-warn      Warn before running a destructive query.
  -e, --execute TEXT      Execute command and quit.
  --help                  Show this message and exit.
root@scw-nervous-mestorf:~/workspace/clean# rm venv/lib/python3.13/site-packages/litecli/packages/special/llm.py
root@scw-nervous-mestorf:~/workspace/clean# ./venv/bin/litecli --help
Traceback (most recent call last):
  File "/root/workspace/clean/./venv/bin/litecli", line 5, in <module>
    from litecli.main import cli
  File "/root/workspace/clean/venv/lib/python3.13/site-packages/litecli/main.py", line 38, in <module>
    from .completion_refresher import CompletionRefresher
  File "/root/workspace/clean/venv/lib/python3.13/site-packages/litecli/completion_refresher.py", line 7, in <module>
    from .packages.special.main import COMMANDS
  File "/root/workspace/clean/venv/lib/python3.13/site-packages/litecli/packages/special/__init__.py", line 22, in <module>
    from . import llm
ImportError: cannot import name 'llm' from partially initialized module 'litecli.packages.special' (most likely due to a circular import) (/root/workspace/clean/venv/lib/python3.13/site-packages/litecli/packages/special/__init__.py)

I believe that Daniel should be able to remove the patches as part of the Debian release that upstreams the v1.17.1 build, but again I am not a maintainer and can't speak to the correct 'flow' for Debian releases.