Skip to content

Copilot/周泓宇#30

Open
IMHEREJOE wants to merge 3 commits into
lights-think:masterfrom
IMHEREJOE:ai-copilot-gpt5-mini-周泓宇

Hidden character warning

The head ref may contain hidden characters: "ai-copilot-gpt5-mini-\u5468\u6cd3\u5b87"
Open

Copilot/周泓宇#30
IMHEREJOE wants to merge 3 commits into
lights-think:masterfrom
IMHEREJOE:ai-copilot-gpt5-mini-周泓宇

Conversation

@IMHEREJOE
Copy link
Copy Markdown

完成能力点:

  • 实现 Planner 从 prompt 提取 SKU 和业务意图,生成确定性工具计划。
  • 实现 Executor 逐步执行 erp.get_inventorybi.get_salesknowledge.searchsupplier.get_riskoa.create_approval_draft,包含参数渲染、错误处理、跳过 OA 草稿、结果汇总。
  • 完成 RAG 权限感知检索,返回 answercitationsfiltered_doc_ids,避免受限文档正文泄露。
  • 在任务创建中接入提示词注入检测,并记录拒绝审计。
  • 增强权限与可见性控制:运行创建前校验工具权限,GET /api/runs/{run_id}/api/runs/{run_id}/events 仅允许请求者、任务创建者或管理员访问。
  • 补齐审计日志与管理后台指标,新增 approval.draft.createrun.events.readtask.rejected,并补充平均耗时、最近失败、队列健康度指标。

核心设计思路与重要取舍:

  • 维护 README/公开契约字段不变,新增字段仅在合规条件下出现,避免修改公开 API 名称。
  • Planner 采用正则抽取 SKU,避免写死样例;OA 草稿受 oa:approval:write 权限和“分析类”语义双重控制。
  • 运行事件按标准工具轨迹记录,敏感字段在工具输出层面脱敏,避免 vendor_secret/unit_cost_usd 泄露。
  • token 费用采用简单完成步骤计数作为实现占位,确保闭环同时保留后续精细计费改进空间。
  • 未改数据库 schema,仅向现有 runsrun_eventsaudit_logs 写入兼容数据。

本地执行过的命令和结果:

  • e:\youliu\Agent-test\.venv\Scripts\python.exe scripts/self_check.py
    • 结果:Exit 0
    • 说明:公共自检脚本运行成功,确认公开 API 基本契约和环境可用。
  • e:\youliu\Agent-test\.venv\Scripts\python.exe -m pytest -q
    • 结果:4 passed, 1 xfailed, 5 xpassed
    • 说明:全量本地测试通过。唯一 xfail 为 tests/test_acceptance_guidance.py::test_acceptance_alice_inventory_replenishment_loop,它是 starter repo 的验收导向标记,其他验收导向测试意外通过。
  • e:\youliu\Agent-test\.venv\Scripts\python.exe -m pytest -q tests/test_public_contract.py
    • 结果:通过
    • 说明:验证公开契约接口结构和权限行为。
  • e:\youliu\Agent-test\.venv\Scripts\python.exe -m pytest -q tests/test_smoke.py
    • 结果:通过
    • 说明:验证健康检查与种子用户创建任务路径。
  • git status --porcelain
    • 结果:仅包含预期已提交文件,说明工作树干净且改动聚焦。

已知未完成项或风险:

  • token 成本计费为简单步骤计数,不是精确 token 计费。
  • Planner 的 SKU/意图解析基于正则,复杂自然语言场景鲁棒性有限。
  • 提示词注入检测为启发式正则规则,未覆盖所有可能变体。
  • tests/test_acceptance_guidance.py::test_acceptance_alice_inventory_replenishment_loop 仍为 xfail,符合 starter repo 验收导向标记。

扩展兼容影响说明:

  • 未修改公开 API 字段名称,RunOutknowledge.searchdashboard contract 保持稳定。
  • 未改数据库 schema,仅使用现有列写入额外事件和审计数据。
  • 新增审计动作名与事件类型遵循 README 语义,不影响现有读取逻辑。
  • COLLABORATION_LOG.md 记录了对历史备注的逐条判断,说明拒绝了写死 SKU、静默吞错、字段重命名等不兼容方案。

AI 软件、模型与人工协作者:

  • 使用 AI 软件:GitHub Copilot,模型辅助为类 GPT-5 mini 风格实现。
  • 角色分工:开发者主导整体设计、功能实现、兼容判断、测试执行、bug 修复和日志整理;AI 辅助生成初始代码结构、补全 TODO 和优化实现思路。

COLLABORATION_LOG.md 关键结论摘要:

  • 记录了任务目标、非目标、假设与歧义处理、历史备注采纳/拒绝。
  • 说明了从占位实现到完成 Planner/Executor/RAG/权限审计的根因分析与修复路径。
  • 记录了验证命令、结果和剩余风险,保证过程证据完整。

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