Skip to content

feat(cleanup): A29 sweep orphaned Svelte components#226

Open
KooshaPari wants to merge 2 commits into
mainfrom
a29-sweep-orphaned-svelte
Open

feat(cleanup): A29 sweep orphaned Svelte components#226
KooshaPari wants to merge 2 commits into
mainfrom
a29-sweep-orphaned-svelte

Conversation

@KooshaPari

Copy link
Copy Markdown
Owner

A29 Sweep — Orphaned Components & Artifact Cleanup

@codeant-ai

codeant-ai Bot commented Jun 25, 2026

Copy link
Copy Markdown

Skipping CodeAnt AI review — this PR changes more than 100 files, which usually means a migration, codemod, or vendored drop. Line-level review on diffs this large produces duplicate findings on the same rewrite pattern and drowns out anything that actually matters.

If you still want a review, comment @codeant-ai : review. For better signal, consider splitting the PR into smaller chunks.

@github-actions

Copy link
Copy Markdown

Legacy Tooling Scan Report

Severity Count
Critical 0
High 0
Medium 0
Low 0

No violations detected.

This is a WARN-mode scan. Fix before strict enforcement begins.

@sonarqubecloud

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 184630e2c4

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@@ -1,44 +0,0 @@
<script lang="ts">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Restore project dialog components still in use

In the current tree, frontend/web/src/components/addProjectDialog.svelte:5 and frontend/web/src/components/projectPopup.svelte:9 still import ./projectForm.svelte, and /home imports both of those components at frontend/web/src/routes/home/+page.svelte:11-12. With this deletion, SvelteKit/Vite cannot resolve the module when building or loading /home; restore the component or update those imports, along with the companion gitSearch/reviewCard imports deleted in the same sweep.

Useful? React with 👍 / 👎.

@KooshaPari KooshaPari added the area:compute-infra Phenotype compute/infra epic label Jun 25, 2026

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request cleans up the repository by removing a large number of local history files. The reviewer recommends adding the .history/ directory to the project's .gitignore file to prevent these IDE-generated files from being committed in the future.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@@ -1,3 +0,0 @@
@import 'tailwindcss/base';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

It appears the .history directory, which is typically generated by IDE extensions like VS Code's Local History, has been committed to the repository. While this PR correctly removes these files, to prevent this from happening in the future, please add the .history/ directory to your project's .gitignore file.

@kilo-code-bot

kilo-code-bot Bot commented Jun 26, 2026

Copy link
Copy Markdown

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 1
WARNING 0
SUGGESTION 0
Issue Details (click to expand)

CRITICAL

File Issue
frontend/web/src/components/addProjectDialog.svelte Imports deleted modules: ./projectForm.svelte, ./reviewCard.svelte, ./gitSearch.svelte - will cause module resolution errors at build/runtime
frontend/web/src/components/projectPopup.svelte Imports deleted modules: ./projectForm.svelte, ./reviewCard.svelte, ./gitSearch.svelte - will cause module resolution errors at build/runtime

Note: These files were NOT modified in this PR. This is a latent issue exposed by the cleanup that was not addressed. The orphaned components were deleted but their consumers were not updated.

Files Reviewed (2796 files)
  • frontend/web/src/components/addProjectDialog.svelte - 1 issue
  • frontend/web/src/components/projectPopup.svelte - 1 issue
  • Various deleted .history/ and .stories/ files - cleanup validated

Fix these issues in Kilo Cloud


Reviewed by laguna-m.1-20260312:free · Input: 2.9M · Output: 14.1K · Cached: 1.1M

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:compute-infra Phenotype compute/infra epic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant