chore(cleanup): Wave 1 — remove 20 orphaned files in mission-control#680
Open
usamaB wants to merge 2 commits into
Open
chore(cleanup): Wave 1 — remove 20 orphaned files in mission-control#680usamaB wants to merge 2 commits into
usamaB wants to merge 2 commits into
Conversation
- Add compose-level healthcheck using node healthcheck.js (hits /api/status?action=health, same as Dockerfile HEALTHCHECK) - Document docker-compose.hardened.yml overlay differences as comments in the base docker-compose.yml for discoverability - Add .env.production to .gitignore Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Deletes 20 grep-verified zero-reference dead files identified in the Wave 1 audit. All files confirmed no importers before deletion. Files deleted: - src/nav-rail.tsx (duplicate; live one is src/components/layout/nav-rail.tsx) - src/styles/design-tokens.ts - src/plugins/hyperbrowser-example.ts - update-db-port.js (one-shot DB patch, hardcoded port, untracked) - src/components/layout/promo-banner.tsx - src/components/ui/online-status.tsx - src/components/panels/documents-panel.tsx - src/components/panels/token-dashboard-panel.tsx - src/lib/plugin-loader.ts (all init was commented out) - src/components/dashboard/agent-network.tsx - src/components/ui/agent-core-node.tsx (only imported by agent-network, co-deleted) - src/components/dashboard/sessions-list.tsx - src/components/dashboard/sidebar.tsx - src/components/dashboard/stats-grid.tsx - src/components/hud/connection-status.tsx - src/components/panels/agent-cost-panel.tsx - src/components/panels/agent-history-panel.tsx - src/components/panels/session-details-panel.tsx - ops/mc-provisioner-daemon.js - scripts/smoke-staging.mjs Dep cleanup (chained from agent-network + agent-core-node deletion): - Removed @xyflow/react and reactflow from package.json (zero remaining importers) - Refreshed pnpm-lock.yaml Config cleanup: - Removed stale src/lib/plugin-loader.ts entry from vitest.config.ts coverage exclude list Validation: tsc --noEmit ✓ | vitest run ✓ (1 pre-existing better-sqlite3 native binding failure, unrelated) | next build ✓ Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
usamaB
force-pushed
the
chore/wave1-dead-code-openclaw
branch
from
de979ce to
46328a4
Compare
Author
|
QA Validation — PASS (ready to merge) Branch rebased on main, conflicts resolved (package.json + pnpm-lock.yaml had @xyflow/react vs xterm additions in main — kept xterm, dropped xyflow as intended). Fix applied during rebase: Validation results:
Branch is pushed and GitHub shows MERGEABLE. Needs a maintainer with merge permission to squash-merge. |
Author
|
QA verified PR is mergeable. Validation passed. Awaiting builderz-labs maintainer merge — cannot merge from contributor account. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Wave 1 dead-code cleanup. Removes 20 grep-verified zero-reference orphaned files from the mission-control package. All files were confirmed to have zero importers before deletion.
Files deleted (20)
src/nav-rail.tsx— duplicate ofsrc/components/layout/nav-rail.tsx(live one); the only importer (page.tsx) uses@/components/layout/nav-railsrc/styles/design-tokens.tssrc/plugins/hyperbrowser-example.tsupdate-db-port.js— one-shot DB patch script with hardcoded port (was untracked)src/components/layout/promo-banner.tsxsrc/components/ui/online-status.tsxsrc/components/panels/documents-panel.tsxsrc/components/panels/token-dashboard-panel.tsxsrc/lib/plugin-loader.ts— all init was commented outsrc/components/dashboard/agent-network.tsxsrc/components/ui/agent-core-node.tsx— only imported byagent-network.tsx(co-deleted)src/components/dashboard/sessions-list.tsxsrc/components/dashboard/sidebar.tsxsrc/components/dashboard/stats-grid.tsxsrc/components/hud/connection-status.tsxsrc/components/panels/agent-cost-panel.tsxsrc/components/panels/agent-history-panel.tsxsrc/components/panels/session-details-panel.tsxops/mc-provisioner-daemon.jsscripts/smoke-staging.mjsDependency cleanup (chained from agent-network + agent-core-node removal)
After deleting
agent-network.tsxandagent-core-node.tsx, grep confirmed zero remaining importers of@xyflow/reactandreactflow. Both packages removed frompackage.jsonandpnpm-lock.yamlrefreshed.Config cleanup
Removed stale
src/lib/plugin-loader.tsentry from thevitest.config.tscoverage exclude list.Validation results
tsc --noEmit— exit 0, zero type errorsvitest run— 712/713 tests pass; 1 pre-existing failure insession-transcript.test.tsdue tobetter-sqlite3native bindings not compiled (unrelated to this PR, caused byERR_PNPM_IGNORED_BUILDSfor native modules)next build— succeeded, zero errorsDeferred to follow-up PR
~90 unused-exports and ~73 unused-types findings (MEDIUM confidence, inside live files) are deferred to a separate follow-up PR to keep this one reviewable.