refactor: add type annotations to smartfetch utils and network

src/tools/smartfetch/network.ts

@@ -16,7 +16,12 @@ import type {
 } from './types';
 import { trimBlankRuns } from './utils';
 
-export function normalizeUrl(input: string) {
+export function normalizeUrl(input: string): {
+  url: string;
+  upgradedToHttps: boolean;
+  fallbackUrl: string | undefined;
+  originalUrl: string;
+} {
   const parsed = new URL(input);
   const originalUrl = parsed.toString();
   let upgradedToHttps = false;
@@ -29,7 +34,7 @@ export function normalizeUrl(input: string) {
   return { url: parsed.toString(), upgradedToHttps, fallbackUrl, originalUrl };
 }
 
-export function isDocsLikeUrl(url: URL) {
+export function isDocsLikeUrl(url: URL): boolean {
   const host = url.hostname.toLowerCase();
   return (
     DOCS_HOST_SUFFIXES.some((suffix) => host.endsWith(suffix)) ||
@@ -40,7 +45,7 @@ export function isDocsLikeUrl(url: URL) {
 export function buildPermissionPatterns(
   normalized: ReturnType<typeof normalizeUrl>,
   shouldProbeLlmsTxt: boolean,
-) {
+): string[] {
   const patterns = new Set<string>([normalized.url]);
   const origins = [new URL(normalized.url).origin];
   if (normalized.fallbackUrl) {

src/tools/smartfetch/utils.ts

@@ -10,7 +10,7 @@ async function getJSDOM() {
   return JSDOM;
 }
 
-export function wordCount(text: string) {
+export function wordCount(text: string): number {
   const trimmed = text.trim();
   if (!trimmed) return 0;
   return trimmed.split(/\s+/).length;
@@ -24,7 +24,7 @@ function quote(value: unknown) {
   return JSON.stringify(value ?? '');
 }
 
-export function frontmatter(metadata: Record<string, unknown>) {
+export function frontmatter(metadata: Record<string, unknown>): string {
   const lines = ['---'];
   for (const [key, value] of Object.entries(metadata)) {
     if (value === undefined) continue;
@@ -43,7 +43,7 @@ export function frontmatter(metadata: Record<string, unknown>) {
   return lines.join('\n');
 }
 
-export function trimBlankRuns(input: string) {
+export function trimBlankRuns(input: string): string {
   return input.replace(/\n{3,}/g, '\n\n').trim();
 }
 
@@ -154,7 +154,7 @@ function extractStructuredText(root: Element | null) {
   return cleanExtractedText(chunks.join(''));
 }
 
-export function cleanHeadingText(input: string) {
+export function cleanHeadingText(input: string): string {
   const normalized = trimBlankRuns(input).replace(/¶+$/g, '').trim();
   if (/^(?:C|F)#$/.test(normalized)) return normalized;
   if (/\s#+$/.test(normalized)) {
@@ -163,7 +163,7 @@ export function cleanHeadingText(input: string) {
   return normalized;
 }
 
-export function cleanFetchedMarkdown(input: string) {
+export function cleanFetchedMarkdown(input: string): string {
   const output = mapOutsideCodeBlocks(input, (value) =>
     value
       .replace(/^\s*!\[[^\]]*\]\([^)]+\)\s*$/gm, 'Image omitted')
@@ -178,7 +178,7 @@ export function cleanFetchedMarkdown(input: string) {
   return trimBlankRuns(output);
 }
 
-export function cleanFetchedText(input: string) {
+export function cleanFetchedText(input: string): string {
   return trimBlankRuns(input);
 }
 
@@ -195,7 +195,7 @@ export function withTruncationMarker(
   content: string,
   format: 'text' | 'markdown' | 'html',
   truncated: boolean,
-) {
+): string {
   if (!truncated) return content;
   if (format === 'html') return `${content}\n<!-- [..content truncated..] -->`;
   return `${content}\n\n[..content truncated..]`;
@@ -205,7 +205,7 @@ export function joinRenderedContent(
   metadata: string,
   content: string,
   format: 'text' | 'markdown' | 'html',
-) {
+): string {
   if (!metadata) return content;
   if (!content) {
     return format === 'html' ? `<!--\n${metadata.trim()}\n-->` : metadata;
@@ -226,7 +226,7 @@ export function joinRenderedContent(
 export function renderMessageForFormat(
   content: string,
   format: 'text' | 'markdown' | 'html',
-) {
+): string {
   if (format === 'html') return `<pre>${escapeHtml(content)}</pre>`;
   return content;
 }