Skip to content

Commit f37fe21

Browse files
committed
fix(platform): mark the resource chrome as client components
`skills/page.tsx` is a Server Component, and this branch moved its `IntegrationTabsHeader` import onto the `@/app/workspace/[workspaceId]/components` barrel. That barrel re-exports `SortDropdown` from `resource-options`, which calls `useState`, so the server graph now reaches a client-only module and `next build` fails. `resource-header` has the same latent problem (`useState`, `useEffect`, `useRef`). Both files are genuinely client components, so they get the directive rather than the page dropping the barrel import — local feature barrels are the convention here. Also drops a stale `lucide-react` mention now that the dependency is gone.
1 parent 0b7e8d2 commit f37fe21

3 files changed

Lines changed: 5 additions & 2 deletions

File tree

apps/sim/app/workspace/[workspaceId]/components/resource/components/resource-header/resource-header.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
'use client'
2+
13
import {
24
type ComponentType,
35
Fragment,

apps/sim/app/workspace/[workspaceId]/components/resource/components/resource-options/resource-options.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
'use client'
2+
13
import { memo, type ReactNode, useState } from 'react'
24
import * as PopoverPrimitive from '@radix-ui/react-popover'
35
import {

apps/sim/app/workspace/[workspaceId]/settings/components/settings-resource-row/settings-resource-row.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,7 @@ interface SettingsResourceRowProps {
8181
/**
8282
* Appends the canonical navigation chevron after `trailing`. Set it on rows that
8383
* open a detail page; leave it off for rows whose `onClick` performs an action
84-
* in place. The row owns the glyph so callers never pick an arrow themselves —
85-
* `lucide-react` and `@sim/emcn/icons` ship visibly different ones.
84+
* in place. The row owns the glyph so callers never pick an arrow themselves.
8685
*/
8786
navigable?: boolean
8887
/**

0 commit comments

Comments
 (0)