refactor(knowledge): make the shell the canonical view

The panel now mounts `KnowledgeView` and imports nothing from the workspace
route tree. That was the last of the four kinds.

The shape of this one differs from tables, and the difference is the useful
part: the route shell owned the *whole* surface — header, breadcrumbs, ten
modals, bulk operations, the context menu — while the extracted `KnowledgeView`
was only the document list. So the shell became the view, and the previous view
became its `DocumentList` child. Promoting the read-only half and leaving the
mutations behind would have left the panel importing the route tree forever.

Dropped `id` from the props: `source` already carries it (`knowledgeResourceId`),
and two spellings of the same identity is exactly what the axes replace.
`knowledgeBaseName` stays, documented as a title hint rather than identity — it
is what keeps both hosts from flashing a placeholder header before load.

Four boundary violations surfaced by `check:resources` once the files were
inside a unit, each fixed at the cause rather than annotated:

- `AddDocumentsModal` read `useParams()` for the workspace id. Inside the panel
  there is no `[workspaceId]` segment, so this was already returning undefined
  there — the modal is reachable from `/home`. Now a required prop.
- `use-connector-config-fields` was route-tree-resident but is unit-internal;
  moved to `@/hooks/kb/`.
- `BreadcrumbItem`/`ResourceAction` came from the route barrel that merely
  re-exports `@/components/resource`. Retargeted to the source.
- The knowledge list page and the document route imported past the unit barrel.
  They render the same tag editor and selection bar against the same data, so
  the barrel exports those three children — and the rule now says when that is
  allowed, because "saves an import hop" must not qualify.

`ConnectOAuthModal` and `WorkspaceHostProvider` looked like they pinned the
connector modals to the route tree. Neither is route-specific: both moved to
`components/` (with `WorkspaceAccessDenied`, which the provider renders), 22
importers repointed. Deleting the blocker beat splitting the surface around it.

nuqs stays outside the unit — R6 has no escape hatch — so
`use-knowledge-list-state` moved to `@/hooks/kb/` and its parsers to
`@/lib/knowledge/detail-search-params`, mirroring the table.

Suite: 20917 passed, unchanged from before the move. R6 and R3c both 0.

Author: waleedlatif1

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

@@ -41,14 +41,21 @@ Every unit has the same layout, so moving between them costs nothing:
 
 ```
 <unit>/
-├── <unit>.tsx                  # THE view — the only component the barrel mounts
+├── <unit>.tsx                  # THE view — what consumers mount
 ├── index.ts                    # the barrel; the only entry point consumers use
 ├── components/<child>/         # <child>.tsx + index.ts, one folder per child
 ├── hooks/<name>.ts             # optional; tests colocated
 ├── utils/<name>.ts             # pure helpers; tests colocated
 └── types.ts                    # optional; shared types the whole unit reads
 ```
 
+A barrel may also export a **child** — but only one a surface outside the unit
+genuinely renders against the same data, where forking it would be the drift the
+unit exists to prevent (`knowledge-view` exports `ActionBar`, `BaseTagsModal` and
+`DocumentTagsModal` for the knowledge list page and the document detail route).
+Exporting a child to save an import hop is not that; the child stays private and
+the consumer mounts the view.
+
 Imports **inside** a unit are absolute (`@/components/resources/<unit>/...`) like
 everywhere else in the app. The one exception is a folder's own `index.ts`
 naming its siblings, which stays relative (`./<child>`) — it is describing its

apps/sim/app/workspace/[workspaceId]/components/invite-modal/invite-modal.test.tsx

@@ -35,7 +35,7 @@ vi.mock('@/lib/auth/auth-client', () => ({
   useSession: () => ({ data: { user: { id: 'user-1', email: 'viewer@example.com' } } }),
 }))
 
