Skip to content

fix: AGENTS.md 12-role rules audit — 3 real fixes across 8 files#107

Merged
TelivANT merged 1 commit intomainfrom
fix/agents-md-audit
Feb 25, 2026
Merged

fix: AGENTS.md 12-role rules audit — 3 real fixes across 8 files#107
TelivANT merged 1 commit intomainfrom
fix/agents-md-audit

Conversation

@TelivANT
Copy link
Copy Markdown
Owner

以 AGENTS.md 中 12 角色的红线/黄线/绿线规则对全部 9 crate 进行扫描。

修复项(3项)

  1. Rust 红线: rate_limit.rs 中 3 个 .unwrap()HeaderValue::from_static()
  2. 日志统一: metrics/lib.rseprintln!tracing::error!(添加 tracing 依赖)
  3. 错误处理: 5 处 let _ = std::fs::create_dir_all(...)if let Err(e) + warn 日志

已知限制(5项,不修)

  • metrics lazy_static .expect() — Prometheus 标准模式
  • FileAuditBackend/FileGdprBackend 用 std::fs — trait 方法是 sync 的
  • RateLimiter 用 tokio::sync::Mutex — 写多读少场景合理
  • API key SHA-256 无 salt — 高熵随机字符串不需要
  • ADMIN_TOKEN 未设置可启动 — 开发环境需要,已有 warn

@TelivANT TelivANT merged commit 86a7557 into main Feb 25, 2026
3 checks passed
@TelivANT TelivANT deleted the fix/agents-md-audit branch February 25, 2026 15:44
TelivANT added a commit that referenced this pull request Feb 25, 2026
- COMPARISON.md: 4 hot-reload claims corrected (table, architecture, advantages, summary)
- ROADMAP.md: comparison table + priority list hot-reload corrected
- DESIGN.md: hot-reload section marked as "design intent (not fully implemented)"
- RELEASE_CHECKLIST.md: PR #107/#108/#109 merge records + Round 11 audit
TelivANT added a commit that referenced this pull request Feb 26, 2026
- COMPARISON.md: 4 hot-reload claims corrected (table, architecture, advantages, summary)
- ROADMAP.md: comparison table + priority list hot-reload corrected
- DESIGN.md: hot-reload section marked as "design intent (not fully implemented)"
- RELEASE_CHECKLIST.md: PR #107/#108/#109 merge records + Round 11 audit
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