Comment rules deep clean

__mocks__/lucide-react.tsx

@@ -7,59 +7,41 @@ import type { ReactElement } from 'react';
 /**
  * Stub for the LocateFixed icon; renders a bare SVG element so tests can locate the icon by test
  * ID.
- *
- * @param props - SVG props forwarded from the component, including optional className.
- * @returns A ReactElement SVG element used as a locate-fixed icon stub in tests.
  */
 export function LocateFixed(props: Readonly<{ className?: string }>): ReactElement {
   return <svg data-testid="locate-fixed-icon" {...props} />;
 }
 
 /**
  * Stub for the Info icon.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as an info icon stub in tests.
  */
 export function Info(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="info-icon" {...props} />;
 }
 
 /**
  * Stub for the Trash2 icon.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as a trash icon stub in tests.
  */
 export function Trash2(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="trash2-icon" {...props} />;
 }
 
 /**
  * Stub for the X icon.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as an X icon stub in tests.
  */
 export function X(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="x-icon" {...props} />;
 }
 
 /**
  * Stub for the Link2 (link) icon used by the between-token link button.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as a link icon stub in tests.
  */
 export function Link2(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="link2-icon" {...props} />;
 }
 
 /**
  * Stub for the Link2Off (unlink) icon used by the arc-split button.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as an unlink icon stub in tests.
  */
 export function Link2Off(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="link2off-icon" {...props} />;
@@ -68,29 +50,20 @@ export function Link2Off(props: Readonly<{ size?: number; className?: string }>)
 /**
  * Stub for the Unlink2 icon used by the between-token unlink button; a broken-chain glyph whose
  * chain sits at the same vertical position as Link2 so their button centers line up.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as an unlink icon stub in tests.
  */
 export function Unlink2(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="unlink2-icon" {...props} />;
 }
 
 /**
  * Stub for the Settings gear icon used by the view-options dropdown button.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as a settings icon stub in tests.
  */
 export function Settings(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="settings-icon" {...props} />;
 }
 
 /**
  * Stub for the Plus icon used by the token chip's suggestion-dropdown toggle.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as a plus icon stub in tests.
  */
 export function Plus(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="plus-icon" {...props} />;
@@ -99,9 +72,6 @@ export function Plus(props: Readonly<{ size?: number; className?: string }>): Re
  * Stub for the Merge icon used by both merge controls (the row-gap merge in the segment list and the
  * cross-segment merge in the continuous strip). A single Y-join glyph, deliberately a different
  * shape from the split marker's arrows-apart glyph.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as a merge icon stub in tests.
  */
 export function Merge(props: Readonly<{ className?: string }>): ReactElement {
   return <svg data-testid="merge-icon" {...props} />;
@@ -110,9 +80,6 @@ export function Merge(props: Readonly<{ className?: string }>): ReactElement {
 /**
  * Stub for the Split icon used by the Alt-gated split markers (token-chip and baseline-text). One
  * stroke diverging into two — the mirror of the `Merge` glyph's join.
- *
- * @param props - SVG props forwarded from the component.
- * @returns A ReactElement SVG element used as a split-marker icon stub in tests.
  */
 export function Split(props: Readonly<{ size?: number; className?: string }>): ReactElement {
   return <svg data-testid="split-icon" {...props} />;

__mocks__/papi-frontend-react.ts

@@ -35,9 +35,6 @@ const useProjectData = jest.fn(() =>
  * Mock for `useProjectSetting`. Returns `[defaultState, setSetting, resetSetting, isLoading]`,
  * passing `defaultState` through unchanged so callers receive a predictable initial value.
  *
- * @param _projectDataProviderSource - Ignored project data provider source.
- * @param _key - Ignored setting key.
- * @param defaultState - Value surfaced as the current setting state.
  * @returns Tuple of `[defaultState, jest.fn(), jest.fn(), false]`.
  */
 const useProjectSetting = jest
@@ -53,7 +50,6 @@ const useProjectSetting = jest
  * Mock for `useLocalizedStrings`. Maps each requested key to itself so tests receive a
  * predictable `Record<string, string>` without a real localization service.
  *
- * @param keys - BCP47-style string keys to resolve.
  * @returns Tuple of `[record, isLoading]` where every key maps to itself and `isLoading` is
  *   `false`.
  */
@@ -66,8 +62,6 @@ const useLocalizedStrings = jest.fn().mockImplementation((keys: string[]) => [
  * Mock for `useSetting`. Returns `[defaultState, setSetting, resetSetting, false]`, passing
  * `defaultState` through unchanged so callers receive a predictable initial value.
  *
- * @param _key - Ignored setting key.
- * @param defaultState - Value surfaced as the current setting state.
  * @returns Tuple of `[defaultState, jest.fn(), jest.fn(), false]`.
  */
 const useSetting = jest
@@ -82,8 +76,6 @@ const useSetting = jest
 /**
  * Mock for `useRecentScriptureRefs`. Returns an empty history and a no-op `addRecentScriptureRef`
  * so components that display recent references render without errors.
- *
- * @returns Object with `recentScriptureRefs` (empty array) and `addRecentScriptureRef` (jest spy).
  */
 const useRecentScriptureRefs = jest
   .fn()