killpy 0.25.0__tar.gz → 1.0.0__tar.gz

{killpy-0.25.0 → killpy-1.0.0}/PKG-INFO

@@ -1,17 +1,25 @@
 Metadata-Version: 2.4
 Name: killpy
-Version: 0.25.0
-Summary: List all .venv directories and Conda environments 🐍 on your system and check how much space they are using. You can then choose which ones to delete in order to free up space 🧹.
+Version: 1.0.0
+Summary: Find and delete unused Python environments (.venv, conda, poetry, pipenv, pipx, pyenv, uv) and caches to free disk space — from a fast terminal UI or a single CLI command.
 Project-URL: Homepage, https://github.com/Tlaloc-Es/killpy
 Project-URL: Documentation, https://tlaloc-es.github.io/killpy/
 Project-URL: Repository, https://github.com/Tlaloc-Es/killpy
 Project-URL: Bug Tracker, https://github.com/Tlaloc-Es/killpy/issues
+Keywords: venv,virtualenv,conda,poetry,pipenv,pipx,pyenv,uv,cleanup,disk-space,cache,cli,tui,textual,developer-tools
+Classifier: Development Status :: 4 - Beta
 Classifier: Environment :: Console
+Classifier: Environment :: Console :: Curses
+Classifier: Intended Audience :: Developers
+Classifier: Intended Audience :: System Administrators
 Classifier: License :: OSI Approved :: MIT License
