@tastic/profile 0.11.0 → 0.12.0

package/src/ProfilesManager.tsx

@@ -14,17 +14,17 @@ import { type ProfilePickerAlignResult } from './ProfilePicker'
 import { clampTag, isValidTag, MAX_PROFILE_NAME_LENGTH } from './profilesValidation'
 import { Profile } from './types'
 
-// Shared by the New Profile row's own dashed circle and every profile row's own ProfileChip — one
+// Shared by the New Profile row's own dashed circle and every profile row's own ProfileChip - one
 // size for every circle on this screen, rather than each picking its own.
 const CHIP_SIZE = 40
 
-// react-native-paper's own MD3 Text variant names — mirrored locally rather than importing its
+// react-native-paper's own MD3 Text variant names - mirrored locally rather than importing its
 // internal VariantProp/MD3TypescaleKey (not exported from the package's public entry point), same
 // as @tastic/hud's BaseStatsScreen does for its own identical titleVariant prop.
 type MD3TextVariant = 'displayLarge' | 'displayMedium' | 'displaySmall' | 'headlineLarge' | 'headlineMedium' | 'headlineSmall' | 'titleLarge' | 'titleMedium' | 'titleSmall' | 'labelLarge' | 'labelMedium' | 'labelSmall' | 'bodyLarge' | 'bodyMedium' | 'bodySmall'
 
