toga-ai 1.0.351 → 1.0.352

package/knowledge/2.0/apps/ai-bdr/INDEX.md

@@ -6,6 +6,6 @@
 | [BDR Web Funnel — Full Implementation Plan](features/bdr-web-funnel-plan.md) | > **Execution status (2026-07-14):** PLAN **Phases 0–5 are BUILT and QA'd** — the web > app engine, full mockup port, and real backend are done. | bdr/PLAN.md, bdr/mockup/app.jsx, bdr/mockup/screens.jsx, bdr/mockup/components.jsx |
 | [Call Orchestration — PHP Worker ↔ Vapi (the integration seam)](features/call-orchestration.md) | The **PHP worker** is the orchestrator; **Vapi** is the actor. | ai-bdr/docs/client-onboarding-sop.md, ai-bdr/docs/vapi-firstmessage-timing-fix.md, ai-bdr/docs/vapi-inbound-callback-assistant-request.md, ai-bdr/docs/vapi-voicemail-iphone-screening.md, ai-bdr/scripts/update_structured_output.py |
 | [Vapi Integration — Assistants, Tools, Structured Output](features/vapi-integration.md) | Everything inside Vapi: the three assistants, the three shared tools, the shared structured-output schema, the Liquid-templated system prompt, and the Python sc | ai-bdr/vapi/templates/assistant.template.json, ai-bdr/vapi/templates/system-prompt.template.md, ai-bdr/prompts/archive/prompt-may-15.txt, ai-bdr/prompts/campaigns/healthcare-2026-03-18.md, ai-bdr/prompts/campaigns/healthcare-v2-2026-03-25.md, ai-bdr/prompts/templates/campaign-template.md, ai-bdr/scripts/update_assistant.py, ai-bdr/scripts/update_system_prompt.py, ai-bdr/scripts/update_structured_output.py, ai-bdr/scripts/update_call_summary_context.py, ai-bdr/scripts/fix_booking_guardrails.py, ai-bdr/scripts/get_assistant.py, ai-bdr/docs/vapi-firstmessage-timing-fix.md, ai-bdr/docs/vapi-voicemail-iphone-screening.md |
-| [Web Funnel — Built Next.js App (structure, stack, how to run)](features/web-funnel-app.md) | The **built** state of the BDR web funnel — the Next.js UI / lead-capture front door of the existing AI-BDR product (see `../architecture.md`). | bdr/PLAN.md, bdr/.gitignore, bdr/eslint.config.mjs, bdr/amplify.yml, bdr/src/app, bdr/src/lib/resolveCampaignId.ts, bdr/src/content, bdr/src/components, bdr/src/flow, bdr/src/server, bdr/src/app/api, bdr/test, bdr/public, bdr/mockup/styles.css |
+| [Web Funnel — Built Next.js App (structure, stack, how to run)](features/web-funnel-app.md) | The **built** state of the BDR web funnel — the Next.js UI / lead-capture front door of the existing AI-BDR product (see `../architecture.md`). | bdr/PLAN.md, bdr/.gitignore, bdr/eslint.config.mjs, bdr/amplify.yml, bdr/src/app, bdr/src/app/layout.tsx, bdr/src/app/studio.css, bdr/src/lib/resolveCampaignId.ts, bdr/src/lib/scheduleSlots.ts, bdr/src/content, bdr/src/components, bdr/src/flow, bdr/src/server, bdr/src/app/api, bdr/test, bdr/public, bdr/mockup/styles.css |
 | [Web Funnel — Next.js UI + Config-Driven Campaign Content](features/web-funnel-content-model.md) | > **Status note:** the content **contract** below (schema, provider seam, adapters) > is now BUILT and QA'd (PLAN Phases 0–5). | bdr/src/content/schema.ts, bdr/src/content/hubspotProvider.ts, bdr/src/content/useCampaign.ts, bdr/src/content/default.ts, bdr/src/lib/resolveCampaignId.ts, bdr/src/server/leadSink.ts, bdr/src/server/callbackService.ts, bdr/src/app/api/call-now/route.ts, bdr/src/app/api/call-later/route.ts, bdr/src/app/api/contact/route.ts |
 | [New-Campaign Onboarding (6-phase runbook)](workflows/new-campaign-onboarding.md) | The end-to-end procedure for launching a new outbound BDR campaign (industry, offer, target persona, geography). | ai-bdr/docs/client-onboarding-sop.md, ai-bdr/docs/calcom-new-campaign-event-guide.md, ai-bdr/prompts/templates/campaign-template.md, ai-bdr/scripts/update_assistant.py, ai-bdr/scripts/update_system_prompt.py |

package/knowledge/2.0/apps/ai-bdr/features/bdr-web-funnel-plan.md

