@kelpie/ui 0.3.0 → 0.4.0

package/dist/registry/visibleNav.js

@@ -0,0 +1,27 @@
+import { useModuleSettings } from '../api/resources/moduleSettings.js';
+import { inSlotOrder } from './registry.js';
+/**
+ * Core's nav items merged with a slot's module contributions, ordered, with the
+ * entries belonging to a disabled module removed.
+ *
+ * One hook rather than the rule written out at each nav surface. The shell's
+ * sidebar and the account tab strip both render a slot, and a workspace that
+ * switched a module off should stop seeing it in both. Two copies of this
+ * filter is exactly the sort of pair that drifts, and the symptom is a tab that
+ * survives its module being disabled and leads to a page answering 403.
+ */
+/**
+ * A nav item whose id does not match its module id. Everything else looks itself
+ * up directly, which is what keeps a toggleable module's own nav entry working
+ * with no change here when one is added later.
+ */
+const NAV_ID_TO_MODULE_ID = {
+    fundraising: 'raises',
+    data: 'import-export',
+};
+export function useVisibleNavItems(coreItems, moduleItems) {
+    const { settings } = useModuleSettings();
+    const disabledModuleIds = new Set(settings.filter((setting) => !setting.enabled).map((setting) => setting.moduleId));
+    return inSlotOrder([...coreItems, ...moduleItems]).filter((item) => !disabledModuleIds.has(NAV_ID_TO_MODULE_ID[item.id] ?? item.id));
+}
+//# sourceMappingURL=visibleNav.js.map

package/dist/registry/visibleNav.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"visibleNav.js","sourceRoot":"","sources":["../../src/registry/visibleNav.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,oCAAoC,CAAA;AAEtE,OAAO,EAAE,WAAW,EAAE,MAAM,eAAe,CAAA;AAE3C;;;;;;;;;GASG;AAEH;;;;GAIG;AACH,MAAM,mBAAmB,GAAqC;IAC5D,WAAW,EAAE,QAAQ;IACrB,IAAI,EAAE,eAAe;CACtB,CAAA;AAED,MAAM,UAAU,kBAAkB,CAChC,SAA6B,EAC7B,WAA+B;IAE/B,MAAM,EAAE,QAAQ,EAAE,GAAG,iBAAiB,EAAE,CAAA;IACxC,MAAM,iBAAiB,GAAG,IAAI,GAAG,CAC/B,QAAQ,CAAC,MAAM,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,CAAC,CAClF,CAAA;IAED,OAAO,WAAW,CAAC,CAAC,GAAG,SAAS,EAAE,GAAG,WAAW,CAAC,CAAC,CAAC,MAAM,CACvD,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,iBAAiB,CAAC,GAAG,CAAC,mBAAmB,CAAC,IAAI,CAAC,EAAE,CAAC,IAAI,IAAI,CAAC,EAAE,CAAC,CAC1E,CAAA;AACH,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kelpie/ui",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "React UI for Kelpie: components, pages, API client, and the extension registry an assembly composes modules through.",
   "keywords": [
     "kelpie",
@@ -48,7 +48,7 @@
     "@dnd-kit/core": "^6.3.1",
     "@dnd-kit/sortable": "^10.0.0",
     "@dnd-kit/utilities": "^3.2.2",
-    "@kelpie/schemas": "^0.3.0",
+    "@kelpie/schemas": "^0.4.0",
     "@tanstack/react-query": "^5.101.4",
     "react-markdown": "^10.1.0",
     "react-router": "^8.3.0",

package/src/api/ApiProvider.tsx

@@ -10,10 +10,10 @@ import { createQueryClient } from './queryClient.ts'
 
 export interface ApiProviderProps {
   /** Origin plus base path, e.g. `/v1` in the browser. */
-  readonly baseUrl?: string
+  readonly baseUrl?: string | undefined
   /** A prepared client, for tests and for assemblies that construct their own. */
-  readonly client?: ApiClient
-  readonly queryClient?: QueryClient
+  readonly client?: ApiClient | undefined
+  readonly queryClient?: QueryClient | undefined
   readonly children: ReactNode
 }
 

package/src/api/client.ts

@@ -173,7 +173,10 @@ export function createApiClient(options: ApiClientOptions): ApiClient {
       method,
       credentials: 'same-origin',
       headers: headersFor(body),
-      body: body === undefined ? undefined : JSON.stringify(body),
+      // Spread rather than `body: undefined`. `RequestInit` comes from the DOM
+      // library, where `body` is optional and does not accept an explicit
+      // `undefined`, and it is not ours to widen. A GET carries no body at all.
+      ...(body === undefined ? {} : { body: JSON.stringify(body) }),
     })
   }
 

