feat(cycle-7): polish — bezier canvas, signal animation, traces UI#12
Open
sameerchereddy wants to merge 1 commit intomainfrom
Open
feat(cycle-7): polish — bezier canvas, signal animation, traces UI#12sameerchereddy wants to merge 1 commit intomainfrom
sameerchereddy wants to merge 1 commit intomainfrom
Conversation
Canvas:
- Replaced straight spoke lines with routed bezier SVG connectors
(vertical exit for top/bottom blocks, horizontal for left/right)
- SVG uses viewBox="0 0 100 100" + preserveAspectRatio="none" —
eliminates ResizeObserver/canvasSize; block % positions map directly
- Animated signal dots travel along each connector via animateMotion+mpath
- activeNodeKey state tracks which block is live per WS event;
active connector highlights and dot accelerates
- Block/tool nodes glow via animate-node-glow when processing
- Colors via style={} (CSS inline) so hsl(var(--channel)) resolves correctly
Backend:
- traces.py: GET /api/traces + GET /api/traces/{id} with asyncpg
- run.py: passes enabled_tool_keys to run_react_loop
- agents.py: auto-seeds default tools on create; seed-defaults endpoint
- loop.py: system prompt guidance to avoid unnecessary tool calls
- main.py: registers traces router
Frontend:
- TracesPage: list with agent_id filter, skeleton loader, fade-in rows
- TraceDetailPage: iteration-by-iteration ReAct breakdown, token usage
- OnboardingPage: 3-step flow with staggered animations, personalized greeting
- index.css: slideIn/scaleIn/nodeGlow keyframes + utility classes
🤖 AI Code ReviewAI review unavailable. Reviewed by GitHub Models (gpt-4o) · Not a substitute for human review |
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
1 participant
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.
Summary
animateMotion+mpath; active connector highlights and dot accelerates during live WS events; block/tool nodes glow when processingGET /api/tracesandGET /api/traces/{id}endpoints (asyncpg); TracesPage with agent filter, skeleton loader, fade-in rows; TraceDetailPage with iteration-by-iteration ReAct breakdown and token usageenabled_tool_keyspassed through torun_react_loop; system prompt guidance to reduce unnecessary tool callsslideIn,scaleIn,nodeGlowkeyframes + utility classes inindex.cssTest plan
GET /api/tracesandGET /api/traces/{id}return correct data