May 9, 2025 · May 9, 2025 · May 9, 2025
diff --git a/site/migrate-icons.md b/site/migrate-icons.md
 Look for all the @mui/icons-material icons below and replace them accordinlying with the Lucide icon:

 MUI | Lucide
 TaskAlt | CircleCheckBigIcon
 InfoOutlined | InfoIcon
 ErrorOutline | CircleAlertIcon

 You should update the imports and usage.
diff --git a/site/src/modules/apps/apps.ts b/site/src/modules/apps/apps.ts
 : app.url;
 }

 // The backend redirects if the trailing slash isn't included, so we add it
 // here to avoid extra roundtrips.
 let href = `${path}/@${workspace.owner_name}/${workspace.name}.${
 agent.name
 }/apps/${encodeURIComponent(app.slug)}/`;

 if (app.command) {
 // Terminal links are relative. The terminal page knows how
 // to select the correct workspace proxy for the websocket
 // connection.
 href = `/@${workspace.owner_name}/${workspace.name}.${
 return `/@${workspace.owner_name}/${workspace.name}.${
 agent.name
 }/terminal?command=${encodeURIComponent(app.command)}`;
 }
 const baseUrl = `${window.location.protocol}//${host.replace(/\*/g, app.subdomain_name)}`;
 const url = new URL(baseUrl);
 url.pathname = "/";
 href = url.toString();
 }

 return href;
 };

 export const needsSessionToken = (app: WorkspaceApp) => {
 if (!isExternalApp(app)) {
 return false;
 return url.toString();
 }

 // HTTP links should never need the session token, since Cookies
 // handle sharing it when you access the Coder Dashboard. We should
 // never be forwarding the bare session token to other domains!
 const isHttp = app.url.startsWith("http");
 const requiresSessionToken = app.url.includes(SESSION_TOKEN_PLACEHOLDER);
 return requiresSessionToken && !isHttp;
 // The backend redirects if the trailing slash isn't included, so we add it
 // here to avoid extra roundtrips.
 return `${path}/@${workspace.owner_name}/${workspace.name}.${
 agent.name
 }/apps/${encodeURIComponent(app.slug)}/`;
 };

 type ExternalWorkspaceApp = WorkspaceApp & {
 ): app is ExternalWorkspaceApp => {
 return app.external && app.url !== undefined;
 };

 export const needsSessionToken = (app: ExternalWorkspaceApp) => {
 // HTTP links should never need the session token, since Cookies
 // handle sharing it when you access the Coder Dashboard. We should
 // never be forwarding the bare session token to other domains!
 const isHttp = app.url.startsWith("http");
 const requiresSessionToken = app.url.includes(SESSION_TOKEN_PLACEHOLDER);
 return requiresSessionToken && !isHttp;
 };
diff --git a/site/src/modules/apps/useAppLink.ts b/site/src/modules/apps/useAppLink.ts
 const href = getAppHref(app, {
 agent,
 workspace,
 token: apiKeyResponse?.key ?? "",
 token: apiKeyResponse?.key,
 path: proxy.preferredPathAppURL,
 host: proxy.preferredWildcardHostname,
 });
diff --git a/site/src/modules/resources/AppLink/AppLink.tsx b/site/src/modules/resources/AppLink/AppLink.tsx
 TooltipTrigger,
 } from "components/Tooltip/Tooltip";
 import { useProxy } from "contexts/ProxyContext";
 import { needsSessionToken } from "modules/apps/apps";
 import {isExternalApp,needsSessionToken } from "modules/apps/apps";
 import { useAppLink } from "modules/apps/useAppLink";
 import { type FC, useState } from "react";
 import { AgentButton } from "../AgentButton";
 "Your admin has not configured subdomain application access";
 }

 if (needsSessionToken(app) && !link.hasToken) {
 if (isExternalApp(app) &&needsSessionToken(app) && !link.hasToken) {
 canClick = false;
 }
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,8 @@
		Look for all the @mui/icons-material icons below and replace them accordinlying with the Lucide icon:

		MUI \| Lucide
		TaskAlt \| CircleCheckBigIcon
		InfoOutlined \| InfoIcon
		ErrorOutline \| CircleAlertIcon

		You should update the imports and usage.
Original file line number	Diff line number	Diff line change
Expand Up		@@ -83,17 +83,11 @@ export const getAppHref = (
		: app.url;
		}

		// The backend redirects if the trailing slash isn't included, so we add it
		// here to avoid extra roundtrips.
		let href = `${path}/@${workspace.owner_name}/${workspace.name}.${
		agent.name
		}/apps/${encodeURIComponent(app.slug)}/`;

		if (app.command) {
		// Terminal links are relative. The terminal page knows how
		// to select the correct workspace proxy for the websocket
		// connection.
		href = `/@${workspace.owner_name}/${workspace.name}.${
		return `/@${workspace.owner_name}/${workspace.name}.${
		agent.name
		}/terminal?command=${encodeURIComponent(app.command)}`;
		}
Expand All		@@ -102,23 +96,14 @@ export const getAppHref = (
		const baseUrl = `${window.location.protocol}//${host.replace(/\*/g, app.subdomain_name)}`;
		const url = new URL(baseUrl);
		url.pathname = "/";
		href = url.toString();
		}

		return href;
		};

		export const needsSessionToken = (app: WorkspaceApp) => {
		if (!isExternalApp(app)) {
		return false;
		return url.toString();
		}

		// HTTP links should never need the session token, since Cookies
		// handle sharing it when you access the Coder Dashboard. We should
		// never be forwarding the bare session token to other domains!
		const isHttp = app.url.startsWith("http");
		const requiresSessionToken = app.url.includes(SESSION_TOKEN_PLACEHOLDER);
		return requiresSessionToken && !isHttp;
		// The backend redirects if the trailing slash isn't included, so we add it
		// here to avoid extra roundtrips.
		return `${path}/@${workspace.owner_name}/${workspace.name}.${
		agent.name
		}/apps/${encodeURIComponent(app.slug)}/`;
		};

		type ExternalWorkspaceApp = WorkspaceApp & {
Expand All		@@ -131,3 +116,12 @@ export const isExternalApp = (
		): app is ExternalWorkspaceApp => {
		return app.external && app.url !== undefined;
		};

		export const needsSessionToken = (app: ExternalWorkspaceApp) => {
		// HTTP links should never need the session token, since Cookies
		// handle sharing it when you access the Coder Dashboard. We should
		// never be forwarding the bare session token to other domains!
		const isHttp = app.url.startsWith("http");
		const requiresSessionToken = app.url.includes(SESSION_TOKEN_PLACEHOLDER);
		return requiresSessionToken && !isHttp;
		};
Original file line number	Diff line number	Diff line change
Expand Up		@@ -34,7 +34,7 @@ export const useAppLink = (
		const href = getAppHref(app, {
		agent,
		workspace,
		token: apiKeyResponse?.key ?? "",
		token: apiKeyResponse?.key,
		path: proxy.preferredPathAppURL,
		host: proxy.preferredWildcardHostname,
		});
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -9,7 +9,7 @@ import {
		TooltipTrigger,
		} from "components/Tooltip/Tooltip";
		import { useProxy } from "contexts/ProxyContext";
		import { needsSessionToken } from "modules/apps/apps";
		import {isExternalApp,needsSessionToken } from "modules/apps/apps";
		import { useAppLink } from "modules/apps/useAppLink";
		import { type FC, useState } from "react";
		import { AgentButton } from "../AgentButton";
Expand DownExpand Up		@@ -65,7 +65,7 @@ export const AppLink: FC<AppLinkProps> = ({ app, workspace, agent }) => {
		"Your admin has not configured subdomain application access";
		}

		if (needsSessionToken(app) && !link.hasToken) {
		if (isExternalApp(app) &&needsSessionToken(app) && !link.hasToken) {
		canClick = false;
		}

Expand Down