fix: critical bugs, security, console.log cleanup, tests, README

- Fix memory leaks in chat_stream_handlers and help_bot_handlers
- Remove hardcoded GitHub OAuth client ID
- Generate random Django SECRET_KEY instead of hardcoded placeholder
- Remove ~40 stray console.log from renderer and main process
- Fix unhandled promise rejection in app_handlers auto-install fallback
- Replace console.log with electron-log in process_manager, ipc_client, terminal_handlers, debug_handlers, start_proxy_server
- Fix key={index} misuse in ChatInput, MessagesList, home, Console, FileTree
- Add aria-labels and type=button to icon-only ChatInput buttons
- Update README: mark Roo Code Cloud done, add missing integrations
- Add unit tests for lib/utils, lib/assert, lib/schemas
- Unskip e2e chat_search tests

Author: SFARPak

README.md

@@ -81,10 +81,25 @@ AliFullStack is evolving rapidly. Here's what’s done and what’s coming soon:
 - [x] [Azure OpenAI](https://azure.microsoft.com/en-us/products/ai-services/openai-service)
 - [x] [xAI](https://x.ai)
 - [x] [AWS Bedrock](https://aws.amazon.com/bedrock/)
-- [ ] [Roo Code Cloud](https://roocode.com)
+- [x] [Roo Code Cloud](https://roocode.com)
 - [ ] Mistral _(planned)_
 - [ ] Cohere _(planned)_
 
+### 🧩 Local & Extended Providers
+
+- [x] [Ollama](https://ollama.ai) (local models)
+- [x] [LM Studio](https://lmstudio.ai) (local models)
+- [x] [NVIDIA NIM](https://build.nvidia.com) (direct OpenAI-compatible)
+- [x] Custom OpenAI-compatible providers
+
+### 🔌 Integrations
+
+- [x] [GitHub](https://github.com) (OAuth + push + repo management)
+- [x] [Vercel](https://vercel.com) (deploy + framework detection)
+- [x] [Supabase](https://supabase.com) (project management + edge functions)
+- [x] [Neon](https://neon.tech) (serverless Postgres)
+- [ ] AWS _(planned)_
+
 ### ✨ In Progress & Planned Features
 
 - [x] Full-stack project generation
@@ -96,6 +111,7 @@ AliFullStack is evolving rapidly. Here's what’s done and what’s coming soon:
   - [x] Self-improving development pipeline
 - [x] **Danger Zone Features** — Advanced app management tools
   - [x] Delete All Apps functionality
+  - [x] Reset Everything functionality
 - [ ] Drag-and-drop UI builder
 - [ ] Advanced code generation patterns
 - [ ] AI-assisted test and schema generation

e2e-tests/chat_search.spec.ts

@@ -1,6 +1,6 @@
 import { test } from "./helpers/test_helper";
 
-test.skip("chat search - basic search dialog functionality", async ({ po }) => {
+test("chat search - basic search dialog functionality", async ({ po }) => {
   await po.setUp({ autoApprove: true });
   await po.importApp("minimal");
 
@@ -41,7 +41,7 @@ test.skip("chat search - basic search dialog functionality", async ({ po }) => {
   await po.page.keyboard.press("Escape");
 });
 
-test.skip("chat search - with named chats for easier testing", async ({
+test("chat search - with named chats for easier testing", async ({
   po,
 }) => {
   await po.setUp({ autoApprove: true });
@@ -85,7 +85,7 @@ test.skip("chat search - with named chats for easier testing", async ({
   await po.page.keyboard.press("Escape");
 });
 
-test.skip("chat search - keyboard shortcut functionality", async ({ po }) => {
+test("chat search - keyboard shortcut functionality", async ({ po }) => {
   await po.setUp({ autoApprove: true });
   await po.importApp("minimal");
 
@@ -102,7 +102,7 @@ test.skip("chat search - keyboard shortcut functionality", async ({ po }) => {
   await po.page.getByTestId("chat-search-dialog").waitFor({ state: "hidden" });
 });
 
-test.skip("chat search - navigation and selection", async ({ po }) => {
+test("chat search - navigation and selection", async ({ po }) => {
   await po.setUp({ autoApprove: true });
   await po.importApp("minimal");
 

src/__tests__/assert.test.ts

@@ -0,0 +1,32 @@
+import { describe, it, expect } from "vitest";
+import { assertExists } from "./assert";
+
+describe("assertExists", () => {
+  it("does not throw for a defined value", () => {
+    expect(() => assertExists("hello", "should exist")).not.toThrow();
+    expect(() => assertExists(0, "should exist")).not.toThrow();
+    expect(() => assertExists(false, "should exist")).not.toThrow();
+    expect(() => assertExists("", "should exist")).not.toThrow();
+  });
+
+  it("throws for undefined", () => {
+    expect(() => assertExists(undefined, "missing value")).toThrow(
+      "missing value",
+    );
+  });
+
+  it("throws for null", () => {
+    expect(() => assertExists(null, "missing value")).toThrow("missing value");
+  });
+
+  it("throws with the provided message", () => {
+    const message = "custom error message";
+    try {
+      assertExists(undefined, message);
+      expect.fail("should have thrown");
+    } catch (e) {
+      expect(e).toBeInstanceOf(Error);
+      expect((e as Error).message).toBe(message);
+    }
+  });
+});

src/__tests__/schemas.test.ts

@@ -0,0 +1,95 @@
+import { describe, it, expect } from "vitest";
+import { z } from "zod";
+import {
+  isAliFullStackProEnabled,
+  hasAliFullStackProKey,
+  ChatSummarySchema,
+  AppSearchResultSchema,
+} from "./schemas";
+
+describe("isAliFullStackProEnabled", () => {
+  it("returns false when enableAliFullStackPro is false", () => {
+    const settings = {
+      enableAliFullStackPro: false,
+      providerSettings: { auto: { apiKey: { value: "key" } } },
+    } as any;
+    expect(isAliFullStackProEnabled(settings)).toBe(false);
+  });
+
+  it("returns false when enableAliFullStackPro is undefined", () => {
+    const settings = {
+      providerSettings: { auto: { apiKey: { value: "key" } } },
+    } as any;
+    expect(isAliFullStackProEnabled(settings)).toBe(false);
+  });
+
+  it("returns false when apiKey is missing", () => {
+    const settings = {
+      enableAliFullStackPro: true,
+      providerSettings: { auto: { apiKey: {} } },
+    } as any;
+    expect(isAliFullStackProEnabled(settings)).toBe(false);
+  });
+
+  it("returns true when enabled and apiKey exists", () => {
+    const settings = {
+      enableAliFullStackPro: true,
+      providerSettings: { auto: { apiKey: { value: "key" } } },
+    } as any;
+    expect(isAliFullStackProEnabled(settings)).toBe(true);
+  });
+});
+
+describe("hasAliFullStackProKey", () => {
+  it("returns true when apiKey value is present", () => {
+    const settings = {
+      providerSettings: { auto: { apiKey: { value: "key" } } },
+    } as any;
+    expect(hasAliFullStackProKey(settings)).toBe(true);
+  });
+
+  it("returns false when providerSettings is missing", () => {
+    const settings = {} as any;
+    expect(hasAliFullStackProKey(settings)).toBe(false);
+  });
+
+  it("returns false when apiKey is empty string", () => {
+    const settings = {
+      providerSettings: { auto: { apiKey: { value: "" } } },
+    } as any;
+    expect(hasAliFullStackProKey(settings)).toBe(false);
+  });
+});
+
+describe("Zod schemas", () => {
+  it("ChatSummarySchema parses valid data", () => {
+    const data = {
+      id: 1,
+      appId: 2,
+      title: "Test Chat",
+      createdAt: new Date(),
+    };
+    expect(() => ChatSummarySchema.parse(data)).not.toThrow();
+  });
+
+  it("ChatSummarySchema rejects missing id", () => {
+    expect(() =>
+      ChatSummarySchema.parse({
+        appId: 2,
+        title: "Test",
+        createdAt: new Date(),
+      }),
+    ).toThrow();
+  });
+
+  it("AppSearchResultSchema parses valid data", () => {
+    const data = {
+      id: 1,
+      name: "My App",
+      createdAt: new Date(),
+      matchedChatTitle: null,
+      matchedChatMessage: null,
+    };
+    expect(() => AppSearchResultSchema.parse(data)).not.toThrow();
+  });
+});

src/__tests__/utils.test.ts

@@ -0,0 +1,39 @@
+import { describe, it, expect } from "vitest";
+import { generateCuteAppName, cn } from "./utils";
+
+describe("generateCuteAppName", () => {
+  it("returns a non-empty string", () => {
+    const name = generateCuteAppName();
+    expect(name).toBeTruthy();
+    expect(typeof name).toBe("string");
+  });
+
+  it("returns a name with at least two words separated by a space", () => {
+    const name = generateCuteAppName();
+    const parts = name.split(" ");
+    expect(parts.length).toBeGreaterThanOrEqual(2);
+  });
+
+  it("returns different names on successive calls (with high probability)", () => {
+    const names = new Set<string>();
+    for (let i = 0; i < 50; i++) {
+      names.add(generateCuteAppName());
+    }
+    // With 50 draws from a large pool, collisions should be rare
+    expect(names.size).toBeGreaterThan(40);
+  });
+});
+
+describe("cn", () => {
+  it("merges class names", () => {
+    expect(cn("foo", "bar")).toBe("foo bar");
+  });
+
+  it("handles undefined and null inputs", () => {
+    expect(cn("foo", undefined, null, "bar")).toBe("foo bar");
+  });
+
+  it("resolves conflicting Tailwind classes with tailwind-merge", () => {
+    expect(cn("px-2", "px-4")).toBe("px-4");
+  });
+});

src/client_logic/template_hook.ts

@@ -9,13 +9,11 @@ export async function neonTemplateHook({
   appId: number;
   appName: string;
 }) {
-  console.log("Creating Neon project");
   const neonProject = await IpcClient.getInstance().createNeonProject({
     name: appName,
     appId: appId,
   });
 
-  console.log("Neon project created", neonProject);
   await IpcClient.getInstance().setAppEnvVars({
     appId: appId,
     envVars: [
@@ -41,5 +39,4 @@ export async function neonTemplateHook({
       },
     ],
   });
-  console.log("App env vars set");
 }

src/components/ChatList.tsx

@@ -57,8 +57,7 @@ export function ChatList({ show }: { show?: boolean }) {
     if (isChatRoute) {
       const id = routerState.location.search.id;
       if (id) {
-        console.log("Setting selected chat id to", id);
-        setSelectedChatId(id);
+         setSelectedChatId(id);
       }
     }
   }, [isChatRoute, routerState.location.search, setSelectedChatId]);

src/components/ChatPanel.tsx

@@ -44,15 +44,6 @@ export function ChatPanel({
   // State to track system messages for auto-scrolling
   const [systemMessageCount, setSystemMessageCount] = useState(0);
 
-  // Debug logging
-  console.log("ChatPanel render:", {
-    isBackendMode,
-    isFullstackMode,
-    isFrontendMode,
-    isTodoPanelOpen,
-    isStepsPanelOpen,
-    showTodoToggle: isFullstackMode || isFrontendMode,
-  });
   // Reference to store the processed prompt so we don't submit it twice
 
   const messagesEndRef = useRef<HTMLDivElement | null>(null);
@@ -89,7 +80,6 @@ export function ChatPanel({
   };
 
   useEffect(() => {
-    console.log("streamCount", streamCount);
     scrollToBottom();
   }, [streamCount]);
 

src/components/TodoListPanel.tsx

@@ -24,16 +24,8 @@ export function TodoListPanel({ isOpen, onClose }: TodoListPanelProps) {
   const [editTitle, setEditTitle] = useState("");
   const [editDescription, setEditDescription] = useState("");
 
-  // Debug logging
-  console.log("TodoListPanel render:", {
-    isOpen,
-    chatId,
-    todosCount: todos.length,
-  });
-
   // Filter todos for current chat
   const currentChatTodos = todos.filter((todo) => todo.chatId === chatId);
-  console.log("Current chat todos:", currentChatTodos.length);
 
   const addTodo = () => {
     if (!newTodoTitle.trim() || !chatId) return;

src/components/backend-chat/BackendChatPanel.tsx

@@ -31,9 +31,6 @@ export function BackendChatPanel({
   const [error, setError] = useState<string | null>(null);
   const streamCount = useAtomValue(chatStreamCountAtom);
 
-  // Debug logging
-  console.log("BackendChatPanel render:", { isTodoPanelOpen, isStepsPanelOpen });
-
   const messagesEndRef = useRef<HTMLDivElement>(null);
   const messagesContainerRef = useRef<HTMLDivElement | null>(null);
 
@@ -68,7 +65,6 @@ export function BackendChatPanel({
   };
 
   useEffect(() => {
-    console.log("streamCount", streamCount);
     scrollToBottom();
   }, [streamCount]);