@multiplatform.one/forms 7.11.0 → 7.16.0

package/src/Form/submissionPaths.spec.tsx

@@ -0,0 +1,214 @@
+// @vitest-environment jsdom
+/**
+ * MPO-275: every way into a form's submission shares one guarded execution and
+ * one outcome. The routes are a native submit button, implicit Enter in the
+ * only field, Button action=submit, the save bar's Save, and Ctrl-S / Cmd-S.
+ */
+import { renderWithProviders } from "@multiplatform.one/test-utils";
+import { act, cleanup, fireEvent, waitFor } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
+import type { ReactElement } from "react";
+import { afterEach, describe, expect, it, vi } from "vitest";
+import { Button } from "../Button";
+import * as saveBarStories from "../ContextualSaveBar/ContextualSaveBar.stories";
+import { Input } from "../fields/Input";
+import { Form, useFormContext } from "./index";
+
+afterEach(cleanup);
+
+const routes = ["html", "enter", "action", "save", "ctrl-s", "cmd-s"] as const;
+type Route = (typeof routes)[number];
+const saveBarRoutes: readonly Route[] = ["save", "ctrl-s", "cmd-s"];
+
+let formState = () => ({ isDirty: false, submissionAttempts: 0 });
+
+function StateProbe() {
+  const form = useFormContext();
+  formState = () => ({
+    isDirty: form.state.isDirty,
+    submissionAttempts: form.state.submissionAttempts,
+  });
+  return null;
+}
+
+function PathForm({
+  route,
+  onSubmit,
+  onSaved,
+  defaultName = "Original",
+  required = false,
+}: {
+  route: Route;
+  onSubmit: (values: Record<string, unknown>) => unknown;
+  onSaved?: (values: Record<string, unknown>) => void;
+  defaultName?: string;
+  required?: boolean;
+}) {
+  return (
+    <Form
+      formOptions={{ defaultValues: { name: defaultName } }}
+      onSubmit={onSubmit as never}
+      saveBar={{ forceVisible: true, onSaved }}
+    >
+      <StateProbe />
+      <Input name="name" label="Name" required={required} />
+      {route === "html" ? <button type="submit">Submit HTML</button> : null}
+      <Button action="submit">Submit action</Button>
+    </Form>
+  );
+}
+
+type View = ReturnType<typeof renderWithProviders>;
+
+const nameInput = (view: View) => view.getByLabelText(/^Name/) as HTMLInputElement;
+
+async function activate(route: Route, view: View) {
+  switch (route) {
+    case "html":
+      await act(async () => fireEvent.click(view.getByRole("button", { name: "Submit HTML" })));
+      return;
+    case "enter": {
+      const user = userEvent.setup();
+      await user.type(nameInput(view), "{Enter}");
+      return;
+    }
+    case "action":
+      await act(async () => fireEvent.click(view.getByRole("button", { name: "Submit action" })));
+      return;
+    case "save":
+      await act(async () => fireEvent.click(view.getByRole("button", { name: "Save" })));
+      return;
+    case "ctrl-s":
+      await act(async () => fireEvent.keyDown(document, { key: "s", ctrlKey: true }));
+      return;
+    case "cmd-s":
+      await act(async () => fireEvent.keyDown(document, { key: "s", metaKey: true }));
+      return;
+  }
+}
+
+describe("every submission path shares one outcome (MPO-275)", () => {
+  it.each(routes)(
+    "%s: a retry clears the failure while it runs, and a failed retry shows its own",
+    async (route) => {
+      let rejectRetry!: (error: Error) => void;
+      const submit = vi
+        .fn()
+        .mockRejectedValueOnce(new Error("First refusal."))
+        .mockImplementationOnce(
+          () =>
+            new Promise((_, reject) => {
+              rejectRetry = reject;
+            }),
+        );
+      const saved = vi.fn();
+      const view = renderWithProviders(
+        <PathForm route={route} onSubmit={submit} onSaved={saved} />,
+      );
+      fireEvent.change(nameInput(view), { target: { value: "Edited" } });
+
+      await activate(route, view);
+      await waitFor(() =>
+        expect(view.getByText(/Couldn't save/).textContent).toContain("First refusal."),
+      );
+      expect(submit).toHaveBeenCalledTimes(1);
+
+      await activate(route, view);
+      await waitFor(() => expect(submit).toHaveBeenCalledTimes(2));
+      await waitFor(() => expect(view.queryByText(/First refusal/)).toBeNull());
+
+      await act(async () => rejectRetry(new Error("Second refusal.")));
+      await waitFor(() =>
+        expect(view.getByText(/Couldn't save/).textContent).toContain("Second refusal."),
+      );
+      expect(view.queryByText(/First refusal/)).toBeNull();
+      expect(submit).toHaveBeenCalledTimes(2);
+      expect(nameInput(view).value).toBe("Edited");
+      expect(saved).not.toHaveBeenCalled();
+    },
+  );
+
+  it.each(routes)(
+    "%s: a validation refusal keeps the field error, the summary and the attempt count",
+    async (route) => {
+      const submit = vi.fn();
+      const saved = vi.fn();
+      const view = renderWithProviders(
+        <PathForm route={route} onSubmit={submit} onSaved={saved} defaultName="" required />,
+      );
+
+      await activate(route, view);
+      await waitFor(() =>
+        expect(view.container.querySelector('[data-testid="error-summary"]')?.textContent).toMatch(
+          /Name is required/,
+        ),
+      );
+      expect(nameInput(view).getAttribute("aria-invalid")).toBe("true");
+      expect(formState().submissionAttempts).toBe(1);
+      expect(submit).not.toHaveBeenCalled();
+      expect(saved).not.toHaveBeenCalled();
+      expect(view.queryByText(/Couldn't save/)).toBeNull();
+    },
+  );
+
+  it.each(routes)(
+    "%s: a success calls the handler once and keeps its path's success policy",
+    async (route) => {
+      const submit = vi.fn().mockResolvedValue(undefined);
+      const saved = vi.fn();
+      const view = renderWithProviders(
+        <PathForm route={route} onSubmit={submit} onSaved={saved} />,
+      );
+      fireEvent.change(nameInput(view), { target: { value: "Edited" } });
+
+      await activate(route, view);
+      await waitFor(() => expect(submit).toHaveBeenCalledTimes(1));
+      const viaSaveBar = saveBarRoutes.includes(route);
+      await waitFor(() => expect(saved).toHaveBeenCalledTimes(viaSaveBar ? 1 : 0));
+      expect(formState().isDirty).toBe(!viaSaveBar);
+      expect(nameInput(view).value).toBe("Edited");
+      expect(view.queryByText(/Couldn't save/)).toBeNull();
+      expect(submit).toHaveBeenCalledTimes(1);
+    },
+  );
+});
+
+describe("the SubmissionPaths story counts onSubmit calls (MPO-275)", () => {
+  function mountStory() {
+    const story = saveBarStories.SubmissionPaths as { render?: () => ReactElement };
+    if (!story.render) throw new Error("SubmissionPaths has no render");
+    return renderWithProviders(story.render());
+  }
+
+  const calls = (view: View) => view.getByTestId("submit-calls").textContent;
+
+  it("adds exactly one call per activation on every path and shows the same refusal", async () => {
+    const view = mountStory();
+    expect(calls(view)).toBe("onSubmit calls: 0");
+    const user = userEvent.setup();
+
+    const paths: Array<[string, () => Promise<unknown>]> = [
+      ["enter", () => user.type(view.getByLabelText("Service name"), "{Enter}")],
+      [
+        "action",
+        () => act(async () => fireEvent.click(view.getByRole("button", { name: "Submit" }))),
+      ],
+      ["save", () => act(async () => fireEvent.click(view.getByRole("button", { name: "Save" })))],
+      ["ctrl-s", () => act(async () => fireEvent.keyDown(document, { key: "s", ctrlKey: true }))],
+      ["cmd-s", () => act(async () => fireEvent.keyDown(document, { key: "s", metaKey: true }))],
+    ];
+
+    let expected = 0;
+    for (const [, run] of paths) {
+      await run();
+      expected += 1;
+      await waitFor(() => expect(calls(view)).toBe(`onSubmit calls: ${expected}`));
+      await waitFor(() =>
+        expect(view.getByText(/Couldn't save/).textContent).toContain(
+          "The record changed. Refresh and retry.",
+        ),
+      );
+    }
+    expect((view.getByLabelText("Service name") as HTMLInputElement).value).toBe("service-example");
+  });
+});

package/src/InputParts/index.tsx

@@ -29,6 +29,7 @@ import {
   ensureFocusVisibleRing,
   FOCUS_RING_CLIPPED_OFFSET,
   FOCUS_RING_HALO_OFFSET,
+  useTouchSurface,
   isTouchSurface,
   recipeFamilies,
   resolveGlyphPaint,
@@ -70,12 +71,14 @@ function useNativeFieldA11yProps() {
 const defaultContextValues = {
   size: "$true" as FontSizeTokens,
   scaleIcon: 1,
+  touch: !isWeb && isTouchSurface(),
   color: undefined as ColorTokens | string | undefined,
 };
 
 export const InputContext = createStyledContext<{
   size: FontSizeTokens;
   scaleIcon: number;
+  touch: boolean;
   color?: ColorTokens | string;
 }>(defaultContextValues);
 
@@ -91,8 +94,10 @@ function endCapVariantsFromHeights(heightVariants: Record<string, { height: numb
 
 const sizeRecipeEndCapVariants = endCapVariantsFromHeights(recipeFamilies.control.heightVariants);
 
-function sizeRecipeBoxFallback(val: unknown) {
-  const recipe = sizeRecipeForToken(String(val ?? "$true"), { touch: isTouchSurface() });
+function sizeRecipeBoxFallback(val: unknown, extras: { props: { touch?: boolean } }) {
+  const recipe = sizeRecipeForToken(String(val ?? "$true"), {
+    touch: extras.props.touch ?? (!isWeb && isTouchSurface()),
+  });
   return {
     height: recipe.height,
     paddingHorizontal: recipe.paddingHorizontal,
@@ -100,19 +105,25 @@ function sizeRecipeBoxFallback(val: unknown) {
   };
 }
 
-function sizeRecipeTypeFallback(val: unknown) {
+function sizeRecipeTypeFallback(val: unknown, extras: { props: { touch?: boolean } }) {
   return {
-    fontSize: sizeRecipeForToken(String(val ?? "$true"), { touch: isTouchSurface() }).fontSize,
+    fontSize: sizeRecipeForToken(String(val ?? "$true"), {
+      touch: extras.props.touch ?? (!isWeb && isTouchSurface()),
+    }).fontSize,
   };
 }
 
-function sizeRecipeIconFallback(val: unknown) {
-  const { iconSize } = sizeRecipeForToken(String(val ?? "$true"), { touch: isTouchSurface() });
+function sizeRecipeIconFallback(val: unknown, extras: { props: { touch?: boolean } }) {
+  const { iconSize } = sizeRecipeForToken(String(val ?? "$true"), {
+    touch: extras.props.touch ?? (!isWeb && isTouchSurface()),
+  });
   return { width: iconSize, height: iconSize };
 }
 
-function sizeRecipeEndCapFallback(val: unknown) {
-  const { height } = sizeRecipeForToken(String(val ?? "$true"), { touch: isTouchSurface() });
+function sizeRecipeEndCapFallback(val: unknown, extras: { props: { touch?: boolean } }) {
+  const { height } = sizeRecipeForToken(String(val ?? "$true"), {
+    touch: extras.props.touch ?? (!isWeb && isTouchSurface()),
+  });
   return { height, width: height, paddingHorizontal: 0 };
 }
 
@@ -350,6 +361,7 @@ const InputGroupFrame = styled(XGroup, {
   ...(isWeb ? { className: "mp-composite-ring" } : {}),
 
   variants: {
+    touch: { true: {}, false: {} },
     unstyled: {
       false: defaultInputGroupStyles,
     },
@@ -431,6 +443,7 @@ const InputGroupImpl = InputGroupFrame.styleable<{
     paddingHorizontal?: unknown;
   };
   const [focused, setFocused] = useState(false);
+  const hydrationTouch = useTouchSurface();
   const { knobProps, control, disabledState } = useResolvedKnobs();
   const grouped = useControlGrouped();
   const inTableCell = useIsInTableCell();
@@ -476,7 +489,7 @@ const InputGroupImpl = InputGroupFrame.styleable<{
   });
   const sizeToken = String((props as { size?: unknown }).size ?? knobProps.sizeToken);
   const familyRecipe = sizeRecipeForToken(sizeToken, {
-    touch: isTouchSurface(),
+    touch: hydrationTouch,
     family: recipeFamily,
   });
   const controlFragment =
@@ -520,6 +533,7 @@ const InputGroupImpl = InputGroupFrame.styleable<{
           onPress,
           disabled,
           <InputGroupFrame
+            touch={hydrationTouch}
             ref={forwardedRef}
             borderWidth={0}
             borderRadius={0}
@@ -559,6 +573,7 @@ const InputGroupImpl = InputGroupFrame.styleable<{
       onPress,
       disabled,
       <InputGroupFrame
+        touch={hydrationTouch}
         ref={forwardedRef}
         borderWidth={0}
         borderRadius={0}
@@ -600,6 +615,7 @@ const InputGroupImpl = InputGroupFrame.styleable<{
         onPress,
         disabled,
         <InputGroupFrame
+          touch={hydrationTouch}
           ref={forwardedRef}
           {...knobProps.body}
           size={knobProps.sizeToken}
@@ -721,8 +737,10 @@ export function mapInputModeToKeyboardType(inputMode?: string): MappedKeyboardTy
   return INPUT_MODE_TO_KEYBOARD[inputMode as keyof typeof INPUT_MODE_TO_KEYBOARD];
 }
 
-export const inputSizeVariant: SizeVariantSpreadFunction<any> = (val = "$true") => {
-  const recipe = sizeRecipeForToken(String(val || "$true"), { touch: isTouchSurface() });
+export const inputSizeVariant: SizeVariantSpreadFunction<any> = (val = "$true", extras) => {
+  const recipe = sizeRecipeForToken(String(val || "$true"), {
+    touch: extras?.props.touch ?? (!isWeb && isTouchSurface()),
+  });
   return {
     fontSize: recipe.fontSize,
     // LC-84: Area is chromeless — inset lives on Input.Box / ThemedTextArea.
@@ -759,6 +777,7 @@ const InputFrame = styled(TInput, {
     outlineColor: "transparent",
   },
   variants: {
+    touch: { true: {}, false: {} },
     size: {
       ...(recipeFamilies.control.typeVariants ?? sizeRecipeTypeVariants),
       "...size": sizeRecipeTypeFallback,
@@ -776,6 +795,7 @@ const InputAreaImpl = InputFrame.styleable<{
   onChangeText?: (text: string) => void;
 }>((areaProps, ref) => {
   const { setFocused } = FocusContext.useStyledContext();
+  const hydrationTouch = useTouchSurface();
   const { size } = InputContext.useStyledContext();
   const {
     secureTextEntry,
@@ -848,6 +868,7 @@ const InputAreaImpl = InputFrame.styleable<{
       {...(inTableCell ? { minWidth: 0 } : undefined)}
     >
       <InputFrame
+        touch={hydrationTouch}
         aria-describedby={fieldDescribedBy}
         {...nativeA11yProps}
         ref={ref}
@@ -930,6 +951,7 @@ const InputSection = styled(XGroup.Item, {
   alignItems: "center",
   context: InputContext,
   variants: {
+    touch: { true: {}, false: {} },
     scaleIcon: {
       ":number": {} as any,
     },
@@ -956,6 +978,7 @@ const InputButton = styled(TButton, {
   },
 
   variants: {
+    touch: { true: {}, false: {} },
     scaleIcon: {
       ":number": {} as any,
     },
@@ -1000,6 +1023,7 @@ const InputGlyphButton = styled(View, {
   focusVisibleStyle: { outlineWidth: 0 },
 
   variants: {
+    touch: { true: {}, false: {} },
     scaleIcon: {
       ":number": {} as any,
     },
@@ -1027,6 +1051,7 @@ const InputButtonImpl = InputButton.styleable<{ glyphRing?: boolean }>((buttonPr
     className?: string;
   };
   const { size: contextSize, color: contextColor } = InputContext.useStyledContext();
+  const hydrationTouch = useTouchSurface();
   const { knobProps } = useResolvedKnobs();
   const theme = useTheme();
   const size = (sizeProp ?? contextSize) as FontSizeTokens;
@@ -1068,11 +1093,12 @@ const InputButtonImpl = InputButton.styleable<{ glyphRing?: boolean }>((buttonPr
     // knob's: a $2 field inside a $4 knob would otherwise ship a 44px stop
     // on a 32px frame. LC-77 keeps the sub-floor case pressable via the
     // caller's hitSlop, never by inflating the cap.
-    const square = sizeRecipeForToken(String(size ?? "$true"), { touch: isTouchSurface() }).height;
+    const square = sizeRecipeForToken(String(size ?? "$true"), { touch: hydrationTouch }).height;
     const disabled = (props as { disabled?: boolean }).disabled === true;
     if (isWeb) ensureCompositeFocusRing();
     return (
       <InputGlyphButton
+        touch={hydrationTouch}
         ref={ref as never}
         role="button"
         size={size}
@@ -1104,6 +1130,7 @@ const InputButtonImpl = InputButton.styleable<{ glyphRing?: boolean }>((buttonPr
   const square = knobProps.control.height;
   return (
     <InputButton
+      touch={hydrationTouch}
       ref={decorativeRef}
       size={size}
       height={square}
@@ -1134,6 +1161,7 @@ export const InputIconFrame = styled(View, {
   context: InputContext,
 
   variants: {
+    touch: { true: {}, false: {} },
     scaleIcon: {
       ":number": {} as any,
     },
@@ -1154,6 +1182,7 @@ const InputIcon = InputIconFrame.styleable<{
   adornment?: "leading" | "trailing";
 }>((iconProps, ref) => {
   const { children, color: colorProp, scaleIcon: scaleIconProp, ...props } = iconProps;
+  const hydrationTouch = useTouchSurface();
   const { size = "$true", color: contextColor, scaleIcon = 1 } = InputContext.useStyledContext();
   const { knobProps } = useResolvedKnobs();
   const resolvedScaleIcon = scaleIconProp ?? scaleIcon;
@@ -1170,6 +1199,7 @@ const InputIcon = InputIconFrame.styleable<{
   const getThemedIcon = useGetThemedIcon({ size: iconSize, color: color as any });
   return (
     <InputIconFrame
+      touch={hydrationTouch}
       ref={ref}
       size={size}
       {...knobProps.controlIcon}
@@ -1192,6 +1222,7 @@ export const InputContainerFrame = styled(View, {
   flexDirection: "column",
 
   variants: {
+    touch: { true: {}, false: {} },
     size: {
       // Size is shared with the child parts; layout spacing belongs to space.
       "...size": () => ({}),
@@ -1213,10 +1244,12 @@ export const InputContainerFrame = styled(View, {
 });
 
 const InputContainerImpl = InputContainerFrame.styleable((props, forwardedRef) => {
+  const hydrationTouch = useTouchSurface();
   const { knobProps } = useResolvedKnobs();
   const inTableCell = useIsInTableCell();
   return (
     <InputContainerFrame
+      touch={hydrationTouch}
       ref={forwardedRef}
       size={knobProps.sizeToken}
       {...knobProps.body}
@@ -1238,6 +1271,7 @@ const InputContainerImpl = InputContainerFrame.styleable((props, forwardedRef) =
 const InputLabelFrame = styled(Label, {
   context: InputContext,
   variants: {
+    touch: { true: {}, false: {} },
     scaleIcon: {
       ":number": {} as any,
     },
@@ -1263,6 +1297,7 @@ const InputInfoFrame = styled(Text, {
   color: formCommonColors.text,
 
   variants: {
+    touch: { true: {}, false: {} },
     scaleIcon: {
       ":number": {} as any,
     },
@@ -1328,6 +1363,7 @@ const InputXGroup = styled(XGroup, {
   context: InputContext,
 
   variants: {
+    touch: { true: {}, false: {} },
     scaleIcon: {
       ":number": {} as any,
     },

package/src/Label/Label.stories.tsx

@@ -1,7 +1,23 @@
 import type { Meta, StoryObj } from "@storybook/react-native-web-vite";
-import { Input as TamaguiInput, YStack } from "tamagui";
+import { YStack } from "tamagui";
+import { Input as InputParts } from "../InputParts";
 import { Label } from "./index";
 
+/**
+ * The house Input chrome without the field's own label: the Label under test
+ * is the one above it. The `Input` field brings its own label, so it cannot
+ * stand here.
+ */
+function Control({ disabled, ...area }: { id: string; placeholder?: string; disabled?: boolean }) {
+  return (
+    <InputParts>
+      <InputParts.Box disabled={disabled}>
+        <InputParts.Area disabled={disabled} {...area} />
+      </InputParts.Box>
+    </InputParts>
+  );
+}
+
 const meta: Meta<typeof Label> = {
   title: "Forms/Label",
   component: Label,
@@ -34,7 +50,7 @@ export const WithControl: Story = {
       <Label htmlFor="label-story-input" required>
         Email address
       </Label>
-      <TamaguiInput id="label-story-input" placeholder="you@example.com" />
+      <Control id="label-story-input" placeholder="you@example.com" />
     </YStack>
   ),
 };
@@ -46,13 +62,13 @@ export const RequiredAndOptional: Story = {
         <Label htmlFor="label-required" required>
           Email address
         </Label>
-        <TamaguiInput id="label-required" />
+        <Control id="label-required" />
       </YStack>
       <YStack gap="$1.5">
         <Label htmlFor="label-optional" required={false} requiredMarking="optional">
           Nickname
         </Label>
-        <TamaguiInput id="label-optional" />
+        <Control id="label-optional" />
       </YStack>
     </YStack>
   ),
@@ -64,7 +80,7 @@ export const ErrorState: Story = {
       <Label htmlFor="label-error" required error>
         Email address
       </Label>
-      <TamaguiInput id="label-error" />
+      <Control id="label-error" />
     </YStack>
   ),
 };
@@ -75,7 +91,7 @@ export const Disabled: Story = {
       <Label htmlFor="label-disabled" disabled>
         Email address
       </Label>
-      <TamaguiInput id="label-disabled" disabled />
+      <Control id="label-disabled" disabled />
     </YStack>
   ),
 };
@@ -86,7 +102,7 @@ export const AsPageHeading: Story = {
       <Label htmlFor="label-heading" asPageHeading>
         Event name
       </Label>
-      <TamaguiInput id="label-heading" />
+      <Control id="label-heading" />
     </YStack>
   ),
 };
@@ -97,7 +113,7 @@ export const Hidden: Story = {
       <Label htmlFor="label-hidden" hidden>
         Search
       </Label>
-      <TamaguiInput id="label-hidden" placeholder="Search" />
+      <Control id="label-hidden" placeholder="Search" />
     </YStack>
   ),
 };
@@ -107,18 +123,18 @@ export const EdgeCases: Story = {
     <YStack gap="$4" maxWidth={360}>
       <YStack gap="$1.5">
         <Label htmlFor="label-empty">{""}</Label>
-        <TamaguiInput id="label-empty" placeholder="empty label above" />
+        <Control id="label-empty" placeholder="empty label above" />
       </YStack>
       <YStack gap="$1.5">
         <Label htmlFor="label-long">
           This is an extremely long label that keeps going and going to check wrapping behaviour of
           label text over multiple lines without breaking the layout of the associated control
         </Label>
-        <TamaguiInput id="label-long" placeholder="long label above" />
+        <Control id="label-long" placeholder="long label above" />
       </YStack>
       <YStack gap="$1.5">
         <Label htmlFor="label-rtl">تسمية باللغة العربية</Label>
-        <TamaguiInput id="label-rtl" placeholder="rtl label above" />
+        <Control id="label-rtl" placeholder="rtl label above" />
       </YStack>
     </YStack>
   ),

package/src/Skeleton/Skeleton.radius.spec.tsx

@@ -0,0 +1,30 @@
+import { cleanup } from "@testing-library/react";
+import { renderWithProviders } from "@multiplatform.one/test-utils";
+import { Preset } from "@multiplatform.one/theme";
+import { afterEach, describe, expect, it } from "vitest";
+import { Skeleton } from "./index";
+
+afterEach(cleanup);
+
+function textBoneRadius(borderRadius: "none" | "small" | "large" | "full") {
+  const { container } = renderWithProviders(
+    <Preset overrides={{ borderRadius }}>
+      <Skeleton variant="text" width={120} />
+    </Preset>,
+  );
+  const bone = container.querySelector(".is_Skeleton") as HTMLElement;
+  const atoms = String(bone.className)
+    .split(" ")
+    .filter((c) => c.startsWith("_btlr-"));
+  cleanup();
+  return atoms;
+}
+
+describe("Skeleton text bones ride DEFAULT (knobs-enumerated Part 2b, MPO-56)", () => {
+  it("takes the token at every stop, square at none", () => {
+    expect(textBoneRadius("none")).toEqual(["_btlr-t-radius-0"]);
+    expect(textBoneRadius("small")).toEqual(["_btlr-t-radius-2"]);
+    expect(textBoneRadius("large")).toEqual(["_btlr-t-radius-6"]);
+    expect(textBoneRadius("full")).toEqual(["_btlr-t-radius-12"]);
+  });
+});

package/src/Skeleton/index.tsx

@@ -109,14 +109,11 @@ export function Skeleton({
       // Radius only — a skeleton is a flat blob, it never draws the knob's border.
       // Consumers that clamp (Checkbox/Radio/Switch) pass an explicit borderRadius
       // which wins via ...props below.
-      // Text lines are R-CLAMP (spec Skeleton): radius scales to the $2
-      // anatomy cap but honors the square world at `none` (Axiom 3 NULL) —
-      // a 5px arc at radius:none was the UX-P06 residual.
-      {...(variant === "rounded"
+      // Text lines and rounded blocks ride DEFAULT (knobs-enumerated Part 2b,
+      // MPO-56): the token at every stop, the paint capping at h/2.
+      {...(variant === "rounded" || variant === "text"
         ? { borderRadius: knobProps.borderRadius.borderRadius }
-        : variant === "text" && knobProps.pointy
-          ? { borderRadius: 0 }
-          : undefined)}
+        : undefined)}
       // R-PILL (spec Skeleton "circle"): the circular variant stays 1:1 and
       // round at every radius value, so it declares the class the
       // `radius:none` sweep licenses it under (D-09 / LC-60).

package/src/Wheel/Wheel.spec.tsx

@@ -7,7 +7,7 @@ import { Wheel } from "./index";
 afterEach(cleanup);
 
 describe("Wheel contrast contract", () => {
-  it("uses the selected-text weight and keeps rounding on the selection band", () => {
+  it("keeps the selected row on the body weight and rounding on the selection band", () => {
     renderWithProviders(
       <Preset overrides={{ borderRadius: "full" }}>
         <Wheel items={[1, 2, 3, 4, 5]} value={3} visibleItems={5} />
@@ -15,8 +15,15 @@ describe("Wheel contrast contract", () => {
     );
     const wheel = screen.getByTestId("wheel");
     const selected = wheel.querySelector('[data-selected="true"]')?.firstElementChild;
+    const idle = wheel.querySelector(
+      '[data-testid="wheel-item"]:not([data-selected="true"])',
+    )?.firstElementChild;
     expect(selected).toBeTruthy();
-    expect(["700", "bold"]).toContain(getComputedStyle(selected as Element).fontWeight);
+    expect(idle).toBeTruthy();
+    expect(getComputedStyle(selected as Element).fontWeight).toBe("400");
+    expect(getComputedStyle(selected as Element).fontWeight).toBe(
+      getComputedStyle(idle as Element).fontWeight,
+    );
     expect(["", "0px"]).toContain(getComputedStyle(wheel).borderTopLeftRadius);
     expect(getComputedStyle(screen.getByTestId("wheel-selection")).borderStartStartRadius).not.toBe(
       "0px",

package/src/Wheel/index.tsx

@@ -847,7 +847,6 @@ export function Wheel<T = string | number>({
       <Text
         {...knobProps.body}
         fontSize={size}
-        fontWeight={isSelected && !disabled ? "bold" : knobProps.textWeight.fontWeight}
         color={itemColor(isSelected)}
         numberOfLines={1}
         ellipsizeMode="tail"
@@ -865,7 +864,6 @@ export function Wheel<T = string | number>({
       width={flex ? "100%" : columnWidth}
       minWidth={flex ? 0 : columnWidth}
       flex={flex}
-      borderRadius={0}
       data-testid="wheel"
       data-disabled={disabled ? true : undefined}
       aria-disabled={disabled || undefined}