Compare commits

..

2 Commits

Author SHA1 Message Date
github-actions[bot]
2ae0e5a07b chore(release): version packages (#7595)
* chore(release): version packages

* deps: install

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: shadcn <m@shadcn.com>
2025-06-11 20:06:18 +04:00
shadcn
431af4f7ff fix(shadcn): semicolon in code style (#7594)
* fix(shadcn): handle semicolon in code style

* chore: changeset

* fix: format
2025-06-11 19:54:04 +04:00
7 changed files with 88 additions and 36 deletions

View File

@@ -86,7 +86,7 @@
"recharts": "2.15.1",
"rehype-pretty-code": "^0.14.1",
"rimraf": "^6.0.1",
"shadcn": "2.6.2",
"shadcn": "2.6.3",
"shiki": "^1.10.1",
"sonner": "^2.0.0",
"tailwind-merge": "^3.0.1",

View File

@@ -88,7 +88,7 @@
"react-resizable-panels": "^2.0.22",
"react-wrap-balancer": "^0.4.1",
"recharts": "2.12.7",
"shadcn": "2.6.2",
"shadcn": "2.6.3",
"sharp": "^0.32.6",
"sonner": "^1.2.3",
"swr": "2.2.6-beta.3",

View File

@@ -1,5 +1,11 @@
# @shadcn/ui
## 2.6.3
### Patch Changes
- [#7594](https://github.com/shadcn-ui/ui/pull/7594) [`431af4f7ff294af032c0687b8b655ed6db2e690f`](https://github.com/shadcn-ui/ui/commit/431af4f7ff294af032c0687b8b655ed6db2e690f) Thanks [@shadcn](https://github.com/shadcn)! - fix: semicolon in code style
## 2.6.2
### Patch Changes

View File

@@ -1,6 +1,6 @@
{
"name": "shadcn",
"version": "2.6.2",
"version": "2.6.3",
"description": "Add components to your apps.",
"publishConfig": {
"access": "public"

View File

@@ -88,7 +88,7 @@ import * as SelectPrimitive from "@radix-ui/react-select"
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from "radix-ui";
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from "radix-ui"
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
@@ -109,7 +109,7 @@ export const DialogRoot = Root
export const DialogTrigger = Trigger
export const SelectContent = Content`
const expected = `import { Root, Trigger, Content } from "radix-ui";
const expected = `import { Root, Trigger, Content } from "radix-ui"
export const DialogRoot = Root
export const DialogTrigger = Trigger
@@ -131,7 +131,7 @@ import { useState } from "react"
export const Dialog = DialogPrimitive.Root
export const Select = SelectRoot`
const expected = `import { Dialog as DialogPrimitive, Root as SelectRoot } from "radix-ui";
const expected = `import { Dialog as DialogPrimitive, Root as SelectRoot } from "radix-ui"
import { useState } from "react"
@@ -184,7 +184,7 @@ export const Dialog = DialogPrimitive.Root`
const expected = `"use client"
import { Dialog as DialogPrimitive } from "radix-ui";
import { Dialog as DialogPrimitive } from "radix-ui"
import { useState } from "react"
export const Dialog = DialogPrimitive.Root`
@@ -206,7 +206,7 @@ export const Dialog = DialogPrimitive.Root`
const expected = `"use client"
import { DropdownMenu as DropdownMenuPrimitive, Dialog as DialogPrimitive } from "radix-ui";
import { DropdownMenu as DropdownMenuPrimitive, Dialog as DialogPrimitive } from "radix-ui"
import { useState } from "react"
@@ -228,7 +228,7 @@ import * as SelectPrimitive from '@radix-ui/react-select'
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from 'radix-ui';
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from 'radix-ui'
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
@@ -248,7 +248,7 @@ import * as SelectPrimitive from "@radix-ui/react-select"
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from 'radix-ui';
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from 'radix-ui'
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
@@ -270,7 +270,7 @@ export type MyDialogProps = ComponentProps
export type MySelectProps = SelectProps
export const Dialog = DialogPrimitive.Root`
const expected = `import { type ComponentProps, type SelectProps, Root, Dialog as DialogPrimitive } from "radix-ui";
const expected = `import { type ComponentProps, type SelectProps, Root, Dialog as DialogPrimitive } from "radix-ui"
export type MyDialogProps = ComponentProps
export type MySelectProps = SelectProps
@@ -291,7 +291,7 @@ import { Root, Trigger } from "@radix-ui/react-dialog"
export type Props = DialogProps
export const DialogRoot = Root`
const expected = `import { type DialogProps, Root, Trigger } from "radix-ui";
const expected = `import { type DialogProps, Root, Trigger } from "radix-ui"
export type Props = DialogProps
export const DialogRoot = Root`
@@ -308,7 +308,7 @@ import * as DialogPrimitive from "@radix-ui/react-dialog"
export type Props = DialogTypes.ComponentProps
export const Dialog = DialogPrimitive.Root`
const expected = `import { type Dialog as DialogTypes, Dialog as DialogPrimitive } from "radix-ui";
const expected = `import { type Dialog as DialogTypes, Dialog as DialogPrimitive } from "radix-ui"
export type Props = DialogTypes.ComponentProps
export const Dialog = DialogPrimitive.Root`
@@ -327,7 +327,7 @@ export const Dialog = DialogPrimitive.Root
export const ChevronDown = ChevronDownIcon`
const expected = `import { ChevronDownIcon, Cross2Icon } from "@radix-ui/react-icons"
import { Dialog as DialogPrimitive, Root } from "radix-ui";
import { Dialog as DialogPrimitive, Root } from "radix-ui"
export const Dialog = DialogPrimitive.Root
export const ChevronDown = ChevronDownIcon`
@@ -349,7 +349,7 @@ export type MyIconProps = IconProps
export type MyDialogProps = ComponentProps`
const expected = `import type { IconProps } from "@radix-ui/react-icons/dist/types"
import { type ComponentProps, Root } from "radix-ui";
import { type ComponentProps, Root } from "radix-ui"
export type MyIconProps = IconProps
export type MyDialogProps = ComponentProps`
@@ -373,7 +373,7 @@ export type Props = IconProps`
const expected = `import * as Icons from "@radix-ui/react-icons"
import { ChevronDownIcon } from "@radix-ui/react-icons"
import type { IconProps } from "@radix-ui/react-icons/dist/types"
import { Dialog as DialogPrimitive, Root } from "radix-ui";
import { Dialog as DialogPrimitive, Root } from "radix-ui"
export const Dialog = DialogPrimitive.Root
export const Icon = ChevronDownIcon
@@ -401,7 +401,7 @@ import {
export const DialogRoot = Root
export const SelectValue = Value`
const expected = `import { Root, Trigger, Content, Value, Item } from "radix-ui";
const expected = `import { Root, Trigger, Content, Value, Item } from "radix-ui"
export const DialogRoot = Root
export const SelectValue = Value`
@@ -425,7 +425,7 @@ import * as SelectPrimitive from "@radix-ui/react-select"
export const Dialog = DialogRoot
export const Select = SelectPrimitive.Root`
const expected = `import { Root as DialogRoot, Trigger, Content, Select as SelectPrimitive } from "radix-ui";
const expected = `import { Root as DialogRoot, Trigger, Content, Select as SelectPrimitive } from "radix-ui"
export const Dialog = DialogRoot
export const Select = SelectPrimitive.Root`
@@ -450,7 +450,7 @@ import {
export type Props = DialogProps`
const expected = `import { type ComponentProps, type DialogProps, type SelectProps, Root } from "radix-ui";
const expected = `import { type ComponentProps, type DialogProps, type SelectProps, Root } from "radix-ui"
export type Props = DialogProps`
@@ -472,7 +472,7 @@ export type Props = DialogProps`
export const DialogRoot = Root`
const expected = `import { Root, Trigger, Content } from "radix-ui";
const expected = `import { Root, Trigger, Content } from "radix-ui"
export const DialogRoot = Root`
@@ -490,7 +490,7 @@ export const DialogRoot = Root`
export const DialogRoot = Root`
const expected = `import { Root, Trigger, Content } from "radix-ui";
const expected = `import { Root, Trigger, Content } from "radix-ui"
export const DialogRoot = Root`
@@ -532,7 +532,7 @@ export const Accordion = AccordionPrimitive.Root
export const AlertDialog = AlertDialogPrimitive.Root
export const Button = Slot`
const expected = `import { Accordion as AccordionPrimitive, AlertDialog as AlertDialogPrimitive, AspectRatio as AspectRatioPrimitive, Avatar as AvatarPrimitive, Slot as SlotPrimitive, Checkbox as CheckboxPrimitive, Collapsible as CollapsiblePrimitive, ContextMenu as ContextMenuPrimitive, Dialog as DialogPrimitive, DropdownMenu as DropdownMenuPrimitive, HoverCard as HoverCardPrimitive, Label as LabelPrimitive, Menubar as MenubarPrimitive, NavigationMenu as NavigationMenuPrimitive, Popover as PopoverPrimitive, Progress as ProgressPrimitive, RadioGroup as RadioGroupPrimitive, ScrollArea as ScrollAreaPrimitive, Select as SelectPrimitive, Separator as SeparatorPrimitive, Slider as SliderPrimitive, Switch as SwitchPrimitive, Tabs as TabsPrimitive, Toggle as TogglePrimitive, ToggleGroup as ToggleGroupPrimitive, Tooltip as TooltipPrimitive } from "radix-ui";
const expected = `import { Accordion as AccordionPrimitive, AlertDialog as AlertDialogPrimitive, AspectRatio as AspectRatioPrimitive, Avatar as AvatarPrimitive, Slot as SlotPrimitive, Checkbox as CheckboxPrimitive, Collapsible as CollapsiblePrimitive, ContextMenu as ContextMenuPrimitive, Dialog as DialogPrimitive, DropdownMenu as DropdownMenuPrimitive, HoverCard as HoverCardPrimitive, Label as LabelPrimitive, Menubar as MenubarPrimitive, NavigationMenu as NavigationMenuPrimitive, Popover as PopoverPrimitive, Progress as ProgressPrimitive, RadioGroup as RadioGroupPrimitive, ScrollArea as ScrollAreaPrimitive, Select as SelectPrimitive, Separator as SeparatorPrimitive, Slider as SliderPrimitive, Switch as SwitchPrimitive, Tabs as TabsPrimitive, Toggle as TogglePrimitive, ToggleGroup as ToggleGroupPrimitive, Tooltip as TooltipPrimitive } from "radix-ui"
export const Accordion = AccordionPrimitive.Root
export const AlertDialog = AlertDialogPrimitive.Root
@@ -578,7 +578,7 @@ import { cn } from "@/lib/utils"
export const Sheet = SheetPrimitive.Root
export const SheetTrigger = SheetPrimitive.Trigger`
const expected = `import { Dialog as SheetPrimitive } from "radix-ui";
const expected = `import { Dialog as SheetPrimitive } from "radix-ui"
import { cn } from "@/lib/utils"
export const Sheet = SheetPrimitive.Root
@@ -597,7 +597,7 @@ import { Slot } from "@radix-ui/react-slot"
export const FormLabel = LabelPrimitive.Root
export const FormControl = Slot`
const expected = `import { Label as LabelPrimitive, Slot as SlotPrimitive } from "radix-ui";
const expected = `import { Label as LabelPrimitive, Slot as SlotPrimitive } from "radix-ui"
export const FormLabel = LabelPrimitive.Root
export const FormControl = SlotPrimitive.Slot`
@@ -664,7 +664,7 @@ export const FormControl = SlotPrimitive.Slot`
expect(result.replacedPackages.sort()).toEqual(allPackages.sort())
// Should be a single unified import from radix-ui
expect(result.content).toContain('from "radix-ui";')
expect(result.content).toContain('from "radix-ui"')
expect(result.content.startsWith("import {")).toBe(true)
expect(result.content).toContain("Slot as SlotPrimitive") // Slot should be aliased as SlotPrimitive
expect(result.content).toContain("Accordion as AccordionPrimitive") // Namespace should be aliased
@@ -685,7 +685,7 @@ const Button = ({ asChild, children }) => {
return <Comp>{children}</Comp>
}`
const expected = `import { Slot as SlotPrimitive } from "radix-ui";
const expected = `import { Slot as SlotPrimitive } from "radix-ui"
const Button = ({ asChild, children }) => {
const Comp = asChild ? SlotPrimitive.Slot : "button"
@@ -708,7 +708,7 @@ const Button = ({ asChild }) => {
return null
}`
const expected = `import { Slot as SlotPrimitive } from "radix-ui";
const expected = `import { Slot as SlotPrimitive } from "radix-ui"
const Button = ({ asChild }) => {
const Comp1 = asChild ? SlotPrimitive.Slot : "button"
@@ -731,7 +731,7 @@ const Button = ({ asChild }) => {
return null
}`
const expected = `import { Slot as SlotComponent } from "radix-ui";
const expected = `import { Slot as SlotComponent } from "radix-ui"
const Button = ({ asChild }) => {
const Comp = asChild ? Slot : "button"
@@ -754,7 +754,7 @@ const Button = ({ asChild }) => {
return <Slot />
}`
const expected = `import { Slot as SlotPrimitive } from "radix-ui";
const expected = `import { Slot as SlotPrimitive } from "radix-ui"
const Button = ({ asChild }) => {
const SlotName = "Slot"
@@ -781,7 +781,7 @@ const Button = ({ asChild, ...props }: ButtonProps) => {
return <Comp {...props} />
}`
const expected = `import { Slot as SlotPrimitive } from "radix-ui";
const expected = `import { Slot as SlotPrimitive } from "radix-ui"
import React from "react"
type ButtonProps = React.ComponentProps<typeof SlotPrimitive.Slot> & {
@@ -811,7 +811,7 @@ const Button = ({ asChild, ...props }: ButtonProps) => {
return <Comp {...props} />
}`
const expected = `import { Slot as SlotPrimitive } from "radix-ui";
const expected = `import { Slot as SlotPrimitive } from "radix-ui"
import { ComponentProps } from "react"
type ButtonProps = ComponentProps<typeof SlotPrimitive.Slot> & {
@@ -827,6 +827,46 @@ const Button = ({ asChild, ...props }: ButtonProps) => {
expect(result.content.trim()).toBe(expected.trim())
expect(result.replacedPackages).toEqual(["@radix-ui/react-slot"])
})
it("should not add double semicolons when import already ends with semicolon", async () => {
const input = `import * as DialogPrimitive from "@radix-ui/react-dialog";
import * as SelectPrimitive from "@radix-ui/react-select";
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from "radix-ui";
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
const result = await migrateRadixFile(input)
expect(result.content.trim()).toBe(expected.trim())
expect(result.replacedPackages).toEqual([
"@radix-ui/react-dialog",
"@radix-ui/react-select",
])
})
it("should not add semicolon when original imports don't have semicolons", async () => {
const input = `import * as DialogPrimitive from "@radix-ui/react-dialog"
import * as SelectPrimitive from "@radix-ui/react-select"
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
const expected = `import { Dialog as DialogPrimitive, Select as SelectPrimitive } from "radix-ui"
export const Dialog = DialogPrimitive.Root
export const Select = SelectPrimitive.Root`
const result = await migrateRadixFile(input)
expect(result.content.trim()).toBe(expected.trim())
expect(result.replacedPackages).toEqual([
"@radix-ui/react-dialog",
"@radix-ui/react-select",
])
})
})
describe("migrateRadix - package.json updates", () => {

View File

@@ -215,13 +215,15 @@ export async function migrateRadixFile(
content: string
): Promise<{ content: string; replacedPackages: string[] }> {
// Enhanced regex to handle type-only imports, but exclude react-icons
// Also capture optional semicolon at the end
const radixImportPattern =
/import\s+(?:(type)\s+)?(?:\*\s+as\s+(\w+)|{([^}]+)})\s+from\s+(["'])@radix-ui\/react-([^"']+)\4/g
/import\s+(?:(type)\s+)?(?:\*\s+as\s+(\w+)|{([^}]+)})\s+from\s+(["'])@radix-ui\/react-([^"']+)\4(;?)/g
const imports: Array<{ name: string; alias?: string; isType?: boolean }> = []
const linesToRemove: string[] = []
const replacedPackages: string[] = []
let quoteStyle = '"' // Default to double quotes
let hasSemicolon = false // Track if any import had a semicolon
let result = content
let match
@@ -235,6 +237,7 @@ export async function migrateRadixFile(
namedImports,
quote,
packageName,
semicolon,
] = match
// Skip react-icons package and any sub-paths (like react-icons/dist/types)
@@ -244,9 +247,10 @@ export async function migrateRadixFile(
linesToRemove.push(fullMatch)
// Use the quote style from the first import
// Use the quote style and semicolon style from the first import
if (linesToRemove.length === 1) {
quoteStyle = quote
hasSemicolon = semicolon === ";"
}
// Track which package we're replacing
@@ -301,7 +305,9 @@ export async function migrateRadixFile(
})
.join(", ")
const unifiedImport = `import { ${importList} } from ${quoteStyle}radix-ui${quoteStyle};`
const unifiedImport = `import { ${importList} } from ${quoteStyle}radix-ui${quoteStyle}${
hasSemicolon ? ";" : ""
}`
// Replace first import with unified import, remove the rest
result = linesToRemove.reduce((acc, line, index) => {

4
pnpm-lock.yaml generated
View File

@@ -322,7 +322,7 @@ importers:
specifier: ^6.0.1
version: 6.0.1
shadcn:
specifier: 2.6.2
specifier: 2.6.3
version: link:../../packages/shadcn
shiki:
specifier: ^1.10.1
@@ -602,7 +602,7 @@ importers:
specifier: 2.12.7
version: 2.12.7(react-dom@18.3.1(react@18.3.1))(react@18.3.1)
shadcn:
specifier: 2.6.2
specifier: 2.6.3
version: link:../../packages/shadcn
sharp:
specifier: ^0.32.6