File zone share: component tests, state machine moved out of ShareSet #116

Merged
void merged 3 commits from void/share-set-tests into dev 2026-11-12 13:15:50 +00:00
Member

Follow-up chat-mind-20261112T124133749Z-t6. The repo has no DOM test environment (server render only), so the panel's state moved into shareController.ts (picked files, phase, one send per room, encrypted refusal at Send, failed files kept by position) and ShareSet.tsx draws it (ShareSetView is pure props). No behavior change intended; flag stays off.

Tests: shareController.test.ts (double tap, re-mount, other rooms, throw frees the room, progress, failed by position with same-named twins, all failed, encrypted refusal at Send not mount); ShareSet.test.tsx (idle row, picked, sending, encrypted note, success; flag off/on/encrypted, hook under the config provider). 87 albums tests pass.

Not covered: a real tap on a real input (no DOM env); that is the phone check.

🤖 Generated with Claude Code

Follow-up `chat-mind-20261112T124133749Z-t6`. The repo has no DOM test environment (server render only), so the panel's state moved into `shareController.ts` (picked files, phase, one send per room, encrypted refusal at Send, failed files kept by position) and `ShareSet.tsx` draws it (`ShareSetView` is pure props). No behavior change intended; flag stays off. Tests: shareController.test.ts (double tap, re-mount, other rooms, throw frees the room, progress, failed by position with same-named twins, all failed, encrypted refusal at Send not mount); ShareSet.test.tsx (idle row, picked, sending, encrypted note, success; flag off/on/encrypted, hook under the config provider). 87 albums tests pass. Not covered: a real tap on a real input (no DOM env); that is the phone check. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
File zone share: panel state machine apart from React, with component tests
Some checks failed
Build pull request / Build pull request (pull_request) Has been cancelled
eb0a1d0ee0
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
File zone share tests: lint
Some checks failed
Build pull request / Build pull request (pull_request) Has been cancelled
86a68839b0
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Author
Member

Soul pass on 86a68839: 17 mutations, 4 survived, plus two code findings. Fixed in the next commit: controller built once per mount (was useMemo over mx/room/queryClient, a rerender with a new Room could drop the pick), success lands in one notification, new tests kill the surviving title-on-partial-failure and onSent timing/throw mutations (checked by re-mutating). Left, stated: key={epoch} on the file input is not observable by server render (phone check); the encrypted-note wording is pinned by literal. 90 albums tests pass. Flag stays off.

Soul pass on 86a68839: 17 mutations, 4 survived, plus two code findings. Fixed in the next commit: controller built once per mount (was useMemo over mx/room/queryClient, a rerender with a new Room could drop the pick), success lands in one notification, new tests kill the surviving title-on-partial-failure and onSent timing/throw mutations (checked by re-mutating). Left, stated: key={epoch} on the file input is not observable by server render (phone check); the encrypted-note wording is pinned by literal. 90 albums tests pass. Flag stays off.
File zone share: controller built once per mount, one notification on success, tests for the surviving mutations
All checks were successful
Build pull request / Build pull request (pull_request) Successful in 3m48s
cf99096d3c
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
void merged commit 494d2a7ed4 into dev 2026-11-12 13:15:50 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
GameCult/Ratatoskr!116
No description provided.