session-flow: continue-in-background dirty-tree gate ignores worktree.bgIsolation
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 76/100
Hướng nghiên cứu
Đọc plugins/session-flow/skills/continue-in-background/SKILL.md, đặc biệt là bước 1, “Dirty-tree gate”, và so sánh cách plugin source-control đọc cài đặt trong plugins/source-control/hooks/worktree-create-gate.sh. Cập nhật cổng kiểm tra và báo cáo khởi chạy để tính đến trường hợp worktree.bgIsolation được đặt thành none; xác minh lý do từ chối và cảnh báo khớp với hành vi mà issue mong đợi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Summary
/session-flow:continue-in-background's dirty-tree gate assumes a background session always moves into an isolated git worktree before editing, unless the current session is already in a linked worktree. The worktree.bgIsolation setting can turn that isolation off, and then the background session edits the working copy directly. The gate does not read the setting, so its reasoning and outcome are wrong in that configuration.
Affected area / component
plugins/session-flow/skills/continue-in-background/SKILL.md (step 1, "Dirty-tree gate")
Steps to reproduce
- In a git repository, set
"worktree": {"bgIsolation": "none"}in.claude/settings.json. - Leave an uncommitted change in the checkout.
- Run
/session-flow:continue-in-background.
Expected behavior
The gate accounts for worktree.bgIsolation. With isolation off, the launched session sees the uncommitted changes, so the "changes won't carry over" reason for refusing does not apply. The real risk is different: the background session and the user would edit the same checkout at the same time, and the launch report should say so.
Actual behavior
SKILL.md:124-130: "Background sessions move into an isolated git worktree (a fresh checkout) before editing files (https://code.claude.com/docs/en/agent-view), so OTHER uncommitted changes in this checkout would NOT carry into the launched agent's edits. Any such changes → do NOT launch". The only exception is a session already inside a linked worktree.- Live docs, https://code.claude.com/docs/en/agent-view#how-file-edits-are-isolated (read 2026-10-12): "To turn off worktree isolation for a repository where git worktrees are impractical, set
worktree.bgIsolationto"none". Background sessions then edit your working copy directly without moving into a worktree first." The same section lists other skip cases (a session moved to the background with/background, a write outside the working directory). - Nothing in
plugins/session-flow/mentionsbgIsolation(grep -rn bgIsolation plugins/session-flowreturns nothing). The source-control plugin already names the setting (plugins/source-control/hooks/worktree-create-gate.sh:64).
Suggested fix
Read the effective worktree.bgIsolation value before the dirty-tree check. When it is "none", skip the "changes won't carry over" refusal and instead warn that the launched session will edit this checkout in place, alongside the user. This is a gate-logic change, not a pointer fix.
Environment
session-flow on main as of 2026-10-12.
Found during the upstream-pointer sweep (PRs #7061-#7274).
- Ngôn ngữ chính
- Shell
- Star
- 22
- Fork
- 2
- Merge trung bình
- 6 giờ 42 phút
- Pull request đã merge (30 ngày)
- 902
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của melodic-software/claude-code-plugins
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdĐang mởneeds-human needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
melodic-software/claude-code-plugins#7286 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs-triage priority: low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
melodic-software/claude-code-plugins#7282 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs-triage priority: medium
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
melodic-software/claude-code-plugins#7273 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs-human needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
melodic-software/claude-code-plugins#7232 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
source-control: read-pr.sh calls GraphQL before its REST read, so it fails in cloud sessionsĐang mởgood first issue needs-triage priority: high
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
melodic-software/claude-code-plugins#7223 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của melodic-software/claude-code-plugins
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
sclorg/s2i-ruby-container#660 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
wazuh/wazuh-docker#2732 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Route 53 DNS analytics/insights adds an observability signal for the DNS layerCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởdocumentation enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100