fix: resolve lint errors (import sort, nested ternary)

- Sort imports alphabetically in config.ts (DEFAULT_MERGER)
- Extract nested ternary in review.ts to separate variable

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Test User

packages/cli/src/commands/review.ts

@@ -166,8 +166,7 @@ export function postReadyForHumanReviewComment(
   finalScore: number | undefined,
   cwd: string,
 ): void {
-  const scoreNote =
-    finalScore !== undefined ? ` (score: ${finalScore}/100)` : '';
+  const scoreNote = finalScore !== undefined ? ` (score: ${finalScore}/100)` : '';
   const body =
     `## ✅ Ready for Human Review\n\n` +
     `Night Watch has reviewed this PR${scoreNote} and found no issues requiring automated fixes.\n\n` +
@@ -520,12 +519,15 @@ export function reviewCommand(program: Command): void {
             const reviewedPrNumbers = parseReviewedPrNumbers(scriptResult?.data.prs);
             const noChangesPrNumbers = parseReviewedPrNumbers(scriptResult?.data.no_changes_prs);
             const fallbackPrNumber = fallbackPrDetails?.number;
+            // Resolve primary PR numbers: prefer reviewed PRs, fall back to fallback PR
+            const primaryPrNumbers: number[] = [];
+            if (reviewedPrNumbers.length > 0) {
+              primaryPrNumbers.push(...reviewedPrNumbers);
+            } else if (fallbackPrNumber !== undefined) {
+              primaryPrNumbers.push(fallbackPrNumber);
+            }
             const notificationTargets = buildReviewNotificationTargets(
-              reviewedPrNumbers.length > 0
-                ? reviewedPrNumbers
-                : fallbackPrNumber !== undefined
-                  ? [fallbackPrNumber]
-                  : [],
+              primaryPrNumbers,
               noChangesPrNumbers,
               legacyNoChangesNeeded,
             );
@@ -567,7 +569,10 @@ export function reviewCommand(program: Command): void {
                   event: reviewEvent,
                   projectName: path.basename(projectDir),
                   exitCode,
-                  provider: formatProviderDisplay(envVars.NW_PROVIDER_CMD, envVars.NW_PROVIDER_LABEL),
+                  provider: formatProviderDisplay(
+                    envVars.NW_PROVIDER_CMD,
+                    envVars.NW_PROVIDER_LABEL,
+                  ),
                   prUrl: prDetails?.url,
                   prTitle: prDetails?.title,
                   prBody: prDetails?.body,

packages/core/src/config.ts

@@ -21,7 +21,6 @@ import {
   DEFAULT_ANALYTICS,
   DEFAULT_AUDIT,
   DEFAULT_AUTO_MERGE,
-  DEFAULT_MERGER,
   DEFAULT_AUTO_MERGE_METHOD,
   DEFAULT_BOARD_PROVIDER,
   DEFAULT_BRANCH_PATTERNS,
@@ -36,6 +35,7 @@ import {
   DEFAULT_MAX_LOG_SIZE,
   DEFAULT_MAX_RETRIES,
   DEFAULT_MAX_RUNTIME,
+  DEFAULT_MERGER,
   DEFAULT_MIN_REVIEW_SCORE,
   DEFAULT_NOTIFICATIONS,
   DEFAULT_PRD_DIR,
@@ -220,7 +220,9 @@ function mergeConfigs(
     merged.merger = {
       ...merged.merger,
       enabled: true,
-      mergeMethod: (merged as unknown as Record<string, unknown>).autoMergeMethod as IMergerConfig['mergeMethod'] ?? 'squash',
+      mergeMethod:
+        ((merged as unknown as Record<string, unknown>)
+          .autoMergeMethod as IMergerConfig['mergeMethod']) ?? 'squash',
     };
   }