fix: stop S7632 flagging NOSONAR mentioned in prose comments

SonarCloud's python:S7632 parses the literal ``# NOSONAR`` token wherever it
appears — including inside explanatory comments that *quote* the directive —
and treats the following text as a malformed suppression. The actual bare
``# NOSONAR`` suppression lines are fine; the flagged lines were the prose
comments describing them. Reword those comments to drop the inner ``#`` so the
analyzer no longer sees a directive.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Chris Coutinho
2026-05-31 21:38:14 +02:00
co-authored by Claude Opus 4.8
parent 1528a1248d
commit 874c7c109b
3 changed files with 6 additions and 6 deletions
@@ -122,8 +122,8 @@ class GatewayProvider(OpenAIProvider):
# AsyncOpenAI rejects an empty key; use a non-secret placeholder when
# the gateway is unauthenticated. When a token provider is configured,
# the real Bearer is set on the client before each request. The bare
# ``# NOSONAR`` silences the hard-coded-credential hotspot — this is a
# public placeholder string, not a secret.
# suppression marker below silences the hard-coded-credential hotspot —
# this is a public placeholder string, not a secret.
super().__init__(
api_key=_UNAUTHENTICATED_PLACEHOLDER, # NOSONAR
base_url=base_url,
+3 -3
View File
@@ -156,9 +156,9 @@ class NatsTaskProducer:
) -> None:
return None
# The bare ``# NOSONAR`` silences ``python:S7503`` (async method without
# await): ``async def`` is required by the TaskProducer protocol, but this
# handle close is a genuine no-op.
# The bare suppression marker silences S7503 (async method without await):
# ``async def`` is required by the TaskProducer protocol, but this handle
# close is a genuine no-op.
async def aclose(self) -> None: # NOSONAR
# Per-handle close (e.g. a per-user scanner clone exiting). The bus
# connection is shared and owned by the lifespan, so this is a no-op;
@@ -45,7 +45,7 @@ class PostgresTaskProducer:
) -> None: # pragma: no cover
return None
# Bare ``# NOSONAR`` silences ``python:S7503`` (async method without await):
# The bare suppression marker silences S7503 (async method without await):
# ``async def`` is required by the TaskProducer protocol; this stub is a
# no-op until the Postgres transport lands.
async def aclose(self) -> None: # NOSONAR # pragma: no cover