code-aide 1.1.0__tar.gz → 1.2.0__tar.gz

{code_aide-1.1.0 → code_aide-1.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-aide
-Version: 1.1.0
+Version: 1.2.0
 Summary: Manage AI coding CLI tools (Claude, Copilot, Cursor, Gemini, Amp, Codex)
 Project-URL: Homepage, https://github.com/dajobe/code-aide
 Project-URL: Repository, https://github.com/dajobe/code-aide

code_aide-1.2.0/TODO.md

@@ -0,0 +1,70 @@
+# TODO
+
+Keep the project focused on managing AI coding CLI tools (install, upgrade,
+remove, status, and version metadata).
+
+## Bugs
+
+- [x] Standardize `--dryrun` flag: `update-versions` uses `--dry-run` but
+  `install` uses `--dryrun`. Change `update-versions` to `--dryrun`.
+- [x] Add missing `success()` message for direct-download installs in non-dryrun
+  mode (`install.py`, `install_tool()`).
+- [x] Replace PID-based temp directory naming with `tempfile.mkdtemp()` in
+  `install.py` `install_direct_download()`.
+- [x] Guard `package.split("/", 1)` in `detection.py`
+  `get_system_package_info()` against missing `/` in subprocess output.
+
+## Reliability and Correctness
+
+- [ ] Handle missing `node` cleanly during prerequisite checks
+  (`FileNotFoundError` path in Node version probing).
+- [ ] Read tool version output from both stdout and stderr so status does not
+  miss installed versions.
+- [ ] Make version cache writes atomic (write temp file + rename) to avoid
+  partial/corrupted `versions.json`. Apply to `save_bundled_versions()` too.
+- [ ] Warn when `versions.json` cache contains invalid JSON instead of silently
+  returning empty data.
+- [ ] Use `os.pathsep` instead of hardcoded `":"` in `prereqs.py`
+  `check_path_directories()`.
+
+## Security and Integrity
+
+- [ ] Add integrity verification for direct-download tarballs (not only install
+  script SHA256).
+- [ ] Extend tool metadata to support tarball checksum/signature fields where
+  applicable.
+
+## CLI and Automation UX
+
+- [ ] Add `--json` output mode for `list`, `status`, and `update-versions`.
+- [ ] Add a focused `doctor` command for environment checks (PATH,
+  prerequisites, command health).
+- [ ] Consider `install --force` for reinstall/repair flows.
+- [ ] Add cleanup support for stale direct-download versions no longer in use.
+- [ ] Expand `upgrade` help text to mention the default "only out-of-date tools"
+  behavior.
+
+## Documentation
+
+- [ ] Document that running `code-aide` with no subcommand defaults to `status`.
+- [ ] Add `install --dryrun` to the README usage examples.
+- [ ] Document environment variables and config file paths
+  (`~/.config/code-aide/versions.json`).
+
+## Platform and Package Detection
+
+- [ ] Broaden system package metadata detection beyond Gentoo-specific tooling
+  where practical.
+
+## Maintainability and Tests
+
+- [ ] Split larger modules into smaller focused files where practical
+  (`commands_actions.py`, `versions.py`, `install.py`).
+- [ ] Add tests for major untested functions:
+  - `upgrade_tool()` (completely untested)
+  - `cmd_remove()` (completely untested)
+  - `fetch_url()` (completely untested)
+  - `check_prerequisites()` / `install_nodejs_npm()` (completely untested)
+  - `save_bundled_versions()` (untested)
+  - `get_system_package_info()` (completely untested)
+- [ ] Add tests for prerequisite edge cases and cache write behavior.

{code_aide-1.1.0 → code_aide-1.2.0}/src/code_aide/__init__.py

@@ -1,3 +1,3 @@
 """code-aide - Manage AI coding CLI tools."""
 
-__version__ = "1.1.0"
+__version__ = "1.2.0"

{code_aide-1.1.0 → code_aide-1.2.0}/src/code_aide/commands_actions.py

@@ -302,7 +302,7 @@ def cmd_update_versions(args: argparse.Namespace) -> None:
 
     updates = [result for result in results if result["update"]]
     if not updates:
-        if version_info_changed and not args.dry_run:
+        if version_info_changed and not args.dryrun:
             dest = _save(config["tools"])
             print(f"Updated latest version info in {dest}.")
         if version_info_changed:
@@ -323,7 +323,7 @@ def cmd_update_versions(args: argparse.Namespace) -> None:
     for result in updates:
         print(f"  {result['tool']}: SHA256 changed")
 
-    if args.dry_run:
+    if args.dryrun:
         print("\nDry run mode - no changes written.")
         return
 

{code_aide-1.1.0 → code_aide-1.2.0}/src/code_aide/detection.py

@@ -105,6 +105,9 @@ def get_system_package_info(binary_path: str) -> Dict[str, Optional[str]]:
     except Exception:
         pass
 
+    if "/" not in package:
+        return result
+
     category, package_name = package.split("/", 1)
     ebuild_dirs = globmod.glob(f"/var/db/repos/*/{category}/{package_name}/")
     ebuilds = []

{code_aide-1.1.0 → code_aide-1.2.0}/src/code_aide/entry.py

@@ -84,7 +84,7 @@ def main() -> None:
     )
     update_versions_parser.add_argument(
         "-n",
-        "--dry-run",
+        "--dryrun",
         action="store_true",
         help="Show changes only, do not write updates",
     )

