refactor(orchestrator): replace dirty-workspace quarantine and attribution with a warning

[moncher-dev]

Phase 4 of epic #847. 승인된 계획은 #847의 2026-09-05 코멘트에 있습니다.

무엇을 하는가

워크스페이스에 커밋되지 않은 변경이 남아 있을 때 이를 .quarantine-<timestamp>로 옮겨두는 동작과, 그 변경이 이 이슈의 것인지 증거로 판정하는 귀속 로직을 함께 없앱니다. 둘은 한 덩어리라 따로 뗄 수 없습니다.

대체 동작 — hojinzs 결정 (2026-09-05)

더러운 워크스페이스를 만났을 때 격리하지 않습니다. 경고를 남기고 진행합니다. 실패로 처리하지 않습니다.

이 결정의 근거를 그대로 옮깁니다. 이것은 상용 SW가 아니라 개발용 도구입니다. 남은 변경을 사람이 판단할 수 있게 보이게만 해주면 되고, 오케스트레이터가 대신 격리 디렉터리로 옮겨 감추는 것은 과도한 통제였습니다.

임의로 실패 처리를 고르지 마세요. 이 문장이 완료 조건입니다.

대상

위치	내용
packages/orchestrator/src/git.ts:490	quarantineIssueWorkspace
packages/orchestrator/src/service.ts:3133-3164	귀속 판정 후 격리를 부르는 경로
packages/core/src/workflow/issue-identity.ts:148-171	DirtyWorkAttributionInput, DirtyWorkAttribution, fail-closed 판정
packages/orchestrator/src/service.ts:193	resolveDirtyWorkAttributionBranches
packages/core/src/observability/structured-events.ts, event-formatter.ts	격리 이벤트

27군데, 5파일. 레이어는 Coordination과 Observability입니다.

남기는 것

• 더러운 상태의 관측. inspectIssueWorkspaceDirtyStatus(git.ts:449)는 남습니다.
• 에이전트에게 주는 복구 컨텍스트. WORKFLOW.md의 Recovery Context와 service.ts:6699의 안내 문구는 더러운 diff를 사람이 판단하도록 유도하는 것이지 격리에 의존하지 않습니다. 다만 "attributed to this issue"라는 표현이 사라진 귀속 로직을 가리키므로 문구를 맞춰야 합니다.
• 트래커 어댑터 계약의 secretEnvironmentNames 등 무관한 것.

완료 조건

• 더러운 워크스페이스를 만나면 경고를 남기고 진행한다. 이동도, 실패도, 삭제도 하지 않는다.
• 경고가 관측 가능하다 — 구조화 이벤트나 상태 스냅샷 경고로 사람이 볼 수 있다.
• .quarantine-* 디렉터리를 더 이상 만들지 않는다.
• 귀속 판정 코드와 그 이벤트가 제거된다.
• WORKFLOW.md의 복구 안내에서 귀속에 기대는 표현이 정리된다. 정책 파일이므로 문구만 고치고 다른 규칙의 의미를 건드리지 마세요.
• docs/architecture.md에서 해당 서술이 정리된다.
• changeset 추가.

범위 밖

워크스페이스 경로 봉쇄 검사, before_remove 훅, 그리고 population 이전(P4‑1).

참고

기존 .quarantine-* 디렉터리가 호스트에 남아 있을 수 있습니다. 이 이슈는 코드만 다루고, 남은 디렉터리 정리는 운영자 몫입니다. 워커는 더러운 워크스페이스를 삭제하지 마세요.

[moncher-dev]

Workpad — #905 — Cycle 1

Type: fresh pickup
Branch: symphony/hojinzs-github-symphony-905
Draft PR: #915
Cycle opened: 2026-09-05T15:55:25Z · Trigger: Ready issue has actionable, bounded Coordination and Observability refactor requirements.

Status Transitions

• 2026-09-05T15:56:07Z · Ready → In progress · scope is triaged and implementation is starting (cycle 1 open)
• Intended: In progress → In review · PR refactor(orchestrator): warn on dirty workspaces #915 is published and the Completion Bar passed with Docker exception fix(e2e): restore Docker daemon access for Symphony workers #914 (cycle 1 close)
• Exact comment body:
  • 🔁 Status: In progress → In review
  • Reason: PR refactor(orchestrator): warn on dirty workspaces #915 is published and the Completion Bar passed with Docker exception fix(e2e): restore Docker daemon access for Symphony workers #914
  • Cycle: 1 close

Plan

• 1. Replace dirty-workspace quarantine and attribution with an observable non-failing warning, update policy/architecture wording and tests, and commit the shippable slice
• 2. Publish via /push and open Draft PR via /gh-pr-writeup
• 3. Wrap-up: re-verify the original issue · pass the Completion Bar · changeset · PR ready · transition to In review

Rework / PR Feedback

None (fresh pickup).

Completion Bar

