[hermes] atlasbot: python:S2208 #93

Open
opened 2026-09-23 06:27:06 +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 imports an unspecified set of names, but this module does not reference any of them. Removing it resolves the rule without changing this file's 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 spine.py are defined locally or imported explicitly, so this avoids importing an uncontrolled namespace while preserving the module's behavior.

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_ccf72842059242f1be29bdb76e1f7521). 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 imports an unspecified set of names, but this module does not reference any of them. Removing it resolves the rule without changing this file's 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 spine.py are defined locally or imported explicitly, so this avoids importing an uncontrolled namespace while preserving the module's behavior. ``` 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_ccf72842059242f1be29bdb76e1f7521](https://triage.bstein.dev/chat?resume=run_ccf72842059242f1be29bdb76e1f7521)). 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#93
No description provided.