ruff: re-enable SIM103 as non-blocking CI advisory

Technologicat · claude · Technologicat · commit e5d8779d3ef6 · 2026-04-12T02:07:45.000+03:00
Two-pass CI: hard errors first, then SIM103 informational.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -27,8 +27,9 @@ jobs:
         python -m pip install --upgrade pip
         pip install ruff
     - name: Lint with ruff
-      run: |
-        ruff check .
+      run: ruff check . --ignore SIM103
+    - name: Lint advisories (non-blocking)
+      run: ruff check . --select SIM103 || true
 
   test:
     needs: lint
diff --git a/pyproject.toml b/pyproject.toml
@@ -115,7 +115,8 @@ ignore = [
     "SIM118",  # in-dict-keys — explicit .keys() marks the variable as a dictlike
     "SIM300",  # yoda conditions — natural reading order preferred
     "SIM910",  # dict.get with None default — explicit None documents programmer intent
-    "SIM103",  # return condition directly — multi-guard patterns; autofix breaks visual consistency
+    # Note: SIM103 (return condition directly) is intentionally NOT ignored here.
+    # It is enabled as an advisory — CI runs it in a non-failing second pass.
 ]
 
 [tool.ruff.lint.per-file-ignores]

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,8 @@ ignore = [`
`115`	`115`	`"SIM118", # in-dict-keys — explicit .keys() marks the variable as a dictlike`
`116`	`116`	`"SIM300", # yoda conditions — natural reading order preferred`
`117`	`117`	`"SIM910", # dict.get with None default — explicit None documents programmer intent`
`118`		`- "SIM103", # return condition directly — multi-guard patterns; autofix breaks visual consistency`
	`118`	`+ # Note: SIM103 (return condition directly) is intentionally NOT ignored here.`
	`119`	`+ # It is enabled as an advisory — CI runs it in a non-failing second pass.`
`119`	`120`	`]`
`120`	`121`
`121`	`122`	`[tool.ruff.lint.per-file-ignores]`