Remove dead skills-related code left after PR #2240 gutted the skills
UI subsystem: 6 orphaned skill handler method types from AppViewState,
and the never-populated debugModels state field + its "Models" card in
the debug view.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

)

Remove dead skills-related code left after PR #2240 gutted the skills
UI subsystem: 6 orphaned skill handler method types from AppViewState,
and the never-populated debugModels state field + its "Models" card in
the debug view.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…ead control after #2240/#2245 (#2520)

The Tools tab in the Agents view rendered in the tab bar but its panel
body was empty (placeholder `${nothing /* tools panel removed */}`).
Clicking Tools showed a highlighted tab with no content — only
user-visible dead control remaining from the post-#2336 audit sweep.

Changes:
- ui/src/ui/views/agents.ts: drop "tools" from `AgentsPanel` type
  union; delete `{ id: "tools", label: "Tools" }` tab entry; delete
  empty placeholder.
- ui/src/ui/app.ts, app-settings.ts, app-view-state.ts: narrow each
  `agentsPanel` literal union to drop "tools" (the only consumer was
  the deleted tab).
- ui/src/ui/app-render.ts: delete two now-impossible
  `panel === "tools"` blocks that lazy-loaded the tools catalog on
  tab activation. Eager `loadToolsCatalog` callers (gateway connect,
  settings apply, refresh handler) preserved — `toolsCatalog*` state
  is still live and asserted by the smoke suite.

Out-of-scope (deferred): orphan `onToolsProfileChange` /
`onToolsOverridesChange` props/handlers in `views/agents.ts` and
`app-render.ts` — distinct dead-code pattern from the visible tab,
warrants its own gut wave. `"skills"` literal in the type union also
preserved (no consumer, but AC silent).

AC verification:
- `grep -rn '"tools"' ui/src/ui/views/agents.ts` → no matches
- `grep -rn 'id: "tools"' ui/src/` → no matches
- `pnpm check` green (oxfmt, tsgo, oxlint, fork-integrity gates)
- `pnpm test` green (7014 + 288 tests)
- `pnpm test:ui:smoke` green (12 tests, browser-mode Chromium)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…ead control after #2240/#2245 (#2520) (#2533)

The Tools tab in the Agents view rendered in the tab bar but its panel
body was empty (placeholder `${nothing /* tools panel removed */}`).
Clicking Tools showed a highlighted tab with no content — only
user-visible dead control remaining from the post-#2336 audit sweep.

Changes:
- ui/src/ui/views/agents.ts: drop "tools" from `AgentsPanel` type
  union; delete `{ id: "tools", label: "Tools" }` tab entry; delete
  empty placeholder.
- ui/src/ui/app.ts, app-settings.ts, app-view-state.ts: narrow each
  `agentsPanel` literal union to drop "tools" (the only consumer was
  the deleted tab).
- ui/src/ui/app-render.ts: delete two now-impossible
  `panel === "tools"` blocks that lazy-loaded the tools catalog on
  tab activation. Eager `loadToolsCatalog` callers (gateway connect,
  settings apply, refresh handler) preserved — `toolsCatalog*` state
  is still live and asserted by the smoke suite.

Out-of-scope (deferred): orphan `onToolsProfileChange` /
`onToolsOverridesChange` props/handlers in `views/agents.ts` and
`app-render.ts` — distinct dead-code pattern from the visible tab,
warrants its own gut wave. `"skills"` literal in the type union also
preserved (no consumer, but AC silent).

AC verification:
- `grep -rn '"tools"' ui/src/ui/views/agents.ts` → no matches
- `grep -rn 'id: "tools"' ui/src/` → no matches
- `pnpm check` green (oxfmt, tsgo, oxlint, fork-integrity gates)
- `pnpm test` green (7014 + 288 tests)
- `pnpm test:ui:smoke` green (12 tests, browser-mode Chromium)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

 sweep

Shadow fields outlived the skills UI code tree after #2240/#2245 gutted it —
no readers remain for any of:

- `AgentsPanel` / `agentsPanel` union `"skills"` member
- `agentSkills{Loading,Error,Report,AgentId}` view-state fields
- `skillsReport` in the host-interface intersection
- `SkillStatusReport` type + its import
- `tabs.skills` + `subtitles.skills` i18n entries across all 6 locales

Type-narrowing deletions only. Smoke suite's host-interface-field assertions
auto-update when `AppViewState` is narrowed — validates the contract stays
coherent post-deletion.

(#2521)

 sweep (#2536)

Shadow fields outlived the skills UI code tree after #2240/#2245 gutted it —
no readers remain for any of:

- `AgentsPanel` / `agentsPanel` union `"skills"` member
- `agentSkills{Loading,Error,Report,AgentId}` view-state fields
- `skillsReport` in the host-interface intersection
- `SkillStatusReport` type + its import
- `tabs.skills` + `subtitles.skills` i18n entries across all 6 locales

Type-narrowing deletions only. Smoke suite's host-interface-field assertions
auto-update when `AppViewState` is narrowed — validates the contract stays
coherent post-deletion.

(#2521)