-// Exported (not just used internally) so a host composing a screen around this component — e.g.
-// @tastic/profile's own ProfilesScreen — can reference this exact shape in its own onCreate/onSave
+// Exported (not just used internally) so a host composing a screen around this component - e.g.
+// @tastic/profile's own ProfilesScreen - can reference this exact shape in its own onCreate/onSave
 // prop types without redeclaring it.
 export interface ProfileEditPatch {
   name: string
@@ -33,36 +33,36 @@ export interface ProfileEditPatch {
 }
 
 export interface ProfilesManagerHandle {
-  // Flushes whatever row is currently being edited — the same thing this component's own unmount
+  // Flushes whatever row is currently being edited - the same thing this component's own unmount
   // effect does, but callable explicitly. Exists because "unmount" isn't a reliable "the user is
   // navigating away" signal on every host: React Navigation's web renderer keeps a popped screen
   // mounted-but-hidden instead of unmounting it, so a host relying only on the unmount fallback
   // silently loses an in-progress edit on `router.back()` there. A host whose router doesn't
   // genuinely unmount this component on navigation should hold a ref and call this right before
   // navigating away (e.g. in its own back-button handler). Safe to call any time, including when
-  // nothing is being edited — a no-op then, same as the unmount effect already is.
+  // nothing is being edited - a no-op then, same as the unmount effect already is.
   commitPendingEdit: () => void
 }
 
 export interface ProfilesManagerProps {
   profiles: Profile[]
-  // The color a brand-new profile's draft starts on — this component doesn't have (and shouldn't
+  // The color a brand-new profile's draft starts on - this component doesn't have (and shouldn't
   // invent) an opinion about the host app's own palette, so this is required rather than defaulted.
   defaultColor: string
   onCreate: (input: ProfileEditPatch) => void
   onSave: (id: string, patch: ProfileEditPatch) => void
-  // Irreversible — this only fires after this component's own confirmation card, not on the first
+  // Irreversible - this only fires after this component's own confirmation card, not on the first
   // tap. If the host app tracks anything else keyed by profile id (stats, history), clear that on
   // its own side when this fires.
   onDelete: (id: string) => void
-  // Rendered inline, before the "Profiles" title, in this component's own header row — a plain
+  // Rendered inline, before the "Profiles" title, in this component's own header row - a plain
   // rendering slot, not an onBack-style callback: this component doesn't know or care that it's a
   // back button, only that the host wants something shown there. Omit to hide the slot.
   headerLeft?: ReactNode
-  // React 19 accepts `ref` as a plain prop on a function component (no forwardRef needed) — see
+  // React 19 accepts `ref` as a plain prop on a function component (no forwardRef needed) - see
   // ProfilesManagerHandle's own doc for why a host may need this.
   ref?: Ref<ProfilesManagerHandle>
-  // Independently optional overrides for this screen's text/card colors — each defaults to the same
+  // Independently optional overrides for this screen's text/card colors - each defaults to the same
   // literal-black/white-by-appearance formula as before when omitted, so every existing caller
   // (none of which pass these) renders identically to before. Same convention as @tastic/hud's
   // BaseStatsScreen fg/cardBg overrides, for a caller with its own app-wide chrome palette instead
@@ -74,10 +74,10 @@ export interface ProfilesManagerProps {
   fgMuted?: string
   cardBg?: string
   // 'headlineSmall' (this screen's own default, below), matching BaseStatsScreen's own settled
-  // default and the reasoning behind it — this title sits next to a small headerLeft back-button
+  // default and the reasoning behind it - this title sits next to a small headerLeft back-button
   // glyph the same way, and the previous hardcoded 'displaySmall' read as oversized there too.
   titleVariant?: MD3TextVariant
-  // Transforms the color swatch grid's own preview per-swatch — forwarded straight to the internal
+  // Transforms the color swatch grid's own preview per-swatch - forwarded straight to the internal
   // InlineColorPicker's identical prop (see its own doc). Leaves draftColor/onCreate/onSave's actual
   // color value untouched; only what's rendered changes. Defaults to ProfileColorProvider's preview.
   colorPreview?: (hex: string) => string
@@ -97,7 +97,7 @@ export interface ProfilesManagerProps {
 
 type Row = { kind: 'profile'; profile: Profile } | { kind: 'new' }
 
-// First letter of the name, uppercased — the tag's own default for as long as it hasn't been typed
+// First letter of the name, uppercased - the tag's own default for as long as it hasn't been typed
 // into directly (see the tagManuallySet state below). Empty name -> empty tag, same as typing
 // nothing into the tag field yourself would leave it.
 function deriveTag(name: string): string {
@@ -105,14 +105,14 @@ function deriveTag(name: string): string {
   return trimmed ? trimmed[0].toUpperCase() : ''
 }
 
-// The profile roster itself — create/rename/recolor/retag/delete, editing inline right in the row
-// you tapped — with no navigation of its own: just data in (`profiles`) and callbacks out, plus
+// The profile roster itself - create/rename/recolor/retag/delete, editing inline right in the row
+// you tapped - with no navigation of its own: just data in (`profiles`) and callbacks out, plus
 // this file's own internal edit-draft state. Deliberately router-agnostic (an `onBack`-style prop
 // would still assume *some* host owns navigation; this component doesn't even go that far), so any
 // consuming screen just renders it as its body inside its own routed wrapper (back button, native
 // push/pop transition, whatever persistence hook backs `profiles`/`onCreate`/`onSave`/`onDelete`).
 // Staying router-agnostic means this component can't know for itself whether "navigating away"
-// really unmounts it on a given host's router — see ProfilesManagerHandle's own doc for the
+// really unmounts it on a given host's router - see ProfilesManagerHandle's own doc for the
 // react-navigation-web case where it doesn't, and why a host there needs the `ref` escape hatch
 // instead of relying on the unmount fallback alone.
 export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDelete, headerLeft, ref, fg: fgOverride, fgMuted: fgMutedOverride, cardBg: cardBgOverride, titleVariant = 'headlineSmall', colorPreview: colorPreviewProp, useColorPickerAlign }: ProfilesManagerProps) {
@@ -130,35 +130,35 @@ export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDe
   const fgMuted = fgMutedOverride ?? (dark ? 'rgba(255,255,255,0.5)' : 'rgba(0,0,0,0.5)')
   const cardBg = cardBgOverride ?? (dark ? '#111111' : '#F2F2F2')
   const cardBorder = dark ? 'rgba(255,255,255,0.2)' : 'rgba(0,0,0,0.2)'
-  // A contained neutral, not an outlined one — reads as "the other button" next to Delete's solid
+  // A contained neutral, not an outlined one - reads as "the other button" next to Delete's solid
   // danger fill, rather than looking unfinished/secondary beside it.
   const cancelBg = dark ? '#2A2A2A' : '#E0E0E0'
   const cancelFg = dark ? '#FFFFFF' : '#000000'
 
-  // 'new' clears back to null once a commit resolves — there's only ever one row open at a time.
+  // 'new' clears back to null once a commit resolves - there's only ever one row open at a time.
   const [editingId, setEditingId] = useState<string | 'new' | null>(null)
   const [draftName, setDraftName] = useState('')
   const [draftColor, setDraftColor] = useState(defaultColor)
   const [draftTag, setDraftTag] = useState('')
-  // Whether the tag has been typed into directly this session — once true, editing the name stops
+  // Whether the tag has been typed into directly this session - once true, editing the name stops
   // overwriting it (see deriveTag's own doc). Starts true for an existing profile that already has
-  // a tag (don't clobber a deliberate choice just because the name changed), false otherwise — a
+  // a tag (don't clobber a deliberate choice just because the name changed), false otherwise - a
   // brand new profile, or one that's never had a tag set at all.
   const [tagManuallySet, setTagManuallySet] = useState(false)
   const [confirmDeleteId, setConfirmDeleteId] = useState<string | null>(null)
 
-  // Commits whichever row is currently open, using its own latest draft state, then closes it —
+  // Commits whichever row is currently open, using its own latest draft state, then closes it -
   // always closes, even when there's nothing valid to save (an empty name just discards the draft
   // rather than leaving the row stuck open). This is the only way a row ever closes: submitting the
-  // name field (see EditRow's own focus-chain wiring — tag submits to name, name submits this),
+  // name field (see EditRow's own focus-chain wiring - tag submits to name, name submits this),
   // switching to a different row (see startEdit, which calls this before opening a new one), or
-  // leaving the screen entirely (see the unmount effect below) — no explicit save/cancel buttons,
+  // leaving the screen entirely (see the unmount effect below) - no explicit save/cancel buttons,
   // just scoped to the whole row instead of one field, since there's more than one field to touch.
   const commitEdit = useCallback(() => {
     const name = draftName.trim()
     if (editingId && name) {
       const tagValid = isValidTag(draftTag)
-      // Surfaced here, not just the field's own live `error` outline — that outline disappears the
+      // Surfaced here, not just the field's own live `error` outline - that outline disappears the
       // moment this row closes, so without this a still-invalid tag would just silently vanish
       // (saved as empty) with no explanation of why it didn't stick.
       if (draftTag && !tagValid) showErrorToast('Invalid tag', 'Use up to 3 letters or a single emoji')
@@ -169,11 +169,11 @@ export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDe
     setEditingId(null)
   }, [editingId, draftName, draftColor, draftTag, onCreate, onSave, showErrorToast])
 
-  // Flushes whatever's currently being edited if this whole screen goes away mid-edit — a hardware
+  // Flushes whatever's currently being edited if this whole screen goes away mid-edit - a hardware
   // back gesture or swipe-back can bypass a host screen's own back button (which should already
   // call commitEdit before navigating away) but still unmounts this component the same as any other
   // exit does. Reads the always-current commitEdit through a ref rather than depending on it
-  // directly in the effect — one that re-ran (and re-cleaned-up) on every keystroke would commit on
+  // directly in the effect - one that re-ran (and re-cleaned-up) on every keystroke would commit on
   // every keystroke too, not just on a genuine unmount.
   const commitEditRef = useRef(commitEdit)
   useEffect(() => {
@@ -181,7 +181,7 @@ export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDe
   })
   useEffect(() => () => commitEditRef.current(), [])
   // The explicit escape hatch for hosts whose "navigate away" doesn't genuinely unmount this
-  // component — see ProfilesManagerHandle's own doc. `[]` deps: the handle object itself never
+  // component - see ProfilesManagerHandle's own doc. `[]` deps: the handle object itself never
   // needs to change identity, since it always calls through the ref to whatever commitEdit is
   // current at call time, the same indirection the unmount effect above already relies on.
   useImperativeHandle(ref, () => ({ commitPendingEdit: () => commitEditRef.current() }), [])
@@ -213,8 +213,8 @@ export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDe
     [tagManuallySet]
   )
 
-  // The functional setDraftTag form — reading `previous` from React's own latest-applied state
-  // rather than closing over the `draftTag` variable — matters here, not just style: fast/rapid
+  // The functional setDraftTag form - reading `previous` from React's own latest-applied state
+  // rather than closing over the `draftTag` variable - matters here, not just style: fast/rapid
   // keystrokes can fire their onChangeText calls before a re-render lands, batched into the same
   // update, and clamping every one of them against the same stale `draftTag` (captured once at
   // render time) would let the later ones silently clobber what the earlier ones already clamped
@@ -233,7 +233,7 @@ export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDe
     setConfirmDeleteId(null)
   }, [confirmDeleteId, editingId, onDelete])
 
-  // Alphabetical — this list has no other natural order (creation order isn't meaningful to look
+  // Alphabetical - this list has no other natural order (creation order isn't meaningful to look
   // something up by), and it's the one that actually stays usable once there are more than a
   // handful of profiles on one shared device.
   const sortedProfiles = useMemo(() => [...profiles].sort((a, b) => a.name.localeCompare(b.name)), [profiles])
@@ -279,7 +279,7 @@ export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDe
               return <EditRow key={profile.id} draftName={draftName} onNameChange={handleNameChange} draftColor={draftColor} onColorChange={setDraftColor} draftTag={draftTag} onTagChange={handleTagChange} onSubmit={commitEdit} onDelete={() => setConfirmDeleteId(profile.id)} fgMuted={fgMuted} host={host} dark={dark} colorPreview={colorPreview} useColorPickerAlign={useColorPickerAlign} />
             }
 
-            // No delete affordance at rest — tap the row to start editing, which is the one place
+            // No delete affordance at rest - tap the row to start editing, which is the one place
             // delete shows (see EditRow): a deliberate extra step before a destructive action is
             // even reachable, not just before it's confirmed.
             return (
@@ -307,13 +307,13 @@ export function ProfilesManager({ profiles, defaultColor, onCreate, onSave, onDe
               <Text variant='bodyLarge' style={[styles.overlayBody, { color: fg }]}>
                 This permanently deletes {confirmDeleteProfile.name}&rsquo;s profile. This cannot be undone.
               </Text>
-              {/* Contained neutral, not buttonColor={colors.primary} — colors.primary is often a
+              {/* Contained neutral, not buttonColor={colors.primary} - colors.primary is often a
               live gameplay/seat color in host apps, so a contained Cancel in that color would just
               be whatever's currently painted there, with no relation to "safe to tap". */}
               <Button mode='contained' onPress={() => setConfirmDeleteId(null)} style={styles.overlayButton} buttonColor={cancelBg} textColor={cancelFg}>
                 Cancel
               </Button>
-              {/* colors.danger/onDanger — auto-paper's actual semantic role for this, not a seat's
+              {/* colors.danger/onDanger - auto-paper's actual semantic role for this, not a seat's
               own live color by luck of the default palette. */}
               <Button mode='contained' onPress={handleConfirmDelete} style={styles.overlayButton} buttonColor={colors.danger} textColor={colors.onDanger}>
                 Delete
@@ -334,7 +334,7 @@ interface EditRowProps {
   draftTag: string
   onTagChange: (tag: string) => void
   onSubmit: () => void
-  // Omitted for the 'new' row — nothing saved yet to delete.
+  // Omitted for the 'new' row - nothing saved yet to delete.
   onDelete?: () => void
   autoFocus?: boolean
   fgMuted: string
@@ -356,29 +356,29 @@ function AlignedColorPicker({ useAlign, ...props }: ComponentProps<typeof Inline
   return <InlineColorPicker {...props} alignOverride={alignOverride} />
 }
 
-// One row, expanded: the color trigger (@tastic/hud's InlineColorPicker) — doubling as the tag's
-// own display via its `tag` prop below, since there's no separate visible tag field anymore — an
+// One row, expanded: the color trigger (@tastic/hud's InlineColorPicker) - doubling as the tag's
+// own display via its `tag` prop below, since there's no separate visible tag field anymore - an
 // invisible TextInput that exists purely to actually capture what's typed (RN has no way to bring
 // up a keyboard without a real focusable input; this one is just never meant to be *seen*), the
-// name taking up the row's remaining space, and — only for an existing profile — a delete icon at
+// name taking up the row's remaining space, and - only for an existing profile - a delete icon at
 // the very end. The two effects below are what wires tapping the swatch to popping the keyboard on
 // the hidden tag field too, so typing a tag and picking a color read as one combined action instead
 // of two separate taps, with the swatch itself as the only place the tag ever visibly shows.
 // Return-key chains tag -> name via @rific/focus-chain rather than each field submitting on its
-// own — submitting the *name* field is what actually commits the row, so its registration's own
-// onSubmitEditing (a no-op — it's last in the chain) is overridden with onSubmit below, matching
+// own - submitting the *name* field is what actually commits the row, so its registration's own
+// onSubmitEditing (a no-op - it's last in the chain) is overridden with onSubmit below, matching
 // how a normal multi-field form reads: fill fields in order, the last one finishes it.
 function EditRow({ draftName, onNameChange, draftColor, onColorChange, draftTag, onTagChange, onSubmit, onDelete, autoFocus, fgMuted, host, dark, colorPreview, useColorPickerAlign }: EditRowProps) {
   const register = useFocusChain()
   const tag = register()
   const name = register()
 
-  // tag.props.focus (from @rific/focus-chain's own register(), not tag.ref — that's a callback
+  // tag.props.focus (from @rific/focus-chain's own register(), not tag.ref - that's a callback
   // ref, not a ref object, so it has no .current to read) is the hook's own imperative "focus this
   // exact registered field" call. Mirrored into a ref every render, same as commitEditRef above,
   // rather than read directly in the effect below: register() returns a brand-new props object on
   // every render (it isn't memoized), so depending on tag.props.focus directly would rerun that
-  // effect — and refocus an already-focused field — on every keystroke into the tag field itself,
+  // effect - and refocus an already-focused field - on every keystroke into the tag field itself,
   // re-triggering its own selectTextOnFocus mid-type and wiping out what's being typed.
   const tagFocusRef = useRef(tag.props.focus)
   useEffect(() => {
@@ -397,9 +397,9 @@ function EditRow({ draftName, onNameChange, draftColor, onColorChange, draftTag,
     <View style={styles.editRow}>
       {useColorPickerAlign ? <AlignedColorPicker useAlign={useColorPickerAlign} id='color' host={host} value={draftColor} onChange={onColorChange} previewValue={colorPreview} tag={draftTag} dark={dark} /> : <InlineColorPicker id='color' host={host} value={draftColor} onChange={onColorChange} previewValue={colorPreview} tag={draftTag} dark={dark} />}
       {/* Invisible on purpose (styles.hiddenTagInput: zero footprint, position: 'absolute' so it
-      doesn't reserve space in the row's own flex layout) — the swatch above is the only place this
+      doesn't reserve space in the row's own flex layout) - the swatch above is the only place this
       value is ever meant to be seen. pointerEvents='none' is what actually keeps it out of the
-      way: without it, this field — even at 1x1 and fully transparent — still sits in front of the
+      way: without it, this field - even at 1x1 and fully transparent - still sits in front of the
       swatch for hit-testing purposes and swallows the tap meant to open the color popover, since
       focusing this field is never done by tapping it directly anyway (see the effect above, which
       focuses it programmatically). accessibilityLabel stands in for the placeholder a visible field
@@ -439,7 +439,7 @@ const styles = StyleSheet.create({
     gap: 4,
     paddingBottom: 8
   },
-  // 1x1 and fully transparent — there's nothing to see here on purpose (see EditRow's own comment
+  // 1x1 and fully transparent - there's nothing to see here on purpose (see EditRow's own comment
   // above the field this styles). position: 'absolute' pulls it out of editRow's flex flow
   // entirely, so it doesn't reserve a gap in the row the way a normal flex sibling would.
   hiddenTagInput: {

package/src/ProfilesScreen.tsx

@@ -6,7 +6,7 @@ import { StyleSheet, View } from 'react-native'
 import { ProfileEditPatch, ProfilesManager, ProfilesManagerHandle, ProfilesManagerProps } from './ProfilesManager'
 import { Profile } from './types'
 
-// react-native-paper's own MD3 Text variant names — mirrored locally rather than importing its
+// react-native-paper's own MD3 Text variant names - mirrored locally rather than importing its
 // internal VariantProp/MD3TypescaleKey (not exported from the package's public entry point), same
 // as ProfilesManager's own identical titleVariant prop does.
 type MD3TextVariant = 'displayLarge' | 'displayMedium' | 'displaySmall' | 'headlineLarge' | 'headlineMedium' | 'headlineSmall' | 'titleLarge' | 'titleMedium' | 'titleSmall' | 'labelLarge' | 'labelMedium' | 'labelSmall' | 'bodyLarge' | 'bodyMedium' | 'bodySmall'
@@ -16,22 +16,22 @@ export interface ProfilesScreenProps {
   // Same reasoning as ProfilesManager's own defaultColor: this component has no opinion on the
   // host's palette, so it stays required rather than defaulted. Callers already differ here today
   // (most apps pass @rific/auto-paper's defaultColors[0].value; one passes its own local
-  // DEFAULT_P1_COLOR constant instead) — this prop is exactly what lets both keep working unchanged.
+  // DEFAULT_P1_COLOR constant instead) - this prop is exactly what lets both keep working unchanged.
   defaultColor: string
-  // Same ProfileEditPatch shape ProfilesManager's own onCreate/onSave already take — deliberately
+  // Same ProfileEditPatch shape ProfilesManager's own onCreate/onSave already take - deliberately
   // NOT widened to accept a host's own richer CreateProfileInput. A host whose profile shape has
   // extra required fields (e.g. a control/key scheme) composes those in its own closure at the call
   // site, exactly as it already does calling <ProfilesManager> today.
   onCreate: (patch: ProfileEditPatch) => void
   onSave: (id: string, patch: ProfileEditPatch) => void
-  // Irreversible, same as ProfilesManager's own onDelete — fires once, after this component's
+  // Irreversible, same as ProfilesManager's own onDelete - fires once, after this component's
   // internal confirmation card. If the host tracks anything else keyed by profile id (stats,
   // achievement-unlock history), clear that on its own side inside this callback, same as every
   // current app's `(id) => { deleteProfile(id); removeProfileStats(id) }`.
   onDelete: (id: string) => void
   onBack: () => void
   // Independently optional overrides, each defaulting to the same literal-black/white-by-appearance
-  // formula every current caller renders with unchanged (none of them pass these today) — same
+  // formula every current caller renders with unchanged (none of them pass these today) - same
   // convention as BaseStatsScreen's and ProfilesManager's own fg/bg-shaped overrides. fg/fgMuted/
   // cardBg/titleVariant/colorPreview are forwarded straight through to the internal ProfilesManager.
   fg?: string
@@ -49,13 +49,13 @@ export interface ProfilesScreenProps {
 // The routed profiles-roster screen shell every app in this fleet hand-rolls around ProfilesManager
 // today: the container View + theme-derived bg/fg + back button + commit-pending-edit-before-
 // navigate composition. Lives here rather than in @tastic/hud since it has zero generic/non-profile
-// UI content — it's one layer up from ProfilesManager, not a different kind of thing. Deliberately
-// still doesn't call useProfiles()/useGameStats() (or any host-specific hook) itself — those two
+// UI content - it's one layer up from ProfilesManager, not a different kind of thing. Deliberately
+// still doesn't call useProfiles()/useGameStats() (or any host-specific hook) itself - those two
 // hooks' shapes are genuinely app-specific (different CreateProfileInput/GameStatsContextValue per
 // app), so this stays props-in/callbacks-out, exactly like ProfilesManager itself.
 export function ProfilesScreen({ profiles, defaultColor, onCreate, onSave, onDelete, onBack, fg: fgOverride, fgMuted, bg: bgOverride, cardBg, titleVariant, colorPreview, useColorPickerAlign }: ProfilesScreenProps) {
   const { dark } = useAutoPaperTheme()
-  // Owns the ref itself — nothing outside this component ever needs it, since this is now the one
+  // Owns the ref itself - nothing outside this component ever needs it, since this is now the one
   // thing that both renders ProfilesManager AND owns the only navigation trigger next to it.
   const managerRef = useRef<ProfilesManagerHandle>(null)
   // Same literal-black/white-by-appearance formula as ProfilesManager's own fg default, resolved

package/src/index.ts

@@ -10,5 +10,6 @@ export { type CreateProfileInput, createProfileRecord, profilesActions, profiles
 export { resolveInitialProfiles } from './redux/resolveInitialProfiles'
 export { createSeatColorsSlice, type SeatColorsActions, type SeatColorsSlice, type SeatColorsState, type SetLastGuestColorPayload, type SetProfileOverridePayload } from './redux/seatColorsSlice'
 export { useSharedProfilesSync, type UseSharedProfilesSyncOptions, type UseSharedProfilesSyncResult } from './redux/useSharedProfilesSync'
-export { isSharedProfileStoreAvailable, loadSharedProfiles, saveSharedProfiles } from './sharedProfileStore'
+export { avoidSeatColorClash } from './seatColorClash'
+export { isSharedProfileStoreAvailable, loadSharedProfiles, readSharedProfiles, saveSharedProfiles } from './sharedProfileStore'
 export type { Profile } from './types'

package/src/profilesValidation.ts

@@ -1,7 +1,7 @@
 import { Profile } from './types'
 
 export const MAX_PROFILE_NAME_LENGTH = 20
-// In code points, not JS string length — a single emoji can span more than one UTF-16 code unit
+// In code points, not JS string length - a single emoji can span more than one UTF-16 code unit
 // (surrogate pairs, variation selectors), which .length would overcount.
 export const MAX_TAG_LENGTH = 3
 
@@ -11,23 +11,23 @@ function isValidHexColor(value: unknown): value is string {
 
 // Codepoint 0xFE0F is the variation selector that forces an emoji-style rendering of an otherwise
 // dual-purpose glyph; 0x200D is the zero-width joiner that chains multiple pictographs into one
-// compound emoji (a family, a flag, a skin-tone variant) — built from String.fromCodePoint rather
+// compound emoji (a family, a flag, a skin-tone variant) - built from String.fromCodePoint rather
 // than embedded directly in a regex literal so this file's own source has no invisible characters
 // in it.
 const VARIATION_SELECTOR = String.fromCodePoint(0xfe0f)
 const ZERO_WIDTH_JOINER = String.fromCodePoint(0x200d)
-// Matches one emoji as a single unit, including a ZWJ-joined compound — good enough for telling
+// Matches one emoji as a single unit, including a ZWJ-joined compound - good enough for telling
 // "the user typed an emoji" from "the user typed letters" without needing full Unicode grapheme
 // segmentation (Intl.Segmenter's own Hermes support is inconsistent enough not to lean on here).
 const EMOJI_PATTERN = new RegExp(`^\\p{Extended_Pictographic}${VARIATION_SELECTOR}?(${ZERO_WIDTH_JOINER}\\p{Extended_Pictographic}${VARIATION_SELECTOR}?)*$`, 'u')
 
-// Presence check, not the full anchored EMOJI_PATTERN above — clampTag below only needs to know
+// Presence check, not the full anchored EMOJI_PATTERN above - clampTag below only needs to know
 // whether a just-typed chunk contains a pictograph at all, not whether an entire value is exactly
 // one (possibly ZWJ-joined) emoji. Same distinction @tastic/hud's InlineColorPicker draws for its
 // own font-sizing check.
 const EMOJI_PRESENT = /\p{Extended_Pictographic}/u
 
-// Empty is valid too — a profile can have no tag yet, same as it can go without ever entering an
+// Empty is valid too - a profile can have no tag yet, same as it can go without ever entering an
 // editor's own tag field at all; every place this renders falls back to a generic icon in that case
 // (see ProfileChip).
 export function isValidTag(value: unknown): value is string {
@@ -45,8 +45,8 @@ export function clampTag(previous: string, next: string): string {
   const previousCodepoints = Array.from(previous)
   const nextCodepoints = Array.from(next)
 
-  // Shrinking (backspace/delete, or a selectTextOnFocus-driven wholesale replacement — see
-  // ProfilesManager's own EditRow — reporting its own replacement text as no longer than what it
+  // Shrinking (backspace/delete, or a selectTextOnFocus-driven wholesale replacement - see
+  // ProfilesManager's own EditRow - reporting its own replacement text as no longer than what it
   // overwrote) always passes through untouched: there's nothing to clamp about removing content.
   if (nextCodepoints.length <= previousCodepoints.length) return next
 
@@ -54,7 +54,7 @@ export function clampTag(previous: string, next: string): string {
   while (sharedPrefixLength < previousCodepoints.length && previousCodepoints[sharedPrefixLength] === nextCodepoints[sharedPrefixLength]) sharedPrefixLength++
   const added = nextCodepoints.slice(sharedPrefixLength).join('')
 
-  // A tag is either a handful of plain characters or exactly one emoji, never a mix — typing more
+  // A tag is either a handful of plain characters or exactly one emoji, never a mix - typing more
   // of either kind on top of the other one starts fresh with just what's newly typed, rather than
   // producing a combination isValidTag would reject anyway (an emoji followed by letters, or a
   // second emoji tacked onto the first).
@@ -63,7 +63,7 @@ export function clampTag(previous: string, next: string): string {
   return nextCodepoints.length > MAX_TAG_LENGTH ? previous : next
 }
 
-// Validates only the fields this package's own `Profile` type declares — an app that extends
+// Validates only the fields this package's own `Profile` type declares - an app that extends
 // Profile with its own fields (a control scheme, anything else) needs its own wrapper that runs
 // this first and then checks its own additions, same as it extends the type itself. See this
 // package's own README for the composition pattern.

package/src/redux/createActionCreator.ts

@@ -10,7 +10,7 @@ export type ActionCreator<P> = ((payload: P) => ReduxAction<P>) & {
 
 // Same low-level shape @rific/core's createSettingsSlice gives every other slice in the fleet
 // (namespaced type string, .type, .match). Shared internally between profilesSlice.ts and
-// profileExtensionSlice.ts rather than each hand-rolling its own copy — not exported from
+// profileExtensionSlice.ts rather than each hand-rolling its own copy - not exported from
 // index.ts, since it's an implementation detail of how this package's own reducers are built, not
 // something a consuming app needs.
 export function createActionCreator<P>(type: string): ActionCreator<P> {

package/src/redux/profileExtensionSlice.ts

@@ -19,7 +19,7 @@ export interface ProfileExtensionSlice<TExtension> {
 
 // A per-profile extension is any app-specific field that doesn't belong on the shared base Profile
 // type (a control scheme, a key-binding preference, a difficulty setting, ...) but still needs to
-// travel WITH a specific profile id, entirely locally — never synced to the shared App Group
+// travel WITH a specific profile id, entirely locally - never synced to the shared App Group
 // roster (a profile created on a sibling @tastic game has no idea this extension exists at all,
 // same as this package's own README describes for any app-specific field). Two real @tastic games
 // (BoxHockey's controlScheme, LightCycles' keyScheme) each hand-rolled an almost-identical
@@ -28,10 +28,10 @@ export interface ProfileExtensionSlice<TExtension> {
 // need a third copy.
 //
 // `namespace` prefixes every action type (e.g. 'profileExtension/controlScheme/set') so an app with
-// more than one independent per-profile extension never collides between them — same convention
+// more than one independent per-profile extension never collides between them - same convention
 // @rific/core's createSettingsSlice uses its own namespace argument for.
 //
-// Deliberately NOT auto-cleared when the base `profiles` slice removes an id — this stays a plain,
+// Deliberately NOT auto-cleared when the base `profiles` slice removes an id - this stays a plain,
 // self-contained reducer rather than importing profilesActions.remove's action type to listen for.
 // An app already dispatches multiple independent actions when deleting a profile (profilesActions.
 // remove, and typically its own local selection-slice cleanup); calling this slice's own `remove`

package/src/redux/profileSelectionSlice.ts

@@ -1,7 +1,7 @@
 import { ActionCreator, createActionCreator } from './createActionCreator'
 
 // redux-persist's own REHYDRATE action-type constant, inlined as a literal rather than taking a
-// dependency on the package itself — this factory otherwise has no opinion on whether the
+// dependency on the package itself - this factory otherwise has no opinion on whether the
 // consuming app uses redux-persist at all, and the string itself is a long-stable part of
 // redux-persist's own public contract (unchanged across every major version). See the reducer's
 // own REHYDRATE case below for why this needs handling at all.
@@ -25,30 +25,30 @@ export interface ProfileSelectionSlice<TSeat extends string | number> {
   reducer: (state: ProfileSelectionState<TSeat> | undefined, action: { type: string; payload?: unknown }) => ProfileSelectionState<TSeat>
 }
 
-// Which profile each seat last picked — always local, never part of the shared App Group roster
+// Which profile each seat last picked - always local, never part of the shared App Group roster
 // (this package's own profiles slice, see profilesSlice.ts): who's selected on THIS device is
 // naturally a per-app, per-device thing, not something a sibling @tastic game should see or
 // influence. Five @tastic games (AirHockey, BoxHockey, Pong, LightCycles, Snake) each hand-rolled
-// an identical Record<Seat, string | null> reducer before this factory existed — same duplication
+// an identical Record<Seat, string | null> reducer before this factory existed - same duplication
 // createProfileExtensionSlice generalized away for per-profile extensions, this is that shape for
 // per-seat selection instead.
 //
 // `namespace` prefixes every action type (e.g. 'profileSelection/loadout/select') so an app with
-// more than one independent seat-selection concern never collides between them — same convention
+// more than one independent seat-selection concern never collides between them - same convention
 // createProfileExtensionSlice's own namespace argument follows.
 //
 // Unlike createProfileExtensionSlice, the seat set can't be derived from TSeat alone (types don't
-// exist at runtime), so callers pass their own `defaultState` — the same literal object (e.g.
+// exist at runtime), so callers pass their own `defaultState` - the same literal object (e.g.
 // `{ 1: null, 2: null }`) each app already declares as its own `defaultProfileSelectionState`
 // constant today. `clearProfile` walks that state's own keys rather than assuming any particular
 // seat count, so it reproduces the 5 apps' current two-seat behavior without hardcoding "1"/"2".
 //
-// `mountKey` defaults to 'profileSelection' — the literal key every current consumer (AirHockey,
+// `mountKey` defaults to 'profileSelection' - the literal key every current consumer (AirHockey,
 // BoxHockey, Pong, LightCycles, Snake) already mounts this reducer under in its own
 // combineReducers({...}) call (e.g. `combineReducers({ ..., profileSelection })`). It has to be
 // passed explicitly rather than inferred, since redux-persist's REHYDRATE payload is keyed by
 // whatever name the app's own root reducer happens to use, which this factory has no way to see
-// from inside its own reducer function — only worth overriding if some future consumer ever mounts
+// from inside its own reducer function - only worth overriding if some future consumer ever mounts
 // this under a different key. Note `namespace` above is a different thing: it's the app's own name
 // (e.g. 'airhockey', 'boxhockey'), used only to prefix action types, not the mount key.
 //
@@ -56,6 +56,15 @@ export interface ProfileSelectionSlice<TSeat extends string | number> {
 // after any profilesActions.setAll (initial reconcile, onRemoteChange), since a sibling app can
 // delete a profile without this app's own deleteProfile ever running. It returns the same state
 // object when nothing changes, so dispatching it on every foreground resume costs no re-render.
+//
+// One profile is never on two seats. `select` picking a profile another seat already has swaps
+// the two: that seat takes this seat's previous pick (a profile, or null for guest), the same rule
+// as @tastic/input's assignSeatScheme. A picker's takenProfileId only hides the other seat's
+// profile while both seats are on screen, so without the swap a 1 Player pick of seat 2's saved
+// profile left both seats on it, and every later 2 Player round recorded a win and a loss for that
+// one profile. A host showing per-seat state tied to the selection (a profile's color override)
+// should reset it for both seats when a pick swaps. REHYDRATE also drops any duplicate saved before
+// this rule existed, keeping the profile on the first seat.
 export function createProfileSelectionSlice<TSeat extends string | number>(namespace: string, defaultState: ProfileSelectionState<TSeat>, mountKey = 'profileSelection'): ProfileSelectionSlice<TSeat> {
   const select = createActionCreator<SelectProfilePayload<TSeat>>(`profileSelection/${namespace}/select`)
   const clearProfile = createActionCreator<string>(`profileSelection/${namespace}/clearProfile`)
@@ -64,16 +73,35 @@ export function createProfileSelectionSlice<TSeat extends string | number>(names
   function reducer(state: ProfileSelectionState<TSeat> = defaultState, action: { type: string; payload?: unknown }): ProfileSelectionState<TSeat> {
     // redux-persist's own default stateReconciler (autoMergeLevel1) HARD-REPLACES this slice's
     // entire persisted sub-state on rehydration rather than backfilling missing fields (confirmed
-    // against autoMergeLevel1's actual source: `newState[key] = inboundState[key]`, not a merge) —
+    // against autoMergeLevel1's actual source: `newState[key] = inboundState[key]`, not a merge) -
     // so a device with a blob persisted before a new seat existed would otherwise rehydrate with
     // that seat's key missing entirely, and every consumer indexes into it (`state[seat]`) with no
     // fallback. Spreading `defaultState` first, then the incoming persisted value, backfills exactly
-    // that gap — the same pattern createSeatColorsSlice's own REHYDRATE case already uses.
+    // that gap - the same pattern createSeatColorsSlice's own REHYDRATE case already uses.
     if (action.type === REHYDRATE) {
       const persisted = (action.payload as Record<string, Partial<ProfileSelectionState<TSeat>>> | undefined)?.[mountKey]
-      return { ...defaultState, ...state, ...persisted }
+      const merged = { ...defaultState, ...state, ...persisted }
+      const seen = new Set<string>()
+      for (const seat of Object.keys(merged) as TSeat[]) {
+        const profileId = merged[seat]
+        if (profileId === null) continue
+        if (seen.has(profileId)) merged[seat] = null
+        else seen.add(profileId)
+      }
+      return merged
+    }
+    if (select.match(action)) {
+      const { profileId, seat } = action.payload
+      const previous = state[seat] ?? null
+      const next = { ...state, [seat]: profileId } as ProfileSelectionState<TSeat>
+      if (profileId !== null) {
+        // Object.keys yields strings, so a numeric seat compares as a string.
+        for (const other of Object.keys(state) as TSeat[]) {
+          if (String(other) !== String(seat) && state[other] === profileId) next[other] = previous === profileId ? null : previous
+        }
+      }
+      return next
     }
-    if (select.match(action)) return { ...state, [action.payload.seat]: action.payload.profileId } as ProfileSelectionState<TSeat>
     if (clearProfile.match(action)) {
       const next = { ...state }
       for (const seat of Object.keys(state) as TSeat[]) {

package/src/redux/resolveInitialProfiles.ts

@@ -1,23 +1,27 @@
-import { isSharedProfileStoreAvailable, loadSharedProfiles, saveSharedProfiles } from '../sharedProfileStore'
+import { isSharedProfileStoreAvailable, readSharedProfiles, saveSharedProfiles } from '../sharedProfileStore'
 import { Profile } from '../types'
 
 // Resolves the roster an app should start with, given whatever it already had saved in its own
-// local fallback storage (however that app stores it — this never touches AsyncStorage itself,
-// only the shared App Group store via loadSharedProfiles/saveSharedProfiles). Every @tastic game
+// local fallback storage (however that app stores it - this never touches AsyncStorage itself,
+// only the shared App Group store via readSharedProfiles/saveSharedProfiles). Every @tastic game
 // hand-rolled an identical version of this exact decision inline in its own useProfiles.tsx; this
 // is that logic, extracted verbatim.
 //
 // - Shared store unavailable (Android/web, or an iOS build that hasn't run `expo prebuild` since
 //   the entitlement was added): `localFallback` IS the roster, verbatim.
+// - Shared store available, but its roster couldn't be read (readSharedProfiles' null):
+//   `localFallback`, and nothing is written. Seeding here would overwrite every other app's
+//   profiles with this app's local snapshot.
 // - Shared store available, and genuinely empty: either a first launch, or the first time THIS
-//   app has synced since the App Group entitlement was added — seed it from `localFallback` (so
+//   app has synced since the App Group entitlement was added - seed it from `localFallback` (so
 //   an upgrading user's own profiles don't just vanish), and mirror that seed to the shared store
 //   (fire-and-forget; a failed seed write just means the next successful write catches up).
 // - Shared store available and non-empty: shared always wins over a stale local snapshot, never
-//   merged with it — avoids resurrecting a profile that was deleted from another app.
+//   merged with it - avoids resurrecting a profile that was deleted from another app.
 export async function resolveInitialProfiles(groupId: string, localFallback: Profile[]): Promise<Profile[]> {
   if (!isSharedProfileStoreAvailable) return localFallback
-  const shared = await loadSharedProfiles(groupId)
+  const shared = await readSharedProfiles(groupId)
+  if (shared === null) return localFallback
   if (shared.length === 0 && localFallback.length > 0) {
     saveSharedProfiles(groupId, localFallback).catch(() => {})
     return localFallback