feat(UI):sse logs complete & dynamic import of cmps#1369
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR completes SSE (Server-Sent Events) logs functionality and refactors the MR view with dynamic component imports. The main purpose is to improve performance through code splitting and enhance the logging system for real-time task monitoring.
- Refactors MR detail page to use dynamic imports and tab-based navigation
- Enhances SSE logging with better duplicate detection and HTTP log fetching
- Consolidates SSE request functions into a centralized module
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| moon/apps/web/pages/[org]/mr/[link]/index.tsx | Refactors to use dynamic imports and tab-based layout for better code splitting |
| moon/apps/web/hooks/SSE/useRequest.ts | Removes old request function, replaced by consolidated module |
| moon/apps/web/hooks/SSE/useGetTaskStatus.ts | Adds new hook for fetching individual task status |
| moon/apps/web/hooks/SSE/useGetMrTaskStatus.ts | Adds new hook for fetching MR-level task status with polling |
| moon/apps/web/hooks/SSE/useGetMrTask.ts | Updates to use consolidated request module |
| moon/apps/web/hooks/SSE/useGetHTTPLog.ts | Adds new hook for HTTP log fetching |
| moon/apps/web/hooks/SSE/ssmRequest.ts | Consolidates all SSE-related API requests |
| moon/apps/web/components/MrView/hook/useSSM.ts | Improves duplicate log detection and data structure |
| moon/apps/web/components/MrView/components/Checks/index.tsx | Enhances with better log management and status handling |
| moon/apps/web/components/MrView/components/Checks/cpns/store.ts | Updates status enum and atom types |
| moon/apps/web/components/MrView/components/Checks/cpns/Task.tsx | Refactors task rendering with improved status handling |
| moon/apps/web/components/Layout/TabLayout.tsx | Adds new tab navigation component |
| moon/apps/web/components/Issues/utils/store.tsx | Renames buildId atom for consistency |
| moon/apps/web/components/Issues/IssueDetailPage.tsx | Updates to use link instead of issueDetail.id |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
benjamin-747
approved these changes
Aug 25, 2025
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.
完善sse日志需求
分离conversation、checks、filechange渲染逻辑