[Bug]: Git 仓库目录所有者与当前运行用户不一致时，审核等功能失败
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 34/100
調査の方向性
まず src/crates/services/services-integrations/src/git/utils.rs の Git コマンド処理と review_platform.rs の workspace 操作を追跡し、次に get_file_diff_tool.rs と remote_git.rs を別々の経路として調査します。既存の Git 関連テストを実行し、一覧にあるエントリポイントを使って不足しているカバレッジを特定します。完了の基準は、影響を受けるローカル、CLI、review、diff、remote の各ワークフローに、認識可能な ownership エラー、スコープを限定した処理、ユーザー向けガイダンス、回帰テストのカバレッジがあることです。
索引モデルが issue の本文から書いたものです。
説明
问题描述
当工作区 Git 仓库根目录的文件系统所有者（owner）与运行 BitFun 的当前操作系统用户不一致时，Git 会触发仓库信任校验（safe.directory），并拒绝在该仓库中执行操作。
典型报错：
fatal: detected dubious ownership in repository at <repo-path>
目前 BitFun 没有对该场景做统一的预检、错误分类或恢复提示，因此“审核/Review”、获取审核 diff、远程仓库探测，以及部分 Git 工作流可能直接失败。
复现条件
- 准备一个目录 owner 与当前运行 BitFun 的用户不同的 Git 仓库（例如共享目录、挂载目录、容器映射目录或以其他系统用户创建的目录）。
- 在 BitFun 中将该目录作为工作区。
- 执行审核/Review、查看 diff 或其他需要 Git 的操作。
- 操作失败，并在日志或错误链中出现
detected dubious ownership/safe.directory相关信息。
代码检查证据
这是对当前仓库代码的静态检查结果：
src/crates/services/services-integrations/src/git/utils.rs:253等 Git 外部命令封装路径没有注入 scopedsafe.directory，命令失败会直接转成CommandFailed。src/crates/services/services-integrations/src/review_platform.rs:790的工作区探测和 Git 操作使用同类外部 Git 调用。src/crates/assembly/core/src/agentic/tools/implementations/get_file_diff_tool.rs:1022在安全审核路径中把 Git diff 失败包装成 “Git diff is unavailable within the safety boundary”，没有针对所有权不一致做处理。src/crates/services/services-integrations/src/remote_ssh/remote_git.rs:23的远程 Git 命令构造同样没有safe.directory策略；远端 SSH 用户与仓库 owner 不一致时也可能触发相同问题。- CLI 中还有若干直接调用 Git 的路径，可能绕过统一封装。
在当前代码中未检索到 safe.directory、dubious ownership 等专门处理逻辑，现有 Git 测试也未覆盖仓库 owner 与当前用户不一致的场景。当前结论来自静态代码检查，尚未在本机通过跨用户目录复现。
期望行为
- 在 Git 服务层增加统一的仓库信任预检和可识别的错误类型，覆盖审核、diff、远程探测、CLI、远端工作区和其他内部 Git 调用。
- 对用户明确选择并确认的工作区，可在单次/单进程 Git 调用中使用规范化后的仓库根目录作为 scoped
safe.directory，不要写入全局 Git 配置，也不要默认使用safe.directory=*。 - 远程工作区应在远端主机上、以实际 SSH 用户身份执行相同的信任判断。
- 向用户显示明确的处理建议，并补充 Unix owner mismatch、Windows ACL/挂载目录、容器/WSL/NFS/SMB、远程 SSH 等回归测试。
- 主要言語
- Rust
- スター
- 2.3k
- フォーク
- 236
- 平均マージ
- 2時間 42分
- マージ済み PR（30日）
- 466
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
GCWing/OpenBitFun のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
GCWing/OpenBitFun#3213 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
GCWing/OpenBitFun#2363 ·
メンテナーはふだん 1 日以内に返信
-
question
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
GCWing/OpenBitFun#2340 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 1/100
GCWing/OpenBitFun#3273 ·
メンテナーはふだん 1 日以内に返信
-
[Feature]: 生态兼容逻辑优化オープン
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
GCWing/OpenBitFun#3271 ·
メンテナーはふだん 1 日以内に返信
GCWing/OpenBitFun の issue をすべて見る
似ている issue
-
component:sight
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
agentic-os-org/ANOLISA#4115 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
A-io-database bug needs triage python
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信
-
Change output crossing a compactsize boundary leaves the fee slightly below the requested feerateオープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
bitcoindevkit/bdk_wallet#578 ·
メンテナーはふだん 8 日以内に返信
-
`python.analysis` setting changes (e.g. `completeFunctionParens`) are not picked up until reloadオープンlanguage-server
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 2 日以内に返信