Merge branch 'remove-apps-for-now' into release/1.22.0

Author: Douwe Osinga

crates/goose/src/agents/apps_extension.rs

@@ -1,3 +1,5 @@
+#![allow(dead_code)]
+
 use crate::agents::extension::PlatformExtensionContext;
 use crate::agents::mcp_client::{Error, McpClientTrait};
 use crate::config::paths::Paths;
@@ -632,6 +634,7 @@ impl McpClientTrait for AppsManagerClient {
     }
 }
 
+#[allow(dead_code)]
 fn schema<T: JsonSchema>() -> JsonObject {
     let mut obj = serde_json::to_value(schema_for!(T))
         .map(|v| v.as_object().unwrap().clone())
@@ -642,13 +645,15 @@ fn schema<T: JsonSchema>() -> JsonObject {
     obj
 }
 
+#[allow(dead_code)]
 fn extract_string(args: &JsonObject, key: &str) -> Result<String, String> {
     args.get(key)
         .and_then(|v| v.as_str())
         .map(|s| s.to_string())
         .ok_or_else(|| format!("Missing or invalid '{}'", key))
 }
 
+#[allow(dead_code)]
 fn extract_tool_response<T: serde::de::DeserializeOwned>(
     response: &Message,
     tool_name: &str,

crates/goose/src/agents/extension.rs

@@ -1,4 +1,3 @@
-use crate::agents::apps_extension;
 use crate::agents::chatrecall_extension;
 use crate::agents::code_execution_extension;
 use crate::agents::extension_manager_extension;
@@ -55,17 +54,6 @@ pub static PLATFORM_EXTENSIONS: Lazy<HashMap<&'static str, PlatformExtensionDef>
             },
         );
 
-        map.insert(
-            apps_extension::EXTENSION_NAME,
-            PlatformExtensionDef {
-                name: apps_extension::EXTENSION_NAME,
-                description:
-                    "Create and manage custom Goose apps through chat. Apps are HTML/CSS/JavaScript and run in sandboxed windows.",
-                default_enabled: true,
-                client_factory: |ctx| Box::new(apps_extension::AppsManagerClient::new(ctx).unwrap()),
-            },
-        );
-
         map.insert(
             chatrecall_extension::EXTENSION_NAME,
             PlatformExtensionDef {

ui/desktop/src/components/GooseSidebar/AppSidebar.tsx

@@ -1,7 +1,6 @@
 import { AppEvents } from '../../constants/events';
 import React, { useEffect, useState } from 'react';
 import {
-  AppWindow,
   ChefHat,
   ChevronRight,
   Clock,
@@ -31,7 +30,6 @@ import { useNavigation } from '../../hooks/useNavigation';
 import { SessionIndicators } from '../SessionIndicators';
 import { useSidebarSessionStatus } from '../../hooks/useSidebarSessionStatus';
 import { getInitialWorkingDir } from '../../utils/workingDir';
-import { useConfig } from '../ConfigContext';
 
 interface SidebarProps {
   onSelectSession: (sessionId: string) => void;
@@ -63,13 +61,6 @@ const menuItems: NavigationEntry[] = [
     icon: FileText,
     tooltip: 'Browse your saved recipes',
   },
-  {
-    type: 'item',
-    path: '/apps',
-    label: 'Apps',
-    icon: AppWindow,
-    tooltip: 'MCP and custom apps',
-  },
   {
     type: 'item',
     path: '/schedules',
@@ -198,11 +189,8 @@ SessionList.displayName = 'SessionList';
 const AppSidebar: React.FC<SidebarProps> = ({ currentPath }) => {
   const navigate = useNavigate();
   const chatContext = useChatContext();
-  const configContext = useConfig();
   const setView = useNavigation();
 
-  const appsExtensionEnabled = !!configContext.extensionsList?.find((ext) => ext.name === 'apps')
-    ?.enabled;
   const [searchParams] = useSearchParams();
   const [recentSessions, setRecentSessions] = useState<Session[]>([]);
   const [isChatExpanded, setIsChatExpanded] = useState(true);
@@ -468,14 +456,6 @@ const AppSidebar: React.FC<SidebarProps> = ({ currentPath }) => {
     );
   };
 
-  const visibleMenuItems = menuItems.filter((entry) => {
-    // Filter out Apps if extension is not enabled
-    if (entry.type === 'item' && entry.path === '/apps') {
-      return appsExtensionEnabled;
-    }
-    return true;
-  });
-
   return (
     <>
       <SidebarContent className="pt-12">
@@ -561,7 +541,7 @@ const AppSidebar: React.FC<SidebarProps> = ({ currentPath }) => {
 
           <SidebarSeparator />
 
-          {visibleMenuItems.map((entry, index) => renderMenuItem(entry, index))}
+          {menuItems.map((entry, index) => renderMenuItem(entry, index))}
         </SidebarMenu>
       </SidebarContent>
     </>