gaard-api 0.2.12__tar.gz → 0.2.14__tar.gz

{gaard_api-0.2.12/src/gaard_api.egg-info → gaard_api-0.2.14}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gaard-api
-Version: 0.2.12
+Version: 0.2.14
 Summary: GAARD backend web services providing admin interface
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{gaard_api-0.2.12 → gaard_api-0.2.14}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "gaard-api"
-version = "0.2.12"
+version = "0.2.14"
 description = "GAARD backend web services providing admin interface"
 readme = "README.md"
 requires-python = ">=3.11"

{gaard_api-0.2.12 → gaard_api-0.2.14}/src/gaard_api/admin/database_initial.sql

@@ -16,6 +16,7 @@ CREATE UNIQUE INDEX IF NOT EXISTS ix_conversation_turns_turn_id ON conversation_
 CREATE TABLE IF NOT EXISTS conversations (id INTEGER NOT NULL, conversation_id VARCHAR(64) NOT NULL, owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, status VARCHAR(50) NOT NULL, datasource_id VARCHAR(255) NOT NULL, datasource_ids_json TEXT NOT NULL, title VARCHAR(255) NOT NULL, summary_json TEXT NOT NULL, created_at DATETIME NOT NULL, updated_at DATETIME NOT NULL, PRIMARY KEY (id));
 CREATE UNIQUE INDEX IF NOT EXISTS ix_conversations_conversation_id ON conversations (conversation_id);
 CREATE TABLE IF NOT EXISTS dashboard_user_states (owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, active_dashboard_id VARCHAR(64) NOT NULL, created_at DATETIME NOT NULL, updated_at DATETIME NOT NULL, PRIMARY KEY (owner_user_id));
+CREATE TABLE IF NOT EXISTS dashboard_shares (id INTEGER NOT NULL, dashboard_id VARCHAR(64) NOT NULL, target_user_id VARCHAR(255) NOT NULL, target_username VARCHAR(255) NOT NULL, access_level VARCHAR(20) NOT NULL, created_by_user_id VARCHAR(255) NOT NULL, created_by_username VARCHAR(255) NOT NULL, created_at DATETIME NOT NULL, updated_at DATETIME NOT NULL, PRIMARY KEY (id), CONSTRAINT uq_dashboard_shares_dashboard_target UNIQUE (dashboard_id, target_user_id));
 CREATE TABLE IF NOT EXISTS dashboard_widgets (id INTEGER NOT NULL, widget_id VARCHAR(64) NOT NULL, dashboard_id VARCHAR(64) NOT NULL, owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, metric_widget_key VARCHAR(255) NOT NULL, title VARCHAR(255) NOT NULL, visualization_type VARCHAR(50) NOT NULL, x INTEGER NOT NULL, y INTEGER NOT NULL, w INTEGER NOT NULL, h INTEGER NOT NULL, created_at DATETIME NOT NULL, updated_at DATETIME NOT NULL, PRIMARY KEY (id), CONSTRAINT uq_dashboard_widgets_widget_id UNIQUE (widget_id));
 CREATE TABLE IF NOT EXISTS dashboards (id INTEGER NOT NULL, dashboard_id VARCHAR(64) NOT NULL, owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, name VARCHAR(255) NOT NULL, description TEXT NOT NULL, created_at DATETIME NOT NULL, updated_at DATETIME NOT NULL, PRIMARY KEY (id));
 CREATE UNIQUE INDEX IF NOT EXISTS ix_dashboards_dashboard_id ON dashboards (dashboard_id);
