[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

followup-reply.md

@@ -1,3 +1,10 @@
-Thanks @hugovk — fair point on the health score. I've reverted the ternaries and fixed the 3.10 timezone issue in the earlier push; this update adds a broader auto-fix pass (format/refactor/ruff aligned with your ruff config) plus a few targeted refactors: suffix lookup in `naturalsize`, `_today_for_value()` to dedupe the `naturalday`/`naturaldate` today logic, and the other safe changes listed in **Changes brought**.
+Thanks @hugovk — fair point on the health score. I've reverted the ternaries and fixed
+the 3.10 timezone issue in the earlier push; this update adds a broader auto-fix pass
+(format/refactor/ruff aligned with your ruff config) plus a few targeted refactors:
+suffix lookup in `naturalsize`, `_today_for_value()` to dedupe the
+`naturalday`/`naturaldate` today logic, and the other safe changes listed in **Changes
+brought**.
 
-CI was green on my side after the revert push (`pre-commit.ci` + RTD). I can't add `changelog:*` labels from the fork — could you add one when you have a moment? Happy to adjust anything that's still not what you want.
+CI was green on my side after the revert push (`pre-commit.ci` + RTD). I can't add
+`changelog:*` labels from the fork — could you add one when you have a moment? Happy to
+adjust anything that's still not what you want.

pr-body.md

@@ -2,7 +2,14 @@
 
 **Repo health:** 68/100
 
-Our analysis found **90 format issues** and about **49 refactoring opportunities** across the reviewed scope. Security scans (gitleaks, bandit, semgrep) were clean; pip-audit reported no known vulnerabilities. Python code duplication is low (~0.7% in `.py` files). Radon maintainability ranks are mostly A-grade, but several functions exceed cyclomatic-complexity thresholds — notably `naturaldelta` and `precisedelta` in `time.py`. This pull request applies an **auto-fix pass** aligned with your project config — **5 files**, ~46 focused line changes — plus a few targeted refactors; it is not a complete cleanup of every finding.
+Our analysis found **90 format issues** and about **49 refactoring opportunities**
+across the reviewed scope. Security scans (gitleaks, bandit, semgrep) were clean;
+pip-audit reported no known vulnerabilities. Python code duplication is low (~0.7% in
+`.py` files). Radon maintainability ranks are mostly A-grade, but several functions
+exceed cyclomatic-complexity thresholds — notably `naturaldelta` and `precisedelta` in
+`time.py`. This pull request applies an **auto-fix pass** aligned with your project
+config — **5 files**, ~46 focused line changes — plus a few targeted refactors; it is
+not a complete cleanup of every finding.
 
 ## What you are doing right
 
@@ -22,37 +29,43 @@ Our analysis found **90 format issues** and about **49 refactoring opportunities
 
 **Duplicate code (follow-up)**
 
-- ~0.7% duplication in Python — primary clone: `time.py` today-calculation for `naturalday` / `naturaldate` — **partially addressed** by extracting `_today_for_value`.
+- ~0.7% duplication in Python — primary clone: `time.py` today-calculation for
+  `naturalday` / `naturaldate` — **partially addressed** by extracting
+  `_today_for_value`.
 
 **Refactoring (~49 opportunities; sample in this PR)**
 
 - Collapse suffix selection in `naturalsize` (`filesize.py`) — **included in this PR**.
 - Flatten `natural_list` branching (`lists.py`) — **included in this PR**.
 - Remove redundant `int()` cast in `metric` (`number.py`) — **included in this PR**.
-- Yoda comparison fix and `zip(..., strict=True)` in `time.py` — **included in this PR**.
+- Yoda comparison fix and `zip(..., strict=True)` in `time.py` — **included in this
+  PR**.
 - Deeper `naturaldelta` branch simplification — follow-up (larger change).
 
 **Lint / style (defer)**
 
-- Many findings reflect intentional API choices (boolean positional args, `format` parameter name, pytest parametrize style).
+- Many findings reflect intentional API choices (boolean positional args, `format`
+  parameter name, pytest parametrize style).
 
 ## Changes brought
 
-| Pass | Files | Fixes / notes | Manual? |
-| --- | --- | --- | --- |
-| `shipgate format --target .` | 0 | Codebase already formatted; skipped pyproject rule renames | no |
-| `shipgate refactor fix` | 0 | No remaining auto-fixable refactors in `src/humanize/` | no |
-| `ruff check --fix` (pyproject `[tool.ruff]`) | 1 | RUF046 redundant `int()` in `metric()` | no |
-| Targeted safe refactors | 4 | `lists` elif flatten; `filesize` suffix lookup; `_today_for_value` dedup; `zip(..., strict=True)` | yes |
-| Revert ternaries (review) | 2 | Restored `if`/`else` per maintainer feedback | revert only |
-| **Net in PR** | **5** | | **0** new manual |
+| Pass                                         | Files | Fixes / notes                                                                                     | Manual?          |
+| -------------------------------------------- | ----- | ------------------------------------------------------------------------------------------------- | ---------------- |
+| `shipgate format --target .`                 | 0     | Codebase already formatted; skipped pyproject rule renames                                        | no               |
+| `shipgate refactor fix`                      | 0     | No remaining auto-fixable refactors in `src/humanize/`                                            | no               |
+| `ruff check --fix` (pyproject `[tool.ruff]`) | 1     | RUF046 redundant `int()` in `metric()`                                                            | no               |
+| Targeted safe refactors                      | 4     | `lists` elif flatten; `filesize` suffix lookup; `_today_for_value` dedup; `zip(..., strict=True)` | yes              |
+| Revert ternaries (review)                    | 2     | Restored `if`/`else` per maintainer feedback                                                      | revert only      |
+| **Net in PR**                                | **5** |                                                                                                   | **0** new manual |
 
 ## Changes
 
 - `src/humanize/lists.py`: flatten `natural_list` branching; use `!s` conversion.
 - `src/humanize/filesize.py`: collapse suffix selection into a single lookup.
-- `src/humanize/number.py`: remove redundant `int()` around `max()` and `math.floor()` in `metric()`.
-- `src/humanize/time.py`: extract `_today_for_value()` for `naturalday` / `naturaldate`; yoda comparison fix; `zip(..., strict=True)` in `precisedelta`.
+- `src/humanize/number.py`: remove redundant `int()` around `max()` and `math.floor()`
+  in `metric()`.
+- `src/humanize/time.py`: extract `_today_for_value()` for `naturalday` / `naturaldate`;
+  yoda comparison fix; `zip(..., strict=True)` in `precisedelta`.
 - `tests/test_time.py`: `zip(..., strict=True)` in `_date_and_delta` test.
 
 ## Verification
@@ -66,14 +79,18 @@ Our analysis found **90 format issues** and about **49 refactoring opportunities
 <details>
 <summary>Other findings (not in this example PR)</summary>
 
-| area | finding | note |
-| --- | --- | --- |
-| Complexity | `naturaldelta` CC 33 | Deferred — larger refactor |
-| Complexity | `precisedelta` CC 26 | Deferred |
-| Lint | Boolean positional args (FBT) | Intentional public API |
-| Lint | `format` parameter name (A002) | Stable API surface |
+| area       | finding                        | note                       |
+| ---------- | ------------------------------ | -------------------------- |
+| Complexity | `naturaldelta` CC 33           | Deferred — larger refactor |
+| Complexity | `precisedelta` CC 26           | Deferred                   |
+| Lint       | Boolean positional args (FBT)  | Intentional public API     |
+| Lint       | `format` parameter name (A002) | Stable API surface         |
 
 </details>
 
 ---
-*We're testing [ShipGate](https://github.com/inquilabee/shipgate) on real-world projects to learn whether it works well in practice. This review summary was generated from ShipGate check output and AI-assisted analysis. Feedback on the findings or approach is welcome and appreciated — [docs](https://inquilabee.github.io/shipgate/).*
+
+_We're testing [ShipGate](https://github.com/inquilabee/shipgate) on real-world projects
+to learn whether it works well in practice. This review summary was generated from
+ShipGate check output and AI-assisted analysis. Feedback on the findings or approach is
+welcome and appreciated — [docs](https://inquilabee.github.io/shipgate/)._

shipgate-review/FINDINGS.md

@@ -2,11 +2,11 @@
 
 Captured 2026-07-30 during follow-up auto-expand pass.
 
-| Artifact | Description |
-| --- | --- |
-| `check.out` | `shipgate check --suite full --target .` |
-| `refactor-strict.out` | `shipgate refactor check --strict .` |
-| `format.out` | `shipgate format --target .` |
+| Artifact              | Description                              |
+| --------------------- | ---------------------------------------- |
+| `check.out`           | `shipgate check --suite full --target .` |
+| `refactor-strict.out` | `shipgate refactor check --strict .`     |
+| `format.out`          | `shipgate format --target .`             |
 
 ## Counts (from captures)
 

shipgate-review/SUMMARY.md

@@ -2,7 +2,14 @@
 
 **Repo health:** 68/100
 
-Our analysis found **90 format issues** and about **49 refactoring opportunities** across the reviewed scope. Security scans (gitleaks, bandit, semgrep) were clean; pip-audit reported no known vulnerabilities. Python code duplication is low (~0.7% in `.py` files). Radon maintainability ranks are mostly A-grade, but several functions exceed cyclomatic-complexity thresholds — notably `naturaldelta` and `precisedelta` in `time.py`. This pull request applies an **auto-fix pass** aligned with your project config — **5 files**, ~46 focused line changes — plus a few targeted refactors; it is not a complete cleanup of every finding.
+Our analysis found **90 format issues** and about **49 refactoring opportunities**
+across the reviewed scope. Security scans (gitleaks, bandit, semgrep) were clean;
+pip-audit reported no known vulnerabilities. Python code duplication is low (~0.7% in
+`.py` files). Radon maintainability ranks are mostly A-grade, but several functions
+exceed cyclomatic-complexity thresholds — notably `naturaldelta` and `precisedelta` in
+`time.py`. This pull request applies an **auto-fix pass** aligned with your project
+config — **5 files**, ~46 focused line changes — plus a few targeted refactors; it is
+not a complete cleanup of every finding.
 
 ## What you are doing right
 
@@ -22,37 +29,43 @@ Our analysis found **90 format issues** and about **49 refactoring opportunities
 
 **Duplicate code (follow-up)**
 
-- ~0.7% duplication in Python — primary clone: `time.py` today-calculation for `naturalday` / `naturaldate` — **partially addressed** by extracting `_today_for_value`.
+- ~0.7% duplication in Python — primary clone: `time.py` today-calculation for
+  `naturalday` / `naturaldate` — **partially addressed** by extracting
+  `_today_for_value`.
 
 **Refactoring (~49 opportunities; sample in this PR)**
 
 - Collapse suffix selection in `naturalsize` (`filesize.py`) — **included in this PR**.
 - Flatten `natural_list` branching (`lists.py`) — **included in this PR**.
 - Remove redundant `int()` cast in `metric` (`number.py`) — **included in this PR**.
-- Yoda comparison fix and `zip(..., strict=True)` in `time.py` — **included in this PR**.
+- Yoda comparison fix and `zip(..., strict=True)` in `time.py` — **included in this
+  PR**.
 - Deeper `naturaldelta` branch simplification — follow-up (larger change).
 
 **Lint / style (defer)**
 
-- Many findings reflect intentional API choices (boolean positional args, `format` parameter name, pytest parametrize style).
+- Many findings reflect intentional API choices (boolean positional args, `format`
+  parameter name, pytest parametrize style).
 
 ## Changes brought
 
-| Pass | Files | Fixes / notes | Manual? |
-| --- | --- | --- | --- |
-| `shipgate format --target .` | 0 | Codebase already formatted; skipped pyproject rule renames | no |
-| `shipgate refactor fix` | 0 | No remaining auto-fixable refactors in `src/humanize/` | no |
-| `ruff check --fix` (pyproject `[tool.ruff]`) | 1 | RUF046 redundant `int()` in `metric()` | no |
-| Targeted safe refactors | 4 | `lists` elif flatten; `filesize` suffix lookup; `_today_for_value` dedup; `zip(..., strict=True)` | yes |
-| Revert ternaries (review) | 2 | Restored `if`/`else` per maintainer feedback | revert only |
-| **Net in PR** | **5** | | **0** new manual |
+| Pass                                         | Files | Fixes / notes                                                                                     | Manual?          |
+| -------------------------------------------- | ----- | ------------------------------------------------------------------------------------------------- | ---------------- |
+| `shipgate format --target .`                 | 0     | Codebase already formatted; skipped pyproject rule renames                                        | no               |
+| `shipgate refactor fix`                      | 0     | No remaining auto-fixable refactors in `src/humanize/`                                            | no               |
+| `ruff check --fix` (pyproject `[tool.ruff]`) | 1     | RUF046 redundant `int()` in `metric()`                                                            | no               |
+| Targeted safe refactors                      | 4     | `lists` elif flatten; `filesize` suffix lookup; `_today_for_value` dedup; `zip(..., strict=True)` | yes              |
+| Revert ternaries (review)                    | 2     | Restored `if`/`else` per maintainer feedback                                                      | revert only      |
+| **Net in PR**                                | **5** |                                                                                                   | **0** new manual |
 
 ## Changes
 
 - `src/humanize/lists.py`: flatten `natural_list` branching; use `!s` conversion.
 - `src/humanize/filesize.py`: collapse suffix selection into a single lookup.
-- `src/humanize/number.py`: remove redundant `int()` around `max()` and `math.floor()` in `metric()`.
-- `src/humanize/time.py`: extract `_today_for_value()` for `naturalday` / `naturaldate`; yoda comparison fix; `zip(..., strict=True)` in `precisedelta`.
+- `src/humanize/number.py`: remove redundant `int()` around `max()` and `math.floor()`
+  in `metric()`.
+- `src/humanize/time.py`: extract `_today_for_value()` for `naturalday` / `naturaldate`;
+  yoda comparison fix; `zip(..., strict=True)` in `precisedelta`.
 - `tests/test_time.py`: `zip(..., strict=True)` in `_date_and_delta` test.
 
 ## Verification
@@ -66,14 +79,18 @@ Our analysis found **90 format issues** and about **49 refactoring opportunities
 <details>
 <summary>Other findings (not in this example PR)</summary>
 
-| area | finding | note |
-| --- | --- | --- |
-| Complexity | `naturaldelta` CC 33 | Deferred — larger refactor |
-| Complexity | `precisedelta` CC 26 | Deferred |
-| Lint | Boolean positional args (FBT) | Intentional public API |
-| Lint | `format` parameter name (A002) | Stable API surface |
+| area       | finding                        | note                       |
+| ---------- | ------------------------------ | -------------------------- |
+| Complexity | `naturaldelta` CC 33           | Deferred — larger refactor |
+| Complexity | `precisedelta` CC 26           | Deferred                   |
+| Lint       | Boolean positional args (FBT)  | Intentional public API     |
+| Lint       | `format` parameter name (A002) | Stable API surface         |
 
 </details>
 
 ---
-*We're testing [ShipGate](https://github.com/inquilabee/shipgate) on real-world projects to learn whether it works well in practice. This review summary was generated from ShipGate check output and AI-assisted analysis. Feedback on the findings or approach is welcome and appreciated — [docs](https://inquilabee.github.io/shipgate/).*
+
+_We're testing [ShipGate](https://github.com/inquilabee/shipgate) on real-world projects
+to learn whether it works well in practice. This review summary was generated from
+ShipGate check output and AI-assisted analysis. Feedback on the findings or approach is
+welcome and appreciated — [docs](https://inquilabee.github.io/shipgate/)._

shipgate-review/check.out

@@ -316,7 +316,7 @@ import-linter.check: error: TOOL_EXIT ╔══╗─────────▶
  ║║╔══╣╔╗║╔╗║╔╣║ ║║ ╔╬╣╔╗║║ ║│║╔═╝
 ╔╣╠╣║║║╚╝║╚╝║║║╚╗║╚═╝║║║║║╚╗║═╣║
 ╚══╩╩╩╣╔═╩══╩╝╚═╝╚═══╩╩╝╚╩═╩╩═╩╝
-  └──▶║║                    ▲ 
+  └──▶║║                    ▲
       ╚╝────────────────────┘
 
 Could not find package 'humanize' in your Python path.