From 838f8d6ff6d21fd4da72de8839ee5321471de6e1 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:06:41 +0000
Subject: [PATCH 01/20] fix: stop silently substituting bids below the real
 minimum
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

setAdGroupDefaultBid (and setTargetBid, which had the same pattern)
validated only that a bid was finite/non-negative, then silently
rewrote anything below Amazon's real $0.02 minimum to $0.02 — a
caller requesting $0.00 got $0.02 back with a history entry claiming
that's what they asked for, contradicting the engine's documented
fail-fast convention.

Added assertValidBid (finite, non-negative, and >= the real minimum)
for these "set an explicit bid on an existing entity" actions, and
removed the clamp. Left addTarget/normalizeCampaign's clamping alone
— those are creation/normalization paths filling in defaults for
otherwise-incomplete data, a different and still-valid use case for
clamping. Tightened the two UI call sites' pre-submit guards to match
the new floor so they no longer throw uncaught.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 .../AdConsole/details/AdGroupsTab.tsx           |  6 +++++-
 src/components/AdConsole/details/TargetsTab.tsx |  3 ++-
 .../ad-console/core/__tests__/adgroup.test.ts   | 14 +++++++++++++-
 .../ad-console/core/__tests__/engine.test.ts    |  6 ++++++
 src/engine/ad-console/core/engine/adgroup.ts    |  8 ++++----
 src/engine/ad-console/core/engine/target.ts     |  8 ++++----
 src/lib/validation.ts                           | 17 +++++++++++++++++
 7 files changed, 51 insertions(+), 11 deletions(-)

diff --git a/src/components/AdConsole/details/AdGroupsTab.tsx b/src/components/AdConsole/details/AdGroupsTab.tsx
index 4283c02..1423be9 100644
--- a/src/components/AdConsole/details/AdGroupsTab.tsx
+++ b/src/components/AdConsole/details/AdGroupsTab.tsx
@@ -10,6 +10,7 @@ import { TextInput } from '@astryxdesign/core/TextInput';
 import type { Campaign } from '@/engine/ad-console/types';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
 import { calc, formatMoney, formatWhole, formatPercent, formatBid, acosClass } from '@/engine/ad-console/core/engine';
+import { MIN_BID } from '@/lib/validation';
 import { EmptyState } from './EmptyState';
 
 interface Props {
@@ -62,7 +63,10 @@ export function AdGroupsTab({ campaign: c }: Props) {
             />
           </div>
           <Button label="Save default bid" variant="primary" style={{ marginTop: 8 }}
-            onClick={() => setAdGroupDefaultBid(c.id, focused.id, Number(adGroupBidEdits[focused.id] ?? focused.defaultBid))} />
+            onClick={() => {
+              const bid = Number(adGroupBidEdits[focused.id] ?? focused.defaultBid);
+              if (bid >= MIN_BID) setAdGroupDefaultBid(c.id, focused.id, bid);
+            }} />
         </Card>
         {!agTargets.length ? (
           <EmptyState icon="target" title={`No targets in "${focused.name}" yet`} message="Add keywords or product targets to this ad group from the Targeting tab." />
diff --git a/src/components/AdConsole/details/TargetsTab.tsx b/src/components/AdConsole/details/TargetsTab.tsx
index e7c4206..30e4c29 100644
--- a/src/components/AdConsole/details/TargetsTab.tsx
+++ b/src/components/AdConsole/details/TargetsTab.tsx
@@ -7,6 +7,7 @@ import { Card } from '@astryxdesign/core/Card';
 import type { Campaign } from '@/engine/ad-console/types';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
 import { calc, formatMoney, formatWhole, formatPercent, formatBid, formatRoas, acosClass } from '@/engine/ad-console/core/engine';
+import { MIN_BID } from '@/lib/validation';
 import { EmptyState } from './EmptyState';
 
 interface Props {
@@ -109,7 +110,7 @@ export function TargetsTab({ campaign: c }: Props) {
                     <Button label="+10%" size="sm" onClick={() => adjustTargetBid(c.id, t.id, 1.1)} />{' '}
                     <Button label="Set" size="sm" onClick={() => {
                       const bid = Number(bidEdits[t.id]);
-                      if (bid && bid > 0) setTargetBid(c.id, t.id, bid);
+                      if (bid >= MIN_BID) setTargetBid(c.id, t.id, bid);
                     }} />{' '}
                     <Button
                       label={t.status === 'Paused' ? 'Enable' : 'Pause'}
diff --git a/src/engine/ad-console/core/__tests__/adgroup.test.ts b/src/engine/ad-console/core/__tests__/adgroup.test.ts
index 9ba39e0..154f2fa 100644
--- a/src/engine/ad-console/core/__tests__/adgroup.test.ts
+++ b/src/engine/ad-console/core/__tests__/adgroup.test.ts
@@ -97,7 +97,7 @@ describe('setAdGroupStatus', () => {
 });
 
 describe('setAdGroupDefaultBid', () => {
-  it('sets the default bid (clamped to minimum)', () => {
+  it('sets the default bid', () => {
     const next = setAdGroupDefaultBid(makeCampaign(), 'AG1', 1.25);
     expect(next.adGroups[0]!.defaultBid).toBe(1.25);
   });
@@ -105,6 +105,18 @@ describe('setAdGroupDefaultBid', () => {
   it('fails fast on unknown ad group id', () => {
     expect(() => setAdGroupDefaultBid(makeCampaign(), 'NOPE', 1)).toThrow();
   });
+
+  it('fails fast on a bid below the real minimum instead of silently substituting it', () => {
+    // Previously: 0 and 0.01 passed the non-negative check and were then
+    // silently rewritten to $0.02, discarding what the caller asked for.
+    expect(() => setAdGroupDefaultBid(makeCampaign(), 'AG1', 0)).toThrow();
+    expect(() => setAdGroupDefaultBid(makeCampaign(), 'AG1', 0.01)).toThrow();
+  });
+
+  it('fails fast on a negative or NaN default bid', () => {
+    expect(() => setAdGroupDefaultBid(makeCampaign(), 'AG1', -1)).toThrow();
+    expect(() => setAdGroupDefaultBid(makeCampaign(), 'AG1', NaN)).toThrow();
+  });
 });
 
 describe('removeAdGroup', () => {
diff --git a/src/engine/ad-console/core/__tests__/engine.test.ts b/src/engine/ad-console/core/__tests__/engine.test.ts
index 3e9bb92..6203f0a 100644
--- a/src/engine/ad-console/core/__tests__/engine.test.ts
+++ b/src/engine/ad-console/core/__tests__/engine.test.ts
@@ -205,6 +205,12 @@ describe('target operations', () => {
     expect(() => setTargetBid(c, 'T1', -5)).toThrow();
   });
 
+  it('fails fast on a bid below the real minimum instead of silently substituting it', () => {
+    const c = makeCampaign({ targets: [{ id: 'T1', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'kw', match: 'Exact', bid: 0.75, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 }] });
+    expect(() => setTargetBid(c, 'T1', 0)).toThrow();
+    expect(() => setTargetBid(c, 'T1', 0.01)).toThrow();
+  });
+
   it('pauses then re-enables a target', () => {
     const c = makeCampaign({ targets: [{ id: 'T1', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'kw', match: 'Exact', bid: 0.75, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 }] });
     expect(pauseTarget(c, 'T1').targets[0]!.status).toBe('Paused');
diff --git a/src/engine/ad-console/core/engine/adgroup.ts b/src/engine/ad-console/core/engine/adgroup.ts
index c070462..a971bc1 100644
--- a/src/engine/ad-console/core/engine/adgroup.ts
+++ b/src/engine/ad-console/core/engine/adgroup.ts
@@ -2,7 +2,7 @@
  * Ad group CRUD operations.
  */
 import type { Campaign, CampaignStatus, AdGroup, ProductAd, Ad } from '../types';
-import { assertNonEmpty, assertFiniteNonNegative, ValidationError } from '../../../../lib/validation';
+import { assertNonEmpty, assertValidBid, ValidationError } from '../../../../lib/validation';
 import { generateId } from './id';
 
 export function addAdGroup(c: Campaign, name: string, defaultBid?: number): Campaign {
@@ -82,15 +82,15 @@ export function setAdGroupStatus(c: Campaign, adGroupId: string, status: Campaig
 }
 
 export function setAdGroupDefaultBid(c: Campaign, adGroupId: string, defaultBid: number): Campaign {
-  assertFiniteNonNegative('default bid', defaultBid);
+  assertValidBid('default bid', defaultBid);
   const ag = c.adGroups.find((a) => a.id === adGroupId);
   if (!ag) throw new ValidationError(`Unknown ad group: ${adGroupId}`);
   return {
     ...c,
     adGroups: c.adGroups.map((a) =>
-      a.id === adGroupId ? { ...a, defaultBid: Math.max(0.02, defaultBid) } : a,
+      a.id === adGroupId ? { ...a, defaultBid } : a,
     ),
-    history: [...c.history, `Ad group "${ag.name}" default bid -> $${Math.max(0.02, defaultBid).toFixed(2)}`],
+    history: [...c.history, `Ad group "${ag.name}" default bid -> $${defaultBid.toFixed(2)}`],
   };
 }
 
diff --git a/src/engine/ad-console/core/engine/target.ts b/src/engine/ad-console/core/engine/target.ts
index e5b6d6d..5183e88 100644
--- a/src/engine/ad-console/core/engine/target.ts
+++ b/src/engine/ad-console/core/engine/target.ts
@@ -5,7 +5,7 @@
 import type {
   Campaign, CampaignStatus, MatchType, Target, TargetType
 } from '../types';
-import { assertNonEmpty, assertFiniteNonNegative, ValidationError } from '../../../../lib/validation';
+import { assertNonEmpty, assertFiniteNonNegative, assertValidBid, ValidationError } from '../../../../lib/validation';
 import { generateId } from './id';
 
 export interface AddTargetOptions {
@@ -132,12 +132,12 @@ export function removeTarget(c: Campaign, targetId: string): Campaign {
 }
 
 export function setTargetBid(c: Campaign, targetId: string, newBid: number): Campaign {
-  assertFiniteNonNegative('bid', newBid);
+  assertValidBid('bid', newBid);
   return {
     ...c,
     targets: c.targets.map((t) =>
       t.id === targetId
-        ? { ...t, bid: Math.max(0.02, newBid) }
+        ? { ...t, bid: newBid }
         : t,
     ),
     history: [
@@ -145,7 +145,7 @@ export function setTargetBid(c: Campaign, targetId: string, newBid: number): Cam
       (() => {
         const t = c.targets.find((x) => x.id === targetId);
         return t
-          ? `Bid for "${t.value}" (${t.type}) changed from $${t.bid.toFixed(2)} to $${Math.max(0.02, newBid).toFixed(2)}`
+          ? `Bid for "${t.value}" (${t.type}) changed from $${t.bid.toFixed(2)} to $${newBid.toFixed(2)}`
           : `Bid updated for target ${targetId}`;
       })(),
     ],
diff --git a/src/lib/validation.ts b/src/lib/validation.ts
index 54eaa17..6220f5d 100644
--- a/src/lib/validation.ts
+++ b/src/lib/validation.ts
@@ -18,6 +18,23 @@ export function assertFiniteNonNegative(name: string, value: number): void {
   }
 }
 
+/** Amazon Ads' real minimum bid — used by functions that set an explicit bid on existing entities. */
+export const MIN_BID = 0.02;
+
+/**
+ * Like assertFiniteNonNegative, but also enforces the platform's real bid
+ * floor. Use this for "set the bid to X" actions on an existing entity,
+ * where silently substituting a different value than what was explicitly
+ * requested would be misleading. Creation/normalization paths that fill in
+ * defaults for otherwise-incomplete data should keep clamping instead.
+ */
+export function assertValidBid(name: string, value: number): void {
+  assertFiniteNonNegative(name, value);
+  if (value < MIN_BID) {
+    throw new ValidationError(`${name} must be at least $${MIN_BID.toFixed(2)}, got ${value}`);
+  }
+}
+
 export function assertNonEmpty(name: string, value: string): void {
   if (typeof value !== 'string' || value.trim().length === 0) {
     throw new ValidationError(`${name} must be a non-empty string`);

From 67194ced89a911a8f9c7740e06378c6fa7280e61 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:07:05 +0000
Subject: [PATCH 02/20] =?UTF-8?q?fix:=20UI=20consistency=20pass=20?=
 =?UTF-8?q?=E2=80=94=20un-Card-wrap=20dense=20tables,=20dedupe=20metrics/f?=
 =?UTF-8?q?ormatters?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Several visual/consistency issues flagged by review:

- Dashboard, OverviewTab, and PortfolioOverview wrapped their dense
  data tables in Card, violating the "Table renders edge-to-edge,
  never Card-wrapped" convention followed everywhere else (Manager*Tab,
  BudgetRulesTab). Split each into a Card for the header/summary
  widget portion and a bare Table sibling below it.
- PortfolioOverview's campaign-name link used ad-hoc inline styles
  instead of the shared .row-link class, so it had no hover affordance
  unlike the identical link on Dashboard and Campaign Manager.
- ManagerCampaignsTab always showed "No campaigns yet — Create your
  first campaign" even when the real cause was an active filter/search
  matching nothing, pointing the user at campaign creation instead of
  clearing filters. Added a distinct "no matches" empty state with a
  Clear filters action, using a new hasAnyCampaigns prop to tell the
  two cases apart.
- Deduplicated three hand-rolled metrics-summing reduce blocks
  (PortfolioOverview x2, CampaignManager) to the canonical
  totalMetrics() engine helper, and Dashboard's local
  fmtMoney/fmtWhole/fmtPercent (which had drifted to 2-decimal
  percentages) to the canonical formatMoney/formatWhole/formatPercent.
- Extracted the "which adFormat means video for this campaign type"
  check (previously an inline compound boolean in OverviewTab) into a
  shared isVideoFormat(type, adFormat) engine helper.

Updated the Astryx Card-contract test that had pinned the old
(incorrect) 3-Card count for OverviewTab down to 2, since "Top
targets" is intentionally no longer Card-wrapped.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 src/components/AdConsole/CampaignManager.tsx  | 22 ++---
 src/components/AdConsole/Dashboard.tsx        | 26 ++---
 .../AdConsole/PortfolioOverview.tsx           | 96 +++++++------------
 .../__tests__/ManagerCampaignsTab.test.tsx    | 26 ++++-
 .../__tests__/contracts/cards-astryx.test.tsx |  6 +-
 .../AdConsole/details/ManagerCampaignsTab.tsx | 10 +-
 .../AdConsole/details/OverviewTab.tsx         | 10 +-
 src/engine/ad-console/core/engine/campaign.ts | 10 ++
 src/engine/ad-console/core/engine/index.ts    |  2 +-
 9 files changed, 106 insertions(+), 102 deletions(-)

diff --git a/src/components/AdConsole/CampaignManager.tsx b/src/components/AdConsole/CampaignManager.tsx
index 9666c2f..5dd8392 100644
--- a/src/components/AdConsole/CampaignManager.tsx
+++ b/src/components/AdConsole/CampaignManager.tsx
@@ -5,7 +5,7 @@ import { Button } from '@astryxdesign/core/Button';
 import { Card } from '@astryxdesign/core/Card';
 import { useCampaignManager } from './hooks/useCampaignManager';
 import { MetricCard } from './metrics/MetricCard';
-import { calc, formatMoney, formatWhole, formatPercent, formatBid, formatRoas, acosClass } from '@/engine/ad-console/core/engine';
+import { calc, totalMetrics, formatMoney, formatWhole, formatPercent, formatBid, formatRoas, acosClass } from '@/engine/ad-console/core/engine';
 import type { FilterState } from '@/engine/ad-console/types';
 import { ManagerCampaignsTab } from './details/ManagerCampaignsTab';
 import { ManagerAdGroupsTab } from './details/ManagerAdGroupsTab';
@@ -15,11 +15,13 @@ import { ManagerNegativesTab } from './details/ManagerNegativesTab';
 
 export function CampaignManager() {
   const {
-    filteredCampaigns, filter, selectedTab, portfolioOptions,
+    campaigns, filteredCampaigns, filter, selectedTab, portfolioOptions,
     setFilter, selectCampaign, setTab, toggleCampaignStatus,
     duplicateCampaign, archiveCampaign, runSimulation, setView,
   } = useCampaignManager();
 
+  const clearFilters = () => setFilter({ type: 'All', status: 'All', portfolio: 'All', search: '' });
+
   const [simulating, setSimulating] = useState(false);
 
   const handleSimulate = async () => {
@@ -84,23 +86,13 @@ export function CampaignManager() {
         <select id="cm-filter-portfolio" className="select" value={filter.portfolio} onChange={(e) => setFilter({ portfolio: e.target.value })}>
           {portfolioOptions.map((x) => <option key={x}>{x}</option>)}
         </select>
-        <Button label="Clear filters" onClick={() => setFilter({ type: 'All', status: 'All', portfolio: 'All', search: '' })} />
+        <Button label="Clear filters" onClick={clearFilters} />
         <Button label="Run 7-day sim" variant="info" onClick={handleSimulate} />
       </div>
 
       <div className="grid-4 cm-metrics-grid">
         {(() => {
-          const m = filteredCampaigns.reduce(
-            (acc, c) => {
-              acc.impressions += c.metrics.impressions;
-              acc.clicks += c.metrics.clicks;
-              acc.spend += c.metrics.spend;
-              acc.sales += c.metrics.sales;
-              acc.orders += c.metrics.orders;
-              return acc;
-            },
-            { impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
-          );
+          const m = totalMetrics(filteredCampaigns);
           const x = calc(m);
           const acosTone = x.acos <= 0 ? '' : x.acos <= 30 ? 'good' : 'bad';
           return (
@@ -129,11 +121,13 @@ export function CampaignManager() {
       {selectedTab === 'campaigns' && (
         <ManagerCampaignsTab
           campaigns={filteredCampaigns}
+          hasAnyCampaigns={campaigns.length > 0}
           onSelect={selectCampaign}
           onToggleStatus={toggleCampaignStatus}
           onDuplicate={duplicateCampaign}
           onArchive={archiveCampaign}
           onCreate={() => setView('create')}
+          onClearFilters={clearFilters}
         />
       )}
       {selectedTab === 'adgroups' && <ManagerAdGroupsTab campaigns={filteredCampaigns} onSelectCampaign={selectCampaign} />}
diff --git a/src/components/AdConsole/Dashboard.tsx b/src/components/AdConsole/Dashboard.tsx
index 75c0bd7..ce95850 100644
--- a/src/components/AdConsole/Dashboard.tsx
+++ b/src/components/AdConsole/Dashboard.tsx
@@ -65,13 +65,13 @@ export function Dashboard() {
       </div>
 
       <div className="split">
-        <Card variant="default" padding={6}>
+        <div>
           <div className="section-head">
             <h2>Campaigns</h2>
             <span className="meta">{enabledCount} enabled · {state.campaigns.length} total</span>
           </div>
           {renderCampaignTable(state.campaigns.slice(0, 8), selectCampaign, calc, setView)}
-        </Card>
+        </div>
         <div>
           <Card variant="default" padding={6} style={{ marginBottom: 'var(--space-4)' }}>
             <div className="section-head">
@@ -114,16 +114,6 @@ export function Dashboard() {
   );
 }
 
-function fmtMoney(n: number) {
-  return '$' + n.toLocaleString(undefined, { minimumFractionDigits: 2, maximumFractionDigits: 2 });
-}
-function fmtWhole(n: number) {
-  return n.toLocaleString();
-}
-function fmtPercent(n: number) {
-  return n.toFixed(2) + '%';
-}
-
 type Tone = '' | 'good' | 'bad';
 
 function kpiDelta(
@@ -219,15 +209,15 @@ function renderCampaignTable(
                     {c.status}
                   </span>
                 </td>
-                <td className="money">{fmtMoney(c.dailyBudget)}</td>
+                <td className="money">{formatMoney(c.dailyBudget)}</td>
                 <td>
                   <span className="muted">{c.targetingMode}</span>
                 </td>
-                <td className="mono">{fmtWhole(c.metrics.impressions)}</td>
-                <td className="mono">{fmtWhole(c.metrics.clicks)}</td>
-                <td className="money">{fmtMoney(c.metrics.spend)}</td>
-                <td className="money">{fmtMoney(c.metrics.sales)}</td>
-                <td className={`mono ${acosClass(x.acos)}`}>{fmtPercent(x.acos)}</td>
+                <td className="mono">{formatWhole(c.metrics.impressions)}</td>
+                <td className="mono">{formatWhole(c.metrics.clicks)}</td>
+                <td className="money">{formatMoney(c.metrics.spend)}</td>
+                <td className="money">{formatMoney(c.metrics.sales)}</td>
+                <td className={`mono ${acosClass(x.acos)}`}>{formatPercent(x.acos)}</td>
               </tr>
             );
           })}
diff --git a/src/components/AdConsole/PortfolioOverview.tsx b/src/components/AdConsole/PortfolioOverview.tsx
index 402140f..6ab86af 100644
--- a/src/components/AdConsole/PortfolioOverview.tsx
+++ b/src/components/AdConsole/PortfolioOverview.tsx
@@ -5,7 +5,7 @@ import { Button } from '@astryxdesign/core/Button';
 import { Table } from '@astryxdesign/core/Table';
 import { Card } from '@astryxdesign/core/Card';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
-import { calc, formatMoney, formatWhole, formatPercent, formatRoas, acosClass } from '@/engine/ad-console/core/engine';
+import { calc, totalMetrics as sumMetrics, formatMoney, formatWhole, formatPercent, formatRoas, acosClass } from '@/engine/ad-console/core/engine';
 
 export function PortfolioOverview() {
   const state = useAdConsoleStore((s) => s.state);
@@ -30,35 +30,11 @@ export function PortfolioOverview() {
     return Array.from(map.entries()).map(([name, camps]) => ({
       name,
       campaigns: camps,
-      metrics: camps.reduce(
-        (acc, c) => {
-          acc.impressions += c.metrics.impressions;
-          acc.clicks += c.metrics.clicks;
-          acc.spend += c.metrics.spend;
-          acc.sales += c.metrics.sales;
-          acc.orders += c.metrics.orders;
-          return acc;
-        },
-        { impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
-      ),
+      metrics: sumMetrics(camps),
     }));
   }, [state.campaigns]);
 
-  const totalMetrics = useMemo(
-    () =>
-      state.campaigns.reduce(
-        (acc, c) => {
-          acc.impressions += c.metrics.impressions;
-          acc.clicks += c.metrics.clicks;
-          acc.spend += c.metrics.spend;
-          acc.sales += c.metrics.sales;
-          acc.orders += c.metrics.orders;
-          return acc;
-        },
-        { impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
-      ),
-    [state.campaigns],
-  );
+  const totalMetrics = useMemo(() => sumMetrics(state.campaigns), [state.campaigns]);
 
   const totalDerived = calc(totalMetrics);
 
@@ -119,36 +95,38 @@ export function PortfolioOverview() {
         portfolios.map((pf) => {
           const x = calc(pf.metrics);
           return (
-            <Card key={pf.name} variant="default" padding={6} style={{ marginBottom: 'var(--space-4)' }}>
-              <div className="card-title">
-                {manageMode ? (
-                  <div style={{ display: 'flex', gap: 8, alignItems: 'center', flex: 1 }}>
-                    <label htmlFor={`po-rename-${pf.name}`} className="visually-hidden">Portfolio name</label>
-                    <input id={`po-rename-${pf.name}`} className="input" style={{ fontWeight: 600, flex: 1 }}
-                      value={renameMap[pf.name] ?? pf.name}
-                      onChange={(e) => setRenameMap((m) => ({ ...m, [pf.name]: e.target.value }))}
-                      onBlur={(e) => {
-                        const v = e.target.value.trim();
-                        if (v && v !== pf.name) renamePortfolio(pf.name, v);
+            <div key={pf.name} style={{ marginBottom: 'var(--space-4)' }}>
+              <Card variant="default" padding={6}>
+                <div className="card-title">
+                  {manageMode ? (
+                    <div style={{ display: 'flex', gap: 8, alignItems: 'center', flex: 1 }}>
+                      <label htmlFor={`po-rename-${pf.name}`} className="visually-hidden">Portfolio name</label>
+                      <input id={`po-rename-${pf.name}`} className="input" style={{ fontWeight: 600, flex: 1 }}
+                        value={renameMap[pf.name] ?? pf.name}
+                        onChange={(e) => setRenameMap((m) => ({ ...m, [pf.name]: e.target.value }))}
+                        onBlur={(e) => {
+                          const v = e.target.value.trim();
+                          if (v && v !== pf.name) renamePortfolio(pf.name, v);
+                        }} />
+                      <span className="muted">{pf.campaigns.length} campaign{pf.campaigns.length !== 1 ? 's' : ''}</span>
+                      <Button label="Delete" variant="destructive" size="sm" onClick={() => {
+                        if (confirm(`Remove portfolio "${pf.name}"? Campaigns will be unassigned.`)) deletePortfolio(pf.name);
                       }} />
-                    <span className="muted">{pf.campaigns.length} campaign{pf.campaigns.length !== 1 ? 's' : ''}</span>
-                    <Button label="Delete" variant="destructive" size="sm" onClick={() => {
-                      if (confirm(`Remove portfolio "${pf.name}"? Campaigns will be unassigned.`)) deletePortfolio(pf.name);
-                    }} />
-                  </div>
-                ) : (
-                  <>
-                    <h2>{pf.name}</h2>
-                    <span>{pf.campaigns.length} campaign{pf.campaigns.length !== 1 ? 's' : ''}</span>
-                  </>
-                )}
-              </div>
-              <div className="grid-4" style={{ marginBottom: 12 }}>
-                <div><span className="muted">Spend</span><div style={{ fontWeight: 600 }}>{formatMoney(pf.metrics.spend)}</div></div>
-                <div><span className="muted">Sales</span><div style={{ fontWeight: 600 }}>{formatMoney(pf.metrics.sales)}</div></div>
-                <div><span className={`mono ${acosClass(x.acos)}`} style={{ fontWeight: 600 }}>{formatPercent(x.acos)}</span><div className="muted">ACOS</div></div>
-                <div><span style={{ fontWeight: 600 }}>{formatRoas(x.roas)}</span><div className="muted">ROAS</div></div>
-              </div>
+                    </div>
+                  ) : (
+                    <>
+                      <h2>{pf.name}</h2>
+                      <span>{pf.campaigns.length} campaign{pf.campaigns.length !== 1 ? 's' : ''}</span>
+                    </>
+                  )}
+                </div>
+                <div className="grid-4">
+                  <div><span className="muted">Spend</span><div style={{ fontWeight: 600 }}>{formatMoney(pf.metrics.spend)}</div></div>
+                  <div><span className="muted">Sales</span><div style={{ fontWeight: 600 }}>{formatMoney(pf.metrics.sales)}</div></div>
+                  <div><span className={`mono ${acosClass(x.acos)}`} style={{ fontWeight: 600 }}>{formatPercent(x.acos)}</span><div className="muted">ACOS</div></div>
+                  <div><span style={{ fontWeight: 600 }}>{formatRoas(x.roas)}</span><div className="muted">ROAS</div></div>
+                </div>
+              </Card>
               <Table>
                   <thead>
                     <tr>
@@ -162,8 +140,8 @@ export function PortfolioOverview() {
                       return (
                         <tr key={c.id}>
                           <td>
-                            <button onClick={() => selectCampaign(c.id)}
-                              style={{ border: 'none', background: 'none', color: 'var(--blue)', cursor: 'pointer', fontWeight: 500 }}>
+                            <button className="row-link" onClick={() => selectCampaign(c.id)}
+                              style={{ border: 'none', background: 'none', color: 'var(--blue)', cursor: 'pointer', fontWeight: 500, textAlign: 'left' }}>
                               {c.name}
                             </button>
                           </td>
@@ -191,7 +169,7 @@ export function PortfolioOverview() {
                     })}
                   </tbody>
                 </Table>
-            </Card>
+            </div>
           );
         })
       )}
diff --git a/src/components/AdConsole/__tests__/ManagerCampaignsTab.test.tsx b/src/components/AdConsole/__tests__/ManagerCampaignsTab.test.tsx
index 49500d3..525bc7d 100644
--- a/src/components/AdConsole/__tests__/ManagerCampaignsTab.test.tsx
+++ b/src/components/AdConsole/__tests__/ManagerCampaignsTab.test.tsx
@@ -79,6 +79,8 @@ describe('ManagerCampaignsTab — column alignment (H-02)', () => {
         onDuplicate={noop}
         onArchive={noop}
         onCreate={noop}
+        onClearFilters={noop}
+        hasAnyCampaigns={true}
       />,
     );
 
@@ -106,7 +108,7 @@ describe('ManagerCampaignsTab — column alignment (H-02)', () => {
     expect(cellText(13)).toBe('8.00x'); // ROAS
   });
 
-  it('shows the empty state when there are no campaigns', () => {
+  it('shows the "create your first campaign" empty state when there are no campaigns at all', () => {
     render(
       <ManagerCampaignsTab
         campaigns={[]}
@@ -115,11 +117,31 @@ describe('ManagerCampaignsTab — column alignment (H-02)', () => {
         onDuplicate={noop}
         onArchive={noop}
         onCreate={noop}
+        onClearFilters={noop}
+        hasAnyCampaigns={false}
       />,
     );
     expect(screen.getByText('No campaigns yet')).toBeDefined();
   });
 
+  it('shows a "no matches" empty state (not "create your first campaign") when filters yield zero results', () => {
+    render(
+      <ManagerCampaignsTab
+        campaigns={[]}
+        onSelect={noop}
+        onToggleStatus={noop}
+        onDuplicate={noop}
+        onArchive={noop}
+        onCreate={noop}
+        onClearFilters={noop}
+        hasAnyCampaigns={true}
+      />,
+    );
+    expect(screen.getByText('No campaigns match your filters')).toBeDefined();
+    expect(screen.queryByText('No campaigns yet')).toBeNull();
+    expect(screen.getByText('Clear filters')).toBeDefined();
+  });
+
   it('renders one row per campaign', () => {
     const a = { ...FIXTURE_CAMPAIGN, id: 'a', name: 'Alpha' };
     const b = { ...FIXTURE_CAMPAIGN, id: 'b', name: 'Bravo' };
@@ -131,6 +153,8 @@ describe('ManagerCampaignsTab — column alignment (H-02)', () => {
         onDuplicate={noop}
         onArchive={noop}
         onCreate={noop}
+        onClearFilters={noop}
+        hasAnyCampaigns={true}
       />,
     );
     const rows = container.querySelectorAll('tbody tr');
diff --git a/src/components/AdConsole/__tests__/contracts/cards-astryx.test.tsx b/src/components/AdConsole/__tests__/contracts/cards-astryx.test.tsx
index 5dcbad0..d7e2f7e 100644
--- a/src/components/AdConsole/__tests__/contracts/cards-astryx.test.tsx
+++ b/src/components/AdConsole/__tests__/contracts/cards-astryx.test.tsx
@@ -44,8 +44,10 @@ describe('Astryx Card contract — presence and variant wiring', () => {
     const c = useAdConsoleStore.getState().state.campaigns[0]!;
     render(<OverviewTab campaign={c} />);
     const astryxCards = document.querySelectorAll('.astryx-card');
-    // OverviewTab has 3 .card.pad sections (Settings, Products, Top targets)
-    expect(astryxCards.length).toBeGreaterThanOrEqual(3);
+    // OverviewTab has 2 Card sections (Settings, Products). "Top targets"
+    // is a dense table and is deliberately NOT Card-wrapped, per the
+    // "Table edge-to-edge, never Card-wrapped" convention.
+    expect(astryxCards.length).toBeGreaterThanOrEqual(2);
   });
 
   it('Astrox cards have data-variant attribute', () => {
diff --git a/src/components/AdConsole/details/ManagerCampaignsTab.tsx b/src/components/AdConsole/details/ManagerCampaignsTab.tsx
index ef94e27..9925b2d 100644
--- a/src/components/AdConsole/details/ManagerCampaignsTab.tsx
+++ b/src/components/AdConsole/details/ManagerCampaignsTab.tsx
@@ -8,16 +8,22 @@ import { EmptyState } from './EmptyState';
 
 interface Props {
   campaigns: Campaign[];
+  hasAnyCampaigns: boolean;
   onSelect: (id: string) => void;
   onToggleStatus: (id: string) => void;
   onDuplicate: (id: string) => void;
   onArchive: (id: string) => void;
   onCreate: () => void;
+  onClearFilters: () => void;
 }
 
-export function ManagerCampaignsTab({ campaigns, onSelect, onToggleStatus, onDuplicate, onArchive, onCreate }: Props) {
+export function ManagerCampaignsTab({ campaigns, hasAnyCampaigns, onSelect, onToggleStatus, onDuplicate, onArchive, onCreate, onClearFilters }: Props) {
   if (!campaigns.length) {
-    return (
+    return hasAnyCampaigns ? (
+      <EmptyState icon="search" title="No campaigns match your filters" message="Try a different search term or adjust the type/status/portfolio filters.">
+        <Button label="Clear filters" onClick={onClearFilters} />
+      </EmptyState>
+    ) : (
       <EmptyState icon="chart" title="No campaigns yet" message="Create your first campaign to start training.">
         <Button label="Create campaign" variant="primary" onClick={onCreate} />
       </EmptyState>
diff --git a/src/components/AdConsole/details/OverviewTab.tsx b/src/components/AdConsole/details/OverviewTab.tsx
index 3ee598f..008d01e 100644
--- a/src/components/AdConsole/details/OverviewTab.tsx
+++ b/src/components/AdConsole/details/OverviewTab.tsx
@@ -8,7 +8,7 @@ import { NumberInput } from '@astryxdesign/core/NumberInput';
 import { Selector } from '@astryxdesign/core/Selector';
 import type { Campaign } from '@/engine/ad-console/types';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
-import { calc, formatMoney, formatWhole, formatPercent, formatBid, formatRoas, acosClass } from '@/engine/ad-console/core/engine';
+import { calc, formatMoney, formatWhole, formatPercent, formatBid, formatRoas, acosClass, isVideoFormat } from '@/engine/ad-console/core/engine';
 import { PRODUCTS } from '@/engine/ad-console/core/scenarios';
 
 interface Props {
@@ -91,8 +91,8 @@ export function OverviewTab({ campaign: c }: Props) {
                   } as any)} />
               </div>
             )}
-            {((c.type === 'SB' || c.type === 'SD') && ((c.type === 'SB' && c.adFormat === 'Video') || (c.type === 'SD' && c.adFormat === 'Video creative'))) && c.creative?.video && (
-              <div className="tag" style={{ background: 'var(--surface-2)', padding: '6px 10px', borderRadius: 6, display: 'inline-flex', alignItems: 'center', gap: 6, marginTop: 4 }}>
+            {isVideoFormat(c.type, c.adFormat) && c.creative?.video && (
+              <div className="tag" style={{ marginTop: 'var(--space-1)' }}>
                 <span style={{ fontSize: 18 }}>&#9654;</span>
                 <span><strong>Video:</strong> {c.creative.video}</span>
               </div>
@@ -102,7 +102,7 @@ export function OverviewTab({ campaign: c }: Props) {
           </div>
         )}
       </Card>
-      <Card variant="default" padding={6} style={{ gridColumn: '1 / -1' }}>
+      <div style={{ gridColumn: '1 / -1' }}>
         <div className="section-head"><h2>Top targets by profit signal</h2><span className="meta">Use to train bid optimization</span></div>
         <Table>
             <thead><tr><th>Target</th><th>Bid</th><th>Impr.</th><th>Clicks</th><th>CPC</th><th>Spend</th><th>Sales</th><th>Orders</th><th>ACOS</th><th>ROAS</th></tr></thead>
@@ -126,7 +126,7 @@ export function OverviewTab({ campaign: c }: Props) {
               })}
             </tbody>
           </Table>
-      </Card>
+      </div>
     </div>
   );
 }
diff --git a/src/engine/ad-console/core/engine/campaign.ts b/src/engine/ad-console/core/engine/campaign.ts
index 5cd7bae..301b264 100644
--- a/src/engine/ad-console/core/engine/campaign.ts
+++ b/src/engine/ad-console/core/engine/campaign.ts
@@ -306,4 +306,14 @@ export function savePlacements(
     placements,
     history: [...c.history, changes.length ? `Placements updated: ${changes.join(', ')}` : 'Placements saved (no changes)'],
   };
+}
+
+/**
+ * Single source of truth for which adFormat value means "video" for a
+ * given campaign type — SB and SD each use a different string for it.
+ */
+export function isVideoFormat(type: CampaignType, adFormat: string | undefined): boolean {
+  if (type === 'SB') return adFormat === 'Video';
+  if (type === 'SD') return adFormat === 'Video creative';
+  return false;
 }
\ No newline at end of file
diff --git a/src/engine/ad-console/core/engine/index.ts b/src/engine/ad-console/core/engine/index.ts
index 8ba9c66..9499a08 100644
--- a/src/engine/ad-console/core/engine/index.ts
+++ b/src/engine/ad-console/core/engine/index.ts
@@ -3,7 +3,7 @@
  */
 export { generateId, resetIdCounter } from './id';
 export { calc, totalMetrics, metricDefaults, formatMoney, formatWhole, formatBid, formatPercent, formatRoas, acosClass } from './metrics';
-export { normalizeCampaign, toggleCampaignStatus, archiveCampaign, duplicateCampaign, updateCampaignSettings, savePlacements } from './campaign';
+export { normalizeCampaign, toggleCampaignStatus, archiveCampaign, duplicateCampaign, updateCampaignSettings, savePlacements, isVideoFormat } from './campaign';
 export { addTarget, addKeyword, addAutoTarget, addAsinTarget, addCategoryTarget, removeTarget, setTargetBid, adjustTargetBid, pauseTarget, setTargetStatus } from './target';
 export { addAdGroup, addProductAd, addAd, renameAdGroup, setAdGroupStatus, setAdGroupDefaultBid, removeAdGroup } from './adgroup';
 export { isFilteredByNegative, addNegative, addNegativeKeyword, addNegativeAsin, addNegativeCategory, harvestTerm, getHarvestCandidates, getNegativeCandidates } from './negative';

From 813a2443887621b697b4f3be3aa1c8d8ab68348b Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:10:46 +0000
Subject: [PATCH 03/20] =?UTF-8?q?fix:=20more=20UI=20consistency=20?=
 =?UTF-8?q?=E2=80=94=20un-Card-wrap=20tables,=20token-ize=20hardcoded=20va?=
 =?UTF-8?q?lues?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- BulkOpsPage's CSV preview table and ReportsPage's selected-report
  table were Card-wrapped; split each into a header + bare Table,
  matching the edge-to-edge convention applied elsewhere.
- .pill.purple (the SD campaign-type badge) used raw hex (#f3e8ff /
  #7c3aed) while every sibling pill variant (.active, .orange, .green,
  .bad) resolves through --info/--accent/--success/--danger tokens.
  Added --purple/--purple-soft tokens with the same values so a future
  theme/rebrand updates this badge along with the rest.
- TargetsTab's "add keyword" inline form used raw pixel gap/margin
  literals (8, 10) instead of the --space-* tokens used for the same
  purpose everywhere else in the app.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 src/app/globals.css                                    |  6 ++++--
 src/components/AdConsole/details/TargetsTab.tsx        |  6 +++---
 src/components/AdConsole/features/bulk/BulkOpsPage.tsx |  4 ++--
 .../AdConsole/features/reports/ReportsPage.tsx         | 10 +++++++---
 4 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/src/app/globals.css b/src/app/globals.css
index 256a5c1..0a59452 100644
--- a/src/app/globals.css
+++ b/src/app/globals.css
@@ -60,6 +60,8 @@
   --danger-soft: #fde7ee;
   --info: #007185;             /* Amazon clickable teal (same as focus) */
   --info-soft: #e0f2f5;
+  --purple: #7c3aed;            /* SD campaign-type badge */
+  --purple-soft: #f3e8ff;
 
   /* Typography — Premium font stack */
   --font-display: 'Geist', -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, sans-serif;
@@ -1243,8 +1245,8 @@ textarea.input {
 }
 
 .pill.purple {
-  background: #f3e8ff;
-  color: #7c3aed;
+  background: var(--purple-soft);
+  color: var(--purple);
 }
 
 .pill.green {
diff --git a/src/components/AdConsole/details/TargetsTab.tsx b/src/components/AdConsole/details/TargetsTab.tsx
index 30e4c29..cd00db0 100644
--- a/src/components/AdConsole/details/TargetsTab.tsx
+++ b/src/components/AdConsole/details/TargetsTab.tsx
@@ -46,8 +46,8 @@ export function TargetsTab({ campaign: c }: Props) {
       )}
 
       {showAddKeywordForm && (
-        <Card variant="default" padding={6} style={{ marginBottom: 10, background: 'var(--surface-2)' }}>
-          <div style={{ display: 'flex', gap: 8, flexWrap: 'wrap', alignItems: 'end' }}>
+        <Card variant="default" padding={6} style={{ marginBottom: 'var(--space-3)', background: 'var(--surface-2)' }}>
+          <div style={{ display: 'flex', gap: 'var(--space-2)', flexWrap: 'wrap', alignItems: 'end' }}>
             <div className="field" style={{ flex: 2, minWidth: 150 }}>
               <label htmlFor="kw-value">Keyword</label>
               <input id="kw-value" className="input full" value={newKeywordValue} onChange={(e) => setNewKeywordValue(e.target.value)} placeholder="Enter keyword" />
@@ -69,7 +69,7 @@ export function TargetsTab({ campaign: c }: Props) {
               </select>
             </div>
           </div>
-          <div style={{ marginTop: 8, display: 'flex', gap: 8 }}>
+          <div style={{ marginTop: 'var(--space-2)', display: 'flex', gap: 'var(--space-2)' }}>
             <Button label="Add keyword" variant="primary" onClick={() => {
               if (!newKeywordValue.trim()) return;
               addKeyword(c.id, newKeywordValue.trim(), newKeywordMatch, newKeywordBid, newKeywordAdGroup);
diff --git a/src/components/AdConsole/features/bulk/BulkOpsPage.tsx b/src/components/AdConsole/features/bulk/BulkOpsPage.tsx
index 6ec7eac..1034439 100644
--- a/src/components/AdConsole/features/bulk/BulkOpsPage.tsx
+++ b/src/components/AdConsole/features/bulk/BulkOpsPage.tsx
@@ -63,7 +63,7 @@ export function BulkOpsPage() {
           )}
 
           {preview.length > 0 && (
-            <Card variant="default" padding={6}>
+            <div>
               <div className="card-title">
                 <h2>Preview</h2>
                 <span className="bulk-status">{preview.length} rows · {valid ? (
@@ -84,7 +84,7 @@ export function BulkOpsPage() {
                     ))}
                   </tbody>
                 </Table>
-            </Card>
+            </div>
           )}
 
           {!preview.length && !errors.length && (
diff --git a/src/components/AdConsole/features/reports/ReportsPage.tsx b/src/components/AdConsole/features/reports/ReportsPage.tsx
index eeb822f..0060569 100644
--- a/src/components/AdConsole/features/reports/ReportsPage.tsx
+++ b/src/components/AdConsole/features/reports/ReportsPage.tsx
@@ -67,12 +67,12 @@ export function ReportsPage() {
       )}
 
       {selected && (
-        <Card variant="default" padding={6}>
+        <div>
           <div className="card-title">
             <h2>{selected.type} report</h2>
             <Button label="Export CSV" size="sm" onClick={() => handleExportCsv(selected.id, selected.type)} />
           </div>
-          {selected.rows.length > 0 && (
+          {selected.rows.length > 0 ? (
             <Table>
                 <thead>
                   <tr>{Object.keys(selected.rows[0]).map((h) => <th key={h}>{h}</th>)}</tr>
@@ -93,8 +93,12 @@ export function ReportsPage() {
                   })}
                 </tbody>
               </Table>
+          ) : (
+            <Card variant="default" padding={6}>
+              <p className="muted">This report type has no data yet.</p>
+            </Card>
           )}
-        </Card>
+        </div>
       )}
 
       {!requests.length && (

From 1ffe450dd0efe4ffcb3ffb4d770cf8424c937436 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:17:01 +0000
Subject: [PATCH 04/20] fix: dedupe ad-hoc ID generation onto the single
 generateId source of truth
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

launchCampaign built campaign ids from Date.now() alone with no
counter, unlike every other entity id in the engine — two campaigns
of the same type launched within the same millisecond (e.g. a
double-clicked "Launch" button) collided, corrupting lookups that
assume unique ids. Now uses generateId('C-' + type), matching the
identical pattern duplicateCampaign already uses.

Also replaced four near-identical hand-rolled uid()/counter/Date.now
helpers in the reports, profiles, integrity, and trainer feature
engines — each was its own independent module-scoped counter that
resets on hot reload — with the shared generateId(prefix), so ID
generation has exactly one implementation instead of five.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 src/engine/ad-console/__tests__/store.test.ts | 23 +++++++++++++++++++
 src/engine/ad-console/core/slices/core.ts     |  2 +-
 .../ad-console/features/integrity/engine.ts   |  5 ++--
 .../ad-console/features/profiles/engine.ts    |  9 ++------
 .../ad-console/features/reports/engine.ts     | 11 +++------
 .../ad-console/features/trainer/engine.ts     |  9 ++------
 6 files changed, 33 insertions(+), 26 deletions(-)

diff --git a/src/engine/ad-console/__tests__/store.test.ts b/src/engine/ad-console/__tests__/store.test.ts
index d3226ac..d251bb1 100644
--- a/src/engine/ad-console/__tests__/store.test.ts
+++ b/src/engine/ad-console/__tests__/store.test.ts
@@ -26,6 +26,29 @@ describe('Store actions', () => {
       expect(campaign?.targets.map(t => t.match)).toEqual(['Exact', 'Phrase', 'Broad']);
     });
 
+    it('assigns unique ids even when launched within the same millisecond', () => {
+      // Regression: the id used to be built from Date.now() alone with no
+      // counter, so two launches in the same tick (e.g. a double-clicked
+      // "Launch" button) produced identical campaign ids.
+      const nowSpy = vi.spyOn(Date, 'now').mockReturnValue(1700000000000);
+      try {
+        const store = useAdConsoleStore.getState();
+        store.updateDraft('name', 'First');
+        store.launchCampaign();
+        store.updateDraft('name', 'Second');
+        store.launchCampaign();
+
+        const state = useAdConsoleStore.getState().state;
+        const first = state.campaigns.find(c => c.name === 'First');
+        const second = state.campaigns.find(c => c.name === 'Second');
+        expect(first?.id).toBeDefined();
+        expect(second?.id).toBeDefined();
+        expect(first?.id).not.toBe(second?.id);
+      } finally {
+        nowSpy.mockRestore();
+      }
+    });
+
     it('does not create campaign if name is empty', () => {
       const store = useAdConsoleStore.getState();
       store.updateDraft('name', '');
diff --git a/src/engine/ad-console/core/slices/core.ts b/src/engine/ad-console/core/slices/core.ts
index 93e388f..2982208 100644
--- a/src/engine/ad-console/core/slices/core.ts
+++ b/src/engine/ad-console/core/slices/core.ts
@@ -67,7 +67,7 @@ export const createCoreSlice = (set: any, get: any, ..._rest: any[]): CoreSlice
   launchCampaign: () => set((s: any) => {
     const d = s.draft;
     if (!d.name.trim()) return s;
-    const id = 'C-' + d.type + '-' + Date.now().toString(36);
+    const id = generateId('C-' + d.type);
     const agId = 'AG-' + id;
     const portfolioName = d.portfolio || 'Training Portfolio';
     const buildTargets = (raw: string, type: 'Keyword' | 'ASIN' | 'Category' | 'Audience - views remarketing', match?: any) =>
diff --git a/src/engine/ad-console/features/integrity/engine.ts b/src/engine/ad-console/features/integrity/engine.ts
index 5ebc3e6..f7f4b66 100644
--- a/src/engine/ad-console/features/integrity/engine.ts
+++ b/src/engine/ad-console/features/integrity/engine.ts
@@ -7,11 +7,10 @@
 import type { Campaign } from '../../core/types';
 import type { IntegrityIssue, IntegrityReport } from './types';
 import { ValidationError } from '../../../../lib/validation';
+import { generateId } from '../../core/engine/id';
 
-let _counter = 0;
 function uid(): string {
-  _counter++;
-  return 'II-' + Date.now().toString(36) + '-' + _counter;
+  return generateId('II');
 }
 
 export function runIntegrityCheck(campaigns: Campaign[]): IntegrityReport {
diff --git a/src/engine/ad-console/features/profiles/engine.ts b/src/engine/ad-console/features/profiles/engine.ts
index df054e5..c2cdbd1 100644
--- a/src/engine/ad-console/features/profiles/engine.ts
+++ b/src/engine/ad-console/features/profiles/engine.ts
@@ -3,16 +3,11 @@
  */
 import type { TraineeProfile } from './types';
 import { assertNonEmpty } from '../../../../lib/validation';
-
-let _counter = 0;
-function uid(): string {
-  _counter++;
-  return 'P-' + Date.now().toString(36) + '-' + _counter;
-}
+import { generateId } from '../../core/engine/id';
 
 export function createProfile(name: string): TraineeProfile {
   return {
-    id: uid(),
+    id: generateId('P'),
     name: name.trim().slice(0, 25) || 'Trainee',
     createdAt: new Date().toISOString(),
     lastActiveAt: new Date().toISOString(),
diff --git a/src/engine/ad-console/features/reports/engine.ts b/src/engine/ad-console/features/reports/engine.ts
index b0b9b38..c979451 100644
--- a/src/engine/ad-console/features/reports/engine.ts
+++ b/src/engine/ad-console/features/reports/engine.ts
@@ -3,19 +3,14 @@
  */
 import type { Report, ReportRequest, ReportType } from './types';
 import { assertNonEmpty, ValidationError } from '../../../../lib/validation';
-
-let _counter = 0;
-function uid(): string {
-  _counter++;
-  return 'R-' + Date.now().toString(36) + '-' + _counter;
-}
+import { generateId } from '../../core/engine/id';
 
 const REPORT_TYPES: ReportType[] = ['campaign', 'adGroup', 'target', 'searchTerm', 'placement'];
 
 export function createReportRequest(type: ReportType): ReportRequest {
   if (!REPORT_TYPES.includes(type)) throw new ValidationError(`Unknown report type: ${type}`);
   return {
-    id: uid(),
+    id: generateId('R'),
     type,
     status: 'pending',
     requestedAt: new Date().toISOString(),
@@ -49,7 +44,7 @@ export function generateReport(type: ReportType): Report {
   }
 
   return {
-    id: uid(),
+    id: generateId('R'),
     type,
     rows,
     generatedAt: now,
diff --git a/src/engine/ad-console/features/trainer/engine.ts b/src/engine/ad-console/features/trainer/engine.ts
index 34b950b..2117482 100644
--- a/src/engine/ad-console/features/trainer/engine.ts
+++ b/src/engine/ad-console/features/trainer/engine.ts
@@ -3,17 +3,12 @@
  */
 import type { TrainerNote, TrainerState } from './types';
 import { assertNonEmpty } from '../../../../lib/validation';
-
-let _counter = 0;
-function uid(): string {
-  _counter++;
-  return 'TN-' + Date.now().toString(36) + '-' + _counter;
-}
+import { generateId } from '../../core/engine/id';
 
 export function addNote(text: string): TrainerNote {
   assertNonEmpty('note text', text);
   return {
-    id: uid(),
+    id: generateId('TN'),
     timestamp: new Date().toISOString(),
     text: text.trim(),
   };

From a945dc96a277adb68c8306c600cf2dfb666d0e25 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:20:18 +0000
Subject: [PATCH 05/20] fix: dedupe training-views list and KPI formulas in
 consoleNav.ts
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The 6-item training-view check (drills/missions/reports/bulk/trainer/
integrity) was repeated as an identical view === 'x' || ... chain in
three functions (getLeftRail, activeTopbarSection,
sidebarSectionForView) — exactly the class of duplication that
already caused audit H-03 (one of the copies falling out of sync).
Extracted a single TRAINING_VIEWS set all three now check against.

getKpiTiles also hand-rolled its own ctr/acos/roas formulas instead
of calling the canonical calc(), duplicating logic that a future
formula fix (e.g. guarding against negative sales) would otherwise
need to be applied in two places.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 src/components/AdConsole/nav/consoleNav.ts | 49 +++++++---------------
 1 file changed, 16 insertions(+), 33 deletions(-)

diff --git a/src/components/AdConsole/nav/consoleNav.ts b/src/components/AdConsole/nav/consoleNav.ts
index 0326dcd..cac25a3 100644
--- a/src/components/AdConsole/nav/consoleNav.ts
+++ b/src/components/AdConsole/nav/consoleNav.ts
@@ -7,6 +7,7 @@
  */
 
 import { ValidationError } from '../../../lib/validation';
+import { calc } from '../../../engine/ad-console/core/engine';
 
 export type NavView =
   | 'dashboard'
@@ -67,6 +68,17 @@ export const GLOBAL_NAV: NavSection[] = [
   { key: 'training', label: 'Training', view: 'drills' },
 ];
 
+/**
+ * The 6 training-product views. Single source of truth for "is this view
+ * part of Training" — previously repeated as an identical `view === 'x' ||
+ * ...` chain in three functions, which is exactly the kind of duplication
+ * that caused audit H-03 (a training view silently falling through to the
+ * wrong default because one of the three lists fell out of sync).
+ */
+const TRAINING_VIEWS: ReadonlySet<NavView> = new Set([
+  'drills', 'missions', 'reports', 'bulk', 'trainer', 'integrity',
+]);
+
 /**
  * Rail items for the Training section. The order matches the typical
  * trainee journey: hands-on drills first, then missions, then reporting
@@ -122,16 +134,7 @@ export function getLeftRail(section: NavView | RailSection): LeftRailItem[] {
   // The training rail is registered under each of the 6 training views
   // in LEFT_RAIL, so passing any of them returns the same training rail.
   if (section === 'training') return TRAINING_RAIL;
-  if (
-    section === 'drills' ||
-    section === 'missions' ||
-    section === 'reports' ||
-    section === 'bulk' ||
-    section === 'trainer' ||
-    section === 'integrity'
-  ) {
-    return TRAINING_RAIL;
-  }
+  if (TRAINING_VIEWS.has(section as NavView)) return TRAINING_RAIL;
   return LEFT_RAIL[section as NavView] ?? LEFT_RAIL.campaigns;
 }
 
@@ -147,16 +150,7 @@ export function getLeftRail(section: NavView | RailSection): LeftRailItem[] {
 export function activeTopbarSection(view: string): RailSection {
   if (view === 'portfolio') return 'portfolio';
   if (view === 'dashboard') return 'dashboard';
-  if (
-    view === 'drills' ||
-    view === 'missions' ||
-    view === 'reports' ||
-    view === 'bulk' ||
-    view === 'trainer' ||
-    view === 'integrity'
-  ) {
-    return 'training';
-  }
+  if (TRAINING_VIEWS.has(view as NavView)) return 'training';
   return 'campaigns';
 }
 
@@ -168,16 +162,7 @@ export function activeTopbarSection(view: string): RailSection {
 export function sidebarSectionForView(view: string): RailSection {
   if (view === 'portfolio') return 'portfolio';
   if (view === 'dashboard') return 'dashboard';
-  if (
-    view === 'drills' ||
-    view === 'missions' ||
-    view === 'reports' ||
-    view === 'bulk' ||
-    view === 'trainer' ||
-    view === 'integrity'
-  ) {
-    return 'training';
-  }
+  if (TRAINING_VIEWS.has(view as NavView)) return 'training';
   return 'campaigns';
 }
 
@@ -254,9 +239,7 @@ export function getKpiTiles(m: MetricsSnapshot): KpiTile[] {
   ) {
     throw new ValidationError('MetricsSnapshot requires finite numeric fields');
   }
-  const ctr = m.impressions ? (m.clicks / m.impressions) * 100 : 0;
-  const acos = m.sales ? (m.spend / m.sales) * 100 : 0;
-  const roas = m.spend ? m.sales / m.spend : 0;
+  const { ctr, acos, roas } = calc(m);
   return [
     { key: 'impressions', label: 'Impressions', value: whole(m.impressions) },
     { key: 'clicks', label: 'Clicks', value: whole(m.clicks) },

From bae73783d4e0201718b7f86ee0ac2bb8693c64d5 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:22:43 +0000
Subject: [PATCH 06/20] fix: remove dead state from CreateCampaignWizard
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

13 useState hooks (keyword/target inputs, SB creative fields) and two
store-action hooks (selectProduct/removeProduct) were initialized
from the draft but never read or updated anywhere in the component —
the actual form state lives in each step's own store subscription
(draft/updateDraft), not these local copies. A future maintainer
editing this file could reasonably assume these hooks were live and
wire new logic to them, silently doing nothing since the real data
flow bypasses them entirely.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 .../AdConsole/wizard/CreateCampaignWizard.tsx | 22 -------------------
 1 file changed, 22 deletions(-)

diff --git a/src/components/AdConsole/wizard/CreateCampaignWizard.tsx b/src/components/AdConsole/wizard/CreateCampaignWizard.tsx
index 027a7e4..adee00d 100644
--- a/src/components/AdConsole/wizard/CreateCampaignWizard.tsx
+++ b/src/components/AdConsole/wizard/CreateCampaignWizard.tsx
@@ -1,11 +1,8 @@
 'use client';
 
-import { useState } from 'react';
 import { Button } from '@astryxdesign/core/Button';
 import { Check } from '@phosphor-icons/react';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
-import type { CampaignType, CampaignDraft } from '@/engine/ad-console/types';
-import { PRODUCTS, BRANDS } from '@/engine/ad-console/core/scenarios';
 
 // Step components
 import { Step1AdType } from './Step1AdType';
@@ -39,28 +36,9 @@ export function CreateCampaignWizard() {
   const launchCampaign = useAdConsoleStore((s) => s.launchCampaign);
   const resetDraft = useAdConsoleStore((s) => s.resetDraft);
   const setView = useAdConsoleStore((s) => s.setView);
-  const selectProductAction = useAdConsoleStore((s) => s.selectProduct);
-  const removeProductAction = useAdConsoleStore((s) => s.removeProduct);
 
   const d = draft;
 
-  // Local state for forms
-  const [exactKeywords, setExactKeywords] = useState(d.exactKeywords || '');
-  const [phraseKeywords, setPhraseKeywords] = useState(d.phraseKeywords || '');
-  const [broadKeywords, setBroadKeywords] = useState(d.broadKeywords || '');
-  const [asinTargets, setAsinTargets] = useState(d.asinTargets || '');
-  const [categoryTargets, setCategoryTargets] = useState(d.categoryTargets || '');
-  const [audienceTargets, setAudienceTargets] = useState(d.audienceTargets || '');
-  const [audienceLookback, setAudienceLookback] = useState(d.audienceLookback || '30');
-
-  // SB-specific state
-  const [storeUrl, setStoreUrl] = useState(d.creative.destination || '');
-  const [brandId, setBrandId] = useState(d.creative.brandName ? BRANDS.find(b => b.name === d.creative.brandName)?.id || '' : '');
-  const [logo, setLogo] = useState(d.creative.logo || '');
-  const [headline, setHeadline] = useState(d.creative.headline || '');
-  const [image, setImage] = useState(d.creative.image || '');
-  const [video, setVideo] = useState(d.creative.video || '');
-
   const isComplete = (step: number) => wizardStep > step;
   const isActive = (step: number) => wizardStep === step;
 

From 7fc10d08633b5fb035e77942ca94b7463923577c Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:25:29 +0000
Subject: [PATCH 07/20] =?UTF-8?q?perf:=20fix=20O(n=C2=B7m)=20search-term?=
 =?UTF-8?q?=20dedup=20loop=20and=20memoize=20dashboard=20aggregates?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- simulation.ts checked each newly generated search term against both
  the campaign's existing terms and the terms generated so far via two
  nested linear scans per candidate. Since searchTerms only grows
  across simulation runs (never pruned), each subsequent 7-day sim
  click re-scanned an ever-larger array. Replaced with a single Set
  for O(1) membership checks.
- Dashboard subscribed to the whole store `state` and recomputed
  totalMetrics/calc/getKpiTiles directly in the render body, so any
  state change anywhere in the app (not just campaign metrics) redid
  the full-campaign-list aggregate. Memoized on state.campaigns, which
  keeps the same array reference for state changes that don't touch
  campaigns.
- ManagerSearchTermsTab did the same flatMap+filter scan (O(search
  terms x negatives)) directly in the render body with no
  memoization; wrapped in useMemo keyed on the campaigns prop.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 src/components/AdConsole/Dashboard.tsx        | 28 ++++++++++++-------
 .../details/ManagerSearchTermsTab.tsx         | 13 ++++++---
 src/engine/ad-console/core/simulation.ts      | 23 +++++++--------
 3 files changed, 37 insertions(+), 27 deletions(-)

diff --git a/src/components/AdConsole/Dashboard.tsx b/src/components/AdConsole/Dashboard.tsx
index ce95850..d79c90d 100644
--- a/src/components/AdConsole/Dashboard.tsx
+++ b/src/components/AdConsole/Dashboard.tsx
@@ -1,5 +1,6 @@
 'use client';
 
+import { useMemo } from 'react';
 import { Button } from '@astryxdesign/core/Button';
 import { Table } from '@astryxdesign/core/Table';
 import { Card } from '@astryxdesign/core/Card';
@@ -15,16 +16,23 @@ export function Dashboard() {
   const selectCampaign = useAdConsoleStore((s) => s.selectCampaign);
   const totalMetrics = useAdConsoleStore((s) => s.totalMetricsCalc);
 
-  const m = totalMetrics();
-  const d = calc(m);
-  const tiles = getKpiTiles({
-    impressions: m.impressions,
-    clicks: m.clicks,
-    spend: m.spend,
-    sales: m.sales,
-    orders: m.orders,
-    units: m.orders,
-  });
+  // state.campaigns keeps the same array reference for any state change
+  // that doesn't touch campaigns (filter/tab/mobile-menu toggles, etc.),
+  // so this skips recomputing the full-campaign-list aggregate on those.
+  const m = useMemo(() => totalMetrics(), [state.campaigns]); // eslint-disable-line react-hooks/exhaustive-deps
+  const d = useMemo(() => calc(m), [m]);
+  const tiles = useMemo(
+    () =>
+      getKpiTiles({
+        impressions: m.impressions,
+        clicks: m.clicks,
+        spend: m.spend,
+        sales: m.sales,
+        orders: m.orders,
+        units: m.orders,
+      }),
+    [m],
+  );
 
   const enabledCount = state.campaigns.filter((c) => c.status === 'Enabled').length;
   const acosHealthy = d.acos > 0 && d.acos <= 30;
diff --git a/src/components/AdConsole/details/ManagerSearchTermsTab.tsx b/src/components/AdConsole/details/ManagerSearchTermsTab.tsx
index ee38ef8..cfffb46 100644
--- a/src/components/AdConsole/details/ManagerSearchTermsTab.tsx
+++ b/src/components/AdConsole/details/ManagerSearchTermsTab.tsx
@@ -1,5 +1,6 @@
 'use client';
 
+import { useMemo } from 'react';
 import { Table } from '@astryxdesign/core/Table';
 import type { Campaign } from '@/engine/ad-console/types';
 import { calc, formatMoney, formatWhole, formatPercent, formatBid, formatRoas, acosClass, isFilteredByNegative } from '@/engine/ad-console/core/engine';
@@ -10,10 +11,14 @@ interface Props {
 }
 
 export function ManagerSearchTermsTab({ campaigns }: Props) {
-  const rows = campaigns.flatMap((c) =>
-    (c.searchTerms || [])
-      .filter((st) => !isFilteredByNegative(st.term, c.negatives))
-      .map((st) => ({ c, st }))
+  const rows = useMemo(
+    () =>
+      campaigns.flatMap((c) =>
+        (c.searchTerms || [])
+          .filter((st) => !isFilteredByNegative(st.term, c.negatives))
+          .map((st) => ({ c, st }))
+      ),
+    [campaigns],
   );
 
   if (!rows.length) {
diff --git a/src/engine/ad-console/core/simulation.ts b/src/engine/ad-console/core/simulation.ts
index c3dde86..7df376a 100644
--- a/src/engine/ad-console/core/simulation.ts
+++ b/src/engine/ad-console/core/simulation.ts
@@ -74,31 +74,28 @@ export function simulateDays(campaigns: Campaign[], days: number = 7): Campaign[
     // Supports SP and SB campaigns (SD doesn't have search terms by design)
     const generatedST: SearchTerm[] = [];
     const shouldGenerateSearchTerms = c.type === 'SP' || c.type === 'SB';
-    
+    // Tracks every term already in the campaign or generated this pass, so
+    // duplicate checks are O(1) instead of two linear scans per candidate
+    // (c.searchTerms only grows across simulation runs, so this mattered).
+    const seenTerms = new Set(c.searchTerms.map((st) => st.term));
+
     if (shouldGenerateSearchTerms) {
       for (let si = 0; si < enabledTargets.length; si++) {
         const tgt = enabledTargets[si];
         if (tgt.type !== 'Keyword') continue;
-        
+
         // Use the new generator with negative filtering during generation
         const generatedTerms = generateSearchTermsForTarget(
           tgt.value,
           tgt.match as 'Exact' | 'Phrase' | 'Broad',
           c.negatives.map(n => ({ value: n.value, type: n.type }))
         );
-        
+
         for (let gi = 0; gi < generatedTerms.length; gi++) {
           const gt = generatedTerms[gi];
-          // Check if already exists in campaign search terms (avoid duplicates)
-          let exists = false;
-          for (let ei = 0; ei < c.searchTerms.length; ei++) {
-            if (c.searchTerms[ei].term === gt) { exists = true; break; }
-          }
-          for (let ei = 0; ei < generatedST.length; ei++) {
-            if (generatedST[ei].term === gt) { exists = true; break; }
-          }
-          if (exists) continue;
-          
+          if (seenTerms.has(gt)) continue;
+          seenTerms.add(gt);
+
           const termShare = 0.15 + Math.random() * 0.1;
           const termClicks = Math.max(1, Math.round(tgt.clicks * termShare));
           const termSpend = tgt.spend * termShare;

From 77b494ef65b764e47215b8606ee6b15c5d944bcc Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:27:48 +0000
Subject: [PATCH 08/20] fix: replace dead Tailwind sizing classes with explicit
 SVG dimensions
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The landing page's feature-grid and CTA icons used Tailwind sizing
classes (w-6 h-6, w-4 h-4), but this repo has no Tailwind/PostCSS
compiler wired up (confirmed: no tailwind.config.*, no
postcss.config.*, not in package.json). Without a compiler these
classes are dead — the SVGs had no explicit width/height, so browsers
fall back to the default inline-SVG size (300x150) instead of the
intended 24px/16px icons.

Replaced with explicit width/height attributes, matching how
EmptyState.tsx already sizes its icon SVGs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012Jpq3N94psa27k8xASDy8B
---
 src/app/page.tsx | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/app/page.tsx b/src/app/page.tsx
index d7503de..f622763 100644
--- a/src/app/page.tsx
+++ b/src/app/page.tsx
@@ -10,7 +10,7 @@ const FEATURES = [
     title: 'Campaign Creation Wizard',
     description: 'Build SP, SB, and SD campaigns step by step ΓÇö the same interface you\'ll use in production.',
     icon: (
-      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" className="w-6 h-6">
+      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M9 5H7a2 2 0 00-2 2v12a2 2 0 002 2h10a2 2 0 002-2V7a2 2 0 00-2-2h-2M9 5a2 2 0 002 2h2a2 2 0 002-2M9 5a2 2 0 012-2h2a2 2 0 012 2m-3 7h3m-3 4h3m-6-4h.01M9 16h.01" />
       </svg>
     ),
@@ -19,7 +19,7 @@ const FEATURES = [
     title: '7-Day Performance Simulation',
     description: 'Watch realistic metrics build ΓÇö ROAS, ACOS, CPC, impressions ΓÇö exactly like a live campaign.',
     icon: (
-      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" className="w-6 h-6">
+      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M9 19v-6a2 2 0 00-2-2H5a2 2 0 00-2 2v6a2 2 0 002 2h2a2 2 0 002-2zm0 0V9a2 2 0 012-2h2a2 2 0 012 2v10m-6 0a2 2 0 002 2h2a2 2 0 002-2m0 0V5a2 2 0 012-2h2a2 2 0 012 2v14a2 2 0 01-2 2h-2a2 2 0 01-2-2z" />
       </svg>
     ),
@@ -28,7 +28,7 @@ const FEATURES = [
     title: 'Search Term Mining',
     description: 'Identify which search terms convert. Harvest winners, negate losers ΓÇö the optimization loop that actually moves ACOS.',
     icon: (
-      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" className="w-6 h-6">
+      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M21 21l-6-6m2-5a7 7 0 11-14 0 7 7 0 0114 0z" />
       </svg>
     ),
@@ -37,7 +37,7 @@ const FEATURES = [
     title: 'Guided Drills',
     description: 'Click-by-click coaching walks you through the console. Track mistakes, earn scores, level up.',
     icon: (
-      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" className="w-6 h-6">
+      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M9.663 17h4.673M12 3v1m6.364 1.636l-.707.707M21 12h-1M4 12H3m3.343-5.657l-.707-.707m2.828 9.9a5 5 0 117.072 0l-.548.547A3.374 3.374 0 0014 18.469V19a2 2 0 11-4 0v-.531c0-.895-.356-1.754-.988-2.386l-.548-.547z" />
       </svg>
     ),
@@ -46,7 +46,7 @@ const FEATURES = [
     title: 'Scenario Missions',
     description: 'Real-world challenges from beginner ACOS reduction to advanced auto-targeting. Get scored, get better.',
     icon: (
-      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" className="w-6 h-6">
+      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M9 12l2 2 4-4m5.618-4.016A11.955 11.955 0 0112 2.944a11.955 11.955 0 01-8.618 3.04A12.02 12.02 0 003 9c0 5.591 3.824 10.29 9 11.622 5.176-1.332 9-6.03 9-11.622 0-1.042-.133-2.052-.382-3.016z" />
       </svg>
     ),
@@ -55,7 +55,7 @@ const FEATURES = [
     title: 'Bulk CSV Operations',
     description: 'Paste your Amazon bulk export. Validate it, preview the changes, apply it ΓÇö no more spreadsheet errors.',
     icon: (
-      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" className="w-6 h-6">
+      <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M4 16v1a3 3 0 003 3h10a3 3 0 003-3v-1m-4-8l-4-4m0 0L8 8m4-4v12" />
       </svg>
     ),
@@ -141,7 +141,7 @@ export default function LandingPage() {
             transition={{ duration: 0.6, ease: [0.16, 1, 0.3, 1] }}
             className="landing-hero-label"
           >
-            <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" className="w-4 h-4">
+            <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="16" height="16">
               <path d="M15 10l4.553-2.276A1 1 0 0121 8.618v6.764a1 1 0 01-1.447.894L15 14M5 18h8a2 2 0 002-2V8a2 2 0 00-2-2H5a2 2 0 00-2 2v8a2 2 0 002 2z" />
             </svg>
             <span>No Amazon account needed</span>
@@ -175,7 +175,7 @@ export default function LandingPage() {
           >
             <Link href="/dashboard" className="landing-cta-primary landing-cta-primary--lg">
               Start Training Free
-              <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2" className="w-4 h-4">
+              <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2" width="16" height="16">
                 <path d="M5 12h14M12 5l7 7-7 7" />
               </svg>
             </Link>
@@ -298,7 +298,7 @@ export default function LandingPage() {
             <p>Build your first campaign in the next 5 minutes.</p>
             <Link href="/dashboard" className="landing-cta-primary landing-cta-primary--lg landing-cta-primary--centered">
               Open the Simulator
-              <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2" className="w-4 h-4">
+              <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2" width="16" height="16">
                 <path d="M5 12h14M12 5l7 7-7 7" />
               </svg>
             </Link>

From eb6d6569213e8b10c397baadee6a34292561624b Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:53:35 +0000
Subject: [PATCH 09/20] Fix mojibake em-dashes/arrow in landing page copy
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

src/app/page.tsx had literal garbled byte sequences (—, →) committed
in place of an em-dash and a right-arrow, likely from an encoding
mismatch at some earlier edit. Replaced with the correct UTF-8
characters.
---
 src/app/page.tsx | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/app/page.tsx b/src/app/page.tsx
index f622763..01c873a 100644
--- a/src/app/page.tsx
+++ b/src/app/page.tsx
@@ -8,7 +8,7 @@ import { useEffect, useRef, useState } from 'react';
 const FEATURES = [
   {
     title: 'Campaign Creation Wizard',
-    description: 'Build SP, SB, and SD campaigns step by step ΓÇö the same interface you\'ll use in production.',
+    description: 'Build SP, SB, and SD campaigns step by step — the same interface you\'ll use in production.',
     icon: (
       <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M9 5H7a2 2 0 00-2 2v12a2 2 0 002 2h10a2 2 0 002-2V7a2 2 0 00-2-2h-2M9 5a2 2 0 002 2h2a2 2 0 002-2M9 5a2 2 0 012-2h2a2 2 0 012 2m-3 7h3m-3 4h3m-6-4h.01M9 16h.01" />
@@ -17,7 +17,7 @@ const FEATURES = [
   },
   {
     title: '7-Day Performance Simulation',
-    description: 'Watch realistic metrics build ΓÇö ROAS, ACOS, CPC, impressions ΓÇö exactly like a live campaign.',
+    description: 'Watch realistic metrics build — ROAS, ACOS, CPC, impressions — exactly like a live campaign.',
     icon: (
       <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M9 19v-6a2 2 0 00-2-2H5a2 2 0 00-2 2v6a2 2 0 002 2h2a2 2 0 002-2zm0 0V9a2 2 0 012-2h2a2 2 0 012 2v10m-6 0a2 2 0 002 2h2a2 2 0 002-2m0 0V5a2 2 0 012-2h2a2 2 0 012 2v14a2 2 0 01-2 2h-2a2 2 0 01-2-2z" />
@@ -26,7 +26,7 @@ const FEATURES = [
   },
   {
     title: 'Search Term Mining',
-    description: 'Identify which search terms convert. Harvest winners, negate losers ΓÇö the optimization loop that actually moves ACOS.',
+    description: 'Identify which search terms convert. Harvest winners, negate losers — the optimization loop that actually moves ACOS.',
     icon: (
       <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M21 21l-6-6m2-5a7 7 0 11-14 0 7 7 0 0114 0z" />
@@ -53,7 +53,7 @@ const FEATURES = [
   },
   {
     title: 'Bulk CSV Operations',
-    description: 'Paste your Amazon bulk export. Validate it, preview the changes, apply it ΓÇö no more spreadsheet errors.',
+    description: 'Paste your Amazon bulk export. Validate it, preview the changes, apply it — no more spreadsheet errors.',
     icon: (
       <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="1.5" width="24" height="24">
         <path d="M4 16v1a3 3 0 003 3h10a3 3 0 003-3v-1m-4-8l-4-4m0 0L8 8m4-4v12" />
@@ -66,7 +66,7 @@ const STEPS = [
   {
     num: '01',
     title: 'Build your first campaign',
-    description: 'Choose SP, SB, or SD. Walk through targeting, bidding, and creative settings ΓÇö exactly like the real console.',
+    description: 'Choose SP, SB, or SD. Walk through targeting, bidding, and creative settings — exactly like the real console.',
   },
   {
     num: '02',
@@ -195,14 +195,14 @@ export default function LandingPage() {
         </div>
       </section>
 
-      {/* Problem → Solution */}
+      {/* Problem → Solution */}
       <section className="landing-problem">
         <div className="container">
           <div className="landing-problem-grid">
             <FadeIn className="landing-problem-col">
               <div className="landing-problem-label">The old way</div>
               <ul className="landing-problem-list">
-                <li>Practice on live campaigns ΓÇö real money, real risk</li>
+                <li>Practice on live campaigns — real money, real risk</li>
                 <li>Learn by trial and expensive error</li>
                 <li>No safe space to experiment with new strategies</li>
                 <li>No guided coaching when you get stuck</li>
@@ -267,7 +267,7 @@ export default function LandingPage() {
         <div className="container">
           <FadeIn className="landing-section-header">
             <h2>The exact interface you&apos;ll use in production</h2>
-            <p>No simplified training wheels ΓÇö this is the real console experience.</p>
+            <p>No simplified training wheels — this is the real console experience.</p>
           </FadeIn>
 
           <FadeIn className="landing-preview-wrapper">

From 8d5e938bdc91185be4bb9619fbb794ec970a484d Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:54:17 +0000
Subject: [PATCH 10/20] Add missing object-fit:cover rule for landing page
 images

Two Next.js <Image fill> elements on the landing page use
className="object-cover" but no CSS rule ever defined that class (no
Tailwind compiler is wired up in this repo), so the images had no
object-fit applied and could render stretched.
---
 src/app/globals.css | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/app/globals.css b/src/app/globals.css
index 0a59452..2a27075 100644
--- a/src/app/globals.css
+++ b/src/app/globals.css
@@ -3138,6 +3138,7 @@ textarea.input {
 }
 
 .text-accent { color: var(--landing-accent); }
+.object-cover { object-fit: cover; }
 
 /* Navigation */
 .landing-nav {

From e7f5520a3ae3eb90f7dfde2600e9987c0a88059c Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:54:30 +0000
Subject: [PATCH 11/20] Fix dashboard sidebar/alerts card overflowing viewport

.split used grid-template-columns: 2fr 1fr, which per the CSS grid
sizing algorithm floors each track at its content's min-content width
rather than actually letting it shrink to its fractional share. With
the app sidebar now taking up real width (see next commit), the 1fr
column (Operator alerts / Training coverage) no longer fit and was
clipped past the right edge of the viewport instead of wrapping.
minmax(0, ...) lets both tracks shrink to fit, so content wraps inside
its column instead of overflowing.
---
 src/app/globals.css | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/globals.css b/src/app/globals.css
index 2a27075..3f21f7d 100644
--- a/src/app/globals.css
+++ b/src/app/globals.css
@@ -1074,7 +1074,7 @@ code, pre {
 
 .split {
   display: grid;
-  grid-template-columns: 2fr 1fr;
+  grid-template-columns: minmax(0, 2fr) minmax(0, 1fr);
   gap: var(--space-5);
 }
 

From edb2369daf242273bbdf29f417a609d1a37e7737 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 13:54:55 +0000
Subject: [PATCH 12/20] =?UTF-8?q?Wire=20the=20Sidebar=20into=20the=20app?=
 =?UTF-8?q?=20layout=20=E2=80=94=20Missions/Reports/Bulk=20ops/Trainer/Int?=
 =?UTF-8?q?egrity=20were=20unreachable?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The Sidebar component (and the getLeftRail/TRAINING_RAIL model behind
it, with its own full unit-test coverage in consoleNav.test.ts) was
fully built but never mounted anywhere in the component tree — the
.app-body/.app-sidebar CSS classes it depends on existed in
globals.css but had no consumer either. Since the top nav's "Training"
button only lands on Drills, there was no way for a desktop user to
reach Missions, Reports, Bulk ops, Trainer, or Integrity, and no way to
jump directly to a campaign's Ad groups/Targeting/etc. tabs or
Portfolio's Budget rules from the nav.

Mount <Sidebar /> in AdConsole.tsx inside a new .app-body wrapper
alongside the existing <main>, and hide it below the tablet breakpoint
(same threshold useBreakpoint uses for isMobileOrTablet) so it doesn't
double up with the mobile hamburger drawer.

MobileNav had the same underlying gap from a different angle: its
`section` was hardcoded to only ever resolve to 'portfolio' or
'campaigns', so training views never got a training rail, and its
active-item logic only matched the 'campaigns' group. Switched both to
the shared sidebarSectionForView/isSidebarItemActive/resolveSidebarClick
helpers that Sidebar itself uses, so mobile and desktop navigation stay
consistent.

Added a contract test pinning that the sidebar renders and actually
drives navigation to the previously-unreachable views.
---
 src/components/AdConsole/AdConsole.tsx        | 14 +--
 .../__tests__/contracts/sidebar.test.tsx      | 88 +++++++++++++++++++
 src/components/AdConsole/mobile/MobileNav.tsx | 14 ++-
 3 files changed, 107 insertions(+), 9 deletions(-)
 create mode 100644 src/components/AdConsole/__tests__/contracts/sidebar.test.tsx

diff --git a/src/components/AdConsole/AdConsole.tsx b/src/components/AdConsole/AdConsole.tsx
index dd8d1f5..84b4247 100644
--- a/src/components/AdConsole/AdConsole.tsx
+++ b/src/components/AdConsole/AdConsole.tsx
@@ -4,6 +4,7 @@ import { Button } from '@astryxdesign/core/Button';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
 
 import { Topbar } from './layout/Topbar';
+import { Sidebar } from './layout/Sidebar';
 import { ErrorBoundary } from './ErrorBoundary';
 import { Dashboard } from './Dashboard';
 import { CampaignManager } from './CampaignManager';
@@ -50,11 +51,14 @@ export function AdConsole() {
     <div className="app-layout">
       <a className="skip-link" href="#main-content">Skip to main content</a>
       <Topbar />
-      <main id="main-content" className="app-main" tabIndex={-1}>
-        <ErrorBoundary>
-          <div className="app-content">{renderView()}</div>
-        </ErrorBoundary>
-      </main>
+      <div className="app-body">
+        <Sidebar />
+        <main id="main-content" className="app-main" tabIndex={-1}>
+          <ErrorBoundary>
+            <div className="app-content">{renderView()}</div>
+          </ErrorBoundary>
+        </main>
+      </div>
     </div>
   );
 }
diff --git a/src/components/AdConsole/__tests__/contracts/sidebar.test.tsx b/src/components/AdConsole/__tests__/contracts/sidebar.test.tsx
new file mode 100644
index 0000000..2312608
--- /dev/null
+++ b/src/components/AdConsole/__tests__/contracts/sidebar.test.tsx
@@ -0,0 +1,88 @@
+/**
+ * Sidebar contract tests.
+ *
+ * Pins the fix for a real navigation gap: `Sidebar` (nav/consoleNav's
+ * `getLeftRail`/training rail) existed with full unit test coverage but was
+ * never mounted into `AdConsole`, so Missions/Reports/Bulk ops/Trainer/
+ * Integrity were unreachable from the desktop UI. This locks in that the
+ * sidebar renders and actually drives navigation for those views.
+ */
+import React from 'react';
+import { describe, it, expect, beforeEach } from 'vitest';
+import { render, fireEvent } from '@testing-library/react';
+import { SessionProvider } from 'next-auth/react';
+import { AdConsole } from '../../AdConsole';
+import { useAdConsoleStore } from '@/engine/ad-console/store';
+
+function resetStore() {
+  useAdConsoleStore.getState().resetAll();
+}
+
+function renderAdConsole() {
+  return render(
+    <SessionProvider session={null}>
+      <AdConsole />
+    </SessionProvider>,
+  );
+}
+
+describe('Sidebar contract — mounted and reachable', () => {
+  beforeEach(resetStore);
+
+  it('renders a nav.app-sidebar inside app-body', () => {
+    renderAdConsole();
+    const sidebar = document.querySelector('.app-body > nav.app-sidebar');
+    expect(sidebar).not.toBeNull();
+  });
+
+  it('shows the training rail (Missions, Reports, Bulk ops, Trainer, Integrity) when on a training view', () => {
+    useAdConsoleStore.getState().setView('drills');
+    renderAdConsole();
+
+    const sidebar = document.querySelector('.app-sidebar')!;
+    for (const label of ['Drills', 'Missions', 'Reports', 'Bulk ops', 'Trainer', 'Integrity']) {
+      const btn = Array.from(sidebar.querySelectorAll('button.sidebar-item')).find(
+        (b) => b.textContent?.trim() === label,
+      );
+      expect(btn, `sidebar item "${label}" must exist`).not.toBeUndefined();
+    }
+  });
+
+  it('clicking "Missions" in the sidebar navigates to the missions view', () => {
+    useAdConsoleStore.getState().setView('drills');
+    renderAdConsole();
+
+    const sidebar = document.querySelector('.app-sidebar')!;
+    const btn = Array.from(sidebar.querySelectorAll('button.sidebar-item')).find(
+      (b) => b.textContent?.trim() === 'Missions',
+    );
+    expect(btn).not.toBeUndefined();
+    fireEvent.click(btn!);
+
+    expect(useAdConsoleStore.getState().view).toBe('missions');
+  });
+
+  it('clicking "Reports" in the sidebar navigates to the reports view', () => {
+    useAdConsoleStore.getState().setView('trainer');
+    renderAdConsole();
+
+    const sidebar = document.querySelector('.app-sidebar')!;
+    const btn = Array.from(sidebar.querySelectorAll('button.sidebar-item')).find(
+      (b) => b.textContent?.trim() === 'Reports',
+    );
+    expect(btn).not.toBeUndefined();
+    fireEvent.click(btn!);
+
+    expect(useAdConsoleStore.getState().view).toBe('reports');
+  });
+
+  it('marks the active view\'s rail item with aria-current="page"', () => {
+    useAdConsoleStore.getState().setView('integrity');
+    renderAdConsole();
+
+    const sidebar = document.querySelector('.app-sidebar')!;
+    const active = sidebar.querySelector('button.sidebar-item[aria-current="page"]');
+    expect(active).not.toBeNull();
+    expect(active?.textContent?.trim()).toBe('Integrity');
+  });
+});
diff --git a/src/components/AdConsole/mobile/MobileNav.tsx b/src/components/AdConsole/mobile/MobileNav.tsx
index fd660ae..d64d174 100644
--- a/src/components/AdConsole/mobile/MobileNav.tsx
+++ b/src/components/AdConsole/mobile/MobileNav.tsx
@@ -3,12 +3,13 @@
 import { useEffect } from 'react';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
 import { useBreakpoint } from '@/lib/useBreakpoint';
-import { getLeftRail, type NavView } from '../nav/consoleNav';
+import { getLeftRail, isSidebarItemActive, resolveSidebarClick, sidebarSectionForView } from '../nav/consoleNav';
 
 const GROUP_TITLES: Record<string, string> = {
   campaigns: 'Campaign Manager',
   portfolios: 'Portfolios',
   measurement: 'Measurement',
+  training: 'Training',
 };
 
 export function MobileNav() {
@@ -19,11 +20,13 @@ export function MobileNav() {
   const toggleMobileMenu = useAdConsoleStore((s) => s.toggleMobileMenu);
   const closeMobileMenu = useAdConsoleStore((s) => s.closeMobileMenu);
   const view = useAdConsoleStore((s) => s.view);
+  const selectedTab = useAdConsoleStore((s) => s.state.selectedTab);
   const setView = useAdConsoleStore((s) => s.setView);
+  const setTab = useAdConsoleStore((s) => s.setTab);
   const runSimulation = useAdConsoleStore((s) => s.runSimulation);
   const resetAll = useAdConsoleStore((s) => s.resetAll);
 
-  const section: NavView = view === 'portfolio' ? 'portfolio' : 'campaigns';
+  const section = sidebarSectionForView(view);
   const items = getLeftRail(section);
 
   const groups: Record<string, typeof items> = {};
@@ -79,9 +82,12 @@ export function MobileNav() {
             {groupItems.map((item) => (
               <button
                 key={item.label}
-                className={`mobile-drawer-item ${view === item.view && group === 'campaigns' ? 'active' : ''}`}
+                className={`mobile-drawer-item ${isSidebarItemActive(item, view, selectedTab) ? 'active' : ''}`}
                 onClick={() => {
-                  setView(item.view);
+                  const action = resolveSidebarClick(item, view);
+                  if (action.type === 'setTab') setTab(action.tab!);
+                  else if (action.type === 'setTabAndView') { setTab(action.tab!); setView(action.view!); }
+                  else setView(action.view!);
                   closeMobileMenu();
                 }}
               >

From 3ca31253ad9fe2a9a3e83f568b7950fa1157db50 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 14:15:35 +0000
Subject: [PATCH 13/20] Fix adjustTargetBid throwing instead of flooring on
 decrement
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Copilot flagged that setTargetBid's new fail-fast MIN_BID check
(previous commit) has a real regression: adjustTargetBid — the
"-10%"/"+10%" bid-adjustment buttons — calls setTargetBid with a
multiplied value it doesn't control, so decrementing a target already
near the floor now throws an uncaught ValidationError instead of the
old clamping behavior. Unlike an explicit "Set bid" action,
adjustTargetBid is a relative nudge, so it should floor at MIN_BID
rather than fail fast.
---
 src/engine/ad-console/core/__tests__/engine.test.ts | 6 ++++++
 src/engine/ad-console/core/engine/target.ts         | 7 +++++--
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/engine/ad-console/core/__tests__/engine.test.ts b/src/engine/ad-console/core/__tests__/engine.test.ts
index 6203f0a..c507a5c 100644
--- a/src/engine/ad-console/core/__tests__/engine.test.ts
+++ b/src/engine/ad-console/core/__tests__/engine.test.ts
@@ -195,6 +195,12 @@ describe('target operations', () => {
     expect(adjustTargetBid(c, 'T1', 1.5).targets[0]!.bid).toBe(1.5);
   });
 
+  it('floors a decrement at the platform minimum instead of throwing (the "-10%" button on an already-cheap bid)', () => {
+    const c = makeCampaign({ targets: [{ id: 'T1', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'kw', match: 'Exact', bid: 0.02, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 }] });
+    expect(() => adjustTargetBid(c, 'T1', 0.9)).not.toThrow();
+    expect(adjustTargetBid(c, 'T1', 0.9).targets[0]!.bid).toBe(0.02);
+  });
+
   it('fails fast on a NaN bid instead of silently storing NaN', () => {
     const c = makeCampaign({ targets: [{ id: 'T1', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'kw', match: 'Exact', bid: 0.75, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 }] });
     expect(() => setTargetBid(c, 'T1', NaN)).toThrow();
diff --git a/src/engine/ad-console/core/engine/target.ts b/src/engine/ad-console/core/engine/target.ts
index 5183e88..39478a3 100644
--- a/src/engine/ad-console/core/engine/target.ts
+++ b/src/engine/ad-console/core/engine/target.ts
@@ -5,7 +5,7 @@
 import type {
   Campaign, CampaignStatus, MatchType, Target, TargetType
 } from '../types';
-import { assertNonEmpty, assertFiniteNonNegative, assertValidBid, ValidationError } from '../../../../lib/validation';
+import { assertNonEmpty, assertFiniteNonNegative, assertValidBid, MIN_BID, ValidationError } from '../../../../lib/validation';
 import { generateId } from './id';
 
 export interface AddTargetOptions {
@@ -155,7 +155,10 @@ export function setTargetBid(c: Campaign, targetId: string, newBid: number): Cam
 export function adjustTargetBid(c: Campaign, targetId: string, multiplier: number): Campaign {
   const t = c.targets.find((x) => x.id === targetId);
   if (!t) return c;
-  return setTargetBid(c, targetId, t.bid * multiplier);
+  // A relative nudge (e.g. the "-10%" button), not an explicit "set to X" —
+  // floor at the platform minimum instead of throwing when decrementing an
+  // already-cheap bid.
+  return setTargetBid(c, targetId, Math.max(MIN_BID, t.bid * multiplier));
 }
 
 export function pauseTarget(c: Campaign, targetId: string): Campaign {

From 15d162e2f8dd8339bc14fabcab27e0fee20cee03 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 14:16:49 +0000
Subject: [PATCH 14/20] Add engine test coverage requested in review

CodeRabbit flagged several changed engine functions without direct
regression tests, per CLAUDE.md's "changes under src/engine/ must
include tests" convention:

- isVideoFormat: SB/SD video-string matches and the SP/undefined
  fallback.
- Shared generateId migration (profiles/reports/trainer/integrity
  engines): each creation path now has a same-millisecond uniqueness
  regression test, mirroring the one already added for
  launchCampaign.
- simulateDays' Set-based search-term dedup: added a test covering
  duplicates across repeated simulateDays calls, complementing the
  existing "does not add duplicate search terms" test which already
  covered duplicates within a single run from overlapping targets.
---
 .../ad-console/core/__tests__/engine.test.ts  | 26 +++++++++++++++++++
 .../core/__tests__/simulation.test.ts         | 15 +++++++++++
 .../integrity/__tests__/engine.test.ts        | 22 +++++++++++++++-
 .../profiles/__tests__/engine.test.ts         | 13 +++++++++-
 .../features/reports/__tests__/engine.test.ts | 15 ++++++++++-
 .../features/trainer/__tests__/engine.test.ts | 13 +++++++++-
 6 files changed, 100 insertions(+), 4 deletions(-)

diff --git a/src/engine/ad-console/core/__tests__/engine.test.ts b/src/engine/ad-console/core/__tests__/engine.test.ts
index c507a5c..1501c60 100644
--- a/src/engine/ad-console/core/__tests__/engine.test.ts
+++ b/src/engine/ad-console/core/__tests__/engine.test.ts
@@ -27,6 +27,7 @@ import {
   campaignById,
   totalMetrics,
   normalizeCampaign,
+  isVideoFormat,
 } from '../engine';
 import type { Campaign, CampaignType, CampaignStatus, Metrics, Target, SearchTerm, Negative } from '../types';
 
@@ -340,4 +341,29 @@ describe('campaign creation flow', () => {
     expect(campaign.targets[0].value).toBe('coffee filter');
     expect(campaign.targets[1].value).toBe('coffee maker');
   });
+});
+
+describe('isVideoFormat', () => {
+  it('is true for an SB campaign with adFormat "Video"', () => {
+    expect(isVideoFormat('SB', 'Video')).toBe(true);
+  });
+
+  it('is false for an SB campaign with any other adFormat', () => {
+    expect(isVideoFormat('SB', 'Product collection')).toBe(false);
+    expect(isVideoFormat('SB', undefined)).toBe(false);
+  });
+
+  it('is true for an SD campaign with adFormat "Video creative"', () => {
+    expect(isVideoFormat('SD', 'Video creative')).toBe(true);
+  });
+
+  it('is false for an SD campaign with any other adFormat', () => {
+    expect(isVideoFormat('SD', 'Video')).toBe(false);
+    expect(isVideoFormat('SD', undefined)).toBe(false);
+  });
+
+  it('is false for SP regardless of adFormat', () => {
+    expect(isVideoFormat('SP', 'Video')).toBe(false);
+    expect(isVideoFormat('SP', undefined)).toBe(false);
+  });
 });
\ No newline at end of file
diff --git a/src/engine/ad-console/core/__tests__/simulation.test.ts b/src/engine/ad-console/core/__tests__/simulation.test.ts
index 6a2a04b..9c2135e 100644
--- a/src/engine/ad-console/core/__tests__/simulation.test.ts
+++ b/src/engine/ad-console/core/__tests__/simulation.test.ts
@@ -171,6 +171,21 @@ describe('simulateDays', () => {
     expect(terms.length).toBe(unique.size);
   });
 
+  it('does not re-add duplicate search terms across repeated simulateDays runs', () => {
+    const c = makeCampaign({
+      targets: [
+        { id: 'T1', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'coffee', match: 'Broad', bid: 1, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
+      ],
+    });
+    const [firstRun] = simulateDays([c], 7);
+    const [secondRun] = simulateDays([firstRun], 7);
+    const terms = secondRun.searchTerms.map(st => st.term);
+    const unique = new Set(terms);
+    expect(terms.length).toBe(unique.size);
+    // The second run should carry forward the first run's terms, not duplicate them.
+    expect(secondRun.searchTerms.length).toBeGreaterThanOrEqual(firstRun.searchTerms.length);
+  });
+
   it('fails fast on a negative days value instead of corrupting metrics', () => {
     const c = makeCampaign();
     expect(() => simulateDays([c], -7)).toThrow();
diff --git a/src/engine/ad-console/features/integrity/__tests__/engine.test.ts b/src/engine/ad-console/features/integrity/__tests__/engine.test.ts
index 3b4955a..8375d1f 100644
--- a/src/engine/ad-console/features/integrity/__tests__/engine.test.ts
+++ b/src/engine/ad-console/features/integrity/__tests__/engine.test.ts
@@ -1,4 +1,4 @@
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { runIntegrityCheck } from '../engine';
 import type { Campaign } from '../../../core/types';
 
@@ -114,4 +114,24 @@ describe('runIntegrityCheck', () => {
   it('fails fast when campaigns is not an array', () => {
     expect(() => runIntegrityCheck(null as unknown as Campaign[])).toThrow();
   });
+
+  it('assigns unique issue ids even when generated within the same millisecond', () => {
+    const nowSpy = vi.spyOn(Date, 'now').mockReturnValue(1700000000000);
+    try {
+      const report = runIntegrityCheck([
+        baseCampaign({
+          status: 'Archived',
+          targets: [
+            { id: 'T1', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'a', match: 'Exact', bid: 0.75, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
+            { id: 'T2', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'b', match: 'Exact', bid: 0.75, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
+            { id: 'T3', campaignId: 'C1', adGroupId: 'AG1', type: 'Keyword', value: 'c', match: 'Exact', bid: 0.75, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
+          ],
+        }),
+      ]);
+      const ids = report.issues.map((i) => i.id);
+      expect(new Set(ids).size).toBe(ids.length);
+    } finally {
+      nowSpy.mockRestore();
+    }
+  });
 });
diff --git a/src/engine/ad-console/features/profiles/__tests__/engine.test.ts b/src/engine/ad-console/features/profiles/__tests__/engine.test.ts
index d6fb2b2..92ee038 100644
--- a/src/engine/ad-console/features/profiles/__tests__/engine.test.ts
+++ b/src/engine/ad-console/features/profiles/__tests__/engine.test.ts
@@ -1,4 +1,4 @@
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import {
   createProfile,
   switchProfile,
@@ -17,6 +17,17 @@ describe('createProfile', () => {
   it('falls back to a default name when blank', () => {
     expect(createProfile('   ').name).toBe('Trainee');
   });
+
+  it('assigns unique ids even when created within the same millisecond', () => {
+    const nowSpy = vi.spyOn(Date, 'now').mockReturnValue(1700000000000);
+    try {
+      const a = createProfile('Ana');
+      const b = createProfile('Ben');
+      expect(a.id).not.toBe(b.id);
+    } finally {
+      nowSpy.mockRestore();
+    }
+  });
 });
 
 describe('profile collection ops', () => {
diff --git a/src/engine/ad-console/features/reports/__tests__/engine.test.ts b/src/engine/ad-console/features/reports/__tests__/engine.test.ts
index e39806e..5c1941d 100644
--- a/src/engine/ad-console/features/reports/__tests__/engine.test.ts
+++ b/src/engine/ad-console/features/reports/__tests__/engine.test.ts
@@ -1,4 +1,4 @@
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { createReportRequest, generateReport, reportToCsv } from '../engine';
 import type { ReportType } from '../types';
 
@@ -14,6 +14,19 @@ describe('createReportRequest', () => {
   it('fails fast on unknown report type', () => {
     expect(() => createReportRequest('bogus' as ReportType)).toThrow();
   });
+
+  it('assigns unique ids even when created within the same millisecond, including across createReportRequest and generateReport', () => {
+    const nowSpy = vi.spyOn(Date, 'now').mockReturnValue(1700000000000);
+    try {
+      const reqA = createReportRequest('campaign');
+      const reqB = createReportRequest('target');
+      const report = generateReport('campaign');
+      const ids = [reqA.id, reqB.id, report.id];
+      expect(new Set(ids).size).toBe(ids.length);
+    } finally {
+      nowSpy.mockRestore();
+    }
+  });
 });
 
 describe('generateReport', () => {
diff --git a/src/engine/ad-console/features/trainer/__tests__/engine.test.ts b/src/engine/ad-console/features/trainer/__tests__/engine.test.ts
index 2438e2b..a36091b 100644
--- a/src/engine/ad-console/features/trainer/__tests__/engine.test.ts
+++ b/src/engine/ad-console/features/trainer/__tests__/engine.test.ts
@@ -1,4 +1,4 @@
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { addNote, calculateCertScore, calculateGrade } from '../engine';
 
 describe('addNote', () => {
@@ -12,6 +12,17 @@ describe('addNote', () => {
   it('fails fast on empty note text', () => {
     expect(() => addNote('   ')).toThrow();
   });
+
+  it('assigns unique ids even when created within the same millisecond', () => {
+    const nowSpy = vi.spyOn(Date, 'now').mockReturnValue(1700000000000);
+    try {
+      const a = addNote('first');
+      const b = addNote('second');
+      expect(a.id).not.toBe(b.id);
+    } finally {
+      nowSpy.mockRestore();
+    }
+  });
 });
 
 describe('calculateCertScore', () => {

From 8c2241e38c7416a238420786d09d81c97a585d7a Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 14:17:21 +0000
Subject: [PATCH 15/20] Address review nits: drop dead import, use EmptyState
 for no-data report

- reports/engine.ts imported assertNonEmpty but never used it
  (pre-existing, unrelated to this PR's changes; Copilot flagged it).
- ReportsPage's "no data yet" fallback used a bare Card instead of the
  EmptyState component already used the same way elsewhere
  (ManagerCampaignsTab, ManagerSearchTermsTab, HistoryTab).
---
 src/components/AdConsole/features/reports/ReportsPage.tsx | 5 ++---
 src/engine/ad-console/features/reports/engine.ts          | 2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/components/AdConsole/features/reports/ReportsPage.tsx b/src/components/AdConsole/features/reports/ReportsPage.tsx
index 0060569..6224375 100644
--- a/src/components/AdConsole/features/reports/ReportsPage.tsx
+++ b/src/components/AdConsole/features/reports/ReportsPage.tsx
@@ -6,6 +6,7 @@ import { Card } from '@astryxdesign/core/Card';
 import { CheckCircle, XCircle } from '@phosphor-icons/react';
 import { useAdConsoleStore } from '@/engine/ad-console/store';
 import { calc, formatMoney, formatWhole, formatPercent } from '@/engine/ad-console/core/engine';
+import { EmptyState } from '../../details/EmptyState';
 
 export function ReportsPage() {
   const requests = useAdConsoleStore((s) => s.reportQueue);
@@ -94,9 +95,7 @@ export function ReportsPage() {
                 </tbody>
               </Table>
           ) : (
-            <Card variant="default" padding={6}>
-              <p className="muted">This report type has no data yet.</p>
-            </Card>
+            <EmptyState icon="chart" title="No data yet" message="This report type has no data yet." />
           )}
         </div>
       )}
diff --git a/src/engine/ad-console/features/reports/engine.ts b/src/engine/ad-console/features/reports/engine.ts
index c979451..a8c355c 100644
--- a/src/engine/ad-console/features/reports/engine.ts
+++ b/src/engine/ad-console/features/reports/engine.ts
@@ -2,7 +2,7 @@
  * Reports — pure engine.
  */
 import type { Report, ReportRequest, ReportType } from './types';
-import { assertNonEmpty, ValidationError } from '../../../../lib/validation';
+import { ValidationError } from '../../../../lib/validation';
 import { generateId } from '../../core/engine/id';
 
 const REPORT_TYPES: ReportType[] = ['campaign', 'adGroup', 'target', 'searchTerm', 'placement'];

From e411d362c1d27899538466b0c660ff156dc8d2e5 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 14:46:01 +0000
Subject: [PATCH 16/20] Fix sitewide Astryx Card padding zeroed out by the
 global CSS reset
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Root cause: Astryx ships its component styles (including the padding
generated from a Card's `padding` prop) inside `@layer astryx-base`/
`@layer astryx-theme`. Per the CSS cascade-layers spec, an unlayered
declaration always wins over a layered one regardless of specificity.
This repo's global reset (`*, *::before, *::after { ...; padding: 0; }`)
is unlayered, so it was unconditionally beating every Astryx `padding`
prop across the entire app — every Card rendered with 0 padding no
matter what value was passed, which is why content (e.g. the "Remove"
button on wizard product cards, headings on dashboard cards) sat
flush against card edges instead of respecting their intended spacing.

Confirmed via computed-style inspection: a Card with `padding={6}`
resolved to `padding: 0px` before this fix and `padding: 23px`
(24px token minus the 1px border inset) after.

Scoped the padding reset down to the two native elements that actually
relied on it (`ul`, `ol`) instead of zeroing it on every element.
---
 src/app/globals.css | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/app/globals.css b/src/app/globals.css
index 3f21f7d..4a6a3cc 100644
--- a/src/app/globals.css
+++ b/src/app/globals.css
@@ -149,8 +149,16 @@
   color-scheme: light;
 }
 
-/* Reset */
-*, *::before, *::after { box-sizing: border-box; margin: 0; padding: 0; }
+/* Reset
+ * NOTE: padding is intentionally NOT zeroed on `*`. Astryx components ship
+ * their own padding as StyleX classes inside `@layer astryx-base`/
+ * `@layer astryx-theme`; per the CSS cascade-layers spec, ANY unlayered
+ * declaration beats a layered one regardless of specificity. A blanket
+ * `* { padding: 0 }` here (unlayered) was silently zeroing out every
+ * Astryx `padding` prop (Card, etc.) sitewide. Reset padding only on the
+ * couple of native elements that actually need it. */
+*, *::before, *::after { box-sizing: border-box; margin: 0; }
+ul, ol { padding: 0; }
 
 html {
   font-family: var(--font-body);

From 37aaf54bfc8ae633e2112e689eff5c27d468a5fe Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 14:46:25 +0000
Subject: [PATCH 17/20] Fix wizard review step showing "Lookback" for
 non-audience campaigns
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

makeDraft() defaults audienceLookback to '30' regardless of campaign
type, so Step6ReviewLaunch's `{d.audienceLookback && ...}` check was
truthy from the moment the wizard opens — every campaign's review
screen showed "Lookback: 30 days", including plain Sponsored Products
campaigns where the concept doesn't exist. Gated it on
`targetingMode.includes('Audiences')` instead, matching the same
condition the SB/SD targeting steps already use to show the lookback
selector in the first place.

Also filled in review-step gaps found during the same audit: ASIN
targets, category targets, and audience targets entered in step 4 had
no summary row at all, and SB/SD creative fields (headline, brand,
destination) set in step 3 weren't shown anywhere before launch.
---
 .../AdConsole/wizard/Step6ReviewLaunch.tsx    |  8 +-
 .../__tests__/Step6ReviewLaunch.test.tsx      | 78 +++++++++++++++++++
 2 files changed, 85 insertions(+), 1 deletion(-)
 create mode 100644 src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx

diff --git a/src/components/AdConsole/wizard/Step6ReviewLaunch.tsx b/src/components/AdConsole/wizard/Step6ReviewLaunch.tsx
index 12fc52c..4fb31a5 100644
--- a/src/components/AdConsole/wizard/Step6ReviewLaunch.tsx
+++ b/src/components/AdConsole/wizard/Step6ReviewLaunch.tsx
@@ -38,7 +38,13 @@ export function Step6ReviewLaunch({ isActive, isComplete }: Step6ReviewLaunchPro
           {d.exactKeywords && <div className="review-row"><span>Exact keywords</span><strong>{d.exactKeywords.split('\n').filter(Boolean).length} entered</strong></div>}
           {d.phraseKeywords && <div className="review-row"><span>Phrase keywords</span><strong>{d.phraseKeywords.split('\n').filter(Boolean).length} entered</strong></div>}
           {d.broadKeywords && <div className="review-row"><span>Broad keywords</span><strong>{d.broadKeywords.split('\n').filter(Boolean).length} entered</strong></div>}
-          {d.audienceLookback && <div className="review-row"><span>Lookback</span><strong>{d.audienceLookback} days</strong></div>}
+          {d.asinTargets && <div className="review-row"><span>ASIN targets</span><strong>{d.asinTargets.split('\n').filter(Boolean).length} entered</strong></div>}
+          {d.categoryTargets && <div className="review-row"><span>Category targets</span><strong>{d.categoryTargets.split('\n').filter(Boolean).length} entered</strong></div>}
+          {d.targetingMode.includes('Audiences') && d.audienceTargets && <div className="review-row"><span>Audience targets</span><strong>{d.audienceTargets.split('\n').filter(Boolean).length} entered</strong></div>}
+          {d.targetingMode.includes('Audiences') && <div className="review-row"><span>Lookback</span><strong>{d.audienceLookback} days</strong></div>}
+          {d.creative.headline && <div className="review-row"><span>Headline</span><strong>{d.creative.headline}</strong></div>}
+          {d.creative.brandName && <div className="review-row"><span>Brand</span><strong>{d.creative.brandName}</strong></div>}
+          {d.creative.destination && <div className="review-row"><span>Destination</span><strong>{d.creative.destination}</strong></div>}
         </div>
         {!d.name.trim() && <div className="coach-tip" style={{ marginTop: 10 }}>Campaign name is required before launch.</div>}
       </Card>
diff --git a/src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx b/src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx
new file mode 100644
index 0000000..91f94f3
--- /dev/null
+++ b/src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx
@@ -0,0 +1,78 @@
+/**
+ * Step6ReviewLaunch pins a real bug found during a wizard walkthrough:
+ * `audienceLookback` defaults to '30' in makeDraft() regardless of
+ * campaign type, so the old `{d.audienceLookback && ...}` condition
+ * showed a "Lookback" row on every campaign's review step — including
+ * Sponsored Products, which has no such concept. The Lookback row should
+ * only appear when targetingMode is actually an audience mode.
+ */
+import { describe, it, expect, beforeEach } from 'vitest';
+import { render, screen } from '@testing-library/react';
+import { Step6ReviewLaunch } from '../Step6ReviewLaunch';
+import { useAdConsoleStore } from '@/engine/ad-console/store';
+
+function resetStore() {
+  useAdConsoleStore.getState().resetAll();
+  useAdConsoleStore.getState().resetDraft();
+}
+
+describe('Step6ReviewLaunch — Lookback row', () => {
+  beforeEach(resetStore);
+
+  it('does not show Lookback for a default (non-audience) SP draft', () => {
+    render(<Step6ReviewLaunch isActive isComplete={false} />);
+    expect(screen.queryByText('Lookback')).toBeNull();
+  });
+
+  it('shows Lookback once targetingMode is an audience mode', () => {
+    useAdConsoleStore.getState().updateDraft('type', 'SD');
+    useAdConsoleStore.getState().updateDraft('targetingMode', 'Audiences - views remarketing');
+    useAdConsoleStore.getState().updateDraft('audienceLookback', '60');
+    render(<Step6ReviewLaunch isActive isComplete={false} />);
+    expect(screen.getByText('Lookback')).toBeTruthy();
+    expect(screen.getByText('60 days')).toBeTruthy();
+  });
+
+  it('hides Lookback again if targetingMode is switched back to a non-audience mode', () => {
+    useAdConsoleStore.getState().updateDraft('type', 'SB');
+    useAdConsoleStore.getState().updateDraft('targetingMode', 'Keyword');
+    render(<Step6ReviewLaunch isActive isComplete={false} />);
+    expect(screen.queryByText('Lookback')).toBeNull();
+  });
+});
+
+describe('Step6ReviewLaunch — targeting/creative summary rows', () => {
+  beforeEach(resetStore);
+
+  it('shows ASIN and category target counts when entered', () => {
+    useAdConsoleStore.getState().updateDraft('asinTargets', 'B0ABC123\nB0DEF456');
+    useAdConsoleStore.getState().updateDraft('categoryTargets', 'Drinkware');
+    render(<Step6ReviewLaunch isActive isComplete={false} />);
+    expect(screen.getByText('ASIN targets')).toBeTruthy();
+    expect(screen.getByText('2 entered')).toBeTruthy();
+    expect(screen.getByText('Category targets')).toBeTruthy();
+    expect(screen.getByText('1 entered')).toBeTruthy();
+  });
+
+  it('shows the SB/SD creative headline, brand, and destination when set', () => {
+    useAdConsoleStore.getState().updateDraft('creative', {
+      headline: 'Discover your perfect brew',
+      brandName: 'Acme Coffee',
+      destination: 'Brand Store',
+    });
+    render(<Step6ReviewLaunch isActive isComplete={false} />);
+    expect(screen.getByText('Headline')).toBeTruthy();
+    expect(screen.getByText('Discover your perfect brew')).toBeTruthy();
+    expect(screen.getByText('Brand')).toBeTruthy();
+    expect(screen.getByText('Acme Coffee')).toBeTruthy();
+    expect(screen.getByText('Destination')).toBeTruthy();
+    expect(screen.getByText('Brand Store')).toBeTruthy();
+  });
+
+  it('does not show creative rows for a fresh SP draft with no creative set', () => {
+    render(<Step6ReviewLaunch isActive isComplete={false} />);
+    expect(screen.queryByText('Headline')).toBeNull();
+    expect(screen.queryByText('Brand')).toBeNull();
+    expect(screen.queryByText('Destination')).toBeNull();
+  });
+});

From 22058f4aa39936ed4f2799fd90a2a74b90ead9be Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 14:58:01 +0000
Subject: [PATCH 18/20] Bump version to 3.6.0
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Reflects this release's changes: sidebar navigation wiring, the
sitewide Astryx Card padding fix, wizard review-step fixes, and the
bid-validation/ID-generation/UI-consistency work — see CHANGELOG.md.
Synced package-lock.json's duplicate version fields and the unused
coreState.version display field to match.
---
 package-lock.json                         | 4 ++--
 package.json                              | 2 +-
 src/engine/ad-console/core/slices/core.ts | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index 052a641..b9a0e56 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "amazon-ad-console",
-  "version": "3.5.0",
+  "version": "3.6.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "amazon-ad-console",
-      "version": "3.5.0",
+      "version": "3.6.0",
       "hasInstallScript": true,
       "dependencies": {
         "@astryxdesign/core": "^0.1.8",
diff --git a/package.json b/package.json
index 671f052..ae5c609 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "amazon-ad-console",
-  "version": "3.5.0",
+  "version": "3.6.0",
   "private": true,
   "description": "Amazon PPC Training Simulator — Amazon Ads Console replica for VA training",
   "scripts": {
diff --git a/src/engine/ad-console/core/slices/core.ts b/src/engine/ad-console/core/slices/core.ts
index 2982208..35dd732 100644
--- a/src/engine/ad-console/core/slices/core.ts
+++ b/src/engine/ad-console/core/slices/core.ts
@@ -36,7 +36,7 @@ export interface CoreSlice {
 }
 
 const coreState: AdConsoleState = {
-  version: '3.6',
+  version: '3.6.0',
   campaigns: defaultCampaigns(),
   filter: { type: 'All', status: 'All', portfolio: 'All', search: '' },
   selectedCampaignId: null,

From 2830a2ffe66bdbd06cb58e75e4ce30ee35f69a84 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 14:58:14 +0000
Subject: [PATCH 19/20] Document 3.6.0 changes and the CSS cascade-layers
 gotcha
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add CHANGELOG.md (new — this repo had no changelog convention before),
  covering every fix/change in this release.
- CLAUDE.md: document MIN_BID/assertValidBid, the isVideoFormat helper,
  the EmptyState component convention, and — most importantly — the
  CSS cascade-layers rule that caused the sitewide Card padding bug, so
  it doesn't get silently reintroduced by a future "helpful" reset rule.
- docs/FEATURES.md: the mobile/responsive breakpoint table described a
  "sidebar collapses to 200px" tablet behavior that was never actually
  implemented (and now that the sidebar is wired up, would be
  misleading) — corrected to match what the code actually does: hidden
  below 1100px, hamburger drawer takes over for both mobile and tablet.
---
 CHANGELOG.md     | 86 ++++++++++++++++++++++++++++++++++++++++++++++++
 CLAUDE.md        |  9 +++--
 docs/FEATURES.md |  8 ++---
 3 files changed, 96 insertions(+), 7 deletions(-)
 create mode 100644 CHANGELOG.md

diff --git a/CHANGELOG.md b/CHANGELOG.md
new file mode 100644
index 0000000..8c76c32
--- /dev/null
+++ b/CHANGELOG.md
@@ -0,0 +1,86 @@
+# Changelog
+
+All notable changes to this project are documented in this file.
+
+The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
+This file starts at 3.6.0 — earlier releases were not retroactively documented.
+
+## [3.6.0] - 2026-08-03
+
+### Fixed
+
+- **Sidebar navigation was never mounted.** The `Sidebar` component and its
+  `getLeftRail`/training-rail nav model (`nav/consoleNav.ts`) were fully
+  built and unit-tested but never rendered into `AdConsole.tsx` — Missions,
+  Reports, Bulk ops, Trainer, and Integrity were unreachable from the
+  desktop UI. `MobileNav` had the same gap from a different angle (its
+  section resolution only ever returned `campaigns`/`portfolio`). Both are
+  now wired through the shared `sidebarSectionForView`/`isSidebarItemActive`/
+  `resolveSidebarClick` helpers.
+- **Every Astryx `Card`'s `padding` prop silently rendered as `0px`
+  sitewide.** Root cause: Astryx ships component styles inside
+  `@layer astryx-base`/`@layer astryx-theme`, and per the CSS
+  cascade-layers spec an unlayered declaration always beats a layered one
+  regardless of specificity. This app's global reset
+  (`*, *::before, *::after { ...; padding: 0; }`) was unlayered, so it
+  unconditionally zeroed every Astryx padding prop — this is why content
+  (buttons, headings, form fields) so often sat flush against card edges.
+  Scoped the reset down to just `ul`/`ol` (the only elements relying on it).
+- Campaign creation wizard's Review & Launch step showed a "Lookback: 30
+  days" row on every campaign, including plain Sponsored Products, because
+  `audienceLookback` defaults to `'30'` regardless of type. Gated it on the
+  campaign's targeting mode actually being an audience mode. Also added
+  missing review rows for ASIN/category/audience targets and SB/SD creative
+  fields (headline, brand, destination) that were entered earlier in the
+  wizard but never shown before launch.
+- `.split`'s `2fr 1fr` grid (dashboard "Operator alerts"/"Training
+  coverage" cards) didn't shrink to fit once the sidebar took up real
+  width, clipping content past the viewport edge. Fixed with
+  `minmax(0, ...)` tracks.
+- `adjustTargetBid` (the "-10%"/"+10%" bid buttons) threw an uncaught
+  error when decrementing an already-cheap bid below the platform
+  minimum, instead of flooring it — a regression from the bid
+  fail-fast change below.
+- `setTargetBid`/`setAdGroupDefaultBid` no longer silently substitute a
+  bid below the real $0.02 minimum; they fail fast via a new
+  `assertValidBid`/`MIN_BID` (`src/lib/validation.ts`), matching this
+  codebase's existing fail-fast convention. Creation/normalization paths
+  (`addTarget`, `normalizeCampaign`) still clamp, since those fill in
+  defaults for incomplete data rather than acting on explicit user intent.
+- Campaign Manager's empty state no longer says "No campaigns yet" when a
+  search/filter simply matched nothing — it now shows a distinct "no
+  matches" state with a "Clear filters" action.
+- Fixed a campaign-ID collision risk in `launchCampaign` (two campaigns of
+  the same type launched within the same millisecond could get the same
+  ID) by switching to the shared `generateId` helper, and deduplicated the
+  same ad-hoc ID-generation pattern across the `profiles`/`reports`/
+  `trainer`/`integrity` feature engines.
+- Fixed literal mojibake (`—`, `→`) in the landing page copy and a
+  missing `.object-cover` rule that left two landing-page images without
+  `object-fit` applied.
+- Replaced dead Tailwind sizing classes (`w-6 h-6`, `w-4 h-4`) on landing
+  page icons with explicit SVG dimensions — this repo has no Tailwind
+  compiler wired up, so the classes were doing nothing.
+
+### Added
+
+- Contract tests pinning the sidebar fix (renders, drives navigation to
+  the previously-unreachable views) and the wizard review-step fix.
+- Regression tests for `isVideoFormat`, the shared `generateId` migration
+  (same-millisecond uniqueness across all four feature engines), and the
+  simulation's search-term dedup across repeated `simulateDays` calls.
+
+### Changed
+
+- Deduplicated hand-rolled metrics/formatter logic in `PortfolioOverview`,
+  `Dashboard`, and `CampaignManager` onto the shared `totalMetrics`/
+  `formatMoney`/`formatWhole`/`formatPercent` engine functions.
+- Un-Card-wrapped dense tables in `PortfolioOverview`, `Dashboard`,
+  `OverviewTab`, `BulkOpsPage`, and `ReportsPage` per this repo's own
+  convention (dense data renders edge-to-edge, `Card` is for dashboard
+  widgets/settings groups only).
+- Fixed an O(n·m) duplicate-detection loop in the search-term simulator
+  (now O(1) via a `Set`) and memoized a few expensive per-render
+  aggregations (`Dashboard`, `ManagerSearchTermsTab`).
+- Removed 13 dead `useState` hooks and unused imports from
+  `CreateCampaignWizard`.
diff --git a/CLAUDE.md b/CLAUDE.md
index c6520c5..04ae677 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -52,7 +52,7 @@ Core engine (src/engine/ad-console/core/) — zero framework dependencies, pure
 **`core/engine/`, `core/types.ts`, and `core/simulation.ts` have zero React/Next/Zustand dependencies.** They are pure TypeScript: given state in, return new state out, no mutation, no side effects. This is the most important invariant in the codebase — it's what makes those modules portable and unit-testable in isolation. Never import React, Next.js, or store code into them. Note that `core/slices/` (below) is the one exception within `core/` — it depends on Zustand's `StateCreator` type by design, since its job is to wrap the pure engine in store slices.
 
 - `core/types.ts` — every domain interface (Campaign, AdGroup, Target, Negative, BudgetRule, Portfolio, Metrics, etc.)
-- `core/engine/` — one module per domain concern: `campaign.ts`, `target.ts`, `adgroup.ts`, `negative.ts`, `budget.ts`, `portfolio.ts`, `draft.ts`, `id.ts`, `metrics.ts`, `responsive.ts`, `search-term-generator.ts`. All re-exported through `core/engine/index.ts`.
+- `core/engine/` — one module per domain concern: `campaign.ts`, `target.ts`, `adgroup.ts`, `negative.ts`, `budget.ts`, `portfolio.ts`, `draft.ts`, `id.ts`, `metrics.ts`, `responsive.ts`, `search-term-generator.ts`. All re-exported through `core/engine/index.ts`. `campaign.ts` also exports `isVideoFormat(type, adFormat)`, the single source of truth for which `adFormat` string means "video" for a given campaign type (SB uses `'Video'`, SD uses `'Video creative'`) — used by both the engine and `OverviewTab`.
 - `core/simulation.ts` — the 7-day performance simulator; metrics cascade target → ad group → campaign → dashboard.
 - `core/slices/` — Zustand-dependent `StateCreator` slices (core, target, adgroup, negative, budget, portfolio, draft) that wrap the pure engine functions with state.
 - `features/<name>/` — self-contained modules (`drills`, `profiles`, `trainer`, `bulk`, `reports`, `missions`, `integrity`), each with its own `types.ts`, `engine.ts`, `store.ts`. Adding a feature means adding a new directory here — existing files shouldn't need edits (open/closed).
@@ -86,13 +86,15 @@ NextAuth v5 (beta), Credentials provider, JWT sessions, bcrypt password hashing.
 
 ### UI conventions (Astryx design system)
 Components come from `@astryxdesign/core` (153 components, theme via `@astryxdesign/theme-neutral`). This is actively used across the component tree (~40 files) — don't hand-roll layout `<div>`s or raw CSS when an Astryx component/prop/token covers it. Key rules (full detail lives in `AGENTS.md`'s Astryx block):
-- No raw `<div>` for layout — components handle layout/spacing (`AppShell` for full pages, `SideNav` for sidebar nav).
+- No raw `<div>` for layout — components handle layout/spacing (`AppShell` for full pages, `SideNav` for sidebar nav). In practice the component layer predates full Astryx adoption and still uses hand-rolled `.app-layout`/`.app-sidebar`/`.app-main` divs throughout (see "known SOLID violations" below) — match the existing pattern in a file rather than mixing conventions mid-component.
 - Dense data → `Table`/`List`/`Item` rows edge-to-edge, never Card-wrapped. `Card` is for dashboard widgets/galleries/settings groups only.
 - Styling values must be tokens (`var(--color-*|--spacing-*|--radius-*)`) — no raw hex/px, no Tailwind utility classes (this repo has no Tailwind compiler wired up despite Tailwind appearing in some older docs).
 - Discover components/props via the CLI: `npm run astryx -- component <Name>`, `npm run astryx -- search "<thing>"`, `npm run astryx -- build "<idea>"`.
+- **Never add `padding` (or any box-model property Astryx components expose as a prop) to a bare-selector reset in `globals.css`** (e.g. `*, *::before, *::after { ... }`). Astryx ships its component styles inside `@layer astryx-base`/`@layer astryx-theme`; per the CSS cascade-layers spec, *any* unlayered declaration beats a layered one regardless of specificity. An unlayered `* { padding: 0 }` silently zeroed every Astryx `padding` prop sitewide until it was found and fixed (3.6.0) — the global reset only zeroes `margin`, plus `padding` on the couple of native elements (`ul`, `ol`) that actually need it. If a future reset-like rule needs to beat Astryx's own styling, put it in the unlayered `src/app/astryx-theme.css` bridge scoped to the specific class/selector, not a wildcard.
+- Empty states use the shared `EmptyState` component (`src/components/AdConsole/details/EmptyState.tsx`) — icon + title + optional message, not a bare `Card` with a muted paragraph.
 
 ### Validation
-Engine functions fail fast: invalid input throws `ValidationError` (`src/lib/validation.ts`) rather than silently clamping or producing `NaN`. Follow this pattern for new engine functions — don't add silent fallbacks.
+Engine functions fail fast: invalid input throws `ValidationError` (`src/lib/validation.ts`) rather than silently clamping or producing `NaN`. Follow this pattern for new engine functions — don't add silent fallbacks. `MIN_BID` and `assertValidBid` (also in `src/lib/validation.ts`) enforce the $0.02 platform bid floor for "set an explicit bid" actions (`setTargetBid`, `setAdGroupDefaultBid`); creation/normalization paths (`addTarget`, `normalizeCampaign`) still clamp instead of throwing, since those fill in defaults for incomplete data rather than acting on explicit user intent. Relative adjustments (`adjustTargetBid`, the "±10%" buttons) floor at `MIN_BID` rather than fail fast, since the caller doesn't fully control the resulting value.
 
 ## Testing conventions
 
@@ -117,6 +119,7 @@ These come from `AGENTS.md`, `LOOP.md`, `loop-constraints.md`, and `gate.yaml` 
 - `legacy/` holds the pre-Next.js prototype (a single-file `amazon_ppc_simulator.html` with inline JS) and its old QA/docs — historical reference only, not part of the current build.
 - `codegraphs/Amazon-ad-console.md` describes that old single-file prototype and is stale relative to the current Next.js/engine architecture described above; don't rely on it.
 - `docs/` has deeper reference material: `ARCHITECTURE.md`, `API.md` (full engine function signatures), `SCHEMA.md`, `FEATURES.md`, `INTEGRATION.md` (porting guide), `AUTH.md`, `AUDIT-FOLLOWUPS.md`.
+- `CHANGELOG.md` (repo root) tracks notable changes per release starting at 3.6.0; bump `version` in `package.json` (and the unused-but-should-stay-in-sync `coreState.version` in `core/slices/core.ts`) together with a new entry when cutting a release.
 - `skills/`, `patterns/`, `gate.yaml`, `STATE.md`, `loop-*.md` support an autonomous triage/fix loop tool used against this repo — not part of the app runtime.
 
 ## Porting the engine
diff --git a/docs/FEATURES.md b/docs/FEATURES.md
index 70d3d76..e9fd5b7 100644
--- a/docs/FEATURES.md
+++ b/docs/FEATURES.md
@@ -405,13 +405,13 @@ When clicking a tab-mapped item:
 
 | Range | Label | Behavior |
 |-------|-------|----------|
-| < 768px | Mobile | Sidebar hidden, hamburger toggle + slide-out drawer |
-| 768–1100px | Tablet | Sidebar collapses to 200px, hamburger still available |
-| > 1100px | Desktop | Full Amazon Console layout |
+| < 768px | Mobile | Desktop sidebar hidden; hamburger toggle + slide-out drawer |
+| 768–1100px | Tablet | Same as mobile — desktop sidebar hidden, hamburger drawer takes over (both are `isMobileOrTablet` in `useBreakpoint`) |
+| > 1100px | Desktop | Full Amazon Console layout with the persistent left sidebar |
 
 ### Mobile Drawer
 - Hamburger button in the global nav toggles a slide-out drawer
-- Drawer contains all sidebar groups: Campaign Manager, Portfolios, Measurement
+- Drawer contains all sidebar groups for the active section: Campaign Manager, Portfolios, Measurement, or Training (Drills/Missions/Reports/Bulk ops/Trainer/Integrity)
 - Backdrop overlay with click-to-close
 - Escape key closes the drawer
 - Animation state machine: closed → open ↔ closing → closed

From 93ba85744f121a7623a34051223afb95173d0e84 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 3 Aug 2026 15:09:11 +0000
Subject: [PATCH 20/20] Add visible bid validation feedback in TargetsTab;
 strengthen two vacuous tests
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

CodeRabbit flagged that AdGroupsTab/TargetsTab silently do nothing when
a typed bid is below MIN_BID, with no visible error. Investigated both:

- AdGroupsTab uses Astryx's NumberInput with min={0.02} — it already
  shows real-time invalid styling (aria-invalid, red border, "Invalid
  number" live-region text) via the component's own built-in min
  validation, with no app code needed. Added a regression test proving
  this rather than adding redundant/dead validation logic.
- TargetsTab uses a raw native <input type="number">, which doesn't
  get that behavior for free — genuinely had no visible feedback.
  Added a red border, aria-invalid, and a title tooltip when the
  edited value is below MIN_BID.

Also fixed two review-flagged test gaps that could pass vacuously:
- The integrity same-millisecond-uniqueness test didn't assert the
  fixture actually produced more than one issue before checking
  uniqueness (an empty or single-item array trivially passes).
- The repeated-simulateDays dedup test didn't assert the first run
  produced any search terms, nor that the second run actually carried
  them forward — both now asserted explicitly.

Declined (with reasoning, per the PR's review thread): rewriting
Step6ReviewLaunch's review-row layout onto different Astryx layout
primitives — `review-row`/`review-box` is the same pattern already
used in OverviewTab.tsx; changing just one of the two would make them
inconsistent with each other, not more consistent.
---
 .../AdConsole/details/TargetsTab.tsx          |  7 +-
 .../bid-validation-feedback.test.tsx          | 71 +++++++++++++++++++
 .../__tests__/Step6ReviewLaunch.test.tsx      |  3 +-
 .../core/__tests__/simulation.test.ts         |  3 +
 .../integrity/__tests__/engine.test.ts        |  1 +
 5 files changed, 83 insertions(+), 2 deletions(-)
 create mode 100644 src/components/AdConsole/details/__tests__/bid-validation-feedback.test.tsx

diff --git a/src/components/AdConsole/details/TargetsTab.tsx b/src/components/AdConsole/details/TargetsTab.tsx
index cd00db0..518eea4 100644
--- a/src/components/AdConsole/details/TargetsTab.tsx
+++ b/src/components/AdConsole/details/TargetsTab.tsx
@@ -93,8 +93,13 @@ export function TargetsTab({ campaign: c }: Props) {
                   <td><span className={`pill ${t.status === 'Enabled' ? 'green' : 'orange'}`}>{t.status}</span></td>
                   <td>
                     <label htmlFor={`t-bid-${t.id}`} className="visually-hidden">Bid for {t.value}</label>
-                    <input id={`t-bid-${t.id}`} className="input" style={{ width: 72, padding: '4px 6px', fontSize: 12 }}
+                    <input id={`t-bid-${t.id}`} className="input" style={{
+                        width: 72, padding: '4px 6px', fontSize: 12,
+                        ...(Number(bidEdits[t.id] ?? t.bid) < MIN_BID ? { borderColor: 'var(--danger)' } : {}),
+                      }}
                       type="number" min="0.02" step="0.01" value={bidEdits[t.id] ?? t.bid}
+                      aria-invalid={Number(bidEdits[t.id] ?? t.bid) < MIN_BID || undefined}
+                      title={Number(bidEdits[t.id] ?? t.bid) < MIN_BID ? `Minimum bid is $${MIN_BID.toFixed(2)}.` : undefined}
                       onChange={(e) => setBidEdits({ ...bidEdits, [t.id]: e.target.value })} />
                   </td>
                   <td className="mono">{formatWhole(t.impressions)}</td>
diff --git a/src/components/AdConsole/details/__tests__/bid-validation-feedback.test.tsx b/src/components/AdConsole/details/__tests__/bid-validation-feedback.test.tsx
new file mode 100644
index 0000000..a048d3b
--- /dev/null
+++ b/src/components/AdConsole/details/__tests__/bid-validation-feedback.test.tsx
@@ -0,0 +1,71 @@
+/**
+ * Pins visible validation feedback for below-minimum bids in TargetsTab and
+ * AdGroupsTab. Previously the "Set"/"Save default bid" buttons silently did
+ * nothing when the typed value was below MIN_BID — no error state, no
+ * indication anything was wrong (flagged in PR review).
+ */
+import { describe, it, expect } from 'vitest';
+import { render, screen, fireEvent } from '@testing-library/react';
+import { TargetsTab } from '../TargetsTab';
+import { AdGroupsTab } from '../AdGroupsTab';
+import type { Campaign } from '@/engine/ad-console/types';
+
+function makeCampaign(): Campaign {
+  const agId = 'AG1';
+  return {
+    id: 'C1',
+    type: 'SP',
+    name: 'Test',
+    portfolio: 'Training Portfolio',
+    status: 'Enabled',
+    dailyBudget: 10,
+    defaultBid: 0.75,
+    startDate: '2026-01-01',
+    endDate: null,
+    targetingMode: 'Manual keyword',
+    adFormat: 'Standard',
+    bidStrategy: 'Dynamic bids - down only',
+    placements: { top: 0, product: 0, rest: 0 },
+    products: ['B0TRAIN001'],
+    creative: null,
+    metrics: { impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
+    adGroups: [{ id: agId, campaignId: 'C1', name: 'AG', status: 'Enabled', defaultBid: 0.75, metrics: { impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 } }],
+    targets: [
+      { id: 'T1', campaignId: 'C1', adGroupId: agId, type: 'Keyword', value: 'kw', match: 'Exact', bid: 0.75, status: 'Enabled', impressions: 0, clicks: 0, spend: 0, sales: 0, orders: 0 },
+    ],
+    searchTerms: [],
+    negatives: [],
+    budgetRules: [],
+    productAds: [],
+    ads: [],
+    history: [],
+  };
+}
+
+describe('TargetsTab — bid validation feedback', () => {
+  it('marks the bid input invalid once it is typed below MIN_BID', () => {
+    render(<TargetsTab campaign={makeCampaign()} />);
+    const input = screen.getByLabelText('Bid for kw') as HTMLInputElement;
+
+    expect(input.getAttribute('aria-invalid')).toBeNull();
+
+    fireEvent.change(input, { target: { value: '0.01' } });
+
+    expect(input.getAttribute('aria-invalid')).toBe('true');
+    expect(input.title).toContain('Minimum bid is $0.02');
+  });
+});
+
+describe('AdGroupsTab — bid validation feedback', () => {
+  it('marks the default-bid NumberInput invalid once it is typed below its min (built into Astryx NumberInput via the min prop)', () => {
+    render(<AdGroupsTab campaign={makeCampaign()} />);
+    fireEvent.click(screen.getByRole('button', { name: /1 target/ }));
+
+    const input = screen.getByLabelText('Default bid') as HTMLInputElement;
+    expect(input.getAttribute('aria-invalid')).not.toBe('true');
+
+    fireEvent.change(input, { target: { value: '0.01' } });
+
+    expect(input.getAttribute('aria-invalid')).toBe('true');
+  });
+});
diff --git a/src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx b/src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx
index 91f94f3..6e25da7 100644
--- a/src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx
+++ b/src/components/AdConsole/wizard/__tests__/Step6ReviewLaunch.test.tsx
@@ -54,7 +54,8 @@ describe('Step6ReviewLaunch — targeting/creative summary rows', () => {
     expect(screen.getByText('1 entered')).toBeTruthy();
   });
 
-  it('shows the SB/SD creative headline, brand, and destination when set', () => {
+  it.each(['SB', 'SD'] as const)('shows the %s creative headline, brand, and destination when set', (type) => {
+    useAdConsoleStore.getState().updateDraft('type', type);
     useAdConsoleStore.getState().updateDraft('creative', {
       headline: 'Discover your perfect brew',
       brandName: 'Acme Coffee',
diff --git a/src/engine/ad-console/core/__tests__/simulation.test.ts b/src/engine/ad-console/core/__tests__/simulation.test.ts
index 9c2135e..912b3bb 100644
--- a/src/engine/ad-console/core/__tests__/simulation.test.ts
+++ b/src/engine/ad-console/core/__tests__/simulation.test.ts
@@ -179,7 +179,10 @@ describe('simulateDays', () => {
     });
     const [firstRun] = simulateDays([c], 7);
     const [secondRun] = simulateDays([firstRun], 7);
+    const firstTerms = firstRun.searchTerms.map(st => st.term);
     const terms = secondRun.searchTerms.map(st => st.term);
+    expect(firstTerms.length).toBeGreaterThan(0);
+    expect(terms).toEqual(expect.arrayContaining(firstTerms));
     const unique = new Set(terms);
     expect(terms.length).toBe(unique.size);
     // The second run should carry forward the first run's terms, not duplicate them.
diff --git a/src/engine/ad-console/features/integrity/__tests__/engine.test.ts b/src/engine/ad-console/features/integrity/__tests__/engine.test.ts
index 8375d1f..5d69d22 100644
--- a/src/engine/ad-console/features/integrity/__tests__/engine.test.ts
+++ b/src/engine/ad-console/features/integrity/__tests__/engine.test.ts
@@ -129,6 +129,7 @@ describe('runIntegrityCheck', () => {
         }),
       ]);
       const ids = report.issues.map((i) => i.id);
+      expect(ids.length).toBeGreaterThan(1);
       expect(new Set(ids).size).toBe(ids.length);
     } finally {
       nowSpy.mockRestore();