Remove dead/commented-out code in web/src

[Copilot]

Multiple files in web/src contained commented-out code blocks, a duplicate import causing a TypeScript error, and a require()-style import flagged by ESLint.

Fixes

TypeScript / ESLint errors

• SpeedDisplay.tsx: Remove duplicate import log from "loglevel" (TS2300)
• JobItem.test.tsx: Replace require("react-router-dom") with a proper ES import (no-require-imports)

Multi-line commented-out code blocks removed

• AssetGridContent.tsx — dead useEffect block
• modelDownloadCheck.ts — dead cached-files check block + orphaned hfModels?.find(...) expression; also removed the now-unused hfModels parameter from the function signature
• GlobalChatStore.test.ts — dead mockServer.stop() block
• numberInputStyles.ts, Welcome.styles.ts, NodeSwitch.tsx, ThemeNodetool.tsx, PlaceholderNode.tsx — dead CSS/style blocks

Single-line commented-out code removed

• useDynamicProperty.ts, workflowUpdates.ts, useFitView.ts, FavoriteStar.tsx, settingsMenuStyles.ts, EditableInput.tsx, NumberInput.tsx

Unused variables

• ChatMarkdown.tsx, CodeBlock.tsx, PreRenderer.tsx: Prefix unused node destructure params with _node (required by ReactMarkdown's renderer interface to prevent inadvertent DOM spreading)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Remove all dead/commented-out code blocks in web/src</issue_title>
<issue_description>## Problem
Multiple files contain commented-out code blocks, unused imports, unused variables, and dead code paths that add noise and make the codebase harder to maintain.

What to do

1. Remove all multi-line commented-out code blocks (not explanatory comments, just dead code)
2. Remove unused imports flagged by TypeScript/ESLint
3. Remove unused variables (prefix with _ if they're required by an interface/callback signature)
4. Remove empty useEffect hooks that do nothing
5. Remove duplicate property definitions

Scope

All files in web/src/. Do NOT remove:

• JSDoc comments
• TODO/FIXME comments
• Single-line explanatory comments
• Code that is commented out with a clear "TODO: re-enable when X" note</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Remove all dead/commented-out code blocks in web/src #2127

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[cloudflare-workers-and-pages]

Deploying nodetool-staging with    Cloudflare Pages

Latest commit:	a59a12d
Status:	⚡️  Build in progress...

View logs

[cloudflare-workers-and-pages]

Deploying nodetool with    Cloudflare Pages

Latest commit:	a59a12d
Status:	✅  Deploy successful!
Preview URL:	https://d01ba389.nodetool.pages.dev
Branch Preview URL:	https://copilot-remove-dead-commente.nodetool.pages.dev

View logs

[georgi]

Review: Clean and well-scoped PR. Changes look correct:

• Removes duplicate import log in SpeedDisplay.tsx
• Fixes require() → ES import in JobItem.test.tsx
• Removes ~15 multi-line commented-out code blocks across styles, hooks, stores
• Removes orphaned hfModels?.find() expression and unused hfModels parameter from modelDownloadCheck.ts
• Properly prefixes unused node params with _node in markdown renderers

One note: the JobItem.test.tsx import fix also appears in other PRs (#2219 merged). Please rebase on latest main to resolve any conflicts.

[georgi]

Unable to rebase this branch automatically — it has merge conflicts with main (likely in SpeedDisplay.tsx and JobItem.test.tsx which were already fixed by other merged PRs). I don't have push access to Copilot-owned branches. @copilot or @georgi please rebase this branch on latest main.

[georgi]

Superseded by #2238 which is rebased onto current main.