+Classifier: Operating System :: OS Independent
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Topic :: System :: Systems Administration
+Classifier: Topic :: Utilities
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 Requires-Dist: click>=8.1.8
@@ -188,7 +196,7 @@ Environments flagged with `⚠️` are actively in use (the environment killpy r
 | Key | Action |
 |-----|--------|
 | `↑` / `↓` or `k` / `j` | Move cursor up / down (vim-style) |
-| `/` | Open live search/filter bar (regex supported) |
+| `/` | Open live search/filter bar (substring match) |
 | `Escape` | Close search bar and clear filter |
 | `T` | Toggle multi-select mode on / off |
 | `Space` | *(Multi-select)* Toggle current row selected |
@@ -203,7 +211,7 @@ Environments flagged with `⚠️` are actively in use (the environment killpy r
 
 ### Search / filter
 
-Press `/` to open the filter bar at the bottom of the screen. Type any string or a Python regex — the venv table updates live as you type. Press `Escape` or submit an empty value to clear the filter and return to the full list.
+Press `/` to open the filter bar at the bottom of the screen. Type any text — the venv table filters by path substring (case-insensitive) and updates live as you type. Press `Escape` or submit an empty value to clear the filter and return to the full list.
 
 ### Multi-select mode
 
@@ -257,7 +265,7 @@ ______________________________________________________________________
 killpy list                               # list all detected environments
 killpy list --path ~/projects             # scan a specific path
 killpy list --type venv --type conda      # filter by type (repeatable)
-killpy list --older-than 90               # not accessed in the last 90 days
+killpy list --older-than 90               # not modified in the last 90 days
 killpy list --json                        # output as a JSON array
 killpy list --json-stream                 # stream as NDJSON — one line per env
 killpy list --quiet                       # suppress progress output (scripts/CI)
@@ -276,7 +284,7 @@ The **Path** column mirrors the form of `--path`: a relative `--path` produces r
     "absolute_path": "/home/user/projects/my-app/.venv",
     "name": "my-app/.venv",
     "type": "venv",
-    "last_accessed": "2025-11-02T14:23:01+00:00",
+    "last_modified": "2025-11-02T14:23:01+00:00",
     "size_bytes": 54393984,
     "size_human": "51.88 MB",
     "managed_by": null,
@@ -409,8 +417,7 @@ Category is assigned deterministically by applying the following rules in order:
 |----------|------|----------|
 | 1 | Orphan (`is_orphan=True`) **and** `age ≥ 180 days` | `HIGH` |
 | 2 | Active git repository **or** `age < 120 days` | `LOW` |
-| 3 | `age ≥ 120 days` | `MEDIUM` |
-| 4 | Fallback | `LOW` |
+| 3 | `age ≥ 120 days` *(exhaustive fallback)* | `MEDIUM` |
 
 Age and orphan status are the dominant signals. Size does not affect the category.
 
@@ -490,7 +497,7 @@ Yes. Run `killpy --path ~` to scan your home folder. The `stats` command gives a
 killpy delete --type venv --older-than 90 --yes
 ```
 
-Deletes every `.venv` / `pyvenv.cfg` env not accessed in the last 90 days, without prompting.
+Deletes every `.venv` / `pyvenv.cfg` env not modified in the last 90 days, without prompting.
 
 **How do I use killpy in a CI pipeline or script?**
 
@@ -596,7 +603,7 @@ ______________________________________________________________________
 
 ```yaml
 - repo: https://github.com/Tlaloc-Es/KillPy
-  rev: 0.24.1
+  rev: 1.0.0
   hooks:
     - id: killpy                  # remove __pycache__ on every commit
     - id: killpy-clean-caches     # also removes .mypy_cache, .pytest_cache, .ruff_cache
@@ -621,7 +628,7 @@ Typical minimal setup (safe for daily use):
 
 ```yaml
 - repo: https://github.com/Tlaloc-Es/KillPy
-  rev: 0.25.0
+  rev: 1.0.0
   hooks:
     - id: killpy
 ```

{killpy-0.25.0 → killpy-1.0.0}/README.md

@@ -167,7 +167,7 @@ Environments flagged with `⚠️` are actively in use (the environment killpy r
 | Key | Action |
 |-----|--------|
 | `↑` / `↓` or `k` / `j` | Move cursor up / down (vim-style) |
-| `/` | Open live search/filter bar (regex supported) |
+| `/` | Open live search/filter bar (substring match) |
 | `Escape` | Close search bar and clear filter |
 | `T` | Toggle multi-select mode on / off |
 | `Space` | *(Multi-select)* Toggle current row selected |
@@ -182,7 +182,7 @@ Environments flagged with `⚠️` are actively in use (the environment killpy r
 
 ### Search / filter
 
-Press `/` to open the filter bar at the bottom of the screen. Type any string or a Python regex — the venv table updates live as you type. Press `Escape` or submit an empty value to clear the filter and return to the full list.
+Press `/` to open the filter bar at the bottom of the screen. Type any text — the venv table filters by path substring (case-insensitive) and updates live as you type. Press `Escape` or submit an empty value to clear the filter and return to the full list.
 
 ### Multi-select mode
 
@@ -236,7 +236,7 @@ ______________________________________________________________________
 killpy list                               # list all detected environments
 killpy list --path ~/projects             # scan a specific path
 killpy list --type venv --type conda      # filter by type (repeatable)
-killpy list --older-than 90               # not accessed in the last 90 days
+killpy list --older-than 90               # not modified in the last 90 days
 killpy list --json                        # output as a JSON array
 killpy list --json-stream                 # stream as NDJSON — one line per env
 killpy list --quiet                       # suppress progress output (scripts/CI)
@@ -255,7 +255,7 @@ The **Path** column mirrors the form of `--path`: a relative `--path` produces r
     "absolute_path": "/home/user/projects/my-app/.venv",
     "name": "my-app/.venv",
     "type": "venv",
-    "last_accessed": "2025-11-02T14:23:01+00:00",
+    "last_modified": "2025-11-02T14:23:01+00:00",
     "size_bytes": 54393984,
     "size_human": "51.88 MB",
     "managed_by": null,
@@ -388,8 +388,7 @@ Category is assigned deterministically by applying the following rules in order:
 |----------|------|----------|
 | 1 | Orphan (`is_orphan=True`) **and** `age ≥ 180 days` | `HIGH` |
 | 2 | Active git repository **or** `age < 120 days` | `LOW` |
-| 3 | `age ≥ 120 days` | `MEDIUM` |
-| 4 | Fallback | `LOW` |
+| 3 | `age ≥ 120 days` *(exhaustive fallback)* | `MEDIUM` |
 
 Age and orphan status are the dominant signals. Size does not affect the category.
 
@@ -469,7 +468,7 @@ Yes. Run `killpy --path ~` to scan your home folder. The `stats` command gives a
 killpy delete --type venv --older-than 90 --yes
 ```
 
-Deletes every `.venv` / `pyvenv.cfg` env not accessed in the last 90 days, without prompting.
+Deletes every `.venv` / `pyvenv.cfg` env not modified in the last 90 days, without prompting.
 
 **How do I use killpy in a CI pipeline or script?**
 
@@ -575,7 +574,7 @@ ______________________________________________________________________
 
 ```yaml
 - repo: https://github.com/Tlaloc-Es/KillPy
-  rev: 0.24.1
+  rev: 1.0.0
   hooks:
     - id: killpy                  # remove __pycache__ on every commit
     - id: killpy-clean-caches     # also removes .mypy_cache, .pytest_cache, .ruff_cache
@@ -600,7 +599,7 @@ Typical minimal setup (safe for daily use):
 
 ```yaml
 - repo: https://github.com/Tlaloc-Es/KillPy
-  rev: 0.25.0
+  rev: 1.0.0
   hooks:
     - id: killpy
 ```

{killpy-0.25.0 → killpy-1.0.0}/killpy/__main__.py

@@ -1,3 +1,7 @@
+"""killpy CLI entry point: the ``cli`` click group and its subcommands."""
+
+from __future__ import annotations
+
 import logging
 import sys
 from pathlib import Path
@@ -10,7 +14,7 @@ from rich.prompt import Confirm
 from killpy.cleaner import Cleaner, CleanerError
 from killpy.cli import TableApp
 from killpy.commands._utils import partition_in_use
-from killpy.commands.clean import clean
+from killpy.commands.clean import clean_cmd
 from killpy.commands.delete import delete_cmd
 from killpy.commands.doctor import doctor_cmd
 from killpy.commands.find import find_cmd
@@ -144,7 +148,7 @@ def cli(ctx, path: Path, exclude: str, delete_all: bool, yes: bool, force: bool)
             app.run()
 
 
-cli.add_command(clean)
+cli.add_command(clean_cmd, name="clean")
 cli.add_command(list_cmd, name="list")
 cli.add_command(delete_cmd, name="delete")
 cli.add_command(stats_cmd, name="stats")

{killpy-0.25.0 → killpy-1.0.0}/killpy/cleaners/__init__.py

@@ -1,3 +1,7 @@
+"""Stateless ``__pycache__`` removal helper used by ``killpy clean``."""
+
+from __future__ import annotations
+
 import os
 import shutil
 from pathlib import Path

{killpy-0.25.0 → killpy-1.0.0}/killpy/cli.py

@@ -1,5 +1,8 @@
+"""Interactive Textual TUI (``TableApp``) for browsing and deleting environments."""
+
+from __future__ import annotations
+
 import asyncio
-import re
 import subprocess
 import sys
 from datetime import datetime
@@ -44,7 +47,7 @@ def _health_text(category: str) -> Text | str:
     return ""
 
 
-def is_venv_tab(func):
+def _is_venv_tab(func):
     def wrapper(self, *args, **kwargs):
         if self.query_one(TabbedContent).active == "venv-tab":
             return func(self, *args, **kwargs)
@@ -52,7 +55,7 @@ def is_venv_tab(func):
     return wrapper
 
 
-def is_pipx_tab(func):
+def _is_pipx_tab(func):
     def wrapper(self, *args, **kwargs):
         if self.query_one(TabbedContent).active == "pipx-tab":
             return func(self, *args, **kwargs)
@@ -60,7 +63,7 @@ def is_pipx_tab(func):
     return wrapper
 
 
-def shorten_path_for_table(path_value, max_parts: int = 2) -> str:
+def _shorten_path_for_table(path_value, max_parts: int = 2) -> str:
     path_text = str(path_value)
     if "/" not in path_text and "\\" not in path_text:
         return path_text
@@ -293,7 +296,7 @@ class TableApp(App):
         yield Label("", id="status-label")
         yield Label("", id="selected-path-label")
         yield Input(
-            placeholder="Filter by path (regex)… Esc to clear",
+            placeholder="Filter by path… Esc to clear",
             id="search-input",
         )
         yield Label("", id="multi-select-label")
@@ -333,7 +336,7 @@ class TableApp(App):
             {
                 "path": str(environment.path),
                 "type": environment.type,
-                "last_modified": environment.last_accessed_str,
+                "last_modified": environment.last_modified_str,
                 "size": environment.size_bytes,
                 "size_human": environment.size_human,
                 "health": self._health_by_path.get(str(environment.path), ""),
@@ -349,7 +352,7 @@ class TableApp(App):
             "type"
         ]
         table.add_row(
-            shorten_path_for_table(row["path"]),
+            _shorten_path_for_table(row["path"]),
             type_label,
             row["last_modified"],
             row["size"],
@@ -378,20 +381,17 @@ class TableApp(App):
         table.clear(columns=True)
         table.add_columns(*self.get_headers_for_table("venv-table"))
         self._venv_display_indices = []
+        query = self._filter_query.lower()
         for i, row in enumerate(self.venv_rows):
-            if self._filter_query:
-                try:
-                    if not re.search(self._filter_query, row["path"], re.IGNORECASE):
-                        continue
-                except re.error:
-                    pass  # invalid regex — show all
+            if query and query not in row["path"].lower():
+                continue
             self._venv_display_indices.append(i)
             env = row["environment"]
             type_label = ("\u26a0\ufe0f " if env.is_system_critical else "") + row[
                 "type"
             ]
             table.add_row(
-                shorten_path_for_table(row["path"]),
+                _shorten_path_for_table(row["path"]),
                 type_label,
                 row["last_modified"],
                 row["size"],
@@ -609,12 +609,16 @@ class TableApp(App):
             row["health"] = self._health_by_path.get(row["path"], "")
         self.render_venv_table()
 
-    async def action_clean_pycache(self):
-        total_freed_space = await asyncio.to_thread(remove_pycache, self.root_dir)
-        self.bytes_release += total_freed_space
+    def _show_bytes_released(self) -> None:
+        """Update the status label with the running total of freed space."""
         self.query_one("#status-label", Label).update(
             f"{format_size(self.bytes_release)} deleted"
         )
+
+    async def action_clean_pycache(self):
+        total_freed_space = await asyncio.to_thread(remove_pycache, self.root_dir)
+        self.bytes_release += total_freed_space
+        self._show_bytes_released()
         self.bell()
 
     def on_data_table_row_highlighted(self, event: DataTable.RowHighlighted) -> None:
@@ -633,7 +637,7 @@ class TableApp(App):
             _, row = resolved
             selected_path_label.update(f"Selected: {row['path']}")
 
-    @is_venv_tab
+    @_is_venv_tab
     def action_confirm_delete(self):
         freed_now = 0
         if self._multi_select_mode and self._selected_venv_paths:
@@ -662,12 +666,10 @@ class TableApp(App):
 
         self._record_deletion(freed_now)
         self.render_venv_table()
-        self.query_one("#status-label", Label).update(
-            f"{format_size(self.bytes_release)} deleted"
-        )
+        self._show_bytes_released()
         self.bell()
 
-    @is_venv_tab
+    @_is_venv_tab
     def action_mark_for_delete(self):
         table = self.query_one("#venv-table", DataTable)
 
@@ -690,7 +692,7 @@ class TableApp(App):
                     EnvStatus.MARKED_TO_DELETE.value,
                 )
 
-    @is_venv_tab
+    @_is_venv_tab
     def action_delete_now(self):
         table = self.query_one("#venv-table", DataTable)
         cursor_cell = table.cursor_coordinate
@@ -709,9 +711,7 @@ class TableApp(App):
                     (cursor_cell.row, self.VENV_COL_STATUS),
                     EnvStatus.DELETED.value,
                 )
-                self.query_one("#status-label", Label).update(
-                    f"{format_size(self.bytes_release)} deleted"
-                )
+                self._show_bytes_released()
         self.bell()
 
     def delete_environment(self, environment: Environment) -> bool:
@@ -729,7 +729,7 @@ class TableApp(App):
             self.query_one("#status-label", Label).update(str(error))
             return False
 
-    @is_pipx_tab
+    @_is_pipx_tab
     def action_uninstall_pipx(self):
         table = self.query_one("#pipx-table", DataTable)
         cursor_cell = table.cursor_coordinate
@@ -742,9 +742,7 @@ class TableApp(App):
                 table.update_cell_at((cursor_cell.row, 3), EnvStatus.DELETED.value)
                 self.bytes_release += int(row["size"])
                 self._record_deletion(int(row["size"]))
-                self.query_one("#status-label", Label).update(
-                    f"{format_size(self.bytes_release)} deleted"
-                )
+                self._show_bytes_released()
 
         self.bell()
 
@@ -772,7 +770,7 @@ class TableApp(App):
             tid = "#venv-table" if active == "venv-tab" else "#pipx-table"
             self.query_one(tid, DataTable).action_cursor_up()
 
-    @is_venv_tab
+    @_is_venv_tab
     def action_open_folder(self) -> None:
         """Open the parent directory of the selected environment in the OS file manager."""  # noqa: E501
         table = self.query_one("#venv-table", DataTable)
@@ -834,7 +832,7 @@ class TableApp(App):
         self._update_multi_select_label()
         self.render_venv_table()
 
-    @is_venv_tab
+    @_is_venv_tab
     def action_multi_select_toggle_row(self) -> None:
         """Toggle current row selection in multi-select mode (Space key)."""
         if not self._multi_select_mode:
@@ -859,7 +857,7 @@ class TableApp(App):
             self._compute_row_status(row),
         )
 
-    @is_venv_tab
+    @_is_venv_tab
     def action_multi_select_all(self) -> None:
         """Toggle select-all / deselect-all in multi-select mode (A key)."""
         if not self._multi_select_mode:

{killpy-0.25.0 → killpy-1.0.0}/killpy/commands/_utils.py

@@ -68,7 +68,7 @@ def filter_envs(
         names such as ``"venv"`` and ``"cache"`` are automatically expanded to
         their concrete sub-type values via :data:`_TYPE_ALIASES`.
     older_than:
-        If provided, only environments not accessed in the last *older_than* days
+        If provided, only environments not modified in the last *older_than* days
         are kept.
     """
     now = datetime.now(tz=timezone.utc)
@@ -84,6 +84,6 @@ def filter_envs(
 
     if older_than is not None:
         cutoff = now - timedelta(days=older_than)
-        result = [e for e in result if e.last_accessed < cutoff]
+        result = [e for e in result if e.last_modified < cutoff]
 
     return result

{killpy-0.25.0 → killpy-1.0.0}/killpy/commands/clean.py

@@ -1,3 +1,7 @@
+"""``killpy clean`` – remove ``__pycache__`` directories under a path."""
+
+from __future__ import annotations
+
 from pathlib import Path
 
 import click
@@ -6,14 +10,15 @@ from killpy.cleaners import remove_pycache
 from killpy.files import format_size
 
 
-@click.command()
+@click.command("clean")
 @click.option(
     "--path",
     default=Path.cwd,
     type=click.Path(path_type=Path, exists=True, file_okay=False, dir_okay=True),
     help="Path to the directory to clean",
 )
-def clean(path: Path):
+def clean_cmd(path: Path) -> None:
+    """Remove all ``__pycache__`` directories under PATH and report freed space."""
     click.echo(f"Cleaning {path}…")
     total_freed_space = remove_pycache(path)
     click.echo(f"{format_size(total_freed_space)} freed")

{killpy-0.25.0 → killpy-1.0.0}/killpy/commands/delete.py

@@ -37,7 +37,7 @@ from killpy.scanner import Scanner
     type=int,
     default=None,
     metavar="DAYS",
-    help="Only delete environments not accessed in the last N days.",
+    help="Only delete environments not modified in the last N days.",
 )
 @click.option(
     "--dry-run",

{killpy-0.25.0 → killpy-1.0.0}/killpy/commands/doctor.py

@@ -13,12 +13,13 @@ from rich.table import Table
 from killpy.detectors import ALL_DETECTORS
 from killpy.files import format_size
 from killpy.intelligence import SuggestionEngine, UsageTracker, score_all
-from killpy.models import Suggestion
+from killpy.models import ScoredEnvironment, Suggestion
 from killpy.scanner import Scanner
 
-# Environments smaller than this threshold are excluded from "wasted" totals.
+# HIGH-category environments below this size don't count toward "wasted" totals.
 _IMPACTFUL_SIZE_BYTES = 10 * 1024 * 1024  # 10 MB
-_SMALL_SIZE_BYTES = 1 * 1024 * 1024  # 1 MB (matches suggestions.py)
+# Environments below this size are reported as "ignored (too small to matter)".
+_SMALL_SIZE_BYTES = 1 * 1024 * 1024  # 1 MB
 
 _CATEGORY_STYLE = {
     "HIGH": "bold red",
@@ -78,7 +79,7 @@ def doctor_cmd(path: Path, as_json: bool, show_all: bool) -> None:
     suggestions = engine.classify_all(scored_envs)
 
     if as_json:
-        _output_json(suggestions, envs)
+        _output_json(suggestions, scored_envs)
         return
 
     _output_rich(console, suggestions, scored_envs, path, show_all=show_all)
@@ -89,39 +90,47 @@ def doctor_cmd(path: Path, as_json: bool, show_all: bool) -> None:
 # ---------------------------------------------------------------------------
 
 
-def _output_json(suggestions: list[Suggestion], envs) -> None:  # type: ignore[type-arg]
-    total_size = sum(e.size_bytes for e in envs)
-    env_by_path = {e.path: e for e in envs}
-    high = [s for s in suggestions if s.category == "HIGH"]
+def _summarise(
+    suggestions: list[Suggestion],
+    scored_envs: list[ScoredEnvironment],
+) -> tuple[int, int, dict[str, int]]:
+    """Return ``(total_size_bytes, wasted_bytes, category_counts)``.
+
+    *wasted* sums the sizes of HIGH-category environments at or above
+    :data:`_IMPACTFUL_SIZE_BYTES`; *category_counts* maps each category to the
+    number of suggestions in it.  Computed once and shared by both output paths.
+    """
+    total_size = sum(se.env.size_bytes for se in scored_envs)
+    high_paths = {s.env_path for s in suggestions if s.category == "HIGH"}
     wasted = sum(
-        env_by_path[s.env_path].size_bytes
-        for s in high
-        if s.env_path in env_by_path
-        and env_by_path[s.env_path].size_bytes >= _IMPACTFUL_SIZE_BYTES
+        se.env.size_bytes
+        for se in scored_envs
+        if se.env.path in high_paths and se.env.size_bytes >= _IMPACTFUL_SIZE_BYTES
+    )
+    counts = {
+        "HIGH": sum(1 for s in suggestions if s.category == "HIGH"),
+        "MEDIUM": sum(1 for s in suggestions if s.category == "MEDIUM"),
+        "LOW": sum(1 for s in suggestions if s.category == "LOW"),
+    }
+    return total_size, wasted, counts
+
+
+def _output_json(
+    suggestions: list[Suggestion], scored_envs: list[ScoredEnvironment]
+) -> None:
+    total_size, wasted, counts = _summarise(suggestions, scored_envs)
+    ignored_small = sum(
+        1 for se in scored_envs if se.env.size_bytes < _SMALL_SIZE_BYTES
     )
-    ignored_small = sum(1 for e in envs if e.size_bytes < _SMALL_SIZE_BYTES)
     data = {
-        "total_environments": len(envs),
+        "total_environments": len(scored_envs),
         "total_size_bytes": total_size,
         "total_size_human": format_size(total_size),
         "wasted_size_bytes": wasted,
         "wasted_size_human": format_size(wasted),
         "ignored_small_envs_count": ignored_small,
-        "counts": {
-            "HIGH": sum(1 for s in suggestions if s.category == "HIGH"),
-            "MEDIUM": sum(1 for s in suggestions if s.category == "MEDIUM"),
-            "LOW": sum(1 for s in suggestions if s.category == "LOW"),
-        },
-        "suggestions": [
-            {
-                "env_path": str(s.env_path),
-                "score": s.score,
-                "category": s.category,
-                "reasons": s.reasons,
-                "recommended_action": s.recommended_action,
-            }
-            for s in suggestions
-        ],
+        "counts": counts,
+        "suggestions": [s.to_dict() for s in suggestions],
     }
     click.echo(json.dumps(data, indent=2))
 
@@ -129,19 +138,12 @@ def _output_json(suggestions: list[Suggestion], envs) -> None:  # type: ignore[t
 def _output_rich(
     console: Console,
     suggestions: list[Suggestion],
-    scored_envs,  # type: ignore[type-arg]
+    scored_envs: list[ScoredEnvironment],
     path: Path,
     *,
     show_all: bool = False,
 ) -> None:
-    total_size = sum(se.env.size_bytes for se in scored_envs)
-    high_suggestions = [s for s in suggestions if s.category == "HIGH"]
-    high_paths = {s.env_path for s in high_suggestions}
-    wasted = sum(
-        se.env.size_bytes
-        for se in scored_envs
-        if se.env.path in high_paths and se.env.size_bytes >= _IMPACTFUL_SIZE_BYTES
-    )
+    total_size, wasted, counts = _summarise(suggestions, scored_envs)
 
     console.print()
     console.rule("[bold cyan]Environment Health Report[/bold cyan]")
@@ -153,11 +155,6 @@ def _output_rich(
     )
 
     # Category summary
-    counts = {
-        "HIGH": sum(1 for s in suggestions if s.category == "HIGH"),
-        "MEDIUM": sum(1 for s in suggestions if s.category == "MEDIUM"),
-        "LOW": sum(1 for s in suggestions if s.category == "LOW"),
-    }
     console.print(
         f"  [bold red]HIGH[/bold red] (safe to delete): {counts['HIGH']}  "
         f"[bold yellow]MEDIUM[/bold yellow] (review): {counts['MEDIUM']}  "
@@ -198,7 +195,7 @@ def _output_rich(
 
         now = datetime.now(tz=timezone.utc)
         for se in top:
-            la = se.env.last_accessed
+            la = se.env.last_modified
             if la.tzinfo is None:
                 la = la.replace(tzinfo=timezone.utc)
             age_days = (now - la).days
@@ -267,7 +264,7 @@ def _print_category_table(
 
     now = datetime.now(tz=timezone.utc)
     for se in scored_envs:
-        la = se.env.last_accessed
+        la = se.env.last_modified
         if la.tzinfo is None:
             la = la.replace(tzinfo=timezone.utc)
         age_days = (now - la).days

{killpy-0.25.0 → killpy-1.0.0}/killpy/commands/list.py

@@ -11,6 +11,7 @@ from rich.table import Table
 
 from killpy.commands._utils import filter_envs
 from killpy.files import format_size
+from killpy.models import Environment
 from killpy.scanner import Scanner
 
 
@@ -38,7 +39,7 @@ def _run_json_stream(
 
 def _scan_with_progress(
     scanner: Scanner, path: Path, quiet: bool, stderr_console: Console
-):
+) -> list[Environment]:
     if quiet:
         return scanner.scan(path)
 
@@ -57,7 +58,7 @@ def _print_table(envs: list, console: Console) -> None:
     table = Table(show_header=True, header_style="bold cyan")
     table.add_column("Type", style="dim", min_width=10)
     table.add_column("Name", min_width=20)
-    table.add_column("Last accessed", min_width=12)
+    table.add_column("Last modified", min_width=12)
     table.add_column("Size", justify="right", min_width=9)
     table.add_column("Path")
 
@@ -65,7 +66,7 @@ def _print_table(envs: list, console: Console) -> None:
         table.add_row(
             env.type,
             env.name,
-            env.last_accessed_str,
+            env.last_modified_str,
             env.size_human,
             str(env.path),
         )
@@ -100,7 +101,7 @@ def _print_table(envs: list, console: Console) -> None:
     type=int,
     default=None,
     metavar="DAYS",
-    help="Only show environments not accessed in the last N days.",
+    help="Only show environments not modified in the last N days.",
 )
 @click.option(
     "--json-stream",