chore(lint): tighten converter typings

Author: alpha912

src/docx.ts

@@ -19,9 +19,12 @@ import {
   ChartBlock,
   DiagramBlock,
   OSFCodeBlock,
+  type TextRun as OSFTextRun,
 } from 'omniscript-parser';
 import { Converter, ConverterOptions, ConversionResult } from './types';
 
+type DocxElement = Paragraph | Table;
+
 export class DOCXConverter implements Converter {
   getSupportedFormats(): string[] {
     return ['docx'];
@@ -51,8 +54,11 @@ export class DOCXConverter implements Converter {
     };
   }
 
-  private generateDocumentElements(document: OSFDocument, options: ConverterOptions): any[] {
-    const elements: any[] = [];
+  private generateDocumentElements(
+    document: OSFDocument,
+    options: ConverterOptions
+  ): DocxElement[] {
+    const elements: DocxElement[] = [];
 
     for (const block of document.blocks) {
       switch (block.type) {
@@ -85,8 +91,8 @@ export class DOCXConverter implements Converter {
     return elements;
   }
 
-  private renderMetaBlock(meta: MetaBlock): any[] {
-    const elements: any[] = [];
+  private renderMetaBlock(meta: MetaBlock): DocxElement[] {
+    const elements: DocxElement[] = [];
 
     if (meta.props.title) {
       elements.push(
@@ -129,9 +135,9 @@ export class DOCXConverter implements Converter {
     return elements;
   }
 
-  private renderDocBlock(doc: DocBlock): any[] {
+  private renderDocBlock(doc: DocBlock): DocxElement[] {
     const content = doc.content || '';
-    const elements: any[] = [];
+    const elements: DocxElement[] = [];
 
     // Split content into lines and process markdown-like syntax
     const lines = content.split('\n');
@@ -212,8 +218,8 @@ export class DOCXConverter implements Converter {
     return elements;
   }
 
-  private renderSlideBlock(slide: SlideBlock): any[] {
-    const elements: any[] = [];
+  private renderSlideBlock(slide: SlideBlock): DocxElement[] {
+    const elements: DocxElement[] = [];
 
     // Add slide title
     if (slide.title) {
@@ -256,8 +262,8 @@ export class DOCXConverter implements Converter {
     return elements;
   }
 
-  private renderSheetBlock(sheet: SheetBlock): any[] {
-    const elements: any[] = [];
+  private renderSheetBlock(sheet: SheetBlock): DocxElement[] {
+    const elements: DocxElement[] = [];
 
     // Add sheet title
     if (sheet.name) {
@@ -361,7 +367,12 @@ export class DOCXConverter implements Converter {
       { regex: /`(.+?)`/g, format: { font: 'Courier New' } },
     ];
 
-    const matches: Array<{ index: number; length: number; text: string; format: any }> = [];
+    const matches: Array<{
+      index: number;
+      length: number;
+      text: string;
+      format: { bold?: boolean; italics?: boolean; font?: string };
+    }> = [];
 
     // Find all formatting matches
     for (const pattern of patterns) {
@@ -410,16 +421,18 @@ export class DOCXConverter implements Converter {
     return runs;
   }
 
-  private extractText(run: any): string {
+  private extractText(run: OSFTextRun): string {
     if (typeof run === 'string') return run;
-    if (run.type === 'link') return run.text;
-    if (run.type === 'image') return run.alt || '';
-    if (run.text) return run.text;
+    if ('type' in run) {
+      if (run.type === 'link') return run.text;
+      if (run.type === 'image') return run.alt || '';
+    }
+    if ('text' in run) return run.text;
     return '';
   }
 
-  private renderChartBlock(chart: ChartBlock): any[] {
-    const elements: any[] = [];
+  private renderChartBlock(chart: ChartBlock): DocxElement[] {
+    const elements: DocxElement[] = [];
 
     if (chart.title) {
       elements.push(
@@ -463,8 +476,8 @@ export class DOCXConverter implements Converter {
     return elements;
   }
 
-  private renderDiagramBlock(diagram: DiagramBlock): any[] {
-    const elements: any[] = [];
+  private renderDiagramBlock(diagram: DiagramBlock): DocxElement[] {
+    const elements: DocxElement[] = [];
 
     if (diagram.title) {
       elements.push(
@@ -510,8 +523,8 @@ export class DOCXConverter implements Converter {
     return elements;
   }
 
-  private renderCodeBlock(code: OSFCodeBlock): any[] {
-    const elements: any[] = [];
+  private renderCodeBlock(code: OSFCodeBlock): DocxElement[] {
+    const elements: DocxElement[] = [];
 
     if (code.caption) {
       elements.push(

src/pdf.ts

@@ -11,6 +11,7 @@ import {
   OSFCodeBlock,
   TableBlock,
 } from 'omniscript-parser';
+import type { TextRun as OSFTextRun } from 'omniscript-parser';
 import { Converter, ConverterOptions, ConversionResult } from './types';
 
 export class PDFConverter implements Converter {
@@ -182,12 +183,14 @@ export class PDFConverter implements Converter {
   }
 
   private renderDocBlock(doc: DocBlock, options: ConverterOptions): string {
+    void options;
     const content = doc.content || '';
     const html = this.renderMarkdown(content);
     return `<div class="doc-block">${html}</div>`;
   }
 
   private renderSlideBlock(slide: SlideBlock, options: ConverterOptions): string {
+    void options;
     let html = '<div class="slide-block">';
 
     if (slide.title) {
@@ -330,6 +333,7 @@ export class PDFConverter implements Converter {
   }
 
   private renderSheetBlock(sheet: SheetBlock, options: ConverterOptions): string {
+    void options;
     let html = '<div class="sheet-block">';
 
     if (sheet.name) {
@@ -381,6 +385,7 @@ export class PDFConverter implements Converter {
   }
 
   private renderTableBlock(table: TableBlock, options: ConverterOptions): string {
+    void options;
     const tableStyle = table.style || 'bordered';
     let html = '<div class="table-block">';
 
@@ -411,11 +416,13 @@ export class PDFConverter implements Converter {
     return html;
   }
 
-  private extractText(run: any): string {
+  private extractText(run: OSFTextRun): string {
     if (typeof run === 'string') return run;
-    if (run.type === 'link') return run.text;
-    if (run.type === 'image') return run.alt || '';
-    if (run.text) return run.text;
+    if ('type' in run) {
+      if (run.type === 'link') return run.text;
+      if (run.type === 'image') return run.alt || '';
+    }
+    if ('text' in run) return run.text;
     return '';
   }
 
@@ -429,6 +436,7 @@ export class PDFConverter implements Converter {
   }
 
   private renderChartBlock(chart: ChartBlock, options: ConverterOptions): string {
+    void options;
     let html = '<div class="chart-block" style="margin: 40px 0; page-break-inside: avoid;">';
 
     if (chart.title) {
@@ -441,10 +449,10 @@ export class PDFConverter implements Converter {
     html += '</div>';
 
     const chartPayload = {
-      labels: chart.data.map((d: any) => d.label),
-      datasets: chart.data.map((d: any, i: number) => ({
-        label: d.label,
-        data: d.values,
+      labels: chart.data.map(series => series.label),
+      datasets: chart.data.map((series, i) => ({
+        label: series.label,
+        data: series.values,
         backgroundColor: chart.options?.colors?.[i] || undefined,
       })),
     };
@@ -469,6 +477,7 @@ export class PDFConverter implements Converter {
   }
 
   private renderDiagramBlock(diagram: DiagramBlock, options: ConverterOptions): string {
+    void options;
     let html = '<div class="diagram-block" style="margin: 40px 0; page-break-inside: avoid;">';
 
     if (diagram.title) {
@@ -484,6 +493,7 @@ export class PDFConverter implements Converter {
   }
 
   private renderCodeBlock(code: OSFCodeBlock, options: ConverterOptions): string {
+    void options;
     let html = '<div class="code-block" style="margin: 40px 0; page-break-inside: avoid;">';
 
     if (code.caption) {