TA-232: Session Graph View Component #13
Conversation
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Summary of ChangesHello @changhoi, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 풀 리퀘스트는 세션 상세 페이지에 혁신적인 그래프 뷰를 도입하여 사용자가 복잡한 에이전트 상호작용을 더 쉽게 이해할 수 있도록 합니다. 메인 세션과 서브 에이전트 간의 호출 관계를 시각적으로 표현하고, 각 단계에서 사용된 토큰 정보를 제공함으로써, 시스템의 동작 방식과 리소스 흐름에 대한 깊이 있는 통찰력을 제공합니다. 이는 디버깅 및 성능 분석에 큰 도움이 될 것입니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| const selectedNodeSessions = useMemo(() => { | ||
| if (!selectedNodeId) return [] | ||
| const node = nodes.find((n) => n.id === selectedNodeId) | ||
| if (!node || !isSessionNode(node)) return [] | ||
| return node.data.sessions | ||
| }, [nodes, selectedNodeId]) | ||
|
|
||
| // Get selected node's label for popover | ||
| const selectedNodeLabel = useMemo(() => { | ||
| if (!selectedNodeId) return '' | ||
| const node = nodes.find((n) => n.id === selectedNodeId) | ||
| if (!node || !isSessionNode(node)) return '' | ||
| return node.data.label | ||
| }, [nodes, selectedNodeId]) |
There was a problem hiding this comment.
두 개의 useMemo 훅에서 selectedNodeSessions와 selectedNodeLabel을 계산하기 위해 각각 nodes 배열을 순회하고 있습니다. 이로 인해 불필요한 중복 계산이 발생할 수 있습니다.
하나의 useMemo 훅을 사용하여 selectedNode를 찾은 다음, 그 결과를 바탕으로 selectedNodeSessions와 selectedNodeLabel을 파생시키는 것이 더 효율적이고 가독성이 좋습니다.
// Get selected node and its data for popover
const selectedNode = useMemo(() => {
if (!selectedNodeId) {
return null
}
const node = nodes.find((n) => n.id === selectedNodeId)
return node && isSessionNode(node) ? node : null
}, [nodes, selectedNodeId])
const selectedNodeSessions = selectedNode?.data.sessions ?? []
const selectedNodeLabel = selectedNode?.data.label ?? ''
| } satisfies SessionNode) | ||
|
|
||
| // Group SubAgents by timestamp to identify parallel spawns | ||
| // Use toolExecutionId for grouping since parallel calls happen in same batch |
There was a problem hiding this comment.
이 주석은 구현 내용과 일치하지 않는 것 같습니다. 코드는 toolExecutionId가 아닌 타임스탬프를 기준으로 서브 에이전트를 그룹화하여 병렬 실행 여부를 판단하고 있습니다.
타임스탬프를 기준으로 그룹화하는 방식은 대부분의 경우에 잘 동작하겠지만, 완전히 견고하지는 않습니다. 순차적으로 실행된 별개의 에이전트 호출이 같은 초(second) 내에 발생할 경우 동일한 타임스탬프 키를 갖게 되어, 병렬로 잘못 렌더링될 수 있습니다.
더 신뢰성 있는 방법은 서브 에이전트를 생성한 부모 에이전트 메시지의 ID를 기준으로 그룹화하는 것입니다. 이를 위해서는 PROGRESS_TYPE_AGENT 이벤트에서부터 원본 에이전트 메시지까지 거슬러 올라가는 추가적인 로직이 필요할 수 있습니다.
우선은 현재 로직을 정확하게 설명하도록 주석을 수정하는 것이 좋겠습니다.
| // Use toolExecutionId for grouping since parallel calls happen in same batch | |
| // Use timestamp for grouping to identify parallel calls |
- Add buildToolExecutionMetadata helper function - Populate ToolExecution.Metadata with IsSubagent and SubagentID - Add unit tests for SubAgent metadata conversion Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…ents - Add maxRetries parameter to FindByBatchID in EventQueryPort - Update MongoDB adapter to filter by retryCount < maxRetries - Pass config maxRetries value in processBatch Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add ProgressTypeSkill constant to v2 domain - Re-add PROGRESS_TYPE_SKILL = 2 to proto definition - Update API gRPC converter to handle ProgressTypeSkill - Update frontend types and components for skill progress Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add GraphPopupState interface for external popup state management - Modify SessionGraphView to accept onNodeSelect callback - Move MessagePopover rendering to parent component Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
TA-233: Fix SubAgent-Tool relationship in claudecodeadapter conversionProblemSubAgent logs were not displaying correctly in the dashboard because Changes
Files Modified
|
TA-234: Fix event retry infinite increase by excluding max-retried eventsProblemEvent retry count was increasing infinitely because events that exceeded the maximum retry count were still being fetched for processing. SolutionSimple fix: Add Changes
Files Modified
|
TA-235: Fix Skill Progress type display in Session Detail UIProblemSession Detail UI was displaying "Unknown" for Skill Progress type messages because the ChangesGo Backend:
Frontend:
Files Modified
|
TA-236: Fix Graph View popup position by moving popup outside containerProblemThe MessagePopover in Graph View was rendered inside a Card container with SolutionMove popup state management to the parent component and render MessagePopover outside the SessionGraphView component, ensuring it displays without container clipping. Changes
Files Modified
|
Resolves TA-232
Summary
Session 상세 페이지에 Graph View를 추가합니다. React Flow를 사용하여 Main Session과 SubAgent들의 호출 관계를 좌우 흐름도(Left to Right) 형태로 시각화합니다.
Features
Issue
https://linear.app/team-attention/issue/TA-232/session-graph-view-component