Skip to content

fix(web): keep settled threads reachable when opened directly - #4413

Merged
t3dotgg merged 8 commits into
mainfrom
sidebar-v2-settled-thread-affordances
Jul 24, 2026
Merged

fix(web): keep settled threads reachable when opened directly#4413
t3dotgg merged 8 commits into
mainfrom
sidebar-v2-settled-thread-affordances

fix(web): re-read useNowMinute snapshot when no timer is live

54b1618
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Effect Service Conventions succeeded Jul 24, 2026 in 29s

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 changed scope is limited to React UI code and a client-side hook: apps/web/src/components/ChatView.tsx, apps/web/src/components/SidebarV2.tsx, and the new apps/web/src/hooks/useNowMinute.ts. These changes add settled-thread affordances (composer banner, sidebar visibility, a shared minute-quantized clock hook) and consume existing helpers/commands at the React application boundary. No Effect service is created, moved, refactored, or defined here: effectiveSettled is a pure helper consumed via a named import (permitted), atom commands and hooks are consumed at the React boundary, and no consolidated "effect" imports, Layer.succeed/ManagedRuntime.make/runPromise misuse, or error-modeling violations are introduced. No convention violations found in the pull request's changed scope.


🔍 Agent activity — reasoning, no tool calls

💭 Looking at the diff to understand what's changing with Effect service conventions. The modifications span React components like ChatView and SidebarV2, plus a new hook for managing minute updates with external store synchronization. The changes appear to handle settled states and refactor time-based logic into a reusable hook, but I'm not seeing any violations of the Effect service patterns yet. I'm verifying there aren't any Effect service violations by checking the actual imports and usage patterns. The diff shows React hooks and atom commands being consumed at the application boundary, which is allowed. effectiveSettled is a pure helper with a named import, which follows conventions. I don't see any Effect service creation, movement, or refactoring happening here, so this appears to be all clear. The changed code only consumes these utilities through React hooks at the application boundary, which is allowed. Since the diff introduces no Effect service violations and doesn't define any service conventions, I'm confident this is clean.


Agent Credits: 20 credits