chiguyong
|
31c65e01b8
|
fix(security): P0 安全加固 + 多实例部署一致性 (U1-U4 + U5c)
Deploy to Production / deploy (push) Has been cancelled
Details
U1: LLM gateway KB 缓存 fail-closed — 异常时默认禁用缓存防止 KB 数据泄漏
U2: MCP 危险工具黑名单过滤 — 6+1 端点覆盖，防止绕过 chat confirmation
U3: SecretsStore Redis 迁移 — 多 worker 共享凭证，内存降级保留开发模式
U4: channels webhook Redis 状态 — ZSET 滑动窗口限流 + nonce dedup + backpressure
U5c: ce-code-review 修复批次:
- P0: 统一 MCP 黑名单与 publisher.py 一致 (terminal_execute -> terminal, +file_read)
- P1: ZSET 限流 member 加 uuid 后缀避免同时间戳碰撞
- P1: SecretsStore redis 参数 Any -> aioredis.Redis | None (AGENTS.md 合规)
- P1: Redis client 添加 socket_timeout 防止单点故障请求挂死
测试: 171 scoped tests pass, ruff clean
|
2026-06-26 04:05:33 +08:00 |
|
chiguyong
|
53faa60472
|
fix(review): ce-code-review P1+P2 修复 — 安全/可靠性/性能
P1 安全与可靠性（4 项）：
- wecom: verify_signature 增加时间戳新鲜度校验（5 分钟窗口防重放）
- cache: should_cache 在 per_user_namespace 开启时拒绝 user_id=None
匿名请求，避免跨用户缓存泄漏（安全要求 a/e）
- channels: webhook receive_message 异常兜底，防止 500 触发平台重试风暴
- app: shutdown 调用 close_all_adapters + await _pending_webhook_tasks，
防止 httpx 连接泄漏和丢失 IM 回复
P2 效率与可维护性（5 项）：
- feishu: _TOKEN_CACHE_TTL 300 → 6900（2h 减 5min 余量，避免 24x 过频刷新）
- channels: _pending_webhook_tasks 有界化（2x 并发上限时 429 拒绝）
- gateway: quota 检查每 period 单次 get_usage，复用 summary 检查 token+cost
- cache_key: generate_cache_key 合并为单次 SHA-256（消除 8-10 次冗余哈希）
- config: ProviderConfig.get_api_key 移除未用的 secrets_store 参数
P3 去重（1 项）：
- channels: _process_inbound_message DIRECT_CHAT 路径提取 _direct_chat 辅助函数
测试：
- test_wecom: 时间戳改用 int(time.time())，新增 test_expired_timestamp_rejected
- test_cache: should_cache 测试覆盖匿名拒绝 + namespace_off 兼容
- test_config_migration: get_api_key 测试适配新签名
- channels/config_migration/quota_enforcement 测试全部通过
|
2026-06-26 01:40:31 +08:00 |
|
chiguyong
|
1ccaf56b9a
|
refactor: ce-simplify-code 审查修复 — 去重 + 效率 + 死代码清理
3 个审查代理（复用/质量/效率）发现 15 个问题，全部修复：
效率与安全（6 项）：
- MCPClient 缓存 MultiServerMCPClient 单例 + aclose()，修复连接/子进程泄漏
- _rate_limits 清理空 IP 条目，修复 X-Forwarded-For 欺骗下内存泄漏
- _seen_nonces 改用 OrderedDict，O(1) 摊销过期清理
- webhook 后台任务加 Semaphore(20) + 任务引用追踪，限制无界并发
- _build_adapter 用 asyncio.gather 并行解密 secrets
- 适配器实例缓存（_adapter_cache），token TTL 缓存跨请求命中
去重（4 项）：
- header_get 提取到 channels/base.py，4 个适配器统一 import
- _get_client/close() 移入 MessageAdapter 基类，子类继承
- URLVerificationChallenge 统一到 base.py，feishu/slack/wecom 共用
- Transport ABC 添加 endpoint_url 属性，from_transport 不再访问私有字段
死代码与类型安全（5 项）：
- detect_cache_hit 死方法替换为 record_cache_result 公开 API
- execution_mode.value == "direct_chat" 改用枚举比较
- 删除 yielded_any 死变量、重复 from fastapi import Request、
多余 getattr 防御
453 tests passed, ruff clean（预存 F841 非本次引入）
|
2026-06-25 23:54:14 +08:00 |
|
chiguyong
|
793476cafa
|
feat(llm): U17 — LiteLLM 语义缓存替换 + per-user/ACL scope 安全隔离
- 新增 LitellmCacheManager：配置 litellm.cache 全局，三级后端 fallback
(RedisSemanticCache -> RedisCache -> InMemoryCache)，redisvl lazy import
- cache_key 扩展 user_id + kb_acl_hash 参数（安全要求 a/b/e）
- gateway 集成：读取 KB caching_disabled flag（安全要求 c），构建带 scope
的 cache_key，命中时 cost=0
- LLMResponse 新增 cache_hit 字段；LLMRequest 新增 cache 参数
- litellm_provider 透传 cache 参数 + 检测 _hidden_params 缓存命中
- 33 个新测试覆盖 13 场景（含 User A != User B 缓存隔离）
- 旧 InMemoryLLMCache/RedisLLMCache 保留向后兼容
|
2026-06-25 22:49:59 +08:00 |
|
chiguyong
|
069dbc22b1
|
feat(llm): U15 — LiteLLM unified provider + api_key encrypted secrets migration
|
2026-06-25 21:41:15 +08:00 |
|
chiguyong
|
00c8386939
|
fix(review): U1 Redis quota enforcement — key construction + fail-closed + degradation recovery + async
|
2026-06-22 16:22:33 +08:00 |
|
chiguyong
|
09feca3307
|
feat(admin): U7 — usage dashboard + quota enforcement
UsageRecord extended with user_id + department_id (backward compatible).
UsageStore Protocol extended: record() accepts user_id/department_id,
get_usage() accepts filters, new get_usage_by_user/department methods.
RedisUsageStore uses versioned keys (v2) for new records.
LLMGateway.chat()/chat_stream() accept user_id, department_ids, db_path.
Quota check before provider call: model whitelist + token limit + cost
limit (daily). Multi-department uses strictest-wins (any exceed → reject).
QuotaExceededError → 429 at route layer.
UsageService: summary, timeseries, by-model, top-users, export (CSV/JSON).
5 new admin endpoints under /admin/usage/*.
llm_gateway.py routes pass DepartmentContext + db_path to gateway,
catch QuotaExceededError → 429 (JSON for /chat, SSE error for /stream).
84 new tests. 441 admin+usage tests pass, no regressions.
|
2026-06-21 17:23:20 +08:00 |
|
chiguyong
|
6e0e081f23
|
feat: gap closure sprint — dark theme, @-mention, LocalComputerUse, tests
P0: U4 UsageStore + U5 CascadeStateStore independent test files (57 tests)
P1: Dark theme — tokens.css [data-theme="dark"] + theme.ts Pinia store
+ TopNav toggle button + App.vue dynamic Ant Design theme
P1: @-mention — MentionDropdown.vue + /skills/mention-suggest API
+ ChatInput integration with @ detection
P2: LocalComputerUseSession — pyautogui + screencapture (replaces Docker stub)
P2: Integration tests for gap closure (12 tests)
Fix: create_cascade_state_store() now passes session_ttl to InMemory fallback
|
2026-06-14 16:16:50 +08:00 |