@@ -6,7 +6,7 @@ project: "AI-BDR"
 client: shared
 type: feature
 status: draft
-updated: 2026-07-14
+updated: 2026-07-15
 owners: [tcox]
 files:
   - bdr/PLAN.md
@@ -515,6 +515,21 @@ so fidelity should be near-exact.
     summaries, accent `RichLine`s) is represented (flat columns can't hold arrays cleanly — child rows or
     a JSON field); who authors/owns the schema; and how the slug relates to the existing
     `hsCampaignId`/Toga campaign UUID used for attribution.
+11. **Privacy Policy & Terms links (§9.11, from the first design review).** The consent
+    footnotes on Call Now / Schedule link to **dead anchors** (`#privacy` / `#terms`,
+    ported verbatim from the mockup; the original `info` app had none). They need real
+    pages or links to `togatech` pages. Wiring is a **2-line content change** — the schema
+    already supports `[label](href)`, and HubSpot campaigns can override the href per
+    campaign.
+12. **Share button is a mock (§9.12).** The Success summary-card Share popover validates
+    the email and shows "Sent" but **sends nothing** (the Copy button IS a real clipboard
+    write). A functional version needs an **email-send route** AND the **real call
+    pipeline** (today's summaries are simulated rotating recaps, not real call data).
+13. **"Avg. connect time" (~7 s) is curated copy, not a metric (§9.13).** It has never
+    been computed in any product version — it is campaign-editable bundle copy. Keep it
+    campaign-editable for launch; a **live calculation is post-launch** (needs a metric
+    definition, a Vapi/Toga call-record source, periodic computation, and real call
+    volume).
 
 ---
 
@@ -558,6 +573,12 @@ so fidelity should be near-exact.
 - [ ] Pixel-perfect parity pass against the mockup screens + screenshots.
 
 ## Change history
+- 2026-07-15 — First UI design review: appended three future items to §9 open questions,
+  mirroring `BDR/PLAN.md` §9.11–§9.13. §9.11 real Privacy/Terms pages (consent footnotes
+  currently link to dead `#privacy`/`#terms` anchors; 2-line content fix). §9.12 Share
+  button is a mock (validates + shows "Sent" but sends nothing; Copy is real) — needs an
+  email-send route plus the real call pipeline. §9.13 "Avg. connect time" (~7 s) is
+  curated campaign copy, never a computed metric; live calculation is post-launch. (tcox)
 - 2026-07-14 — Phases 0–5 executed and QA'd (engine + full mockup port + real backend);
   built codebase now documented in `web-funnel-app.md`. `BDR/PLAN.md` itself corrected:
   absolute local paths genericized, and the stale "not yet in TOGA registry" note fixed

package/knowledge/2.0/apps/ai-bdr/features/web-funnel-app.md

@@ -6,7 +6,7 @@ project: AI-BDR
 client: shared
 type: feature
 status: active
-updated: 2026-07-14
+updated: 2026-07-15
 owners: [tcox]
 files:
   - bdr/PLAN.md
@@ -14,7 +14,10 @@ files:
   - bdr/eslint.config.mjs
   - bdr/amplify.yml
   - bdr/src/app
+  - bdr/src/app/layout.tsx
+  - bdr/src/app/studio.css
   - bdr/src/lib/resolveCampaignId.ts
+  - bdr/src/lib/scheduleSlots.ts
   - bdr/src/content
   - bdr/src/components
   - bdr/src/flow
@@ -55,6 +58,11 @@ is **decision-gated, not code** (see "What is parked / not built").
   hand-authored CSS (ported verbatim as `src/app/studio.css`) and threaten pixel
   parity. Do **not** add Tailwind to this repo — the styling is intentionally the
   mockup's own CSS-variable system. See the styles.css escape-sequence gotcha below.
+- **Font: Plus Jakarta Sans via `next/font/google`** (`Plus_Jakarta_Sans`, weights
+  400/500/600/700/800, `display: swap`, self-hosted at build). Loaded in
+  `src/app/layout.tsx` and exposed as `--font-jakarta` on `<html>`; `studio.css`
+  `--font-display` / `--font-body` resolve `var(--font-jakarta)` first with the
+  original stack kept as fallback. See the font-port gotcha below.
 - **ESLint flat config** carries a **no-em-dash rule** on shipped strings
   (`Literal` / `TemplateElement` / `JSXText` selectors) — enforces the mockup copy
   rule on in-repo strings (`eslint.config.mjs`).
@@ -82,7 +90,10 @@ is **decision-gated, not code** (see "What is parked / not built").
   behind `LeadSink` + `CallbackService` adapters (detail in
   `web-funnel-content-model.md`).
 - `src/app/api/*` — route handlers `call-now`, `call-later`, `contact`.
-- `public/*` — assets pruned to the **default campaign only** (~6.2 MB).
+- `public/*` — assets pruned to the **default campaign only**, **plus AutoBuild's
+  FLOURISH characters** (Owl 1 `owl-02` + Robot 1 `robot-02`, art + talk videos) which
+  the assembly cycle needs even though they are not the campaign agent. See the
+  asset-pruning gotcha below.
 - `test/*` — the 67-test Vitest suite.
 
 ## How the built app behaves (key runtime facts)
@@ -99,6 +110,12 @@ is **decision-gated, not code** (see "What is parked / not built").
 - **Boundary validation on every route handler.** 16 KB body cap; phone / UUID / ISO
   checks; all handlers return the `{isSuccess, status, error, messages, data}`
   envelope (same contract as `info`/Toga).
+- **Schedule timezone is already calculated (not a gap).** `detectLocalTz()`
+  (`src/lib/scheduleSlots.ts`) derives the visitor's zone via the `Intl` short name and
+  renders it in the "All times in `<tz>`" note + the confirmation label; the backend
+  (`src/server/toga.ts`) converts the picked slot to `America/Chicago` for Toga. If
+  `Intl` is unavailable the note simply is not rendered (graceful, no crash). A design
+  request to "make tz calculated" was already satisfied — do not re-implement.
 
 ## What is parked / not built
 
@@ -122,6 +139,20 @@ is **decision-gated, not code** (see "What is parked / not built").
   parsers. This was **fixed in the ported `src/app/studio.css`**, but
   `mockup/styles.css` was intentionally left pristine — so **anyone re-porting the
   mockup CSS must re-apply this fix.**
+- **Port the mockup's `<head>` resources, not just its CSS.** The initial port mirrored
+  the mockup's CSS but not its `<head>` font `<link>`: `studio.css` named Plus Jakarta
+  Sans but nothing ever loaded it, so the whole site silently fell back to `system-ui`.
+  Fixed by loading the font via `next/font/google` in `layout.tsx` (see Stack). **Anyone
+  porting mockup screens must check `<head>` resources (fonts, preconnects, meta) too —
+  a named font in CSS does not mean the font is loaded.**
+- **Asset pruning must include AutoBuild's flourish characters.** Phase 2 pruning (§5.8)
+  kept only the campaign agent's art, which left AutoBuild's FLOURISH cycle (Owl 1
+  `owl-02`, Robot 1 `robot-02`) rendering blank — only the final agent showed. The
+  preloader is **silently tolerant** of missing files (resolves `false`, never blocks),
+  so the gap produced no error and was easy to miss. Fixed by copying `owl-02.png`,
+  `owl-02-talk.mp4`, `robot-02.png`, `robot-02-talk.mp4` into `public/agents/`. **Check
+  `public/agents/` against every form/slot the flow actually cycles, not just the
+  campaign agent** — the tolerant preloader will not warn you.
 - **No PII in logs.** All of `info`'s PII-logging was stripped on port (identifier-
   only logs); the Toga error paths **strip query strings** because the contact
   lookup carries the email in the query. Do not reintroduce value logging.
