Skip to content

feat: resources ux improvements - #609

Merged
cmsparks merged 12 commits into
mainfrom
resources-ux-improvements
Oct 13, 2026
Merged

cmsparks merged 12 commits into
mainfrom
resources-ux-improvements

Conversation

@cmsparks

@cmsparks cmsparks commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

redesign the look of the models, machines, and skills pages

@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit 5ed56bc.

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread frontend/apps/web/src/components/agents/AgentProfilesSection.tsx
@cmsparks
cmsparks changed the base branch from main to agent-page-improvements October 5, 2026 20:19
@cmsparks
cmsparks force-pushed the agent-page-improvements branch 2 times, most recently from 5607c49 to 5ac49f2 Compare October 6, 2026 20:51
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from 19e6dea to 9fd2098 Compare October 6, 2026 22:25

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread frontend/apps/web/src/components/projects/ProjectMachinesView.tsx Outdated
Comment thread frontend/apps/web/src/components/projects/grant-override-diffs.ts
@cmsparks
cmsparks force-pushed the agent-page-improvements branch from 98f4757 to 816fcef Compare October 6, 2026 23:41
@cmsparks cmsparks changed the title Resources ux improvements feat: resources ux improvements Oct 6, 2026
@github-actions github-actions Bot added the feat PRs of type feat (auto-applied from title) label Oct 6, 2026
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from 9fd2098 to bc9151b Compare October 6, 2026 23:52
@cmsparks
cmsparks changed the base branch from agent-page-improvements to main October 7, 2026 01:21
@cmsparks
cmsparks requested a review from asherdale October 7, 2026 01:21
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch 2 times, most recently from 2ce3c3a to f010450 Compare October 7, 2026 01:25

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread frontend/apps/web/src/components/projects/ProjectModelProviderView.tsx Outdated
Comment thread frontend/apps/web/src/components/overview/ModelProvidersSection.tsx Outdated
Comment thread frontend/apps/web/src/components/projects/ProjectMachinesView.tsx Outdated
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from f010450 to 155ea9c Compare October 7, 2026 23:17

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread frontend/apps/web/src/components/integrations/IntegrationProfileList.tsx Outdated
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from 155ea9c to 98055d9 Compare October 7, 2026 23:52

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

itemLabel: (profile) => profile.name,
renderItem: profileLabel,
renderValue: profileLabel,
renderValue: (profile) => <LinkedProfileLabel profile={profile} />,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Profile chips discard unsaved settings

Medium Severity

The Slack and Discord profile chips now render as links that navigate to the profile page. The integration form has no leave warning, so clicking a selected profile name drops unsaved mention or launcher edits. The single-profile picker keeps the name inert and uses a separate Open control instead.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 98055d9. Configure here.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like the chips still link out, and the integration form has no leave warning, so clicking a profile name drops unsaved launcher edits. IntegrationForm already tracks dirty (line 27), so wiring useUnsavedChangesWarning like the agent editor and memory viewer would keep the links and cover the sidebar too

Comment thread frontend/apps/web/src/components/projects/ProjectMachinesView.tsx
Comment thread frontend/apps/web/src/components/skills/SkillCard.tsx Outdated
Comment thread frontend/apps/web/src/components/overview/ModelProvidersSection.tsx Outdated
Comment thread frontend/apps/web/src/components/overview/ProviderModelList.tsx
Comment thread frontend/apps/web/src/components/skills/SkillView.tsx
Comment thread frontend/apps/web/src/components/skills/SkillCard.tsx Outdated
Comment thread frontend/apps/web/src/routes/IntegrationPage.tsx Outdated
Comment thread frontend/apps/web/src/routes/OrgIntegrationsPage.tsx Outdated
Comment thread frontend/apps/web/src/components/overview/MachinePoolsSection.tsx Outdated
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from 98055d9 to 42f39d9 Compare October 8, 2026 21:54

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread frontend/apps/web/src/routes/ModelProviderPage.tsx
Comment thread frontend/apps/web/src/components/overview/MachinePoolsSection.tsx Outdated

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread frontend/apps/web/src/components/overview/MachinePoolsSection.tsx

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread frontend/apps/web/src/components/overview/MachinePoolsSection.tsx
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from 9138589 to aca8f5b Compare October 9, 2026 00:23

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread internal/httpapi/machine_routes.go

@asherdale asherdale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! just some minor/straightforward stuff

Comment thread frontend/apps/web/src/components/skills/SkillView.tsx
itemLabel: (profile) => profile.name,
renderItem: profileLabel,
renderValue: profileLabel,
renderValue: (profile) => <LinkedProfileLabel profile={profile} />,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like the chips still link out, and the integration form has no leave warning, so clicking a profile name drops unsaved launcher edits. IntegrationForm already tracks dirty (line 27), so wiring useUnsavedChangesWarning like the agent editor and memory viewer would keep the links and cover the sidebar too

Comment thread frontend/apps/web/src/components/overview/MachineInventory.tsx
Comment thread frontend/apps/web/src/components/app-shell/OrganizationNav.tsx
Comment thread frontend/apps/web/src/components/app-shell/NavUser.tsx Outdated
Comment thread frontend/apps/web/src/routes/ProjectSkillPage.tsx
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from 4238fe5 to 31384f2 Compare October 13, 2026 17:39
cmsparks and others added 12 commits October 13, 2026 10:39
…ctor

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Drop the skill card file count (and its per-card detail fetch)
- Skip manage-only model provider queries for org members
- Keep the integration settings tab in the URL (?tab=)
- Restore the BYO "Connect machine" / "Share machines" header buttons
- Update model, skill, and machine docs for the new UI
- Split API tokens into org and personal access token pages
- Model combobox and credential secret field layout tweaks

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Add GET /orgs/{orgID}/integrations (listOrgIntegrations), scoped to the
  projects the caller can read, with name filter and cursor paging
- Add ListIntegrationsForProjects store query and integration test
- Add useOrgIntegrations hook; integration mutations also refresh the org list
- Replace the org integrations project picker with an org-wide list of
  integration cards tagged by project
- Share the project picker button between New agent and Add integration

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Skip manage-only provider queries for members on the provider page and
  show an access message instead of "may have been deleted"
- Show org members the machines they can see instead of the manage-only
  pools list; share the machine table via MachineInventory
- Drop the redundant "Not part of any pool" BYO card footers
- Split ProviderCard to reduce complexity; replace the orphan label in
  CredentialSecretField with a group heading
- Keep combobox selections when Escape is pressed with the popup closed
- E2E: reopen the launcher tab after reconnecting an integration

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rences

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cmsparks
cmsparks force-pushed the resources-ux-improvements branch from 31384f2 to 5ed56bc Compare October 13, 2026 17:40

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5ed56bc. Configure here.

}{filters.SourceKinds}
SourceKinds []executionstore.MachineSourceKind
MachinePoolID uuid.UUID `json:",omitzero"`
}{filters.SourceKinds, filters.MachinePoolID}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Project machine list ignores pool filter

Medium Severity

The new machine_pool_id query parameter is applied only on the org listVisibleMachines path. listVisibleProjectMachines never reads it, so a project pool page that asks for one pool still returns every machine visible in the project. The same parameter on listProjectMachinePoolGrants is also specified but unused.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 5ed56bc. Configure here.

@cmsparks
cmsparks merged commit d63edb3 into main Oct 13, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat PRs of type feat (auto-applied from title)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants