fix(approval): 도메인 해제 승인을 별도 유형 DOMAIN_UNBIND 로 (#8) - #292
Merged
Conversation
도메인 삭제도 승인을 만드는데(operation=DELETE 인 DOMAIN_BIND 스텝) bind 와 같은 DOMAIN_BINDING 유형이라, 화면 승인 카드가 "도메인 연결 승인"으로 떴다 — 실제로는 해제인데. 되돌리기 어려운 삭제 승인에서 반대 문구는 위험하다(배포 e2e 발견 #8). FE 와 (b) 별도 유형으로 합의: 화면은 모르는 유형에 원시 라벨을 그대로 노출하므로 (?? type), operation 메타(a)는 신호를 놓치면 "연결"로 폴백해 거짓이 되지만, 별도 유형은 최악이 못생긴 라벨(거짓 아님) — 되돌릴 수 없는 승인엔 모호 > 틀림. 플래그로 묶으면 다음에 문구 손대는 사람이 놓쳐 재발한다는 점도. - ApprovalType.DOMAIN_UNBIND 추가. - AgentOrchestrator.toApprovalType: DOMAIN_BIND 를 operation=DELETE 로 분기(INFRA_OPERATE 가 이미 operation 으로 분기하는 것과 같은 패턴). bind 는 그대로 DOMAIN_BINDING. - ProjectApprovalPolicy.requires: DOMAIN_UNBIND 도 domainApprovalRequired(연결과 같은 정책) — 승인 요구 여부는 안 바꾸고 유형 라벨만 분리. exhaustive switch 라 컴파일이 강제. - approve/reject 라우팅은 task_id 기반(isStandalone)이라 DOMAIN_UNBIND(agent-task)도 DOMAIN_BINDING 과 동일하게 태스크 resume — 무변경. FE 는 이 유형 문구를 이미 pre-add(#67) — 이 배포부터 "도메인 연결 해제 승인"으로 제대로 읽힌다. 테스트: DOMAIN_BIND+operation=DELETE → DOMAIN_UNBIND 승인 생성 단언. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WHX8hzKT3JsTTRRvdUkL5K
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
문제 (배포 e2e 발견 #8)
도메인 삭제도 승인을 만드는데(operation=DELETE) bind 와 같은
DOMAIN_BINDING유형이라, 승인 카드 제목이 "도메인 연결 승인" 으로 떴다 — 실제로는 해제인데. 되돌리기 어려운 삭제에서 반대 문구는 위험.해결 — (b) 별도 유형 (FE 와 합의)
화면은 모르는 유형에 원시 라벨을 그대로 노출(
?? type)하므로:(a) operation 메타 → 신호 놓치면 "연결"로 폴백 = 거짓
(b) 별도 유형 → 최악이 못생긴 라벨 = 거짓 아님. 되돌릴 수 없는 승인엔 모호 > 틀림.
ApprovalType.DOMAIN_UNBIND추가toApprovalType: DOMAIN_BIND 를operation=DELETE로 분기(INFRA_OPERATE 패턴과 동일). bind→DOMAIN_BINDINGProjectApprovalPolicy: DOMAIN_UNBIND→domainApprovalRequired(연결과 동일 정책, 요구 여부 불변)approve 라우팅은 task_id 기반이라 무변경(agent-task resume 동일)
검증
agent.orchestrator.*·policy 통과.트랙
승인 트랙(#1·#7·#8) 중 #8. #7 FE 완료(#68), #1(배포 승인 게이트) 다음.
🤖 Generated with Claude Code