Repository navigation
fix: finish UI cleanup and OpenRouter model catalog - #47
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (60)
💤 Files with no reviewable changes (7)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughChangesModel catalog and entitlement handling
Estimated code review effort: 5 (Critical) | ~90 minutes Sequence Diagram(s)sequenceDiagram
participant Browser
participant Bootstrap
participant Main
participant AppLayout
participant HomePage
Browser->>Bootstrap: load application
Bootstrap->>Main: import entry module
Main->>AppLayout: mount application shell
AppLayout->>HomePage: render grouped tools
HomePage-->>Browser: display responsive navigation
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Essentials by visiting https://app.coderabbit.ai/settings/billing. Comment |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Entitlement-read failures can permanently downgrade Pro models, empty Pro configs default incorrectly, and localized model claims remain stale.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
Open (2)
What changed in this PR
This PR completes UI/navigation cleanup and expands the shared OpenRouter model catalog.
Changes:
- Improves responsive navigation, startup shells, homepage grouping, landing previews, and feature layouts.
- Adds ten-model Basic/Pro routing, reasoning rules, migrations, prompt updates, translations, and tests.
- Follow-up is needed for entitlement-read failures, empty Pro defaults, and stale localized model claims.
| File | Reviewed change |
|---|---|
supabase/README.md |
Documents the expanded model catalog. |
supabase/functions/ai/handler-research.test.ts |
Tests catalog routing and reasoning. |
supabase/functions/ai/generate-honesty_test.ts |
Updates model fixtures. |
supabase/functions/ai-stream/prompt.test.ts |
Tests shared prompt rules. |
supabase/functions/ai-stream/handler_test.ts |
Updates streaming model fixtures. |
supabase/functions/_shared/prompts.ts |
Adds plain-language prompt guidance. |
supabase/functions/_shared/prompts.test.ts |
Tests prompt calibration. |
supabase/functions/_shared/model-catalog.ts |
Defines the expanded model catalog. |
supabase/functions/_shared/model-catalog.test.ts |
Validates catalog and reasoning rules. |
src/test/vercel-deployment-config.test.ts |
Tests startup-shell configuration. |
src/test/stores/settings-store-extended.test.ts |
Updates settings defaults. |
src/test/stores/settings-store-branches.test.ts |
Tests settings migration. |
src/test/stores/settings-store-additional.test.ts |
Updates settings behavior tests. |
src/test/settings-store.test.ts |
Tests entitlement-aware model hydration. |
src/test/security-hardening.test.ts |
Updates routing security assertions. |
src/test/providers.test.ts |
Tests the provider catalog. |
src/test/pages/SparringPage.test.tsx |
Updates Sparring UI tests. |
src/test/pages/SettingsAdminNavigation.test.tsx |
Tests admin-role navigation. |
src/test/pages/pages.test.tsx |
Tests homepage and landing changes. |
src/test/pages/HistoryPage.test.tsx |
Updates heading tests. |
src/test/observability.test.ts |
Updates model telemetry fixtures. |
src/test/components/SparringSetup.test.tsx |
Tests compact Sparring setup. |
src/test/components/ModelSelector.test.tsx |
Tests catalog and entitlement controls. |
src/test/components/AppLayout.test.tsx |
Tests responsive navigation. |
src/test/ai/streaming.test.ts |
Updates streaming model fixtures. |
src/pages/TimerPage.tsx |
Removes redundant subtitle. |
src/pages/SettingsPage.tsx |
Adds cached admin-role querying. |
src/pages/LibraryPage.tsx |
Removes redundant descriptions. |
src/pages/HomePage.tsx |
Groups tools into Prepare, Practice, and Utilities. |
src/pages/HistoryPage.tsx |
Simplifies page content. |
src/pages/GeneratePage.tsx |
Removes redundant subtitle. |
src/pages/FlowchartPage.tsx |
Simplifies the header. |
src/pages/FactoidsPage.tsx |
Removes redundant subtitle. |
src/pages/CrossfirePage.tsx |
Removes redundant subtitle. |
src/pages/BrainstormPage.tsx |
Removes redundant subtitle. |
src/pages/AttackPage.tsx |
Removes redundant subtitle. |
src/locales/hu.json |
Updates Hungarian translations. |
src/locales/es.json |
Updates Spanish translations. |
src/locales/en.json |
Updates English translations. |
src/lib/stores/settings-store.ts |
Adds model migration and entitlement normalization. |
src/components/layout/AppLayout.tsx |
Adds responsive desktop/mobile navigation. |
src/components/landing/WhatsNew.tsx |
Adds new tool highlights. |
src/components/landing/SocialProof.tsx |
Updates tool count. |
src/components/landing/Hero.tsx |
Adds interactive mode previews. |
src/components/landing/Footer.tsx |
Removes redundant tagline. |
src/components/landing/Features.tsx |
Adds Crossfire and Sparring. |
src/components/features/sparring/SparringSetup.tsx |
Compacts setup controls. |
src/components/features/onboarding/OnboardingTour.tsx |
Resolves visible tour targets. |
src/components/features/factoids/FactoidsForm.tsx |
Aligns timeframe controls. |
src/bootstrap.ts |
Improves startup and recovery handling. |
public/theme-init.js |
Applies the startup shell and theme early. |
index.html |
Adds themed startup and error shells. |
e2e/startup-shell.spec.ts |
Tests startup behavior. |
e2e/sparring.spec.ts |
Updates Sparring browser coverage. |
e2e/home-navigation.spec.ts |
Tests navigation responsiveness. |
e2e/authed-pages.spec.ts |
Tests Factoids alignment. |
docs/architecture.md |
Documents the expanded catalog. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The app homepage no longer fit its feature count, navigation overflowed during startup and refresh, admin navigation appeared late, and the Factoids and Sparring layouts had alignment and space problems. The landing preview, debate prompts, and model catalog also no longer matched the product.
Result
Validation
npm run verifyCI=1 PORT=18080 PLAYWRIGHT_BASE_URL=http://127.0.0.1:18080 npm run test:e2e:webkit: 79 passed./.superset/db-gate.sh: 27 pgTAP tests passed; local Supabase was stopped afterwardProtected staging E2E was not run because
STAGING_SUPABASE_URL,STAGING_SUPABASE_ANON_KEY, andSTAGING_DEMO_CODEare absent. Production was not used as a substitute.This PR is intentionally left unmerged for Adam's review.
Summary by CodeRabbit
New Features
Improvements