feat: Knowledge Collection 운영 관리 기능 보강 - #442
Conversation
64cf6f5 to
44ed651
Compare
44ed651 to
7f49f2b
Compare
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7f49f2b78c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| target_count_bucket=bucket_count(len(collections)), | ||
| changed_count_bucket=bucket_count(changed_count), | ||
| unchanged_count_bucket=bucket_count(len(collections) - changed_count), |
There was a problem hiding this comment.
11개 이상(최대 50개) KC를 bulk grant/revoke하면 여기서 공통 bucket_count가 11-100을 반환하지만 KnowledgeCollectionPermissionBulkBundleResponse는 11-50만 허용합니다. 이 모델 생성은 이미 self.db.commit() 이후라 권한 변경은 저장되고 API는 Pydantic ValidationError/500으로 실패해 UI가 실패로 표시하며 재시도도 계속 실패합니다. Bulk 전용 버킷 함수나 응답 스키마를 서로 일치시켜야 합니다.
Useful? React with 👍 / 👎.
| self._require_manual_collection(collection) | ||
| if collection.lifecycle_state != "active": | ||
| self.unit_of_work.rollback() | ||
| raise CollectionStateConflict("collection_not_active") |
There was a problem hiding this comment.
삭제된 manual Collection에 reorder 요청이 들어오면 repository가 deleted row를 반환한 뒤 이 분기에서 collection_not_active 409를 내므로, 기존 서비스의 lifecycle_state != "deleted" hidden 처리와 달리 같은 조직의 호출자가 삭제된 리소스 존재를 404와 구분할 수 있습니다. reorder()도 archive/restore처럼 deleted 상태를 먼저 CollectionHidden으로 처리해야 합니다.
Useful? React with 👍 / 👎.
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1 participant
변경 사항
관련 이슈
Linear: MBA-264
변경 유형
테스트
로컬에서 테스트 완료
기존 테스트 통과 확인
MBA-264 집중 검증: Gateway/Shared 101 passed, PostgreSQL 통합 4 passed, Client API 28 passed, Client component 7 passed
변경 파일 Ruff 및 ESLint 검증 완료, Client production build 통과
전체 회귀 테스트는 PR 전 1회 실행했습니다.
apscheduler가 없어 중단되었으며, 변경 범위 집중 테스트는 통과했습니다.스크린샷 (UI 변경 시)