refactor: clean up formatting and improve readability in multiple files

- Standardized string formatting by removing unnecessary line breaks in descriptions across tools.
- Enhanced readability by consolidating filter conditions into single lines in `get-team-patterns.ts` and `search-codebase.ts`.
- Improved error handling response structure in `index.ts` for consistency.
- Overall, these changes aim to maintain code clarity and consistency throughout the codebase.

Author: PatrickSys

src/index.ts

@@ -332,7 +332,8 @@ async function generateCodebaseContext(): Promise<string> {
     lines.push('# Codebase Intelligence');
     lines.push('');
     lines.push(
-      `Index: ${index.status} (${index.confidence}, ${index.action})${index.reason ? ` — ${index.reason}` : ''
+      `Index: ${index.status} (${index.confidence}, ${index.action})${
+        index.reason ? ` — ${index.reason}` : ''
       }`
     );
     lines.push('');
@@ -600,30 +601,45 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     // Gate INDEX_CONSUMING tools on a valid, healthy index
     let indexSignal: IndexSignal | undefined;
     if ((INDEX_CONSUMING_TOOL_NAMES as readonly string[]).includes(name)) {
-      if (indexState.status === "indexing") {
+      if (indexState.status === 'indexing') {
         return {
-          content: [{ type: "text", text: JSON.stringify({
-            status: "indexing",
-            message: "Index build in progress — please retry shortly",
-          }) }],
+          content: [
+            {
+              type: 'text',
+              text: JSON.stringify({
+                status: 'indexing',
+                message: 'Index build in progress — please retry shortly'
+              })
+            }
+          ]
         };
       }
-      if (indexState.status === "error") {
+      if (indexState.status === 'error') {
         return {
-          content: [{ type: "text", text: JSON.stringify({
-            status: "error",
-            message: `Indexer error: ${indexState.error}`,
-          }) }],
+          content: [
+            {
+              type: 'text',
+              text: JSON.stringify({
+                status: 'error',
+                message: `Indexer error: ${indexState.error}`
+              })
+            }
+          ]
         };
       }
       indexSignal = await ensureValidIndexOrAutoHeal();
-      if (indexSignal.action === "rebuild-failed") {
+      if (indexSignal.action === 'rebuild-failed') {
         return {
-          content: [{ type: "text", text: JSON.stringify({
-            error: "Index is corrupt and could not be rebuilt automatically.",
-            index: indexSignal,
-          }) }],
-          isError: true,
+          content: [
+            {
+              type: 'text',
+              text: JSON.stringify({
+                error: 'Index is corrupt and could not be rebuilt automatically.',
+                index: indexSignal
+              })
+            }
+          ],
+          isError: true
         };
       }
     }
@@ -632,7 +648,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       indexState,
       paths: PATHS,
       rootPath: ROOT_PATH,
-      performIndexing,
+      performIndexing
     };
 
     const result = await dispatchTool(name, args ?? {}, ctx);
@@ -641,15 +657,25 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     if (indexSignal !== undefined && result.content?.[0]) {
       try {
         const parsed = JSON.parse(result.content[0].text);
-        result.content[0] = { type: 'text', text: JSON.stringify({ ...parsed, index: indexSignal }) };
-      } catch { /* response wasn't JSON, skip injection */ }
+        result.content[0] = {
+          type: 'text',
+          text: JSON.stringify({ ...parsed, index: indexSignal })
+        };
+      } catch {
+        /* response wasn't JSON, skip injection */
+      }
     }
 
     return result;
   } catch (error) {
     return {
-      content: [{ type: "text", text: `Unexpected error: ${error instanceof Error ? error.message : String(error)}` }],
-      isError: true,
+      content: [
+        {
+          type: 'text',
+          text: `Unexpected error: ${error instanceof Error ? error.message : String(error)}`
+        }
+      ],
+      isError: true
     };
   }
 });

src/tools/detect-circular-dependencies.ts

@@ -14,8 +14,7 @@ export const definition: Tool = {
     properties: {
       scope: {
         type: 'string',
-        description:
-          "Optional path prefix to limit analysis (e.g., 'src/features', 'libs/shared')"
+        description: "Optional path prefix to limit analysis (e.g., 'src/features', 'libs/shared')"
       }
     }
   }

src/tools/get-symbol-references.ts

@@ -11,8 +11,7 @@ export const definition: Tool = {
     properties: {
       symbol: {
         type: 'string',
-        description:
-          'Symbol name to find references for (for example: parseConfig or UserService)'
+        description: 'Symbol name to find references for (for example: parseConfig or UserService)'
       },
       limit: {
         type: 'number',

src/tools/get-team-patterns.ts

@@ -73,9 +73,7 @@ export async function handle(
       };
 
       const relevantCategories = categoryMap[category || 'all'] || [];
-      const matchingMemories = allMemories.filter((m) =>
-        relevantCategories.includes(m.category)
-      );
+      const matchingMemories = allMemories.filter((m) => relevantCategories.includes(m.category));
 
       if (matchingMemories.length > 0) {
         result.memories = matchingMemories;

src/tools/search-codebase.ts

@@ -147,9 +147,7 @@ export async function handle(
   const searcher = new CodebaseSearcher(ctx.rootPath);
   let results: any[];
   const searchProfile =
-    intent && ['explore', 'edit', 'refactor', 'migrate'].includes(intent)
-      ? intent
-      : 'explore';
+    intent && ['explore', 'edit', 'refactor', 'migrate'].includes(intent) ? intent : 'explore';
 
   try {
     results = await searcher.search(queryStr, limit || 5, filters, {
@@ -252,9 +250,7 @@ export async function handle(
     const allImports = intelligence.internalFileGraph.imports as Record<string, string[]>;
     for (const [file, deps] of Object.entries(allImports)) {
       if (
-        deps.some((dep: string) =>
-          resultPaths.some((rp) => dep.endsWith(rp) || rp.endsWith(dep))
-        )
+        deps.some((dep: string) => resultPaths.some((rp) => dep.endsWith(rp) || rp.endsWith(dep)))
       ) {
         if (!resultPaths.some((rp) => file.endsWith(rp) || rp.endsWith(file))) {
           impactCandidates.push(file);
@@ -267,9 +263,7 @@ export async function handle(
   // Build reverse import map from intelligence graph
   const reverseImports = new Map<string, string[]>();
   if (intelligence?.internalFileGraph?.imports) {
-    for (const [file, deps] of Object.entries<string[]>(
-      intelligence.internalFileGraph.imports
-    )) {
+    for (const [file, deps] of Object.entries<string[]>(intelligence.internalFileGraph.imports)) {
       for (const dep of deps) {
         if (!reverseImports.has(dep)) reverseImports.set(dep, []);
         reverseImports.get(dep)!.push(file);
@@ -292,9 +286,7 @@ export async function handle(
     // imports: files this result depends on (forward lookup)
     const imports: string[] = [];
     if (intelligence?.internalFileGraph?.imports) {
-      for (const [file, deps] of Object.entries<string[]>(
-        intelligence.internalFileGraph.imports
-      )) {
+      for (const [file, deps] of Object.entries<string[]>(intelligence.internalFileGraph.imports)) {
         if (file.endsWith(rPath) || rPath.endsWith(file)) {
           imports.push(...deps);
         }
@@ -547,10 +539,7 @@ export async function handle(
 
   // For edit/refactor/migrate: return full preflight card (risk, patterns, impact, etc.).
   // For explore or lite-only: return flattened { ready, reason }.
-  let preflightPayload:
-    | { ready: boolean; reason?: string }
-    | Record<string, unknown>
-    | undefined;
+  let preflightPayload: { ready: boolean; reason?: string } | Record<string, unknown> | undefined;
   if (preflight) {
     const el = preflight.evidenceLock;
     // Full card per tool schema; add top-level ready/reason for backward compatibility