Skip to content

Conversation

@ComputelessComputer
Copy link
Collaborator

No description provided.

@coderabbitai
Copy link

coderabbitai bot commented Nov 7, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Restructures product routes: adds multiple new product pages, removes several old ones, refactors ai-notetaking into a template-driven flow, renames and updates ai-assistant route and local-ai content, adjusts landing image tags and minor copy, and converts navigation/footer to data-driven lists.

Changes

Cohort / File(s) Summary
Landing page updates
apps/web/src/routes/_view/index.tsx
Replaced three Image usages with native img in CoolStuffSection; added trailing periods in two HowItWorksSection sentences.
Route path & copy update
apps/web/src/routes/_view/product/ai-assistant.tsx
Changed createFileRoute path from /_view/product/chat/_view/product/ai-assistant; updated title, headings, privacy copy, and CTAs (waitlist → direct download / link to Notepad).
AI notetaking refactor
apps/web/src/routes/_view/product/ai-notetaking.tsx
Rewrote page to template-driven multi-step notetaking; added getCategory, getIconForTemplate, getTemplatesByCategory, TemplateCard and UseCase components; renders templates by category.
Local AI content overhaul
apps/web/src/routes/_view/product/local-ai.tsx
Renamed to "Local AI", updated metadata and intro; restructured feature sections (comparison Local vs Cloud, AI models, compliance), updated CTAs to include Learn about Notepad link.
New product pages (marketing/static)
apps/web/src/routes/_view/product/notepad.tsx, .../api.tsx, .../bot.tsx, .../web.tsx
Added four new route files exporting Route via createFileRoute with static presentational components and meta.
New integration & feature pages
apps/web/src/routes/_view/product/calendar.tsx, .../contacts.tsx, .../daily-note.tsx, .../extensibility.tsx, .../floating-panel.tsx
Added five new multi-section product pages (marketing content), each exporting a Route and head meta.
New OSS & deployment pages
apps/web/src/routes/_view/product/owhisper.tsx, .../self-hosting.tsx
Added OWhisper (detailed marketing) and Self-Hosting (Coming Soon) route pages with Route exports.
Deleted product pages
apps/web/src/routes/_view/product/hybrid-notetaking.tsx, .../local.tsx, .../notetaking.tsx, .../summary.tsx, .../templates.tsx, .../transcript.tsx
Removed six full route files and their exported Route constants, components, and helper functions.
Navigation & footer refactor
apps/web/src/routes/_view/route.tsx
Replaced hardcoded productLinks with productsList and featuresList; data-driven rendering for Products/Features dropdowns and simplified footer grid layout.

Sequence Diagram(s)

(omitted — changes are primarily static page additions, deletions, and UI rendering updates; no new inter-component control flow to diagram)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

  • High-attention areas:
    • apps/web/src/routes/_view/product/ai-notetaking.tsx — new template grouping logic, icon mapping, and component signatures.
    • apps/web/src/routes/_view/route.tsx — navigation data structures and dropdown rendering (desktop + mobile).
    • Add/remove route files — ensure routing registration and links (internal/external) are correct.

Possibly related PRs

  • landing #1617 — modifies image handling in apps/web/src/routes/_view/index.tsx; related to the landing image changes.
  • expanding landing page #1632 — touches route files and templates; related to the ai-notetaking/templates restructuring and multiple route edits.
  • ui fixes #1623 — changes navigation/layout in apps/web/src/routes/_view/route.tsx; related to navigation refactor here.

Suggested reviewers

  • yujonglee
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jj-branch-14

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1117c38 and ce8f47a.

⛔ Files ignored due to path filters (2)
  • apps/web/src/routeTree.gen.ts is excluded by !**/*.gen.ts
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (22)
  • apps/web/src/routes/_view/index.tsx (5 hunks)
  • apps/web/src/routes/_view/product/ai-assistant.tsx (4 hunks)
  • apps/web/src/routes/_view/product/ai-notetaking.tsx (5 hunks)
  • apps/web/src/routes/_view/product/api.tsx (1 hunks)
  • apps/web/src/routes/_view/product/bot.tsx (1 hunks)
  • apps/web/src/routes/_view/product/calendar.tsx (1 hunks)
  • apps/web/src/routes/_view/product/contacts.tsx (1 hunks)
  • apps/web/src/routes/_view/product/daily-note.tsx (1 hunks)
  • apps/web/src/routes/_view/product/extensibility.tsx (1 hunks)
  • apps/web/src/routes/_view/product/floating-panel.tsx (1 hunks)
  • apps/web/src/routes/_view/product/hybrid-notetaking.tsx (0 hunks)
  • apps/web/src/routes/_view/product/local-ai.tsx (2 hunks)
  • apps/web/src/routes/_view/product/local.tsx (0 hunks)
  • apps/web/src/routes/_view/product/notepad.tsx (1 hunks)
  • apps/web/src/routes/_view/product/notetaking.tsx (0 hunks)
  • apps/web/src/routes/_view/product/owhisper.tsx (1 hunks)
  • apps/web/src/routes/_view/product/self-hosting.tsx (1 hunks)
  • apps/web/src/routes/_view/product/summary.tsx (0 hunks)
  • apps/web/src/routes/_view/product/templates.tsx (0 hunks)
  • apps/web/src/routes/_view/product/transcript.tsx (0 hunks)
  • apps/web/src/routes/_view/product/web.tsx (1 hunks)
  • apps/web/src/routes/_view/route.tsx (5 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ComputelessComputer ComputelessComputer merged commit 6249208 into main Nov 7, 2025
3 checks passed
@yujonglee yujonglee deleted the jj-branch-14 branch November 7, 2025 22:38
This was referenced Nov 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants