refactor(resources): migrate log and knowledge onto the axes; collapse tab chrome

Continues the resource-view layer. Three kinds now have a canonical view
(file, log, knowledge) and only `table` remains — deliberately, because the
mothership mounts its editing shell, which is a product decision rather than
unfinished work.

**log** — `log-details/` splits into `components/resources/log-view/` (the view)
and a 198-line shell that keeps resize, keyboard nav and close chrome. Four
leaked contexts resolved: `useParams` → `source.workspaceId`, `useRouter` →
an `onNavigate` prop, `useQueryState(tab)` → view-owned state gated by
`hostOwnsUrl(host)`, `usePermissionConfig` → a required `showExecutionInternals`
prop (deliberately not folded into `grants`: it gates *seeing* trace internals,
not write or run, and an optional field would default permissive — the wrong
failure mode for a permission-group restriction). A fifth leak the plan missed —
`file-download.tsx` holding its own `useRouter` — was caught by the gate.

**knowledge** — `base.tsx` (1733 lines) splits into a 909-line canonical view
and a 974-line shell. The read surface (document list, filters, sort,
pagination, unavailable state) is the view; upload, connectors, tag editing,
rename, delete and bulk operations stay in the shell, exactly as the tables
editing grid kept its write path. All six nuqs keys now route through
`useKnowledgeListState(host)`, whose every write sits behind `hostOwnsUrl`.

**tab chrome** — the five `Embedded*Actions` collapse into one kind-keyed
`ResourceTabActions`. They differed only in icon, copy and destination, which is
a config table. `knowledge` and `table` destinations now resolve through
`workspaceSource().hrefFor()` instead of hand-built strings.

**schedule** is removed as a resource kind: its page was deleted upstream, its
`resourceHref` case pointed at a route that no longer exists, and
`MothershipResourceType` never carried it, so it was unreachable.

Behavior change, intended and limited to embedded surfaces: the mothership panel
no longer writes the log `?tab` key or the six knowledge keys into the host
page's address bar. That is the bug the `host` axis exists to fix. Both route
pages keep their deep-linkable params unchanged.

`Resource` and `InlineRenameInput` moved to `components/` — mandatory, since a
canonical unit may not import the workspace route tree. The workspace barrel
re-exports both, so its consumers are byte-unchanged.

Ratchets: shadow-named components 8 → 2 (the 2 left are deliberate —
`EmbeddedWorkflow`, because a workflow is a live collaborative session rather
than a document with an address, and `EmbeddedFolder`, because a folder is
structure inside a resource); cross-tree imports 38 → 37. None raised.

Verification: apps/sim + 22 packages typecheck, biome clean, 18626 tests pass,
and all 15 repo gates pass including both strict variants. Not visually verified.

Author: waleedlatif1

.claude/rules/sim-resource-views.md

@@ -12,7 +12,7 @@ paths:
 
 # Resource Views
 
-A **resource** is a thing a workspace holds that can also be shared: a file, a table, an interface, a knowledge base, a log, a scheduled task. A resource with a canonical view has **exactly one**, and every consumer mounts that one — the workspace route page, the mothership panel, an interface module, and the public share page.
+A **resource** is a thing a workspace holds that can also be shared: a file, a table, an interface, a knowledge base, a log. A resource with a canonical view has **exactly one**, and every consumer mounts that one — the workspace route page, the mothership panel, an interface module, and the public share page.
 
 **One view per resource. Consumers construct the axes and mount it. They never wrap it.**
 
@@ -30,14 +30,14 @@ Enforced by `bun run check:resources` (strict CI gate: `bun run check:resources:
 
 There is no fourth axis. Agent streaming is **one optional prop on `FileView`** (`streaming?: FileViewStreaming`), because only files stream.
 
-`ShareSource` declares `workspaceId?: never` and `resourceId?: never`, and `WorkspaceSource` declares `token?: never` and `seed?: never`. A share source **cannot** carry a workspace id — that is a compile error, not a convention. A kind whose seed is typed `never` (`knowledge`, `log`, `schedule`) structurally cannot construct a share source at all: "no public surface" is a compile-time fact.
+`ShareSource` declares `workspaceId?: never` and `resourceId?: never`, and `WorkspaceSource` declares `token?: never` and `seed?: never`. A share source **cannot** carry a workspace id — that is a compile error, not a convention. A kind whose seed is typed `never` (`table`, `knowledge`, `log`) structurally cannot construct a share source at all: "no public surface" is a compile-time fact.
 
 ```
 apps/sim/resources/                    # kinds.ts · source.ts · grants.ts · host.ts — pure TS
 apps/sim/components/resources/<unit>/  # 'use client' — THE view, one per resource
 ```
 
-A resource kind with no canonical view yet (`table`, `knowledge`, `log`, `schedule`) is simply **absent** from `CANONICAL_UNITS` in the check. That is the correct state for an unmigrated kind. Do not add a flag, a shim, or a placeholder entry for it.
+A resource kind with no canonical view yet (`table`, `knowledge`, `log`) is simply **absent** from `CANONICAL_UNITS` in the check. That is the correct state for an unmigrated kind. Do not add a flag, a shim, or a placeholder entry for it.
 
 ## Consume: construct the axes, then mount
 

CLAUDE.md

@@ -416,15 +416,15 @@ Principles when building or migrating shared UI:
 
 ## Resource Views
 
-A **resource** is a thing a workspace holds that can also be shared — a file, a table, a knowledge base, a log, a scheduled task. A resource with a canonical view has exactly ONE, in `apps/sim/components/resources/<unit>/`, mounted by every consumer: the workspace route page, the mothership panel, the public share page.
+A **resource** is a thing a workspace holds that can also be shared — a file, a table, a knowledge base, a log. A resource with a canonical view has exactly ONE, in `apps/sim/components/resources/<unit>/`, mounted by every consumer: the workspace route page, the mothership panel, the public share page.
 
 Views are mounted against exactly **three axes**, defined in `apps/sim/resources/**` (pure TypeScript — no React, no `'use client'`, because a Server Component builds a share source during SSR):
 
 - `source` — where the data comes from and by what address: `WorkspaceSource<K> | ShareSource<K>`, discriminated on `via`. Replaces `workspaceId`, `token`, `contentSource`, `isPublic`. `ShareSource` declares `workspaceId?: never`, so a share token can no longer be laundered through a workspace-shaped slot.
 - `grants` — what this viewer may do: `{ write, run }`. Replaces `canEdit`, `canRun`, `canAdmin`, `disableEdit/Insert/Delete`.
 - `host` — who owns the URL, the router, the document frame: `'page' | 'panel' | 'public'`. Replaces `embedded`. `hostOwnsUrl(host)` is the one place the "embedded views do not write nuqs keys" rule lives.
 
-There is no fourth axis; agent streaming is one optional prop on `FileView`. Consumers CONSTRUCT the axes and MOUNT the view — never wrap it in a passthrough, never reach past its barrel, never reimplement its UI because it lacks a seam (add the seam), never import `@/app/workspace/[workspaceId]/**` from an anonymous surface (`app/f/**`, `app/(interfaces)/**`), and never read `useRouter`/`useParams`/`useQueryState`/`useUserPermissionsContext` inside a unit. A kind with no canonical view yet (`knowledge`, `log`, `schedule`) is simply absent from the check's `CANONICAL_UNITS` — no flag, shim, or placeholder.
+There is no fourth axis; agent streaming is one optional prop on `FileView`. Consumers CONSTRUCT the axes and MOUNT the view — never wrap it in a passthrough, never reach past its barrel, never reimplement its UI because it lacks a seam (add the seam), never import `@/app/workspace/[workspaceId]/**` from an anonymous surface (`app/f/**`, `app/(interfaces)/**`), and never read `useRouter`/`useParams`/`useQueryState`/`useUserPermissionsContext` inside a unit. A kind with no canonical view yet (`knowledge`, `log`) is simply absent from the check's `CANONICAL_UNITS` — no flag, shim, or placeholder.
 
 Enforced by `bun run check:resources` (strict: `check:resources:strict`), which ratchets counters for wrappers, imports past a barrel, cross-tree imports, unsanctioned props, token-as-`workspaceId`, and context leaks. Escape hatches — reason mandatory, on the line directly above: `// boundary-resource-wrapper:`, `// boundary-resource-internal:`, `// boundary-resource-tree:`, `// boundary-resource-prop:`. Full rules in `.claude/rules/sim-resource-views.md`.
 

apps/sim/app/_shell/desktop-title-bar-surfaces.test.ts

@@ -40,7 +40,7 @@ const logoShell = read('../(landing)/components/logo-shell/logo-shell.tsx')
 const pageHeaderBar = read('../../components/page-header-bar.ts')
 const settingsHeader = read('../../components/settings/settings-header.tsx')
 const resourceHeader = read(
-  '../workspace/[workspaceId]/components/resource/components/resource-header/resource-header.tsx'
+  '../../components/resource/components/resource-header/resource-header.tsx'
 )
 const mothershipView = read(
   '../workspace/[workspaceId]/home/components/mothership-view/mothership-view.tsx'

apps/sim/app/workspace/[workspaceId]/components/folders/folder-breadcrumbs.ts

@@ -3,7 +3,7 @@ import type {
   BreadcrumbEditing,
   BreadcrumbItem,
   DropdownOption,
-} from '@/app/workspace/[workspaceId]/components/resource/components/resource-header'
+} from '@/components/resource/components/resource-header'
 import type { WorkflowFolder } from '@/stores/folders/types'
 
 export interface FolderBreadcrumbItemsOptions {

apps/sim/app/workspace/[workspaceId]/components/folders/folder-row.tsx

@@ -1,9 +1,6 @@
 import { Folder } from '@sim/emcn/icons'
+import type { ResourceCell, ResourceRow } from '@/components/resource/resource'
 import { folderRowId } from '@/app/workspace/[workspaceId]/components/folders/folder-row-id'
-import type {
-  ResourceCell,
-  ResourceRow,
-} from '@/app/workspace/[workspaceId]/components/resource/resource'
 import type { WorkflowFolder } from '@/stores/folders/types'
 
 const FOLDER_ICON = <Folder className='size-[14px]' />

apps/sim/app/workspace/[workspaceId]/components/folders/use-folder-row-drag-drop.ts

@@ -1,8 +1,8 @@
 'use client'
 
 import { type DragEvent, useCallback, useEffect, useMemo, useRef, useState } from 'react'
+import type { RowDragDropConfig } from '@/components/resource/resource'
 import { parseFolderedRowId } from '@/app/workspace/[workspaceId]/components/folders/folder-row-id'
-import type { RowDragDropConfig } from '@/app/workspace/[workspaceId]/components/resource/resource'
 
 /**
  * Private drag payload, namespaced so a drag started on another Sim surface (or an external

apps/sim/app/workspace/[workspaceId]/components/index.ts

@@ -1,47 +1,47 @@
-export { ConversationListItem } from './conversation-list-item'
-export type { ErrorBoundaryProps, ErrorStateProps } from './error'
-export { ErrorShell, ErrorState } from './error'
-export { InlineRenameInput } from './inline-rename-input'
-export { IntegrationTabsHeader } from './integration-tabs-header'
-export { MessageActions } from './message-actions'
-export { FloatingOverflowText } from './resource/components/floating-overflow-text'
+/**
+ * `Resource` and `InlineRenameInput` are re-exported from `@/components/**`:
+ * they moved out of this tree because the canonical resource views under
+ * `components/resources/**` mount them too, and a shared unit may not import
+ * `@/app/workspace/[workspaceId]/**`. The workspace tree keeps reaching them
+ * through this barrel, which stays its single aggregated entry point.
+ */
+export { InlineRenameInput } from '@/components/inline-rename-input'
 export {
+  type BreadcrumbEditing,
+  type BreadcrumbItem,
+  type ChromeActionSpec,
+  type ColumnOption,
+  type DropdownOption,
+  EMPTY_CELL_PLACEHOLDER,
+  type FilterConfig,
+  type FilterTag,
+  FloatingOverflowText,
   type MemberFilterOption,
   memberFilterOptions,
   ownerCell,
-} from './resource/components/owner-cell'
-export {
-  type ChromeActionSpec,
+  type PaginationConfig,
+  Resource,
+  type ResourceAction,
+  type ResourceCell,
+  type ResourceCellEditing,
   ResourceChromeFallback,
-} from './resource/components/resource-chrome-fallback'
-export type {
-  BreadcrumbEditing,
-  BreadcrumbItem,
-  DropdownOption,
-  ResourceAction,
-} from './resource/components/resource-header'
-export type {
-  ColumnOption,
-  FilterConfig,
-  FilterTag,
-  SearchConfig,
-  SearchTag,
-  SortConfig,
-} from './resource/components/resource-options'
-export { SortDropdown } from './resource/components/resource-options'
-export { timeCell } from './resource/components/time-cell'
-export type {
-  PaginationConfig,
-  ResourceCell,
-  ResourceCellEditing,
-  ResourceColumn,
-  ResourceRow,
-  ResourceTableHandle,
-  RowDragDropConfig,
-  SelectableConfig,
-} from './resource/resource'
-export { EMPTY_CELL_PLACEHOLDER, Resource } from './resource/resource'
-export { useBackgroundContextMenu } from './resource/use-background-context-menu'
+  type ResourceColumn,
+  type ResourceRow,
+  type ResourceTableHandle,
+  type RowDragDropConfig,
+  type SearchConfig,
+  type SearchTag,
+  type SelectableConfig,
+  type SortConfig,
+  SortDropdown,
+  timeCell,
+  useBackgroundContextMenu,
+} from '@/components/resource'
+export { ConversationListItem } from './conversation-list-item'
+export type { ErrorBoundaryProps, ErrorStateProps } from './error'
+export { ErrorShell, ErrorState } from './error'
+export { IntegrationTabsHeader } from './integration-tabs-header'
+export { MessageActions } from './message-actions'
 export { ResourceTile } from './resource-tile'
 export { ShareModal, type ShareModalProps } from './share-modal'
 export { SkillTile } from './skill-tile'

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/add-resource-dropdown/add-resource-dropdown.tsx

