Skip to content

Commit d289e36

Browse files
committed
style: Fix Prettier formatting issues
1 parent e00c6a6 commit d289e36

3 files changed

Lines changed: 26 additions & 34 deletions

File tree

src/orchestration/builtin-skills.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -454,7 +454,10 @@ const requirementsReviewerSkill = {
454454
}
455455

456456
default:
457-
return { success: false, error: 'Invalid action. Use: review, fagan, pbr, metrics, correct' };
457+
return {
458+
success: false,
459+
error: 'Invalid action. Use: review, fagan, pbr, metrics, correct',
460+
};
458461
}
459462
},
460463
};
@@ -499,7 +502,8 @@ const designReviewerSkill = {
499502
{
500503
name: 'focus',
501504
type: 'array',
502-
description: 'Review focus areas: solid, patterns, coupling-cohesion, error-handling, security, all',
505+
description:
506+
'Review focus areas: solid, patterns, coupling-cohesion, error-handling, security, all',
503507
required: false,
504508
},
505509
{ name: 'outputFormat', type: 'string', description: 'json, markdown', required: false },
@@ -685,7 +689,8 @@ const designReviewerSkill = {
685689
default:
686690
return {
687691
success: false,
688-
error: 'Invalid action. Use: review, solid, patterns, coupling, security, c4, adr, correct',
692+
error:
693+
'Invalid action. Use: review, solid, patterns, coupling, security, c4, adr, correct',
689694
};
690695
}
691696
},

src/validators/design-reviewer.js

Lines changed: 12 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -463,9 +463,7 @@ class DesignReviewer {
463463
}
464464

465465
// DIP (Dependency Inversion Principle) チェック
466-
if (
467-
/directly\s+depend|concrete\s+class|tight\s+coupling|/gi.test(content)
468-
) {
466+
if (/directly\s+depend|concrete\s+class|tight\s+coupling|/gi.test(content)) {
469467
issues.push(
470468
new DesignIssue({
471469
id: `DES-SOLID-${String(issueCounter++).padStart(3, '0')}`,
@@ -545,8 +543,7 @@ class DesignReviewer {
545543
severity: IssueSeverity.SUGGESTION,
546544
title: 'Consider Observer Pattern',
547545
description: 'Event handling mentioned. Observer pattern might be beneficial.',
548-
recommendation:
549-
'Consider Observer/Pub-Sub pattern for decoupled event notification.',
546+
recommendation: 'Consider Observer/Pub-Sub pattern for decoupled event notification.',
550547
})
551548
);
552549
}
@@ -570,8 +567,7 @@ class DesignReviewer {
570567
severity: IssueSeverity.MAJOR,
571568
title: 'High Coupling Detected',
572569
description: 'Design mentions tight coupling between components',
573-
recommendation:
574-
'Reduce coupling through interfaces, events, or dependency injection.',
570+
recommendation: 'Reduce coupling through interfaces, events, or dependency injection.',
575571
})
576572
);
577573
}
@@ -657,8 +653,7 @@ class DesignReviewer {
657653
severity: IssueSeverity.MINOR,
658654
title: 'Missing Retry Strategy',
659655
description: 'External service integration without retry strategy',
660-
recommendation:
661-
'Add retry with exponential backoff for external service calls.',
656+
recommendation: 'Add retry with exponential backoff for external service calls.',
662657
})
663658
);
664659
}
@@ -675,8 +670,7 @@ class DesignReviewer {
675670
severity: IssueSeverity.MINOR,
676671
title: 'Consider Circuit Breaker',
677672
description: 'Distributed system without circuit breaker pattern',
678-
recommendation:
679-
'Implement circuit breaker pattern to prevent cascade failures.',
673+
recommendation: 'Implement circuit breaker pattern to prevent cascade failures.',
680674
})
681675
);
682676
}
@@ -690,8 +684,7 @@ class DesignReviewer {
690684
severity: IssueSeverity.SUGGESTION,
691685
title: 'Consider Graceful Degradation',
692686
description: 'No graceful degradation strategy documented',
693-
recommendation:
694-
'Define fallback behaviors for when components fail.',
687+
recommendation: 'Define fallback behaviors for when components fail.',
695688
})
696689
);
697690
}
@@ -844,8 +837,7 @@ class DesignReviewer {
844837
severity: IssueSeverity.MINOR,
845838
title: 'Missing Component Diagram',
846839
description: 'C4 Component diagram not found',
847-
recommendation:
848-
'Add Component diagram for key containers showing internal structure.',
840+
recommendation: 'Add Component diagram for key containers showing internal structure.',
849841
})
850842
);
851843
}
@@ -875,8 +867,7 @@ class DesignReviewer {
875867
severity: IssueSeverity.MINOR,
876868
title: 'Missing ADR Status',
877869
description: 'ADR status not specified',
878-
recommendation:
879-
'Add status: proposed/accepted/deprecated/superseded.',
870+
recommendation: 'Add status: proposed/accepted/deprecated/superseded.',
880871
})
881872
);
882873
}
@@ -889,8 +880,7 @@ class DesignReviewer {
889880
severity: IssueSeverity.MAJOR,
890881
title: 'Missing ADR Context',
891882
description: 'ADR context/background not documented',
892-
recommendation:
893-
'Add Context section explaining the problem/situation.',
883+
recommendation: 'Add Context section explaining the problem/situation.',
894884
})
895885
);
896886
}
@@ -903,8 +893,7 @@ class DesignReviewer {
903893
severity: IssueSeverity.CRITICAL,
904894
title: 'Missing ADR Decision',
905895
description: 'ADR decision not clearly stated',
906-
recommendation:
907-
'Add Decision section clearly stating what was decided.',
896+
recommendation: 'Add Decision section clearly stating what was decided.',
908897
})
909898
);
910899
}
@@ -917,8 +906,7 @@ class DesignReviewer {
917906
severity: IssueSeverity.MAJOR,
918907
title: 'Missing ADR Consequences',
919908
description: 'ADR consequences not documented',
920-
recommendation:
921-
'Add Consequences section with both positive and negative impacts.',
909+
recommendation: 'Add Consequences section with both positive and negative impacts.',
922910
})
923911
);
924912
}
@@ -931,8 +919,7 @@ class DesignReviewer {
931919
severity: IssueSeverity.MINOR,
932920
title: 'Missing ADR Alternatives',
933921
description: 'ADR alternatives considered not documented',
934-
recommendation:
935-
'Add Alternatives section showing other options that were considered.',
922+
recommendation: 'Add Alternatives section showing other options that were considered.',
936923
})
937924
);
938925
}

