Skip to content

fix: 修复多个模块 bug (denial, long_term, MCP, lazy_ignore, api_server)#49

Merged
afine907 merged 1 commit into
masterfrom
fix/bugs-and-lint
May 30, 2026
Merged

fix: 修复多个模块 bug (denial, long_term, MCP, lazy_ignore, api_server)#49
afine907 merged 1 commit into
masterfrom
fix/bugs-and-lint

Conversation

@afine907
Copy link
Copy Markdown
Owner

Summary

修复 5 个模块中的 bug,提升代码健壮性。

Changes

denial.py

  • 修复 line 139-140 缩进错误,_generate_denial_reason 方法体缩进不一致

long_term.py

  • 移除未使用的 Path 导入

mcp/client.py

  • _discover_tools 支持嵌套 result 结构 (result.result.tools)
  • MCP 协议响应可能将工具列表包装在 result 字段中

lazy_ignore.py

  • 重写 should_ignore 方法,正确处理否定模式 (!pattern)
  • 分离普通模式和否定模式,确保否定模式优先级高于普通模式
  • 例如: *.pyc + !important.pycimportant.pyc 不被忽略

api_server.py

  • 使用 app.router.add_route(method, path, handler) 替代 method_map
  • 修复 UrlDispatcher 没有 post/put/delete/patch 属性的问题

Test Plan

  • uv run ruff check src/ - 0 errors
  • 所有修改的模块原有测试仍然通过

- 修复 denial.py 缩进错误 (line 139-140)
- 修复 long_term.py 未使用的 Path 导入
- 修复 mcp/client.py 工具发现: 支持嵌套 result 结构
- 修复 lazy_ignore.py: 支持否定模式 (!pattern) 优先级
- 修复 api_server.py: 使用 add_route 替代 method_map
@afine907 afine907 merged commit 6c7acab into master May 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant