bluera-knowledge 0.9.32 → 0.9.36

package/src/cli/commands/store.test.ts

@@ -82,7 +82,7 @@ describe('store command execution', () => {
       mockServices.store.list.mockResolvedValue(mockStores);
 
       const command = createStoreCommand(getOptions);
-      const listCommand = command.commands.find(c => c.name() === 'list');
+      const listCommand = command.commands.find((c) => c.name() === 'list');
       const actionHandler = listCommand?._actionHandler;
 
       await actionHandler!([]);
@@ -109,7 +109,7 @@ describe('store command execution', () => {
       mockServices.store.list.mockResolvedValue(mockStores);
 
       const command = createStoreCommand(getOptions);
-      const listCommand = command.commands.find(c => c.name() === 'list');
+      const listCommand = command.commands.find((c) => c.name() === 'list');
       const actionHandler = listCommand?._actionHandler;
 
       listCommand.parseOptions(['--type', 'file']);
@@ -123,7 +123,7 @@ describe('store command execution', () => {
       mockServices.store.list.mockResolvedValue([]);
 
       const command = createStoreCommand(getOptions);
-      const listCommand = command.commands.find(c => c.name() === 'list');
+      const listCommand = command.commands.find((c) => c.name() === 'list');
       const actionHandler = listCommand?._actionHandler;
 
       await actionHandler!([]);
@@ -153,7 +153,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const listCommand = command.commands.find(c => c.name() === 'list');
+      const listCommand = command.commands.find((c) => c.name() === 'list');
       const actionHandler = listCommand?._actionHandler;
 
       await actionHandler!([]);
@@ -193,7 +193,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const listCommand = command.commands.find(c => c.name() === 'list');
+      const listCommand = command.commands.find((c) => c.name() === 'list');
       const actionHandler = listCommand?._actionHandler;
 
       await actionHandler!([]);
@@ -221,10 +221,17 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
-      createCommand.parseOptions(['--type', 'file', '--source', '/path/to/files', '--description', 'My file store']);
+      createCommand.parseOptions([
+        '--type',
+        'file',
+        '--source',
+        '/path/to/files',
+        '--description',
+        'My file store',
+      ]);
       await actionHandler!(['my-files']);
 
       expect(mockServices.store.create).toHaveBeenCalledWith({
@@ -257,7 +264,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'repo', '--source', '/path/to/repo']);
@@ -271,9 +278,7 @@ describe('store command execution', () => {
         description: undefined,
         tags: undefined,
       });
-      expect(consoleLogSpy).toHaveBeenCalledWith(
-        expect.stringContaining('Created store: my-repo')
-      );
+      expect(consoleLogSpy).toHaveBeenCalledWith(expect.stringContaining('Created store: my-repo'));
     });
 
     it('creates a web store successfully', async () => {
@@ -293,7 +298,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'web', '--source', 'https://docs.example.com']);
@@ -307,9 +312,7 @@ describe('store command execution', () => {
         description: undefined,
         tags: undefined,
       });
-      expect(consoleLogSpy).toHaveBeenCalledWith(
-        expect.stringContaining('Created store: my-docs')
-      );
+      expect(consoleLogSpy).toHaveBeenCalledWith(expect.stringContaining('Created store: my-docs'));
     });
 
     it('creates store with tags', async () => {
@@ -329,10 +332,17 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
-      createCommand.parseOptions(['--type', 'file', '--source', '/path/to/files', '--tags', 'typescript, react, frontend']);
+      createCommand.parseOptions([
+        '--type',
+        'file',
+        '--source',
+        '/path/to/files',
+        '--tags',
+        'typescript, react, frontend',
+      ]);
       await actionHandler!(['tagged-store']);
 
       expect(mockServices.store.create).toHaveBeenCalledWith({
@@ -368,7 +378,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'file', '--source', '/path/to/files']);
@@ -388,7 +398,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'file', '--source', '/path/to/files']);
@@ -414,7 +424,7 @@ describe('store command execution', () => {
       mockServices.store.getByIdOrName.mockResolvedValue(mockStore);
 
       const command = createStoreCommand(getOptions);
-      const infoCommand = command.commands.find(c => c.name() === 'info');
+      const infoCommand = command.commands.find((c) => c.name() === 'info');
       const actionHandler = infoCommand?._actionHandler;
 
       await actionHandler!(['my-store']);
@@ -449,7 +459,7 @@ describe('store command execution', () => {
       mockServices.store.getByIdOrName.mockResolvedValue(mockStore);
 
       const command = createStoreCommand(getOptions);
-      const infoCommand = command.commands.find(c => c.name() === 'info');
+      const infoCommand = command.commands.find((c) => c.name() === 'info');
       const actionHandler = infoCommand?._actionHandler;
 
       await actionHandler!(['my-repo']);
@@ -477,7 +487,7 @@ describe('store command execution', () => {
       mockServices.store.getByIdOrName.mockResolvedValue(mockStore);
 
       const command = createStoreCommand(getOptions);
-      const infoCommand = command.commands.find(c => c.name() === 'info');
+      const infoCommand = command.commands.find((c) => c.name() === 'info');
       const actionHandler = infoCommand?._actionHandler;
 
       await actionHandler!(['my-docs']);
@@ -509,7 +519,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const infoCommand = command.commands.find(c => c.name() === 'info');
+      const infoCommand = command.commands.find((c) => c.name() === 'info');
       const actionHandler = infoCommand?._actionHandler;
 
       await actionHandler!(['my-store']);
@@ -523,7 +533,7 @@ describe('store command execution', () => {
       mockServices.store.getByIdOrName.mockResolvedValue(undefined);
 
       const command = createStoreCommand(getOptions);
-      const infoCommand = command.commands.find(c => c.name() === 'info');
+      const infoCommand = command.commands.find((c) => c.name() === 'info');
       const actionHandler = infoCommand?._actionHandler;
 
       await expect(actionHandler!(['nonexistent'])).rejects.toThrow('process.exit: 3');
@@ -546,7 +556,7 @@ describe('store command execution', () => {
       mockServices.store.getByIdOrName.mockResolvedValue(mockStore);
 
       const command = createStoreCommand(getOptions);
-      const infoCommand = command.commands.find(c => c.name() === 'info');
+      const infoCommand = command.commands.find((c) => c.name() === 'info');
       const actionHandler = infoCommand?._actionHandler;
 
       await actionHandler!([storeId]);
@@ -573,7 +583,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
       const actionHandler = deleteCommand?._actionHandler;
 
       deleteCommand.parseOptions(['--force']);
@@ -601,7 +611,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
       const actionHandler = deleteCommand?._actionHandler;
 
       deleteCommand.parseOptions(['--yes']);
@@ -615,7 +625,7 @@ describe('store command execution', () => {
       mockServices.store.getByIdOrName.mockResolvedValue(undefined);
 
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
       const actionHandler = deleteCommand?._actionHandler;
 
       deleteCommand.parseOptions(['--force']);
@@ -645,7 +655,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
       const actionHandler = deleteCommand?._actionHandler;
 
       deleteCommand.parseOptions(['--force']);
@@ -675,7 +685,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
       const actionHandler = deleteCommand?._actionHandler;
 
       await expect(actionHandler!(['my-store'])).rejects.toThrow('process.exit: 1');
@@ -728,7 +738,7 @@ describe('store command execution', () => {
       }));
 
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
       const actionHandler = deleteCommand?._actionHandler;
 
       await actionHandler!(['my-store']);
@@ -775,7 +785,7 @@ describe('store command execution', () => {
       }));
 
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
       const actionHandler = deleteCommand?._actionHandler;
 
       await expect(actionHandler!(['my-store'])).rejects.toThrow('process.exit: 0');
@@ -798,25 +808,25 @@ describe('store command execution', () => {
 
       expect(command.name()).toBe('store');
       expect(command.commands.length).toBe(4);
-      expect(command.commands.map(c => c.name())).toEqual(['list', 'create', 'info', 'delete']);
+      expect(command.commands.map((c) => c.name())).toEqual(['list', 'create', 'info', 'delete']);
     });
 
     it('list subcommand has type option', () => {
       const command = createStoreCommand(getOptions);
-      const listCommand = command.commands.find(c => c.name() === 'list');
-      const typeOption = listCommand?.options.find(o => o.long === '--type');
+      const listCommand = command.commands.find((c) => c.name() === 'list');
+      const typeOption = listCommand?.options.find((o) => o.long === '--type');
 
       expect(typeOption).toBeDefined();
     });
 
     it('create subcommand has required options', () => {
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
 
-      const typeOption = createCommand?.options.find(o => o.long === '--type');
-      const sourceOption = createCommand?.options.find(o => o.long === '--source');
-      const descriptionOption = createCommand?.options.find(o => o.long === '--description');
-      const tagsOption = createCommand?.options.find(o => o.long === '--tags');
+      const typeOption = createCommand?.options.find((o) => o.long === '--type');
+      const sourceOption = createCommand?.options.find((o) => o.long === '--source');
+      const descriptionOption = createCommand?.options.find((o) => o.long === '--description');
+      const tagsOption = createCommand?.options.find((o) => o.long === '--tags');
 
       expect(typeOption).toBeDefined();
       expect(typeOption?.mandatory).toBe(true);
@@ -830,10 +840,10 @@ describe('store command execution', () => {
 
     it('delete subcommand has force and yes options', () => {
       const command = createStoreCommand(getOptions);
-      const deleteCommand = command.commands.find(c => c.name() === 'delete');
+      const deleteCommand = command.commands.find((c) => c.name() === 'delete');
 
-      const forceOption = deleteCommand?.options.find(o => o.long === '--force');
-      const yesOption = deleteCommand?.options.find(o => o.long === '--yes');
+      const forceOption = deleteCommand?.options.find((o) => o.long === '--force');
+      const yesOption = deleteCommand?.options.find((o) => o.long === '--yes');
 
       expect(forceOption).toBeDefined();
       expect(yesOption).toBeDefined();
@@ -858,10 +868,17 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
-      createCommand.parseOptions(['--type', 'file', '--source', '/path/to/files', '--tags', 'tag1, tag2, tag3']);
+      createCommand.parseOptions([
+        '--type',
+        'file',
+        '--source',
+        '/path/to/files',
+        '--tags',
+        'tag1, tag2, tag3',
+      ]);
       await actionHandler!(['tagged-store']);
 
       expect(mockServices.store.create).toHaveBeenCalledWith(
@@ -888,10 +905,17 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
-      createCommand.parseOptions(['--type', 'file', '--source', '/path/to/files', '--tags', '  tag1  ,  tag2  ']);
+      createCommand.parseOptions([
+        '--type',
+        'file',
+        '--source',
+        '/path/to/files',
+        '--tags',
+        '  tag1  ,  tag2  ',
+      ]);
       await actionHandler!(['tagged-store']);
 
       expect(mockServices.store.create).toHaveBeenCalledWith(
@@ -919,7 +943,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'file', '--source', '/local/path']);
@@ -951,7 +975,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'repo', '--source', '/repo/path']);
@@ -984,7 +1008,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'repo', '--source', 'https://github.com/user/repo']);
@@ -1018,10 +1042,15 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
-      createCommand.parseOptions(['--type', 'repo', '--source', 'http://internal-git.example.com/repo']);
+      createCommand.parseOptions([
+        '--type',
+        'repo',
+        '--source',
+        'http://internal-git.example.com/repo',
+      ]);
       await actionHandler!(['repo-http-store']);
 
       expect(mockServices.store.create).toHaveBeenCalledWith({
@@ -1051,7 +1080,7 @@ describe('store command execution', () => {
       });
 
       const command = createStoreCommand(getOptions);
-      const createCommand = command.commands.find(c => c.name() === 'create');
+      const createCommand = command.commands.find((c) => c.name() === 'create');
       const actionHandler = createCommand?._actionHandler;
 
       createCommand.parseOptions(['--type', 'web', '--source', 'https://example.com']);

package/src/cli/commands/store.ts

@@ -1,10 +1,12 @@
 import { Command } from 'commander';
 import { createServices, destroyServices } from '../../services/index.js';
-import type { GlobalOptions } from '../program.js';
 import type { StoreType } from '../../types/store.js';
+import type { GlobalOptions } from '../program.js';
 
 export function createStoreCommand(getOptions: () => GlobalOptions): Command {
-  const store = new Command('store').description('Manage knowledge stores (collections of indexed documents)');
+  const store = new Command('store').description(
+    'Manage knowledge stores (collections of indexed documents)'
+  );
 
   store
     .command('list')
@@ -42,48 +44,63 @@ export function createStoreCommand(getOptions: () => GlobalOptions): Command {
   store
     .command('create <name>')
     .description('Create a new store pointing to a local path or URL')
-    .requiredOption('-t, --type <type>', 'Store type: file (local dir), repo (git), web (crawled site)')
+    .requiredOption(
+      '-t, --type <type>',
+      'Store type: file (local dir), repo (git), web (crawled site)'
+    )
     .requiredOption('-s, --source <path>', 'Local path for file/repo stores, URL for web stores')
     .option('-d, --description <desc>', 'Optional description for the store')
     .option('--tags <tags>', 'Comma-separated tags for filtering')
-    .action(async (name: string, options: {
-      type: StoreType;
-      source: string;
-      description?: string;
-      tags?: string;
-    }) => {
-      const globalOpts = getOptions();
-      const services = await createServices(globalOpts.config, globalOpts.dataDir);
-      let exitCode = 0;
-      try {
-        // Detect if source is a URL (for repo stores that should clone from remote)
-        const isUrl = options.source.startsWith('http://') || options.source.startsWith('https://');
-        const result = await services.store.create({
-          name,
-          type: options.type,
-          path: options.type === 'file' || (options.type === 'repo' && !isUrl) ? options.source : undefined,
-          url: options.type === 'web' || (options.type === 'repo' && isUrl) ? options.source : undefined,
-          description: options.description,
-          tags: options.tags?.split(',').map((t) => t.trim()),
-        });
+    .action(
+      async (
+        name: string,
+        options: {
+          type: StoreType;
+          source: string;
+          description?: string;
+          tags?: string;
+        }
+      ) => {
+        const globalOpts = getOptions();
+        const services = await createServices(globalOpts.config, globalOpts.dataDir);
+        let exitCode = 0;
+        try {
+          // Detect if source is a URL (for repo stores that should clone from remote)
+          const isUrl =
+            options.source.startsWith('http://') || options.source.startsWith('https://');
+          const result = await services.store.create({
+            name,
+            type: options.type,
+            path:
+              options.type === 'file' || (options.type === 'repo' && !isUrl)
+                ? options.source
+                : undefined,
+            url:
+              options.type === 'web' || (options.type === 'repo' && isUrl)
+                ? options.source
+                : undefined,
+            description: options.description,
+            tags: options.tags?.split(',').map((t) => t.trim()),
+          });
 
-        if (result.success) {
-          if (globalOpts.format === 'json') {
-            console.log(JSON.stringify(result.data, null, 2));
+          if (result.success) {
+            if (globalOpts.format === 'json') {
+              console.log(JSON.stringify(result.data, null, 2));
+            } else {
+              console.log(`\nCreated store: ${result.data.name} (${result.data.id})\n`);
+            }
           } else {
-            console.log(`\nCreated store: ${result.data.name} (${result.data.id})\n`);
+            console.error(`Error: ${result.error.message}`);
+            exitCode = 1;
           }
-        } else {
-          console.error(`Error: ${result.error.message}`);
-          exitCode = 1;
+        } finally {
+          await destroyServices(services);
+        }
+        if (exitCode !== 0) {
+          process.exit(exitCode);
         }
-      } finally {
-        await destroyServices(services);
-      }
-      if (exitCode !== 0) {
-        process.exit(exitCode);
       }
-    });
+    );
 
   store
     .command('info <store>')
@@ -137,7 +154,9 @@ export function createStoreCommand(getOptions: () => GlobalOptions): Command {
         const skipConfirmation = options.force === true || options.yes === true;
         if (!skipConfirmation) {
           if (!process.stdin.isTTY) {
-            console.error('Error: Use --force or -y to delete without confirmation in non-interactive mode');
+            console.error(
+              'Error: Use --force or -y to delete without confirmation in non-interactive mode'
+            );
             process.exit(1);
           }
           // Interactive confirmation

package/src/cli/program.ts

@@ -1,7 +1,7 @@
-import { Command } from 'commander';
 import { readFileSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
 import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { Command } from 'commander';
 
 interface PackageJson {
   version: string;

package/src/crawl/article-converter.test.ts

@@ -40,7 +40,10 @@ describe('convertHtmlToMarkdown', () => {
       const html = '<html><body><article><h1>Title</h1><p>Content</p></article></body></html>';
       const result = await convertHtmlToMarkdown(html, 'https://example.com');
 
-      expect(vi.mocked(articleExtractor.extractFromHtml)).toHaveBeenCalledWith(html, 'https://example.com');
+      expect(vi.mocked(articleExtractor.extractFromHtml)).toHaveBeenCalledWith(
+        html,
+        'https://example.com'
+      );
       expect(result.success).toBe(true);
     });
 

package/src/crawl/article-converter.ts

@@ -27,10 +27,7 @@ export interface ConversionResult {
  * 3. Convert to markdown with Turndown + GFM
  * 4. Cleanup markdown (regex patterns)
  */
-export async function convertHtmlToMarkdown(
-  html: string,
-  url: string,
-): Promise<ConversionResult> {
+export async function convertHtmlToMarkdown(html: string, url: string): Promise<ConversionResult> {
   logger.debug({ url, htmlLength: html.length }, 'Starting HTML conversion');
 
   try {
@@ -40,28 +37,37 @@ export async function convertHtmlToMarkdown(
 
     try {
       const article = await extractFromHtml(html, url);
-      if (article !== null && article.content !== undefined && article.content !== '') {
+      if (article?.content !== undefined && article.content !== '') {
         articleHtml = article.content;
         title = article.title !== undefined && article.title !== '' ? article.title : undefined;
-        logger.debug({
-          url,
-          title,
-          extractedLength: articleHtml.length,
-          usedFullHtml: false,
-        }, 'Article content extracted');
+        logger.debug(
+          {
+            url,
+            title,
+            extractedLength: articleHtml.length,
+            usedFullHtml: false,
+          },
+          'Article content extracted'
+        );
       } else {
         // Fallback to full HTML if extraction fails
         articleHtml = html;
-        logger.debug({ url, usedFullHtml: true }, 'Article extraction returned empty, using full HTML');
+        logger.debug(
+          { url, usedFullHtml: true },
+          'Article extraction returned empty, using full HTML'
+        );
       }
     } catch (extractError) {
       // Fallback to full HTML if extraction fails
       articleHtml = html;
-      logger.debug({
-        url,
-        usedFullHtml: true,
-        error: extractError instanceof Error ? extractError.message : String(extractError),
-      }, 'Article extraction failed, using full HTML');
+      logger.debug(
+        {
+          url,
+          usedFullHtml: true,
+          error: extractError instanceof Error ? extractError.message : String(extractError),
+        },
+        'Article extraction failed, using full HTML'
+      );
     }
 
     // Step 2: Preprocess HTML for code blocks
@@ -100,18 +106,24 @@ export async function convertHtmlToMarkdown(
     // Step 4: Cleanup markdown with comprehensive regex patterns
     const markdown = cleanupMarkdown(rawMarkdown);
 
-    logger.debug({
-      url,
-      title,
-      rawMarkdownLength: rawMarkdown.length,
-      finalMarkdownLength: markdown.length,
-    }, 'HTML to markdown conversion complete');
+    logger.debug(
+      {
+        url,
+        title,
+        rawMarkdownLength: rawMarkdown.length,
+        finalMarkdownLength: markdown.length,
+      },
+      'HTML to markdown conversion complete'
+    );
 
     // Log markdown preview at trace level
-    logger.trace({
-      url,
-      markdownPreview: truncateForLog(markdown, 1000),
-    }, 'Markdown content preview');
+    logger.trace(
+      {
+        url,
+        markdownPreview: truncateForLog(markdown, 1000),
+      },
+      'Markdown content preview'
+    );
 
     return {
       markdown,
@@ -119,10 +131,13 @@ export async function convertHtmlToMarkdown(
       success: true,
     };
   } catch (error) {
-    logger.error({
-      url,
-      error: error instanceof Error ? error.message : String(error),
-    }, 'HTML to markdown conversion failed');
+    logger.error(
+      {
+        url,
+        error: error instanceof Error ? error.message : String(error),
+      },
+      'HTML to markdown conversion failed'
+    );
 
     return {
       markdown: '',