feat(web): revamp Company Brain configure page#1306
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 20e585c | Jul 18 2026, 06:49 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 20e585c | Commit Preview URL Branch Preview URL |
Jul 18 2026, 06:50 PM |
How to use the Graphite Merge QueueAdd the label Main to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Claude finished @MaheshtheDev's task in 1m 26s —— View job Reviewing PR
Review submitted. This is a clean frontend refactor with solid code patterns — no bugs, security issues, or logic errors found. |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview: Revamps the Company Brain configure page with a new tabbed sidebar layout (Integrations, Models, Automations), merges org/personal connection tabs into a unified grid with scope chips, and shows only unconnected apps on the home dashboard.
Issues found: None — this looks good to ship.
The code is well-structured with good patterns:
- Error handling in
connections-board.tsxuses independent try-catch blocks so one failing API call doesn't break the others - All fetch calls correctly use
credentials: "include"for cookie-based auth - Custom MCP URL input is properly guarded behind
isStaffcheck - The UTC cron conversion functions handle timezone/day boundary crossings correctly
- React's automatic escaping prevents XSS vectors
Score: 10/10
|
Note Production impact unlikely. Re-reviewed at f9f24ef: second commit cleanly removes the three Company Brain tabs from the settings modal with no dangling openSettings calls. All components re-homed in ConfigureView, no backend or Worker binding changes. Polylane analysed |
There was a problem hiding this comment.
Overview: Revamps the Company Brain configure page with a new tabbed sidebar layout (Integrations, Models, Automations), merges org/personal connection tabs into a unified grid with scope chips, and shows only unconnected apps on the home dashboard.
Issues found: None — this looks good to ship.
The code is well-structured with good patterns:
- Error handling in
connections-board.tsxuses independent try-catch blocks so one failing API call doesn't break the others - All fetch calls correctly use
credentials: "include"for cookie-based auth - Custom MCP URL input is properly guarded behind
isStaffcheck - The UTC cron conversion functions handle timezone/day boundary crossings correctly
- React's automatic escaping prevents XSS vectors
- Admin checks are in place for org-level operations (disconnect workspace, model changes)
Score: 10/10
Merge activity
|
- New Configure tab with sidebar sections: Integrations, Models, Automations - Merge org/personal connection tabs into one grid: per-card scope chips, admin scope dropdown, Slack as a card, custom MCP via modal - Automations: always-visible template cards with dashed blank-create tile - Home: Connect your tools only shows unconnected apps Fixes ENG-1075
f9f24ef to
20e585c
Compare
## Stack context Stacks on #1306, which moves Company Brain settings into the revamped Configure page. Backend contract and runtime support: supermemoryai/mono#2581. The UI safely hides any effort controls omitted by an older backend response. ## What changed - Adds independent Low, Medium, High, and Extra high reasoning controls for Main, Triage, and Research. - Saves model and reasoning edits together through the existing partial PATCH. - Keeps controls visible but disabled for non-admin members. - Explains that Extra high maps to High for Grok and GPT providers. ## Validation - `bunx biome check apps/web/hooks/use-brain-models.ts apps/web/components/settings/company-brain-models.tsx` - `bun run build` in `apps/web` - `git diff --check` The standalone web TypeScript command still reports pre-existing unrelated errors outside these files. --- **Session Details** - Session: [View Session](https://supermemory.us1.vorflux.com/agent-sessions/1c4f3ec7-a536-4105-bbe7-8b19e61f245f) - Requested by: Sreeram Sreedhar (sreeram@supermemory.com) - Address comments on this PR. Add `(aside)` to your comment to have me ignore it.
2 participants
Fixes ENG-1075