> For the complete documentation index, see [llms.txt](https://ai-os-and-trend-finder.gitbook.io/ai-os-and-trend-finder-docs/llms.txt). Markdown versions of documentation pages are available by appending `.md` to page URLs; this page is available as [Markdown](https://ai-os-and-trend-finder.gitbook.io/ai-os-and-trend-finder-docs/.spec_system/archive/sessions/phase33-session05-polish-public-demo-ui-states/implementation-notes.md).

# Implementation Notes

**Session ID**: `phase33-session05-polish-public-demo-ui-states` **Started**: 2026-06-25 06:42 **Last Updated**: 2026-06-25 07:18

***

## Session Progress

| Metric              | Value   |
| ------------------- | ------- |
| Tasks Completed     | 20 / 20 |
| Estimated Remaining | 0 hours |
| Blockers            | 0       |

***

## Task Log

### 2026-06-25 - Session Start

**Environment verified**:

* [x] Prerequisites confirmed
* [x] Tools available
* [x] Directory structure ready

***

### Task T001 - Verify Phase 33 prior session evidence and Session 05 scope

**Started**: 2026-06-25 06:42 **Completed**: 2026-06-25 06:44 **Duration**: 2 minutes

**Notes**:

* Confirmed Phase 33 Sessions 01 through 04 have closeout summaries and validation artifacts showing completed work before this UI-polish session.
* Confirmed deterministic state points at `phase33-session05-polish-public-demo-ui-states` as the current session.
* Confirmed Session 05 scope is limited to public-demo UI copy, disabled controls, route labels, browser-local state explanations, and focused checks.

**Files Changed**:

* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T001 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T001 complete after evidence was recorded.

**Verification**:

* Command/check: `sed -n '1,130p' .spec_system/specs/phase33-session01-capture-local-demo-runs/IMPLEMENTATION_SUMMARY.md`
  * Result: PASS - Session 01 summary exists and records 16 completed tasks, capture review, and no blockers.
  * Evidence: Summary states the local Trend Finder and Dream candidate was produced and reviewed for later public fixture work.
* Command/check: `sed -n '1,130p' .spec_system/specs/phase33-session02-freeze-public-fixtures/IMPLEMENTATION_SUMMARY.md`
  * Result: PASS - Session 02 summary exists and records 18 completed tasks, fixture freezing, and no blockers.
  * Evidence: Summary states public fixture files were generated through the snapshot exporter and Dream/advanced Trend Finder work was handed off.
* Command/check: `sed -n '1,130p' .spec_system/specs/phase33-session03-harden-trend-finder-projection/IMPLEMENTATION_SUMMARY.md`
  * Result: PASS - Session 03 summary exists and records 16 completed tasks, projection hardening, and no blockers.
  * Evidence: Summary states Trend Finder public projection now preserves relationship-aware public output and sanitized Engine Replay state.
* Command/check: `sed -n '1,130p' .spec_system/specs/phase33-session04-harden-dream-projection/IMPLEMENTATION_SUMMARY.md`
  * Result: PASS - Session 04 summary exists and records 18 completed tasks, Dream allowlist projection, and no blockers.
  * Evidence: Summary states Dream Review now exports a small public-safe branch with 4 allowlisted prescriptions.
* Command/check: `jq '{current_session, completed_contains_s01:(.completed_sessions|index("phase33-session01-capture-local-demo-runs")!=null), completed_contains_s02:(.completed_sessions|index("phase33-session02-freeze-public-fixtures")!=null), completed_contains_s03:(.completed_sessions|index("phase33-session03-harden-trend-finder-projection")!=null), completed_contains_s04:(.completed_sessions|index("phase33-session04-harden-dream-projection")!=null)}' .spec_system/state.json`
  * Result: PASS - State records Session 05 as current and Sessions 01 through 04 in `completed_sessions`.
  * Evidence: All four completed-session booleans returned `true`.
* UI product-surface check: N/A - This task audited workflow artifacts only.
* UI craft check: N/A - This task did not change UI.

***

### Task T002 - Audit public-demo copy, Dream controls, and Trend Finder route notices

**Started**: 2026-06-25 06:44 **Completed**: 2026-06-25 06:45 **Duration**: 1 minute

**Notes**:

* Confirmed shared public-demo copy already covers disabled aggregate refresh, Dream run, source setup, Trend Finder run, scheduler, assets, Creator Lens, and browser-local AI Rogue state.
* Identified shared-copy gaps for explicit frozen Trend Finder snapshot, frozen Dream Review snapshot, unavailable Dream snapshot, browser-local watchlist/workbench controls, and frozen Brief/export language.
* Confirmed Dream run and aggregate/run hooks short-circuit public demo mode before `/__token`, `/__run_dream`, `/__refresh_data`, or `/__run_trend_finder` fetches.
* Identified UI copy gaps: `DreamHero` still tells hosted users to run local scheduler commands for empty Dream state; Trend Finder empty states still mention source collectors without public-demo-specific frozen snapshot language; Source Setup/Scheduler panels are network-guarded but still show action controls and local loading labels in public demo mode.

**Files Changed**:

* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T002 audit evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T002 complete after evidence was recorded.

**Verification**:

* Command/check: `sed -n '1,260p' src/lib/public-demo.ts`
  * Result: PASS - Shared public-demo helper coverage and missing copy variants were inspected.
  * Evidence: Existing `PublicDemoUnavailableControl` variants include runtime controls but not dedicated frozen Dream snapshot, Trend Finder snapshot, Brief/export, Workbench, Watchlist, or unavailable snapshot variants.
* Command/check: `sed -n '1,260p' src/components/home/dream-hero.tsx`
  * Result: PASS - Hosted-runtime copy gap found.
  * Evidence: Empty Dream copy still mentions `AI_OS_DREAM_ENABLED=true bun run scheduler:dream:run`, which is not valid public-demo product copy.
* Command/check: `sed -n '1,260p' src/components/dream-run-button.tsx && sed -n '1,260p' src/lib/use-dream-run.ts`
  * Result: PASS - Public-demo Dream run button/hook behavior inspected.
  * Evidence: `useDreamRun()` resolves to `public-demo`, disables the button, and returns before fetching `/__token` or posting `/__run_dream`.
* Command/check: `sed -n '1,220p' src/lib/use-aggregate-refresh.ts && sed -n '220,520p' src/extensions/trend-finder/hooks/use-trend-finder-run.ts`
  * Result: PASS - Public-demo aggregate and Trend Finder run guards inspected.
  * Evidence: Both hooks return before local bridge fetches when `isBrowserPublicDemoMode()` is true.
* Command/check: `rg -n "demo|Demo|fixture|snapshot|runtime|scheduler|collector|source setup|setup|local|bridge|upload|mutation|refresh|run|public" src/extensions/trend-finder/views src/components/app-sidebar.tsx src/lib/public-demo.ts src/components/home/dream-hero.tsx src/components/dream-run-button.tsx src/lib/use-dream-run.ts`
  * Result: PASS - Route notice and hosted-runtime copy hotspots identified.
  * Evidence: Matches highlighted public notices in Trends, Engine Replay, Sources, Workbench, Watchlist, and Brief plus local-command copy in Dream.
* UI product-surface check: PASS - Audited the normal product surfaces for public-demo diagnostics and hosted-runtime claims.
  * Evidence: Main issue is misleading local-command copy, not exposed debug panels; later tasks will replace local-command and generic runtime copy with product-facing public-demo copy.
* UI craft check: PASS - Audited copy density against PRD\_UX requirements for concise trust signals.
  * Evidence: Route notices are already compact, but shared labels need more precise frozen/public-source/browser-local variants.

***

### Task T003 - Audit route matrix, sidebar reachability, and focused test coverage

**Started**: 2026-06-25 06:45 **Completed**: 2026-06-25 06:46 **Duration**: 1 minute

**Notes**:

* Confirmed the Pages demo route matrix includes host, agent, extension, Trend Finder Trends, Engine Replay, Hidden Gems, Sources, Workbench, Watchlist, Brief, and AI Rogue routes.
* Confirmed sidebar route reachability includes Trend Finder extension nav and a dedicated Engine Replay utility link, with public-demo identity label set to `hosted demo`.
* Identified route-matrix gaps: home does not assert Dream Review copy, several Trend Finder routes assert only generic headings/text, and desktop/mobile e2e route checks currently pass after the first matching surface check instead of requiring all listed product-surface checks.
* Confirmed existing focused tests cover public-demo source/scheduler no-bridge behavior, Dream run disabled behavior, route matrix ordering, and no `/__*` requests in Pages route smoke; later tasks need to extend these tests with frozen snapshot, Dream unavailable/populated, browser-local, and stricter route-surface assertions.

**Files Changed**:

* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T003 audit evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T003 complete after evidence was recorded.

**Verification**:

* Command/check: `sed -n '1,560p' scripts/lib/pages-demo-routes.ts`
  * Result: PASS - Route matrix inspected.
  * Evidence: Required public demo routes are present, but Trend Finder route surface checks are mostly generic and home only checks the skills section.
* Command/check: `sed -n '1,320p' scripts/lib/__tests__/pages-demo-routes.test.ts`
  * Result: PASS - Route registry test coverage inspected.
  * Evidence: Tests assert order, unique IDs/paths, route groups, and product-facing check text, but do not yet require Dream or per-route frozen/browser-local checks.
* Command/check: `sed -n '1,320p' src/components/app-sidebar.tsx`
  * Result: PASS - Sidebar reachability inspected.
  * Evidence: Enabled extension nav produces `/extensions/trend-finder/{viewId}` links, Engine Replay appears as a utility route, and public demo identity uses `hosted demo`.
* Command/check: `sed -n '1,260p' tests/e2e/pages-demo-routes.spec.ts && sed -n '1,260p' tests/e2e/pages-demo-mobile.spec.ts`
  * Result: PASS - Desktop/mobile route-smoke harness inspected.
  * Evidence: Both specs record and fail on `/__*` requests; both return from `expectRouteSurface()` after the first passing check, so additional matrix checks are currently alternatives rather than cumulative requirements.
* Command/check: `sed -n '1,260p' src/lib/__tests__/public-demo.test.ts && sed -n '1,260p' src/lib/__tests__/use-dream-run.test.tsx && sed -n '1,260p' src/components/__tests__/dream-run-button.test.tsx && sed -n '1,280p' src/components/home/__tests__/dream-hero.test.tsx`
  * Result: PASS - Existing public-demo and Dream test coverage inspected.
  * Evidence: Public-demo helper and Dream run public-mode tests exist; Dream hero still expects first-run scheduler-command copy.
* UI product-surface check: PASS - Verified route and sidebar surfaces are reachable but route checks need more product-specific public-demo assertions.
  * Evidence: Later route matrix updates will assert visible frozen snapshot, disabled runtime, Dream Review, and browser-local copy instead of only generic headings.
* UI craft check: PASS - Verified the matrix targets desktop/mobile route readability and no-overflow tests.
  * Evidence: Mobile route smoke already checks document overflow; public-demo text fit will be preserved through concise labels.

***

### Task T004 - Extend shared public-demo copy

**Started**: 2026-06-25 06:46 **Completed**: 2026-06-25 06:46 **Duration**: 1 minute

**Notes**:

* Added shared copy variants for frozen real Trend Finder snapshots, frozen real Dream Review snapshots, unavailable Dream snapshots, frozen Brief export, browser-local Workbench state, browser-local Watchlist state, and browser-local visibility settings.
* Tightened existing Dream and Trend Finder run copy to say `AI runtime disabled` instead of only `Frozen snapshot`, separating frozen data from disabled hosted runtime actions.
* Updated source setup copy to say reviewed public-source snapshot data without implying local source config reads.

**Files Changed**:

* `src/lib/public-demo.ts` - Extended the public-demo copy union and central copy map.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T004 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T004 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/public-demo.test.ts`
  * Result: PASS - Public-demo helper suite passed.
  * Evidence: 1 test file passed, 27 tests passed.
* Command/check: `sed -n '1,180p' src/lib/public-demo.ts`
  * Result: PASS - Shared copy map inspected after edits.
  * Evidence: Added variants are present and all copy is ASCII, product-facing, and free of `/__*` bridge strings.
* UI product-surface check: PASS - Shared copy now distinguishes frozen data, disabled hosted runtime, and browser-local state.
  * Evidence: New labels include `Frozen real Trend Finder snapshot`, `Frozen real Dream Review snapshot`, `AI runtime disabled`, and browser-local Workbench/Watchlist labels.
* UI craft check: PASS - Copy stays concise enough for compact route notices and button titles.
  * Evidence: Short labels are available for compact controls; longer explanatory text lives in titles/messages.

**BQC Fixes**:

* Product surface discipline: Replaced generic disabled-run copy with explicit runtime-disabled language so public surfaces do not imply broken local jobs (`src/lib/public-demo.ts`).

***

### Task T005 - Add Dream public-demo populated and unavailable presentation branches

**Started**: 2026-06-25 06:46 **Completed**: 2026-06-25 06:48 **Duration**: 2 minutes

**Notes**:

* Added public-demo branching to `DreamHero` so populated Dream data renders as a frozen real Dream Review snapshot instead of an overnight live review.
* Added an unavailable public-demo branch for empty Dream data that avoids local scheduler commands and explains that no hosted action can run.
* Added browser-local dismissal copy for the case where public-demo users hide all snapshot recommendations.

**Files Changed**:

* `src/components/home/dream-hero.tsx` - Added public-demo snapshot, unavailable, and browser-local dismissal copy paths.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T005 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T005 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/components/home/__tests__/dream-hero.test.tsx`
  * Result: PASS - Existing Dream hero behavior remains intact.
  * Evidence: 1 test file passed, 2 tests passed.
* Command/check: `sed -n '1,150p' src/components/home/dream-hero.tsx`
  * Result: PASS - Public-demo branches inspected after edits.
  * Evidence: Component now uses `dream-snapshot` and `dream-unavailable` shared copy and no public-demo branch mentions local scheduler commands.
* UI product-surface check: PASS - Dream public-demo copy is product-facing and does not expose local command instructions.
  * Evidence: Populated state says `Frozen real Dream Review snapshot`; empty state says the hosted demo did not receive reviewed Dream output.
* UI craft check: PASS - Copy is confined to the existing hero metadata and empty-state block without adding a large explanatory panel.
  * Evidence: Header, detail line, button, and empty-state surfaces retain the existing layout structure.

**BQC Fixes**:

* Product surface discipline: Removed local scheduler command guidance from public-demo Dream empty state (`src/components/home/dream-hero.tsx`).

***

### Task T006 - Tighten Dream run public-demo button and hook semantics

**Started**: 2026-06-25 06:48 **Completed**: 2026-06-25 06:49 **Duration**: 1 minute

**Notes**:

* Updated `DreamRunButton` so disabled states return before click effects and `run()` calls, keeping public-demo disabled behavior inert even if a click event is dispatched.
* Kept `useDreamRun()` public-demo message sourced from the shared public-demo copy object.
* Reconfirmed non-public Dream run success and server duplicate-run behavior after the public-demo copy changes.

**Files Changed**:

* `src/components/dream-run-button.tsx` - Added disabled-click guard and reused the public-demo state boolean for label, title, and aria copy.
* `src/lib/use-dream-run.ts` - Reused shared public-demo copy for resolved public-demo error message.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T006 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T006 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/use-dream-run.test.tsx -t "looks up a token|marks a 409 response"`
  * Result: PASS - Non-public Dream run success and duplicate server response behavior still pass.
  * Evidence: 1 test file passed; 2 tests passed and 5 skipped by filter.
* Command/check: `sed -n '1,120p' src/components/dream-run-button.tsx && sed -n '68,140p' src/lib/use-dream-run.ts`
  * Result: PASS - Button disabled guard and shared public-demo hook copy inspected.
  * Evidence: Button returns before `fireClickEffect()` and `dreamRun.run()` when `dreamRun.isDisabled` is true; hook returns shared `dream-run` message in public demo mode.
* UI product-surface check: PASS - Disabled Dream run button uses shared public-demo label/title/aria copy and does not imply a hosted job will run.
  * Evidence: Shared label now says `AI runtime disabled`; aria title says Dream runs are unavailable in the hosted demo.
* UI craft check: PASS - Compact Dream button behavior changed without altering layout dimensions.
  * Evidence: Existing compact size classes and icon treatment were preserved.

**BQC Fixes**:

* Duplicate action prevention: Disabled Dream buttons now return before click effects and run calls, covering synthetic or stale disabled click paths (`src/components/dream-run-button.tsx`).

***

### Task T007 - Define stricter public-demo route matrix and navigation checks

**Started**: 2026-06-25 06:49 **Completed**: 2026-06-25 06:50 **Duration**: 1 minute

**Notes**:

* Added product-specific public-demo surface checks for home Dream Review and each Trend Finder route.
* Added route checks for frozen Trend Finder/Dream snapshots, disabled AI runtime labels, reviewed media, read-only source/scheduler state, browser-local Workbench/Watchlist state, and frozen Brief export copy.
* Added a shared navigation-check export for hosted demo identity, Trend Finder, Engine Replay, and AI Rogue navigation smoke coverage.

**Files Changed**:

* `scripts/lib/pages-demo-routes.ts` - Added stricter surface checks and `PAGES_DEMO_NAVIGATION_CHECKS`.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T007 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T007 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- scripts/lib/__tests__/pages-demo-routes.test.ts`
  * Result: PASS - Route matrix registry suite passed.
  * Evidence: 1 test file passed, 5 tests passed.
* Command/check: `sed -n '1,560p' scripts/lib/pages-demo-routes.ts`
  * Result: PASS - Updated route matrix inspected.
  * Evidence: Trend Finder routes now include frozen/browser-local/read-only labels and navigation checks are exported for smoke tests.
* UI product-surface check: PASS - Matrix now targets product-facing copy instead of only generic route headings.
  * Evidence: Added checks include `Frozen real Trend Finder snapshot`, `Frozen real Dream Review snapshot`, `AI runtime disabled`, and `Browser-local watchlist`.
* UI craft check: PASS - Matrix changes do not alter UI; they define concise labels for later browser assertions.
  * Evidence: No layout code changed in this task.

***

### Task T008 - Update Trend Finder Trends public-demo states

**Started**: 2026-06-25 06:50 **Completed**: 2026-06-25 06:52 **Duration**: 2 minutes

**Notes**:

* Updated the Trends public-demo notice to show frozen real Trend Finder snapshot, reviewed public-source snapshot, and AI runtime disabled copy together.
* Changed public-demo loading, error, disabled, timestamp, offline, and empty ranked-trend states to use frozen snapshot language instead of live refresh or collector language.
* Updated focused dashboard assertions to verify the new public-demo copy and disabled run behavior.

**Files Changed**:

* `src/extensions/trend-finder/views/trends-view.tsx` - Polished public-demo notice, timestamp, loading/error/empty/offline copy, and disabled runtime labels.
* `src/lib/__tests__/trend-finder-dashboard.test.tsx` - Updated public-demo Trend Finder assertions for the new product-facing copy.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T008 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T008 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/trend-finder-dashboard.test.tsx`
  * Result: PASS - Trend Finder dashboard suite passed.
  * Evidence: 1 test file passed, 36 tests passed.
* Command/check: `rg -n "Frozen real Trend Finder snapshot|Reviewed public-source snapshot|AI runtime disabled|collectors run|Snapshot generated|Public demo snapshot is loading|Failed to load frozen" src/extensions/trend-finder/views/trends-view.tsx src/lib/__tests__/trend-finder-dashboard.test.tsx`
  * Result: PASS - Public-demo copy and updated tests inspected.
  * Evidence: Public-demo branches include frozen snapshot/loading/error/timestamp copy; remaining `collectors run` text is only in the non-public branch.
* UI product-surface check: PASS - Trends public-demo route now presents frozen product output and disabled hosted runtime without live-hosted-work claims.
  * Evidence: Normal public notice text includes frozen snapshot, public-source snapshot, and AI runtime disabled labels.
* UI craft check: PASS - Added compact badges and reused existing notice/card layout without adding large explanatory blocks above ranked trends.
  * Evidence: First-viewport structure remains Trend dashboard, KPI row, run control, and ranked trends.

**BQC Fixes**:

* Product surface discipline: Removed public-demo live refresh/collector implications from Trends empty and timestamp states (`src/extensions/trend-finder/views/trends-view.tsx`).

***

### Task T009 - Update Engine Replay public-demo replay and runtime copy

**Started**: 2026-06-25 06:52 **Completed**: 2026-06-25 06:54 **Duration**: 2 minutes

**Notes**:

* Updated the Engine Replay public-demo notice to use frozen real Trend Finder snapshot, reviewed media, and AI runtime disabled labels.
* Replaced public-demo scheduler/live-progress context with read-only schedule and frozen trace copy instead of showing live polling context.
* Renamed the run-header aggregate label to `Snapshot aggregate run` in public demo mode.
* Updated focused Engine Replay assertions for the new product-facing copy.

**Files Changed**:

* `src/extensions/trend-finder/views/engine-replay-view.tsx` - Polished public-demo replay notice, scheduler context, live-progress copy, and aggregate-run label.
* `src/lib/__tests__/trend-finder-engine-replay.test.tsx` - Updated public-demo replay copy assertions.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T009 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T009 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/trend-finder-engine-replay.test.tsx`
  * Result: PASS - Engine Replay focused suite passed.
  * Evidence: 1 test file passed, 23 tests passed.
* Command/check: `rg -n "Frozen real Trend Finder snapshot|Reviewed media|AI runtime disabled|Read-only schedule|Live progress disabled|Snapshot aggregate run" src/extensions/trend-finder/views/engine-replay-view.tsx src/lib/__tests__/trend-finder-engine-replay.test.tsx`
  * Result: PASS - Updated public-demo replay labels and tests inspected.
  * Evidence: Test asserts frozen real snapshot, reviewed media, and AI runtime disabled; view renders live-progress-disabled and snapshot aggregate labels in public-demo branches.
* UI product-surface check: PASS - Engine Replay now presents sanitized frozen trace data rather than live hosted runtime work.
  * Evidence: Public demo scheduler context states read-only schedule and frozen trace instead of live progress polling.
* UI craft check: PASS - Copy remains in the existing compact notice and context-band areas.
  * Evidence: No new large panel or layout reflow was introduced.

**BQC Fixes**:

* Product surface discipline: Replaced public-demo live scheduler/progress implications with read-only scheduler and frozen trace copy (`src/extensions/trend-finder/views/engine-replay-view.tsx`).

***

### Task T010 - Update Sources public-demo setup and scheduler panels

**Started**: 2026-06-25 06:54 **Completed**: 2026-06-25 06:56 **Duration**: 2 minutes

**Notes**:

* Added reviewed public-source snapshot copy to the Sources public-demo notice.
* Replaced public-demo Source Setup and Scheduler action panels with read-only hosted-demo summary cards.
* Preserved the local-mode Source Setup and Scheduler panels unchanged outside public-demo mode.

**Files Changed**:

* `src/extensions/trend-finder/views/sources-view.tsx` - Added public-demo read-only source/scheduler cards and reviewed-source snapshot notice copy.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T010 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T010 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/trend-finder-dashboard.test.tsx`
  * Result: PASS - Dashboard suite including public-demo Sources no-bridge coverage passed.
  * Evidence: 1 test file passed, 36 tests passed.
* Command/check: `rg -n "Reviewed public-source snapshot|Read-only sources|Read-only schedule|Hosted demo|Source and schedule controls|public snapshot" src/extensions/trend-finder/views/sources-view.tsx src/lib/__tests__/trend-finder-dashboard.test.tsx`
  * Result: PASS - Sources public-demo copy and existing assertions inspected.
  * Evidence: View renders reviewed public-source, read-only sources, read-only schedule, hosted demo, and public snapshot labels.
* UI product-surface check: PASS - Public-demo Sources route no longer shows setup/scheduler write controls as active product actions.
  * Evidence: Public mode renders read-only summary cards instead of `SourceSetupPanel` and `SchedulerFirstRunPanel`.
* UI craft check: PASS - Read-only cards reuse compact dashboard card styling and keep source health first.
  * Evidence: Cards sit in the existing two-column panel area below source health metrics.

**BQC Fixes**:

* Product surface discipline: Removed public-demo source setup and scheduler action controls from the hosted demo surface (`src/extensions/trend-finder/views/sources-view.tsx`).

***

### Task T011 - Update Hidden Gems frozen rows and browser-local filters

**Started**: 2026-06-25 06:56 **Completed**: 2026-06-25 06:57 **Duration**: 1 minute

**Notes**:

* Added public-demo toolbar labels for frozen generated rows and browser-local filters.
* Added public-demo empty states for no hidden gems and no browser-local watched hidden gems.
* Kept local-mode Hidden Gems behavior and copy unchanged.

**Files Changed**:

* `src/extensions/trend-finder/views/hidden-gems-view.tsx` - Added public-demo labels and empty-state copy.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T011 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T011 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/trend-finder-dashboard.test.tsx`
  * Result: PASS - Dashboard suite including Hidden Gems rendering passed.
  * Evidence: 1 test file passed, 36 tests passed.
* Command/check: `rg -n "Frozen generated rows|Browser-local filters|No browser-local watched hidden gems|Frozen snapshot has no hidden gems|committed public snapshot" src/extensions/trend-finder/views/hidden-gems-view.tsx`
  * Result: PASS - Hidden Gems public-demo copy inspected.
  * Evidence: View contains frozen generated row, browser-local filter, and committed public snapshot empty-state copy.
* UI product-surface check: PASS - Hidden Gems distinguishes frozen generated rows from browser-local watch/filter state.
  * Evidence: Public-demo toolbar chips and empty states now use explicit frozen/browser-local language.
* UI craft check: PASS - Added chips are compact and stay inside the existing toolbar area.
  * Evidence: No new section was added before the hidden-gem list.

**BQC Fixes**:

* Product surface discipline: Added browser-local filter copy so Hidden Gems watch filters do not imply fixture mutation (`src/extensions/trend-finder/views/hidden-gems-view.tsx`).

***

### Task T012 - Update Signal Workbench browser-local public-demo copy

**Started**: 2026-06-25 06:57 **Completed**: 2026-06-25 06:58 **Duration**: 1 minute

**Notes**:

* Added public-demo Workbench chips for frozen public snapshot, browser-local workbench triage, and browser-local visibility.
* Added explanatory copy that notes, tags, visibility changes, and triage decisions do not update committed public fixtures.
* Updated loading, empty, offline, and timestamp copy for public-demo Workbench state.

**Files Changed**:

* `src/extensions/trend-finder/views/signal-workbench-view.tsx` - Added public-demo state chips, browser-local explanation, and public-demo status messages.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T012 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T012 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/extensions/trend-finder/components/__tests__/signal-workbench-view.test.tsx`
  * Result: PASS - Signal Workbench focused suite passed.
  * Evidence: 1 test file passed, 12 tests passed.
* Command/check: `rg -n "Frozen public snapshot|Browser-local workbench|Browser-local visibility|browser-local triage|Snapshot generated|committed public fixtures|committed public demo snapshot" src/extensions/trend-finder/views/signal-workbench-view.tsx`
  * Result: PASS - Workbench public-demo copy inspected.
  * Evidence: View contains frozen snapshot loading/status and browser-local triage/offline/timestamp copy; browser-local labels come from shared public-demo copy.
* UI product-surface check: PASS - Workbench public-demo state now says triage and visibility edits are browser-local only.
  * Evidence: Public-demo explanatory line combines Workbench and visibility shared messages.
* UI craft check: PASS - Added chips and one compact explanation in the existing status area above controls.
  * Evidence: Table and filters remain the primary work surface.

**BQC Fixes**:

* Product surface discipline: Added browser-local Workbench and visibility copy so notes, tags, and hidden panels do not imply committed fixture mutation (`src/extensions/trend-finder/views/signal-workbench-view.tsx`).

***

### Task T013 - Update Watchlist generated rows and browser-local state copy

**Started**: 2026-06-25 06:58 **Completed**: 2026-06-25 06:59 **Duration**: 1 minute

**Notes**:

* Added public-demo Watchlist chips for browser-local watchlist state and frozen generated rows.
* Updated local pinned topic cards to say browser-local pins in public demo mode.
* Added public-demo empty-state copy for missing generated Watchlist rows and local-pin-only payloads.

**Files Changed**:

* `src/extensions/trend-finder/views/watchlist-view.tsx` - Added public-demo Watchlist labels, local pin copy, and empty-state copy.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T013 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T013 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/trend-finder-dashboard.test.tsx`
  * Result: PASS - Dashboard suite including Watchlist rendering passed.
  * Evidence: 1 test file passed, 36 tests passed.
* Command/check: `rg -n "Browser-local watchlist|Generated rows are frozen|Browser-local pin|Frozen snapshot has no Watchlist topics|committed public snapshot|Generated Watchlist rows are frozen" src/extensions/trend-finder/views/watchlist-view.tsx`
  * Result: PASS - Watchlist public-demo copy inspected.
  * Evidence: View contains browser-local pin, frozen generated row, and committed public snapshot empty-state copy; browser-local watchlist label comes from shared public-demo copy.
* UI product-surface check: PASS - Watchlist public-demo route now distinguishes frozen generated rows from browser-local pins, notes, tags, and baselines.
  * Evidence: Toolbar chips and local pin cards use explicit public-demo labels.
* UI craft check: PASS - Added chips fit inside existing toolbar and local pin card metadata.
  * Evidence: No new section was added above the Watchlist table.

**BQC Fixes**:

* Product surface discipline: Added frozen-generated and browser-local Watchlist copy so pins, notes, tags, and baselines do not imply committed fixture mutation (`src/extensions/trend-finder/views/watchlist-view.tsx`).

***

### Task T014 - Update Brief frozen snapshot, export, clipboard, and unavailable copy

**Started**: 2026-06-25 06:59 **Completed**: 2026-06-25 07:00 **Duration**: 1 minute

**Notes**:

* Added public-demo frozen Brief copy to the Brief summary and export controls.
* Added export explanation that clipboard actions copy public snapshot content only and do not run collection or update committed fixtures.
* Added public-demo loading, error, disabled, and no-data states for missing Brief snapshot content.

**Files Changed**:

* `src/extensions/trend-finder/views/brief-view.tsx` - Added public-demo Brief summary, export, clipboard, and unavailable-state copy.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T014 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T014 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/extensions/trend-finder/views/__tests__/brief-view.test.tsx`
  * Result: PASS - Brief focused suite including clipboard behavior passed.
  * Evidence: 1 test file passed, 13 tests passed.
* Command/check: `rg -n "Frozen brief|Clipboard exports copy the public snapshot content only|Public Brief snapshot|Frozen Brief snapshot|snapshot items|Brief-ready" src/extensions/trend-finder/views/brief-view.tsx`
  * Result: PASS - Brief public-demo branches inspected.
  * Evidence: View includes public Brief loading/unavailable/no-data copy and uses shared frozen Brief export copy at runtime.
* UI product-surface check: PASS - Brief no longer implies live collection in public-demo unavailable or export states.
  * Evidence: Export controls state clipboard copies public snapshot content only.
* UI craft check: PASS - Frozen Brief copy appears in existing summary and export-control areas.
  * Evidence: Existing Brief layout and clipboard button sizing were preserved.

**BQC Fixes**:

* Duplicate action prevention: Existing pending-state clipboard guard remains covered by the Brief suite after public-demo copy changes (`src/extensions/trend-finder/views/brief-view.tsx`).
* Product surface discipline: Added frozen Brief and public snapshot export copy so hosted users do not infer live collection or fixture mutation (`src/extensions/trend-finder/views/brief-view.tsx`).

***

### Task T015 - Verify and adjust sidebar/navigation public-demo labels

**Started**: 2026-06-25 07:03 **Completed**: 2026-06-25 07:04 **Duration**: 1 minute

**Notes**:

* Made the sidebar identity select the public-demo name, avatar, and profile label explicitly at render time.
* Preserved local-mode operator initials, local storage event updates, extension navigation, Engine Replay utility navigation, and mobile sidebar close behavior.
* Confirmed the public-demo identity surface stays product-facing as `AI OS Public Demo` / `hosted demo` and does not show env or localStorage operator values.

**Files Changed**:

* `src/components/app-sidebar.tsx` - Added explicit public-demo display name/avatar/profile selection while preserving local fallback initials.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T015 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T015 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/components/__tests__/app-sidebar.test.tsx`
  * Result: PASS - Sidebar focused suite passed.
  * Evidence: 1 test file passed, 28 tests passed.
* Command/check: `rg -n "AI OS Public Demo|hosted demo|Engine Replay|Trend Finder" src/components/app-sidebar.tsx src/components/__tests__/app-sidebar.test.tsx`
  * Result: PASS - Sidebar public-demo identity and navigation coverage inspected.
  * Evidence: Test asserts hosted-demo identity ignores env/localStorage values and the sidebar still exposes Trend Finder and Engine Replay navigation.
* UI product-surface check: PASS - Public-demo sidebar identity does not imply a live local operator or local runtime session.
  * Evidence: Hosted mode renders the shared public-demo identity and profile label instead of private operator storage.
* UI craft check: PASS - Sidebar layout, compact identity block, and footer navigation structure were unchanged.
  * Evidence: The edit only changes selected display values; sizing classes and nav order remain intact.

**BQC Fixes**:

* Product surface discipline: Public-demo sidebar identity is selected explicitly at render time so stale local state cannot surface private operator copy in hosted mode (`src/components/app-sidebar.tsx`).

***

### Task T016 - Add public-demo copy and Dream run tests

**Started**: 2026-06-25 07:04 **Completed**: 2026-06-25 07:05 **Duration**: 1 minute

**Notes**:

* Expanded public-demo helper coverage to include every new frozen snapshot, disabled runtime, browser-local, and frozen Brief copy variant.
* Updated Dream run hook and button tests to assert the shared `AI runtime disabled` public-demo copy.
* Reconfirmed public-demo Dream run controls do not fetch `/__token`, do not post Dream endpoints, do not refresh live data, and remain inert when disabled.

**Files Changed**:

* `src/lib/__tests__/public-demo.test.ts` - Added coverage for new public-demo copy variants and exact frozen/browser-local labels.
* `src/lib/__tests__/use-dream-run.test.tsx` - Updated disabled public-demo Dream run message expectation.
* `src/components/__tests__/dream-run-button.test.tsx` - Updated accessible disabled-control label/title expectations.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T016 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T016 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/public-demo.test.ts src/lib/__tests__/use-dream-run.test.tsx src/components/__tests__/dream-run-button.test.tsx`
  * Result: PASS - Public-demo helper, Dream run hook, and Dream run button suites passed.
  * Evidence: 3 test files passed, 40 tests passed.
* Command/check: `rg -n "AI runtime disabled|Frozen real Dream Review snapshot|Browser-local workbench|Browser-local watchlist|not.toHaveBeenCalled|dream runs are unavailable" src/lib/__tests__/public-demo.test.ts src/lib/__tests__/use-dream-run.test.tsx src/components/__tests__/dream-run-button.test.tsx`
  * Result: PASS - Disabled runtime, frozen snapshot, browser-local, no-fetch/no-refresh, and accessible-label assertions inspected.
  * Evidence: Tests assert `AI runtime disabled`, frozen Dream snapshot copy, browser-local Workbench/Watchlist labels, no fetch/refresh calls, and public-demo button aria label.
* UI product-surface check: PASS - Disabled Dream run controls now test the same product-facing shared copy shown in the UI.
  * Evidence: Button test asserts visible `AI runtime disabled` and hosted-demo aria label.
* UI craft check: PASS - Tests preserve the compact button accessibility surface while changing only public-demo copy expectations.
  * Evidence: Existing idle/running/failed/skipped button states remain covered.

**BQC Fixes**:

* Duplicate action prevention: Tests verify disabled public-demo Dream button clicks do not invoke `run()` and hook public-demo mode does not fetch or refresh (`src/components/__tests__/dream-run-button.test.tsx`, `src/lib/__tests__/use-dream-run.test.tsx`).

***

### Task T017 - Add Dream home public-demo snapshot tests

**Started**: 2026-06-25 07:05 **Completed**: 2026-06-25 07:06 **Duration**: 1 minute

**Notes**:

* Added a populated public-demo DreamHero test that asserts frozen reviewed snapshot copy and no scheduler command text.
* Added an unavailable public-demo DreamHero test that asserts hosted-demo unavailable copy and no local Dream scheduler instructions.
* Reset Vite env stubs after each DreamHero test to prevent public-demo mode from leaking across cases.

**Files Changed**:

* `src/components/home/__tests__/dream-hero.test.tsx` - Added public-demo populated and unavailable Dream snapshot coverage.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T017 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T017 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/components/home/__tests__/dream-hero.test.tsx`
  * Result: PASS - DreamHero focused suite passed.
  * Evidence: 1 test file passed, 4 tests passed.
* Command/check: `rg -n "Frozen real Dream Review snapshot|reviewed improvements in frozen snapshot|Dream Review snapshot is unavailable|AI_OS_DREAM_ENABLED|scheduler:dream:run|unstubAllEnvs" src/components/home/__tests__/dream-hero.test.tsx`
  * Result: PASS - Public-demo Dream snapshot assertions inspected.
  * Evidence: Tests assert frozen reviewed output, unavailable snapshot copy, no local scheduler command, and env cleanup.
* UI product-surface check: PASS - Dream home public-demo tests now enforce product-facing frozen/unavailable states.
  * Evidence: Tests fail if public-demo DreamHero reintroduces local scheduler instructions.
* UI craft check: PASS - Tests target existing hero label/detail/empty-state surfaces without requiring layout changes.
  * Evidence: Existing navigation, dismiss, and restore test remains intact.

**BQC Fixes**:

* Product surface discipline: Public-demo DreamHero tests guard against local scheduler command copy returning to hosted UI (`src/components/home/__tests__/dream-hero.test.tsx`).

***

### Task T018 - Add Trend Finder public-demo unit tests

**Started**: 2026-06-25 07:06 **Completed**: 2026-06-25 07:09 **Duration**: 3 minutes

**Notes**:

* Added dashboard coverage for public-demo Sources reviewed-source copy, Hidden Gems frozen/browser-local toolbar and empty states, and Watchlist frozen/browser-local labels and empty snapshot state.
* Added Signal Workbench coverage for public-demo frozen loading/empty states, browser-local Workbench/visibility chips, browser-local explanation copy, and snapshot timestamp label.
* Added Brief coverage for public-demo frozen Brief summary/export labels and clipboard-only public snapshot export copy.
* Re-ran the focused Trend Finder suites and fixed an assertion to allow repeated reviewed-source labels across the notice chip and read-only summary card.

**Files Changed**:

* `src/lib/__tests__/trend-finder-dashboard.test.tsx` - Added public-demo Sources, Hidden Gems, Watchlist, and no-bridge label coverage.
* `src/extensions/trend-finder/components/__tests__/signal-workbench-view.test.tsx` - Added public-demo Workbench frozen/browser-local state coverage and env cleanup.
* `src/extensions/trend-finder/views/__tests__/brief-view.test.tsx` - Added public-demo frozen Brief export coverage and env cleanup.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T018 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T018 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/trend-finder-dashboard.test.tsx src/extensions/trend-finder/components/__tests__/signal-workbench-view.test.tsx src/extensions/trend-finder/views/__tests__/brief-view.test.tsx`
  * Result: PASS - Dashboard, Signal Workbench, and Brief focused suites passed.
  * Evidence: 3 test files passed, 65 tests passed.
* Command/check: `rg -n "Reviewed public-source snapshot|Frozen generated rows|Browser-local filters|No browser-local watched hidden gems|Frozen snapshot has no hidden gems|Browser-local watchlist|Generated rows are frozen|Frozen snapshot has no Watchlist topics|Frozen public snapshot|Browser-local workbench|Browser-local visibility|Frozen brief|Clipboard exports copy the public snapshot content only|not.toHaveBeenCalled|/__run_trend_finder" src/lib/__tests__/trend-finder-dashboard.test.tsx src/extensions/trend-finder/components/__tests__/signal-workbench-view.test.tsx src/extensions/trend-finder/views/__tests__/brief-view.test.tsx`
  * Result: PASS - Trend Finder public-demo labels, empty states, browser-local copy, and no-bridge assertions inspected.
  * Evidence: Tests now assert reviewed-source, frozen/generated rows, browser-local filter/watchlist/workbench/visibility, frozen Brief export, no fetch/refresh calls, and no public-demo run bridge calls.
* UI product-surface check: PASS - Tests now guard public-demo Trend Finder routes against live-collection or committed-fixture mutation claims.
  * Evidence: Hidden Gems, Watchlist, Workbench, and Brief tests assert frozen or browser-local language on visible surfaces.
* UI craft check: PASS - Test additions target existing toolbar chips, notices, empty states, and export panels without requiring layout changes.
  * Evidence: Existing route rendering and interaction coverage remains intact.

**BQC Fixes**:

* Product surface discipline: Added tests that fail if Trend Finder public-demo routes drop frozen snapshot, browser-local, or clipboard-only export copy (`src/lib/__tests__/trend-finder-dashboard.test.tsx`, `src/extensions/trend-finder/components/__tests__/signal-workbench-view.test.tsx`, `src/extensions/trend-finder/views/__tests__/brief-view.test.tsx`).

***

### Task T019 - Update Pages route registry and smoke expectations

**Started**: 2026-06-25 07:09 **Completed**: 2026-06-25 07:10 **Duration**: 1 minute

**Notes**:

* Added route-registry unit assertions for shared desktop navigation smoke checks.
* Added route-registry assertions that key public-demo routes keep multiple cumulative product-surface checks for frozen Dream, Trend Finder, Sources, Hidden Gems, Workbench, Watchlist, and Brief copy.
* Updated desktop and mobile Pages smoke helpers so every configured route surface check must be visible, rather than passing after the first visible check.
* Added shared desktop navigation smoke checks for hosted demo identity, Trend Finder, Engine Replay, and AI Rogue navigation surfaces.

**Files Changed**:

* `scripts/lib/__tests__/pages-demo-routes.test.ts` - Added shared navigation and cumulative public-demo surface registry assertions.
* `tests/e2e/pages-demo-routes.spec.ts` - Required every desktop route surface check and added shared navigation smoke checks.
* `tests/e2e/pages-demo-mobile.spec.ts` - Required every mobile route surface check.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T019 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T019 complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- scripts/lib/__tests__/pages-demo-routes.test.ts`
  * Result: PASS - Pages route registry suite passed.
  * Evidence: 1 test file passed, 7 tests passed.
* Command/check: `rg -n "PAGES_DEMO_NAVIGATION_CHECKS|did not render every expected|expectNavigationSurface|Frozen real Dream Review snapshot|Clipboard exports copy" scripts/lib/__tests__/pages-demo-routes.test.ts tests/e2e/pages-demo-routes.spec.ts tests/e2e/pages-demo-mobile.spec.ts`
  * Result: PASS - Registry and smoke expectation changes inspected.
  * Evidence: Unit test imports navigation checks, desktop smoke calls `expectNavigationSurface`, and desktop/mobile helpers now fail when any configured route surface check is missing.
* UI product-surface check: PASS - Route smoke readiness now treats public-demo labels as cumulative product requirements.
  * Evidence: Key routes must keep frozen snapshot, disabled runtime, read-only, browser-local, and Brief export labels in the route matrix.
* UI craft check: PASS - Smoke changes verify existing visible surfaces without changing UI layout.
  * Evidence: Mobile smoke continues to check route surfaces and document overflow, while desktop checks persistent navigation surfaces.

**BQC Fixes**:

* Product surface discipline: Pages smoke helpers now require every configured public-demo route label, preventing weak first-match route coverage (`tests/e2e/pages-demo-routes.spec.ts`, `tests/e2e/pages-demo-mobile.spec.ts`).

***

### Task T020 - Run focused checks, Pages smoke readiness, ASCII, and LF validation

**Started**: 2026-06-25 07:10 **Completed**: 2026-06-25 07:18 **Duration**: 8 minutes

**Notes**:

* Ran the required focused Vitest command from the task checklist.
* Re-ran the sidebar focused suite because T015 changed sidebar identity rendering outside the required command.
* Ran the Pages desktop/mobile public-demo Playwright smoke specs against the `pages-demo-chromium` project.
* Fixed the smoke helper after the first Playwright run found duplicate exact text where the first DOM match was hidden on mobile; helpers now select the first visible matching locator.
* Ran ASCII, CRLF, `git diff --check`, project typecheck, scripts typecheck, and direct Playwright spec typecheck.

**Files Changed**:

* `tests/e2e/pages-demo-routes.spec.ts` - Added visible-match helper for cumulative desktop route and navigation checks.
* `tests/e2e/pages-demo-mobile.spec.ts` - Added visible-match helper for cumulative mobile route checks.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/implementation-notes.md` - Recorded T020 evidence and progress.
* `.spec_system/specs/phase33-session05-polish-public-demo-ui-states/tasks.md` - Marked T020 and completion checklist complete after evidence was recorded.

**Verification**:

* Command/check: `bun run test -- src/lib/__tests__/public-demo.test.ts src/lib/__tests__/use-dream-run.test.tsx src/components/__tests__/dream-run-button.test.tsx src/components/home/__tests__/dream-hero.test.tsx src/lib/__tests__/trend-finder-dashboard.test.tsx src/lib/__tests__/trend-finder-engine-replay.test.tsx src/extensions/trend-finder/components/__tests__/signal-workbench-view.test.tsx src/extensions/trend-finder/views/__tests__/brief-view.test.tsx scripts/lib/__tests__/pages-demo-routes.test.ts`
  * Result: PASS - Required focused Vitest command passed.
  * Evidence: 9 test files passed, 139 tests passed.
* Command/check: `bun run test -- src/components/__tests__/app-sidebar.test.tsx`
  * Result: PASS - Sidebar focused suite passed.
  * Evidence: 1 test file passed, 28 tests passed.
* Command/check: `bun run test:e2e -- tests/e2e/pages-demo-routes.spec.ts tests/e2e/pages-demo-mobile.spec.ts --project=pages-demo-chromium`
  * Result: PASS - Pages desktop/mobile public-demo smoke readiness passed after visible-match helper fix.
  * Evidence: 48 tests passed across desktop and mobile route matrix specs.
* Command/check: `if LC_ALL=C rg -n "[^[:ascii:]]" $(git diff --name-only) $(git ls-files --others --exclude-standard); then exit 1; else test $? -eq 1; fi`
  * Result: PASS - Changed and untracked session files contain only ASCII.
  * Evidence: Command exited 0 with no matches.
* Command/check: `if rg -n $'\r' $(git diff --name-only) $(git ls-files --others --exclude-standard); then exit 1; else test $? -eq 1; fi`
  * Result: PASS - Changed and untracked session files use LF line endings.
  * Evidence: Command exited 0 with no carriage-return matches.
* Command/check: `git diff --check`
  * Result: PASS - No whitespace errors in tracked diffs.
  * Evidence: Command exited 0 with no output.
* Command/check: `bun run typecheck`
  * Result: PASS - Project typecheck passed.
  * Evidence: `tsc --noEmit` exited 0.
* Command/check: `bun run typecheck:scripts`
  * Result: PASS - Scripts typecheck passed.
  * Evidence: `tsc --noEmit -p tsconfig.scripts.json` exited 0.
* Command/check: `bunx tsc --ignoreConfig --noEmit --target ES2022 --module ESNext --moduleResolution Bundler --lib ES2022,DOM,DOM.Iterable --types node --skipLibCheck tests/e2e/pages-demo-routes.spec.ts tests/e2e/pages-demo-mobile.spec.ts`
  * Result: PASS - Direct Playwright spec typecheck passed.
  * Evidence: Command exited 0.
* UI product-surface check: PASS - Focused unit and e2e checks verify frozen snapshot, disabled runtime, read-only source/scheduler, browser-local, and clipboard-only public-demo surfaces.
  * Evidence: Vitest and Playwright route smoke commands passed with cumulative product-surface checks.
* UI craft check: PASS - Desktop/mobile smoke checks passed without document overflow on mobile and with visible route/navigation surfaces.
  * Evidence: 48 Playwright route tests passed, including mobile overflow assertions.

**BQC Fixes**:

* Product surface discipline: Playwright route helpers now select visible matches before asserting cumulative labels, preventing hidden duplicate DOM text from masking visible product surfaces (`tests/e2e/pages-demo-routes.spec.ts`, `tests/e2e/pages-demo-mobile.spec.ts`).


---

# Agent Instructions
This documentation is published with GitBook. GitBook is the documentation platform designed so that both humans and AI agents can read, navigate, and reason over technical content effectively. Learn more at gitbook.com.

## Querying This Documentation
If you need additional information that is not directly available in this page, you can query the documentation dynamically by asking a question.

Perform an HTTP GET request on the current page URL with the `ask` query parameter, and the optional `goal` query parameter:

```
GET https://ai-os-and-trend-finder.gitbook.io/ai-os-and-trend-finder-docs/.spec_system/archive/sessions/phase33-session05-polish-public-demo-ui-states/implementation-notes.md?ask=<question>&goal=<endgoal>
```

`ask` is the immediate question: it should be specific, self-contained, and written in natural language.
`goal` is optional and describes the broader end goal you are ultimately trying to accomplish on behalf of the user. GitBook uses it to tailor the answer towards what is most useful for that goal.

The response will contain a direct answer to the question and relevant excerpts and sources from the documentation.

Use this mechanism when the answer is not explicitly present in the current page, you need clarification or additional context, or you want to retrieve related documentation sections.