Skip to content

Conversation

@Anchel123
Copy link
Contributor

@Anchel123 Anchel123 commented Dec 10, 2025

  • 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.

PR Summary by Typo

Overview

This PR introduces a major refactoring of the graph visualization by migrating from react-force-graph-2d to the new @falkordb/canvas library. It also updates dependencies, refines API error handling, and enhances Playwright test configurations with sharding.

Key Changes

  • Migrated graph visualization from react-force-graph-2d to @falkordb/canvas, introducing a new ForceGraph component.
  • Updated dependencies, including adding @falkordb/canvas and @radix-ui/react-switch.
  • Enhanced API error handling in app/api/repo/route.ts for more consistent responses.
  • Implemented Playwright test sharding and updated E2E tests to accommodate the new graph component.
  • Updated README.md to reflect current language support (Java & Python) for code analysis.
  • Introduced a new Radix UI Switch component.

Work Breakdown

Category Lines Changed
New Work 241 (37.8%)
Churn 268 (42.0%)
Refactor 129 (20.2%)
Total Changes 638
To turn off PR summary, please visit Notification settings.

AviAvni and others added 21 commits July 8, 2025 02:12
Replaced the multiple H1 tags with H2 tags, keeping 'CODE GRAPH' as sole H1 tag
- 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.
@Anchel123 Anchel123 requested a review from Naseem77 December 10, 2025 11:09
@vercel
Copy link

vercel bot commented Dec 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
code-graph Ready Ready Preview, Comment Feb 3, 2026 9:11am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 10, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch merge-main-to-staging

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

…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.
Copy link
Contributor

@Naseem77 Naseem77 left a comment

Choose a reason for hiding this comment

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

@Anchel123

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants