Add handoff docs, UI primitives, Explain flow, font migration, and backend-unavailable messaging

✅ merged · #16 · foolish-bandit/gary ← foolish-bandit/gary · opened 19d ago by foolish-bandit · merged 19d ago by foolish-bandit · self · +326 across 5 files · ↗ on GitHub

From the PR description

Motivation

  • Provide handoff and operational documentation to capture architecture, deployment status, and roadmap for the project.
  • Surface additional assistant workflows (Explain, Review, Draft) and surface clearer UX for when the backend is not connected.
  • Move away from runtime next/font/google usage toward local font assets and add small UI primitives to standardize component usage.

Description

  • Added handoff and project docs: docs/HANDOFF.md, docs/ARCHITECTURE.md, and docs/DEPLOYMENT_STATUS.md, and a recommended docs/ROADMAP.md.
  • Introduced an Explain flow and page at frontend/src/app/(pages)/explain/page.tsx and wired an Explain action into the assistant initial view.
  • Updated Review and Draft pages to show a standardized backend-unavailable message when handleNewChat cannot start a chat, and added the same fallback for upload failure UI in InitialView.tsx.
  • Added many UI primitive components under frontend/src/components/ui/ (card, alert, label, separator, tabs, textarea, skeleton) and imported them into pages.
  • Migrated font strategy: removed next/font/google usage from frontend/src/app/layout.tsx, added @fontsource/inter in frontend/package.json, and imported Inter weights in frontend/src/app/globals.css while updating CSS font variables.
  • Minor UI and behavior fixes: corrected sidebar localStorage handling in layout.tsx, changed assistant sidebar label to Ask Gary, adjusted the initial action grid sizing, and added dev-auth bypass notice on login/signup when NEXT_PUBLIC_GARY_SKIP_AUTH=true.

Testing

  • Built the frontend app with npm run build --prefix frontend to validate bundling after the font and component changes (build succeeded).
  • Performed basic smoke interactions in local dev (npm run dev --prefix frontend) to verify navigation to Explain, Review, and Draft shells and to confirm backend-unavailable messaging appears when handleNewChat returns no chat id (smoke checks passed).

Codex Task

Our analysis

Frontend polish, Explain flow, and local font migration for Gary — read the full analysis →

Think the analysis missed something the PR description covers?

Commits in this PR (2)

SHA Subject Author Date
3a52bcd0 Add detailed GaryOSS handoff and architecture docs Zack Brenner 2026-05-07 ↗ GitHub
a7d88b83 Merge branch 'main' into codex/create-pr-#10-frontend-polish-and-bugfix-xsy6ae Zack Brenner 2026-05-07 ↗ GitHub

Capture this PR into my fork

Download a Markdown prompt that tells Claude how to port every commit in this PR into your working tree. Run it via claude -p < capture-pull-16.md from inside the repo you want the changes in.

⬇ Download capture-pull-16.md