-vi.mock('@/app/workspace/[workspaceId]/providers/workspace-host-provider', () => ({
+vi.mock('@/components/workspace-host-provider', () => ({
   useWorkspaceHostContext: () => hostContext.current,
 }))
 

apps/sim/app/workspace/[workspaceId]/components/invite-modal/invite-modal.tsx

@@ -13,13 +13,13 @@ import {
   toast,
 } from '@sim/emcn'
 import { createLogger } from '@sim/logger'
+import { useWorkspaceHostContext } from '@/components/workspace-host-provider'
 import type { BatchInvitationResult } from '@/lib/api/contracts/invitations'
 import { useSession } from '@/lib/auth/auth-client'
 import { isEnterprise } from '@/lib/billing/plan-helpers'
 import { isBillingEnabled } from '@/lib/core/config/env-flags'
 import { quickValidateEmail } from '@/lib/messaging/email/validation'
 import type { PermissionType } from '@/lib/workspaces/permissions/utils'
-import { useWorkspaceHostContext } from '@/app/workspace/[workspaceId]/providers/workspace-host-provider'
 import { useSendWorkspaceInvitations } from '@/hooks/queries/invitations'
 import { useOrganizationBilling } from '@/hooks/queries/organization'
 import { useAdminWorkspaces } from '@/hooks/queries/workspace'

apps/sim/app/workspace/[workspaceId]/home/components/credits-chip/credits-chip.test.tsx

@@ -56,7 +56,7 @@ vi.mock('@/lib/auth/auth-client', () => ({
   useSession: () => ({ data: mockSession.current }),
 }))
 
-vi.mock('@/app/workspace/[workspaceId]/providers/workspace-host-provider', () => ({
+vi.mock('@/components/workspace-host-provider', () => ({
   useWorkspaceHostContext: () => mockHostContext.current,
 }))
 

apps/sim/app/workspace/[workspaceId]/home/components/credits-chip/credits-chip.tsx

@@ -5,12 +5,12 @@ import { Chip, ChipTag, Tooltip } from '@sim/emcn'
 import { Credit } from '@sim/emcn/icons'
 import { useQueryClient } from '@tanstack/react-query'
 import { useParams, useRouter } from 'next/navigation'
+import { useWorkspaceHostContext } from '@/components/workspace-host-provider'
 import { useSession } from '@/lib/auth/auth-client'
 import { formatCredits } from '@/lib/billing/credits/conversion'
 import { buildUpgradeHref } from '@/lib/billing/upgrade-reasons'
 import { canManageWorkspaceBilling } from '@/lib/billing/workspace-permissions'
 import { isBillingEnabled } from '@/lib/core/config/env-flags'
-import { useWorkspaceHostContext } from '@/app/workspace/[workspaceId]/providers/workspace-host-provider'
 import { prefetchWorkspaceSettings } from '@/hooks/queries/workspace'
 import { useWorkspaceCreditAvailability } from '@/hooks/queries/workspace-usage'
 

apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/special-tags/special-tags.tsx

@@ -31,6 +31,7 @@ import {
   type WorkspaceResourceTagType,
 } from '@/components/chat/special-tags/parse'
 import { ThinkingLoader } from '@/components/ui'
+import { useWorkspaceHostContext } from '@/components/workspace-host-provider'
 import { useSession } from '@/lib/auth/auth-client'
 import { buildHostedUpgradeUrl, HOSTED_BILLING_SETTINGS_URL } from '@/lib/billing/upgrade-reasons'
 import { canManageWorkspaceBilling } from '@/lib/billing/workspace-permissions'
@@ -65,7 +66,6 @@ import type {
 // ConnectServiceAccountModal, and that edge would pull the modal into this
 // chunk and defeat the lazy() split below.
 import { useServiceAccountConnectTarget } from '@/app/workspace/[workspaceId]/integrations/components/connect-service-account-modal/use-service-account-connect'
-import { useWorkspaceHostContext } from '@/app/workspace/[workspaceId]/providers/workspace-host-provider'
 import { useUserPermissionsContext } from '@/app/workspace/[workspaceId]/providers/workspace-permissions-provider'
 import { useWorkspaceCredential } from '@/hooks/queries/credentials'
 import {

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

@@ -25,9 +25,11 @@ import {
 import { createLogger } from '@sim/logger'
 import { useRouter } from 'next/navigation'
 import { FileView, type PreviewMode, resolveFileCategory } from '@/components/resources/file-view'
+import { KnowledgeView } from '@/components/resources/knowledge-view'
 import { LogView } from '@/components/resources/log-view'
 import { ResourceEmptyState } from '@/components/resources/resource-empty-state'
 import { TableView } from '@/components/resources/table-view'
+import { useWorkspaceHostContext } from '@/components/workspace-host-provider'
 import { isApiClientError } from '@/lib/api/client/errors'
 import { useSession } from '@/lib/auth/auth-client'
 import { getWorkspaceUsageLimitAction } from '@/lib/billing/workspace-permissions'
@@ -54,8 +56,6 @@ import type {
   MothershipResource,
   MothershipResourceType,
 } from '@/app/workspace/[workspaceId]/home/types'
-import { KnowledgeBase } from '@/app/workspace/[workspaceId]/knowledge/[id]/knowledge-base'
-import { useWorkspaceHostContext } from '@/app/workspace/[workspaceId]/providers/workspace-host-provider'
 import {
   useUserPermissionsContext,
   useWorkspacePermissionsContext,
@@ -381,9 +381,8 @@ export const ResourceContent = memo(function ResourceContent({
 
     case 'knowledgebase':
       return (
-        <KnowledgeBase
+        <KnowledgeView
           key={resource.id}
-          id={resource.id}
           knowledgeBaseName={resource.title}
           source={knowledgeSource}
           grants={grants}

apps/sim/app/workspace/[workspaceId]/home/components/suggested-actions/suggested-actions.tsx

@@ -8,6 +8,7 @@ import { stripVersionSuffix } from '@sim/utils/string'
 import { useParams } from 'next/navigation'
 import { usePostHog } from 'posthog-js/react'
 import { ActionRow } from '@/components/action-row'
+import { ConnectOAuthModal } from '@/components/connect-oauth-modal'
 import { GmailIcon, SlackIcon } from '@/components/icons'
 import {
   INTEGRATIONS,
@@ -16,7 +17,6 @@ import {
   resolveOAuthServiceForSlug,
 } from '@/lib/integrations'
 import { captureEvent } from '@/lib/posthog/client'
-import { ConnectOAuthModal } from '@/app/workspace/[workspaceId]/components/connect-oauth-modal'
 import { getBareIconStyle } from '@/blocks/brand-icon-style'
 import { getAllBlockMeta } from '@/blocks/registry'
 import type { ModuleTag } from '@/blocks/types'

apps/sim/app/workspace/[workspaceId]/integrations/[block]/integration-block-detail.tsx

@@ -5,6 +5,7 @@ import { Chip, ChipDropdown, ChipLink, cn } from '@sim/emcn'
 import { ArrowLeft, Plus } from '@sim/emcn/icons'
 import { useRouter } from 'next/navigation'
 import { useQueryState } from 'nuqs'
+import { ConnectOAuthModal } from '@/components/connect-oauth-modal'
 import { HEADER_ACTION_CLUSTER, PAGE_HEADER_BAR } from '@/components/page-header-bar'
 import { isChatEnabled } from '@/lib/core/config/env-flags'
 import {
@@ -14,7 +15,6 @@ import {
   resolveOAuthServiceForIntegration,
 } from '@/lib/integrations'
 import { credentialProviderMatchesService } from '@/lib/oauth'
-import { ConnectOAuthModal } from '@/app/workspace/[workspaceId]/components/connect-oauth-modal'
 import { RESOURCE_TILE_BASE } from '@/app/workspace/[workspaceId]/components/resource-tile'
 import { IntegrationSkillsSection } from '@/app/workspace/[workspaceId]/integrations/[block]/integration-skills-section'
 import { connectParam } from '@/app/workspace/[workspaceId]/integrations/[block]/search-params'

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

@@ -1,4 +1,3 @@
 export { ChunkContextMenu } from './chunk-context-menu'
 export { ChunkEditor } from './chunk-editor'
 export { DeleteChunkModal } from './delete-chunk-modal'
-export { DocumentTagsModal } from './document-tags-modal'