[scanner] fix: address oversized source files - #22577
[scanner] fix: address oversized source files#22577kubestellar-hive[bot] wants to merge 1 commit into
Conversation
✅ Deploy Preview for kubestellarconsole ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 Hey @kubestellar-hive[bot] — thanks for opening this PR!
This is an automated message. |
Auto Test GeneratorThe following new files have no corresponding test file:
Please add tests or apply the |
|
| New file | Suggested test location |
|---|---|
web/src/components/cards/Missions.helpers.tsx |
web/src/components/cards/__tests__/Missions.helpers.test.tsx |
web/src/components/cards/StockMarketTicker.data.ts |
web/src/components/cards/__tests__/StockMarketTicker.data.test.tsx |
web/src/components/cards/StockMarketTicker.helpers.tsx |
web/src/components/cards/__tests__/StockMarketTicker.helpers.test.tsx |
web/src/components/cards/rss/RSSFeed.fetch.ts |
web/src/components/cards/rss/__tests__/RSSFeed.fetch.test.tsx |
web/src/components/cards/rss/RSSFeed.hooks.ts |
web/src/components/cards/rss/__tests__/RSSFeed.hooks.test.tsx |
web/src/components/cards/rss/RSSFeed.state.ts |
web/src/components/cards/rss/__tests__/RSSFeed.state.test.tsx |
web/src/components/mission-control/FlightPlanBlueprint/BlueprintCanvas.tsx |
web/src/components/mission-control/FlightPlanBlueprint/__tests__/BlueprintCanvas.test.tsx |
web/src/components/mission-control/FlightPlanBlueprint/BlueprintInfoSidebar.tsx |
web/src/components/mission-control/FlightPlanBlueprint/__tests__/BlueprintInfoSidebar.test.tsx |
web/src/components/mission-control/FlightPlanBlueprint/BlueprintToolbar.tsx |
web/src/components/mission-control/FlightPlanBlueprint/__tests__/BlueprintToolbar.test.tsx |
web/src/components/mission-control/FlightPlanBlueprint/FlightPlanBlueprint.helpers.ts |
web/src/components/mission-control/FlightPlanBlueprint/__tests__/FlightPlanBlueprint.helpers.test.tsx |
web/src/components/mission-control/FlightPlanBlueprint/MissionPreviewModal.tsx |
web/src/components/mission-control/FlightPlanBlueprint/__tests__/MissionPreviewModal.test.tsx |
Checked against origin/main. Remove the needs-tests label once tests are added.
♿ Accessibility Audit (WCAG 2.1 AA)✅ No WCAG 2.1 AA violations detected in audited routes. Powered by axe-core. Target: WCAG 2.1 AA compliance. |
| const mockFetchLLMdModels = vi.fn() | ||
|
|
||
| vi.mock('../../lib/cache', () => ({ | ||
| createCachedHook: vi.fn(), |
| const mockClusterCacheRef = vi.hoisted(() => ({ clusters: [] as Array<{ name: string; context?: string; reachable?: boolean }> })) | ||
|
|
||
| vi.mock('../../lib/cache', () => ({ | ||
| createCachedHook: vi.fn(), |
| import React from 'react' | ||
| import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' | ||
| import { render, screen, waitFor, act } from '@testing-library/react' | ||
| import userEvent from '@testing-library/user-event' |
| import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' | ||
| import { render, screen, waitFor, act } from '@testing-library/react' | ||
| import userEvent from '@testing-library/user-event' | ||
| import { DynamicCard, Tier1CardRuntime, Tier2CardRuntime } from '../DynamicCard' |
| import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' | ||
| import { render, screen, waitFor, act } from '@testing-library/react' | ||
| import userEvent from '@testing-library/user-event' | ||
| import { DynamicCard, Tier1CardRuntime, Tier2CardRuntime } from '../DynamicCard' |
| import React from 'react' | ||
| import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' | ||
| import { render, screen, waitFor, act } from '@testing-library/react' | ||
| import userEvent from '@testing-library/user-event' |
| import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' | ||
| import { render, screen, waitFor, act } from '@testing-library/react' | ||
| import userEvent from '@testing-library/user-event' | ||
| import { DynamicCard, Tier1CardRuntime, Tier2CardRuntime } from '../DynamicCard' |
| } as DynamicCardDefinition | ||
| } | ||
|
|
||
| function makeT2Definition(overrides: Partial<DynamicCardDefinition> = {}): DynamicCardDefinition { |
| // Default useCardData return value | ||
| // --------------------------------------------------------------------------- | ||
|
|
||
| function makeUseCardDataReturn(items: Record<string, unknown>[] = []) { |
| emptyMessage: 'Nothing here.', | ||
| } | ||
|
|
||
| function makeT1Definition(overrides: Partial<DynamicCardDefinition> = {}): DynamicCardDefinition { |
Split 20 oversized source files into smaller, focused modules: - Test files split by describe blocks into per-module files - Component files split into sub-components and utilities - All files now under 500 lines for better maintainability Reduces TypeScript compile time and IDE responsiveness. Fixes #22551 Signed-off-by: kubestellar-hive[bot] <bot@kubestellar.io>
342503b to
045df2d
Compare
|
🐝 Hive Agent: |
🛑 Fix loop escalated — human attention neededThis PR has failed CI on 1 distinct fix attempts (new commits, still red). The hive has stopped dispatching further automated fixes for it. Failing checks: All Cards TTFI (Hard Gate), App Visual Regression, Build Frontend, Lint warning regression check, build (linux/amd64, ubuntu-24.04, amd64), build (linux/arm64, ubuntu-24.04-arm, arm64), build-gate, fullstack-smoke, pr-check Raw failure evidence (from check-run annotations): Remove the |
🛑 Fix loop escalated — human attention neededThis PR has failed CI on 1 distinct fix attempts (new commits, still red). The hive has stopped dispatching further automated fixes for it. Failing checks: All Cards TTFI (Hard Gate), App Visual Regression, Build Frontend, Lint warning regression check, build (linux/amd64, ubuntu-24.04, amd64), build (linux/arm64, ubuntu-24.04-arm, arm64), build-gate, fullstack-smoke, pr-check Raw failure evidence (from check-run annotations): Remove the |
3 similar comments
🛑 Fix loop escalated — human attention neededThis PR has failed CI on 1 distinct fix attempts (new commits, still red). The hive has stopped dispatching further automated fixes for it. Failing checks: All Cards TTFI (Hard Gate), App Visual Regression, Build Frontend, Lint warning regression check, build (linux/amd64, ubuntu-24.04, amd64), build (linux/arm64, ubuntu-24.04-arm, arm64), build-gate, fullstack-smoke, pr-check Raw failure evidence (from check-run annotations): Remove the |
🛑 Fix loop escalated — human attention neededThis PR has failed CI on 1 distinct fix attempts (new commits, still red). The hive has stopped dispatching further automated fixes for it. Failing checks: All Cards TTFI (Hard Gate), App Visual Regression, Build Frontend, Lint warning regression check, build (linux/amd64, ubuntu-24.04, amd64), build (linux/arm64, ubuntu-24.04-arm, arm64), build-gate, fullstack-smoke, pr-check Raw failure evidence (from check-run annotations): Remove the |
🛑 Fix loop escalated — human attention neededThis PR has failed CI on 1 distinct fix attempts (new commits, still red). The hive has stopped dispatching further automated fixes for it. Failing checks: All Cards TTFI (Hard Gate), App Visual Regression, Build Frontend, Lint warning regression check, build (linux/amd64, ubuntu-24.04, amd64), build (linux/arm64, ubuntu-24.04-arm, arm64), build-gate, fullstack-smoke, pr-check Raw failure evidence (from check-run annotations): Remove the |
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Fixes #22551
Splits 20 oversized source files into smaller, focused modules for better maintainability and compilation performance. All files now under 500 lines.
— hive: agent=scanner backend=copilot model=auto