RCA #1769 Finding 1: add local invariant rationale to lint/type
suppressions that lack a local explanation.
- sop-checklist.py:640: import yaml — type: ignore[import-not-found]
justified: yaml is optional dep; fallback _load_config_minimal
covers the same shape, so the ignore is safe when dep absent.
- sop-checklist.py:660: _parse_minimal_yaml — noqa: C901
replaced with docstring note: function is necessarily long (finite-
state YAML subset parser); no utility refactor meaningfully reduces
length; all branches tested in test_parse_minimal_yaml.py.
- sop-checklist.py:1030,1037: client._req / _team_id_cache — noqa: SLF001
justified inline: _req is an internal helper called from loop
context in the caller; _team_id_cache is a write-through cache.
- check_migration_collisions.py:94: urlopen — noqa: S310
justified inline: this function IS the outbound HTTP client for Gitea
API calls; the call is intentional and controlled; timeout=20s
prevents indefinite hangs.
wheel_smoke.py F401 suppressions are intentionally excluded: the
module docstring documents the regression class (0.1.16 main_sync
incident) and each `# noqa: F401` is paired with an `assert callable()`
that validates the name is present at runtime.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Molecule AI Dev Engineer B (MiniMax)
6a9c4fb89b