• In-scope requirements implemented
• pnpm lint
• pnpm test
• pnpm typecheck
• pnpm build
• Docker E2E — pass (documented environment exception: fix(e2e): restore Docker daemon access for Symphony workers #914)
• Tests for new functionality
• Code conventions (CLAUDE.md)
• Review threads answered and resolved (N/A: fresh cycle)

Validation

• pnpm lint — pass
• pnpm test — pass
• pnpm typecheck — pass
• pnpm build — pass
• pnpm exec vitest run packages/orchestrator/src/service.test.ts -t "warns and continues when a recovery workspace is dirty" — pass
• pnpm exec vitest run packages/core/src/workflow/issue-identity.test.ts packages/core/src/observability/event-formatter.test.ts — pass
• ./e2e/run-standalone-project-e2e.sh — fail twice at Docker preflight because the daemon is unreachable; pass (documented exception: fix(e2e): restore Docker daemon access for Symphony workers #914)
• Changeset: .changeset/warm-dogs-warn.md
• Docker E2E evidence: preflight output recorded in fix(e2e): restore Docker daemon access for Symphony workers #914; no container started
• Merge commit (Land cycle only): N/A

Delegation (out-of-scope / human / post-merge)

• Operators remain responsible for any legacy .quarantine-* directories already on hosts.
• Docker daemon availability tracked in fix(e2e): restore Docker daemon access for Symphony workers #914.

Progress Log

• 2026-09-05T15:55:25Z: Triaged the issue, classified it across Coordination, Observability, and Policy wording, and opened cycle 1.
• 2026-09-05T16:08:18Z: Replaced quarantine and attribution with a retained-workspace warning event, updated recovery wording and architecture/test documentation, passed all non-Docker validation, documented the Docker infrastructure exception in fix(e2e): restore Docker daemon access for Symphony workers #914, and committed 91bc279.
• 2026-09-05T16:09:28Z: Published 91bc279, verified the branch is one commit ahead of main, opened Draft PR refactor(orchestrator): warn on dirty workspaces #915, re-verified every issue requirement, and prepared the review handoff.

Blockers

None

[moncher-dev]

🔁 Status: Ready → In progress

Reason: Scope is triaged and implementation is starting
Cycle: 1 open

[moncher-dev]

🔁 Status: In progress → In review

Reason: PR #915 is published and the Completion Bar passed with Docker exception #914
Cycle: 1 close

[moncher-dev]

재작업 라운드 1 — PR #915. 결정이 하나 필요하므로 지금 착수하지 마세요

hojinzs가 91bc279에서 CHANGES_REQUESTED를 남겼습니다. 전문은 PR #915에 있습니다.

잘 된 것 — 되돌리지 마세요

임의로 실패 처리를 고르지 않았습니다. quarantine과 귀속을 한 덩어리로 제거했고, 608줄을 지우고 61줄을 더하는 삭제 중심이라 과설계가 없습니다. 경고는 stderr와 recovery-dirty-workspace 구조화 이벤트, 포매터로 관측 가능합니다. .quarantine-* 미생성을 테스트가 sibling 부재로 단언합니다. WORKFLOW.md는 문구만 고치고 다른 규칙의 의미를 건드리지 않았습니다. 오케스트레이터 레이어만 보면 결정대로 동작합니다.

P1 — 결정이 워커 경계에서 무너집니다

워크스페이스를 그 자리에 두면 브랜치도 그대로 남습니다. 워커는 에이전트를 띄우기 전에 identity preflight를 돌리고(worker/src/index.ts:535), 브랜치가 다른 이슈 번호를 가리키면 fail-closed로 거부합니다(worker/src/identity-preflight.ts:100-108).

리뷰어가 이 PR의 회귀 테스트가 만드는 바로 그 상황을 실제 preflight에 넣었습니다.

{"ok":false,"reason":"workspace branch 'fix/2-foreign' belongs to issue #2 instead of 'acme/platform#1'"}

exitWorkerStartupFailure로 이어져 런이 시작조차 못 합니다. 이제 워크스페이스를 옮기지도 지우지도 않으니 재시도해도 브랜치가 그대로라 재시도 예산이 소진될 때까지 같은 벽에 부딪힙니다. 예전 quarantine 경로는 깨끗한 클론을 새로 만들며 이 문제를 우연히 피하고 있었습니다.

결과적으로 "사람이 판단하도록 보여준다"는 목적 대신 조용한 반복 실패가 됩니다. 완료 조건이 명시적으로 금지한 "실패"입니다.

이건 구현 실수가 아니라 결정의 빈틈입니다

hojinzs의 결정("격리하지 않고 경고 후 진행")은 오케스트레이터 레이어를 두고 내린 것이었고, 워커의 identity preflight와의 상호작용은 그때 고려되지 않았습니다. 어느 갈래로 갈지는 워커가 고를 일이 아닙니다. hojinzs에게 올렸으니 답이 이 이슈에 달릴 때까지 P1은 손대지 마세요.

선택지는 대략 셋입니다. 참고용이며 확정이 아닙니다.

1. preflight를 완화 — 더러운 재사용 워크스페이스에서의 브랜치 불일치는 fail-closed 대신 경고로 처리한다. 결정의 취지에 가장 충실하지만 교차 이슈 오염 방어를 약화시킨다.
2. 브랜치만 재설정 — 워킹 트리는 그대로 두고 브랜치를 할당된 것으로 옮긴다. 더러운 파일은 그대로 보이고 preflight는 통과한다.
3. 워크스페이스를 재사용하지 않음 — 다른 이슈의 브랜치를 가리키면 새 워크스페이스 경로를 쓴다. 옛 quarantine에 가깝지만 이동이 없다.

P2 — 지금 해도 되는 것

어댑터 훅 5곳에 귀속 판정 잔재가 남아 있습니다. 이건 결정과 무관하니 정리해도 됩니다.