refactor(web): the Crew request poll uses the shared visible-refresh hook

[bryantderosier]

Problem

After #309 merged, the Crew seat-approval poll still had a hand-written lifecycle: module-level consumer counting, a poll key, start/stop functions, and a startVisibleRefresh export. Jackson's review on #309 pointed out that the gating atom already does the gating, so none of that was needed.

What I changed

• apps/web/src/j5/crew/crewRuntimeRequestsClient.ts: useCrewRuntimeRequestsRefresh is now createVisibleRefreshHook(refreshPolledEnvironments, CREW_RUNTIME_REQUESTS_POLL_INTERVAL_MS). The removed pieces are consumers, polledKey, stopPoll, syncPoll, the consumer-count and key effects, and the useEffect import.
• apps/web/src/j5/useVisibleRefresh.ts: the startVisibleRefresh export is gone, and the file is back to its shape before feat(crews): answer a Crew thread's provider approvals and questions from the Inbox #309.
• refreshCrewRuntimeRequests, the forced re-read after an answer, is unchanged.

Why this shape

• polledCrewRuntimeRequestsAtom (apps/web/src/j5/state.ts) only mounts an environment's query while that environment has a pending approval.
• A newly mounted query fetches on its own, because J5 queries are Atom.swr({ revalidateOnMount: true }).
• refreshPolledEnvironments sends nothing while no environment is polled.

So the shared hook gives the same result with far less code.

Invariants

• With nothing pending, a tick sends zero requests.
• An answer still forces an immediate re-read.

Surfaces

Surface	Decision
Entry points	Unaffected.
Clients (web, desktop, mobile)	Web and desktop; mobile has no Crew Inbox.
Providers	Unaffected.
Contracts	Unaffected.
Reverse states	Not applicable.
Connection modes	Unaffected.
Upstream files / FORK.md	None.
Docs	Unaffected.

Out of scope

• Nothing. This is only the simplification Jackson suggested on feat(crews): answer a Crew thread's provider approvals and questions from the Inbox #309.

Upgrade and data

None.

Verification

• apps/web typechecks (exit 0).
• vp test run apps/web/src/j5: 329 passed. The pure crewApprovalPollPlan tests still cover the gating decision.
• Lint and format are clean.

Review focus

• The tradeoff Jackson named: a second approval on an environment that's already polled now shows on the next 7.5s tick instead of immediately, the same latency as every other J5 poll.

Claude Opus 5.5 via Claude Code in J5 Code

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository: Jacksondr5/j5code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 46554b8e-f25d-421e-b7e3-1b09a4473642

---

_{Comment @coderabbitai help to get the list of available commands.}