Skip to content

fix: update top repos selection to include all repositories#96

Merged
rahul-vyas-dev merged 1 commit into
AOSSIE-Org:mainfrom
rahul-vyas-dev:fix/network-page
Jul 3, 2026
Merged

fix: update top repos selection to include all repositories#96
rahul-vyas-dev merged 1 commit into
AOSSIE-Org:mainfrom
rahul-vyas-dev:fix/network-page

Conversation

@rahul-vyas-dev

@rahul-vyas-dev rahul-vyas-dev commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Screenshots/Recordings:

Before: -
Screenshot_3-7-2026_18196_localhost

After: -
Screenshot_3-7-2026_181932_localhost

Additional Notes:

Fix network page to include all the repos when PAT is available.

Checklist

  • My code follows the project's code style and conventions
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contributing Guidelines

⚠️ AI Notice - Important!

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact.

Summary by CodeRabbit

  • Bug Fixes
    • The network visualization now includes the full repository list instead of only a limited subset, so the displayed connections and nodes better reflect the complete dataset.

@github-actions github-actions Bot added no-issue-linked PR has no linked issue frontend Frontend changes javascript JavaScript/TypeScript changes labels Jul 3, 2026
@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The topRepos variable in NetworkPage.jsx now references the full model.allRepos array instead of a truncated slice of the first 30 repositories, changing the set of nodes and links used to render the D3 network visualization.

Changes

Network data source update

Layer / File(s) Summary
Use full repo list for network graph
src/pages/NetworkPage.jsx
topRepos now references model.allRepos directly instead of model.allRepos.slice(0, 30), changing the repo nodes and links rendered in the network visualization.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested labels: Typescript Lang

Suggested reviewers: Zahnentferner, bhavik-mangla

Poem

One line changed, a slice let go,
Now every repo joins the show,
Nodes and links in fuller bloom,
A rabbit hops with graphing room,
Thump-thump-thump, the network grows! 🐇🕸️

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: expanding the repo selection to include all repositories.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions github-actions Bot added size/XS 1-10 lines changed external-contributor External contributor and removed size/XS 1-10 lines changed labels Jul 3, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/pages/NetworkPage.jsx`:
- Around line 29-31: The NetworkPage data selection is still passing unbounded
model.allRepos and model.contributors into the graph, which can make the
SVG/force simulation too expensive for large orgs. Update the
top-repo/top-contributor handling in NetworkPage to cap or slice the datasets
before they reach the renderer, and make sure the nearby comment reflects the
new bounded behavior instead of “Top repos and contributors for performance.”
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 91718099-1b03-49cd-9f50-17680195707a

📥 Commits

Reviewing files that changed from the base of the PR and between a91f8a4 and 92cc501.

📒 Files selected for processing (1)
  • src/pages/NetworkPage.jsx

Comment thread src/pages/NetworkPage.jsx
@rahul-vyas-dev rahul-vyas-dev merged commit 89fe60a into AOSSIE-Org:main Jul 3, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external-contributor External contributor frontend Frontend changes javascript JavaScript/TypeScript changes no-issue-linked PR has no linked issue size/XS 1-10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant