[ac-pplx]

Summary

This change removes UI code that nothing calls. Behaviour does not change.

• api.postPreferDirect and the "prefer-direct" request mode behind it. For a console served over plain HTTP on port 3000 from a non-loopback host, without external sign-in, it sent the request straight to port 8080 on that host and fell back to the same-origin proxy on a network error. Nothing used it, so request() drops its mode parameter.
• getPublicSandbox, deletePublicOperation and runPublicSandboxOperation in ui/src/lib/public-api.ts, plus the two helpers only they used (publicSandboxToConsoleSandbox, failedConditionMessage).
• ui/src/lib/sandbox-actions.ts, which nothing imports.
• On the nodes page: refreshAll.
• On the templates page: reconcileTemplateInventory, snapshotPolicyFanoutMessage, pluralize, positiveOrDefault, and the SnapshotPolicyFanoutResponse and TemplateNetworkPolicyUpdateResponse types.
• Unused networkPoliciesEqual and Form imports (both in sandbox-detail.tsx; networkPoliciesEqual also in sandboxes/page.tsx).

The nodes page's rendering of a node's queued reconcile trigger is kept. It is unreachable today: the admin node response has no reconcile_trigger field, and buildNodeView sets it to null. Showing it would need both the response field and an adapter change.

Stack

Independent of the other open console PRs. Whichever of this PR and #1244 / #1248 merges later gets a conflict in ui/src/lib/public-api.ts, next to getConsoleSandboxes: keep that branch's new getConsoleSandboxes and drop getPublicSandbox. The import removals in sandboxes/page.tsx and sandbox-detail.tsx merge cleanly.

Validation

• npm ci, npm test (50 pass) and npm run build in ui/. The type check confirms nothing referenced the removed symbols.
• A full Playwright loop over every console route and flow against a local Kind stack, on this branch's Kind space-ui image. No new console errors, page errors or failed requests compared with main.

Severity and risk

Severity: does not apply (cleanup).

Risk: low. A deployment that patched in a caller of any removed export (api.postPreferDirect, the three public-api.ts functions, publicSandboxToConsoleSandbox, or @/lib/sandbox-actions) fails at build time. Roll back by reverting the commit.

AI assistance: Claude Code wrote the change and this description, and ran the validation; a maintainer reviews it.

[github-actions]

Thanks for opening this pull request! A SPACE maintainer will take a look.

CONTRIBUTING.md explains how we review outside contributions.

Withdrawing this approval for now. Any feedback above still stands.

[johnlouie-pplx]

Approving. I checked every removed symbol against the rest of the repo at this head and nothing still references them, including the UI tests, the Python smoke code and the other open console PRs. There are no dynamic imports or file-based routes involved, and sandbox-actions.ts had no side effects, so deleting it is safe. The request() change keeps the same behavior: every remaining call already used the same-origin path, and the three authApi calls now pass the base in the right position.

Locally on 8ddfc2e: npm ci, npm test (50/50), tsc --noEmit and npm run build all pass. Worth knowing that no PR CI job builds ui/ (the Kind smoke was skipped for this path set), so the local build is what covers this change. I didn't run the Playwright pass myself.