refactor(shared): remove unused viewport formatters - #9970
Effect Service Conventions: No issues found
All clear
Details
Note
Your check run agent prompt is: .macroscope/check-run-agents/effect-service-conventions.md
More information about how Check Run Agents work can be found in our Help Center.
The PR removes two unused pure formatter helpers (previewViewportLabel, previewViewportPresetOrientation) from packages/shared/src/previewViewport.ts and their corresponding test assertions. No Effect service is created, moved, refactored, or consumed; no imports, error definitions, layers, or runtime boundaries are touched. Verified that neither removed symbol has any remaining consumers or barrel re-exports in the repository at head, so the removal introduces no dangling references.
🔍 Agent activity — 1 tool call
💭 The diff just removes two formatter functions and their test, which doesn't violate the Effect service convention check itself, though I should verify nothing else still depends on those functions before confirming it's safe to remove.
1. browse_code — Are the functions previewViewportLabel and previewViewportPresetOrientation referenced anywhere in the repo (imports, usages, barrel re-exports)? List all files and lines.
💭 Checking for dead code and Effect service convention issues turns up nothing worth flagging.
Agent Credits: 208 credits
Token usage: 50 uncached input · 1,316,301 cache-read input · 150,340 cache-write input · 15,214 output
Agent Credits may also include non-token charges from external tools such as web research.