bugcap 0.4.0__tar.gz → 0.5.1__tar.gz

{bugcap-0.4.0/src/bugcap.egg-info → bugcap-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bugcap
-Version: 0.4.0
+Version: 0.5.1
 Summary: Local-first, agent-readable bug capture: annotated screenshots + notes, stored locally, synced to trackers on your own schedule.
 Author: Konrad Tagnon Amen ALAHASSA
 License: MIT
@@ -12,10 +12,11 @@ Provides-Extra: mcp
 Requires-Dist: mcp<2,>=1.2; extra == "mcp"
 Provides-Extra: dev
 Requires-Dist: pytest>=7; extra == "dev"
+Requires-Dist: ruff>=0.6; extra == "dev"
 Dynamic: license-file
 
 <p align="center">
-  <img src="docs/assets/logo.svg" alt="bugcap" width="96">
+  <img src="https://raw.githubusercontent.com/amenalahassa/bugcap/master/docs/assets/logo.svg" alt="bugcap" width="96">
 </p>
 
 <h1 align="center">bugcap</h1>
@@ -28,7 +29,7 @@ Dynamic: license-file
 <p align="center">
   <a href="https://github.com/amenalahassa/bugcap/actions/workflows/release.yml"><img alt="tests" src="https://github.com/amenalahassa/bugcap/actions/workflows/release.yml/badge.svg"></a>
   <img alt="python" src="https://img.shields.io/badge/python-3.10%2B-3776ab?logo=python&logoColor=white">
-  <a href="LICENSE"><img alt="license" src="https://img.shields.io/badge/license-MIT-green"></a>
+  <a href="https://github.com/amenalahassa/bugcap/blob/master/LICENSE"><img alt="license" src="https://img.shields.io/badge/license-MIT-green"></a>
   <a href="https://amenalahassa.github.io/bugcap/"><img alt="docs" src="https://img.shields.io/badge/docs-website-d9480f"></a>
 </p>
 
@@ -37,11 +38,11 @@ Dynamic: license-file
   <a href="https://amenalahassa.github.io/bugcap/usage.html">Usage guide</a> ·
   <a href="#quick-start">Quick start</a> ·
   <a href="#connect-your-agent-mcp">Agents (MCP)</a> ·
-  <a href="CHANGELOG.md">Changelog</a>
+  <a href="https://github.com/amenalahassa/bugcap/blob/master/CHANGELOG.md">Changelog</a>
 </p>
 
 <p align="center">
-  <img src="docs/assets/demo.gif" alt="bugcap capturing an annotated bug and listing the report" width="760">
+  <img src="https://raw.githubusercontent.com/amenalahassa/bugcap/master/docs/assets/demo.gif" alt="bugcap live: screenshot a broken page, file a bug with a note, then read it back with bugcap show" width="760">
 </p>
 
 ---
@@ -120,7 +121,7 @@ the error toast"*. It can read the picture because it's a local file, not a CDN
 Working. Roadmap items 1–5, 7 and 8 are implemented and tested, along with spec 002 (images,
 `@` references, recording, live mode and the dashboard). S3/R2 object storage (item 6) and other
 trackers (item 9) are deferred. The `Destination` seam is in place, so they can be added without
-changing the CLI. See [ROADMAP.md](ROADMAP.md).
+changing the CLI. See [ROADMAP.md](https://github.com/amenalahassa/bugcap/blob/master/ROADMAP.md).
 
 ## Installation
 
@@ -150,9 +151,13 @@ Notes on the more involved behaviour:
 - **Repo values** (`init`, `sync`, `config repo set`) are checked for format and, with `gh`, for existence. Changing the repo identity offers to move existing reports.
 - **Media sync** over `[sync] max_upload_mb` (default 25) is skipped with a warning.
 
+## Contributing
+
+Contributions are welcome. Read [CONTRIBUTING.md](https://github.com/amenalahassa/bugcap/blob/master/CONTRIBUTING.md) for the workflow and project rules, and the [Code of Conduct](https://github.com/amenalahassa/bugcap/blob/master/CODE_OF_CONDUCT.md). Report security issues privately, see [SECURITY.md](https://github.com/amenalahassa/bugcap/blob/master/SECURITY.md).
+
 ## Development
 
-See [DEVELOPMENT.md](DEVELOPMENT.md). Run the tests with:
+See [DEVELOPMENT.md](https://github.com/amenalahassa/bugcap/blob/master/DEVELOPMENT.md). Run the tests with:
 
 ```bash
 uv run --extra dev --extra mcp pytest -q
@@ -203,7 +208,7 @@ succeeds, so a re-run skips what is already recorded.
 
 ## Roadmap
 
-See [ROADMAP.md](ROADMAP.md).
+See [ROADMAP.md](https://github.com/amenalahassa/bugcap/blob/master/ROADMAP.md).
 
 ## Design notes / deliberate non-goals
 
@@ -222,4 +227,4 @@ See [ROADMAP.md](ROADMAP.md).
 
 ## License
 
-MIT — see [LICENSE](LICENSE).
+MIT — see [LICENSE](https://github.com/amenalahassa/bugcap/blob/master/LICENSE).

{bugcap-0.4.0 → bugcap-0.5.1}/README.md

@@ -1,5 +1,5 @@
 <p align="center">
-  <img src="docs/assets/logo.svg" alt="bugcap" width="96">
+  <img src="https://raw.githubusercontent.com/amenalahassa/bugcap/master/docs/assets/logo.svg" alt="bugcap" width="96">
 </p>
 
 <h1 align="center">bugcap</h1>
@@ -12,7 +12,7 @@
 <p align="center">
   <a href="https://github.com/amenalahassa/bugcap/actions/workflows/release.yml"><img alt="tests" src="https://github.com/amenalahassa/bugcap/actions/workflows/release.yml/badge.svg"></a>
   <img alt="python" src="https://img.shields.io/badge/python-3.10%2B-3776ab?logo=python&logoColor=white">
-  <a href="LICENSE"><img alt="license" src="https://img.shields.io/badge/license-MIT-green"></a>
+  <a href="https://github.com/amenalahassa/bugcap/blob/master/LICENSE"><img alt="license" src="https://img.shields.io/badge/license-MIT-green"></a>
   <a href="https://amenalahassa.github.io/bugcap/"><img alt="docs" src="https://img.shields.io/badge/docs-website-d9480f"></a>
 </p>
 
@@ -21,11 +21,11 @@
   <a href="https://amenalahassa.github.io/bugcap/usage.html">Usage guide</a> ·
   <a href="#quick-start">Quick start</a> ·
   <a href="#connect-your-agent-mcp">Agents (MCP)</a> ·
-  <a href="CHANGELOG.md">Changelog</a>
+  <a href="https://github.com/amenalahassa/bugcap/blob/master/CHANGELOG.md">Changelog</a>
 </p>
 
 <p align="center">
-  <img src="docs/assets/demo.gif" alt="bugcap capturing an annotated bug and listing the report" width="760">
+  <img src="https://raw.githubusercontent.com/amenalahassa/bugcap/master/docs/assets/demo.gif" alt="bugcap live: screenshot a broken page, file a bug with a note, then read it back with bugcap show" width="760">
 </p>
 
 ---
@@ -104,7 +104,7 @@ the error toast"*. It can read the picture because it's a local file, not a CDN
 Working. Roadmap items 1–5, 7 and 8 are implemented and tested, along with spec 002 (images,
 `@` references, recording, live mode and the dashboard). S3/R2 object storage (item 6) and other
 trackers (item 9) are deferred. The `Destination` seam is in place, so they can be added without
-changing the CLI. See [ROADMAP.md](ROADMAP.md).
+changing the CLI. See [ROADMAP.md](https://github.com/amenalahassa/bugcap/blob/master/ROADMAP.md).
 
 ## Installation
 
@@ -134,9 +134,13 @@ Notes on the more involved behaviour:
 - **Repo values** (`init`, `sync`, `config repo set`) are checked for format and, with `gh`, for existence. Changing the repo identity offers to move existing reports.
 - **Media sync** over `[sync] max_upload_mb` (default 25) is skipped with a warning.
 
+## Contributing
+
+Contributions are welcome. Read [CONTRIBUTING.md](https://github.com/amenalahassa/bugcap/blob/master/CONTRIBUTING.md) for the workflow and project rules, and the [Code of Conduct](https://github.com/amenalahassa/bugcap/blob/master/CODE_OF_CONDUCT.md). Report security issues privately, see [SECURITY.md](https://github.com/amenalahassa/bugcap/blob/master/SECURITY.md).
+
 ## Development
 
-See [DEVELOPMENT.md](DEVELOPMENT.md). Run the tests with:
+See [DEVELOPMENT.md](https://github.com/amenalahassa/bugcap/blob/master/DEVELOPMENT.md). Run the tests with:
 
 ```bash
 uv run --extra dev --extra mcp pytest -q
@@ -187,7 +191,7 @@ succeeds, so a re-run skips what is already recorded.
 
 ## Roadmap
 
-See [ROADMAP.md](ROADMAP.md).
+See [ROADMAP.md](https://github.com/amenalahassa/bugcap/blob/master/ROADMAP.md).
 
 ## Design notes / deliberate non-goals
 
@@ -206,4 +210,4 @@ See [ROADMAP.md](ROADMAP.md).
 
 ## License
 
-MIT — see [LICENSE](LICENSE).
+MIT — see [LICENSE](https://github.com/amenalahassa/bugcap/blob/master/LICENSE).

{bugcap-0.4.0 → bugcap-0.5.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "bugcap"
-version = "0.4.0"
+version = "0.5.1"
 description = "Local-first, agent-readable bug capture: annotated screenshots + notes, stored locally, synced to trackers on your own schedule."
 readme = "README.md"
 requires-python = ">=3.10"
@@ -14,7 +14,7 @@ dependencies = ["tomli>=2; python_version < '3.11'"]
 
 [project.optional-dependencies]
 mcp = ["mcp>=1.2,<2"]
-dev = ["pytest>=7"]
+dev = ["pytest>=7", "ruff>=0.6"]
 
 [project.scripts]
 bugcap = "bugcap.cli:main"
@@ -27,3 +27,13 @@ bugcap = ["dashboard/static/*"]
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
+
+[tool.ruff]
+line-length = 120
+target-version = "py310"
+
+[tool.ruff.lint]
+select = ["E4", "E7", "E9", "F", "I"]
+
+[tool.ruff.lint.per-file-ignores]
+"tests/*" = ["E702", "E731"]

bugcap-0.5.1/src/bugcap/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.5.1"

{bugcap-0.4.0 → bugcap-0.5.1}/src/bugcap/cli.py

@@ -7,13 +7,12 @@ import threading
 from pathlib import Path
 from typing import Optional
 
-from . import backends, capture, config, ghcli, recorder, refs, repo, service, sync, validation
+from . import __version__, backends, capture, config, ghcli, recorder, refs, repo, service, sync, validation
 from .capture import CaptureError
 from .errors import ServiceError
 from .ghcli import GhError
 from .store import STATUSES, Store
 
-
 # --- shared helpers (T014) ----------------------------------------------------
 
 def current_repo():
@@ -152,12 +151,8 @@ def cmd_list(args: argparse.Namespace) -> int:
     return 0
 
 
-def cmd_show(args: argparse.Namespace) -> int:
-    with Store() as store:
-        report = resolve_report(store, args.id)
-        if report is None:
-            return 1
-
+def print_report(report) -> None:
+    """The multi-line details of one report, as `bugcap show` prints them."""
     print(f"#{report.id}  {report.title}")
     print(f"  created_at: {report.created_at}")
     print(f"  status:     {report.status}")
@@ -165,7 +160,7 @@ def cmd_show(args: argparse.Namespace) -> int:
         print(f"  repo:       {report.repo}")
     print(f"  tags:       {', '.join(report.tags) or '(none)'}")
     print(f"  notes:      {refs.display_notes(report.notes, report.media) or '(none)'}")
-    print(f"  images:")
+    print("  images:")
     for path in report.image_paths:
         print(f"    - {path}")
     detailed = [m for m in report.media if not (m.kind == "image" and m.source == "legacy")]
@@ -175,12 +170,21 @@ def cmd_show(args: argparse.Namespace) -> int:
             where = m.path or f"({len(m.frames)} frames)"
             origin = f"  (source: {m.source})" if m.source else ""
             print(f"    {refs.media_token(m)}  {m.kind:8s} {m.label or '-':12s} {where}  {human_size(m.size_bytes)}{origin}")
-    print(f"  synced_refs:")
+    print("  synced_refs:")
     if report.synced_refs:
         for tracker, ref in report.synced_refs.items():
             print(f"    - {tracker}: {ref}")
     else:
         print("    (not synced to any tracker yet)")
+
+
+def cmd_show(args: argparse.Namespace) -> int:
+    with Store() as store:
+        report = resolve_report(store, args.id)
+        if report is None:
+            return 1
+
+    print_report(report)
     return 0
 
 
@@ -574,6 +578,8 @@ def cmd_attach(args: argparse.Namespace) -> int:
         print(f"Attached {shot_path} to report #{args.id}")
     elif note and added:
         print(f"Added note to report #{args.id}")
+    print()
+    print_report(current)
     return 1 if rejected or (note and not added) else 0
 
 
@@ -843,6 +849,7 @@ def build_parser() -> argparse.ArgumentParser:
         prog="bugcap",
         description="Local-first, agent-readable bug capture: annotated screenshots + notes.",
     )
+    parser.add_argument("--version", action="version", version=f"bugcap {__version__}")
     sub = parser.add_subparsers(dest="command", required=True)
 
     p = sub.add_parser("setup", help="Detect a capture tool, or recommend/install one.")

{bugcap-0.4.0 → bugcap-0.5.1}/src/bugcap/ingest.py

@@ -1,11 +1,13 @@
 """Bring existing images into the store: local paths, globs and http(s) URLs.
 
 Every input is validated by its magic bytes (never its extension or Content-Type), copied
-into the images directory under a fresh name (the original path is not kept), and written
-atomically. Network access is limited to plain http(s) GETs with no credentials."""
+into the images directory under a name derived from its content (the original path is not
+kept) and written atomically. The same image is therefore stored once, however many reports
+use it. Network access is limited to plain http(s) GETs with no credentials."""
 from __future__ import annotations
 
 import glob
+import hashlib
 import os
 import socket
 import urllib.error
@@ -32,6 +34,7 @@ class StoredFile:
     mime: str
     size_bytes: int
     source: str
+    reused: bool = False  # the content was already in the store; no new file was written
 
 
 @dataclass
@@ -70,10 +73,16 @@ def sniff_image(data: bytes) -> str:
     raise ServiceError("invalid_image", "not an image (PNG, JPEG, GIF or WebP expected)")
 
 
+def content_name(data: bytes, mime: str) -> str:
+    return f"{hashlib.sha256(data).hexdigest()}{_EXT[mime]}"
+
+
 def _store(data: bytes, source: str) -> StoredFile:
     mime = sniff_image(data)
-    dest = images_dir() / f"{uuid.uuid4()}{_EXT[mime]}"
-    tmp = dest.with_name(dest.name + ".part")
+    dest = images_dir() / content_name(data, mime)
+    if dest.is_file() and dest.stat().st_size == len(data):
+        return StoredFile(to_data_relative(dest), dest, mime, len(data), source, reused=True)
+    tmp = dest.with_name(f"{dest.name}.{uuid.uuid4().hex}.part")
     try:
         tmp.write_bytes(data)
         os.replace(tmp, dest)
@@ -82,6 +91,29 @@ def _store(data: bytes, source: str) -> StoredFile:
     return StoredFile(to_data_relative(dest), dest, mime, len(data), source)
 
 
+def adopt(path) -> Path:
+    """Give a screenshot the capture tool already saved in the store its content-derived
+    name. If the same image is stored already, the new copy is dropped and the existing
+    path returned. Anything that is not a recognisable image, or sits outside the store, is
+    left where it is."""
+    path = Path(path)
+    if path.parent != images_dir():
+        return path
+    try:
+        data = path.read_bytes()
+        mime = sniff_image(data)
+    except (OSError, ServiceError):
+        return path
+    dest = images_dir() / content_name(data, mime)
+    if dest == path:
+        return path
+    if dest.is_file() and dest.stat().st_size == len(data):
+        path.unlink(missing_ok=True)
+        return dest
+    os.replace(path, dest)
+    return dest
+
+
 def import_local(path: Path) -> StoredFile:
     path = Path(path).expanduser()
     if path.is_dir():

{bugcap-0.4.0 → bugcap-0.5.1}/src/bugcap/live.py

@@ -85,7 +85,7 @@ class _App:
 
     def style(self) -> None:
         """One consistent look for every window: a flat light theme with an accent button."""
-        tk, ttk = self.tk, self.ttk
+        ttk = self.ttk
         import tkinter.font as tkfont
 
         base = tkfont.nametofont("TkDefaultFont")

{bugcap-0.4.0 → bugcap-0.5.1}/src/bugcap/service.py

@@ -81,6 +81,8 @@ class AddResult:
 
 def _discard(stored_files) -> None:
     for stored in stored_files:
+        if stored.reused:  # shared with other media; not ours to delete
+            continue
         try:
             stored.abs_path.unlink()
         except OSError:
@@ -158,6 +160,7 @@ def create_report(
         with store.transaction():
             report = store.add(title=title, notes="", tags=tags, repo=repo, body=body, status=status)
             if captured is not None:
+                captured = str(ingest.adopt(captured))
                 result.added.append(
                     store.insert_media(report.id, kind="image", path=captured,
                                        label=captured_label or None, source="captured")
@@ -194,7 +197,7 @@ def delete_report(store: Store, report_id: int) -> Report:
     report = _require(store, report_id)
     store.delete(report_id)
     for item in report.media:
-        _delete_files(item)
+        _delete_files(store, item)
     return report
 
 
@@ -202,6 +205,7 @@ def attach_captured(store: Store, report_id: int, path: str, label: Optional[str
     """Attach a screenshot the capture tool already saved in the store (no copy)."""
     report = _require(store, report_id)
     _check_new_labels(report.media, [label])
+    path = ingest.adopt(path)
     return store.insert_media(report_id, kind="image", path=str(path), label=label or None, source="captured")
 
 
@@ -343,14 +347,18 @@ def remove_media(store: Store, report_id: int, ref: str, force: bool = False) ->
         notes, count = refs.rewrite_references_counted(report.notes, mapping)
         if count:
             store.update(report_id, notes=notes)
-    _delete_files(item)
+    _delete_files(store, item)
     return count
 
 
-def _delete_files(item: Media) -> None:
+def _delete_files(store: Store, item: Media) -> None:
+    """Delete an item's files unless another media item still uses them (identical images
+    are stored once and shared). Call after the item's rows are gone."""
     paths = [item.path] if item.path else []
     paths += [f.path for f in item.frames]
     for rel in paths:
+        if store.path_in_use(rel):
+            continue
         try:
             os.unlink(resolve_data_path(rel))
         except (ValueError, OSError):

{bugcap-0.4.0 → bugcap-0.5.1}/src/bugcap/store.py

@@ -425,6 +425,14 @@ class Store:
         ).fetchall()
         return [self._load_media(r) for r in rows]
 
+    def path_in_use(self, path: str) -> bool:
+        """Whether any media item or frame still points at this stored file."""
+        row = self._conn.execute(
+            "SELECT EXISTS(SELECT 1 FROM media WHERE path = ?) OR EXISTS(SELECT 1 FROM media_frames WHERE path = ?)",
+            (path, path),
+        ).fetchone()
+        return bool(row[0])
+
     def get_media(self, media_id: int) -> Optional[Media]:
         row = self._conn.execute("SELECT * FROM media WHERE id = ?", (media_id,)).fetchone()
         return self._load_media(row) if row else None

{bugcap-0.4.0 → bugcap-0.5.1/src/bugcap.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bugcap
-Version: 0.4.0
+Version: 0.5.1
 Summary: Local-first, agent-readable bug capture: annotated screenshots + notes, stored locally, synced to trackers on your own schedule.
 Author: Konrad Tagnon Amen ALAHASSA
 License: MIT
@@ -12,10 +12,11 @@ Provides-Extra: mcp
 Requires-Dist: mcp<2,>=1.2; extra == "mcp"
 Provides-Extra: dev
 Requires-Dist: pytest>=7; extra == "dev"
+Requires-Dist: ruff>=0.6; extra == "dev"
 Dynamic: license-file
 
 <p align="center">
-  <img src="docs/assets/logo.svg" alt="bugcap" width="96">
+  <img src="https://raw.githubusercontent.com/amenalahassa/bugcap/master/docs/assets/logo.svg" alt="bugcap" width="96">
 </p>
 
 <h1 align="center">bugcap</h1>
@@ -28,7 +29,7 @@ Dynamic: license-file
 <p align="center">
   <a href="https://github.com/amenalahassa/bugcap/actions/workflows/release.yml"><img alt="tests" src="https://github.com/amenalahassa/bugcap/actions/workflows/release.yml/badge.svg"></a>
   <img alt="python" src="https://img.shields.io/badge/python-3.10%2B-3776ab?logo=python&logoColor=white">
-  <a href="LICENSE"><img alt="license" src="https://img.shields.io/badge/license-MIT-green"></a>
+  <a href="https://github.com/amenalahassa/bugcap/blob/master/LICENSE"><img alt="license" src="https://img.shields.io/badge/license-MIT-green"></a>
   <a href="https://amenalahassa.github.io/bugcap/"><img alt="docs" src="https://img.shields.io/badge/docs-website-d9480f"></a>
 </p>
 
@@ -37,11 +38,11 @@ Dynamic: license-file
   <a href="https://amenalahassa.github.io/bugcap/usage.html">Usage guide</a> ·
   <a href="#quick-start">Quick start</a> ·
   <a href="#connect-your-agent-mcp">Agents (MCP)</a> ·
-  <a href="CHANGELOG.md">Changelog</a>
+  <a href="https://github.com/amenalahassa/bugcap/blob/master/CHANGELOG.md">Changelog</a>
 </p>
 
 <p align="center">
-  <img src="docs/assets/demo.gif" alt="bugcap capturing an annotated bug and listing the report" width="760">
+  <img src="https://raw.githubusercontent.com/amenalahassa/bugcap/master/docs/assets/demo.gif" alt="bugcap live: screenshot a broken page, file a bug with a note, then read it back with bugcap show" width="760">
 </p>
 
 ---
@@ -120,7 +121,7 @@ the error toast"*. It can read the picture because it's a local file, not a CDN
 Working. Roadmap items 1–5, 7 and 8 are implemented and tested, along with spec 002 (images,
 `@` references, recording, live mode and the dashboard). S3/R2 object storage (item 6) and other
 trackers (item 9) are deferred. The `Destination` seam is in place, so they can be added without
-changing the CLI. See [ROADMAP.md](ROADMAP.md).
+changing the CLI. See [ROADMAP.md](https://github.com/amenalahassa/bugcap/blob/master/ROADMAP.md).
 
 ## Installation
 
@@ -150,9 +151,13 @@ Notes on the more involved behaviour:
 - **Repo values** (`init`, `sync`, `config repo set`) are checked for format and, with `gh`, for existence. Changing the repo identity offers to move existing reports.
 - **Media sync** over `[sync] max_upload_mb` (default 25) is skipped with a warning.
 
+## Contributing
+
+Contributions are welcome. Read [CONTRIBUTING.md](https://github.com/amenalahassa/bugcap/blob/master/CONTRIBUTING.md) for the workflow and project rules, and the [Code of Conduct](https://github.com/amenalahassa/bugcap/blob/master/CODE_OF_CONDUCT.md). Report security issues privately, see [SECURITY.md](https://github.com/amenalahassa/bugcap/blob/master/SECURITY.md).
+
 ## Development
 
-See [DEVELOPMENT.md](DEVELOPMENT.md). Run the tests with:
+See [DEVELOPMENT.md](https://github.com/amenalahassa/bugcap/blob/master/DEVELOPMENT.md). Run the tests with:
 
 ```bash
 uv run --extra dev --extra mcp pytest -q
@@ -203,7 +208,7 @@ succeeds, so a re-run skips what is already recorded.
 
 ## Roadmap
 
-See [ROADMAP.md](ROADMAP.md).
+See [ROADMAP.md](https://github.com/amenalahassa/bugcap/blob/master/ROADMAP.md).
 
 ## Design notes / deliberate non-goals
 
@@ -222,4 +227,4 @@ See [ROADMAP.md](ROADMAP.md).
 
 ## License
 
-MIT — see [LICENSE](LICENSE).
+MIT — see [LICENSE](https://github.com/amenalahassa/bugcap/blob/master/LICENSE).

{bugcap-0.4.0 → bugcap-0.5.1}/src/bugcap.egg-info/requires.txt

@@ -4,6 +4,7 @@ tomli>=2
 
 [dev]
 pytest>=7
+ruff>=0.6
 
 [mcp]
 mcp<2,>=1.2

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_backends.py

@@ -1,5 +1,4 @@
 """T011: backend detection, recommendation, install-command and guidance per simulated OS."""
-import os
 
 import pytest
 

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_cli_attach.py

@@ -1,7 +1,7 @@
 """T026: `bugcap attach`."""
 import pytest
-
 from fixtures.make_images import png_bytes
+
 from bugcap import backends, capture, cli
 from bugcap.store import Store
 
@@ -169,3 +169,77 @@ def test_pull_ask_flow_appends_note(bugcap_home, monkeypatch, tmp_path):
         sync._attach_answer(store, rid, None)
         report = store.get(rid)
     assert len(report.media) == 2 and report.notes.endswith("@i1: seen on pull")
+
+
+# --- details after attach / shared storage -----------------------------------------------
+
+def test_attach_prints_the_bug_details_after_the_usual_output(bugcap_home, tmp_path, capsys):
+    with Store() as store:
+        rid = store.add("Login fails", tags=["auth"]).id
+    img = tmp_path / "new.png"
+    img.write_bytes(png_bytes())
+    assert cli.main(["attach", str(rid), "--image", str(img), "--note", "see this"]) == 0
+    out = capsys.readouterr().out
+    assert out.index("added @i1") < out.index(f"#{rid}  Login fails")
+    assert "tags:       auth" in out and "see this" in out and "media:" in out and "i1" in out
+
+
+def test_attach_with_a_rejected_image_still_shows_the_unchanged_bug(bugcap_home, tmp_path, capsys):
+    with Store() as store:
+        rid = store.add("Bug").id
+    assert cli.main(["attach", str(rid), "--image", str(tmp_path / "missing.png")]) == 1
+    captured = capsys.readouterr()
+    assert "skipped" in captured.err and f"#{rid}  Bug" in captured.out and "media:" not in captured.out
+
+
+def test_attach_unknown_id_prints_no_details(bugcap_home, capsys):
+    assert cli.main(["attach", "999", "--image", "x.png"]) == 1
+    assert capsys.readouterr().out == ""
+
+
+def test_same_image_on_two_bugs_is_stored_once(bugcap_home, tmp_path):
+    from bugcap.paths import images_dir
+
+    with Store() as store:
+        a, b = store.add("A").id, store.add("B").id
+    img = tmp_path / "same.png"
+    img.write_bytes(png_bytes())
+    assert cli.main(["attach", str(a), "--image", str(img)]) == 0
+    assert cli.main(["attach", str(b), "--image", str(img)]) == 0
+    with Store() as store:
+        pa, pb = store.get(a).media[0].path, store.get(b).media[0].path
+    assert pa == pb and len(list(images_dir().iterdir())) == 1
+
+
+def test_shared_file_survives_until_its_last_user_is_gone(bugcap_home, tmp_path):
+    from bugcap import service
+
+    with Store() as store:
+        a, b = store.add("A").id, store.add("B").id
+    img = tmp_path / "same.png"
+    img.write_bytes(png_bytes())
+    for rid in (a, b):
+        assert cli.main(["attach", str(rid), "--image", str(img)]) == 0
+    with Store() as store:
+        path = store.get(a).media[0].abs_path
+        service.remove_media(store, a, "i1")
+        assert __import__("os").path.isfile(path)  # B still uses it
+        service.delete_report(store, b)
+    assert not __import__("os").path.exists(path)
+
+
+def test_same_image_twice_in_one_attach_and_failed_attach_keep_shared_file(bugcap_home, tmp_path):
+    from bugcap import service
+
+    with Store() as store:
+        a, b = store.add("A").id, store.add("B").id
+        img = tmp_path / "same.png"
+        img.write_bytes(png_bytes())
+        service.add_media(store, a, [str(img)])
+        # B asks for the same image with a label that cannot be used: nothing may be deleted
+        with pytest.raises(Exception):
+            service.add_media(store, b, [str(img)], labels=["i1"])
+        path = store.get(a).media[0].abs_path
+        assert __import__("os").path.isfile(path)
+        res = service.add_media(store, b, [str(img), str(img)])
+        assert len(res.added) == 2 and len({m.path for m in res.added}) == 1

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_cli_init_capture_list.py

@@ -1,7 +1,7 @@
 """T016: init writes config; capture scopes inside a repo; list filters; --all; legacy rows."""
-import pytest
 
 from fixtures.make_images import png_bytes
+
 from bugcap import cli, repo
 from bugcap.store import Store
 

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_cli_live_headless.py

@@ -20,8 +20,12 @@ def test_missing_tkinter_exits_3_with_install_hint(monkeypatch, capsys):
     assert "python3-tk" in capsys.readouterr().err
 
 
+@pytest.mark.skipif(
+    sys.platform in ("win32", "darwin"),
+    reason="DISPLAY only means something on X11; elsewhere Tk opens a real window and blocks",
+)
 def test_unreachable_display_exits_3(monkeypatch, capsys):
-    tk = pytest.importorskip("tkinter")
+    pytest.importorskip("tkinter")
     monkeypatch.setenv("DISPLAY", ":99999")
     assert cli.main(["live"]) == 3
     assert "cannot open a window" in capsys.readouterr().err

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_cli_media.py

@@ -1,8 +1,8 @@
-import pytest
+
+from fixtures.make_v1_db import build
 
 from bugcap import capture, cli
 from bugcap.store import Store
-from fixtures.make_v1_db import build
 
 
 def _boom(*a, **k):

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_cli_no_image_delete.py

@@ -1,7 +1,7 @@
 """`capture --no-image` saves a text-only report; `delete` removes a report and its files."""
-import pytest
 
 from fixtures.make_images import png_bytes
+
 from bugcap import cli, service
 from bugcap.paths import resolve_data_path
 from bugcap.store import Store

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_cli_setup.py

@@ -79,3 +79,12 @@ def test_no_package_manager_prints_guidance(monkeypatch, record_run, capsys):
     assert cli.main(["setup"]) == 1
     assert "manually" in capsys.readouterr().err.lower()
     assert record_run == []
+
+
+def test_version_flag(capsys):
+    from bugcap import __version__
+
+    with pytest.raises(SystemExit) as exc:
+        cli.main(["--version"])
+    assert exc.value.code == 0
+    assert capsys.readouterr().out.strip() == f"bugcap {__version__}"

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_cli_triage.py

@@ -1,5 +1,4 @@
 """T022: `bugcap edit` and `bugcap tag`."""
-import pytest
 
 from bugcap import cli
 from bugcap.store import Store

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_drafts.py

@@ -1,8 +1,9 @@
 import json
 
-from bugcap import drafts
 from fixtures.make_images import png_bytes
 
+from bugcap import drafts
+
 
 def _png(tmp_path, name="s.png"):
     p = tmp_path / name

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_github_media_sync.py

@@ -2,10 +2,11 @@
 import base64
 import json
 
+from fixtures.make_images import png_bytes
+
 from bugcap import service, sync
 from bugcap.store import Store
 from bugcap.sync import GitHubDestination, ImagesTarget, SyncOptions
-from fixtures.make_images import png_bytes
 
 
 def _setup(fake_gh):

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_github_sync.py

@@ -4,7 +4,7 @@ import json
 
 import pytest
 
-from bugcap import config, sync
+from bugcap import sync
 from bugcap.store import Store
 from bugcap.sync import GitHubDestination, ImagesTarget, SyncOptions
 

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_ingest.py

@@ -1,8 +1,10 @@
+from pathlib import Path
+
 import pytest
+from fixtures.make_images import gif_bytes, jpeg_bytes, png_bytes, webp_bytes
 
 from bugcap import ingest
 from bugcap.errors import ServiceError
-from fixtures.make_images import gif_bytes, jpeg_bytes, png_bytes, webp_bytes
 
 
 def test_classify_sources(tmp_path):
@@ -54,7 +56,7 @@ def test_import_copies_into_store(bugcap_home, sample_images):
 
 def test_glob_sorted_and_no_match(sample_images):
     matches = ingest.resolve_glob(str(sample_images / "sample.*"))
-    assert [m.rsplit("/", 1)[1] for m in matches] == ["sample.gif", "sample.jpg", "sample.png", "sample.webp"]
+    assert [Path(m).name for m in matches] == ["sample.gif", "sample.jpg", "sample.png", "sample.webp"]
     items = ingest.expand_sources([str(sample_images / "nope-*.png")])
     assert items[0].error.code == "not_found" and "nope-*.png" in items[0].error.message
 

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_live_session.py

@@ -1,13 +1,13 @@
 from pathlib import Path
 
 import pytest
+from fixtures.make_images import png_bytes
 
 from bugcap import drafts, live_session, recorder
 from bugcap.errors import ServiceError
 from bugcap.live_session import LiveSession, LiveState
 from bugcap.paths import images_dir, media_dir
 from bugcap.store import Store
-from fixtures.make_images import png_bytes
 
 
 class Cfg:
@@ -97,7 +97,7 @@ def test_save_combines_images_and_a_video_in_order(session):
     assert report.title == "Login broken" and report.repo == "owner/proj"
     assert report.tags == ["auth", "proj"] and report.status == "in-progress"
     assert [(m.idx, m.kind) for m in report.media] == [(1, "image"), (2, "video"), (3, "image")]
-    assert first.exists()  # captured files live on in the store, not copied away
+    assert all(Path(m.abs_path).is_file() for m in report.media)  # captured files stay in the store
 
 
 def test_two_reports_in_a_row(session):

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_mcp.py

@@ -17,7 +17,6 @@ EXPECTED_TOOLS = {
 
 
 def _seed_report(home) -> int:
-    env = dict(os.environ, BUGCAP_HOME=str(home))
     os.environ["BUGCAP_HOME"] = str(home)
     from bugcap.store import Store
 

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_mcp_logging.py

@@ -1,6 +1,5 @@
 """BUGS.md: `bugcap mcp-serve` logs startup, client initialize, tool calls and errors, never to stdout."""
 import asyncio
-import io
 import logging
 import os
 import subprocess

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_recorder_run.py

@@ -1,5 +1,6 @@
 import signal
 import subprocess
+from pathlib import Path
 
 import pytest
 
@@ -156,11 +157,10 @@ def test_animated_runs_postprocess_and_removes_raw(env, tmp_path):
 
 
 def test_frames_result(env, tmp_path):
-    frames_dir_files = []
 
     def runner(argv, **kw):
         pattern = argv[-1]
-        folder = tmp_path / pattern.split("/")[-2]
+        folder = Path(pattern).parent
         for n in (1, 2, 3):
             (folder / f"frame-00{n}.png").write_bytes(b"x")
         return subprocess.CompletedProcess(argv, 0, "", "")
@@ -168,7 +168,7 @@ def test_frames_result(env, tmp_path):
     result = recorder.run_recording("ffmpeg", "frames", 5, 30, 25, tmp_path, lambda: True,
                                     **env.kwargs(runner=runner))
     assert result.kind == "frames" and len(result.frames) == 3 and result.path is None
-    assert [p.rsplit("/", 1)[1] for p, _ in result.frames] == ["frame-001.png", "frame-002.png", "frame-003.png"]
+    assert [Path(p).name for p, _ in result.frames] == ["frame-001.png", "frame-002.png", "frame-003.png"]
 
 
 def test_macos_probes_screen_device(env, tmp_path):

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_refs_validate_rewrite.py

@@ -2,7 +2,9 @@ import pytest
 
 from bugcap.errors import ServiceError
 from bugcap.refs import (
-    display_notes, rewrite_references, validate_references,
+    display_notes,
+    rewrite_references,
+    validate_references,
 )
 from bugcap.store import Media
 

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_regression_output.py

@@ -2,9 +2,9 @@
 
 The expected strings below are the exact pre-feature formats; the test fails if any
 scoping/repo change leaks into output when there is no .bugcap.toml in scope."""
-import pytest
 
 from fixtures.make_images import png_bytes
+
 from bugcap import cli
 from bugcap.store import Store
 

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_repo_values.py

@@ -4,7 +4,6 @@ import pytest
 from bugcap import cli, ghcli, repo, validation
 from bugcap.store import Store
 
-
 # --- validation -------------------------------------------------------------------
 
 @pytest.mark.parametrize("slug", ["owner/repo", "my-org/my.repo_1", "a/b"])

{bugcap-0.4.0 → bugcap-0.5.1}/tests/test_sync_consent.py

@@ -1,9 +1,6 @@
 """T031: image-commit consent rules (FR-018a) and images-target resolution (FR-018b/R2)."""
-import base64
 import json
 
-import pytest
-
 from bugcap import config, repo, sync
 from bugcap.store import Store
 from bugcap.sync import GitHubDestination, ImagesTarget, SyncOptions, resolve_images_target

bugcap-0.4.0/src/bugcap/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.4.0"