polish: fold final-review nits

- resolution.ts: thread leader substitution into pickLine for receiver-side,
  self-build, death, FR, PreRoundMood, and PostRoundReaction events; prevents
  literal '{leader}' leaking when bank categories are empty (Netanyahoo's
  propagandaReceive / buildFactory)
- Setup: require non-empty name + country for every human before Start enables
- SoftWarnPanel: remove unused 'orders' prop

Author: emp3thy

src/engine/resolution.ts

@@ -33,7 +33,7 @@ export function resolveRound(state: GameState): ResolveResult {
     const bank = getBank(id);
     if (!bank) continue;
     const snapBack = s.lastColumnNamedLeader === id;
-    const r = pickLine(bank, 'preRoundMood', s.rngState, { snapBack });
+    const r = pickLine(bank, 'preRoundMood', s.rngState, { snapBack, substitutions: { leader: s.leaders[id].name } });
     s.rngState = r.rngState;
     events.push({ kind: 'PreRoundMood', leaderId: id, quote: r.quote, snapBack });
   }
@@ -178,7 +178,7 @@ export function resolveRound(state: GameState): ResolveResult {
           senderQuote = p.quote;
         }
         if (receiverBank) {
-          const p = pickLine(receiverBank, 'propagandaReceive', s.rngState);
+          const p = pickLine(receiverBank, 'propagandaReceive', s.rngState, { substitutions: { leader: s.leaders[e.to].name } });
           s.rngState = p.rngState;
           receiverQuote = p.quote;
         }
@@ -196,7 +196,7 @@ export function resolveRound(state: GameState): ResolveResult {
           senderQuote = p.quote;
         }
         if (receiverBank) {
-          const p = pickLine(receiverBank, 'beingWooed', s.rngState);
+          const p = pickLine(receiverBank, 'beingWooed', s.rngState, { substitutions: { leader: s.leaders[e.to].name } });
           s.rngState = p.rngState;
           receiverQuote = p.quote;
         }
@@ -206,31 +206,31 @@ export function resolveRound(state: GameState): ResolveResult {
       case 'FactoryBuilt': {
         const bank = isHuman(e.by) ? undefined : getBank(e.by);
         if (!bank) break;
-        const p = pickLine(bank, 'buildFactory', s.rngState);
+        const p = pickLine(bank, 'buildFactory', s.rngState, { substitutions: { leader: s.leaders[e.by].name } });
         s.rngState = p.rngState;
         events[i] = { ...e, quote: p.quote };
         break;
       }
       case 'DefenceBuilt': {
         const bank = isHuman(e.by) ? undefined : getBank(e.by);
         if (!bank) break;
-        const p = pickLine(bank, 'buildDefence', s.rngState);
+        const p = pickLine(bank, 'buildDefence', s.rngState, { substitutions: { leader: s.leaders[e.by].name } });
         s.rngState = p.rngState;
         events[i] = { ...e, quote: p.quote };
         break;
       }
       case 'LeaderEliminated': {
         const bank = isHuman(e.id) ? undefined : getBank(e.id);
         if (!bank) break;
-        const p = pickLine(bank, 'death', s.rngState);
+        const p = pickLine(bank, 'death', s.rngState, { substitutions: { leader: s.leaders[e.id].name } });
         s.rngState = p.rngState;
         events[i] = { ...e, quote: p.quote };
         break;
       }
       case 'FinalRetaliationTriggered': {
         const bank = isHuman(e.by) ? undefined : getBank(e.by);
         if (!bank) break;
-        const p = pickLine(bank, 'finalRetaliation', s.rngState);
+        const p = pickLine(bank, 'finalRetaliation', s.rngState, { substitutions: { leader: s.leaders[e.by].name } });
         s.rngState = p.rngState;
         events[i] = { ...e, quote: p.quote };
         break;
@@ -262,7 +262,7 @@ export function resolveRound(state: GameState): ResolveResult {
     if (!s.leaders[id].alive) continue;
     const bank = getBank(id);
     if (!bank) continue;
-    const r = pickLine(bank, 'reaction', s.rngState);
+    const r = pickLine(bank, 'reaction', s.rngState, { substitutions: { leader: s.leaders[id].name } });
     s.rngState = r.rngState;
     events.push({ kind: 'PostRoundReaction', leaderId: id, quote: r.quote });
   }

src/ui/components/SoftWarnPanel.tsx

@@ -1,13 +1,12 @@
-import type { GameState, Order, SoftWarning } from '../../engine/types';
+import type { GameState, SoftWarning } from '../../engine/types';
 import styles from './SoftWarnPanel.module.css';
 
 interface Props {
   warnings: SoftWarning[];
-  orders: Order[];
   game: GameState;
 }
 
-export default function SoftWarnPanel({ warnings, orders: _orders, game }: Props) {
+export default function SoftWarnPanel({ warnings, game }: Props) {
   if (warnings.length === 0) return null;
   return (
     <div className={styles.panel}>

src/ui/screens/Planning.tsx

@@ -136,7 +136,7 @@ export default function Planning({ state, dispatch }: ScreenProps) {
         <div className={`${styles.apSummary} ${overBudget ? styles.over : ''}`}>
           AP used: {apUsed} / {apTotal}
         </div>
-        <SoftWarnPanel warnings={softWarnings} orders={orders} game={game} />
+        <SoftWarnPanel warnings={softWarnings} game={game} />
       </section>
 
       <OrderForm state={game} playerId={activeId} committedOrders={orders} onAdd={addOrder} />

src/ui/screens/Setup.tsx

@@ -28,7 +28,8 @@ export default function Setup({ dispatch }: ScreenProps) {
   const [difficulty, setDifficulty] = useState<Difficulty>('normal');
   const [seedInput, setSeedInput] = useState('');
 
-  const canStart = selectedAi.length >= 2 && selectedAi.length <= 4;
+  const allHumansComplete = humans.every((h) => h.name.trim() !== '' && h.country.trim() !== '');
+  const canStart = selectedAi.length >= 2 && selectedAi.length <= 4 && allHumansComplete;
 
   function addHuman() {
     if (humans.length >= 5) return;