display tasks tab conditionally and remove ai-tasks experiment

hugodutka · hugodutka · commit d2ce9562e00c · 2025-06-23T18:20:00.000Z
diff --git a/coderd/coderd.go b/coderd/coderd.go
@@ -1536,7 +1536,6 @@ func New(options *Options) *API {
 	// Add CSP headers to all static assets and pages. CSP headers only affect
 	// browsers, so these don't make sense on api routes.
 	cspMW := httpmw.CSPHeaders(
-		api.Experiments,
 		options.Telemetry.Enabled(), func() []*proxyhealth.ProxyHost {
 			if api.DeploymentValues.Dangerous.AllowAllCors {
 				// In this mode, allow all external requests.
diff --git a/coderd/httpmw/csp.go b/coderd/httpmw/csp.go
@@ -6,7 +6,6 @@ import (
 	"strings"
 
 	"github.com/coder/coder/v2/coderd/proxyhealth"
-	"github.com/coder/coder/v2/codersdk"
 )
 
 // cspDirectives is a map of all csp fetch directives to their values.
@@ -59,7 +58,7 @@ const (
 //     Example: https://github.com/coder/coder/issues/15118
 //
 //nolint:revive
-func CSPHeaders(experiments codersdk.Experiments, telemetry bool, proxyHosts func() []*proxyhealth.ProxyHost, staticAdditions map[CSPFetchDirective][]string) func(next http.Handler) http.Handler {
+func CSPHeaders(telemetry bool, proxyHosts func() []*proxyhealth.ProxyHost, staticAdditions map[CSPFetchDirective][]string) func(next http.Handler) http.Handler {
 	return func(next http.Handler) http.Handler {
 		return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 			// Content-Security-Policy disables loading certain content types and can prevent XSS injections.
@@ -124,9 +123,7 @@ func CSPHeaders(experiments codersdk.Experiments, telemetry bool, proxyHosts fun
 			if len(extraConnect) > 0 {
 				for _, extraHost := range extraConnect {
 					// Allow embedding the app host.
-					if experiments.Enabled(codersdk.ExperimentAITasks) {
-						cspSrcs.Append(CSPDirectiveFrameSrc, extraHost.AppHost)
-					}
+					cspSrcs.Append(CSPDirectiveFrameSrc, extraHost.AppHost)
 					if extraHost.Host == "*" {
 						// '*' means all
 						cspSrcs.Append(CSPDirectiveConnectSrc, "*")
diff --git a/coderd/httpmw/csp_test.go b/coderd/httpmw/csp_test.go
@@ -10,7 +10,6 @@ import (
 
 	"github.com/coder/coder/v2/coderd/httpmw"
 	"github.com/coder/coder/v2/coderd/proxyhealth"
-	"github.com/coder/coder/v2/codersdk"
 )
 
 func TestCSP(t *testing.T) {
@@ -50,9 +49,7 @@ func TestCSP(t *testing.T) {
 	r := httptest.NewRequest(http.MethodGet, "/", nil)
 	rw := httptest.NewRecorder()
 
-	httpmw.CSPHeaders(codersdk.Experiments{
-		codersdk.ExperimentAITasks,
-	}, false, func() []*proxyhealth.ProxyHost {
+	httpmw.CSPHeaders(false, func() []*proxyhealth.ProxyHost {
 		return proxyHosts
 	}, map[httpmw.CSPFetchDirective][]string{
 		httpmw.CSPDirectiveMediaSrc: expectedMedia,
diff --git a/codersdk/deployment.go b/codersdk/deployment.go
@@ -3369,7 +3369,6 @@ const (
 	ExperimentWebPush            Experiment = "web-push"             // Enables web push notifications through the browser.
 	ExperimentWorkspacePrebuilds Experiment = "workspace-prebuilds"  // Enables the new workspace prebuilds feature.
 	ExperimentAgenticChat        Experiment = "agentic-chat"         // Enables the new agentic AI chat feature.
-	ExperimentAITasks            Experiment = "ai-tasks"             // Enables the new AI tasks feature.
 )
 
 // ExperimentsSafe should include all experiments that are safe for
diff --git a/site/src/modules/dashboard/Navbar/NavbarView.tsx b/site/src/modules/dashboard/Navbar/NavbarView.tsx
@@ -1,5 +1,4 @@
 import { API } from "api/api";
-import { experiments } from "api/queries/experiments";
 import type * as TypesGen from "api/typesGenerated";
 import { Button } from "components/Button/Button";
 import { ExternalImage } from "components/ExternalImage/ExternalImage";
@@ -10,7 +9,6 @@ import { useWebpushNotifications } from "contexts/useWebpushNotifications";
 import { useEmbeddedMetadata } from "hooks/useEmbeddedMetadata";
 import { NotificationsInbox } from "modules/notifications/NotificationsInbox/NotificationsInbox";
 import type { FC } from "react";
-import { useQuery } from "react-query";
 import { NavLink, useLocation } from "react-router-dom";
 import { cn } from "utils/cn";
 import { DeploymentDropdown } from "./DeploymentDropdown";
@@ -145,7 +143,6 @@ const NavItems: FC<NavItemsProps> = ({ className }) => {
 	const location = useLocation();
 	const agenticChat = useAgenticChat();
 	const { metadata } = useEmbeddedMetadata();
-	const experimentsQuery = useQuery(experiments(metadata.experiments));
 
 	return (
 		<nav className={cn("flex items-center gap-4 h-full", className)}>
@@ -178,7 +175,7 @@ const NavItems: FC<NavItemsProps> = ({ className }) => {
 					Chat
 				</NavLink>
 			)}
-			{experimentsQuery.data?.includes("ai-tasks") && (
+			{metadata.tasksTabVisible.value && (
 				<NavLink
 					className={({ isActive }) => {
 						return cn(linkStyles.default, isActive ? linkStyles.active : "");

Original file line number	Diff line number	Diff line change
`@@ -3369,7 +3369,6 @@ const (`
`3369`	`3369`	`ExperimentWebPush Experiment = "web-push" // Enables web push notifications through the browser.`
`3370`	`3370`	`ExperimentWorkspacePrebuilds Experiment = "workspace-prebuilds" // Enables the new workspace prebuilds feature.`
`3371`	`3371`	`ExperimentAgenticChat Experiment = "agentic-chat" // Enables the new agentic AI chat feature.`
`3372`		`- ExperimentAITasks Experiment = "ai-tasks" // Enables the new AI tasks feature.`
`3373`	`3372`	`)`
`3374`	`3373`
`3375`	`3374`	`// ExperimentsSafe should include all experiments that are safe for`