@@ -131,6 +162,17 @@ is **decision-gated, not code** (see "What is parked / not built").
   harness; pull it with `npx toga-ai`.
 
 ## Change history
+- 2026-07-15 — First UI design review pass. FIXED: site-wide font silently fell back to
+  `system-ui` (mockup's `<head>` Google-Fonts link was never ported) — now Plus Jakarta
+  Sans via `next/font/google` in `layout.tsx`, exposed as `--font-jakarta`, resolved
+  first in `studio.css` (added font-port gotcha + Stack note). FIXED: AutoBuild flourish
+  agents (`owl-02`, `robot-02`) rendered blank because asset pruning kept only the
+  campaign agent and the preloader is silently tolerant of missing files — restored their
+  art + talk videos to `public/agents/` (added asset-pruning gotcha, corrected the
+  `public/*` pruning note). CONFIRMED: Schedule-screen timezone is already calculated
+  (`detectLocalTz()` + backend `America/Chicago` conversion) — documented as a runtime
+  fact so it is not mistaken for a gap. All changes uncommitted in the working tree at
+  capture time. (tcox)
 - 2026-07-14 — Initial doc: recorded the BUILT state of the BDR web funnel (PLAN
   Phases 0–5 executed + QA'd). Next.js 16.2.3 App Router/TS, deliberate no-Tailwind,
   ESLint no-em-dash rule, `resolveCampaignId`, full mockup component/flow port

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toga-ai",
-  "version": "1.0.351",
+  "version": "1.0.352",
   "description": "TOGA Technology Team Claude Knowledge System — shared AI coding harness with skills, knowledge base CLI, and project installer for Claude Code.",
   "keywords": [
     "claude",