fix(mobile): make usePoseStream read-only#274
Conversation
Keep WebSocket ownership in App-level wiring so screens stop subscribing and reconnecting again every time mounts. Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
|
Follow-up note: I reran the PR-relevant mobile tests locally on this branch: |
|
Quick sync from my side: this branch is still clean locally and unchanged relative to its fork tracking branch ( Current state I rechecked today:
If this looks good on your end, it seems ready for review/merge. |
|
Closing this stale radar lane to keep the authored-PR queue current. I refreshed/revalidated the branch recently, but there has still been no maintainer response on this thread, so I am closing it for now instead of letting it sit indefinitely. If maintainers want this change later, I can reopen or resend it on a fresh branch. |
Summary
wsService.subscribe()/connect()side effects fromusePoseStreamApp.tsxusePoseStreamno longer subscribes or reconnectsValidation
cd ui/mobile && npm test -- --runInBand src/__tests__/hooks/usePoseStream.test.tsESLINT_USE_FLAT_CONFIG=false npx -y eslint@8 -c .eslintrc.js ...still fails on the repo’s pre-existing ESLint/@typescript-eslint rule mismatch, not on this patchNotes
#151,#236, and#240; it only touches the passive hook layer undersrc/hooks/usePoseStream.ts