package/src/api/resources/agentTasks.ts

@@ -106,8 +106,8 @@ const runs = createReadOnlyResourceHooks<AgentRun>({
 })
 
 export interface AgentRunFilters {
-  readonly agentId?: string
-  readonly status?: AgentRunStatus
+  readonly agentId?: string | undefined
+  readonly status?: AgentRunStatus | undefined
 }
 
 export function useAgentRuns(

package/src/api/resources/candidates.ts

@@ -32,12 +32,12 @@ const candidates = createResourceHooks<Candidate, CreateCandidateInput, Candidat
 /** The documented filters on `GET /v1/candidates`. */
 export interface CandidateFilters {
   /** Candidacies on any of these roles. Repeats on the wire. */
-  readonly roleIds?: readonly string[]
+  readonly roleIds?: readonly string[] | undefined
   /** Candidacies held by any of these people. Repeats on the wire. */
-  readonly personIds?: readonly string[]
+  readonly personIds?: readonly string[] | undefined
   /** Candidacies in any of these pipeline states. Repeats on the wire. */
-  readonly statuses?: readonly CandidateStatus[]
-  readonly limit?: number
+  readonly statuses?: readonly CandidateStatus[] | undefined
+  readonly limit?: number | undefined
 }
 
 function candidateQuery(filters: CandidateFilters): QueryParameters {

package/src/api/resources/companies.ts

@@ -31,12 +31,12 @@ const companies = createResourceHooks<Company, CreateCompanyInput, CompanyInput>
 /** The documented filters on `GET /v1/companies`. */
 export interface CompanyFilters {
   /** Matches name, domain, industry, summary, account type, and tags. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** Companies where any of these people holds a position. Repeats on the wire. */
-  readonly personIds?: readonly string[]
-  readonly limit?: number
+  readonly personIds?: readonly string[] | undefined
+  readonly limit?: number | undefined
   /** `field` ascending, `-field` descending. Only `name`, `created_at`, `updated_at` are sortable. */
-  readonly sort?: string
+  readonly sort?: string | undefined
 }
 
 function companyQuery(filters: CompanyFilters): QueryParameters {

package/src/api/resources/deals.ts

@@ -35,16 +35,16 @@ const deals = createResourceHooks<Deal, CreateDealInput, DealInput>({
 /** The documented filters on `GET /v1/deals`. */
 export interface DealFilters {
   /** Matches the deal's name, summary, tags, competitors, and its company's name. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** Deals at any of these companies. Repeats on the wire. */
-  readonly companyIds?: readonly string[]
+  readonly companyIds?: readonly string[] | undefined
   /** Deals sitting in any of these stages. Repeats on the wire. */
-  readonly stageIds?: readonly string[]
+  readonly stageIds?: readonly string[] | undefined
   /** Deals any of these people are on. Repeats on the wire. */
-  readonly personIds?: readonly string[]
-  readonly limit?: number
+  readonly personIds?: readonly string[] | undefined
+  readonly limit?: number | undefined
   /** `field` ascending, `-field` descending. Only `name`, `created_at`, `updated_at` are sortable. */
-  readonly sort?: string
+  readonly sort?: string | undefined
 }
 
 function dealQuery(filters: DealFilters): QueryParameters {

package/src/api/resources/decisions.ts

@@ -29,10 +29,10 @@ const decisions = createResourceHooks<Decision, CreateDecisionInput, DecisionInp
 })
 
 export interface DecisionFilters {
-  readonly targetType?: RecordTargetType
-  readonly targetId?: string
+  readonly targetType?: RecordTargetType | undefined
+  readonly targetId?: string | undefined
   /** Matches the body, the rationale, the target type, and the target's name. */
-  readonly term?: string
+  readonly term?: string | undefined
 }
 
 function decisionQuery(filters: DecisionFilters): QueryParameters {

package/src/api/resources/forms.ts

@@ -60,10 +60,10 @@ const forms = createResourceHooks<Form, CreateFormInput, FormSettingsInput>({
 
 export interface FormFilters {
   /** Matches a form's name and its description. */
-  readonly term?: string
-  readonly status?: 'active' | 'paused'
+  readonly term?: string | undefined
+  readonly status?: 'active' | 'paused' | undefined
   /** `field` ascending, `-field` descending. Only `name`, `created_at`, `updated_at` are sortable. */
-  readonly sort?: string
+  readonly sort?: string | undefined
 }
 
 function formQuery(filters: FormFilters): QueryParameters {

package/src/api/resources/handbookPages.ts

@@ -26,10 +26,10 @@ const pages = createResourceHooks<HandbookPage, CreateHandbookPageInput, Handboo
 /** The documented filters on `GET /v1/handbook_pages`. */
 export interface HandbookPageFilters {
   /** Matches a page's title and its body. A handbook is searched for what it says. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** Pages at these slugs. Repeats on the wire. */
-  readonly slugs?: readonly string[]
-  readonly limit?: number
+  readonly slugs?: readonly string[] | undefined
+  readonly limit?: number | undefined
 }
 
 function handbookQuery(filters: HandbookPageFilters): QueryParameters {

package/src/api/resources/notes.ts

@@ -32,8 +32,8 @@ export interface NoteFilters {
   readonly targetType: RecordTargetType
   /** Notes on any of these records. Repeats on the wire; all one target type. */
   readonly targetIds: readonly string[]
-  readonly pinned?: boolean
-  readonly limit?: number
+  readonly pinned?: boolean | undefined
+  readonly limit?: number | undefined
 }
 
 function noteQuery(filters: NoteFilters): QueryParameters {

package/src/api/resources/opportunities.ts

@@ -34,16 +34,16 @@ const opportunities = createResourceHooks<Opportunity, CreateOpportunityInput, O
 /** The documented filters on `GET /v1/opportunities`. */
 export interface OpportunityFilters {
   /** Matches the opportunity's name, kind, summary, tags, and its company's name. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** Exact kinds ("Grant", "Accelerator", …). Repeats on the wire. */
-  readonly kinds?: readonly string[]
+  readonly kinds?: readonly string[] | undefined
   /** Opportunities at any of these companies. Repeats on the wire. */
-  readonly companyIds?: readonly string[]
+  readonly companyIds?: readonly string[] | undefined
   /** Opportunities sitting in any of these stages. Repeats on the wire. */
-  readonly stageIds?: readonly string[]
-  readonly limit?: number
+  readonly stageIds?: readonly string[] | undefined
+  readonly limit?: number | undefined
   /** `field` ascending, `-field` descending. Only `name`, `created_at`, `updated_at` are sortable. */
-  readonly sort?: string
+  readonly sort?: string | undefined
 }
 
 function opportunityQuery(filters: OpportunityFilters): QueryParameters {

package/src/api/resources/partnerships.ts

@@ -36,18 +36,18 @@ const partnerships = createResourceHooks<Partnership, CreatePartnershipInput, Pa
 /** The documented filters on `GET /v1/partnerships`. */
 export interface PartnershipFilters {
   /** Matches the partnership's name, kind, summary, tags, and its company's name. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** Exact kinds ("Integration", "Co-marketing", …). Repeats on the wire. */
-  readonly kinds?: readonly string[]
+  readonly kinds?: readonly string[] | undefined
   /** Partnerships with any of these companies. Repeats on the wire. */
-  readonly companyIds?: readonly string[]
+  readonly companyIds?: readonly string[] | undefined
   /** Partnerships sitting in any of these stages. Repeats on the wire. */
-  readonly stageIds?: readonly string[]
+  readonly stageIds?: readonly string[] | undefined
   /** Partnerships any of these people are on. Repeats on the wire. */
-  readonly personIds?: readonly string[]
-  readonly limit?: number
+  readonly personIds?: readonly string[] | undefined
+  readonly limit?: number | undefined
   /** `field` ascending, `-field` descending. Only `name`, `created_at`, `updated_at` are sortable. */
-  readonly sort?: string
+  readonly sort?: string | undefined
 }
 
 function partnershipQuery(filters: PartnershipFilters): QueryParameters {

package/src/api/resources/people.ts

@@ -31,12 +31,12 @@ const people = createResourceHooks<Person, CreatePersonInput, PersonInput>({
 /** The documented filters on `GET /v1/people`. There is no generic filter DSL. */
 export interface PeopleFilters {
   /** Matches name, email, summary, tags, and the titles and companies they hold. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** People holding a position at any of these companies. Repeats on the wire. */
-  readonly companyIds?: readonly string[]
-  readonly limit?: number
+  readonly companyIds?: readonly string[] | undefined
+  readonly limit?: number | undefined
   /** `field` ascending, `-field` descending. Only `name`, `created_at`, `updated_at` are sortable. */
-  readonly sort?: string
+  readonly sort?: string | undefined
 }
 
 function peopleQuery(filters: PeopleFilters): QueryParameters {

package/src/api/resources/planItems.ts

@@ -35,15 +35,15 @@ const planItems = createResourceHooks<PlanItem, CreatePlanItemInput, PlanItemInp
 export const MAX_PAGE_SIZE = 200
 
 export interface PlanItemFilters {
-  readonly targetType?: PipelineKind
+  readonly targetType?: PipelineKind | undefined
   /** The records to read steps for. Repeats on the wire: `?target_id=a&target_id=b`. */
-  readonly targetIds?: readonly string[]
-  readonly statuses?: readonly PlanItemStatus[]
+  readonly targetIds?: readonly string[] | undefined
+  readonly statuses?: readonly PlanItemStatus[] | undefined
   /** Inclusive `YYYY-MM-DD` bounds. The calendar asks for one month with the pair. */
-  readonly from?: string
-  readonly to?: string
+  readonly from?: string | undefined
+  readonly to?: string | undefined
   /** Page size. Worth setting only when the filter already bounds the answer. */
-  readonly limit?: number
+  readonly limit?: number | undefined
 }
 
 function planItemQuery(filters: PlanItemFilters): QueryParameters {

package/src/api/resources/positions.ts

@@ -37,10 +37,10 @@ const positions = createResourceHooks<Position, CreatePositionInput, PositionTit
 
 export interface PositionFilters {
   /** Positions held by any of these people. Repeats on the wire. */
-  readonly personIds?: readonly string[]
+  readonly personIds?: readonly string[] | undefined
   /** Positions held at any of these companies. Repeats on the wire. */
-  readonly companyIds?: readonly string[]
-  readonly limit?: number
+  readonly companyIds?: readonly string[] | undefined
+  readonly limit?: number | undefined
 }
 
 function positionQuery(filters: PositionFilters): QueryParameters {

package/src/api/resources/raises.ts

@@ -35,16 +35,16 @@ const raises = createResourceHooks<Raise, CreateRaiseInput, RaiseInput>({
 /** The documented filters on `GET /v1/raises`. */
 export interface RaiseFilters {
   /** Matches the raise's name, summary, tags, and its firm's name. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** Raises with any of these firms. Repeats on the wire. */
-  readonly companyIds?: readonly string[]
+  readonly companyIds?: readonly string[] | undefined
   /** Raises sitting in any of these stages. Repeats on the wire. */
-  readonly stageIds?: readonly string[]
+  readonly stageIds?: readonly string[] | undefined
   /** Raises any of these people are key on. Repeats on the wire. */
-  readonly personIds?: readonly string[]
-  readonly limit?: number
+  readonly personIds?: readonly string[] | undefined
+  readonly limit?: number | undefined
   /** `field` ascending, `-field` descending. Only `name`, `created_at`, `updated_at` are sortable. */
-  readonly sort?: string
+  readonly sort?: string | undefined
 }
 
 function raiseQuery(filters: RaiseFilters): QueryParameters {

package/src/api/resources/roles.ts

@@ -31,10 +31,10 @@ const roles = createResourceHooks<Role, CreateRoleInput, RoleInput>({
 /** The documented filters on `GET /v1/roles`. */
 export interface RoleFilters {
   /** Matches the role's title, which is what the mockup's filter box matches. */
-  readonly term?: string
+  readonly term?: string | undefined
   /** Roles in any of these statuses. Repeats on the wire. */
-  readonly statuses?: readonly RoleStatus[]
-  readonly limit?: number
+  readonly statuses?: readonly RoleStatus[] | undefined
+  readonly limit?: number | undefined
 }
 
 function roleQuery(filters: RoleFilters): QueryParameters {

package/src/api/resources/webhooks.ts

@@ -41,7 +41,7 @@ const webhooks = createResourceHooks<Webhook, CreateWebhookInput, WebhookInput>(
 })
 
 export interface WebhookFilters {
-  readonly status?: Webhook['status']
+  readonly status?: Webhook['status'] | undefined
 }
 
 function webhookQuery(filters: WebhookFilters): QueryParameters {
@@ -61,7 +61,7 @@ export function useDeleteWebhook(): MutationResult<string, void> {
 }
 
 export interface WebhookDeliveryFilters {
-  readonly status?: WebhookDelivery['status']
+  readonly status?: WebhookDelivery['status'] | undefined
 }
 
 /**

package/src/app/KelpieApp.tsx

@@ -47,6 +47,7 @@ import { HandbookStepPage } from '../pages/onboarding/HandbookStepPage.tsx'
 import { InvitesStepPage } from '../pages/onboarding/InvitesStepPage.tsx'
 import { WorkspaceStepPage } from '../pages/onboarding/WorkspaceStepPage.tsx'
 import { UiExtensionProvider } from '../registry/UiExtensionProvider.tsx'
+import type { RouteContribution } from '../registry/contributions.ts'
 import { useModuleRoutes } from '../registry/context.ts'
 import type { UiExtensions } from '../registry/registry.ts'
 import { SessionGate } from './SessionGate.tsx'
@@ -80,8 +81,25 @@ export function KelpieApp({ extensions, baseUrl }: KelpieAppProps): React.JSX.El
   )
 }
 
+/**
+ * A module route under `account/` renders inside `AccountLayout`, everything
+ * else directly under the shell.
+ *
+ * The prefix is the whole rule, and it is what makes `nav.account` mean
+ * anything: a module contributing a tab there needs its page to keep the tab
+ * strip, and a route mounted as a sibling of `account` would lose it the moment
+ * the tab was clicked. Core's own account pages are nested for the same reason.
+ */
+const ACCOUNT_PREFIX = 'account/'
+
+function isAccountRoute(route: RouteContribution): boolean {
+  return route.path.startsWith(ACCOUNT_PREFIX)
+}
+
 function AppRoutes(): React.JSX.Element {
   const moduleRoutes = useModuleRoutes()
+  const accountRoutes = moduleRoutes.filter(isAccountRoute)
+  const shellRoutes = moduleRoutes.filter((route) => !isAccountRoute(route))
 
   return (
     <Routes>
@@ -146,8 +164,15 @@ function AppRoutes(): React.JSX.Element {
             <Route path="profile" element={<ProfilePage />} />
             <Route path="security" element={<SecurityPage />} />
             <Route path="preferences" element={<PreferencesPage />} />
+            {accountRoutes.map((route) => (
+              <Route
+                key={route.path}
+                path={route.path.slice(ACCOUNT_PREFIX.length)}
+                element={route.element}
+              />
+            ))}
           </Route>
-          {moduleRoutes.map((route) => (
+          {shellRoutes.map((route) => (
             <Route key={route.path} path={route.path} element={route.element} />
           ))}
           {/* Last so every named route above wins first. Nested here rather than

package/src/components/DataTable.tsx

@@ -3,10 +3,10 @@ import type { ReactNode } from 'react'
 export interface Column<TRow> {
   readonly key: string
   readonly header: string
-  readonly className?: string
+  readonly className?: string | undefined
   readonly render: (row: TRow) => ReactNode
   /** The `?sort=` field this column drives. Omit for a column with no server-side sort. */
-  readonly sortKey?: string
+  readonly sortKey?: string | undefined
 }
 
 export interface DataTableGroup<TRow> {
@@ -22,19 +22,19 @@ export interface EmptyStateAction {
 
 export interface DataTableProps<TRow> {
   readonly columns: readonly Column<TRow>[]
-  readonly rows?: readonly TRow[]
-  readonly groups?: readonly DataTableGroup<TRow>[]
-  readonly onRowClick?: (row: TRow) => void
-  readonly emptyMessage?: string
+  readonly rows?: readonly TRow[] | undefined
+  readonly groups?: readonly DataTableGroup<TRow>[] | undefined
+  readonly onRowClick?: ((row: TRow) => void) | undefined
+  readonly emptyMessage?: string | undefined
   /** Replaces the default "Add a record to get started." line under `emptyMessage`. */
-  readonly emptyDescription?: string
+  readonly emptyDescription?: string | undefined
   /** A button rendered in the empty state. Omit when there is nothing to do about it — a filter miss, not a genuinely empty list. */
-  readonly emptyAction?: EmptyStateAction
+  readonly emptyAction?: EmptyStateAction | undefined
   readonly getRowId: (row: TRow) => string
   /** Current `?sort=` value: `field` ascending, `-field` descending, `undefined` for the resource's default. */
-  readonly sort?: string
+  readonly sort?: string | undefined
   /** Fires with the next `?sort=` value when a sortable header is clicked. Required for any column to be clickable. */
-  readonly onSortChange?: (sort: string | undefined) => void
+  readonly onSortChange?: ((sort: string | undefined) => void) | undefined
 }
 
 type SortDirection = 'asc' | 'desc'
@@ -162,7 +162,7 @@ function GroupRows<TRow>({
 }: {
   readonly group: DataTableGroup<TRow>
   readonly columns: readonly Column<TRow>[]
-  readonly onRowClick?: (row: TRow) => void
+  readonly onRowClick?: ((row: TRow) => void) | undefined
   readonly getRowId: (row: TRow) => string
 }): React.JSX.Element {
   return (
@@ -189,7 +189,7 @@ function DataRow<TRow>({
 }: {
   readonly row: TRow
   readonly columns: readonly Column<TRow>[]
-  readonly onRowClick?: (row: TRow) => void
+  readonly onRowClick?: ((row: TRow) => void) | undefined
 }): React.JSX.Element {
   return (
     <tr

package/src/components/EntitySearch.tsx

@@ -13,7 +13,7 @@ import { useEffect, useId, useMemo, useRef, useState } from 'react'
 export interface SearchOption {
   readonly id: string
   readonly label: string
-  readonly meta?: string
+  readonly meta?: string | undefined
 }
 
 export interface EntitySearchProps {
@@ -21,17 +21,17 @@ export interface EntitySearchProps {
   readonly value: string
   readonly onChange: (id: string) => void
   /** Set to run the search server-side. Receives what has been typed, trimmed of nothing. */
-  readonly onQueryChange?: (query: string) => void
+  readonly onQueryChange?: ((query: string) => void) | undefined
   /** When set, typing a name that is not an exact match offers a create row. */
-  readonly onCreate?: (label: string) => void
-  readonly createLabel?: (query: string) => string
-  readonly placeholder?: string
-  readonly emptyMessage?: string
+  readonly onCreate?: ((label: string) => void) | undefined
+  readonly createLabel?: ((query: string) => string) | undefined
+  readonly placeholder?: string | undefined
+  readonly emptyMessage?: string | undefined
   /** Most rows shown at once. The list says so when there are more. */
-  readonly limit?: number
-  readonly required?: boolean
-  readonly className?: string
-  readonly size?: 'sm' | 'md'
+  readonly limit?: number | undefined
+  readonly required?: boolean | undefined
+  readonly className?: string | undefined
+  readonly size?: 'sm' | 'md' | undefined
 }
 
 export function EntitySearch({

package/src/components/KanbanBoard.tsx

@@ -28,8 +28,8 @@ export interface KanbanCard {
   readonly id: string
   readonly stage: string
   readonly title: string
-  readonly meta?: string
-  readonly valueLabel?: string
+  readonly meta?: string | undefined
+  readonly valueLabel?: string | undefined
   readonly href: string
 }
 

package/src/components/Shell.tsx

@@ -2,13 +2,12 @@ import { useEffect, useRef, useState } from 'react'
 import { NavLink, Outlet, useLocation, useNavigate } from 'react-router'
 
 import { useAccount, useTheme } from '../api/resources/account.ts'
-import { useModuleSettings } from '../api/resources/moduleSettings.ts'
 import { useLogOut } from '../api/resources/session.ts'
 import { initialsOf } from '../lib/names.ts'
 import type { ThemePreference } from '../lib/theme.ts'
 import { useNavItems } from '../registry/context.ts'
 import type { NavItem } from '../registry/contributions.ts'
-import { inSlotOrder } from '../registry/registry.ts'
+import { useVisibleNavItems } from '../registry/visibleNav.ts'
 import { ErrorBoundary } from './ErrorBoundary.tsx'
 
 /**
@@ -51,16 +50,6 @@ const CORE_NAV: readonly NavItem[] = [
  */
 const TOP_LEVEL_NAV_IDS: ReadonlySet<string> = new Set(['dashboard', 'handbook', 'planning', 'decisions'])
 
-/**
- * A nav item whose id does not match its module id. Everything else looks
- * itself up directly, which is what keeps a toggleable module's own nav entry
- * working with no change here when one is added later.
- */
-const NAV_ID_TO_MODULE_ID: Readonly<Record<string, string>> = {
-  fundraising: 'raises',
-  data: 'import-export',
-}
-
 function linkClass({ isActive }: { isActive: boolean }): string {
   return [
     'block rounded-md px-2 py-1 text-[13px] transition-colors duration-100',
@@ -89,12 +78,6 @@ export function Shell(): React.JSX.Element {
   const logOut = useLogOut()
   const moduleNav = useNavItems('primary')
   const moduleAdminNav = useNavItems('admin')
-  const { settings } = useModuleSettings()
-  const disabledModuleIds = new Set(
-    settings.filter((setting) => !setting.enabled).map((setting) => setting.moduleId),
-  )
-  const isModuleHidden = (navId: string): boolean =>
-    disabledModuleIds.has(NAV_ID_TO_MODULE_ID[navId] ?? navId)
   const [menuOpen, setMenuOpen] = useState(false)
   // The same preference the account page writes, so this button and that page
   // cannot disagree about which theme the account is on.
@@ -128,12 +111,10 @@ export function Shell(): React.JSX.Element {
     }
   }, [menuOpen])
 
-  const navItems = inSlotOrder([...CORE_NAV, ...moduleNav]).filter((item) => !isModuleHidden(item.id))
+  const navItems = useVisibleNavItems(CORE_NAV, moduleNav)
   const topLevelItems = navItems.filter((item) => TOP_LEVEL_NAV_IDS.has(item.id))
   const crmItems = navItems.filter((item) => !TOP_LEVEL_NAV_IDS.has(item.id))
-  const adminItems = inSlotOrder([...CORE_ADMIN_NAV, ...moduleAdminNav]).filter(
-    (item) => !isModuleHidden(item.id),
-  )
+  const adminItems = useVisibleNavItems(CORE_ADMIN_NAV, moduleAdminNav)
 
   return (
     <div className="flex min-h-screen bg-surface">

package/src/index.ts

@@ -155,12 +155,10 @@ export type { DueBucketId, PlanAttention as PlanAttentionSummary } from './lib/p
 export { applyTheme, getStoredTheme, resolveTheme, setStoredTheme, watchSystemTheme } from './lib/theme.ts'
 export type { ThemePreference } from './lib/theme.ts'
 
-export { EXTENSIBLE_RECORD_TYPES, INTEGRATION_CATEGORIES, NAV_SLOTS } from './registry/contributions.ts'
+export { EXTENSIBLE_RECORD_TYPES, NAV_SLOTS } from './registry/contributions.ts'
 export type {
   DashboardCard,
   ExtensibleRecordType,
-  IntegrationCategory,
-  IntegrationProvider,
   NavItem,
   NavSlot,
   RecordContext,
@@ -177,7 +175,6 @@ export type { UiExtensions, UiModule, UiModuleContext } from './registry/registr
 
 export {
   useDashboardCards,
-  useIntegrationProviders,
   useModuleRoutes,
   useNavItems,
   useOverridable,