Enforce error handling for promises and improve command execution

eslint.config.mjs

@@ -33,17 +33,17 @@ export default tseslint.config(
           varsIgnorePattern: "^_",
         },
       ],
-      "@typescript-eslint/no-floating-promises": "warn",
-      "@typescript-eslint/no-misused-promises": "warn",
-      "@typescript-eslint/no-unsafe-argument": "warn",
-      "@typescript-eslint/no-unsafe-assignment": "warn",
-      "@typescript-eslint/no-unsafe-member-access": "warn",
-      "@typescript-eslint/no-unsafe-call": "warn",
-      "@typescript-eslint/no-unsafe-return": "warn",
-      "@typescript-eslint/no-base-to-string": "warn",
+      "@typescript-eslint/no-floating-promises": "error",
+      "@typescript-eslint/no-misused-promises": "error",
+      "@typescript-eslint/no-unsafe-argument": "error",
+      "@typescript-eslint/no-unsafe-assignment": "error",
+      "@typescript-eslint/no-unsafe-member-access": "error",
+      "@typescript-eslint/no-unsafe-call": "error",
+      "@typescript-eslint/no-unsafe-return": "error",
+      "@typescript-eslint/no-base-to-string": "error",
       "@typescript-eslint/prefer-promise-reject-errors": "warn",
       "@typescript-eslint/no-unnecessary-type-assertion": "warn",
-      "@typescript-eslint/restrict-template-expressions": "warn",
+      "@typescript-eslint/restrict-template-expressions": "error",
       "@typescript-eslint/no-redundant-type-constituents": "warn",
       "@typescript-eslint/require-await": "warn",
       "complexity": ["warn", { max: 20 }],
@@ -59,7 +59,7 @@ export default tseslint.config(
         "warn",
         { groups: ["builtin", "external", "internal", "parent", "sibling"] },
       ],
-      "import-x/no-duplicates": "warn",
+      "import-x/no-duplicates": "error",
 
       // unicorn (modern JS best practices)
       "unicorn/no-array-for-each": "warn",

src/core/parser-shared.test.ts

@@ -4,8 +4,7 @@
  *--------------------------------------------------------------------------------------------*/
 
 import { describe, it, expect } from 'vitest';
-import { detectDevcontainerFromRequests, extractSkillPathsFromText } from './parser-shared';
-import { createRequest, createSession } from './parser-shared';
+import { detectDevcontainerFromRequests, extractSkillPathsFromText, createRequest, createSession } from './parser-shared';
 import { SessionRequest } from './types';
 
 function makeReq(overrides: Partial<SessionRequest> = {}): SessionRequest {

src/extension.ts

@@ -36,7 +36,7 @@ function loadPanelModule(): Promise<PanelModule> {
 async function exportSummaryFromLogs(): Promise<void> {
   const dirs = findLogsDirs();
   if (dirs.length === 0) {
-    vscode.window.showErrorMessage('No AI coding session log directories found.');
+    void vscode.window.showErrorMessage('No AI coding session log directories found.');
     return;
   }
 
@@ -91,7 +91,7 @@ async function reviewPendingTrust(context: vscode.ExtensionContext): Promise<Set
     await approveTrust(context.globalState, item.entry.filePath, item.entry.content);
     approved.add(item.entry.filePath);
   }
-  vscode.window.showInformationMessage(
+  void vscode.window.showInformationMessage(
     `Approved ${approved.size} file${approved.size === 1 ? '' : 's'}. Reloading rules...`,
   );
   return approved;
@@ -185,7 +185,7 @@ export function activate(context: vscode.ExtensionContext) {
           for (const p of Object.keys(listApproved(context.globalState))) {
             await revokeTrust(context.globalState, p);
           }
-          vscode.window.showInformationMessage('All approvals revoked. Reload the dashboard to re-scan.');
+          void vscode.window.showInformationMessage('All approvals revoked. Reload the dashboard to re-scan.');
         }
         return;
       }

src/webview/page-burndown.ts

@@ -137,7 +137,7 @@ export function renderBurndown(container: HTMLElement, currentFilter: DateFilter
   }
 
   // Load disk-persisted budgets on first render
-  loadModelBudgetsFromDisk();
+  void loadModelBudgetsFromDisk();
 
   const now = new Date();
 
@@ -194,7 +194,7 @@ export function renderBurndown(container: HTMLElement, currentFilter: DateFilter
     // Auto-discover models on first render
     if (!modelsLoaded) {
       render(html`<div class="budget-loading">Loading models\u2026</div>`, target);
-      fetchHistoricalBudgets().then(peak => {
+      void fetchHistoricalBudgets().then(peak => {
         discoveredModels = peak;
         modelsLoaded = true;
         // Seed modelBudgets with discovered models (keep existing budgets)

src/webview/panel-request-service.ts

@@ -292,7 +292,7 @@ Generate 3 ${context.difficulty} interview-style questions tailored to this deve
     const prompt = isString(params.prompt) ? params.prompt : '';
     if (!prompt) return;
 
-    vscode.commands.executeCommand('workbench.action.chat.open', {
+    void vscode.commands.executeCommand('workbench.action.chat.open', {
       query: prompt,
     }).then(
       () => postResponse(this.webview, msg.id, { ok: true }),

src/webview/panel-sidebar.ts

@@ -28,7 +28,7 @@ export class DashboardSidebarProvider implements vscode.WebviewViewProvider {
     webviewView.webview.html = this.renderHtml(webviewView.webview);
 
     webviewView.webview.onDidReceiveMessage((msg: { command: string }) => {
-      vscode.commands.executeCommand(msg.command);
+      void vscode.commands.executeCommand(msg.command);
     });
   }