Repository navigation
Conversation
- PATCH /release/disabled, /release/mandatory 신규 (label 생략 시 최신) - updatePackageMetadata 부분 UPDATE로 stale 캐시 lost update 차단 - Slack MandatoryOn/Off 액션 추가, 헤더 이모지 🟢/🔴 → ✓/✕ 단순화 - 웹 Mandatory/Optional 토글 + 승격 경고 confirm, 카드 버튼 outline 통일 - api-client 재생성 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…atory=★) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
on/off는 헤더 텍스트가 isDisabled/isMandatory 값으로 표현 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- 인증·권한 통과 후 업로드 파이프라인(롤아웃 미완/중복 번들/R2·D1 오류) 실패 시 UploadFailed(🚨) 알림 발사 - 에러 메시지를 본문 코드 블록으로 첨부 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Mandatory 버튼: Zap 아이콘 (Mandatory로 전환 시 amber, Optional 전환 시 muted) - Disable 버튼: Power 아이콘 (Disable 시 red, Enable 시 emerald) - 버튼 골격은 무채색 outline 유지 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- 버튼 h-9 → h-7, 패딩 px-2, 글자 text-xs - 아이콘 [&_svg]:size-3 강제 (버튼 베이스 [&_svg]:size-4 오버라이드), gap-1 - 버튼 사이 간격 gap-0.5 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- label 지정/생략(최신) 대상 선택, 부분 UPDATE(다른 필드 불변), 404, 401 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- hooks/: useHistoryTarget(앱·배포 선택), useReleaseHistory(이력·지표·페이지), useReleaseToggles(disabled/mandatory 토글) - components/: ReleaseCard, HistoryPagination 분리 (formatBytes/formatTime은 ReleaseCard로 이동) - types.ts로 응답 타입 분리 - useEffect 3개 기명화: selectFirstAppOnLoad / selectDefaultDeployment / resetPageOnTargetChange - 동작 동일, 내부 구조만 변경 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
vitest-pool-workers가 wrangler.toml 경유로 .dev.vars의 CODE_PUSH_NOTI_SLACK을 로드해 토글 테스트가 실채널로 webhook을 쏘고 있었음. 테스트 바인딩에서 빈 값으로 고정. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
6 tasks done
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
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.
요약
배경
patch -m으로 mandatory 변경이 가능했으나, 이 재구현 서버는PackageInfoUpdateSchema에isMandatory가 없고updatePackage도 UPDATE하지 않아 200 응답을 주면서 조용히 무시하는 상태.변경 사항
management.ts— disabled/mandatory 전용 PATCH 엔드포인트 2개 신설/release하나로 appVersion/description/isDisabled만 수정 가능, isMandatory는 침묵 무시/release/disabled{label?, isDisabled}//release/mandatory{label?, isMandatory}로 각 상태를 따로 관리/release는 CLI(code-push-standalone patch) 호환을 위해 그대로 유지d1.ts/storage.ts—updatePackageMetadata부분 UPDATE 신설updatePackage가 캐시에서 읽은 release의 여러 필드를 통째 SETslack.ts— 액션 종류 재구성 + 업로드 실패 알림 추가Disabled true/false,Mandatory true/false로 표현 (이모지는 종류 구분만)HistoryPage.tsx(web) — Mandatory 토글 + 버튼 단순화disableMutation을 신규/release/disabled로 교체,mandatoryMutation신설api-client—pnpm generate재생성 (신규 메서드 2개)테스트
/release200 불변참고
/release(CLI 경로)에는 전체 필드 SET 패턴이 남아 있음 — pre-existing이며 범위 밖 (별도 트랙 후보)is_mandatory컬럼 기존재)🤖 Generated with Claude Code