[Auto-QA] Oversized source files detected

[clubanderson]

Auto-QA [Performance]: Large Source Files

Detected: 2026-09-07T13:33:52.807Z | Focus: Performance | Commit: c888992 | Run: View

Findings

### Source files over 500 lines (20 found)
Large files are harder to maintain and may impact compile performance:
  - `src/contexts/AlertsContext.lifecycle.test.tsx` (847 lines)
  - `src/lib/__tests__/kubectlProxy.core.connection.test.ts` (839 lines)
  - `src/hooks/mcp/__tests__/clusters.health.test.ts` (839 lines)
  - `src/hooks/mcp/__tests__/rbac.test.ts` (837 lines)
  - `src/components/cards/CardWrapper.tsx` (837 lines)
  - `src/lib/__tests__/analytics-emit-extended.test.ts` (835 lines)
  - `src/hooks/mcp/__tests__/events.test.ts` (834 lines)
  - `src/contexts/AlertsContext.deepcover.test.tsx` (834 lines)
  - `src/hooks/mcp/__tests__/workloads.extended.test.ts` (833 lines)
  - `src/components/cards/EnterpriseComplianceCards.tsx` (833 lines)
  - `src/hooks/__tests__/useResolutions.utils.resolution.test.ts` (829 lines)
  - `src/components/cards/console-missions/ConsoleOfflineDetectionCard.tsx` (829 lines)
  - `src/lib/unified/__tests__/registerHooks-hooks.test.ts` (824 lines)
  - `src/hooks/mcp/helm.ts` (824 lines)
  - `src/components/cards/__tests__/UserManagement.test.tsx` (820 lines)
  - `src/hooks/__tests__/useLocalClusterTools.crud.test.ts` (816 lines)
  - `src/lib/__tests__/sseClient-expand.test.ts` (811 lines)
  - `src/hooks/mcp/storage.ts` (811 lines)
  - `src/components/cards/TrestleScan.test.tsx` (808 lines)
  - `src/hooks/__tests__/useStackDiscovery.advanced.test.ts` (801 lines)

Suggested Improvements

• Split files over 500 lines into multiple focused modules
• Extract reusable hooks into hooks/ directory
• Move co-located sub-components into their own files under a component directory
• Large files slow TypeScript type-checking and IDE responsiveness

PR Guidance: Keep changes under 50 lines (size S) for best acceptance rate. Split larger fixes into multiple PRs.

---

This issue was automatically created by the Auto-QA workflow during Performance focus day.

[github-actions]

🐝 Hi @clubanderson! I'm kubestellar-hive[bot], an automation bot for this repo.

Trusted users — org members and contributors with write access — can mention @kubestellar-hive in a comment to trigger repo automation.
On issues, that mention queues an automated fix attempt. On pull requests, it records extra context for existing automation.
This is not an interactive Q&A bot, so mentions should be treated as requests for automation rather than a conversation.

Automation may take a moment to start, and follow-up happens through workflow activity rather than chat replies.

[hivecommons-hive]

Triage: PARTIALLY COVERED (leave open; created child for uncovered file)

Issue #23247 lists 20 oversized source files. Of the 5 source components (rest are test files), coverage:

Already covered by child issues:

• ✅ src/components/cards/CardWrapper.tsx (837 lines) → [Auto-QA] Split CardWrapper.tsx into container + presentational components #22966 (child of [Auto-QA] High-complexity components could be split #22955)
• ✅ src/components/cards/EnterpriseComplianceCards.tsx (833 lines) → [scanner] Split EnterpriseComplianceCards.tsx (833 lines, 28 card components) into domain-grouped modules #23156
• ✅ src/hooks/mcp/helm.ts (824 lines) → [scanner] Split web/src/hooks/mcp/helm.ts (824) and storage.ts (811) — one hook per file with barrel re-exports #23155
• ✅ src/hooks/mcp/storage.ts (811 lines) → [scanner] Split web/src/hooks/mcp/helm.ts (824) and storage.ts (811) — one hook per file with barrel re-exports #23155

Newly created child:

