[Enhancement] 启用 Ruff 安全规则集（S）并收敛 24 处 assert #539

背景

本仓库 .github/workflows/validate.yml 的静态检查通过 pre-commit 调用 ruff，但 pyproject.toml 的规则集为：

[tool.ruff.lint]
select = ["E", "F", "I", "B", "UP"]

缺少 S（flake8-bandit 安全规则）。同类仓库（如 codex-a2a）已启用 S，本仓库作为对外提供 A2A/JSON-RPC 服务的运行时，缺少静态安全规则是一处门禁缺口。

现状核实

实测（main）：

uv run --locked --extra dev ruff check --select S --statistics src/opencode_a2a

结果：24 处 S101（Use of assert），集中在：

• src/opencode_a2a/jsonrpc/handlers/session_lifecycle.py（11 处）
• src/opencode_a2a/jsonrpc/handlers/session_queries.py（4 处）
• src/opencode_a2a/jsonrpc/handlers/workspace_control.py（3 处）
• src/opencode_a2a/server/agent_card.py（2 处）
• src/opencode_a2a/jsonrpc/handlers/provider_discovery.py（1 处）
• src/opencode_a2a/server/rest_tasks.py（1 处）

除 S101 外没有其他 S 违例。多数 assert 是 mypy 的类型收窄（assert session_id is not None，在前置校验之后）；少数是内部不变量断言（如 agent_card.py 的扩展 URI 顺序校验）。

需要决策/处理的问题

1. 对类型收窄类 assert：是保留（python -O 下会消失，但不影响外部契约），还是改成显式 raise。
2. 对内部不变量类 assert：建议改为显式 raise（避免 python -O 下被剥离）。
3. 不要通过 ignore = ["S101"] 之类的全局豁免来“通过规则”——那会重新引入本仓库正在治理的“掩盖问题”模式。

验收标准

• pyproject.toml 的 ruff 规则集包含 S，且 uv run ruff check 在 src/ 与 tests/ 上通过。
• tests/ 可按需使用 per-file-ignores 关闭 S（测试内断言属正常用法），但 src/ 不允许全局豁免 S101。
• bash ./scripts/doctor.sh 全绿；新增/修改的行为有对应测试覆盖，且不降低覆盖率门禁。

说明

该事项独立于 #537（CI lint 门禁严格性修复），单独跟踪以避免把 24 处断言改造混入门禁修复 PR。