{code_aide-1.1.0 → code_aide-1.2.0}/src/code_aide/install.py

@@ -7,6 +7,7 @@ import platform
 import shutil
 import subprocess
 import tarfile
+import tempfile
 from typing import Any, Dict, Optional
 
 from code_aide.constants import TOOLS
@@ -162,8 +163,11 @@ def install_direct_download(
         tarball_data, _ = fetch_url(download_url, timeout=120)
         success(f"Downloaded {len(tarball_data)} bytes")
 
-        temp_dir = install_dir + f".tmp-{os.getpid()}"
-        os.makedirs(temp_dir, exist_ok=True)
+        install_parent = os.path.dirname(install_dir) or "."
+        os.makedirs(install_parent, exist_ok=True)
+        temp_dir = tempfile.mkdtemp(
+            prefix=os.path.basename(install_dir) + ".tmp-", dir=install_parent
+        )
 
         try:
             with tarfile.open(fileobj=io.BytesIO(tarball_data), mode="r:gz") as tf:
@@ -275,6 +279,7 @@ def install_tool(tool_name: str, dryrun: bool = False) -> bool:
                 if dryrun:
                     success(f"{tool_config['name']} verification passed")
                 else:
+                    success(f"{tool_config['name']} installed successfully")
                     info(tool_config["next_steps"])
                     if "docs_url" in tool_config:
                         info(f"Documentation: {tool_config['docs_url']}")

{code_aide-1.1.0 → code_aide-1.2.0}/tests/test_commands_actions.py

@@ -59,7 +59,7 @@ class TestCmdUpdateVersions(unittest.TestCase):
             (),
             {
                 "tools": ["missing"],
-                "dry_run": False,
+                "dryrun": False,
                 "yes": False,
                 "verbose": False,
                 "bundled": False,
@@ -81,7 +81,7 @@ class TestCmdUpdateVersions(unittest.TestCase):
             (),
             {
                 "tools": [],
-                "dry_run": True,
+                "dryrun": True,
                 "yes": False,
                 "verbose": False,
                 "bundled": False,
@@ -131,7 +131,7 @@ class TestCmdUpdateVersions(unittest.TestCase):
             (),
             {
                 "tools": [],
-                "dry_run": False,
+                "dryrun": False,
                 "yes": True,
                 "verbose": False,
                 "bundled": True,

{code_aide-1.1.0 → code_aide-1.2.0}/tests/test_install.py

@@ -122,3 +122,57 @@ class TestInstallDirectDownloadDryrun(unittest.TestCase):
 
         result = cli_install.install_direct_download("test", tool_config, dryrun=True)
         self.assertFalse(result)
+
+
+class TestInstallDirectDownload(unittest.TestCase):
+    """Tests for install_direct_download in non-dryrun mode."""
+
+    @mock.patch.object(cli_install, "detect_os_arch", return_value=("linux", "x64"))
+    @mock.patch.object(cli_install, "fetch_url")
+    def test_creates_missing_install_parent_directory(self, mock_fetch, mock_os_arch):
+        script_content = b"echo install"
+        expected_sha256 = cli_install.hashlib.sha256(script_content).hexdigest()
+
+        tarball_data = io.BytesIO()
+        with tarfile.open(fileobj=tarball_data, mode="w:gz") as tf:
+            payload = b"#!/bin/sh\necho ok\n"
+            info = tarfile.TarInfo("package/test-bin")
+            info.mode = 0o755
+            info.size = len(payload)
+            tf.addfile(info, io.BytesIO(payload))
+        tarball_bytes = tarball_data.getvalue()
+
+        def _fake_fetch(url, timeout=30):
+            if url.endswith("/install"):
+                return script_content, None
+            return tarball_bytes, None
+
+        mock_fetch.side_effect = _fake_fetch
+
+        tool_config = {
+            "name": "Test Tool",
+            "install_url": "https://example.com/install",
+            "install_sha256": expected_sha256,
+            "download_url_template": "https://example.com/{version}/{os}/{arch}/pkg.tar.gz",
+            "install_dir": "/tmp/test-{version}",
+            "bin_dir": "/tmp/test-bin",
+            "symlinks": {"test": "test-bin"},
+            "latest_version": "1.0.0",
+        }
+
+        with tempfile.TemporaryDirectory() as td:
+            install_dir = os.path.join(td, "missing", "versions", "1.0.0")
+            bin_dir = os.path.join(td, "bin")
+
+            result = cli_install.install_direct_download(
+                "test",
+                tool_config,
+                dryrun=False,
+                install_dir_override=install_dir,
+                bin_dir_override=bin_dir,
+            )
+
+            self.assertTrue(result)
+            self.assertTrue(os.path.isdir(os.path.dirname(install_dir)))
+            self.assertTrue(os.path.exists(os.path.join(install_dir, "test-bin")))
+            self.assertTrue(os.path.islink(os.path.join(bin_dir, "test")))

code_aide-1.1.0/TODO.md

@@ -1,39 +0,0 @@
-# TODO
-
-Keep the project focused on managing AI coding CLI tools (install, upgrade,
-remove, status, and version metadata).
-
-## Reliability and Correctness
-
-- [ ] Handle missing `node` cleanly during prerequisite checks
-  (`FileNotFoundError` path in Node version probing).
-- [ ] Read tool version output from both stdout and stderr so status does not
-  miss installed versions.
-- [ ] Make version cache writes atomic (write temp file + rename) to avoid
-  partial/corrupted `versions.json`.
-
-## Security and Integrity
-
-- [ ] Add integrity verification for direct-download tarballs (not only install
-  script SHA256).
-- [ ] Extend tool metadata to support tarball checksum/signature fields where
-  applicable.
-
-## CLI and Automation UX
-
-- [ ] Add `--json` output mode for `list`, `status`, and `update-versions`.
-- [ ] Add a focused `doctor` command for environment checks (PATH,
-  prerequisites, command health).
-- [ ] Consider `install --force` for reinstall/repair flows.
-- [ ] Add cleanup support for stale direct-download versions no longer in use.
-
-## Platform and Package Detection
-
-- [ ] Broaden system package metadata detection beyond Gentoo-specific tooling
-  where practical.
-
-## Maintainability and Tests
-
-- [ ] Split larger modules into smaller focused files where practical
-  (`commands_actions.py`, `versions.py`, `install.py`).
-- [ ] Add tests for prerequisite edge cases and cache write behavior.