@pennyfarthing/cyclist 10.4.0 → 11.0.0

package/src/public/hooks/useHealthScore.ts

@@ -1,69 +0,0 @@
-import { useState, useCallback, useRef, useEffect } from 'react';
-
-export interface HealthScoreDimension {
-  name: string;
-  score: number | null;
-  weight: number;
-}
-
-export interface HealthScoreData {
-  success: boolean;
-  composite_score: number;
-  dimensions: HealthScoreDimension[];
-  cached: boolean;
-  error?: string;
-}
-
-export interface UseHealthScoreReturn {
-  data: HealthScoreData | null;
-  isLoading: boolean;
-  error: Error | null;
-  lastFetchedAt: number | null;
-  refresh: () => void;
-}
-
-export function useHealthScore(): UseHealthScoreReturn {
-  const [data, setData] = useState<HealthScoreData | null>(null);
-  const [isLoading, setIsLoading] = useState(false);
-  const [error, setError] = useState<Error | null>(null);
-  const [lastFetchedAt, setLastFetchedAt] = useState<number | null>(null);
-  const abortRef = useRef<AbortController | null>(null);
-
-  const refresh = useCallback(() => {
-    if (abortRef.current) {
-      abortRef.current.abort();
-    }
-
-    const controller = new AbortController();
-    abortRef.current = controller;
-
-    setIsLoading(true);
-    setError(null);
-
-    fetch('/api/health-score', { signal: controller.signal })
-      .then((res) => {
-        if (!res.ok) {
-          throw new Error(`HTTP ${res.status}: ${res.statusText}`);
-        }
-        return res.json();
-      })
-      .then((json: HealthScoreData) => {
-        setData(json);
-        setLastFetchedAt(Date.now());
-        setIsLoading(false);
-      })
-      .catch((err) => {
-        if (err.name === 'AbortError') return;
-        setError(err instanceof Error ? err : new Error(String(err)));
-        setIsLoading(false);
-      });
-  }, []);
-
-  useEffect(() => {
-    return () => {
-      abortRef.current?.abort();
-    };
-  }, []);
-
-  return { data, isLoading, error, lastFetchedAt, refresh };
-}

package/src/public/hooks/useHotspots.ts

@@ -1,123 +0,0 @@
-import { useState, useCallback, useRef, useEffect } from 'react';
-
-// Types matching Python HotspotResult / MultiRepoHotspotResult
-export interface FileHotspot {
-  path: string;
-  change_count: number;
-  bug_fix_count: number;
-  author_count: number;
-  lines_added: number;
-  lines_deleted: number;
-  churn: number;
-  last_changed: string;
-  hotspot_score: number;
-}
-
-export interface DirectoryHotspot {
-  path: string;
-  file_count: number;
-  total_changes: number;
-  total_bug_fixes: number;
-  avg_author_count: number;
-  hotspot_score: number;
-}
-
-export interface HotspotRepoResult {
-  success: boolean;
-  repo_name: string;
-  repo_path: string;
-  time_window_days: number;
-  commit_count: number;
-  file_hotspots: FileHotspot[];
-  directory_hotspots: DirectoryHotspot[];
-  error?: string;
-}
-
-export interface HotspotData {
-  success: boolean;
-  // Single-repo result fields (when --path is used)
-  repo_name?: string;
-  repo_path?: string;
-  time_window_days?: number;
-  commit_count?: number;
-  file_hotspots?: FileHotspot[];
-  directory_hotspots?: DirectoryHotspot[];
-  // Multi-repo result fields
-  repo_results?: HotspotRepoResult[];
-  error?: string;
-}
-
-export interface UseHotspotsOptions {
-  days: number;
-  repo?: string;
-  skipTypes?: string[];
-  includeOrchestrator?: boolean;
-}
-
-export interface UseHotspotsReturn {
-  data: HotspotData | null;
-  isLoading: boolean;
-  error: Error | null;
-  refresh: () => void;
-}
-
-export function useHotspots(options: UseHotspotsOptions): UseHotspotsReturn {
-  const [data, setData] = useState<HotspotData | null>(null);
-  const [isLoading, setIsLoading] = useState(false);
-  const [error, setError] = useState<Error | null>(null);
-  const abortRef = useRef<AbortController | null>(null);
-
-  const fetchHotspots = useCallback(() => {
-    // Cancel any in-flight request
-    if (abortRef.current) {
-      abortRef.current.abort();
-    }
-
-    const controller = new AbortController();
-    abortRef.current = controller;
-
-    setIsLoading(true);
-    setError(null);
-
-    const params = new URLSearchParams({ days: String(options.days) });
-    if (options.repo) {
-      params.set('repo', options.repo);
-    }
-
-    // Determine skip_type values: use explicit skipTypes, or default to ['orchestrator']
-    // unless includeOrchestrator is true
-    const skipTypes = options.skipTypes ??
-      (options.includeOrchestrator ? [] : ['orchestrator']);
-    for (const st of skipTypes) {
-      params.append('skip_type', st);
-    }
-
-    fetch(`/api/hotspots?${params}`, { signal: controller.signal })
-      .then((res) => {
-        if (!res.ok) {
-          throw new Error(`HTTP ${res.status}: ${res.statusText}`);
-        }
-        return res.json();
-      })
-      .then((json: HotspotData) => {
-        setData(json);
-        setIsLoading(false);
-      })
-      .catch((err) => {
-        if (err.name === 'AbortError') return;
-        setError(err instanceof Error ? err : new Error(String(err)));
-        setIsLoading(false);
-      });
-  }, [options.days, options.repo, options.skipTypes, options.includeOrchestrator]);
-
-  // Cleanup abort controller on unmount
-  useEffect(() => {
-    return () => {
-      if (abortRef.current) {
-        abortRef.current.abort();
-      }
-    };
-  }, []);
-
-  return { data, isLoading, error, refresh: fetchHotspots };
-}