@@ -16,6 +16,7 @@ import {
   Tooltip,
 } from '@sim/emcn'
 import { Folder, Plus } from '@sim/emcn/icons'
+import { formatDate } from '@/components/resources/log-view'
 import { isBrowserAgentAvailable } from '@/lib/browser-agent/transport'
 import {
   BROWSER_SESSION_RESOURCE_ID,
@@ -36,7 +37,6 @@ import type {
   MothershipResource,
   MothershipResourceType,
 } from '@/app/workspace/[workspaceId]/home/types'
-import { formatDate } from '@/app/workspace/[workspaceId]/logs/utils'
 import { listIntegrations } from '@/blocks/integration-matcher'
 import { useFolders } from '@/hooks/queries/folders'
 import { useKnowledgeBasesQuery } from '@/hooks/queries/kb/knowledge'

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/index.ts

@@ -1,5 +1,5 @@
 export { AddResourceDropdown, useAvailableResources } from './add-resource-dropdown'
-export { ResourceActions, ResourceContent } from './resource-content'
+export { ResourceContent, ResourceTabActions } from './resource-content'
 export type { ResourceTypeConfig } from './resource-registry'
 export {
   getResourceConfig,

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/index.ts

@@ -1 +1 @@
-export { ResourceActions, ResourceContent } from './resource-content'
+export { ResourceContent, ResourceTabActions } from './resource-content'