chore(lint): clean all ESLint warnings (#57)

eslint.config.js

@@ -21,7 +21,7 @@ export default [
       },
     },
     rules: {
-      "no-unused-vars": ["warn", { argsIgnorePattern: "^_" }],
+      "no-unused-vars": ["warn", { argsIgnorePattern: "^_", caughtErrorsIgnorePattern: "^_", varsIgnorePattern: "^_" }],
     },
   },
 ];

src/observability/collectors/reporter.js

@@ -217,7 +217,6 @@ export function renderDashboardHtml(report) {
   const tasks = Object.values(report.tasks);
   const passCount      = tasks.filter((t) => t.status === 'PASS').length;
   const failCount      = tasks.filter((t) => t.status === 'FAIL').length;
-  const activeCount    = tasks.filter((t) => !t.status || t.status === 'PENDING' || t.status === 'READY' || t.status === 'IN_PROGRESS').length;
   const totalTools     = tasks.reduce((s, t) => s + t.tool_call_count, 0);
   const totalGuardrail = tasks.reduce((s, t) => s + t.guardrail_hit_count, 0);
 

tests/approval-security.test.js

@@ -8,7 +8,7 @@
 
 import test from 'node:test';
 import assert from 'node:assert/strict';
-import { existsSync, mkdtempSync, mkdirSync, readdirSync, rmSync, writeFileSync } from 'node:fs';
+import { existsSync, mkdtempSync, mkdirSync, rmSync, writeFileSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import {

tests/extension-stage-attribution.test.js

@@ -25,7 +25,7 @@ const mockPi = {
   registerCommand(name, command) { this.commands[name] = command; },
 };
 
-test('sdlc_validate attributes stage_completed to canonical stages, not task ids', async (t) => {
+test('sdlc_validate attributes stage_completed to canonical stages, not task ids', async () => {
   const projectRoot = mkdtempSync(join(tmpdir(), 'rstack-stage-attr-'));
   const previousProjectRoot = process.env.RSTACK_PROJECT_ROOT;
   const previousWebhook = process.env.RSTACK_SLACK_WEBHOOK;

tests/harness-checkpoints-signatures.test.js

@@ -117,11 +117,6 @@ test('RStack Checkpoints, Signatures, & Model Escalation E2E', async (t) => {
     const fs = await import('node:fs/promises');
     await fs.writeFile(eventsPath, mockEvents.map(e => JSON.stringify(e)).join('\n') + '\n');
 
-    // Run sdlc_delegate or standard builder prep and check escalated model
-    const registry = [
-      { id: 'agent.00-environment', name: 'agent.00-environment', kind: 'agent', path: 'agents/00-environment.md', domains: ['env'], stageAffinity: [] }
-    ];
-
     // Create mock agent file
     const agentDir = join(projectRoot, 'agents');
     mkdirSync(agentDir, { recursive: true });
@@ -130,7 +125,7 @@ test('RStack Checkpoints, Signatures, & Model Escalation E2E', async (t) => {
     // Override process env to escalate to gemini-2.5-pro on attempts >= 2
     process.env.RSTACK_ESCALATED_MODEL = 'gemini-2.5-pro-escalated';
 
-    const res = await mockPi.tools.sdlc_delegate.execute('2', {
+    await mockPi.tools.sdlc_delegate.execute('2', {
       agent: 'agent.00-environment',
       task: 'Check environment setup',
     });

tests/harness-dashboard.test.js

@@ -1,6 +1,6 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync, existsSync, readFileSync } from 'node:fs';
+import { mkdtempSync, rmSync, existsSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { updateRunMetrics, startDashboardServer } from '../src/index.js';