fix: address PR #55 review feedback

- Remove unused .project-list-card-clickable-content CSS class
- Reduce !important usage in schema select styles, increase specificity instead
- Replace duplicate API call with explorerService.setProjectSchema
- Replace message.success/error with useNotificationService for consistency
- Show error InfoChip when schema list is empty instead of hiding dropdown
- Revert project_schema max_length change (will be added to PR #54 with migration)

Author: wicky-zipstack

backend/backend/core/models/project_details.py

@@ -102,7 +102,7 @@ def delete(self, *args, **kwargs):
     project_path = models.CharField(max_length=100)
     profile_path = models.CharField(max_length=100)
 
-    project_schema = models.CharField(max_length=1024, blank=True, null=True)
+    project_schema = models.CharField(max_length=20, blank=True, null=True)
     # User specific access control fields
     created_by = models.JSONField(default=dict)
     created_at = models.DateTimeField(auto_now_add=True)

frontend/src/base/components/project-list/ProjectListCard.css

@@ -171,10 +171,6 @@
 }
 
 /* ---------- body ---------- */
-.project-list-card-clickable-content {
-  /* clickable via parent wrapper */
-}
-
 .project-list-card-detail-section {
   padding: 12px 14px;
 }

frontend/src/ide/chat-ai/ChatAI.css

@@ -443,24 +443,25 @@
 .chat-ai-info-chip:has(.chat-ai-schema-select) {
   padding: 3px 4px 3px 10px;
 }
-.chat-ai-schema-select {
+/* Schema select inside info chip — !important needed to override Ant Design inline styles */
+.chat-ai-info-chip .chat-ai-schema-select {
   max-width: 180px;
 }
-.chat-ai-schema-select.ant-select {
+.chat-ai-info-chip .chat-ai-schema-select.ant-select {
   height: 16px !important;
 }
-.chat-ai-schema-select .ant-select-selector {
+.chat-ai-info-chip .chat-ai-schema-select .ant-select-selector {
   padding: 0 !important;
   height: 16px !important;
   min-height: unset !important;
 }
-.chat-ai-schema-select .ant-select-selection-item {
-  font-size: 10px !important;
-  line-height: 16px !important;
-  padding-inline-end: 14px !important;
+.chat-ai-info-chip .chat-ai-schema-select .ant-select-selection-item {
+  font-size: 10px;
+  line-height: 16px;
+  padding-inline-end: 14px;
 }
-.chat-ai-schema-select .ant-select-arrow {
-  font-size: 8px !important;
+.chat-ai-info-chip .chat-ai-schema-select .ant-select-arrow {
+  font-size: 8px;
   right: 0;
   inset-block-start: 50%;
 }

frontend/src/ide/chat-ai/PromptActions.jsx

@@ -1,13 +1,5 @@
 import { memo, useCallback, useEffect, useMemo } from "react";
-import {
-  Space,
-  Typography,
-  Select,
-  Switch,
-  Segmented,
-  Tooltip,
-  message,
-} from "antd";
+import { Space, Typography, Select, Switch, Segmented, Tooltip } from "antd";
 import {
   ConsoleSqlOutlined,
   DatabaseOutlined,
@@ -16,15 +8,15 @@ import {
   WalletOutlined,
 } from "@ant-design/icons";
 import PropTypes from "prop-types";
-import Cookies from "js-cookie";
 
 import { CHAT_INTENTS } from "./helper";
 import CircularTokenDisplay from "./CircularTokenDisplay";
+import InfoChip from "./InfoChip";
 import { useTokenStore } from "../../store/token-store";
 import { useSessionStore } from "../../store/session-store";
 import { useProjectStore } from "../../store/project-store";
-import { orgStore } from "../../store/org-store";
-import { useAxiosPrivate } from "../../service/axios-service";
+import { explorerService } from "../explorer/explorer-service";
+import { useNotificationService } from "../../service/notification-service";
 
 // Define hidden intents and a fixed order array
 const HIDDEN_CHAT_INTENTS = ["AUTO", "NOTA", "INFO"];
@@ -66,8 +58,8 @@ const PromptActions = memo(function PromptActions({
   const setCurrentSchema = useProjectStore((state) => state.setCurrentSchema);
   const schemaList = useProjectStore((state) => state.schemaList);
   const projectId = useProjectStore((state) => state.projectId);
-  const { selectedOrgId } = orgStore();
-  const axios = useAxiosPrivate();
+  const expService = explorerService();
+  const { notify } = useNotificationService();
 
   const schemaOptions = useMemo(
     () => schemaList.map((s) => ({ label: s, value: s })),
@@ -76,25 +68,18 @@ const PromptActions = memo(function PromptActions({
 
   const handleSchemaChange = useCallback(
     (value) => {
-      const csrfToken = Cookies.get("csrftoken");
-      axios({
-        url: `/api/v1/visitran/${
-          selectedOrgId || "default_org"
-        }/project/${projectId}/set_schema`,
-        method: "POST",
-        data: { schema_name: value },
-        headers: { "X-CSRFToken": csrfToken },
-      })
+      expService
+        .setProjectSchema(projectId, value)
         .then(() => {
           setCurrentSchema(value);
-          message.success("Schema updated successfully");
+          notify({ type: "success", message: "Schema updated successfully" });
         })
         .catch((error) => {
           console.error(error);
-          message.error("Failed to update schema");
+          notify({ error });
         });
     },
-    [axios, selectedOrgId, projectId, setCurrentSchema]
+    [expService, projectId, setCurrentSchema, notify]
   );
 
   const llmOptions = useMemo(
@@ -213,7 +198,7 @@ const PromptActions = memo(function PromptActions({
         )}
 
         {/* Schema selector */}
-        {schemaList.length > 0 && (
+        {schemaList.length > 0 ? (
           <div className="chat-ai-info-chip chat-ai-info-chip-clickable">
             <DatabaseOutlined className="chat-ai-info-chip-icon" />
             <Select
@@ -228,6 +213,13 @@ const PromptActions = memo(function PromptActions({
               className="chat-ai-schema-select"
             />
           </div>
+        ) : (
+          <InfoChip
+            icon={<DatabaseOutlined className="chat-ai-info-chip-icon" />}
+            text="No schema"
+            tooltipTitle="No schemas available. Please configure a database connection and select a schema from the explorer."
+            className="chat-ai-info-chip-error"
+          />
         )}
       </Space>