@@ -63,6 +64,12 @@ CREATE INDEX IF NOT EXISTS ix_conversations_owner_username ON conversations (own
 CREATE INDEX IF NOT EXISTS ix_conversations_status ON conversations (status);
 CREATE INDEX IF NOT EXISTS ix_dashboard_user_states_active_dashboard_id ON dashboard_user_states (active_dashboard_id);
 CREATE INDEX IF NOT EXISTS ix_dashboard_user_states_owner_username ON dashboard_user_states (owner_username);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_access_level ON dashboard_shares (access_level);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_created_by_user_id ON dashboard_shares (created_by_user_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_created_by_username ON dashboard_shares (created_by_username);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_dashboard_id ON dashboard_shares (dashboard_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_target_user_id ON dashboard_shares (target_user_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_target_username ON dashboard_shares (target_username);
 CREATE INDEX IF NOT EXISTS ix_dashboard_widgets_dashboard_id ON dashboard_widgets (dashboard_id);
 CREATE INDEX IF NOT EXISTS ix_dashboard_widgets_metric_widget_key ON dashboard_widgets (metric_widget_key);
 CREATE INDEX IF NOT EXISTS ix_dashboard_widgets_owner_user_id ON dashboard_widgets (owner_user_id);
@@ -97,6 +104,7 @@ CREATE UNIQUE INDEX IF NOT EXISTS ix_conversation_turns_turn_id ON conversation_
 CREATE TABLE IF NOT EXISTS conversations (id SERIAL NOT NULL, conversation_id VARCHAR(64) NOT NULL, owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, status VARCHAR(50) NOT NULL, datasource_id VARCHAR(255) NOT NULL, datasource_ids_json TEXT NOT NULL, title VARCHAR(255) NOT NULL, summary_json TEXT NOT NULL, created_at TIMESTAMP WITH TIME ZONE NOT NULL, updated_at TIMESTAMP WITH TIME ZONE NOT NULL, PRIMARY KEY (id));
 CREATE UNIQUE INDEX IF NOT EXISTS ix_conversations_conversation_id ON conversations (conversation_id);
 CREATE TABLE IF NOT EXISTS dashboard_user_states (owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, active_dashboard_id VARCHAR(64) NOT NULL, created_at TIMESTAMP WITH TIME ZONE NOT NULL, updated_at TIMESTAMP WITH TIME ZONE NOT NULL, PRIMARY KEY (owner_user_id));
+CREATE TABLE IF NOT EXISTS dashboard_shares (id SERIAL NOT NULL, dashboard_id VARCHAR(64) NOT NULL, target_user_id VARCHAR(255) NOT NULL, target_username VARCHAR(255) NOT NULL, access_level VARCHAR(20) NOT NULL, created_by_user_id VARCHAR(255) NOT NULL, created_by_username VARCHAR(255) NOT NULL, created_at TIMESTAMP WITH TIME ZONE NOT NULL, updated_at TIMESTAMP WITH TIME ZONE NOT NULL, PRIMARY KEY (id), CONSTRAINT uq_dashboard_shares_dashboard_target UNIQUE (dashboard_id, target_user_id));
 CREATE TABLE IF NOT EXISTS dashboard_widgets (id SERIAL NOT NULL, widget_id VARCHAR(64) NOT NULL, dashboard_id VARCHAR(64) NOT NULL, owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, metric_widget_key VARCHAR(255) NOT NULL, title VARCHAR(255) NOT NULL, visualization_type VARCHAR(50) NOT NULL, x INTEGER NOT NULL, y INTEGER NOT NULL, w INTEGER NOT NULL, h INTEGER NOT NULL, created_at TIMESTAMP WITH TIME ZONE NOT NULL, updated_at TIMESTAMP WITH TIME ZONE NOT NULL, PRIMARY KEY (id), CONSTRAINT uq_dashboard_widgets_widget_id UNIQUE (widget_id));
 CREATE TABLE IF NOT EXISTS dashboards (id SERIAL NOT NULL, dashboard_id VARCHAR(64) NOT NULL, owner_user_id VARCHAR(255) NOT NULL, owner_username VARCHAR(255) NOT NULL, name VARCHAR(255) NOT NULL, description TEXT NOT NULL, created_at TIMESTAMP WITH TIME ZONE NOT NULL, updated_at TIMESTAMP WITH TIME ZONE NOT NULL, PRIMARY KEY (id));
 CREATE UNIQUE INDEX IF NOT EXISTS ix_dashboards_dashboard_id ON dashboards (dashboard_id);
@@ -144,6 +152,12 @@ CREATE INDEX IF NOT EXISTS ix_conversations_owner_username ON conversations (own
 CREATE INDEX IF NOT EXISTS ix_conversations_status ON conversations (status);
 CREATE INDEX IF NOT EXISTS ix_dashboard_user_states_active_dashboard_id ON dashboard_user_states (active_dashboard_id);
 CREATE INDEX IF NOT EXISTS ix_dashboard_user_states_owner_username ON dashboard_user_states (owner_username);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_access_level ON dashboard_shares (access_level);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_created_by_user_id ON dashboard_shares (created_by_user_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_created_by_username ON dashboard_shares (created_by_username);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_dashboard_id ON dashboard_shares (dashboard_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_target_user_id ON dashboard_shares (target_user_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_target_username ON dashboard_shares (target_username);
 CREATE INDEX IF NOT EXISTS ix_dashboard_widgets_dashboard_id ON dashboard_widgets (dashboard_id);
 CREATE INDEX IF NOT EXISTS ix_dashboard_widgets_metric_widget_key ON dashboard_widgets (metric_widget_key);
 CREATE INDEX IF NOT EXISTS ix_dashboard_widgets_owner_user_id ON dashboard_widgets (owner_user_id);

{gaard_api-0.2.12 → gaard_api-0.2.14}/src/gaard_api/admin/database_updates.sql

@@ -8,3 +8,16 @@ UPDATE identity_privilege_datasource_permissions AS permission SET identity_id =
 -- table: identity_privilege_table_permissions
 DELETE FROM identity_privilege_table_permissions AS legacy WHERE EXISTS (SELECT 1 FROM admin_users AS mapped_user WHERE legacy.identity_id IN ('local:' || CAST(mapped_user.id AS VARCHAR(255)), mapped_user.auth_provider || ':' || mapped_user.username) AND EXISTS (SELECT 1 FROM identity_privilege_table_permissions AS duplicate WHERE duplicate.connector_id = legacy.connector_id AND duplicate.table_name = legacy.table_name AND duplicate.id <> legacy.id AND (duplicate.identity_id = CAST(mapped_user.id AS VARCHAR(255)) OR (duplicate.id < legacy.id AND duplicate.identity_id IN ('local:' || CAST(mapped_user.id AS VARCHAR(255)), mapped_user.auth_provider || ':' || mapped_user.username)))));
 UPDATE identity_privilege_table_permissions AS permission SET identity_id = (SELECT CAST(mapped_user.id AS VARCHAR(255)) FROM admin_users AS mapped_user WHERE permission.identity_id IN ('local:' || CAST(mapped_user.id AS VARCHAR(255)), mapped_user.auth_provider || ':' || mapped_user.username) ORDER BY mapped_user.id LIMIT 1) WHERE EXISTS (SELECT 1 FROM admin_users AS mapped_user WHERE permission.identity_id IN ('local:' || CAST(mapped_user.id AS VARCHAR(255)), mapped_user.auth_provider || ':' || mapped_user.username));
+
+-- tag: 2026-08-01.dashboard-sharing.v1
+-- dialect: sqlite
+CREATE TABLE IF NOT EXISTS dashboard_shares (id INTEGER NOT NULL, dashboard_id VARCHAR(64) NOT NULL, target_user_id VARCHAR(255) NOT NULL, target_username VARCHAR(255) NOT NULL, access_level VARCHAR(20) NOT NULL, created_by_user_id VARCHAR(255) NOT NULL, created_by_username VARCHAR(255) NOT NULL, created_at DATETIME NOT NULL, updated_at DATETIME NOT NULL, PRIMARY KEY (id), CONSTRAINT uq_dashboard_shares_dashboard_target UNIQUE (dashboard_id, target_user_id));
+-- dialect: postgresql
+CREATE TABLE IF NOT EXISTS dashboard_shares (id SERIAL NOT NULL, dashboard_id VARCHAR(64) NOT NULL, target_user_id VARCHAR(255) NOT NULL, target_username VARCHAR(255) NOT NULL, access_level VARCHAR(20) NOT NULL, created_by_user_id VARCHAR(255) NOT NULL, created_by_username VARCHAR(255) NOT NULL, created_at TIMESTAMP WITH TIME ZONE NOT NULL, updated_at TIMESTAMP WITH TIME ZONE NOT NULL, PRIMARY KEY (id), CONSTRAINT uq_dashboard_shares_dashboard_target UNIQUE (dashboard_id, target_user_id));
+-- dialect: all
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_access_level ON dashboard_shares (access_level);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_created_by_user_id ON dashboard_shares (created_by_user_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_created_by_username ON dashboard_shares (created_by_username);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_dashboard_id ON dashboard_shares (dashboard_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_target_user_id ON dashboard_shares (target_user_id);
+CREATE INDEX IF NOT EXISTS ix_dashboard_shares_target_username ON dashboard_shares (target_username);

{gaard_api-0.2.12 → gaard_api-0.2.14}/src/gaard_api/admin/models.py

@@ -209,6 +209,31 @@ class DashboardUserState(Base):
     )
 
 
+class DashboardShare(Base):
+    __tablename__ = "dashboard_shares"
+    __table_args__ = (
+        UniqueConstraint(
+            "dashboard_id",
+            "target_user_id",
+            name="uq_dashboard_shares_dashboard_target",
+        ),
+    )
+
+    id: Mapped[int] = mapped_column(Integer, primary_key=True)
+    dashboard_id: Mapped[str] = mapped_column(String(64), index=True)
+    target_user_id: Mapped[str] = mapped_column(String(255), index=True)
+    target_username: Mapped[str] = mapped_column(String(255), index=True, default="")
+    access_level: Mapped[str] = mapped_column(String(20), index=True)
+    created_by_user_id: Mapped[str] = mapped_column(String(255), index=True)
+    created_by_username: Mapped[str] = mapped_column(String(255), index=True, default="")
+    created_at: Mapped[datetime] = mapped_column(DateTime(timezone=True), default=utc_now)
+    updated_at: Mapped[datetime] = mapped_column(
+        DateTime(timezone=True),
+        default=utc_now,
+        onupdate=utc_now,
+    )
+
+
 class UserDatasourceSelection(Base):
     """Client datasource selections, scoped to one authenticated user."""
 

{gaard_api-0.2.12 → gaard_api-0.2.14}/src/gaard_api/admin/services.py

@@ -2229,6 +2229,7 @@ def build_table_settings(
         tables[table.name] = {
             "selected": bool(existing.get("selected", True)),
             "description": str(existing.get("description", "")),
+            "view_description": str(existing.get("view_description", "")),
             "primary_key_prompt": str(existing.get("primary_key_prompt", "")),
             "foreign_key_prompt": str(existing.get("foreign_key_prompt", "")),
             "join_logic": str(existing.get("join_logic", "")),

{gaard_api-0.2.12 → gaard_api-0.2.14}/src/gaard_api/admin-web/assets/main.js

@@ -100,7 +100,7 @@ var state = {
     loadedName: "",
     loadedSql: "",
     name: "",
-    description: "",
+    view_description: "",
     sql: "",
     error: "",
     success: "",
@@ -1926,7 +1926,7 @@ function renderDatasourceViewDialog() {
         </div>
         <form id="datasource-view-form" class="form-grid">
           <label>View name<input name="name" value="${escapeHtml(dialog.name || "")}" autocomplete="off" /></label>
-          <label>View description<textarea name="description">${escapeHtml(dialog.description || "")}</textarea></label>
+          <label>View description<textarea name="view_description">${escapeHtml(dialog.view_description || "")}</textarea></label>
           <label>SQL<textarea name="sql" class="textarea-small" spellcheck="false">${escapeHtml(dialog.sql || "")}</textarea></label>
           ${busy === "load" ? `<p class="muted"><span class="spinner" aria-hidden="true"></span> Loading view definition.</p>` : ""}
           ${dialog.error ? `<p class="error">${escapeHtml(dialog.error)}</p>` : ""}
@@ -1976,6 +1976,7 @@ function getDatasourceSchemaDraftTables(rawTables, tableSettings) {
     state.datasourceSchemaDraftTables[table.name] = {
       selected: settings.selected !== false,
       description: settings.description || "",
+      view_description: settings.view_description || "",
       primary_key_prompt: settings.primary_key_prompt || "",
       foreign_key_prompt: settings.foreign_key_prompt || "",
       join_logic: settings.join_logic || ""
@@ -3340,7 +3341,7 @@ function defaultDatasourceViewDialog() {
     loadedName: "",
     loadedSql: "",
     name: "",
-    description: "",
+    view_description: "",
     sql: "",
     error: "",
     success: "",
@@ -3367,8 +3368,7 @@ async function openEditDatasourceViewDialog(event) {
   const rawTables = state.datasourceSchema?.item?.raw_schema?.tables || [];
   const tableSettings = state.datasourceSchema?.item?.table_settings?.tables || {};
   const draftSettings = getDatasourceSchemaDraftTables(rawTables, tableSettings)[viewName] || {};
-  const hasDraftDescription = Object.prototype.hasOwnProperty.call(draftSettings, "description");
-  const draftDescription = hasDraftDescription ? draftSettings.description || "" : "";
+  const draftViewDescription = draftSettings.view_description || "";
   state.datasourceViewDialog = {
     ...defaultDatasourceViewDialog(),
     open: true,
@@ -3376,14 +3376,14 @@ async function openEditDatasourceViewDialog(event) {
     originalName: viewName,
     loadedName: viewName,
     name: viewName,
-    description: draftDescription,
+    view_description: draftViewDescription,
     busy: "load"
   };
   render();
   try {
     const result = await api(`/api/v1/admin/datasources/${selected.id}/schema/views/${encodeURIComponent(viewName)}`);
     const item = result.item || {};
-    const description = hasDraftDescription ? draftDescription : item.description || "";
+    const viewDescription = draftViewDescription || item.view_description || item.description || "";
     state.datasourceViewDialog = {
       ...state.datasourceViewDialog,
       mode: "edit",
@@ -3391,7 +3391,7 @@ async function openEditDatasourceViewDialog(event) {
       loadedName: item.name || viewName,
       loadedSql: item.sql || "",
       name: item.name || viewName,
-      description,
+      view_description: viewDescription,
       sql: item.sql || "",
       busy: "",
       error: "",
@@ -3420,14 +3420,14 @@ function readDatasourceViewDialogPayload() {
     const dialog = state.datasourceViewDialog || {};
     return {
       name: String(dialog.name || "").trim(),
-      description: String(dialog.description || ""),
+      view_description: String(dialog.view_description || ""),
       sql: String(dialog.sql || "")
     };
   }
   const data = new FormData(form);
   return {
     name: String(data.get("name") || "").trim(),
-    description: String(data.get("description") || ""),
+    view_description: String(data.get("view_description") || ""),
     sql: String(data.get("sql") || "")
   };
 }
@@ -3464,7 +3464,7 @@ async function generateDatasourceViewSql() {
   try {
     const result = await api(`/api/v1/admin/datasources/${selected.id}/schema/views/generate-sql`, {
       method: "POST",
-      body: JSON.stringify({ name: payload.name, description: payload.description })
+      body: JSON.stringify({ name: payload.name, view_description: payload.view_description })
     });
     state.datasourceViewDialog = {
       ...state.datasourceViewDialog,
@@ -3546,7 +3546,7 @@ async function saveDatasourceView(event) {
     const result = canSaveSettingsOnly
       ? await api(`/api/v1/admin/datasources/${selected.id}/schema/views/${encodeURIComponent(dialog.originalName || payload.name)}`, {
           method: "PUT",
-          body: JSON.stringify({ description: payload.description })
+          body: JSON.stringify({ view_description: payload.view_description })
         })
       : dialog.mode === "edit"
         ? await api(`/api/v1/admin/datasources/${selected.id}/schema/views/${encodeURIComponent(dialog.originalName || payload.name)}/execute`, {
@@ -3689,6 +3689,7 @@ async function saveDatasourceSchema(event) {
     tables[table.name] = {
       selected: draft.selected !== false,
       description: draft.description || "",
+      view_description: draft.view_description || "",
       primary_key_prompt: draft.primary_key_prompt || "",
       foreign_key_prompt: draft.foreign_key_prompt || "",
       join_logic: draft.join_logic || ""

{gaard_api-0.2.12 → gaard_api-0.2.14}/src/gaard_api/api/v1/admin.py

@@ -363,22 +363,33 @@ class DatasourceSchemaTableSettingsRequest(BaseModel):
 class DatasourceViewSqlGenerationRequest(BaseModel):
     name: str = Field(min_length=1, max_length=128)
     description: str = Field(default="", max_length=20000)
+    view_description: str | None = Field(default=None, max_length=20000)
 
 
 class DatasourceViewExecuteRequest(BaseModel):
     name: str = Field(min_length=1, max_length=128)
     description: str = Field(default="", max_length=20000)
+    view_description: str | None = Field(default=None, max_length=20000)
     sql: str = Field(min_length=1, max_length=200000)
 
 
 class DatasourceViewUpdateRequest(BaseModel):
     name: str = Field(min_length=1, max_length=128)
     description: str = Field(default="", max_length=20000)
+    view_description: str | None = Field(default=None, max_length=20000)
     sql: str = Field(min_length=1, max_length=200000)
 
 
 class DatasourceViewSettingsRequest(BaseModel):
     description: str = Field(default="", max_length=20000)
+    view_description: str | None = Field(default=None, max_length=20000)
+
+
+def request_view_description(request: Any) -> str:
+    view_description = getattr(request, "view_description", None)
+    if view_description is not None:
+        return view_description
+    return str(getattr(request, "description", ""))
 
 
 class BusinessLogicSuggestionUpdateRequest(BaseModel):
@@ -3248,6 +3259,27 @@ def datasource_view_description_from_cache(
     connector: DatasourceConnector,
     view_name: str,
     actor: str,
+) -> str:
+    cache = get_datasource_schema_cache(session, connector.id)
+    if cache is None:
+        cache = introspect_datasource_connector(session, connector, actor)
+    settings = json_loads(cache.table_settings_json)
+    tables = settings.get("tables", {})
+    direct = tables.get(view_name, {})
+    if direct:
+        return str(direct.get("view_description", direct.get("description", "")))
+    lowered = view_name.lower()
+    for name, item in tables.items():
+        if str(name).lower() == lowered:
+            return str(item.get("view_description", item.get("description", "")))
+    return ""
+
+
+def datasource_view_logic_from_cache(
+    session: Session,
+    connector: DatasourceConnector,
+    view_name: str,
+    actor: str,
 ) -> str:
     cache = get_datasource_schema_cache(session, connector.id)
     if cache is None:
@@ -3282,14 +3314,23 @@ def get_datasource_view_definition(
     finally:
         engine.dispose()
 
+    view_description = datasource_view_description_from_cache(
+        session,
+        connector,
+        existing_name,
+        actor,
+    )
+    datasource_logic = datasource_view_logic_from_cache(
+        session,
+        connector,
+        existing_name,
+        actor,
+    )
     return {
         "name": existing_name,
-        "description": datasource_view_description_from_cache(
-            session,
-            connector,
-            existing_name,
-            actor,
-        ),
+        "description": view_description,
+        "view_description": view_description,
+        "datasource_logic": datasource_logic,
         "sql": view_sql_for_editor(connector, existing_name, definition),
     }
 
@@ -3298,7 +3339,7 @@ def save_datasource_view_settings_to_cache(
     session: Session,
     cache: DatasourceSchemaCache,
     view_name: str,
-    description: str,
+    view_description: str,
     actor: str,
 ) -> DatasourceSchemaCache:
     schema = json_loads(cache.schema_json)
@@ -3315,7 +3356,7 @@ def save_datasource_view_settings_to_cache(
     table_settings = settings.setdefault("tables", {})
     existing = table_settings.setdefault(actual_view_name, {})
     existing["selected"] = True
-    existing["description"] = description.strip()
+    existing["view_description"] = view_description.strip()
     return update_schema_table_settings(
         session=session,
         cache=cache,
@@ -3328,7 +3369,7 @@ def execute_datasource_view_sql(
     session: Session,
     connector: DatasourceConnector,
     view_name: str,
-    description: str,
+    view_description: str,
     sql: str,
     actor: str,
 ) -> DatasourceSchemaCache:
@@ -3352,7 +3393,7 @@ def execute_datasource_view_sql(
         session,
         cache,
         normalized_name,
-        description,
+        view_description,
         actor,
     )
 
@@ -3361,7 +3402,7 @@ def save_datasource_view_settings(
     session: Session,
     connector: DatasourceConnector,
     view_name: str,
-    description: str,
+    view_description: str,
     actor: str,
 ) -> DatasourceSchemaCache:
     normalized_name = normalize_datasource_view_name(view_name)
@@ -3372,7 +3413,7 @@ def save_datasource_view_settings(
         session,
         cache,
         normalized_name,
-        description,
+        view_description,
         actor,
     )
 
@@ -3382,7 +3423,7 @@ def update_datasource_view_sql(
     connector: DatasourceConnector,
     current_view_name: str,
     next_view_name: str,
-    description: str,
+    view_description: str,
     sql: str,
     actor: str,
 ) -> DatasourceSchemaCache:
@@ -3416,7 +3457,7 @@ def update_datasource_view_sql(
         session,
         cache,
         next_name,
-        description,
+        view_description,
         actor,
     )
 
@@ -3448,11 +3489,11 @@ def generate_datasource_view_sql(
     session: Session,
     connector: DatasourceConnector,
     view_name: str,
-    description: str,
+    view_description: str,
     actor: str,
 ) -> str:
     normalized_name = normalize_datasource_view_name(view_name)
-    if not description.strip():
+    if not view_description.strip():
         raise ValueError("View description is required to generate SQL.")
 
     cache = get_or_create_datasource_schema_cache(session, connector, actor)
@@ -3464,8 +3505,8 @@ def generate_datasource_view_sql(
     query_request = QueryRequest(
         question=(
             f"Create a SQL SELECT statement for a database view named {normalized_name}. "
-            "The SELECT must implement this datasource logic:\n"
-            f"{description.strip()}\n\n"
+            "The SELECT must implement this view description:\n"
+            f"{view_description.strip()}\n\n"
             "Return only one SELECT query. Do not create, drop, alter, insert, update or delete data."
         ),
         datasource_id=connector.connector_key,
@@ -3615,7 +3656,7 @@ def generate_datasource_view_sql_preview(
             session=session,
             connector=connector,
             view_name=request.name,
-            description=request.description,
+            view_description=request_view_description(request),
             actor=user.username,
         )
     except (
@@ -3656,7 +3697,7 @@ def execute_datasource_view_sql_endpoint(
             session=session,
             connector=connector,
             view_name=request.name,
-            description=request.description,
+            view_description=request_view_description(request),
             sql=request.sql,
             actor=user.username,
         )
@@ -3705,7 +3746,7 @@ def update_datasource_view_sql_endpoint(
             connector=connector,
             current_view_name=view_name,
             next_view_name=request.name,
-            description=request.description,
+            view_description=request_view_description(request),
             sql=request.sql,
             actor=user.username,
         )
@@ -3756,7 +3797,7 @@ def save_datasource_view_settings_endpoint(
             session=session,
             connector=connector,
             view_name=view_name,
-            description=request.description,
+            view_description=request_view_description(request),
             actor=user.username,
         )
     except (SQLAlchemyError, ValueError) as exc: