paris 0.17.1 → 0.17.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # paris
 
+## 0.17.3
+
+### Patch Changes
+
+- 1f67456: chore: add typecheck script and PR check CI workflow
+
 ## 0.17.1
 
 ### Patch Changes

package/package.json

@@ -2,7 +2,7 @@
   "name": "paris",
   "author": "Sanil Chawla <sanil@slingshot.fm> (https://sanil.co)",
   "description": "Paris is Slingshot's React design system. It's a collection of reusable components, design tokens, and guidelines that help us build consistent, accessible, and performant user interfaces.",
-  "version": "0.17.1",
+  "version": "0.17.3",
   "homepage": "https://paris.slingshot.fm",
   "license": "MIT",
   "repository": {
@@ -64,9 +64,9 @@
   "dependencies": {
     "@ariakit/react": "^0.4.20",
     "@emotion/is-prop-valid": "^1.4.0",
-    "@fortawesome/fontawesome-svg-core": "^6.4.2",
-    "@fortawesome/free-regular-svg-icons": "^6.4.2",
-    "@fortawesome/free-solid-svg-icons": "^6.4.2",
+    "@fortawesome/fontawesome-svg-core": "^6.7.2",
+    "@fortawesome/free-regular-svg-icons": "^6.7.2",
+    "@fortawesome/free-solid-svg-icons": "^6.7.2",
     "@fortawesome/react-fontawesome": "^0.2.2",
     "@headlessui/react": "^2.2.4",
     "@radix-ui/react-checkbox": "^1.3.3",
@@ -81,6 +81,8 @@
     "ts-deepmerge": "^6.0.3"
   },
   "peerDependencies": {
+    "@types/react": "^19",
+    "@types/react-dom": "^19",
     "react": "^19.x",
     "react-dom": "^19.x",
     "sass": "^1.x",
@@ -144,6 +146,7 @@
     "generate:theme": "pte export ./src/stories/theme/themes.ts LightTheme -o ./public/pte.css",
     "generate:text": "ts-node --esm ./scripts/text.ts",
     "release": "pnpm generate:exports && pnpm changeset publish",
+    "typecheck": "tsc --noEmit",
     "lint": "eslint ."
   }
 }

package/src/stories/select/Select.tsx

@@ -151,6 +151,14 @@ export const Select = forwardRef(<T = Record<string, any>>({
 }: SelectProps<T>, ref: ForwardedRef<any>) => {
     const inputID = useId();
     const multiItems = multipleItemsName || 'items';
+
+    // TypeScript can't track discriminated union correlation through destructuring and JSX conditionals.
+    // For Listbox: supports both single and multi via overloads, needs explicit union types
+    // For RadioGroup: only supports single-select, needs narrowed types
+    const listboxValue = value as string | string[] | null | undefined;
+    const listboxOnChange = onChange as ((value: string | string[] | null) => void) | undefined;
+    const singleValue = value as string | null | undefined;
+    const singleOnChange = onChange as ((value: string | null) => void) | undefined;
     const buttonText = () => {
         if (!value || value.length === 0) {
             return placeholder || 'Select an option';
@@ -182,8 +190,8 @@ export const Select = forwardRef(<T = Record<string, any>>({
                 <Listbox
                     as="div"
                     ref={ref}
-                    value={value}
-                    onChange={onChange}
+                    value={listboxValue}
+                    onChange={listboxOnChange}
                     multiple={multiple}
                 >
                     <ListboxButton
@@ -273,7 +281,7 @@ export const Select = forwardRef(<T = Record<string, any>>({
                 </Listbox>
             )}
             {kind === 'radio' && (
-                <RadioGroup ref={ref} as="div" className={styles.radioContainer} value={value} onChange={onChange}>
+                <RadioGroup ref={ref} as="div" className={styles.radioContainer} value={singleValue} onChange={singleOnChange}>
                     {options.map((option) => (
                         <Radio
                             as="div"
@@ -294,7 +302,7 @@ export const Select = forwardRef(<T = Record<string, any>>({
                 </RadioGroup>
             )}
             {kind === 'card' && (
-                <RadioGroup ref={ref} as="div" className={styles.cardContainer} value={value} onChange={onChange}>
+                <RadioGroup ref={ref} as="div" className={styles.cardContainer} value={singleValue} onChange={singleOnChange}>
                     {options.map((option) => (
                         <Radio
                             as="div"
@@ -316,7 +324,7 @@ export const Select = forwardRef(<T = Record<string, any>>({
                 </RadioGroup>
             )}
             {kind === 'segmented' && (
-                <RadioGroup ref={ref} as="div" className={styles.segmentedContainer} value={value || options[0].id} onChange={onChange}>
+                <RadioGroup ref={ref} as="div" className={styles.segmentedContainer} value={singleValue || options[0].id} onChange={singleOnChange}>
                     {options.map((option) => (
                         <Radio
                             as="div"