Skip to content

refactor: reduce IssueFinder cognitive complexity - #79

Merged
arnabnandy7 merged 2 commits into
mainfrom
bugfix/sonarIssue0.1
Sep 3, 2026
Merged

arnabnandy7 merged 2 commits into
mainfrom
bugfix/sonarIssue0.1

Conversation

@arnabnandy7

Copy link
Copy Markdown
Owner

Summary

Refactors the IssueFinder UI structure to reduce its cognitive complexity while preserving existing behavior.

Changes

  • Extracted the sidebar into a dedicated IssueFinderSidebar component.
  • Moved quick searches, supported labels, saved searches, digest controls, and account-specific cards out of the main component.
  • Replaced deeply nested conditional rendering with focused component props and callbacks.
  • Kept search, saved-search, digest, and authentication behavior unchanged.

Verification

  • All 33 focused IssueFinder tests pass.
  • ESLint passes.
  • Commit is signed and includes a sign-off.

Signed-off-by: Arnab Nandy <arnab_nandy7@yahoo.com>
@arnabnandy7 arnabnandy7 added this to the 1.2.0 milestone Sep 3, 2026
@arnabnandy7 arnabnandy7 self-assigned this Sep 3, 2026
@arnabnandy7 arnabnandy7 added the bug Something isn't working label Sep 3, 2026
@vercel

vercel Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
openissue-dev Ready Ready Preview Sep 3, 2026 3:34pm UTC

Signed-off-by: Arnab Nandy <arnab_nandy7@yahoo.com>
@sonarqubecloud

sonarqubecloud Bot commented Sep 3, 2026

Copy link
Copy Markdown

@arnabnandy7
arnabnandy7 merged commit 4d2ab20 into main Sep 3, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from To Do to Completed in Open issue portal development Sep 3, 2026

This branch was successfully deployed

1 active deployment
Preview — eea0895f Deployed Sep 3, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Development

Successfully merging this pull request may close these issues.

1 participant