package/src/public/hooks/useLayoutPersistence.ts

@@ -1,138 +0,0 @@
-/**
- * useLayoutPersistence Hook
- *
- * React hook for saving and restoring layout state to config.local.yaml.
- * Story MSSCI-12706 - Layout Persistence
- *
- * Uses native Dockview SerializedDockview format for complete layout fidelity.
- *
- * REST API:
- * - GET /api/settings/layout - Load layout
- * - PATCH /api/settings/layout - Save layout
- *
- * Features:
- * - Load layout from config on mount
- * - Save layout on changes (debounced)
- * - Per-project independent layouts
- * - Graceful handling of corrupted/missing config
- * - Native Dockview serialization for perfect restore
- */
-
-import { useState, useEffect, useCallback, useRef } from 'react';
-import type { SerializedDockview } from 'dockview-react';
-
-const DEBOUNCE_DELAY = 300;
-
-interface UseLayoutPersistenceResult {
-  layout: SerializedDockview | null;
-  isLoading: boolean;
-  isSaving: boolean;
-  error: Error | null;
-  saveLayout: (layout: SerializedDockview) => void;
-}
-
-/**
- * Validate that the layout has the native Dockview structure
- */
-function isValidDockviewLayout(layout: unknown): layout is SerializedDockview {
-  if (!layout || typeof layout !== 'object') return false;
-  const layoutObj = layout as Record<string, unknown>;
-
-  // Check for native Dockview structure: grid and panels are required
-  if (!layoutObj.grid || typeof layoutObj.grid !== 'object') return false;
-  if (!layoutObj.panels || typeof layoutObj.panels !== 'object') return false;
-
-  const grid = layoutObj.grid as Record<string, unknown>;
-  // Grid should have root, width, height, orientation
-  if (!grid.root || typeof grid.width !== 'number' || typeof grid.height !== 'number') return false;
-
-  return true;
-}
-
-export function useLayoutPersistence(): UseLayoutPersistenceResult {
-  const [layout, setLayout] = useState<SerializedDockview | null>(null);
-  const [isLoading, setIsLoading] = useState(true);
-  const [isSaving, setIsSaving] = useState(false);
-  const [error, setError] = useState<Error | null>(null);
-
-  const debounceRef = useRef<ReturnType<typeof setTimeout> | null>(null);
-  const pendingLayoutRef = useRef<SerializedDockview | null>(null);
-
-  // Load layout on mount via REST API
-  useEffect(() => {
-    const loadLayout = async () => {
-      try {
-        const response = await fetch('/api/settings/layout');
-        if (response.ok) {
-          const data = await response.json();
-          if (data.layout && isValidDockviewLayout(data.layout)) {
-            // Use native Dockview layout directly
-            setLayout(data.layout as SerializedDockview);
-          } else {
-            // No saved layout or invalid format - let DockviewWorkspace build default
-            setLayout(null);
-          }
-        } else {
-          // API error - let DockviewWorkspace build default
-          setLayout(null);
-        }
-      } catch (err) {
-        // On error, let DockviewWorkspace build default
-        console.error('[useLayoutPersistence] Failed to load layout:', err);
-        setLayout(null);
-        setError(err instanceof Error ? err : new Error('Failed to load layout'));
-      } finally {
-        setIsLoading(false);
-      }
-    };
-
-    loadLayout();
-  }, []);
-
-  // Debounced save function via REST API - saves native Dockview format directly
-  const saveLayout = useCallback((newLayout: SerializedDockview) => {
-    pendingLayoutRef.current = newLayout;
-
-    // Clear existing debounce timer
-    if (debounceRef.current) {
-      clearTimeout(debounceRef.current);
-    }
-
-    // Set new debounce timer
-    debounceRef.current = setTimeout(async () => {
-      const layoutToSave = pendingLayoutRef.current;
-      if (!layoutToSave) return;
-
-      setIsSaving(true);
-      try {
-        // Save native Dockview format directly - no conversion needed
-        const response = await fetch('/api/settings/layout', {
-          method: 'PATCH',
-          headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify(layoutToSave),
-        });
-
-        if (!response.ok) {
-          throw new Error('Failed to save layout');
-        }
-        setError(null);
-      } catch (err) {
-        console.error('[useLayoutPersistence] Failed to save layout:', err);
-        setError(err instanceof Error ? err : new Error('Failed to save layout'));
-      } finally {
-        setIsSaving(false);
-      }
-    }, DEBOUNCE_DELAY);
-  }, []);
-
-  // Cleanup debounce timer on unmount
-  useEffect(() => {
-    return () => {
-      if (debounceRef.current) {
-        clearTimeout(debounceRef.current);
-      }
-    };
-  }, []);
-
-  return { layout, isLoading, isSaving, error, saveLayout };
-}

