Skip to content

feat(dataset): connect delete functionality and diagnosis data linking#7

Merged
FluxloopAdmin merged 6 commits into
mainfrom
feature/dataset-functionality
Jan 29, 2026
Merged

feat(dataset): connect delete functionality and diagnosis data linking#7
FluxloopAdmin merged 6 commits into
mainfrom
feature/dataset-functionality

Conversation

@tendtoyj
Copy link
Copy Markdown
Collaborator

Summary

데이터셋 상세 페이지의 백엔드 기능을 연결하고 UI를 개선하는 PR입니다. 삭제 기능, 진단 데이터 연동, 자동 네비게이션 등 핵심 기능을 구현했습니다.

Changes

  • 삭제 기능 연결: 데이터셋 삭제 시 확인 다이얼로그 표시 후 deleteFileAsset/dropCache API 호출, 성공 시 목록 페이지로 이동
  • 헤더 UI 간소화: 불필요한 펜슬 아이콘 버튼 제거, Export as CSV 메뉴 제거 (백엔드 미지원), Rename 메뉴 비활성화 상태로 추가
  • 진단 데이터 연동: FileAssetFileDiagnosis 모델에 diagnosis_id 필드 추가, 파일 임포트 시 진단 데이터와 연결, 상세 페이지에서 실제 LLM 분석 결과 표시
  • 자동 네비게이션: 데이터셋 생성 완료 후 자동으로 해당 데이터셋 상세 페이지로 이동
  • 메모 상태 리셋 버그 수정: 데이터셋 전환 시 이전 메모 내용이 유지되던 문제 수정
  • LLM 제안 이름 사용: 파일 임포트 시 원본 파일명 대신 LLM이 제안한 이름 또는 사용자 편집 이름 사용

Test plan

  • 새 데이터셋 업로드 후 자동으로 상세 페이지로 이동하는지 확인
  • 상세 페이지의 Summary 탭에서 LLM 진단 결과가 표시되는지 확인
  • 데이터셋 삭제 시 확인 다이얼로그가 나타나고 삭제가 정상 동작하는지 확인
  • 여러 데이터셋 간 전환 시 메모 내용이 올바르게 리셋되는지 확인
  • 업로드된 데이터셋의 이름이 LLM 제안 이름으로 설정되는지 확인

🤖 Generated with Claude Code

tendtoyj and others added 5 commits January 28, 2026 11:44
Previously, the importFile API was always receiving the original
filename for the `name` parameter, ignoring the LLM-suggested name
or user-edited name that was correctly being used for `table_name`.

Changed to use the already-computed `tableName` variable which contains
the LLM suggestion, user edit, or auto-generated fallback.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Pass created FileAsset from AddDatasetModal to page.tsx callback,
enabling automatic selection of the new dataset and navigation to
the datasets view after successful import.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add diagnosis_id field to FileAsset and FileDiagnosis models
- Add GET /files/{file_id}/diagnosis endpoint to retrieve linked diagnosis
- Add get_diagnosis_by_id method for direct lookup by diagnosis ID
- Pass diagnosis_id when importing files to establish linkage
- Replace mock agent analysis with real LLM analysis in dataset detail view
- Add loading state for diagnosis data in summary tab

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Previously, memo content persisted in React state when navigating
between datasets, causing memos to appear shared across all datasets.
Now properly resets to empty string when no saved memo exists.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Remove standalone pencil icon button from header
- Remove Export as CSV menu item (no backend support)
- Add disabled Rename menu item in dropdown menu
- Connect Delete dataset to deleteFileAsset/dropCache API
- Show confirmation dialog before deletion
- Navigate back to list after successful deletion

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@tendtoyj tendtoyj self-assigned this Jan 29, 2026
- Remove debug console.log statements from page.tsx, AddDatasetModal,
  DatasetDetailView, and fileAssetApi
- Add CellValue type alias for row data instead of using any[][]
- Simplify redundant ternary expression in DatasetDetailView

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@FluxloopAdmin FluxloopAdmin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM - Approved

핵심 기능이 잘 구현되었습니다!

👍 Highlights

  • Diagnosis ↔ FileAsset 연결: diagnosis_id 링크 + 새 API 엔드포인트
  • Import 후 자동 네비게이션: 생성된 데이터셋으로 바로 이동
  • Mock → 실제 데이터: LLM 분석 결과 실제 연동
  • 삭제 기능: 확인 다이얼로그 포함 연결
  • 코드 정리: console.log 제거, 타입 안전성 개선

💡 향후 개선 (Optional)

  1. Logger import 위치 정리 (import 블록 후로)
  2. 문서 PR 분리 고려 (~9K+ lines)
  3. 한국어/영어 UI 텍스트 일관성

기능 코드 깔끔합니다! 🚀

@FluxloopAdmin FluxloopAdmin merged commit 1e7a4f4 into main Jan 29, 2026
1 check passed
@tendtoyj tendtoyj deleted the feature/dataset-functionality branch January 29, 2026 09:53
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.

2 participants