pix-cli 0.1.1__tar.gz → 0.1.2__tar.gz

{pix_cli-0.1.1 → pix_cli-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: pix-cli
-Version: 0.1.1
+Version: 0.1.2
 Summary: Batch AVIF shrink and image-host sync CLI
 Project-URL: Homepage, https://github.com/uzvg/pix
 Project-URL: Repository, https://github.com/uzvg/pix

{pix_cli-0.1.1 → pix_cli-0.1.2}/docs/architecture.md

@@ -51,8 +51,8 @@ src/pix/
 ├── __init__.py     # 包版本
 ├── py.typed
 ├── cli.py          # 组合根：parser、main、子命令分派
-├── config.py       # 默认值、TOML 读取与校验、parse_size、config init
-├── default_config.toml  # config init 写入的模板（随包分发）
+├── config.py       # TOML 读取与校验、parse_size、config init
+├── default_config.toml  # 默认值唯一来源，也是 config init 的模板（随包分发）
 ├── logger.py       # stderr 日志
 ├── shrink.py       # shrink 子命令
 ├── rclone.py       # rclone 探测、版本检查与执行
@@ -150,7 +150,7 @@ pix config init [--force]
 - Linux/macOS：`$XDG_CONFIG_HOME/pix/config.toml`，未设置时为 `~/.config/pix/config.toml`。
 - Windows：`%APPDATA%\pix\config.toml`。
 
-文件不存在时使用内置默认值。
+文件不存在时使用内置默认值（即 `src/pix/default_config.toml`）。
 
 ```toml
 [shrink]
@@ -230,6 +230,7 @@ binary = "rclone"
 13. 汇总输出：转换、跳过（状态命中）、跳过（太小）、跳过（空源保护）、失败、删除。
 14. 状态在编码过程中每成功 200 个文件刷新一次（原子写），并在正常结束或 `KeyboardInterrupt` 时通过 `finally` 再写一次，避免 Ctrl-C 丢掉整批进度。写入失败记录错误并返回 1，下次运行会重新编码未记录的文件。
 15. 状态损坏、版本不识别或 JSON 解析失败时输出警告，按空状态全量重建，不中止。
+16. 编码每个文件前在 stderr 输出进度 `processing <序号>/<总数>: <相对路径>`；终端下原地刷新，重定向时逐行输出，`--quiet` 抑制。日志与汇总会先结束进度行。
 
 ### 状态文件
 
@@ -343,7 +344,7 @@ config /home/uzvg/.config/pix/config.toml (ok)
 
 ## 9. 日志与退出码
 
-- 默认：阶段信息和汇总，输出到 stderr。
+- 默认：阶段信息、编码进度和汇总，输出到 stderr。终端下进度原地刷新，重定向时逐行输出。
 - `-v`：输出每个文件的处理/跳过原因，以及 rclone 的进度。
 - `--quiet`：只输出错误。
 - `doctor` 报告输出到 stdout；`--dry-run` 的计划文件列表和 `--urls` 的 JSON 输出到 stdout。

{pix_cli-0.1.1 → pix_cli-0.1.2}/docs/decisions.md

@@ -172,6 +172,6 @@
 
 ## D28. 默认配置用内置模板生成
 
-**决策**：`pix config init` 写出随包分发的 `src/pix/default_config.toml`，而不是把 `DEFAULTS` 序列化成 TOML。目标文件已存在时拒绝覆盖并退出 2，`--force` 才覆盖；该命令在 `load_settings` 之前分派。
+**决策**：随包分发的 `src/pix/default_config.toml` 是默认值的唯一来源：`load_settings` 通过 `load_defaults()` 解析它，`pix config init` 直接复制它（保留注释）。目标文件已存在时拒绝覆盖并退出 2，`--force` 才覆盖；该命令在 `load_settings` 之前分派。
 
-**原因**：Python 3.11 标准库只有 `tomllib`，没有 TOML 写入器；引入 `tomli-w` 违反“仅标准库 + pyvips”的约束。独立文件可以直接带注释，用户打开就能改，编辑器也能按 TOML 高亮；测试把模板解析后与 `DEFAULTS` 比对，防止两者漂移。拒绝覆盖保护用户已经修改过的配置，误覆盖的代价远高于多敲一个 `--force`。提前分派让 `--config` 指向新路径时也能创建工作，而不是被“config file not found”挡住。
+**原因**：Python 3.11 标准库只有 `tomllib`，没有 TOML 写入器；引入 `tomli-w` 违反“仅标准库 + pyvips”的约束。独立文件可以直接带注释，用户打开就能改，编辑器也能按 TOML 高亮；默认值只在这一处维护，避免代码常量与模板漂移。拒绝覆盖保护用户已经修改过的配置，误覆盖的代价远高于多敲一个 `--force`。提前分派让 `--config` 指向新路径时也能创建工作，而不是被“config file not found”挡住。

{pix_cli-0.1.1 → pix_cli-0.1.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pix-cli"
-version = "0.1.1"
+version = "0.1.2"
 description = "Batch AVIF shrink and image-host sync CLI"
 readme = "README.md"
 license = { text = "MIT" }

{pix_cli-0.1.1 → pix_cli-0.1.2}/src/pix/config.py

@@ -3,7 +3,6 @@
 from __future__ import annotations
 
 import argparse
-import copy
 import os
 import re
 import sys
@@ -11,29 +10,6 @@ import tomllib
 from pathlib import Path
 from typing import Any
 
-DEFAULTS: dict[str, Any] = {
-    "shrink": {
-        "width": 1920,
-        "quality": 85,
-        "effort": 5,
-        "min_size": 0,
-        "include": ["jpg", "jpeg", "png", "webp"],
-    },
-    "backup": {
-        "exclude": [],
-        "max_delete": 10,
-    },
-    "publish": {
-        "base_url": "",
-        "urls": False,
-        "exclude": [],
-        "max_delete": 10,
-    },
-    "rclone": {
-        "binary": "rclone",
-    },
-}
-
 _SIZE_RE = re.compile(r"^(\d+)([KMG]?)$", re.IGNORECASE)
 _SIZE_UNITS = {"": 1, "K": 1024, "M": 1024**2, "G": 1024**3}
 
@@ -42,6 +18,18 @@ class ConfigError(Exception):
     """Invalid or unreadable configuration."""
 
 
+def load_defaults() -> dict[str, Any]:
+    """Read the packaged template, the single source of default values."""
+    path = Path(__file__).with_name("default_config.toml")
+    try:
+        with path.open("rb") as handle:
+            return tomllib.load(handle)
+    except OSError as exc:
+        raise ConfigError(f"{path}: {exc}") from exc
+    except tomllib.TOMLDecodeError as exc:
+        raise ConfigError(f"{path}: {exc}") from exc
+
+
 def parse_size(value: str | int) -> int:
     if isinstance(value, int) and not isinstance(value, bool):
         if value < 0:
@@ -97,7 +85,7 @@ def load_settings(explicit: Path | None = None) -> dict[str, Any]:
     if explicit is not None and not path.is_file():
         raise ConfigError(f"config file not found: {path}")
     overrides = load_config(path)
-    settings = copy.deepcopy(DEFAULTS)
+    settings = load_defaults()
     _deep_update(settings, overrides)
     validate_settings(settings)
     return settings

{pix_cli-0.1.1 → pix_cli-0.1.2}/src/pix/logger.py

@@ -9,18 +9,37 @@ class Logger:
     def __init__(self, verbose: bool = False, quiet: bool = False) -> None:
         self.verbose = verbose
         self.quiet = quiet
+        self._progress_active = False
+
+    def _end_progress(self) -> None:
+        if self._progress_active:
+            print(file=sys.stderr)
+            self._progress_active = False
 
     def info(self, message: str) -> None:
         if not self.quiet:
+            self._end_progress()
             print(message, file=sys.stderr)
 
     def detail(self, message: str) -> None:
         if self.verbose and not self.quiet:
+            self._end_progress()
             print(message, file=sys.stderr)
 
     def warning(self, message: str) -> None:
         if not self.quiet:
+            self._end_progress()
             print(f"pix: warning: {message}", file=sys.stderr)
 
     def error(self, message: str) -> None:
+        self._end_progress()
         print(f"pix: error: {message}", file=sys.stderr)
+
+    def progress(self, message: str) -> None:
+        if self.quiet:
+            return
+        if sys.stderr.isatty():
+            print(f"\r{message}\033[K", end="", file=sys.stderr, flush=True)
+            self._progress_active = True
+        else:
+            print(message, file=sys.stderr)

{pix_cli-0.1.1 → pix_cli-0.1.2}/src/pix/shrink.py

@@ -258,9 +258,10 @@ def shrink(args: argparse.Namespace, settings: dict[str, Any], log: Logger) -> i
     status = 0
     pyvips = None
     encoded: set[str] = set()
+    total = len(sources)
 
     try:
-        for src in sources:
+        for index, src in enumerate(sources, 1):
             rel = src.relative_to(source).as_posix()
             try:
                 src_stat = src.stat()
@@ -287,6 +288,8 @@ def shrink(args: argparse.Namespace, settings: dict[str, Any], log: Logger) -> i
                 counters["converted"] += 1
                 continue
 
+            log.progress(f"processing {index}/{total}: {rel}")
+
             if pyvips is None:
                 try:
                     import pyvips as pyvips_module

{pix_cli-0.1.1 → pix_cli-0.1.2}/tests/test_cli.py

@@ -1,3 +1,4 @@
+import io
 import json
 import os
 import tomllib
@@ -7,6 +8,7 @@ import pytest
 
 from pix import cli, doctor, rclone
 from pix import config as pix_config
+from pix.logger import Logger
 
 
 def test_parse_size():
@@ -20,6 +22,22 @@ def test_parse_size():
             pix_config.parse_size(bad)
 
 
+def test_logger_progress_rewrites_on_tty(monkeypatch):
+    class FakeStderr(io.StringIO):
+        def isatty(self):
+            return True
+
+    stream = FakeStderr()
+    monkeypatch.setattr("pix.logger.sys.stderr", stream)
+
+    log = Logger()
+    log.progress("first")
+    log.progress("second")
+    log.info("done")
+
+    assert stream.getvalue() == "\rfirst\033[K\rsecond\033[K\ndone\n"
+
+
 def test_version_exits_zero(capsys):
     assert cli.main(["--version"]) == 0
     assert capsys.readouterr().out.strip() == f"pix {cli.__version__}"
@@ -38,7 +56,7 @@ def test_config_priority_and_defaults(tmp_path: Path):
 
     assert settings["shrink"]["width"] == 1000
     assert settings["shrink"]["quality"] == 70
-    assert settings["shrink"]["effort"] == pix_config.DEFAULTS["shrink"]["effort"]
+    assert settings["shrink"]["effort"] == pix_config.load_defaults()["shrink"]["effort"]
     assert settings["publish"]["max_delete"] == 10
 
 
@@ -74,7 +92,7 @@ def test_config_init_writes_defaults(tmp_path: Path, capsys):
     assert cli.main(["--config", str(config), "config", "init"]) == 0
 
     assert capsys.readouterr().out.strip() == f"wrote {config}"
-    assert tomllib.loads(config.read_text(encoding="utf-8")) == pix_config.DEFAULTS
+    assert tomllib.loads(config.read_text(encoding="utf-8")) == pix_config.load_defaults()
     assert pix_config.load_settings(config)["shrink"]["width"] == 1920
 
 
@@ -94,7 +112,7 @@ def test_config_init_force_overwrites(tmp_path: Path, capsys):
 
     assert cli.main(["--config", str(config), "config", "init", "--force"]) == 0
 
-    assert tomllib.loads(config.read_text(encoding="utf-8")) == pix_config.DEFAULTS
+    assert tomllib.loads(config.read_text(encoding="utf-8")) == pix_config.load_defaults()
     assert capsys.readouterr().out.strip() == f"wrote {config}"
 
 
@@ -142,6 +160,16 @@ def test_shrink_encodes_and_writes_state(tmp_path: Path):
     assert entry["out_size"] == output.stat().st_size
 
 
+def test_shrink_reports_progress(tmp_path: Path, capsys):
+    source = tmp_path / "src"
+    dest = tmp_path / "out"
+    make_png(source / "a.png")
+
+    assert run_shrink(source, dest, "-w", "100") == 0
+
+    assert "processing 1/1: a.png" in capsys.readouterr().err
+
+
 def test_shrink_second_run_skips_unchanged(tmp_path: Path, capsys):
     source = tmp_path / "src"
     dest = tmp_path / "out"

{pix_cli-0.1.1 → pix_cli-0.1.2}/uv.lock

@@ -129,7 +129,7 @@ wheels = [
 
 [[package]]
 name = "pix-cli"
-version = "0.1.0"
+version = "0.1.1"
 source = { editable = "." }
 dependencies = [
     { name = "pyvips", extra = ["binary"] },