Skip to content

(janitor/dead-code): remove orphaned cloud-agent-next demo-session components - #6880

Merged
kilo-code-bot[bot] merged 1 commit into
mainfrom
janitor/dead-code/cloud-agent-next-demo
Sep 29, 2026
Merged

kilo-code-bot[bot] merged 1 commit into
mainfrom
janitor/dead-code/cloud-agent-next-demo

Conversation

@kilo-code-bot

@kilo-code-bot kilo-code-bot Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

What was deleted

Three files from apps/web/src/components/cloud-agent-next/:

  • DemoSessionModal.tsx
  • DemoSessionCTA.tsx
  • demo-config.ts

These form the leftover "demo session" UI. The two components are the only importers of the module, and nothing imports either component, so the whole cluster is unreachable.

Evidence it was unreachable

  • DemoSessionModal and DemoSessionCTA appear only in their own definition files. A repo-wide search (all file types, excluding node_modules) finds no import, re-export, or other reference.
  • demo-config.ts is imported only by those two components (import { DEMO_SOURCE_REPO, type DemoConfig } from './demo-config'). Its other exports (DEMO_CONFIGS, templatePrompt) have zero references anywhere, including within the dead components.
  • The demo-session feature has no feature flag, config, or route wiring: searching for demo session/demoSession/DEMO_CONFIGS/templatePrompt yields only the deleted files (the unrelated sales-demo-sort.ts matches "sales demo", not session demo).
  • A separate live apps/web/src/components/cloud-agent/demo-config.ts still exports DEMO_SOURCE_OWNER/DEMO_SOURCE_REPO_NAME and remains used by lib/cloud-agent/github-integration-helpers.ts; it is untouched.

Dynamic and configured entry points checked

  • Not a Next.js convention file (not page/layout/route/loading/error/not-found/instrumentation); plain components in components/.
  • No barrel/index file in cloud-agent-next/ re-exports them (export *), and no parent barrel references them.
  • No dynamic import() or require() resolves to them; no string-based module construction references the filenames.
  • Not referenced by tests (the tool-card parity test covers TodoWriteToolCard, not the demo components), Storybook stories, MDX, scripts, CI workflows, or Wrangler/Cloudflare config.
  • git log --follow shows these files date to the initial "Cloud Agent Next" commits and have only been moved since; they were never wired into the current rendering path.

Contract and compatibility considerations

  • All three files are internal to apps/web and not exported from any published package or SDK. No public or package contract is affected.
  • No persisted data, migration, or rollback depends on them. No module-level side effects (the components only render UI; the config only declares constants/types).
  • Deletion changes no production behavior and no public surface.

Validation performed

  • git diff --check — clean.
  • tsgo --noEmit -p apps/web/tsconfig.json — passes (exit 0).
  • oxlint --config .oxlintrc.json apps/web/src/components/cloud-agent-next — 0 warnings, 0 errors.

Not run (resource-limited sandbox, no Postgres/Redis): full repo pnpm validate, Jest unit suites, and any DB-dependent tests.

Delete DemoSessionModal, DemoSessionCTA, and their demo-config module. The
config is imported only by these two components, and neither component is
imported, rendered, or referenced anywhere else in the repo, so the whole
cluster is unreachable from every entry point.
@kilo-code-bot kilo-code-bot Bot added the janitor Automated dead-code/duplication cleanup label Sep 29, 2026
@kilo-code-bot
kilo-code-bot Bot enabled auto-merge (squash) September 29, 2026 07:50
@kilo-code-bot

kilo-code-bot Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Verified deletion of three unreachable cloud-agent-next demo-session files; no importers, barrel re-exports, dynamic imports, or type consumers remain, and the live cloud-agent/demo-config.ts is distinct and untouched.

Files Reviewed (3 files)
  • apps/web/src/components/cloud-agent-next/DemoSessionCTA.tsx (deleted)
  • apps/web/src/components/cloud-agent-next/DemoSessionModal.tsx (deleted)
  • apps/web/src/components/cloud-agent-next/demo-config.ts (deleted)

Verification notes: repo-wide search finds no reference to DemoSessionModal, DemoSessionCTA, DEMO_CONFIGS, templatePrompt, DemoConfig, or cloud-agent-next/demo-config; the only demo-config importers resolve to the separate, still-present @/components/cloud-agent/demo-config. Deletion-only change, so no RIGHT-side lines exist to anchor inline comments.


Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0

Review guidance: REVIEW.md from base branch main

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

Labels

janitor Automated dead-code/duplication cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant