-
Notifications
You must be signed in to change notification settings - Fork 44
Update dependencies and improve test configurations #500
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: staging
Are you sure you want to change the base?
Conversation
Fix tests
Update README.md
Replaced the multiple H1 tags with H2 tags, keeping 'CODE GRAPH' as sole H1 tag
Update page.tsx
Update Next.js to version 15.5.7
- Bumped various dependencies including `next`, `playwright`, and `sharp` to their latest versions. - Updated Playwright configuration for improved test reporting and parallel execution. - Enhanced README for clarity on supported languages. - Refactored test cases for better maintainability and consistency in UI responses. - Added new utility functions and improved existing ones for better node handling in tests. - Fixed minor formatting issues in configuration files for consistency.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…a to window object
…error message formatting; update zoomToFit logic in Chat component to utilize a nodes map for improved performance.
…bject for improved data retrieval; clean up unnecessary whitespace in codeGraph.
…and package-lock.json
…and package-lock.json
…and package-lock.json
Naseem77
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isPath property not syncing, When user clicks "Show the path" between two nodes, the UI highlights the path visually, but the exposed data doesn't include isPath: true because the canvas doesn't know about this application state change.
Node drag positions not updating, When user drags a node to a new position, the canvas renders it in the new location, but the exposed data returns the old x/y coordinates because those values don't update after drag.
visible property not syncing, When user hides/unhides nodes via the element menu, the UI updates correctly, but the exposed data doesn't reflect the new visible: false/true state because the canvas filters out invisible nodes before receiving them and doesn't track visibility changes.
I just checked the Canvas tests only, and there are the bugs I listed above. check the rest of the tests
…latten node properties
…and package-lock.json
…p unused properties in Node interface
next,playwright, andsharpto their latest versions.PR Summary by Typo
Overview
This PR introduces a major refactoring of the graph visualization by migrating from
react-force-graph-2dto the new@falkordb/canvaslibrary. It also updates dependencies, refines API error handling, and enhances Playwright test configurations with sharding.Key Changes
react-force-graph-2dto@falkordb/canvas, introducing a newForceGraphcomponent.@falkordb/canvasand@radix-ui/react-switch.app/api/repo/route.tsfor more consistent responses.README.mdto reflect current language support (Java & Python) for code analysis.Switchcomponent.Work Breakdown
To turn off PR summary, please visit Notification settings.