ref(tools): extract 28 inline types to named interfaces

src/Internal/LoggerLib/Logger.ts

@@ -707,7 +707,13 @@ export const consoleTransport: ITransport = {
     },
 };
 
-export const fileTransport = (options: { filePath: string; append?: boolean; encoding?: string }): ITransport => ({
+interface FileTransportOptions {
+    filePath: string;
+    append?: boolean;
+    encoding?: string;
+}
+
+export const fileTransport = (options: FileTransportOptions): ITransport => ({
     log(props) {
         // Here you would implement file writing logic
         // Using appropriate React Native or Node.js APIs

src/ask/AIChat.ts

@@ -19,6 +19,11 @@ import { providerManager } from "@ask/providers/ProviderManager";
 import type { ChatConfig } from "@ask/types";
 import { getLanguageModel } from "@ask/types";
 
+interface EngineWithRestore {
+    engine: ChatEngine;
+    restore: () => void;
+}
+
 const DEFAULT_SESSION_DIR = resolve(homedir(), ".genesis-tools/ai-chat/sessions");
 
 export class AIChat {
@@ -241,7 +246,7 @@ export class AIChat {
     /** Get a ChatEngine instance with optional per-call overrides.
      * Returns a restore function that resets the engine to its previous state.
      */
-    private _getEngine(override?: SendOptions["override"]): { engine: ChatEngine; restore: () => void } {
+    private _getEngine(override?: SendOptions["override"]): EngineWithRestore {
         if (!this._engine) {
             throw new Error("AIChat not initialized");
         }

src/ask/providers/ProviderManager.ts

@@ -18,6 +18,11 @@ import { getLanguageModel } from "@ask/types";
 import type { AskConfig } from "@ask/types/config";
 import { generateText } from "ai";
 
+interface ModelMetadata {
+    id: string;
+    description?: string;
+}
+
 export class ProviderManager {
     private detectedProviders: Map<string, DetectedProvider> = new Map();
     private initialized = false;
@@ -598,7 +603,7 @@ export class ProviderManager {
         };
     }
 
-    private parseCapabilities(model: { id: string; description?: string }): string[] {
+    private parseCapabilities(model: ModelMetadata): string[] {
         const capabilities: string[] = ["chat"];
 
         if (model.description?.toLowerCase().includes("vision") || model.id.toLowerCase().includes("vision")) {

src/ask/utils/cli.ts

@@ -3,6 +3,16 @@ import { SafeJSON } from "@app/utils/json";
 import type { Args, CLIOptions, OutputFormat } from "@ask/types";
 import { Command } from "commander";
 
+interface ValidationResult {
+    valid: boolean;
+    errors: string[];
+}
+
+interface OutputFormatResult {
+    type: OutputFormat;
+    filename?: string;
+}
+
 export function parseCLIArguments(): Args {
     const program = new Command()
         .name("ask")
@@ -162,7 +172,7 @@ export function showVersion(): void {
     console.log("Multi-provider LLM chat application for GenesisTools");
 }
 
-export function validateOptions(options: CLIOptions): { valid: boolean; errors: string[] } {
+export function validateOptions(options: CLIOptions): ValidationResult {
     const errors: string[] = [];
 
     // Validate temperature
@@ -253,7 +263,7 @@ export function shouldShowVersion(options: CLIOptions): boolean {
  *  -f json                → json to stdout
  *  -f markdown -o out.md  → markdown written to file
  */
-export function getOutputFormat(options: CLIOptions): { type: OutputFormat; filename?: string } | undefined {
+export function getOutputFormat(options: CLIOptions): OutputFormatResult | undefined {
     // -o implies file output
     if (options.output) {
         return { type: "file", filename: options.output };

src/ask/utils/websearch.ts

@@ -1,6 +1,12 @@
 import logger from "@app/logger";
 import type { SearchResult, WebSearchOptions } from "@ask/types";
 
+interface WebSearchParams {
+    query: string;
+    numResults?: number;
+    safeSearch?: string;
+}
+
 export class WebSearchTool {
     private apiKey?: string;
     private baseURL = "https://api.search.brave.com/res/v1";
@@ -175,7 +181,7 @@ export class WebSearchTool {
                     optional: true,
                 },
             },
-            execute: async (params: { query: string; numResults?: number; safeSearch?: string }) => {
+            execute: async (params: WebSearchParams) => {
                 try {
                     const results = await this.searchWeb(params.query, {
                         numResults: params.numResults,

src/automate/lib/builtins.ts

@@ -3,7 +3,7 @@
 import { SafeJSON } from "@app/utils/json";
 import * as p from "@clack/prompts";
 import { resolveExpression, resolveParams } from "./expressions.ts";
-import type { ExecutionContext, PresetStep, StepResult } from "./types.ts";
+import type { ExecutionContext, PresetStep, StepHandlerResult, StepResult } from "./types.ts";
 
 /** The set of built-in action names that are handled directly (not via Bun.spawn) */
 export const BUILTIN_ACTIONS = new Set(["if", "log", "prompt", "shell", "set"]);
@@ -20,7 +20,7 @@ export function isBuiltinAction(action: string): boolean {
 export async function executeBuiltin(
     step: PresetStep,
     ctx: ExecutionContext
-): Promise<{ result: StepResult; jumpTo?: string }> {
+): Promise<StepHandlerResult> {
     const start = Date.now();
 
     switch (step.action) {
@@ -40,7 +40,7 @@ export async function executeBuiltin(
 }
 
 /** if -- evaluate condition expression, return jumpTo target step ID */
-function handleIf(step: PresetStep, ctx: ExecutionContext, start: number): { result: StepResult; jumpTo?: string } {
+function handleIf(step: PresetStep, ctx: ExecutionContext, start: number): StepHandlerResult {
     if (!step.condition) {
         throw new Error(`Step "${step.id}": "if" action requires a "condition" field`);
     }
@@ -60,7 +60,7 @@ function handleIf(step: PresetStep, ctx: ExecutionContext, start: number): { res
 }
 
 /** log -- print a resolved message to console via @clack/prompts */
-function handleLog(step: PresetStep, ctx: ExecutionContext, start: number): { result: StepResult } {
+function handleLog(step: PresetStep, ctx: ExecutionContext, start: number): StepHandlerResult {
     const params = step.params ? resolveParams(step.params as Record<string, unknown>, ctx) : {};
     const message = String(params.message ?? "");
 
@@ -76,7 +76,7 @@ function handleLog(step: PresetStep, ctx: ExecutionContext, start: number): { re
 }
 
 /** prompt -- ask user a question interactively, store answer as output */
-async function handlePrompt(step: PresetStep, ctx: ExecutionContext, start: number): Promise<{ result: StepResult }> {
+async function handlePrompt(step: PresetStep, ctx: ExecutionContext, start: number): Promise<StepHandlerResult> {
     const params = step.params ? resolveParams(step.params as Record<string, unknown>, ctx) : {};
     const message = String(params.message ?? "Enter value:");
     const defaultValue = params.default != null ? String(params.default) : undefined;
@@ -108,7 +108,7 @@ async function handlePrompt(step: PresetStep, ctx: ExecutionContext, start: numb
 }
 
 /** shell -- run a raw shell command via bash, capture stdout/stderr */
-async function handleShell(step: PresetStep, ctx: ExecutionContext, start: number): Promise<{ result: StepResult }> {
+async function handleShell(step: PresetStep, ctx: ExecutionContext, start: number): Promise<StepHandlerResult> {
     const params = step.params ? resolveParams(step.params as Record<string, unknown>, ctx) : {};
     const command = String(params.command ?? params.cmd ?? "");
 
@@ -162,7 +162,7 @@ async function handleShell(step: PresetStep, ctx: ExecutionContext, start: numbe
 }
 
 /** set -- set key-value pairs into ctx.vars */
-function handleSet(step: PresetStep, ctx: ExecutionContext, start: number): { result: StepResult } {
+function handleSet(step: PresetStep, ctx: ExecutionContext, start: number): StepHandlerResult {
     const params = step.params ? resolveParams(step.params as Record<string, unknown>, ctx) : {};
 
     for (const [key, value] of Object.entries(params)) {

src/automate/lib/step-runner.ts

@@ -5,7 +5,7 @@ import { executeBuiltin, isBuiltinAction } from "./builtins.ts";
 import { resolveExpression, resolveParams } from "./expressions.ts";
 import type { StepContext } from "./registry.ts";
 import { resolveStepHandler } from "./registry.ts";
-import type { ExecutionContext, PresetStep, StepResult } from "./types.ts";
+import type { ExecutionContext, PresetStep, StepHandlerResult, StepResult } from "./types.ts";
 
 /**
  * Execute a single step.
@@ -18,7 +18,7 @@ export async function executeStep(
     step: PresetStep,
     ctx: ExecutionContext,
     options: { dryRun?: boolean; verbose?: boolean }
-): Promise<{ result: StepResult; jumpTo?: string }> {
+): Promise<StepHandlerResult> {
     // Dispatch built-in actions
     if (isBuiltinAction(step.action)) {
         if (options.dryRun) {

src/automate/lib/types.ts

@@ -75,6 +75,12 @@ export interface StepResult {
     error?: string;
 }
 
+/** Result from a step handler, with optional jump target for branching */
+export interface StepHandlerResult {
+    result: StepResult;
+    jumpTo?: string;
+}
+
 /** Run options from CLI flags */
 export interface RunOptions {
     dryRun?: boolean;

src/azure-devops/commands/timelog/prepare-import.ts

@@ -42,6 +42,27 @@ interface PrepareImportFile {
     entries: StoredEntry[];
 }
 
+interface TimelogAddOptions {
+    from?: string;
+    to?: string;
+    name?: string;
+    entry: string;
+}
+
+interface TimelogRemoveOptions {
+    name: string;
+    id: string;
+}
+
+interface TimelogListOptions {
+    name: string;
+    format?: string;
+}
+
+interface TimelogClearOptions {
+    name: string;
+}
+
 // ============= Helpers =============
 
 const storage = new Storage("azure-devops");
@@ -83,7 +104,7 @@ function printEntry(entry: StoredEntry): void {
 
 // ============= Subcommand Actions =============
 
-async function handleAdd(options: { from?: string; to?: string; name?: string; entry: string }): Promise<void> {
+async function handleAdd(options: TimelogAddOptions): Promise<void> {
     const fileName = resolveFileName(options);
 
     // Parse and validate entry JSON
@@ -187,7 +208,7 @@ async function handleAdd(options: { from?: string; to?: string; name?: string; e
     printEntry(storedEntry);
 }
 
-async function handleRemove(options: { name: string; id: string }): Promise<void> {
+async function handleRemove(options: TimelogRemoveOptions): Promise<void> {
     const key = cacheKey(options.name);
 
     const updated = await storage.atomicUpdate<PrepareImportFile>(key, (current) => {
@@ -207,7 +228,7 @@ async function handleRemove(options: { name: string; id: string }): Promise<void
     console.log(`Entry ${options.id} removed. ${updated.entries.length} entries remaining.`);
 }
 
-async function handleList(options: { name: string; format?: string }): Promise<void> {
+async function handleList(options: TimelogListOptions): Promise<void> {
     const key = cacheKey(options.name);
     const data = await storage.getCacheFile<PrepareImportFile>(key, "30 days");
 
@@ -276,7 +297,7 @@ async function handleList(options: { name: string; format?: string }): Promise<v
     console.log(`\nGrand total: ${formatMinutes(grandTotal)}`);
 }
 
-async function handleClear(options: { name: string }): Promise<void> {
+async function handleClear(options: TimelogClearOptions): Promise<void> {
     const key = cacheKey(options.name);
     await storage.deleteCacheFile(key);
     console.log(`Prepare-import file "${options.name}" cleared.`);

src/claude-history-dashboard/src/integrations/tanstack-query/root-provider.tsx

@@ -1,12 +1,17 @@
 import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
 
+interface ProviderProps {
+	children: React.ReactNode;
+	queryClient: QueryClient;
+}
+
 export function getContext() {
 	const queryClient = new QueryClient();
 	return {
 		queryClient,
 	};
 }
 
-export function Provider({ children, queryClient }: { children: React.ReactNode; queryClient: QueryClient }) {
+export function Provider({ children, queryClient }: ProviderProps) {
 	return <QueryClientProvider client={queryClient}>{children}</QueryClientProvider>;
 }

src/github-release-notes/index.ts

@@ -16,6 +16,11 @@ interface GitHubRelease {
     html_url: string;
 }
 
+interface RepoIdentity {
+    owner: string;
+    repo: string;
+}
+
 interface ScriptOptions {
     owner: string;
     repo: string;
@@ -124,7 +129,7 @@ function generateMarkdown(releases: GitHubRelease[], owner: string, repo: string
     return headerContent + releasesContent;
 }
 
-function parseRepoArg(repoArg: string): { owner: string; repo: string } | null {
+function parseRepoArg(repoArg: string): RepoIdentity | null {
     // Accepts owner/repo or full github.com URL
     if (!repoArg) {
         return null;

src/macos-resources/index.tsx

@@ -111,6 +111,24 @@ interface CommandPerformance {
     timestamp: Date;
 }
 
+interface MemoizedHeaderProps {
+    children: React.ReactNode;
+    sortBy: "cpu" | "pid" | "files";
+}
+
+interface MemoizedCellProps {
+    children: React.ReactNode;
+    column: number;
+}
+
+interface MemoizedNotificationsPanelProps {
+    notifications: Notification[];
+}
+
+interface MemoizedCommandPanelProps {
+    commandHistory: CommandPerformance[];
+}
+
 // Custom Table component using Ink's Box and Text
 // const Table: React.FC<{
 //     data: Array<Record<string, string>>;
@@ -147,7 +165,7 @@ const generateTableData = (processes: ProcessInfo[], selectedIndex: number) => {
 
 // Memoized header component
 const MemoizedHeader = React.memo(
-    ({ children, sortBy }: { children: React.ReactNode; sortBy: "cpu" | "pid" | "files" }) => {
+    ({ children, sortBy }: MemoizedHeaderProps) => {
         const headerText = String(children);
         let sortIndicator = "";
         let color = "blue";
@@ -173,7 +191,7 @@ const MemoizedHeader = React.memo(
 );
 
 // Memoized cell component
-const MemoizedCell = React.memo(({ children, column }: { children: React.ReactNode; column: number }) => {
+const MemoizedCell = React.memo(({ children, column }: MemoizedCellProps) => {
     return (
         <Text color={column === 0 ? "green" : column === 2 ? "yellow" : column === 4 ? "cyan" : undefined}>
             {children}
@@ -182,7 +200,7 @@ const MemoizedCell = React.memo(({ children, column }: { children: React.ReactNo
 });
 
 // Memoized notifications panel
-const MemoizedNotificationsPanel = React.memo(({ notifications }: { notifications: Notification[] }) => {
+const MemoizedNotificationsPanel = React.memo(({ notifications }: MemoizedNotificationsPanelProps) => {
     return (
         <Box flexDirection="column" width="25%" height="50" marginLeft={1} borderStyle="single">
             <Box marginBottom={1}>
@@ -204,7 +222,7 @@ const MemoizedNotificationsPanel = React.memo(({ notifications }: { notification
 });
 
 // Memoized command performance panel
-const MemoizedCommandPanel = React.memo(({ commandHistory }: { commandHistory: CommandPerformance[] }) => {
+const MemoizedCommandPanel = React.memo(({ commandHistory }: MemoizedCommandPanelProps) => {
     return (
         <Box flexDirection="column" width="25%" height="50%" marginLeft={1} borderStyle="single" overflow="hidden">
             <Box marginBottom={1}>

src/markdown-cli/index.ts

@@ -5,6 +5,13 @@ import { type MarkdownRenderOptions, renderMarkdownToCli } from "@app/utils/mark
 import chokidar from "chokidar";
 import { Command, Option } from "commander";
 
+interface MarkdownCLIOptions {
+    watch?: boolean;
+    width?: number;
+    theme?: string;
+    color?: boolean;
+}
+
 const program = new Command();
 
 program
@@ -19,7 +26,7 @@ program
             .default("dark")
     )
     .option("--no-color", "Strip ANSI color codes from output")
-    .action((file?: string, opts?: { watch?: boolean; width?: number; theme?: string; color?: boolean }) => {
+    .action((file?: string, opts?: MarkdownCLIOptions) => {
         const renderOpts: MarkdownRenderOptions = {
             width: opts?.width && !Number.isNaN(opts.width) ? opts.width : undefined,
             theme: (opts?.theme as MarkdownRenderOptions["theme"]) || "dark",