fix: clear setTimeout on useEffect cleanup (issue #1095)

- Fixed setTimeout cleanup for copy-feedback timer
- Ensures timer is properly cleared when component unmounts
- Moves cleanup return outside conditional to guarantee cleanup runs

surfsense_web/hooks/use-api-key.ts

@@ -17,10 +17,10 @@ export function useApiKey(): UseApiKeyReturn {
 
 	// Clear copied state after 2 seconds
 	useEffect(() => {
-		if (copied) {
-			const timer = setTimeout(() => setCopied(false), 2000);
-			return () => clearTimeout(timer);
-		}
+		if (!copied) return;
+
+		const timer = setTimeout(() => setCopied(false), 2000);
+		return () => clearTimeout(timer);
 	}, [copied]);
 
 	useEffect(() => {