Skip to content

chore: remove confirmed unused files - #6863

Merged
waleedlatif1 merged 1 commit into
stagingfrom
chore/remove-confirmed-unused-files
Aug 20, 2026
Merged

chore: remove confirmed unused files#6863
waleedlatif1 merged 1 commit into
stagingfrom
chore/remove-confirmed-unused-files

Conversation

@j15z

@j15z j15z commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

This removes maintenance surface that no longer participates in any shipped Sim entry point. A conservative reachability audit reduces React Doctor's deslop/unused-file findings from 119 to 31 while retaining every framework, generator, task, transient, intentionally parked, or otherwise ambiguous path.

The retained BuildCallout barrel is now the component's import boundary, which keeps the landing-specific directory convention without leaving another unused file.

Conservative scope

  • Deleted 86 confirmed-unreachable files (4,723 lines), including the retired landing preview and goo-mark subtrees plus isolated unused barrels, helpers, contracts, and types.
  • Kept 31 reported paths: parked animation/calendar work, Trigger.dev tasks, Docker/build/generator inputs, compile-time fixtures, an intentional manifest barrel, an ambiguous workflow operation, and transient upload artifacts.
  • Did not suppress or disable React Doctor rules.

Validation

  • bun run type-check — 24/24 workspaces pass.
  • bun run check:api-validation — passes.
  • bun run check:openapi — 3 test files and 103 tests pass; 7 specs and 141 contracts cross-checked.
  • bun run docs:check — generated integration docs are in sync.
  • bunx react-doctor@latest --verbosedeslop/unused-file is 31, down from 119; none of the deleted files remains in the report.
  • Structured code review completed (run_id: 20260819-131347-b863ea52). Its one convention finding was fixed and independently validated.

The local production build generated sandbox bundles successfully, but the Next.js optimized compile did not finish within ten minutes and was stopped without diagnostics. CI remains the complete production-build check.

Post-Deploy Monitoring & Validation

No runtime behavior or data path is expected to change. Use the normal deployment/build checks as the primary signal, and watch for missing-module or module-resolution errors during rollout. Revert this commit if a convention-based consumer not represented in the repository graph surfaces.


Compound Engineering

@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 19, 2026 9:12pm

Request Review

@cursor

cursor Bot commented Aug 19, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Pure deletion of unreachable modules with type-check and API validation passing; risk is only an undiscovered dynamic import outside the audit graph.

Overview
Removes ~86 unreachable files (~4,700 lines) identified by a conservative reachability audit, cutting React Doctor deslop/unused-file findings from 119 to 31 without changing shipped behavior.

The largest cut is the retired landing preview subtree (interactive workspace demo, chat pane, React Flow previews, sidebar, tables/files/knowledge/logs mocks, useLandingSubmit, and related workflow seed data). Also removed are Mothership goo-mark SVG components, unused barrel re-exports across workspace settings and workflow UI, Martian Mono font config, Zoho tool types/utils, plan-view / subscription-permissions hooks, MCP OAuth contracts, landing-content integration generator input, and other isolated helpers.

features.tsx now imports BuildCallout through its retained barrel (build-callout/index.ts) instead of a deeper path. Ambiguous, generator, Trigger.dev, and parked paths stay in the repo per the audit.

Reviewed by Cursor Bugbot for commit dc462d6. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes 86 files determined to be unreachable and redirects the landing page’s BuildCallout import through its retained barrel.

  • Removes the retired landing-preview and goo-mark component trees.
  • Removes unused workspace component barrels and isolated helpers, contracts, hooks, and types.
  • Preserves the landing feature component’s import boundary through the existing build-callout/index.ts.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete reachable build, runtime, data, or security regression identified.

The only retained code change resolves through an existing barrel that exports the same component, while the removed files have no established remaining consumers or shipped entry-point dependencies.

Important Files Changed

Filename Overview
apps/sim/app/(landing)/components/features/features.tsx Redirects BuildCallout through an existing barrel that re-exports the same named component without changing its runtime boundary.
apps/sim/app/(landing)/components/landing-preview/landing-preview.tsx Deletes the root of the retired landing-preview subtree along with its supporting components and hooks.
apps/sim/app/(landing)/components/mothership/components/goo-marks/index.ts Removes the unused goo-mark export boundary together with the complete implementation subtree.
apps/sim/app/api/v2/credentials/utils.ts Removes an isolated credential utility for which no reachable remaining consumer was established.
apps/sim/lib/api/contracts/mcp-oauth.ts Removes an unused MCP OAuth contract module without changing the active OAuth implementation.
apps/sim/executor/types/parallel.ts Removes an unused parallel-executor type file with no identified remaining import path.
packages/emcn/src/components/chip-switch/index.ts Removes an unused component-level barrel without changing the component implementation or public package root.

Reviews (1): Last reviewed commit: "chore: remove confirmed unused files" | Re-trigger Greptile

@j15z
j15z requested a review from waleedlatif1 August 20, 2026 01:15
@waleedlatif1
waleedlatif1 merged commit 8aab6d5 into staging Aug 20, 2026
30 checks passed
@waleedlatif1
waleedlatif1 deleted the chore/remove-confirmed-unused-files branch August 20, 2026 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants