Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(site): expose workspace apps in chat workspace pill #24295

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
DanielleMaywood merged 26 commits into from
Apr 15, 2026
Merged

feat(site): expose workspace apps in chat workspace pill #24295

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
51d076f
feat(site): expose workspace apps in chat workspace pill
DanielleMaywood Apr 13, 2026
057ac8d
test(site): add storybook stories for WorkspacePill
DanielleMaywood Apr 13, 2026
dd3f0be
refactor(site): drop appendChatIdToHref, match AgentRow pattern
DanielleMaywood Apr 13, 2026
da60ae8
chore(site): remove stale-prone comments from WorkspacePill
DanielleMaywood Apr 13, 2026
fd165c9
refactor(site): WorkspacePill owns its status display, eliminate Reac…
DanielleMaywood Apr 13, 2026
f366b30
fix(site): remove stray whitespace expression in ChatTopBar
DanielleMaywood Apr 13, 2026
ba6fda9
fix(site): update AgentChatPage story for meatball menu changes
DanielleMaywood Apr 13, 2026
3441075
refactor(site): use Popover styling from #24308 for WorkspacePill
DanielleMaywood Apr 13, 2026
9689199
refactor(site): move Copy SSH and View Workspace from meatball to pill
DanielleMaywood Apr 13, 2026
6610110
fix(site): flip chevron direction for top popover, suppress tooltip w…
DanielleMaywood Apr 13, 2026
20de227
fix(site): show all workspace apps in pill, not just external ones
DanielleMaywood Apr 13, 2026
9c4be48
fix(site): address self-review findings (separator, overflow, story)
DanielleMaywood Apr 13, 2026
29ae400
fix(site): address review feedback (folder, status indicator, null gu…
DanielleMaywood Apr 13, 2026
0dcd9bd
fix(site): address panel review feedback (dropdown, token guard, stor…
DanielleMaywood Apr 13, 2026
5d6aa30
fix(site): address self-review findings (mutation unmount, disabled s…
DanielleMaywood Apr 13, 2026
559e05b
fix(site): address self-review round 2 (tooltip, alt, folder fallback…
DanielleMaywood Apr 13, 2026
1b186da
refactor(site): inline cls variables and remove workspaceStatusDisplay
DanielleMaywood Apr 13, 2026
19ba116
refactor(site): extract StatusIcon component to deduplicate status ic…
DanielleMaywood Apr 13, 2026
a95bf52
fix(site): address R3 review feedback (shared status, mutation lift, …
DanielleMaywood Apr 14, 2026
831572d
fix(site/src/pages/AgentsPage): add external app disabled assertions,…
DanielleMaywood Apr 14, 2026
6373d75
fix(site/src/pages/AgentsPage): fix WithWorkspace story props, add en…
DanielleMaywood Apr 14, 2026
ef908e8
fix(site/src/pages/AgentsPage): useClipboard, restore Cursor, fix fak…
DanielleMaywood Apr 14, 2026
95d78c5
fix(site/src/pages/AgentsPage): remove builtin Cursor, fix clipboard/…
DanielleMaywood Apr 14, 2026
cff4e04
fix(site/src/pages/AgentsPage): assert href absence on disabled exter…
DanielleMaywood Apr 14, 2026
aedb81c
fix(site/src/pages/AgentsPage): group launch items above separator
DanielleMaywood Apr 14, 2026
ed94b48
fix(site/src/pages/AgentsPage): use consistent app names without Open…
DanielleMaywood Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix(site/src/pages/AgentsPage): use consistent app names without Open… 
… prefix

Drop 'Open in' from VS Code/VS Code Insiders and 'Open' from
Terminal labels to match user-configured app naming (just the app
name). All menu items now consistently show their name only.
  • Loading branch information
@DanielleMaywood
DanielleMaywood committed Apr 14, 2026
commit ed94b483b0f22cc12c39ff69f40059df11b293f1
32 changes: 13 additions & 19 deletions site/src/pages/AgentsPage/components/WorkspacePill.stories.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,18 +126,16 @@ export const WithAllApps: Story = {

await waitFor(() => {
Comment thread
DanielleMaywood marked this conversation as resolved.
const body = within(document.body);
expect(body.getByText("Open in VS Code")).toBeInTheDocument();
expect(body.getByText("Open in VS Code Insiders")).toBeInTheDocument();
expect(body.getByText("VS Code")).toBeInTheDocument();
expect(body.getByText("VS Code Insiders")).toBeInTheDocument();
expect(body.getByText("JetBrains Gateway")).toBeInTheDocument();
expect(body.getByText("Cursor")).toBeInTheDocument();
expect(body.getByText("Open Terminal")).toBeInTheDocument();
expect(body.getByText("Terminal")).toBeInTheDocument();
expect(body.getByText("Copy SSH Command")).toBeInTheDocument();
expect(body.getByText("View Workspace")).toBeInTheDocument();

// Verify items are enabled on a running workspace.
const vscodeItem = body
.getByText("Open in VS Code")
.closest("[role=menuitem]");
const vscodeItem = body.getByText("VS Code").closest("[role=menuitem]");
expect(vscodeItem).not.toHaveAttribute("aria-disabled", "true");

// External apps should be enabled with API key mock.
Expand Down Expand Up @@ -165,13 +163,11 @@ export const WithBuiltinAppsOnly: Story = {

await waitFor(() => {
const body = within(document.body);
expect(body.getByText("Open in VS Code")).toBeInTheDocument();
expect(body.getByText("Open Terminal")).toBeInTheDocument();
expect(body.getByText("VS Code")).toBeInTheDocument();
expect(body.getByText("Terminal")).toBeInTheDocument();
expect(body.getByText("View Workspace")).toBeInTheDocument();
// No external apps or VS Code Insiders.
expect(
body.queryByText("Open in VS Code Insiders"),
).not.toBeInTheDocument();
expect(body.queryByText("VS Code Insiders")).not.toBeInTheDocument();
});
},
};
Expand All @@ -193,8 +189,8 @@ export const WithExternalAppsOnly: Story = {
expect(body.getByText("Cursor")).toBeInTheDocument();
expect(body.getByText("View Workspace")).toBeInTheDocument();
// No built-in apps.
expect(body.queryByText("Open in VS Code")).not.toBeInTheDocument();
expect(body.queryByText("Open Terminal")).not.toBeInTheDocument();
expect(body.queryByText("VS Code")).not.toBeInTheDocument();
expect(body.queryByText("Terminal")).not.toBeInTheDocument();
});
},
};
Expand Down Expand Up @@ -231,7 +227,7 @@ export const WithHiddenApp: Story = {
await waitFor(() => {
const body = within(document.body);
// Visible apps should appear.
expect(body.getByText("Open in VS Code")).toBeInTheDocument();
expect(body.getByText("VS Code")).toBeInTheDocument();
expect(body.getByText("JetBrains Gateway")).toBeInTheDocument();
// Hidden app should NOT appear.
expect(body.queryByText("Hidden Internal Tool")).not.toBeInTheDocument();
Expand All @@ -254,19 +250,17 @@ export const WithStoppedWorkspace: Story = {
const body = within(document.body);

// VS Code items should be present but disabled.
Comment thread
DanielleMaywood marked this conversation as resolved.
const vscodeItem = body
.getByText("Open in VS Code")
.closest("[role=menuitem]");
const vscodeItem = body.getByText("VS Code").closest("[role=menuitem]");
expect(vscodeItem).toHaveAttribute("aria-disabled", "true");

const vscodeInsidersItem = body
.getByText("Open in VS Code Insiders")
.getByText("VS Code Insiders")
.closest("[role=menuitem]");
expect(vscodeInsidersItem).toHaveAttribute("aria-disabled", "true");

// Terminal item should be disabled.
const terminalItem = body
.getByText("Open Terminal")
.getByText("Terminal")
.closest("[role=menuitem]");
expect(terminalItem).toHaveAttribute("aria-disabled", "true");

Expand Down
4 changes: 2 additions & 2 deletions site/src/pages/AgentsPage/components/WorkspacePill.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ const VSCodeMenuItem: FC<{
disabled={isGeneratingKey || !isRunning}
>
<ExternalLinkIcon className="size-3.5" />
Open in {label}
{label}
</DropdownMenuItem>
);
};
Expand Down Expand Up @@ -279,7 +279,7 @@ const TerminalMenuItem: FC<{
disabled={!isRunning}
>
<SquareTerminalIcon className="size-3.5" />
Open Terminal
Terminal
</DropdownMenuItem>
);
};
Expand Down
Loading