src/validators/requirements-reviewer.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -631,8 +631,7 @@ class RequirementsReviewer {
631631
title: `Missing Boundary Values: ${req.id}`,
632632
description: 'Requirement mentions ranges but boundary values are not specified.',
633633
evidence: req.fullText,
634-
recommendation:
635-
'Specify exact minimum and maximum values for boundary testing.',
634+
recommendation: 'Specify exact minimum and maximum values for boundary testing.',
636635
})
637636
);
638637
}
@@ -668,8 +667,7 @@ class RequirementsReviewer {
668667
type: DefectType.MISSING,
669668
title: 'Missing Scalability Requirements',
670669
description: 'No requirements found for system scalability.',
671-
recommendation:
672-
'Define expected load, concurrent users, and scaling requirements.',
670+
recommendation: 'Define expected load, concurrent users, and scaling requirements.',
673671
})
674672
);
675673
}
@@ -684,7 +682,8 @@ class RequirementsReviewer {
684682
type: DefectType.MISSING,
685683
title: 'Missing Availability Requirements',
686684
description: 'No availability/SLA requirements defined.',
687-
recommendation: 'Specify target availability (e.g., 99.9% uptime) and recovery time objectives.',
685+
recommendation:
686+
'Specify target availability (e.g., 99.9% uptime) and recovery time objectives.',
688687
})
689688
);
690689
}
@@ -924,7 +923,8 @@ class RequirementsReviewer {
924923
* 修正レポートを生成
925924
*/
926925
generateCorrectionReport(correctionResult) {
927-
const { changesApplied, rejectedFindings, updatedQualityGate, filesModified } = correctionResult;
926+
const { changesApplied, rejectedFindings, updatedQualityGate, filesModified } =
927+
correctionResult;
928928

929929
let report = `## 📝 Correction Report\n\n`;
930930
report += `**Correction Date**: ${new Date().toISOString().split('T')[0]}\n\n`;

0 commit comments

Comments
 (0)