Skip to content

feat(approval): connect desktop approval actions to API#2

Merged
BBAK-jun merged 1 commit intomainfrom
desktop-approval-writeflow
Mar 30, 2026
Merged

feat(approval): connect desktop approval actions to API#2
BBAK-jun merged 1 commit intomainfrom
desktop-approval-writeflow

Conversation

@BBAK-jun
Copy link
Copy Markdown
Owner

@BBAK-jun BBAK-jun commented Mar 30, 2026

What Changed

  • desktop approvals 화면에서 문서별 승인 요청을 실제 API write에 연결했습니다.
  • reviewer에게 할당된 open approval에 대해 desktop에서 바로 승인수정 요청을 처리할 수 있게 했습니다.
  • 승인 액션이 끝나면 bootstrap 상태를 다시 불러와 approval queue, 문서 상태, 버튼 disabled 상태가 바로 갱신되도록 했습니다.

Guardrails

  • 자신에게 승인 요청하는 self-review는 desktop 추천 단계와 API authoritative validation 양쪽에서 막았습니다.
  • approval decision은 해당 approval에 할당된 reviewer membership만 수행할 수 있도록 API에서 검증합니다.
  • refresh 실패가 approval write 실패처럼 보이지 않도록 mutation 성공과 후속 refresh 실패를 분리했습니다.

Verification

  • pnpm --filter @harness-docs/desktop typecheck
  • pnpm --filter @harness-docs/desktop test
  • pnpm --filter @harness-docs/desktop build
  • pnpm --filter @harness-docs/api build
  • pnpm test:api (fails in local env due to existing Docker container name /harness-docs-db-test)

@BBAK-jun BBAK-jun merged commit 76c42db into main Mar 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant