From 257a53f7f9f06dd93953aea27fe2f36ad9630020 Mon Sep 17 00:00:00 2001 From: jinsongwang Date: Sat, 30 May 2026 13:42:55 +0800 Subject: [PATCH] =?UTF-8?q?fix:=20import=20=E7=AE=A1=E7=90=86=E8=A7=84?= =?UTF-8?q?=E8=8C=83=E5=8C=96=20=E2=80=94=20E402=20=E6=B8=85=E9=9B=B6=20+?= =?UTF-8?q?=20CI/pyproject=20=E6=96=87=E6=A1=A3=E5=8C=96=20(Issue=20#31)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - pyproject.toml: F401 (unused-import) 显式标注 - CI: ruff lint 步骤添加 F401 注释 - api/cli/pipeline/role_skill_map: logger 定义移至所有 import 之后，消除 E402 - ruff E402 (模块级导入顺序): 0 errors - ruff F401 (unused-import): 0 errors Co-Authored-By: Claude Opus 4.8 --- .github/workflows/test.yml | 2 +- agent_go/api.py | 6 +++--- agent_go/cli.py | 3 ++- agent_go/pipeline.py | 3 ++- agent_go/role_skill_map.py | 3 ++- pyproject.toml | 3 ++- 6 files changed, 12 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index c6fb261..61befab 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -25,7 +25,7 @@ jobs: run: | pytest tests/ -q - - name: Ruff lint + - name: Ruff lint (includes F401 unused-import) run: | ruff check agent_go/ --select=E,F,W --ignore=E501 diff --git a/agent_go/api.py b/agent_go/api.py index 8d5735f..0ad6b27 100644 --- a/agent_go/api.py +++ b/agent_go/api.py @@ -1,15 +1,15 @@ +import hashlib import json, re, time, logging from pathlib import Path from datetime import datetime from typing import Any, Optional -logger = logging.getLogger(__name__) - from .config import get_api_key, log_event, DECOMPOSE_RULES, AGENT_GO_DIR from .git_utils import analyze_project, get_git_info, get_resource_map from .skills import list_skills from .role_skill_map import load_role_skill_map -import hashlib + +logger = logging.getLogger(__name__) __all__ = [ "call_api", "generate_plan", "decompose_fallback", diff --git a/agent_go/cli.py b/agent_go/cli.py index 7d6f9a0..eec04ed 100644 --- a/agent_go/cli.py +++ b/agent_go/cli.py @@ -3,7 +3,6 @@ from datetime import datetime from typing import Any, Optional -logger = logging.getLogger(__name__) from .config import load_config, safe_input, setup_logger, AGENT_GO_DIR from .api import generate_plan, decompose_fallback @@ -16,6 +15,8 @@ from .tui import cmd_status_tui from .workflow_gen import cmd_ci +logger = logging.getLogger(__name__) + __all__ = [ "main", "cmd_run", "cmd_resume", "cmd_list", "cmd_show", "cmd_status", "cmd_config", "cmd_clean", "cmd_pr", "cmd_review", diff --git a/agent_go/pipeline.py b/agent_go/pipeline.py index 0cc5659..56a7002 100644 --- a/agent_go/pipeline.py +++ b/agent_go/pipeline.py @@ -3,11 +3,12 @@ from pathlib import Path from typing import Any, Optional -logger = logging.getLogger(__name__) from .executor import run_subtask from .git_utils import _set_gc_auto, _worktree_remove, _worktree_prune +logger = logging.getLogger(__name__) + __all__: list[str] = [] def _run_pipeline(confirmed: list[dict[str, Any]], repo: Path, task_dir: Path, logger: logging.Logger, config: dict[str, Any], headless: bool, parallel: int, issue_ref: str, meta: dict[str, Any], diff --git a/agent_go/role_skill_map.py b/agent_go/role_skill_map.py index ddcab52..9e4f508 100644 --- a/agent_go/role_skill_map.py +++ b/agent_go/role_skill_map.py @@ -4,10 +4,11 @@ from fnmatch import fnmatch from typing import Any, Optional -logger = logging.getLogger(__name__) from .config import AGENT_GO_DIR +logger = logging.getLogger(__name__) + __all__ = ["load_role_skill_map", "apply_rules"] DEFAULT_MAP = { diff --git a/pyproject.toml b/pyproject.toml index 4ef2aa5..d80d135 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -3,8 +3,9 @@ target-version = "py39" line-length = 120 [tool.ruff.lint] +# E: pycodestyle F: Pyflakes (含 F401 unused-import) W: pycodestyle warnings select = ["E", "F", "W"] -ignore = ["E501"] +ignore = ["E501"] # 行长度由 formatter 管理 [tool.mypy] python_version = "3.9"