Skip to content

2: Created the repositories section#9

Merged
abhiiiiiii-21 merged 2 commits into
mainfrom
Abhishek_Repositories
Oct 4, 2025
Merged

2: Created the repositories section#9
abhiiiiiii-21 merged 2 commits into
mainfrom
Abhishek_Repositories

Conversation

@abhiiiiiii-21
Copy link
Copy Markdown
Collaborator

@abhiiiiiii-21 abhiiiiiii-21 commented Oct 4, 2025

Summary by CodeRabbit

  • New Features

    • Repositories view with search and language filters.
    • Issues view with search and tag filters, including empty-state messaging.
    • Repository-specific issues page with back navigation.
    • Tabbed navigation to switch between Repositories and Issues.
    • Logout now shows a confirmation dialog.
    • Persistent sidebar layout with automatic active-state highlighting.
  • Chores

    • Added UI and auth dependencies to support dialogs, selects, and tabs.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 4, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Adds repository and issues views with search/filter UIs, introduces per-repository issue page, updates the main repositories page with tabbed navigation, integrates a sidebar with logout confirmation, wraps layout with SidebarProvider, and adds new UI primitives (Dialog, AlertDialog, Select, Tabs, Card). Updates package.json with Radix and Clerk dependencies.

Changes

Cohort / File(s) Summary
Repositories pages and components
app/main/Repositories/page.jsx, app/main/Repositories/_components/MainRepository.jsx, app/main/Repositories/_components/MainIssues.jsx
New tabbed Repositories page rendering repositories or issues with search and multi-select filters via Dialogs; animated tab indicator; static data-driven lists with derived filters.
Per-repository issues page
app/main/Repositories/[id]/page.jsx, app/main/Repositories/[id]/_components/RepoIsuue.jsx
New dynamic route rendering RepoIssue component showing searchable, tag-filterable open issues; includes back navigation and empty state. Note: import references “RepoIssue” and path “RepoIsuue.jsx”.
Layout and navigation
app/main/layout.jsx, app/main/_components/SidebarMain.jsx
Layout now uses SidebarProvider and persistent SidebarMain; Sidebar computes active item from pathname; adds logout confirmation via AlertDialog; removes manual active-state handlers.
UI primitives (Radix-based)
components/ui/alert-dialog.jsx, components/ui/dialog.jsx, components/ui/select.jsx, components/ui/tabs.jsx, components/ui/card.jsx
New reusable UI components wrapping Radix primitives with consistent slots, styling (via cn), and transitions; exports multiple building blocks (AlertDialog, Dialog, Select, Tabs, Card and subcomponents).
Dependencies
package.json
Adds dependencies: @clerk/nextjs, @radix-ui/react-alert-dialog, @radix-ui/react-select, @radix-ui/react-tabs.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant Page as /main/Repositories
  participant Tabs as Tabs UI
  participant MainRepo as MainRepository
  participant MainIssues as MainIssues
  Note over Page,Tabs: Page mounts with "repositories" as default tab
  User->>Tabs: Click "Issues" tab
  Tabs-->>Page: onValueChange("issues")
  alt Tab = repositories
    Page->>MainRepo: Render with {searchQuery, selectedLanguages}
  else Tab = issues
    Page->>MainIssues: Render with {searchQuery, selectedTags}
  end
Loading
sequenceDiagram
  autonumber
  actor User
  participant Issues as MainIssues/RepoIssue
  participant Dialog as Filter Dialog
  User->>Issues: Type in Search
  Issues-->>Issues: Recompute filteredIssues
  User->>Issues: Click "Filter"
  Issues->>Dialog: Open
  User->>Dialog: Toggle tags, Apply
  Dialog-->>Issues: Close with selectedTags
  Issues-->>Issues: Recompute filteredIssues
  alt No matches
    Issues->>User: Show "No issues found"
  else Matches
    Issues->>User: Render cards
  end
Loading
sequenceDiagram
  autonumber
  actor User
  participant Sidebar as SidebarMain
  participant AD as AlertDialog
  User->>Sidebar: Click "Logout"
  Sidebar->>AD: Open confirmation
  alt Confirm
    AD-->>Sidebar: onAction
    Sidebar->>Sidebar: Proceed with logout handler (external wiring)
  else Cancel
    AD-->>Sidebar: onCancel
    Sidebar-->>User: Remain signed in
  end
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

  • added clerk auth #1 — Also adds @clerk/nextjs in package.json; likely related to introducing Clerk integration used alongside these UI changes.

Poem

A hop through tabs, a leap through lists,
I nose the filters, twitching wrists.
Dialogs bloom like moonlit clover,
Click—confirm—the logout’s over.
Repos rustle, issues shine,
I stamp approval: ship the design! 🐇✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch Abhishek_Repositories

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1f1bc3f and f8b74b1.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (13)
  • app/main/Repositories/[id]/_components/RepoIsuue.jsx (1 hunks)
  • app/main/Repositories/[id]/page.jsx (1 hunks)
  • app/main/Repositories/_components/MainIssues.jsx (1 hunks)
  • app/main/Repositories/_components/MainRepository.jsx (1 hunks)
  • app/main/Repositories/page.jsx (1 hunks)
  • app/main/_components/SidebarMain.jsx (3 hunks)
  • app/main/layout.jsx (1 hunks)
  • components/ui/alert-dialog.jsx (1 hunks)
  • components/ui/card.jsx (1 hunks)
  • components/ui/dialog.jsx (1 hunks)
  • components/ui/select.jsx (1 hunks)
  • components/ui/tabs.jsx (1 hunks)
  • package.json (1 hunks)

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.

@abhiiiiiii-21 abhiiiiiii-21 merged commit 4d860ea into main Oct 4, 2025
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant