[hermes] atlasbot: python:S2208 #163

Open
opened 2026-10-02 07:23:51 +00:00 by hermes-automation · 0 comments

SonarQube reports python:S2208 in atlasbot/engine/answerer/spine.py. The build is green; this is a standing finding, not a failure.

What is wrong

The wildcard import from ._base makes this module's dependencies implicit, but no names from _base are referenced in this file. Removing the unused import resolves S2208 without changing runtime behavior.

Ariadne opened no pull request for it: no automated patch was possible for this finding.

Facts

  • gitea — Import only needed names or import the module and then use its members. (CRITICAL, 5min estimated) (atlasbot/engine/answerer/spine.py:9)

Suggested fix (not applied)

Hermes could not open a pull request for this, so the change below was not written, validated, or pushed anywhere. It is a starting point for whoever picks this up, not a reviewed patch.

atlasbot/engine/answerer/spine.py — Remove the unused wildcard import. All names used by this module are already imported explicitly or defined locally, so this preserves its behavior while making dependencies explicit.

from __future__ import annotations

import re
from typing import Any

from atlasbot.engine.intent_router import IntentMatch
from atlasbot.snapshot.builder import summary_text

Filed automatically by Ariadne from a Hermes Agent diagnosis (run run_a8597670689f424bad1902ce22194da5). Hermes has no write access to this repository; no files or infrastructure were changed.

SonarQube reports **python:S2208** in `atlasbot/engine/answerer/spine.py`. The build is green; this is a standing finding, not a failure. ## What is wrong The wildcard import from ._base makes this module's dependencies implicit, but no names from _base are referenced in this file. Removing the unused import resolves S2208 without changing runtime behavior. Ariadne opened no pull request for it: no automated patch was possible for this finding. ## Facts - **gitea** — Import only needed names or import the module and then use its members. (CRITICAL, 5min estimated) (`atlasbot/engine/answerer/spine.py:9`) ## Suggested fix (not applied) Hermes could not open a pull request for this, so the change below was not written, validated, or pushed anywhere. It is a starting point for whoever picks this up, not a reviewed patch. **`atlasbot/engine/answerer/spine.py`** — Remove the unused wildcard import. All names used by this module are already imported explicitly or defined locally, so this preserves its behavior while making dependencies explicit. ``` from __future__ import annotations import re from typing import Any from atlasbot.engine.intent_router import IntentMatch from atlasbot.snapshot.builder import summary_text ``` ## Links - SonarQube finding: https://quality.bstein.dev/project/issues?resolved=false&id=atlasbot&open=AZ2y3T38Ky9i4pkIpSiV - Full evidence bundle and audit trail live in Ariadne at `/api/admin/audit/events`, event types `hermes_autotriage_incident` and `hermes_autotriage_diagnosis`. Filed automatically by Ariadne from a Hermes Agent diagnosis (run [run_a8597670689f424bad1902ce22194da5](https://triage.bstein.dev/chat?resume=run_a8597670689f424bad1902ce22194da5)). Hermes has no write access to this repository; no files or infrastructure were changed. <!-- hermes-triage job=atlasbot classification=python:S2208 incident=sonar/atlasbot/python:S2208/AZ2y3T38Ky9i4pkIpSiV -->
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bstein/atlasbot#163
No description provided.