@soleri/core 9.2.0 → 9.3.0

package/src/planning/gap-passes.test.ts

@@ -25,13 +25,40 @@ function makePlan(overrides: Partial<Plan> = {}): Plan {
     scope: 'Auth module, middleware, and user service. Does not include OAuth providers.',
     status: 'draft',
     decisions: [
-      { decision: 'Use JWT for stateless auth', rationale: 'Because it scales horizontally without shared session store' },
+      {
+        decision: 'Use JWT for stateless auth',
+        rationale: 'Because it scales horizontally without shared session store',
+      },
     ],
     tasks: [
-      { id: 'task-1', title: 'Add JWT signing', description: 'Implement JWT sign/verify using built-in crypto module', status: 'pending', updatedAt: Date.now() },
-      { id: 'task-2', title: 'Add auth middleware', description: 'Create Express middleware that validates JWT from Authorization header', status: 'pending', updatedAt: Date.now() },
-      { id: 'task-3', title: 'Add login endpoint', description: 'POST /auth/login returns JWT after verifying credentials', status: 'pending', updatedAt: Date.now() },
-      { id: 'task-4', title: 'Add test coverage', description: 'Test JWT signing, middleware rejection, and login flow end-to-end', status: 'pending', updatedAt: Date.now() },
+      {
+        id: 'task-1',
+        title: 'Add JWT signing',
+        description: 'Implement JWT sign/verify using built-in crypto module',
+        status: 'pending',
+        updatedAt: Date.now(),
+      },
+      {
+        id: 'task-2',
+        title: 'Add auth middleware',
+        description: 'Create Express middleware that validates JWT from Authorization header',
+        status: 'pending',
+        updatedAt: Date.now(),
+      },
+      {
+        id: 'task-3',
+        title: 'Add login endpoint',
+        description: 'POST /auth/login returns JWT after verifying credentials',
+        status: 'pending',
+        updatedAt: Date.now(),
+      },
+      {
+        id: 'task-4',
+        title: 'Add test coverage',
+        description: 'Test JWT signing, middleware rejection, and login flow end-to-end',
+        status: 'pending',
+        updatedAt: Date.now(),
+      },
     ],
     checks: [],
     createdAt: Date.now(),
@@ -63,7 +90,9 @@ describe('Pattern constants (passes 5-8)', () => {
   });
 
   it('GENERIC_OBJECTIVE_PATTERNS does not match detailed objectives', () => {
-    expect(GENERIC_OBJECTIVE_PATTERNS.some((p) => p.test('Create a user auth module with JWT'))).toBe(false);
+    expect(
+      GENERIC_OBJECTIVE_PATTERNS.some((p) => p.test('Create a user auth module with JWT')),
+    ).toBe(false);
   });
 
   it('RATIONALE_INDICATORS contains reasoning words', () => {
@@ -107,7 +136,13 @@ describe('Pass 5: Clarity', () => {
   it('flags tasks with very short descriptions', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'Do thing', description: 'Short', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'Do thing',
+          description: 'Short',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
         { id: 't2', title: 'Do other', description: '', status: 'pending', updatedAt: Date.now() },
       ],
     });
@@ -123,7 +158,8 @@ describe('Pass 5: Clarity', () => {
 
   it('limits ambiguous words shown to 5', () => {
     const plan = makePlan({
-      objective: 'Maybe perhaps we might could possibly somehow probably do various several things soon with some easy simple appropriate changes etc',
+      objective:
+        'Maybe perhaps we might could possibly somehow probably do various several things soon with some easy simple appropriate changes etc',
       scope: 'Everything. Not limited.',
     });
     const gaps = analyzeClarity(plan);
@@ -158,7 +194,13 @@ describe('Pass 6: Semantic Quality', () => {
   it('flags too few tasks', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'Single task', description: 'Do everything in one task', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'Single task',
+          description: 'Do everything in one task',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeSemanticQuality(plan);
@@ -167,7 +209,11 @@ describe('Pass 6: Semantic Quality', () => {
 
   it('flags too many tasks (> 20)', () => {
     const tasks = Array.from({ length: 21 }, (_, i) => ({
-      id: `t${i}`, title: `Task ${i}`, description: `Description for task ${i} with enough detail`, status: 'pending' as const, updatedAt: Date.now(),
+      id: `t${i}`,
+      title: `Task ${i}`,
+      description: `Description for task ${i} with enough detail`,
+      status: 'pending' as const,
+      updatedAt: Date.now(),
     }));
     const plan = makePlan({ tasks });
     const gaps = analyzeSemanticQuality(plan);
@@ -184,7 +230,9 @@ describe('Pass 6: Semantic Quality', () => {
 
   it('does not flag decisions with proper rationale', () => {
     const plan = makePlan({
-      decisions: [{ decision: 'Use JWT', rationale: 'This is better because it scales horizontally' }],
+      decisions: [
+        { decision: 'Use JWT', rationale: 'This is better because it scales horizontally' },
+      ],
     });
     const gaps = analyzeSemanticQuality(plan);
     expect(gaps.some((g) => g._trigger === 'shallow_rationale')).toBe(false);
@@ -193,9 +241,27 @@ describe('Pass 6: Semantic Quality', () => {
   it('flags duplicate task titles', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'Implement feature', description: 'First implementation', status: 'pending', updatedAt: Date.now() },
-        { id: 't2', title: 'Implement feature', description: 'Duplicate title', status: 'pending', updatedAt: Date.now() },
-        { id: 't3', title: 'Test feature', description: 'Test the feature with assertions', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'Implement feature',
+          description: 'First implementation',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't2',
+          title: 'Implement feature',
+          description: 'Duplicate title',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't3',
+          title: 'Test feature',
+          description: 'Test the feature with assertions',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeSemanticQuality(plan);
@@ -212,7 +278,13 @@ describe('Pass 6: Semantic Quality', () => {
     const plan = makePlan({
       decisions: [],
       tasks: [
-        { id: 't1', title: 'Single task', description: 'Do the thing', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'Single task',
+          description: 'Do the thing',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeSemanticQuality(plan);
@@ -224,9 +296,28 @@ describe('Pass 7: Knowledge Depth', () => {
   it('awards bonus for 5+ vault pattern references', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'Apply patterns', description: 'Use zod-form-validation and react-query-caching and error-boundary-pattern', status: 'pending', updatedAt: Date.now() },
-        { id: 't2', title: 'More patterns', description: 'Use accessibility-focus-ring and semantic-token-usage and component-variant-pattern', status: 'pending', updatedAt: Date.now() },
-        { id: 't3', title: 'Testing', description: 'Test with vitest-snapshot-testing approach', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'Apply patterns',
+          description: 'Use zod-form-validation and react-query-caching and error-boundary-pattern',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't2',
+          title: 'More patterns',
+          description:
+            'Use accessibility-focus-ring and semantic-token-usage and component-variant-pattern',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't3',
+          title: 'Testing',
+          description: 'Test with vitest-snapshot-testing approach',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeKnowledgeDepth(plan);
@@ -237,9 +328,27 @@ describe('Pass 7: Knowledge Depth', () => {
   it('awards bonus for 2-4 vault pattern references', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'Apply patterns', description: 'Use zod-form-validation and react-query-caching', status: 'pending', updatedAt: Date.now() },
-        { id: 't2', title: 'Build', description: 'Build the component', status: 'pending', updatedAt: Date.now() },
-        { id: 't3', title: 'Test', description: 'Run the test suite', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'Apply patterns',
+          description: 'Use zod-form-validation and react-query-caching',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't2',
+          title: 'Build',
+          description: 'Build the component',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't3',
+          title: 'Test',
+          description: 'Run the test suite',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeKnowledgeDepth(plan);
@@ -249,11 +358,46 @@ describe('Pass 7: Knowledge Depth', () => {
   it('awards bonus for high acceptance criteria coverage', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'T1', description: 'Desc 1', status: 'pending', updatedAt: Date.now(), acceptanceCriteria: ['Criteria A'] },
-        { id: 't2', title: 'T2', description: 'Desc 2', status: 'pending', updatedAt: Date.now(), acceptanceCriteria: ['Criteria B'] },
-        { id: 't3', title: 'T3', description: 'Desc 3', status: 'pending', updatedAt: Date.now(), acceptanceCriteria: ['Criteria C'] },
-        { id: 't4', title: 'T4', description: 'Desc 4', status: 'pending', updatedAt: Date.now(), acceptanceCriteria: ['Criteria D'] },
-        { id: 't5', title: 'T5', description: 'Desc 5', status: 'pending', updatedAt: Date.now(), acceptanceCriteria: ['Criteria E'] },
+        {
+          id: 't1',
+          title: 'T1',
+          description: 'Desc 1',
+          status: 'pending',
+          updatedAt: Date.now(),
+          acceptanceCriteria: ['Criteria A'],
+        },
+        {
+          id: 't2',
+          title: 'T2',
+          description: 'Desc 2',
+          status: 'pending',
+          updatedAt: Date.now(),
+          acceptanceCriteria: ['Criteria B'],
+        },
+        {
+          id: 't3',
+          title: 'T3',
+          description: 'Desc 3',
+          status: 'pending',
+          updatedAt: Date.now(),
+          acceptanceCriteria: ['Criteria C'],
+        },
+        {
+          id: 't4',
+          title: 'T4',
+          description: 'Desc 4',
+          status: 'pending',
+          updatedAt: Date.now(),
+          acceptanceCriteria: ['Criteria D'],
+        },
+        {
+          id: 't5',
+          title: 'T5',
+          description: 'Desc 5',
+          status: 'pending',
+          updatedAt: Date.now(),
+          acceptanceCriteria: ['Criteria E'],
+        },
       ],
     });
     const gaps = analyzeKnowledgeDepth(plan);
@@ -263,7 +407,14 @@ describe('Pass 7: Knowledge Depth', () => {
   it('does not award acceptance criteria bonus below 80% threshold', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'T1', description: 'Desc', status: 'pending', updatedAt: Date.now(), acceptanceCriteria: ['A'] },
+        {
+          id: 't1',
+          title: 'T1',
+          description: 'Desc',
+          status: 'pending',
+          updatedAt: Date.now(),
+          acceptanceCriteria: ['A'],
+        },
         { id: 't2', title: 'T2', description: 'Desc', status: 'pending', updatedAt: Date.now() },
         { id: 't3', title: 'T3', description: 'Desc', status: 'pending', updatedAt: Date.now() },
       ],
@@ -273,7 +424,8 @@ describe('Pass 7: Knowledge Depth', () => {
   });
 
   it('awards bonus for rich task descriptions (avg >= 80 chars)', () => {
-    const longDesc = 'This is a very detailed task description that provides specific technical context about what needs to be implemented.';
+    const longDesc =
+      'This is a very detailed task description that provides specific technical context about what needs to be implemented.';
     const plan = makePlan({
       tasks: [
         { id: 't1', title: 'T1', description: longDesc, status: 'pending', updatedAt: Date.now() },
@@ -288,9 +440,27 @@ describe('Pass 7: Knowledge Depth', () => {
   it('awards bonus for domain knowledge indicators', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'A11y audit', description: 'Check WCAG 2.1 compliance and aria-label usage with vault patterns', status: 'pending', updatedAt: Date.now() },
-        { id: 't2', title: 'Contrast', description: '4.5:1 contrast ratio for all text, anti-pattern detection', status: 'pending', updatedAt: Date.now() },
-        { id: 't3', title: 'Touch', description: '44px touch target minimum, acceptance criteria for all buttons', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'A11y audit',
+          description: 'Check WCAG 2.1 compliance and aria-label usage with vault patterns',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't2',
+          title: 'Contrast',
+          description: '4.5:1 contrast ratio for all text, anti-pattern detection',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
+        {
+          id: 't3',
+          title: 'Touch',
+          description: '44px touch target minimum, acceptance criteria for all buttons',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeKnowledgeDepth(plan);
@@ -300,7 +470,13 @@ describe('Pass 7: Knowledge Depth', () => {
   it('returns no bonuses for basic plan', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'Do thing', description: 'Do it', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'Do thing',
+          description: 'Do it',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeKnowledgeDepth(plan);
@@ -311,7 +487,13 @@ describe('Pass 7: Knowledge Depth', () => {
   it('excludes common hyphenated words from pattern refs', () => {
     const plan = makePlan({
       tasks: [
-        { id: 't1', title: 'T1', description: 'Use front-end and back-end and real-time and client-side and server-side', status: 'pending', updatedAt: Date.now() },
+        {
+          id: 't1',
+          title: 'T1',
+          description: 'Use front-end and back-end and real-time and client-side and server-side',
+          status: 'pending',
+          updatedAt: Date.now(),
+        },
       ],
     });
     const gaps = analyzeKnowledgeDepth(plan);

package/src/planning/gap-passes.ts

@@ -6,20 +6,26 @@
 
 import type { Plan } from './planner.js';
 import type { PlanGap } from './gap-types.js';
-import {
-  gap,
-  taskText,
-  decisionText,
-  decisionsText,
-  containsAny,
-} from './gap-patterns.js';
+import { gap, taskText, decisionText, decisionsText, containsAny } from './gap-patterns.js';
 
 // ─── Pattern Constants (Passes 5-8) ─────────────────────────────
 
 export const AMBIGUOUS_WORDS = [
-  'maybe', 'perhaps', 'might', 'could', 'some', 'etc', 'soon',
-  'simple', 'easy', 'appropriate', 'various', 'several',
-  'probably', 'possibly', 'somehow',
+  'maybe',
+  'perhaps',
+  'might',
+  'could',
+  'some',
+  'etc',
+  'soon',
+  'simple',
+  'easy',
+  'appropriate',
+  'various',
+  'several',
+  'probably',
+  'possibly',
+  'somehow',
 ];
 
 export const GENERIC_OBJECTIVE_PATTERNS = [
@@ -29,8 +35,13 @@ export const GENERIC_OBJECTIVE_PATTERNS = [
 ];
 
 export const RATIONALE_INDICATORS = [
-  'because', 'since', 'due to', 'in order to',
-  'so that', 'given that', 'as a result',
+  'because',
+  'since',
+  'due to',
+  'in order to',
+  'so that',
+  'given that',
+  'as a result',
 ];
 
 export const SHALLOW_INDICATORS = ['better', 'good', 'best', 'nice', 'great', 'improved'];
@@ -64,10 +75,12 @@ export function analyzeClarity(plan: Plan): PlanGap[] {
   if (found.length > 0) {
     gaps.push(
       gap(
-        'minor', 'clarity',
+        'minor',
+        'clarity',
         `Ambiguous language detected: ${found.slice(0, 5).join(', ')}${found.length > 5 ? ` (+${found.length - 5} more)` : ''}.`,
         'Replace vague terms with concrete, specific language.',
-        undefined, `ambiguous_words:${found.join(',')}`,
+        undefined,
+        `ambiguous_words:${found.join(',')}`,
       ),
     );
   }
@@ -76,10 +89,12 @@ export function analyzeClarity(plan: Plan): PlanGap[] {
   if (shortTasks.length > 0) {
     gaps.push(
       gap(
-        'minor', 'clarity',
+        'minor',
+        'clarity',
         `${shortTasks.length} task(s) with very short descriptions: ${shortTasks.map((t) => t.id).join(', ')}.`,
         'Add detailed descriptions to all tasks explaining what needs to be done.',
-        'tasks', 'short_task_descriptions',
+        'tasks',
+        'short_task_descriptions',
       ),
     );
   }
@@ -99,10 +114,12 @@ export function analyzeSemanticQuality(plan: Plan): PlanGap[] {
     if (isGeneric || words.length < 5) {
       gaps.push(
         gap(
-          'major', 'semantic-quality',
+          'major',
+          'semantic-quality',
           `Objective is too generic${words.length < 5 ? ` (${words.length} words)` : ''}: "${plan.objective.trim()}".`,
           'Expand the objective to describe the specific outcome, context, and constraints.',
-          'objective', 'generic_objective',
+          'objective',
+          'generic_objective',
         ),
       );
     }
@@ -111,19 +128,23 @@ export function analyzeSemanticQuality(plan: Plan): PlanGap[] {
   if (plan.tasks.length > 0 && plan.tasks.length < 3) {
     gaps.push(
       gap(
-        'minor', 'semantic-quality',
+        'minor',
+        'semantic-quality',
         `Only ${plan.tasks.length} task(s) — plan may lack sufficient breakdown.`,
         'Break down the work into 3-15 well-defined tasks for better tracking.',
-        'tasks', 'too_few_tasks',
+        'tasks',
+        'too_few_tasks',
       ),
     );
   } else if (plan.tasks.length > 20) {
     gaps.push(
       gap(
-        'major', 'semantic-quality',
+        'major',
+        'semantic-quality',
         `${plan.tasks.length} tasks — plan scope may be too large.`,
         'Split into multiple plans or consolidate related tasks to stay under 20.',
-        'tasks', 'too_many_tasks',
+        'tasks',
+        'too_many_tasks',
       ),
     );
   }
@@ -135,10 +156,12 @@ export function analyzeSemanticQuality(plan: Plan): PlanGap[] {
     if (hasShallow && !hasRationale) {
       gaps.push(
         gap(
-          'minor', 'semantic-quality',
+          'minor',
+          'semantic-quality',
           `Decision ${i + 1} uses subjective language without justification.`,
           'Replace "better/good/best" with concrete reasoning using "because/since/due to".',
-          `decisions[${i}]`, 'shallow_rationale',
+          `decisions[${i}]`,
+          'shallow_rationale',
         ),
       );
     }
@@ -153,10 +176,12 @@ export function analyzeSemanticQuality(plan: Plan): PlanGap[] {
   if (duplicates.length > 0) {
     gaps.push(
       gap(
-        'minor', 'semantic-quality',
+        'minor',
+        'semantic-quality',
         `Duplicate task titles: ${[...new Set(duplicates)].join(', ')}.`,
         'Give each task a unique, descriptive title.',
-        'tasks', 'duplicate_task_titles',
+        'tasks',
+        'duplicate_task_titles',
       ),
     );
   }
@@ -164,10 +189,12 @@ export function analyzeSemanticQuality(plan: Plan): PlanGap[] {
   if (plan.tasks.length >= 3 && plan.decisions.length === 0) {
     gaps.push(
       gap(
-        'major', 'semantic-quality',
+        'major',
+        'semantic-quality',
         `${plan.tasks.length} tasks but no decisions documented.`,
         'Document key decisions and their rationale — at least 1 per 3 tasks.',
-        'decisions', 'no_decisions',
+        'decisions',
+        'no_decisions',
       ),
     );
   }
@@ -196,20 +223,35 @@ export function analyzeKnowledgeDepth(plan: Plan): PlanGap[] {
 
   if (namedPatternCount >= 5) {
     gaps.push(
-      gap('bonus', 'knowledge-depth',
+      gap(
+        'bonus',
+        'knowledge-depth',
         `${namedPatternCount} vault pattern references across tasks — strong knowledge-informed plan.`,
-        '', 'tasks', 'vault_pattern_refs_high'),
+        '',
+        'tasks',
+        'vault_pattern_refs_high',
+      ),
     );
     gaps.push(
-      gap('bonus', 'knowledge-depth',
+      gap(
+        'bonus',
+        'knowledge-depth',
         'Vault pattern density indicates expert-level domain knowledge.',
-        '', 'tasks', 'vault_pattern_density'),
+        '',
+        'tasks',
+        'vault_pattern_density',
+      ),
     );
   } else if (namedPatternCount >= 2) {
     gaps.push(
-      gap('bonus', 'knowledge-depth',
+      gap(
+        'bonus',
+        'knowledge-depth',
         `${namedPatternCount} vault pattern references across tasks.`,
-        '', 'tasks', 'vault_pattern_refs_medium'),
+        '',
+        'tasks',
+        'vault_pattern_refs_medium',
+      ),
     );
   }
 
@@ -224,9 +266,14 @@ export function analyzeKnowledgeDepth(plan: Plan): PlanGap[] {
 
   if (plan.tasks.length > 0 && tasksWithCriteria / plan.tasks.length >= 0.8) {
     gaps.push(
-      gap('bonus', 'knowledge-depth',
+      gap(
+        'bonus',
+        'knowledge-depth',
         `${tasksWithCriteria}/${plan.tasks.length} tasks have acceptance criteria (${totalCriteria} total).`,
-        '', 'tasks', 'high_acceptance_criteria'),
+        '',
+        'tasks',
+        'high_acceptance_criteria',
+      ),
     );
   }
 
@@ -237,9 +284,14 @@ export function analyzeKnowledgeDepth(plan: Plan): PlanGap[] {
 
   if (indicatorHits >= 4) {
     gaps.push(
-      gap('bonus', 'knowledge-depth',
+      gap(
+        'bonus',
+        'knowledge-depth',
         `${indicatorHits} domain-specific knowledge indicators found (WCAG, ARIA, contrast ratios, touch targets, etc.).`,
-        '', 'tasks', 'domain_knowledge_indicators'),
+        '',
+        'tasks',
+        'domain_knowledge_indicators',
+      ),
     );
   }
 
@@ -248,9 +300,14 @@ export function analyzeKnowledgeDepth(plan: Plan): PlanGap[] {
       plan.tasks.reduce((sum, t) => sum + (t.description?.length ?? 0), 0) / plan.tasks.length;
     if (avgDescLength >= 80) {
       gaps.push(
-        gap('bonus', 'knowledge-depth',
+        gap(
+          'bonus',
+          'knowledge-depth',
           `Task descriptions average ${Math.round(avgDescLength)} chars — detailed and specific.`,
-          '', 'tasks', 'rich_task_descriptions'),
+          '',
+          'tasks',
+          'rich_task_descriptions',
+        ),
       );
     }
   }
@@ -266,30 +323,42 @@ export function analyzeAlternatives(plan: Plan): PlanGap[] {
 
   if (!alts || alts.length === 0) {
     gaps.push(
-      gap('major', 'alternative-analysis',
+      gap(
+        'major',
+        'alternative-analysis',
         'No alternatives considered — risk of tunnel vision.',
         'Add at least 2 rejected alternatives with pros, cons, and rejection rationale.',
-        'alternatives', 'no_alternatives'),
+        'alternatives',
+        'no_alternatives',
+      ),
     );
     return gaps;
   }
 
   if (alts.length < 2) {
     gaps.push(
-      gap('minor', 'alternative-analysis',
+      gap(
+        'minor',
+        'alternative-analysis',
         `Only ${alts.length} alternative explored — consider at least 2.`,
         'Add another rejected alternative to strengthen decision rationale.',
-        'alternatives', 'few_alternatives'),
+        'alternatives',
+        'few_alternatives',
+      ),
     );
   }
 
   for (let i = 0; i < alts.length; i++) {
     if (!alts[i].rejected_reason || alts[i].rejected_reason.trim().length === 0) {
       gaps.push(
-        gap('minor', 'alternative-analysis',
+        gap(
+          'minor',
+          'alternative-analysis',
           `Alternative ${i + 1} ("${alts[i].approach}") missing rejection rationale.`,
           'Explain why this alternative was rejected.',
-          `alternatives[${i}]`, 'missing_rejection_rationale'),
+          `alternatives[${i}]`,
+          'missing_rejection_rationale',
+        ),
       );
     }
   }