closecode-ai 0.4.0__tar.gz → 0.5.0__tar.gz

{closecode_ai-0.4.0 → closecode_ai-0.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: closecode-ai
-Version: 0.4.0
+Version: 0.5.0
 Summary: CloseCode — an agentic terminal coding assistant (LangGraph + OpenRouter + MCP)
 Author: Om Gite
 License-Expression: MIT
@@ -20,6 +20,8 @@ Requires-Dist: langchain-openrouter
 Requires-Dist: rich
 Requires-Dist: pyfiglet
 Requires-Dist: textual
+Provides-Extra: dev
+Requires-Dist: pytest>=8; extra == "dev"
 
 <div align="center">
 
@@ -99,6 +101,22 @@ closecode
 `pipx` is recommended over `pip` for the packaged version since it installs CLI tools into
 an isolated environment and puts them straight on your `PATH`.
 
+## Tests
+
+```bash
+pip install -e ".[dev]"
+pytest
+```
+
+429 tests covering guardrails, sandbox path resolution, undo/checkpoints, background
+processes, todo-store invariants, plan-mode tool filtering, session persistence, config
+permissions, and the search tools. The suite is fully isolated: `config.py` and
+`session.py` are redirected to temp dirs, no test touches the network, and nothing is
+written to your real `~/.closecode`.
+
+Some tests are marked `xfail` for **known bugs** rather than fixed behaviour — they
+document a gap and will flip to passing when it's fixed. Run `pytest -rx` to list them.
+
 ## Quick start
 
 1. Get a free API key at [openrouter.ai/settings/keys](https://openrouter.ai/settings/keys)
@@ -138,13 +156,26 @@ an isolated environment and puts them straight on your `PATH`.
                                               │ (render.py)          │
                                               └──────┬──────────┬────┘
                                                      ▼          ▼
-                                              tui.py (Textual)  ui.py (classic)
+                                               tui.py + tui_*.py  ui.py + ui_*.py
+                                               (Textual)          (classic)
 ```
 
 A single `Renderer` interface (`render.py`) decouples the agent loop from presentation, so
 the Textual TUI and the classic REPL are two implementations of the same contract rather
 than two copies of the agent logic.
 
+Each frontend is a small set of single-purpose modules behind a facade, so the entry module
+stays readable and the parts can be reused or tested on their own:
+
+| Frontend | Facade | Split into |
+|---|---|---|
+| Textual | `tui.py` — app, message handlers, input, suggestions | `tui_theme.py` (banner, palette, CSS) · `tui_messages.py` (Message classes, `TuiRenderer`) · `tui_widgets.py` (conversation blocks, input box, modals, `ConfirmBridge`) |
+| Classic | `ui.py` — the `print_*` / `stream_*` renderers | `ui_theme.py` (shared `Console`, palette, banner) · `ui_prompts.py` (permission dialog, input, `EscListener`) |
+
+Each facade re-exports the names its split modules define, so `from ui import ...` and
+`from tui import ...` keep working exactly as before.
+
+
 ## Modes
 
 | Mode | Command | Tools available | Use for |
@@ -235,7 +266,8 @@ models is the first thing to try before changing anything else.
 
 ## Roadmap
 
-- [ ] Test suite + CI (guardrails, sandbox path resolution, todo-store invariants)
+- [x] Test suite (`pytest tests/`) — guardrails, sandbox path resolution, todo-store invariants
+- [ ] CI (run the suite on push)
 - [ ] Docker-based sandbox for shell execution, not just path restriction
 - [ ] Client/server split — `build_graph()` behind FastAPI/WebSocket, thin streaming client
 - [ ] PyPI release + prebuilt binaries (PyInstaller) for no-Python-required installs

{closecode_ai-0.4.0 → closecode_ai-0.5.0}/README.md

@@ -76,6 +76,22 @@ closecode
 `pipx` is recommended over `pip` for the packaged version since it installs CLI tools into
 an isolated environment and puts them straight on your `PATH`.
 
+## Tests
+
+```bash
+pip install -e ".[dev]"
+pytest
+```
+
+429 tests covering guardrails, sandbox path resolution, undo/checkpoints, background
+processes, todo-store invariants, plan-mode tool filtering, session persistence, config
+permissions, and the search tools. The suite is fully isolated: `config.py` and
+`session.py` are redirected to temp dirs, no test touches the network, and nothing is
+written to your real `~/.closecode`.
+
+Some tests are marked `xfail` for **known bugs** rather than fixed behaviour — they
+document a gap and will flip to passing when it's fixed. Run `pytest -rx` to list them.
+
 ## Quick start
 
 1. Get a free API key at [openrouter.ai/settings/keys](https://openrouter.ai/settings/keys)
@@ -115,13 +131,26 @@ an isolated environment and puts them straight on your `PATH`.
                                               │ (render.py)          │
                                               └──────┬──────────┬────┘
                                                      ▼          ▼
-                                              tui.py (Textual)  ui.py (classic)
+                                               tui.py + tui_*.py  ui.py + ui_*.py
+                                               (Textual)          (classic)
 ```
 
 A single `Renderer` interface (`render.py`) decouples the agent loop from presentation, so
 the Textual TUI and the classic REPL are two implementations of the same contract rather
 than two copies of the agent logic.
 
+Each frontend is a small set of single-purpose modules behind a facade, so the entry module
+stays readable and the parts can be reused or tested on their own:
+
+| Frontend | Facade | Split into |
+|---|---|---|
+| Textual | `tui.py` — app, message handlers, input, suggestions | `tui_theme.py` (banner, palette, CSS) · `tui_messages.py` (Message classes, `TuiRenderer`) · `tui_widgets.py` (conversation blocks, input box, modals, `ConfirmBridge`) |
+| Classic | `ui.py` — the `print_*` / `stream_*` renderers | `ui_theme.py` (shared `Console`, palette, banner) · `ui_prompts.py` (permission dialog, input, `EscListener`) |
+
+Each facade re-exports the names its split modules define, so `from ui import ...` and
+`from tui import ...` keep working exactly as before.
+
+
 ## Modes
 
 | Mode | Command | Tools available | Use for |
@@ -212,7 +241,8 @@ models is the first thing to try before changing anything else.
 
 ## Roadmap
 
-- [ ] Test suite + CI (guardrails, sandbox path resolution, todo-store invariants)
+- [x] Test suite (`pytest tests/`) — guardrails, sandbox path resolution, todo-store invariants
+- [ ] CI (run the suite on push)
 - [ ] Docker-based sandbox for shell execution, not just path restriction
 - [ ] Client/server split — `build_graph()` behind FastAPI/WebSocket, thin streaming client
 - [ ] PyPI release + prebuilt binaries (PyInstaller) for no-Python-required installs
@@ -226,4 +256,4 @@ harness root, and add it to the Plan-mode allowlist only if it's genuinely read-
 
 ## License
 
-MIT — see [LICENSE](LICENSE).
+MIT — see [LICENSE](LICENSE).

{closecode_ai-0.4.0 → closecode_ai-0.5.0}/closecode_ai.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: closecode-ai
-Version: 0.4.0
+Version: 0.5.0
 Summary: CloseCode — an agentic terminal coding assistant (LangGraph + OpenRouter + MCP)
 Author: Om Gite
 License-Expression: MIT
@@ -20,6 +20,8 @@ Requires-Dist: langchain-openrouter
 Requires-Dist: rich
 Requires-Dist: pyfiglet
 Requires-Dist: textual
+Provides-Extra: dev
+Requires-Dist: pytest>=8; extra == "dev"
 
 <div align="center">
 
@@ -99,6 +101,22 @@ closecode
 `pipx` is recommended over `pip` for the packaged version since it installs CLI tools into
 an isolated environment and puts them straight on your `PATH`.
 
+## Tests
+
+```bash
+pip install -e ".[dev]"
+pytest
+```
+
+429 tests covering guardrails, sandbox path resolution, undo/checkpoints, background
+processes, todo-store invariants, plan-mode tool filtering, session persistence, config
+permissions, and the search tools. The suite is fully isolated: `config.py` and
+`session.py` are redirected to temp dirs, no test touches the network, and nothing is
+written to your real `~/.closecode`.
+
+Some tests are marked `xfail` for **known bugs** rather than fixed behaviour — they
+document a gap and will flip to passing when it's fixed. Run `pytest -rx` to list them.
+
 ## Quick start
 
 1. Get a free API key at [openrouter.ai/settings/keys](https://openrouter.ai/settings/keys)
@@ -138,13 +156,26 @@ an isolated environment and puts them straight on your `PATH`.
                                               │ (render.py)          │
                                               └──────┬──────────┬────┘
                                                      ▼          ▼
-                                              tui.py (Textual)  ui.py (classic)
+                                               tui.py + tui_*.py  ui.py + ui_*.py
+                                               (Textual)          (classic)
 ```
 
 A single `Renderer` interface (`render.py`) decouples the agent loop from presentation, so
 the Textual TUI and the classic REPL are two implementations of the same contract rather
 than two copies of the agent logic.
 
+Each frontend is a small set of single-purpose modules behind a facade, so the entry module
+stays readable and the parts can be reused or tested on their own:
+
+| Frontend | Facade | Split into |
+|---|---|---|
+| Textual | `tui.py` — app, message handlers, input, suggestions | `tui_theme.py` (banner, palette, CSS) · `tui_messages.py` (Message classes, `TuiRenderer`) · `tui_widgets.py` (conversation blocks, input box, modals, `ConfirmBridge`) |
+| Classic | `ui.py` — the `print_*` / `stream_*` renderers | `ui_theme.py` (shared `Console`, palette, banner) · `ui_prompts.py` (permission dialog, input, `EscListener`) |
+
+Each facade re-exports the names its split modules define, so `from ui import ...` and
+`from tui import ...` keep working exactly as before.
+
+
 ## Modes
 
 | Mode | Command | Tools available | Use for |
@@ -235,7 +266,8 @@ models is the first thing to try before changing anything else.
 
 ## Roadmap
 
-- [ ] Test suite + CI (guardrails, sandbox path resolution, todo-store invariants)
+- [x] Test suite (`pytest tests/`) — guardrails, sandbox path resolution, todo-store invariants
+- [ ] CI (run the suite on push)
 - [ ] Docker-based sandbox for shell execution, not just path restriction
 - [ ] Client/server split — `build_graph()` behind FastAPI/WebSocket, thin streaming client
 - [ ] PyPI release + prebuilt binaries (PyInstaller) for no-Python-required installs

{closecode_ai-0.4.0 → closecode_ai-0.5.0}/closecode_ai.egg-info/SOURCES.txt

@@ -16,10 +16,24 @@ todos.py
 token_tracker.py
 tools.py
 tui.py
+tui_messages.py
+tui_theme.py
+tui_widgets.py
 ui.py
+ui_prompts.py
+ui_theme.py
 closecode_ai.egg-info/PKG-INFO
 closecode_ai.egg-info/SOURCES.txt
 closecode_ai.egg-info/dependency_links.txt
 closecode_ai.egg-info/entry_points.txt
 closecode_ai.egg-info/requires.txt
-closecode_ai.egg-info/top_level.txt
+closecode_ai.egg-info/top_level.txt
+tests/test_config.py
+tests/test_guardrails.py
+tests/test_harness.py
+tests/test_llm.py
+tests/test_modes.py
+tests/test_search.py
+tests/test_session.py
+tests/test_todos.py
+tests/test_token_tracker.py

{closecode_ai-0.4.0 → closecode_ai-0.5.0}/closecode_ai.egg-info/requires.txt

@@ -12,3 +12,6 @@ langchain-openrouter
 rich
 pyfiglet
 textual
+
+[dev]
+pytest>=8

{closecode_ai-0.4.0 → closecode_ai-0.5.0}/closecode_ai.egg-info/top_level.txt

@@ -14,4 +14,9 @@ todos
 token_tracker
 tools
 tui
+tui_messages
+tui_theme
+tui_widgets
 ui
+ui_prompts
+ui_theme

{closecode_ai-0.4.0 → closecode_ai-0.5.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "closecode-ai"
-version = "0.4.0"
+version = "0.5.0"
 description = "CloseCode — an agentic terminal coding assistant (LangGraph + OpenRouter + MCP)"
 readme = "README.md"
 requires-python = ">=3.10"
@@ -27,10 +27,23 @@ dependencies = [
     "textual",
 ]
 
+[project.optional-dependencies]
+dev = ["pytest>=8"]
+
 # `closecode` command -> main:cli_entry
 [project.scripts]
 closecode = "main:cli_entry"
 
+[tool.pytest.ini_options]
+# tests/ imports the app modules by bare name (flat layout), which
+# tests/conftest.py arranges by putting the project root on sys.path.
+testpaths = ["tests"]
+python_files = ["test_*.py"]
+addopts = "-q --strict-markers"
+filterwarnings = [
+    "ignore::UserWarning:langchain_core.utils.pydantic",
+]
+
 # Flat layout: every top-level .py file is a module to ship.
 [tool.setuptools]
 py-modules = [
@@ -50,5 +63,10 @@ py-modules = [
     "token_tracker",
     "tools",
     "tui",
+    "tui_messages",
+    "tui_theme",
+    "tui_widgets",
     "ui",
+    "ui_prompts",
+    "ui_theme",
 ]

closecode_ai-0.5.0/tests/test_config.py

@@ -0,0 +1,175 @@
+"""Tests for config.py — the per-user API key / model store.
+
+The file holds a secret, so two properties matter as much as correctness:
+it is created 0600, and a corrupt or hostile file can't crash the launcher.
+Both are tested here against a CLOSECODE_CONFIG_DIR temp dir.
+"""
+
+import json
+import os
+import stat
+
+import pytest
+
+import config
+from config import (
+    CONFIG_DIR_ENV,
+    CONFIG_FILENAME,
+    KEY_API,
+    KEY_MODEL,
+    config_dir,
+    config_path,
+    load_config,
+    save_config_value,
+)
+
+
+# ---------------------------------------------------------------------------
+# location
+# ---------------------------------------------------------------------------
+
+def test_config_dir_honours_the_env_override(isolated_home):
+    assert config_dir() == isolated_home.config_dir
+
+
+def test_config_path_is_config_json_in_that_dir(isolated_home):
+    assert config_path() == isolated_home.config_dir / CONFIG_FILENAME
+    assert config_path().name == "config.json"
+
+
+def test_config_dir_is_created_on_demand(tmp_path, monkeypatch):
+    target = tmp_path / "brand" / "new"
+    monkeypatch.setenv(CONFIG_DIR_ENV, str(target))
+    assert config_dir() == target
+    assert target.is_dir()
+
+
+def test_blank_env_override_falls_back_to_default(monkeypatch, tmp_path):
+    """An empty/whitespace override must not produce a path of '' — that would
+    write the API key into the current working directory. HOME is redirected so
+    the fallback creates a temp dir, not a real ~/.closecode."""
+    monkeypatch.setenv("HOME", str(tmp_path))
+    monkeypatch.setenv(CONFIG_DIR_ENV, "   ")
+    assert config_dir() == tmp_path / ".closecode"
+    assert config_dir().is_dir()
+
+
+def test_env_override_expands_a_tilde(monkeypatch, tmp_path):
+    monkeypatch.setenv("HOME", str(tmp_path))
+    monkeypatch.setenv(CONFIG_DIR_ENV, "~/somewhere")
+    assert config_dir() == tmp_path / "somewhere"
+
+
+# ---------------------------------------------------------------------------
+# load_config
+# ---------------------------------------------------------------------------
+
+def test_load_missing_file_returns_empty_dict(isolated_home):
+    assert load_config() == {}
+
+
+def test_load_reads_back_saved_values(isolated_home):
+    save_config_value(KEY_API, "sk-or-v1-abc")
+    save_config_value(KEY_MODEL, "some/model:free")
+    assert load_config() == {KEY_API: "sk-or-v1-abc", KEY_MODEL: "some/model:free"}
+
+
+@pytest.mark.parametrize(
+    "content",
+    [
+        "{not valid json",
+        "",
+        "null",
+        "[]",
+        '"a string"',
+        "123",
+        "\x00\x01binary",
+    ],
+)
+def test_load_corrupt_or_non_object_file_returns_empty_dict(isolated_home, content):
+    """A mangled config must degrade to 'no config' rather than raising on
+    every launch — the user would be locked out of the app entirely."""
+    config_path().write_text(content, encoding="utf-8")
+    assert load_config() == {}
+
+
+def test_load_survives_a_directory_in_place_of_the_file(isolated_home):
+    config_path().mkdir()
+    assert load_config() == {}
+
+
+# ---------------------------------------------------------------------------
+# save_config_value
+# ---------------------------------------------------------------------------
+
+def test_save_merges_rather_than_replaces(isolated_home):
+    save_config_value(KEY_API, "key-one")
+    save_config_value(KEY_MODEL, "model-one")
+    data = json.loads(config_path().read_text())
+    assert data == {KEY_API: "key-one", KEY_MODEL: "model-one"}
+
+
+def test_save_overwrites_a_single_key(isolated_home):
+    save_config_value(KEY_API, "old")
+    save_config_value(KEY_API, "new")
+    assert load_config()[KEY_API] == "new"
+    assert len(load_config()) == 1
+
+
+def test_save_returns_the_path_written(isolated_home):
+    assert save_config_value(KEY_API, "k") == config_path()
+
+
+def test_save_recovers_from_a_corrupt_file(isolated_home):
+    """A user who hand-edited the file badly should still be able to save."""
+    config_path().write_text("{{{", encoding="utf-8")
+    save_config_value(KEY_API, "k")
+    assert load_config() == {KEY_API: "k"}
+
+
+def test_save_written_file_is_valid_json_with_trailing_newline(isolated_home):
+    save_config_value(KEY_API, "k")
+    raw = config_path().read_text(encoding="utf-8")
+    assert raw.endswith("\n")
+    assert json.loads(raw) == {KEY_API: "k"}
+
+
+def test_save_preserves_unicode_values(isolated_home):
+    save_config_value(KEY_MODEL, "modèle/百万:free")
+    assert load_config()[KEY_MODEL] == "modèle/百万:free"
+
+
+# ---------------------------------------------------------------------------
+# permissions — the file holds a secret
+# ---------------------------------------------------------------------------
+
+@pytest.mark.skipif(os.name == "nt", reason="POSIX permission bits")
+def test_config_file_is_owner_only(isolated_home):
+    save_config_value(KEY_API, "sk-secret")
+    mode = stat.S_IMODE(config_path().stat().st_mode)
+    assert mode == 0o600, f"expected 0600, got {oct(mode)}"
+
+
+@pytest.mark.skipif(os.name == "nt", reason="POSIX permission bits")
+def test_config_file_is_not_world_or_group_readable(isolated_home):
+    save_config_value(KEY_API, "sk-secret")
+    mode = stat.S_IMODE(config_path().stat().st_mode)
+    assert not mode & stat.S_IRGRP
+    assert not mode & stat.S_IROTH
+
+
+@pytest.mark.skipif(os.name == "nt", reason="POSIX permission bits")
+def test_resaving_keeps_the_tight_permissions(isolated_home):
+    save_config_value(KEY_API, "one")
+    save_config_value(KEY_API, "two")
+    assert stat.S_IMODE(config_path().stat().st_mode) == 0o600
+
+
+@pytest.mark.skipif(os.name == "nt", reason="POSIX permission bits")
+def test_permissions_are_tightened_on_an_existing_loose_file(isolated_home):
+    """A file left 0644 by an earlier version or a manual edit gets fixed on
+    the next save, not just on creation."""
+    config_path().write_text("{}", encoding="utf-8")
+    os.chmod(config_path(), 0o644)
+    save_config_value(KEY_API, "k")
+    assert stat.S_IMODE(config_path().stat().st_mode) == 0o600