fix (tasks): fixes to task page, fixes to executor, fixes to settings and onboarding, fixes to integrations page

Author: itsskofficial

src/client/app/api/settings/ai-personality/route.js

@@ -45,6 +45,7 @@ import {
 	MorphingDialogContainer
 } from "@components/ui/morphing-dialog"
 import { Tooltip } from "react-tooltip"
+import ModalDialog from "@components/ModalDialog"
 
 import IconBrandTodoist from "@components/icons/IconBrandTodoist"
 
@@ -457,6 +458,7 @@ const IntegrationsPage = () => {
 	const [searchQuery, setSearchQuery] = useState("")
 	const [activeCategory, setActiveCategory] = useState("All")
 	const [selectedIntegration, setSelectedIntegration] = useState(null)
+	const [activeManualIntegration, setActiveManualIntegration] = useState(null)
 	const [whatsAppToConnect, setWhatsAppToConnect] = useState(null)
 	const posthog = usePostHog()
 
@@ -509,87 +511,87 @@ const IntegrationsPage = () => {
 		}
 	}, [])
 
-		const handleTrelloConnect = (integration) => {
-			const trelloApiKey = integration.client_id
-			if (!trelloApiKey) {
-				toast.error(
-					"Trello API Key is not configured by the administrator."
-				)
-				return
-			}
+	const handleTrelloConnect = (integration) => {
+		const trelloApiKey = integration.client_id
+		if (!trelloApiKey) {
+			toast.error(
+				"Trello API Key is not configured by the administrator."
+			)
+			return
+		}
 
-			const returnUrl = `${window.location.origin}/integrations`
-			const authUrl = `https://trello.com/1/authorize?expiration=never&name=Sentient&scope=read,write&response_type=token&key=${trelloApiKey}&return_url=${encodeURIComponent(returnUrl)}&callback_method=postMessage`
+		const returnUrl = `${window.location.origin}/integrations`
+		const authUrl = `https://trello.com/1/authorize?expiration=never&name=Sentient&scope=read,write&response_type=token&key=${trelloApiKey}&return_url=${encodeURIComponent(returnUrl)}&callback_method=postMessage`
 
-			const authWindow = window.open(
-				authUrl,
-				"trelloAuth",
-				"width=600,height=700,noopener,noreferrer"
-			)
+		const authWindow = window.open(
+			authUrl,
+			"trelloAuth",
+			"width=600,height=700,noopener,noreferrer"
+		)
 
-			// --- ADDED FOR DEBUGGING ---
-			// This listener will log EVERY message event that comes into the window,
-			// helping us see what Trello is sending back, even if it fails the security checks.
-			const debugListener = (event) => {
-				console.log("DEBUG: Received a postMessage event:", event)
-				console.log("DEBUG: Event Origin:", event.origin)
-				console.log("DEBUG: Event Data:", event.data)
-			}
-			window.addEventListener("message", debugListener)
-			// --- END OF DEBUGGING CODE ---
+		// --- ADDED FOR DEBUGGING ---
+		// This listener will log EVERY message event that comes into the window,
+		// helping us see what Trello is sending back, even if it fails the security checks.
+		const debugListener = (event) => {
+			console.log("DEBUG: Received a postMessage event:", event)
+			console.log("DEBUG: Event Origin:", event.origin)
+			console.log("DEBUG: Event Data:", event.data)
+		}
+		window.addEventListener("message", debugListener)
+		// --- END OF DEBUGGING CODE ---
 
-			const handleMessage = async (event) => {
-				// Basic security checks
-				if (
-					event.source !== authWindow ||
-					event.origin !== "https://trello.com" ||
-					!event.data
-				) {
-					return
-				}
+		const handleMessage = async (event) => {
+			// Basic security checks
+			if (
+				event.source !== authWindow ||
+				event.origin !== "https://trello.com" ||
+				!event.data
+			) {
+				return
+			}
 
-				const token = event.data
-				// Trello tokens are 64-char hex strings
-				if (token && /^[0-9a-f]{64}$/.test(token)) {
-					// --- ADDED FOR DEBUGGING ---
-					// Clean up both listeners once we have a valid token
-					window.removeEventListener("message", debugListener)
-					// --- END OF DEBUGGING CODE ---
+			const token = event.data
+			// Trello tokens are 64-char hex strings
+			if (token && /^[0-9a-f]{64}$/.test(token)) {
+				// --- ADDED FOR DEBUGGING ---
+				// Clean up both listeners once we have a valid token
+				window.removeEventListener("message", debugListener)
+				// --- END OF DEBUGGING CODE ---
 
-					window.removeEventListener("message", handleMessage)
-					authWindow.close()
+				window.removeEventListener("message", handleMessage)
+				authWindow.close()
 
-					setProcessingIntegration("trello")
-					try {
-						const response = await fetch(
-							"/api/settings/integrations/connect/manual",
-							{
-								method: "POST",
-								headers: { "Content-Type": "application/json" },
-								body: JSON.stringify({
-									service_name: "trello",
-									credentials: { token: token }
-								})
-							}
+				setProcessingIntegration("trello")
+				try {
+					const response = await fetch(
+						"/api/settings/integrations/connect/manual",
+						{
+							method: "POST",
+							headers: { "Content-Type": "application/json" },
+							body: JSON.stringify({
+								service_name: "trello",
+								credentials: { token: token }
+							})
+						}
+					)
+					if (!response.ok)
+						throw new Error(
+							(await response.json()).error ||
+								"Failed to save Trello token."
 						)
-						if (!response.ok)
-							throw new Error(
-								(await response.json()).error ||
-									"Failed to save Trello token."
-							)
-						toast.success("Trello connected successfully!")
-						fetchIntegrations()
-					} catch (error) {
-						toast.error(error.message)
-					} finally {
-						setProcessingIntegration(null)
-					}
+					toast.success("Trello connected successfully!")
+					fetchIntegrations()
+				} catch (error) {
+					toast.error(error.message)
+				} finally {
+					setProcessingIntegration(null)
 				}
 			}
-
-			window.addEventListener("message", handleMessage, false)
 		}
 
+		window.addEventListener("message", handleMessage, false)
+	}
+
 	const handleConnect = (integration) => {
 		if (integration.name === "whatsapp") {
 			setWhatsAppToConnect(integration)
@@ -671,6 +673,7 @@ const IntegrationsPage = () => {
 				)
 		} else if (integration.auth_type === "manual") {
 			if (MANUAL_INTEGRATION_CONFIGS[integration.name]) {
+				setActiveManualIntegration(integration)
 			} else {
 				toast.error(`UI for ${integration.display_name} not found.`)
 			}
@@ -1013,31 +1016,13 @@ const IntegrationsPage = () => {
 				</main>
 			</div>
 			<AnimatePresence>
-				{activeManualIntegration && (
-					<ManualTokenEntryModal
-						integration={activeManualIntegration}
-						onClose={() => setActiveManualIntegration(null)}
-						onSuccess={fetchIntegrations}
-					/>
-				)}
 				{whatsAppToConnect && (
 					<WhatsAppConnectModal
 						integration={whatsAppToConnect}
 						onClose={() => setWhatsAppToConnect(null)}
 						onSuccess={fetchIntegrations}
 					/>
 				)}
-				{selectedIntegration && (
-					<IntegrationDetailsModal
-						integration={selectedIntegration}
-						onClose={() => setSelectedIntegration(null)}
-						onConnect={handleConnect}
-						onDisconnect={handleDisconnect}
-						isProcessing={
-							processingIntegration === selectedIntegration?.name
-						}
-					/>
-				)}
 			</AnimatePresence>
 		</div>
 	)

src/client/app/api/settings/linkedin/route.js

@@ -46,10 +46,6 @@ const questionSections = {
 	context: {
 		title: "About You",
 		icon: <IconWorld />
-	},
-	personality: {
-		title: "AI Personality",
-		icon: <IconPalette />
 	}
 }
 
@@ -114,7 +110,7 @@ const questions = [
 			"e.g., I enjoy hiking on weekends, I'm learning to play the guitar...",
 		section: "context",
 		icon: <IconHeart />
-	},
+	}
 ]
 
 const WhatsAppSettings = () => {
@@ -1045,7 +1041,7 @@ const ProfilePage = () => {
 			/>
 			<div className="flex-1 flex flex-col overflow-hidden relative w-full md:pl-20 pb-16 md:pb-0">
 				<main className="flex-1 overflow-y-auto p-4 sm:p-6 md:p-10 custom-scrollbar">
-					<HelpTooltip content="Customize your experience here. Teach Sentient about yourself, change its personality, manage notifications, and connect your LinkedIn profile." />
+					<HelpTooltip content="Customize your experience here." />
 					<div className="w-full max-w-5xl mx-auto space-y-10">
 						<ProfileHeader />
 						<ProfileSettings