package/src/public/hooks/useMarkdownParser.ts

@@ -1,36 +0,0 @@
-/**
- * useMarkdownParser Hook
- *
- * Story MSSCI-13969: React hook for parsing markdown to HTML with security.
- * Extracted from js/components/message-view/markdown-parser.js
- *
- * Features:
- * - XSS prevention via HTML escaping
- * - CYCLIST marker stripping
- * - Memoization for performance
- */
-
-import { useMemo } from 'react';
-import { parseMarkdown } from '../utils/markdown.js';
-
-export interface UseMarkdownParserResult {
-  html: string;
-  isLoading: boolean;
-  error: Error | null;
-}
-
-/**
- * React hook for parsing markdown to HTML with security and memoization.
- * @param markdown - Raw markdown text (or null)
- * @returns Object with html string, loading state, and error
- */
-export function useMarkdownParser(
-  markdown: string | null
-): UseMarkdownParserResult {
-  const html = useMemo(() => {
-    if (!markdown) return '';
-    return parseMarkdown(markdown);
-  }, [markdown]);
-
-  return { html, isLoading: false, error: null };
-}

package/src/public/hooks/useMarkerActions.ts

@@ -1,234 +0,0 @@
-/**
- * useMarkerActions Hook
- *
- * Detects CYCLIST markers in message content and returns action metadata
- * for rendering QuickActions buttons.
- *
- * Story: MSSCI-12787 - Implement CYCLIST Marker Parsing and Action Buttons
- *
- * @see sprint/context/MSSCI-12787-reference/quick-actions.js.deleted
- */
-
-import { useMemo } from 'react';
-import { detectMarkers, stripMarkers, MARKER_TYPES } from '@pennyfarthing/shared/browser';
-
-/**
- * Action types that map to different UI presentations
- */
-export type ActionType =
-  | 'handoff'   // Show agent button + "Not yet" option
-  | 'yesno'     // Show Yes/No buttons
-  | 'open'      // Show text input
-  | 'choices'   // Show choice buttons
-  | 'continue'  // Show Continue button
-  | 'invoke'    // Auto-execute (no buttons)
-  | 'context_clear'; // Clear context and reload
-
-/**
- * Result from marker detection
- */
-export interface MarkerAction {
-  type: ActionType;
-  value?: string;
-  responses?: string[];
-  choices?: Array<{ number: number; text: string }>;
-  autoExecute?: boolean;
-}
-
-/**
- * Extract option text from message content for numbered choices.
- * Looks for patterns like "1. Option text" or "1) Option text".
- */
-function extractChoiceTexts(
-  text: string,
-  choiceNumbers: number[]
-): Array<{ number: number; text: string }> {
-  if (!text) {
-    return choiceNumbers.map(num => ({ number: num, text: `Option ${num}` }));
-  }
-
-  // Remove code blocks
-  const withoutCode = text.replace(/```[\s\S]*?```/g, '');
-
-  // Patterns for numbered lists
-  const patterns = [
-    /^\s*(\d+)\.\s+(.+)$/gm,           // "1. Option text"
-    /^\s*(\d+)\)\s+(.+)$/gm,           // "1) Option text"
-    /\*\*(\d+)[.)]\*\*\s*(.+)/gm,      // "**1.** Option text"
-  ];
-
-  const foundChoices = new Map<number, string>();
-
-  for (const pattern of patterns) {
-    pattern.lastIndex = 0;
-    let match;
-    while ((match = pattern.exec(withoutCode)) !== null) {
-      const num = parseInt(match[1], 10);
-      if (choiceNumbers.includes(num) && !foundChoices.has(num)) {
-        // Clean up the text - remove trailing markdown/formatting
-        let optionText = match[2].trim();
-        // Remove trailing description after " - " or " — " for cleaner button labels
-        const dashIndex = optionText.search(/\s+[-—]\s+/);
-        if (dashIndex > 0 && dashIndex < 30) {
-          optionText = optionText.substring(0, dashIndex);
-        }
-        foundChoices.set(num, optionText);
-      }
-    }
-  }
-
-  // Return choices in order, falling back to "Option N" if not found
-  return choiceNumbers.map(num => ({
-    number: num,
-    text: foundChoices.get(num) || `Option ${num}`,
-  }));
-}
-
-/**
- * Process detected markers into action metadata
- */
-function processMarkers(
-  markers: ReturnType<typeof detectMarkers>,
-  fullText: string
-): MarkerAction | null {
-  if (!markers || markers.length === 0) return null;
-
-  const primaryMarker = markers[0];
-
-  switch (primaryMarker.type) {
-    case MARKER_TYPES.HANDOFF:
-      return {
-        type: 'handoff',
-        value: primaryMarker.value,
-        responses: [primaryMarker.value, 'Not yet'],
-      };
-
-    case MARKER_TYPES.INVOKE:
-      return {
-        type: 'invoke',
-        value: primaryMarker.value,
-        autoExecute: true,
-      };
-
-    case MARKER_TYPES.QUESTION: {
-      if (primaryMarker.value === 'yesno') {
-        return {
-          type: 'yesno',
-          responses: ['Yes', 'No'],
-        };
-      }
-      // Handle open questions - may have suggested prompt: "open" or "open:suggested text"
-      if (primaryMarker.value?.startsWith('open')) {
-        // Check for suggested prompt after "open:"
-        const colonIndex = primaryMarker.value.indexOf(':');
-        if (colonIndex !== -1) {
-          const suggestion = primaryMarker.value.substring(colonIndex + 1).trim();
-          if (suggestion) {
-            return {
-              type: 'open',
-              responses: [suggestion],
-            };
-          }
-        }
-        // Plain open question without suggestion
-        return {
-          type: 'open',
-        };
-      }
-      // Check for accompanying CHOICES marker
-      const choicesMarker = markers.find(m => m.type === MARKER_TYPES.CHOICES);
-      if (choicesMarker) {
-        return processChoicesMarker(choicesMarker.value, fullText);
-      }
-      return null;
-    }
-
-    case MARKER_TYPES.CHOICES:
-      return processChoicesMarker(primaryMarker.value, fullText);
-
-    case MARKER_TYPES.CONTEXT_CLEAR:
-      return {
-        type: 'context_clear',
-        value: primaryMarker.value,
-      };
-
-    case MARKER_TYPES.CONTINUE:
-      return {
-        type: 'continue',
-        responses: ['Continue'],
-      };
-
-    default:
-      return null;
-  }
-}
-
-/**
- * Process a CHOICES marker value into choice buttons
- */
-function processChoicesMarker(
-  value: string,
-  fullText: string
-): MarkerAction {
-  const choiceValues = value.split(',').map(v => v.trim());
-  const firstValue = choiceValues[0];
-  const isNumeric = /^\d+$/.test(firstValue);
-
-  let choices: Array<{ number: number; text: string }>;
-
-  if (isNumeric) {
-    // Legacy numeric format - extract text from message
-    const choiceNumbers = choiceValues.map(n => parseInt(n, 10));
-    choices = extractChoiceTexts(fullText, choiceNumbers);
-  } else {
-    // Text label format - use labels directly
-    choices = choiceValues.map((text, index) => ({
-      number: index + 1,
-      text,
-    }));
-  }
-
-  return {
-    type: 'choices',
-    choices,
-    responses: choices.map(c => c.text),
-  };
-}
-
-/**
- * Hook to detect CYCLIST markers in content and return action metadata.
- *
- * @param content - Message content to analyze
- * @returns MarkerAction if marker detected, null otherwise
- *
- * @example
- * ```tsx
- * const actions = useMarkerActions(message.content);
- * if (actions?.type === 'yesno') {
- *   // Render Yes/No buttons
- * }
- * ```
- */
-export function useMarkerActions(content: string | undefined): MarkerAction | null {
-  return useMemo(() => {
-    if (!content) return null;
-
-    const markers = detectMarkers(content);
-    if (!markers) return null;
-
-    return processMarkers(markers, content);
-  }, [content]);
-}
-
-/**
- * Strip CYCLIST markers from content for display.
- *
- * @param content - Content that may contain markers
- * @returns Content with markers removed
- */
-export function useStrippedContent(content: string | undefined): string {
-  return useMemo(() => {
-    if (!content) return '';
-    return stripMarkers(content);
-  }, [content]);
-}