canopycms 0.0.67-int.90 → 0.0.67

package/dist/editor/group-manager/ExternalGroupsTab.d.ts

@@ -1,6 +1,3 @@
-/**
- * External groups tab component
- */
 import React from 'react';
 import type { ExternalGroup } from './types.js';
 export interface ExternalGroupsTabProps {

package/dist/editor/group-manager/ExternalGroupsTab.js

@@ -3,7 +3,6 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Alert, Group, Loader, Paper, ScrollArea, Stack, Text, TextInput } from '@mantine/core';
 import { IconAlertCircle, IconBuilding, IconSearch } from '@tabler/icons-react';
 export const ExternalGroupsTab = ({ canEdit, searchQuery, searchResults, isSearching, searchError, onSearchQueryChange, canSearch, }) => {
-    // Read-only view (when !canEdit)
     if (!canEdit) {
         return (_jsx("div", { style: {
                 flex: 1,

package/dist/editor/group-manager/GroupCard.d.ts

@@ -1,6 +1,3 @@
-/**
- * Card component for displaying a single group with its members
- */
 import React from 'react';
 import type { InternalGroup, UserSearchResult, CanopyGroupId, CanopyUserId } from './types.js';
 export interface GroupCardProps {

package/dist/editor/group-manager/GroupForm.d.ts

@@ -1,6 +1,3 @@
-/**
- * Modal form for creating/editing groups
- */
 import React from 'react';
 import type { InternalGroup, GroupFormData } from './types.js';
 export interface GroupFormProps {

package/dist/editor/group-manager/InternalGroupsTab.d.ts

@@ -1,6 +1,3 @@
-/**
- * Internal groups tab component
- */
 import React from 'react';
 import type { InternalGroup, UserSearchResult, CanopyGroupId, CanopyUserId } from './types.js';
 export interface InternalGroupsTabProps {

package/dist/editor/group-manager/InternalGroupsTab.js

@@ -4,7 +4,6 @@ import { Button, Group, Paper, ScrollArea, Stack, Text } from '@mantine/core';
 import { IconPlus, IconUsers } from '@tabler/icons-react';
 import { GroupCard } from './GroupCard.js';
 export const InternalGroupsTab = ({ groups, canEdit, onCreateGroup, onEditGroup, onDeleteGroup, onAddMember, onRemoveMember, onGetUserMetadata, activeSearchGroupId, searchQuery, searchResults, isSearching, searchError, onSearchQueryChange, onShowSearch, onHideSearch, canSearch, }) => {
-    // Read-only view
     if (!canEdit) {
         return (_jsx("div", { style: {
                 flex: 1,
@@ -12,6 +11,5 @@ export const InternalGroupsTab = ({ groups, canEdit, onCreateGroup, onEditGroup,
                 paddingTop: 'var(--mantine-spacing-md)',
             }, children: groups.length === 0 ? (_jsxs(Paper, { withBorder: true, p: "md", style: { textAlign: 'center' }, children: [_jsx(IconUsers, { size: 32, color: "gray", style: { margin: '0 auto', marginBottom: 8 } }), _jsx(Text, { size: "sm", c: "dimmed", children: "No internal groups" })] })) : (_jsx(Stack, { gap: "xs", pb: "md", children: groups.map((group) => (_jsx(Paper, { withBorder: true, p: "sm", children: _jsxs(Group, { justify: "space-between", children: [_jsxs("div", { children: [_jsx(Text, { size: "sm", fw: 500, children: group.name }), group.description && (_jsx(Text, { size: "xs", c: "dimmed", children: group.description }))] }), _jsxs(Text, { size: "sm", variant: "light", children: [group.members?.length || 0, " members"] })] }) }, group.id))) })) }));
     }
-    // Editable view
     return (_jsx(ScrollArea, { style: { height: '100%' }, pt: "md", children: _jsxs(Stack, { gap: "sm", pb: "md", children: [_jsxs(Group, { justify: "space-between", mb: "xs", children: [_jsx(Text, { size: "sm", fw: 500, c: "dimmed", children: "Manage your internal groups" }), _jsx(Button, { size: "xs", variant: "light", leftSection: _jsx(IconPlus, { size: 14 }), onClick: onCreateGroup, children: "Create Group" })] }), groups.length === 0 ? (_jsxs(Paper, { withBorder: true, p: "md", style: { textAlign: 'center' }, children: [_jsx(IconUsers, { size: 32, color: "gray", style: { margin: '0 auto', marginBottom: 8 } }), _jsx(Text, { size: "sm", c: "dimmed", children: "No internal groups yet. Create one to get started." })] })) : (groups.map((group) => (_jsx(GroupCard, { group: group, onEdit: onEditGroup, onDelete: onDeleteGroup, onAddMember: onAddMember, onRemoveMember: onRemoveMember, onGetUserMetadata: onGetUserMetadata, showUserSearch: activeSearchGroupId === group.id, searchQuery: searchQuery, searchResults: searchResults, isSearching: isSearching, searchError: searchError, onSearchQueryChange: onSearchQueryChange, onShowSearch: () => onShowSearch(group.id), onHideSearch: onHideSearch, canSearch: canSearch }, group.id))))] }) }));
 };

package/dist/editor/group-manager/MemberList.d.ts

@@ -1,6 +1,3 @@
-/**
- * Member list component with user search for adding members
- */
 import React from 'react';
 import type { UserSearchResult, CanopyGroupId, CanopyUserId } from './types.js';
 export interface MemberListProps {

package/dist/editor/group-manager/hooks/useExternalGroupSearch.d.ts

@@ -1,6 +1,3 @@
-/**
- * Hook for external group search functionality
- */
 import type { ExternalGroup } from '../types.js';
 export interface UseExternalGroupSearchOptions {
     onSearchExternalGroups?: (query: string) => Promise<ExternalGroup[]>;

package/dist/editor/group-manager/hooks/useExternalGroupSearch.js

@@ -1,6 +1,3 @@
-/**
- * Hook for external group search functionality
- */
 import { useState, useEffect } from 'react';
 export function useExternalGroupSearch({ onSearchExternalGroups, }) {
     const [searchQuery, setSearchQuery] = useState('');

package/dist/editor/group-manager/hooks/useGroupState.d.ts

@@ -1,6 +1,3 @@
-/**
- * Hook for managing internal group state
- */
 import type { InternalGroup, CanopyGroupId, CanopyUserId } from '../types.js';
 export interface UseGroupStateOptions {
     initialGroups: InternalGroup[];

package/dist/editor/group-manager/hooks/useGroupState.js

@@ -1,6 +1,3 @@
-/**
- * Hook for managing internal group state
- */
 import { useState, useEffect, useCallback } from 'react';
 const TEMP_GROUP_ID_PREFIX = 'temp-';
 const isTempGroupId = (id) => id.startsWith(TEMP_GROUP_ID_PREFIX);
@@ -9,7 +6,6 @@ export function useGroupState({ initialGroups, onSave, }) {
     const [isDirty, setIsDirty] = useState(false);
     const [isSaving, setIsSaving] = useState(false);
     const [error, setError] = useState(null);
-    // Sync groups when prop changes
     useEffect(() => {
         setGroups(initialGroups);
         setIsDirty(false);
@@ -57,7 +53,6 @@ export function useGroupState({ initialGroups, onSave, }) {
     const save = useCallback(async () => {
         if (!onSave)
             return;
-        // Validate that Admins group is not empty
         const adminsGroup = groups.find((g) => g.id === 'Admins');
         if (!adminsGroup || !adminsGroup.members || adminsGroup.members.length === 0) {
             setError('Cannot remove the last admin. At least one admin is required.');

package/dist/editor/group-manager/hooks/useUserSearch.d.ts

@@ -1,6 +1,3 @@
-/**
- * Hook for user search functionality
- */
 import type { UserSearchResult } from '../types.js';
 export interface UseUserSearchOptions {
     onSearchUsers?: (query: string, limit?: number) => Promise<UserSearchResult[]>;

package/dist/editor/group-manager/hooks/useUserSearch.js

@@ -1,6 +1,3 @@
-/**
- * Hook for user search functionality
- */
 import { useState, useEffect, useCallback } from 'react';
 export function useUserSearch({ onSearchUsers }) {
     const [searchQuery, setSearchQuery] = useState('');

package/dist/editor/group-manager/index.d.ts

@@ -1,9 +1,3 @@
-/**
- * GroupManager - Main component for managing internal and external groups.
- *
- * This component provides a tabbed UI for managing internal groups with
- * member assignment, and browsing external groups from the organization.
- */
 import React from 'react';
 import type { GroupManagerProps } from './types.js';
 export declare const GroupManager: React.FC<GroupManagerProps>;

package/dist/editor/group-manager/index.js

@@ -1,11 +1,5 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-/**
- * GroupManager - Main component for managing internal and external groups.
- *
- * This component provides a tabbed UI for managing internal groups with
- * member assignment, and browsing external groups from the organization.
- */
 import { useState, useCallback } from 'react';
 import { Alert, Button, Group, Loader, Stack, Tabs, Text } from '@mantine/core';
 import { IconAlertCircle } from '@tabler/icons-react';
@@ -16,16 +10,12 @@ import { InternalGroupsTab } from './InternalGroupsTab.js';
 import { ExternalGroupsTab } from './ExternalGroupsTab.js';
 import { GroupForm } from './GroupForm.js';
 export const GroupManager = ({ internalGroups: initialInternalGroups, loading = false, canEdit, onSave, onSearchUsers, onGetUserMetadata, onSearchExternalGroups, onClose: _, }) => {
-    // Group state management
     const { groups, isDirty, isSaving, error, setError, createGroup, updateGroup, deleteGroup, addMember, removeMember, save, discard, } = useGroupState({
         initialGroups: initialInternalGroups,
         onSave,
     });
-    // User search for adding members
     const userSearch = useUserSearch({ onSearchUsers });
-    // External group search
     const externalGroupSearch = useExternalGroupSearch({ onSearchExternalGroups });
-    // Modal state for creating/editing groups
     const [isModalOpen, setIsModalOpen] = useState(false);
     const [editingGroup, setEditingGroup] = useState(null);
     const [formData, setFormData] = useState({

package/dist/editor/group-manager/types.d.ts

@@ -1,11 +1,8 @@
-/**
- * Type definitions for GroupManager module
- */
-import type { UserSearchResult, GroupMetadata } from '../../auth/types.js';
+import type { UserSearchResult } from '../../auth/types.js';
 import type { CanopyGroupId, CanopyUserId } from '../../types.js';
 import type { InternalGroup } from '../../authorization/index.js';
 import type { ExternalGroup } from '../../api/groups.js';
-export type { UserSearchResult, GroupMetadata, InternalGroup, ExternalGroup };
+export type { UserSearchResult, InternalGroup, ExternalGroup };
 export type { CanopyGroupId, CanopyUserId };
 export interface GroupManagerProps {
     internalGroups: InternalGroup[];

package/dist/editor/group-manager/types.js

@@ -1,4 +1 @@
-/**
- * Type definitions for GroupManager module
- */
 export {};

package/dist/editor/hooks/index.d.ts

@@ -1,14 +1,9 @@
-export * from './useEditorLayout.js';
-export { useBranchManager, type UseBranchManagerOptions } from './useBranchManager.js';
-export { useEntryManager, listAllEntries } from './useEntryManager.js';
-export * from './useDraftManager.js';
+export { useEditorLayout } from './useEditorLayout.js';
+export { useBranchManager } from './useBranchManager.js';
+export { useEntryManager } from './useEntryManager.js';
+export { useDraftManager } from './useDraftManager.js';
 export { useCommentSystem } from './useCommentSystem.js';
-export { useBranchesData, fetchBranches, BRANCHES_KEY, type BranchesData } from './useBranchesData.js';
-export { fetchEntriesAndSchema, entriesKey, type EntriesData, type FetchEntriesParams, } from './useEntriesData.js';
-export { useCommentsData, fetchComments, commentsKey, type CommentsData } from './useCommentsData.js';
 export { useGroupManager } from './useGroupManager.js';
 export { usePermissionManager } from './usePermissionManager.js';
-export { useBranchActions } from './useBranchActions.js';
-export { useUserContext, type UseUserContextReturn } from './useUserContext.js';
-export { useReferenceResolution, type UseReferenceResolutionOptions, type UseReferenceResolutionResult, } from './useReferenceResolution.js';
-export { useSchemaManager, type UseSchemaManagerOptions, type UseSchemaManagerReturn, } from './useSchemaManager.js';
+export { useUserContext } from './useUserContext.js';
+export { useSchemaManager } from './useSchemaManager.js';

package/dist/editor/hooks/index.js

@@ -1,17 +1,11 @@
-// Barrel export for all custom hooks
-// Export hooks as they are created
-// Note: resetApiClient functions are not exported - they're test-only utilities
-export * from './useEditorLayout.js';
+// Only the hooks Editor.tsx and media/MediaLibraryBody.tsx import. Everything
+// else in this directory is deep-imported from its own module.
+export { useEditorLayout } from './useEditorLayout.js';
 export { useBranchManager } from './useBranchManager.js';
-export { useEntryManager, listAllEntries } from './useEntryManager.js';
-export * from './useDraftManager.js';
+export { useEntryManager } from './useEntryManager.js';
+export { useDraftManager } from './useDraftManager.js';
 export { useCommentSystem } from './useCommentSystem.js';
-export { useBranchesData, fetchBranches, BRANCHES_KEY } from './useBranchesData.js';
-export { fetchEntriesAndSchema, entriesKey, } from './useEntriesData.js';
-export { useCommentsData, fetchComments, commentsKey } from './useCommentsData.js';
 export { useGroupManager } from './useGroupManager.js';
 export { usePermissionManager } from './usePermissionManager.js';
-export { useBranchActions } from './useBranchActions.js';
 export { useUserContext } from './useUserContext.js';
-export { useReferenceResolution, } from './useReferenceResolution.js';
-export { useSchemaManager, } from './useSchemaManager.js';
+export { useSchemaManager } from './useSchemaManager.js';

package/dist/editor/hooks/useBranchActions.d.ts

@@ -15,19 +15,5 @@ export interface UseBranchActionsReturn {
 }
 /**
  * Custom hook for branch navigation actions with dirty check support.
- *
- * Handles:
- * - Branch switching with unsaved changes confirmation
- * - Creating new branches with dirty check
- *
- * @example
- * ```tsx
- * const { handleBranchChange, handleCreateBranch } = useBranchActions({
- *   branchName,
- *   setBranchName,
- *   isAnyDirty,
- *   onReloadBranches
- * })
- * ```
  */
 export declare function useBranchActions(options: UseBranchActionsOptions): UseBranchActionsReturn;

package/dist/editor/hooks/useBranchActions.js

@@ -5,31 +5,11 @@ import { Text } from '@mantine/core';
 import { useApiClient } from '../context/index.js';
 /**
  * Custom hook for branch navigation actions with dirty check support.
- *
- * Handles:
- * - Branch switching with unsaved changes confirmation
- * - Creating new branches with dirty check
- *
- * @example
- * ```tsx
- * const { handleBranchChange, handleCreateBranch } = useBranchActions({
- *   branchName,
- *   setBranchName,
- *   isAnyDirty,
- *   onReloadBranches
- * })
- * ```
  */
 export function useBranchActions(options) {
     const apiClient = useApiClient();
-    // Helper: Perform branch switch with URL update
     const performBranchSwitch = (next) => {
-        // Update branchName state - hooks will react:
-        // - useEntryManager clears selectedPath and refreshes entries
-        // - useDraftManager clears drafts/loadedValues
-        // - useCommentSystem loads comments
         options.setBranchName(next);
-        // Update URL
         if (typeof window !== 'undefined') {
             const url = new URL(window.location.href);
             url.searchParams.set('branch', next);
@@ -37,7 +17,6 @@ export function useBranchActions(options) {
         }
         options.onBranchSwitch?.(next);
     };
-    // Helper: Check for dirty state and show modal
     const confirmIfDirty = async (message) => {
         if (!options.isAnyDirty())
             return true;
@@ -64,7 +43,6 @@ export function useBranchActions(options) {
         const confirmed = await confirmIfDirty('Create new branch without saving changes?');
         if (!confirmed)
             return;
-        // Create the branch via API
         try {
             const result = await apiClient.branches.create({
                 branch: branch.name,

package/dist/editor/hooks/useBranchManager.d.ts

@@ -2,10 +2,7 @@ import type { ConflictStatus, PullRequestState, SyncStatus } from '../../types.j
 import type { OperatingMode } from '../../operating-mode/index.js';
 import type { CommentThread } from '../../comment-store.js';
 import type { BranchListItem } from '../../api/branch.js';
-/**
- * Branch summary for display in BranchManager component.
- */
-export interface BranchSummary {
+interface BranchSummary {
     name: string;
     status: string;
     createdBy?: string;
@@ -50,21 +47,9 @@ export interface BranchSummary {
     submitBlocked: boolean;
 }
 export interface UseBranchManagerOptions {
-    /**
-     * Initial branch name.
-     */
     initialBranch: string;
-    /**
-     * Operating mode (dev, etc.).
-     */
     operatingMode: OperatingMode;
-    /**
-     * Callback to set busy state.
-     */
     setBusy: (busy: boolean) => void;
-    /**
-     * Current comments (for computing comment counts per branch).
-     */
     comments: CommentThread[];
 }
 export interface UseBranchManagerReturn {
@@ -82,37 +67,6 @@ export interface UseBranchManagerReturn {
 }
 /**
  * Custom hook for managing git branches.
- *
- * Handles:
- * - Loading branches from API
- * - Branch switching with unsaved changes confirmation
- * - Creating new branches
- * - Branch workflow (submit, withdraw, request changes)
- * - URL synchronization for branch parameter
- *
- * @example
- * ```tsx
- * const {
- *   branchName,
- *   branches,
- *   currentBranch,
- *   handleBranchChange,
- *   handleCreateBranch,
- *   handleSubmit
- * } = useBranchManager({
- *   initialBranch: 'main',
- *   operatingMode: 'collaboration',
- *   selectedPath,
- *   drafts,
- *   loadedValues,
- *   setDrafts,
- *   setLoadedValues,
- *   setSelectedId,
- *   setEntries,
- *   onEntriesRefresh: refreshEntries,
- *   onCommentsLoad: loadComments,
- *   setBusy
- * })
- * ```
  */
 export declare function useBranchManager(options: UseBranchManagerOptions): UseBranchManagerReturn;
+export {};

package/dist/editor/hooks/useBranchManager.js

@@ -49,9 +49,6 @@ const confirmModalHandlers = (onConfirm, onDismiss) => {
         },
     };
 };
-/**
- * Helper function to show confirmation modal for branch submit action.
- */
 const showSubmitConfirmation = (branchName, onConfirm, onDismiss) => {
     modals.openConfirmModal({
         title: 'Submit Branch for Review',
@@ -61,9 +58,6 @@ const showSubmitConfirmation = (branchName, onConfirm, onDismiss) => {
         ...confirmModalHandlers(onConfirm, onDismiss),
     });
 };
-/**
- * Helper function to show confirmation modal for branch withdraw action.
- */
 const showWithdrawConfirmation = (branchName, onConfirm, onDismiss) => {
     modals.openConfirmModal({
         title: 'Withdraw Branch from Review',
@@ -74,8 +68,6 @@ const showWithdrawConfirmation = (branchName, onConfirm, onDismiss) => {
     });
 };
 /**
- * Helper function to show confirmation modal for branch delete action.
- *
  * Unlike submit/withdraw (both reversible), delete is irreversible -- it
  * unlinks branch.json, removes the clone, and removes the branch head from
  * the local git mirror. Before this, `handleDelete` below called
@@ -95,54 +87,18 @@ const showDeleteConfirmation = (branchName, onConfirm, onDismiss) => {
         // (apps/test-app/e2e/fixtures/branch-page.ts's deleteBranch) already
         // anticipated a delete confirmation and clicks
         // [data-testid="confirm-delete-branch"] when it appears. Without this the
-        // modal opens, nothing dismisses it, and the branch never disappears --
-        // which is exactly how the e2e suite caught this confirmation being added.
+        // modal opens, nothing dismisses it, and the branch never disappears.
         confirmProps: { color: 'red', 'data-testid': 'confirm-delete-branch' },
         ...confirmModalHandlers(onConfirm, onDismiss),
     });
 };
 /**
  * Custom hook for managing git branches.
- *
- * Handles:
- * - Loading branches from API
- * - Branch switching with unsaved changes confirmation
- * - Creating new branches
- * - Branch workflow (submit, withdraw, request changes)
- * - URL synchronization for branch parameter
- *
- * @example
- * ```tsx
- * const {
- *   branchName,
- *   branches,
- *   currentBranch,
- *   handleBranchChange,
- *   handleCreateBranch,
- *   handleSubmit
- * } = useBranchManager({
- *   initialBranch: 'main',
- *   operatingMode: 'collaboration',
- *   selectedPath,
- *   drafts,
- *   loadedValues,
- *   setDrafts,
- *   setLoadedValues,
- *   setSelectedId,
- *   setEntries,
- *   onEntriesRefresh: refreshEntries,
- *   onCommentsLoad: loadComments,
- *   setBusy
- * })
- * ```
  */
 export function useBranchManager(options) {
     const apiClient = useApiClient();
     const { mutate: globalMutate } = useSWRConfig();
     const [branchName, setBranchName] = useState(options.initialBranch);
-    // Automatic load, deduped by SWR (e.g. React Strict Mode's double effect
-    // invoke collapses to a single request). Not keyed by branchName -- the
-    // endpoint returns every branch regardless of which one is selected.
     const { data: branchesData, error: branchesError, isValidating: branchesIsValidating, } = useBranchesData(apiClient);
     const branches = useMemo(() => branchesData?.branches ?? [], [branchesData]);
     // Adopt the server's default branch once data arrives, if nothing pinned one.
@@ -151,7 +107,7 @@ export function useBranchManager(options) {
             setBranchName(branchesData.defaultBranch);
         }
     }, [branchesData, branchName]);
-    // Surface/clear the sticky error toast the same way loadBranches() used to.
+    // Surfaces/clears the sticky error toast the same way loadBranches() does.
     useEffect(() => {
         if (branchesError) {
             console.error(branchesError);
@@ -183,7 +139,6 @@ export function useBranchManager(options) {
     // resolves to the branch the server actually persisted (e.g. "feature-x").
     const currentBranch = branches.find((b) => b.name === branchName) ??
         branches.find((b) => b.name === sanitizeBranchName(branchName));
-    // Compute branch summaries with comment counts
     const branchSummaries = useMemo(() => {
         return branches.map((b) => {
             const branchComments = b.name === branchName ? options.comments : [];
@@ -244,8 +199,6 @@ export function useBranchManager(options) {
         catch (err) {
             console.error(err);
             const message = err instanceof Error ? err.message : 'Failed to load branches';
-            // Fixed id: retries update the existing toast instead of stacking; sticky
-            // because the editor cannot function without the branch list.
             notifications.show({
                 id: 'canopy-branches-load-failed',
                 message,
@@ -315,13 +268,7 @@ export function useBranchManager(options) {
                 finally {
                     options.setBusy(false);
                 }
-            }, 
-            // Dismissal is not an error. Rejecting here meant clicking Cancel
-            // logged a console error at the only call site (Editor.tsx catches
-            // these into console.error), which is both noise and a trip hazard
-            // for the CI=1 stray-console gate. Settled once, by onCancel OR
-            // onClose, so an Escape/overlay dismissal can't leave this pending.
-            () => resolve());
+            }, () => resolve());
         });
     };
     const handleRequestChanges = async (branchNameForChanges) => {
@@ -367,19 +314,12 @@ export function useBranchManager(options) {
                     // rethrown). Cancellation resolves too -- see below.
                     resolve();
                 }
-            }, 
-            // Dismissal is not an error. Rejecting here meant clicking Cancel
-            // logged a console error at the only call site (Editor.tsx catches
-            // these into console.error), which is both noise and a trip hazard
-            // for the CI=1 stray-console gate. Settled once, by onCancel OR
-            // onClose, so an Escape/overlay dismissal can't leave this pending.
-            () => resolve());
+            }, () => resolve());
         });
     };
     const handleReloadBranchData = async () => {
         await loadBranches();
     };
-    // Sync branch to URL parameter
     useEffect(() => {
         if (typeof window === 'undefined')
             return;

package/dist/editor/hooks/useBranchesData.d.ts

@@ -29,6 +29,7 @@ export declare function fetchBranches(apiClient: Pick<ApiClient, 'branches'>): P
  * How often to re-poll while at least one branch is mid-flight, in ms.
  * Only ever active in the transient window below, so this is not a
  * standing background poll.
+ * @internal Exported only for the test that pins it.
  */
 export declare const IN_FLIGHT_POLL_MS = 15000;
 /**
@@ -36,12 +37,12 @@ export declare const IN_FLIGHT_POLL_MS = 15000;
  *
  * `pending-sync` -> `synced` | `sync-failed` happens asynchronously on the
  * EC2 worker when it drains the task queue, with nothing pushed to the
- * browser. Before SWR, the branches list happened to refetch on every
- * branch switch, so these badges (PR #160) converged incidentally as the
- * user moved around. That effect is gone now -- correctly, since the list
- * isn't branch-scoped -- so without this a user who submits a branch would
- * watch "Pending sync" forever until they reloaded the page or performed
- * some unrelated branch mutation.
+ * browser. The branches list is not refetched on a branch switch (it isn't
+ * branch-scoped), so an in-flight badge converges only via this poll --
+ * without it, a user who submits a branch would watch "Pending sync"
+ * forever until they reloaded the page or performed some unrelated branch
+ * mutation.
+ * @internal Exported only for the test that pins it.
  */
 export declare function hasInFlightBranch(data: BranchesData | undefined): boolean;
 /**

package/dist/editor/hooks/useBranchesData.js

@@ -35,6 +35,7 @@ export async function fetchBranches(apiClient) {
  * How often to re-poll while at least one branch is mid-flight, in ms.
  * Only ever active in the transient window below, so this is not a
  * standing background poll.
+ * @internal Exported only for the test that pins it.
  */
 export const IN_FLIGHT_POLL_MS = 15_000;
 /**
@@ -42,12 +43,12 @@ export const IN_FLIGHT_POLL_MS = 15_000;
  *
  * `pending-sync` -> `synced` | `sync-failed` happens asynchronously on the
  * EC2 worker when it drains the task queue, with nothing pushed to the
- * browser. Before SWR, the branches list happened to refetch on every
- * branch switch, so these badges (PR #160) converged incidentally as the
- * user moved around. That effect is gone now -- correctly, since the list
- * isn't branch-scoped -- so without this a user who submits a branch would
- * watch "Pending sync" forever until they reloaded the page or performed
- * some unrelated branch mutation.
+ * browser. The branches list is not refetched on a branch switch (it isn't
+ * branch-scoped), so an in-flight badge converges only via this poll --
+ * without it, a user who submits a branch would watch "Pending sync"
+ * forever until they reloaded the page or performed some unrelated branch
+ * mutation.
+ * @internal Exported only for the test that pins it.
  */
 export function hasInFlightBranch(data) {
     return (data?.branches ?? []).some((b) => b.syncStatus === 'pending-sync');

package/dist/editor/hooks/useCommentSystem.d.ts

@@ -1,37 +1,13 @@
 import type { CommentThread } from '../../comment-store.js';
 import type { EditorEntry } from '../Editor.js';
 export interface UseCommentSystemOptions {
-    /**
-     * Current branch name for loading/saving comments.
-     */
     branchName: string;
-    /**
-     * Currently selected entry path.
-     */
     selectedPath: string;
-    /**
-     * Current entry being edited.
-     */
     currentEntry: EditorEntry | undefined;
-    /**
-     * Current user identifier.
-     */
     currentUser: string;
-    /**
-     * Whether the current user can resolve comment threads.
-     */
     canResolveComments: boolean;
-    /**
-     * Callback to change the selected entry.
-     */
     setSelectedPath: (id: string) => void;
-    /**
-     * Callback to open the branch manager.
-     */
     setBranchManagerOpen: (open: boolean) => void;
-    /**
-     * Optional callback when comments are loaded/updated.
-     */
     onCommentsChange?: (comments: CommentThread[]) => void;
 }
 export interface UseCommentSystemReturn {
@@ -63,30 +39,5 @@ export interface UseCommentSystemReturn {
 }
 /**
  * Custom hook for managing the comment system.
- *
- * Handles:
- * - Loading comments from API
- * - Adding comments to threads
- * - Resolving comment threads
- * - Field focus highlighting from preview frame
- * - Active comment context tracking
- *
- * @example
- * ```tsx
- * const {
- *   comments,
- *   activeThreads,
- *   handleAddComment,
- *   handleResolveThread,
- *   loadComments
- * } = useCommentSystem({
- *   branchName,
- *   selectedPath,
- *   currentEntry,
- *   currentUser,
- *   canResolveComments,
- *   onReloadBranches
- * })
- * ```
  */
 export declare function useCommentSystem(options: UseCommentSystemOptions): UseCommentSystemReturn;