• 🆕 src/components/cards/console-missions/ConsoleOfflineDetectionCard.tsx (829 lines) → new child issue (to be assigned)

Note on test files:
15 test files (847–801 lines) listed but not being split in this initiative. Auto-QA flags large test files, but refactoring test suites is lower-priority than refactoring source components. May be revisited separately per project priorities.

Recommendation:

• Assign newly created child issue when ready
• Keep [Auto-QA] Oversized source files detected #23247 open as a tracking issue for all 20 flagged files
• Test-file refactoring can be deferred to a separate initiative if prioritized later

— Triaged by @clubanderson via scanner

🐝 Hive Agent: scanner | Instance: hosted-kubestellar-console-4vkt | SHA: 1f8aabca41

— hive: agent=scanner backend=copilot model=claude-fable-5 copilot=1.0.78

[hivecommons-hive]

Scanner triage: status re-check against origin/main

Verified all 20 flagged files (5 source components + 15 test files) against a fresh origin/main worktree.

Source components (5) — status:

File	Original	Current	Status
src/components/cards/CardWrapper.tsx	837	688	Reduced via merged PR #23466 (child #22966/#23118, tracked under #22955/#22959). Still >500; no open follow-up currently tracks the remainder.
src/components/cards/EnterpriseComplianceCards.tsx	833	39	✅ Fully resolved — split into domain-grouped modules via merged PR #23496, closing child #23156.
src/hooks/mcp/helm.ts	824	824	In progress — child #23155, open PR #23475.
src/components/cards/console-missions/ConsoleOfflineDetectionCard.tsx	829	829	Tracked — child #23486 (open, no PR yet).
src/hooks/mcp/storage.ts	811	811	In progress — same child #23155 / PR #23475 as helm.ts.

Test files (15, 801–847 lines): per this issue's own earlier triage comment, test-file refactoring was explicitly deferred as lower priority than source-component splits and is not currently tracked by child issues. Leaving that decision in place — no new issues filed for test files.

Summary: 1 of 5 source files fully resolved (EnterpriseComplianceCards.tsx). The remaining 4 are all covered by existing open child issues/PRs (#23155/#23475, #23486, and CardWrapper's partial progress under #22955/#22959). No net-new child issues needed here beyond what's already tracked. Keeping #23247 open until #23155's PR merges and #23486 gets a fix.

🐝 Hive Agent: scanner | Instance: hosted-kubestellar-console-4vkt | SHA: 1f8aabca41

— hive: agent=scanner backend=copilot model=claude-fable-5 copilot=1.0.78

[hivecommons-hive]

Scanner re-triage against origin/main (2026-09-17)

Re-verified all 5 flagged source components (the other 15 are test files, previously deprioritized — see prior triage comment):

File	Original	Current	Status
src/components/cards/EnterpriseComplianceCards.tsx	833	39	✅ Done — merged PR #23496, closed #23156
src/components/cards/CardWrapper.tsx	837	688	Reduced via merged PR #23466 (closed #22966/#23118), still over target. New child filed: #23505
src/hooks/mcp/helm.ts	824	824	In progress — child #23155, open PR #23475
src/hooks/mcp/storage.ts	811	811	In progress — same child #23155 / PR #23475
src/components/cards/console-missions/ConsoleOfflineDetectionCard.tsx	829	829	Tracked — child #23486, open, no PR yet

Test files (15, 801–847 lines): still explicitly deprioritized per earlier triage decision on this issue — no new issues filed for these.

Summary: 1 of 5 source files fully resolved. The remaining 4 are all covered by existing/newly-filed child issues (#23155/PR #23475, #23486, #23505). No further net-new issues needed.

Recommendation: keep #23247 open until #23155's PR merges, #23486 gets a fix, and #23505 (CardWrapper part 2) is addressed.

— Triaged by scanner agent.

🐝 Hive Agent: scanner | Instance: hosted-kubestellar-console-4vkt | SHA: 1f8aabca41

— hive: agent=scanner backend=copilot model=claude-fable-5 copilot=1.0.78