feat(generic): add generic resource tab, refactor home structure, and UI polish#3803
feat(generic): add generic resource tab, refactor home structure, and UI polish#3803waleedlatif1 merged 8 commits intostagingfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Reworks the public chat experience to match the landing theme (dark variables), including a substantial Cleans up/reshapes the site structure by removing most Written by Cursor Bugbot for commit 161bf02. Configure here. |
|
@greptile |
|
@cursor review |
Greptile SummaryThis PR adds the infrastructure for a generic "Results" resource tab, refactors resource-routing logic into a single source of truth, and polishes several UI/structural details across the home feature.\n\nKey changes:\n\n- Confidence Score: 4/5Safe to merge — no runtime regressions introduced; the only concern is staged helper declarations that may surface as linting warnings. The core logic changes are well-structured and type-safe. Prior concerns (whitelabeling regression, duplicated auth button styles) have been resolved. The one remaining item is that genericEntryMap, appendGenericEntry, and updateGenericEntry are declared in live code in use-chat.ts with all call sites commented out, which may trigger no-unused-vars linting warnings. Everything else — persistence guards, registry exhaustiveness, animation cleanup, structural refactoring — is clean. apps/sim/app/workspace/[workspaceId]/home/hooks/use-chat.ts (staged helper declarations without active call sites) Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[SSE tool_call event] --> B{TOOL_PANEL_BEHAVIOR lookup}
B -->|dedicated| C[Open dedicated resource tab\ntable / file / workflow / knowledgebase]
B -->|deferred| D{Does tool_result\nproduce a resource?}
B -->|excluded| E[No resource panel update]
B -->|not in map| F[TODO: Open generic Results tab\ncommented out]
D -->|yes| C
D -->|no| F
C --> G{isEphemeralResource?}
F --> G
G -->|false| H[persistChatResources\nto DB]
G -->|true streaming-file or generic| I[Client-only state only\nskip DB + skip remove mutation\nskip reorder mutation]
Reviews (2): Last reviewed commit: "fix(auth): extract shared auth button cl..." | Re-trigger Greptile |
- Extract AUTH_SUBMIT_BTN constant to (auth)/components/auth-button-classes.ts, replacing 10 copy-pasted identical className strings across 7 files - Update SSOLoginButton primary variant to use AUTH_SUBMIT_BTN instead of hardcoded purple gradient, making it consistent with all other auth form submit buttons - Fix missing isEphemeralResource import in lib/copilot/resources.ts (was re-exported but not available in local scope)
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Summary
Type of Change
Testing
Tested manually
Checklist