session隔离讨论

[spongehah]

1 同git仓库使用git worktree隔离 （deprecated）
2 不同目录clone仓库隔离 (当前)
3 虚拟化隔离

哪种更好？

[spongehah]

Status

• completed

Summary

• 已完成 issue 与仓库现状分析，并给出可执行的隔离策略规划。
• 核心判断：当前仓库已深度落地「不同目录 clone 仓库隔离（方案 2）」；短中期应继续以方案 2 为主，虚拟化隔离（方案 3）作为可选增强层，git worktree（方案 1）不建议回归为默认路径。

Verification

• 已阅读 issue 正文与现有评论（spongehah/Openagent-Github-Bridge#16）。
• 已检查仓库关键实现与约束：AGENTS.md、README.md、internal/agent/workspace_manager.go、plugins/workspace-manager/main.go、internal/config/config.go、internal/service/prompt.go。
• 已确认现有验证入口：make test（Go 全量测试）、以及 plugins/workspace-manager/main_test.go / internal/agent/opencode_test.go 等相关单测。

Outcome

计划内容

1. 先固化决策标准（本 issue 对齐）

   • 用 4 个维度统一比较三种方案：隔离强度（安全/污染风险）、一致性（可复现性）、成本（磁盘/网络/耗时）、运维复杂度。
   • 在该仓库语境下，给出默认结论：
     • 方案 2（独立 clone）= 默认推荐。
     • 方案 3（虚拟化）= 安全增强场景可选。
     • 方案 1（worktree）= 不作为主路径（仅保留历史兼容讨论，不新增投入）。

2. 把“默认方案 2 + 可选方案 3”沉淀为配置化架构

   • 在 internal/config/config.go 增加隔离策略配置（例如 opencode.isolation_mode），建议值：clone（默认）/virtualized（预留）。
   • 保持现有 workspace_manager_host 与 repo 级 override 机制不变，避免破坏多仓库路由。

3. 在 workspace-manager 抽象隔离后端（不改变对 Bridge 的主协议）

   • 以 plugins/workspace-manager/main.go 当前 createOrReuse 为入口，抽象 WorkspaceProvider（例如 CloneProvider / VirtualizedProvider）。
   • 保持 POST /workspaces/create-or-reuse 和返回 worktreePath 的契约稳定，Bridge 侧 internal/agent/workspace_manager.go 仅感知路径，不感知底层实现细节。

4. 分阶段落地，先“文档与开关”，再“虚拟化实现”

   • 阶段 A：仅补充配置与文档说明（默认仍是 clone），明确适用边界与风险。
   • 阶段 B：实现 virtualized provider（容器/VM 任选其一），并通过配置灰度启用。
   • 阶段 C：在高风险仓库或特定任务类型（如外部 PR review）启用虚拟化策略。

5. 测试与回归策略

   • 单测：扩展 plugins/workspace-manager/main_test.go，覆盖 provider 选择、路径回收、安全校验（禁止删除受管目录外路径）。
   • 集成：覆盖 issue/pr 两条链路的 create-or-reuse、复用、强制重建（force=true）与 PR head 刷新逻辑。
   • 回归：验证 internal/agent/opencode.go 的 session 复用逻辑不受影响（仅 workspace 准备阶段可插拔）。

补充

• 现状分析：当前实现已是“按 {owner}/{repo}/{kind}/{number} 固定目录复用 + 首次 clone + 后续复用/刷新”，与方案 2 完全一致；且提示词和会话管理也围绕“预先准备好的独立工作目录”构建。
• 风险与影响：若直接切到虚拟化作为默认，会引入资源调度、镜像管理、启动耗时、调试复杂度和运维门槛上升；但在安全隔离诉求高的环境中价值明显。
• 推荐结论：默认继续方案 2，按需引入方案 3；方案 1 维持 deprecated。

疑问

• 是否需要我在下一步补一份“决策矩阵阈值”草案（例如：当仓库敏感级别/并发量达到何条件时，从 clone 升级到 virtualized）？
• Plan / open questions / follow-up link