feat(web): UX sweep — unified landings, /connectors rename, skeleton loading#726
Conversation
…loading Bumps owletto-web to feat/ux-sweep-2. Full notes in the submodule commit; high-level: - Pill + routes: Connections → Connectors. - /events, /connectors, /agents share a consistent landing shape (icon + lede + stats + side-by-side setup cards). - Dedicated focused-create routes: /agents/create, /connectors/create, /devices/connect. - Sidebar: section header icons, skeleton row loading (with "no org yet" handled), pill-click memory, active-pill jumps home, connection-status dot now matches "active". - Connector picker: Skills row (default open) with copy-prompt for `npx skills add lobu-ai/lobu --skill lobu`; Lobu CLI install row. - Connection settings: humanized auth section + CLI reauth hint; Run-on Refresh button inline; Activity links to /events filtered by connection_id. - Feed detail: connector + device crosslinks; duplicate status badge suppressed; "no configurable parameters" placeholder when configSchema is absent. - "View in knowledge" renamed to "View in memory". - App-wide TooltipProvider; native title= attrs swept to Radix.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe PR updates the ChangesWeb submodule update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/web`:
- Line 1: The submodule pointer for packages/web is pinned to a SHA not
reachable from owletto-web/main which fails the ancestry check; update the
packages/web submodule reference to point to a commit that is an ancestor of
origin/main in the owletto-web repo (or first merge the feature SHA into
owletto-web/main and then update the submodule) by adjusting the submodule
pointer commit (the SHA in the packages/web entry) and committing the updated
gitlink so the new pointer is reachable from owletto-web/main.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
2 participants
Summary
A broad UX pass on top of #723. Bumps the
owletto-websubmodule tofeat/ux-sweep-2.Routes
Connections→Connectors. The "Connectors" landing now hosts both the connection-create and device-add flows side-by-side (the older page mixed both under a single "Connections" name)./agents/create,/connectors/create,/devices/connect— used by the sidebar "+" buttons. The landing pages keep the inline forms for power users./devices/indexis a redirect to/devices/connect(single source of truth for "add a device")./clients(listing) removed in favor of/clients/$clientKeydetail +/clients/connect. Sidebar "All clients" → "Connected apps".Landings
/events: real header (Database icon + lede) + stats strip (Events / Entities / Entity types / Connections)./connectors: two cards reusing shared<ConnectionCreate>and<DeviceConnect>components./agents: BYOA (McpConnect) + Lobu agent (AgentEditorForm) side-by-side. Page H1 uses a composed mark — both Lobu wings flank a section-specific glyph (<SectionMark>).Loading + chrome
<StatsStrip>with pulsing placeholders so the layout doesn't shift between pages.orgId/slugtreat "no org yet" as loading, so the skeleton bridges the resolution gap.TooltipProvider; nativetitle=attrs on icon buttons swept to Radix Tooltip.Connector picker (
McpConnect)npx skills add lobu-ai/lobu --skill lobu.npm i -g @lobu/cli+ login + memory configure.Connection settings
(optional)dropped from the Run-on label./events?connection_ids=….Feed detail
connection_statusdiffers fromfeed.status).configSchemais absent (fixes Spotify saved-tracks / playlists / recently-played feeling empty).Sidebar fixes
'active'(the canonical server value) in addition to'connected'— was always falling to amber.Misc
DEFAULT_APP_ICON(Plug) fallback for unbranded MCP clients.Test plan
/events,/connectors,/agentsshow the new shared layout + stats strip; numbers render once data loads, skeleton stays put until then./connectors/create, add a device via/devices/connect; both reachable from the sidebar "+" buttons./devices/$deviceId.activeconnections (not amber).Summary by CodeRabbit