feat(control-plane): "not enabled" splash for disabled audit logs & LLM requests (+ bank name fix) (#1950)

* feat(control-plane): show "not enabled" splash for disabled audit logs & LLM requests

Add a reusable FeatureNotEnabled component (centered icon + title +
description) and use it for the Audit Logs and LLM Requests tabs, plus
refactor the existing Observations splash to reuse it. Tabs gain an
"Off" badge when the feature is disabled.

To let the UI detect server-side gating, expose audit_log and llm_trace
in the /version features object (sourced from config.audit_log_enabled /
config.llm_trace_enabled), wire them through the features context and the
control-plane SDK type, and add i18n keys across all 10 locales.

* fix(retain): default bank name to bank_id in ensure_bank_exists

ensure_bank_exists inserted banks without a name (NULL), unlike the other
creation path (get_or_create_bank_profile, which defaults name to bank_id).
Since #1940 wired PATCH /config to ensure_bank_exists, a config PATCH on a
never-retained bank (and any retain-only bank) produced a NULL name, which
then 500'd the deprecated GET /profile endpoint (name is typed as a required
str). Default name to bank_id at insert so every creation path is consistent.

Extends the #1940 regression test to assert the auto-created bank's profile
returns 200 with name == bank_id.

* test(api): assert audit_log and llm_trace flags in /version response

* chore: regenerate openapi spec and client SDKs for new feature flags

Author: nicoloboschi

hindsight-api-slim/hindsight_api/api/http.py

@@ -2382,6 +2382,8 @@ class FeaturesInfo(BaseModel):
     file_upload_api: bool = Field(description="Whether file upload/conversion API is enabled")
     document_export_api: bool = Field(description="Whether the document export endpoint is enabled")
     document_import_api: bool = Field(description="Whether the document import endpoint is enabled")
+    audit_log: bool = Field(description="Whether audit logging is enabled")
+    llm_trace: bool = Field(description="Whether per-bank LLM request tracing is enabled")
 
 
 class VersionResponse(BaseModel):
@@ -3047,6 +3049,8 @@ async def version_endpoint() -> VersionResponse:
                 file_upload_api=config.enable_file_upload_api,
                 document_export_api=config.enable_document_export_api,
                 document_import_api=config.enable_document_import_api,
+                audit_log=config.audit_log_enabled,
+                llm_trace=config.llm_trace_enabled,
             ),
         )
 

hindsight-api-slim/hindsight_api/engine/retain/fact_storage.py

@@ -147,12 +147,13 @@ async def ensure_bank_exists(conn, bank_id: str, ops=None) -> None:
     internal_id = uuid.uuid4()
     inserted = await conn.fetchval(
         f"""
-        INSERT INTO {fq_table("banks")} (bank_id, disposition, mission, internal_id)
-        VALUES ($1, $2::jsonb, $3, $4)
+        INSERT INTO {fq_table("banks")} (bank_id, name, disposition, mission, internal_id)
+        VALUES ($1, $2, $3::jsonb, $4, $5)
         ON CONFLICT (bank_id) DO NOTHING
         RETURNING bank_id
         """,
         bank_id,
+        bank_id,  # Default name is the bank_id (matches get_or_create_bank_profile)
         json.dumps(DEFAULT_DISPOSITION),
         "",
         internal_id,

hindsight-api-slim/tests/test_http_api_integration.py

@@ -1104,6 +1104,8 @@ async def test_version_endpoint_returns_correct_version(api_client):
     assert isinstance(features["observations"], bool)
     assert isinstance(features["mcp"], bool)
     assert isinstance(features["worker"], bool)
+    assert isinstance(features["audit_log"], bool)
+    assert isinstance(features["llm_trace"], bool)
 
     print(f"Version endpoint returned: api_version={result['api_version']}, features={features}")
 
@@ -1344,6 +1346,12 @@ async def test_patch_config_persists_override_for_uncreated_bank(api_client, fie
     assert body["config"][field] is False
     assert body["overrides"].get(field) is False
 
+    # The auto-created bank must have a name (defaults to bank_id). A NULL name
+    # would 500 the profile endpoint, whose response types name as a required str.
+    response = await api_client.get(f"/v1/default/banks/{test_bank_id}/profile")
+    assert response.status_code == 200, response.text
+    assert response.json()["name"] == test_bank_id
+
 
 @pytest.mark.hs_llm_core
 @pytest.mark.asyncio

hindsight-clients/go/api/openapi.yaml

@@ -5418,11 +5418,21 @@ components:
           description: Whether the document import endpoint is enabled
           title: Document Import Api
           type: boolean
+        audit_log:
+          description: Whether audit logging is enabled
+          title: Audit Log
+          type: boolean
+        llm_trace:
+          description: Whether per-bank LLM request tracing is enabled
+          title: Llm Trace
+          type: boolean
       required:
+      - audit_log
       - bank_config_api
       - document_export_api
       - document_import_api
       - file_upload_api
+      - llm_trace
       - mcp
       - observations
       - worker

hindsight-clients/go/model_features_info.go

@@ -33,7 +33,9 @@ class FeaturesInfo(BaseModel):
     file_upload_api: StrictBool = Field(description="Whether file upload/conversion API is enabled")
     document_export_api: StrictBool = Field(description="Whether the document export endpoint is enabled")
     document_import_api: StrictBool = Field(description="Whether the document import endpoint is enabled")
-    __properties: ClassVar[List[str]] = ["observations", "mcp", "worker", "bank_config_api", "file_upload_api", "document_export_api", "document_import_api"]
+    audit_log: StrictBool = Field(description="Whether audit logging is enabled")
+    llm_trace: StrictBool = Field(description="Whether per-bank LLM request tracing is enabled")
+    __properties: ClassVar[List[str]] = ["observations", "mcp", "worker", "bank_config_api", "file_upload_api", "document_export_api", "document_import_api", "audit_log", "llm_trace"]
 
     model_config = ConfigDict(
         populate_by_name=True,
@@ -92,7 +94,9 @@ def from_dict(cls, obj: Optional[Dict[str, Any]]) -> Optional[Self]:
             "bank_config_api": obj.get("bank_config_api"),
             "file_upload_api": obj.get("file_upload_api"),
             "document_export_api": obj.get("document_export_api"),
-            "document_import_api": obj.get("document_import_api")
+            "document_import_api": obj.get("document_import_api"),
+            "audit_log": obj.get("audit_log"),
+            "llm_trace": obj.get("llm_trace")
         })
         return _obj
 

hindsight-clients/python/hindsight_client_api/models/features_info.py

@@ -1676,6 +1676,18 @@ export type FeaturesInfo = {
    * Whether the document import endpoint is enabled
    */
   document_import_api: boolean;
+  /**
+   * Audit Log
+   *
+   * Whether audit logging is enabled
+   */
+  audit_log: boolean;
+  /**
+   * Llm Trace
+   *
+   * Whether per-bank LLM request tracing is enabled
+   */
+  llm_trace: boolean;
 };
 
 /**

hindsight-clients/typescript/generated/types.gen.ts

@@ -19,6 +19,7 @@ import { MentalModelsView } from "@/components/mental-models-view";
 import { WebhooksView } from "@/components/webhooks-view";
 import { AuditLogsView } from "@/components/audit-logs-view";
 import { LLMRequestsView } from "@/components/llm-requests-view";
+import { FeatureNotEnabled } from "@/components/feature-not-enabled";
 import { useFeatures } from "@/lib/features-context";
 import { useBank } from "@/lib/bank-context";
 import { bankRoute } from "@/lib/bank-url";
@@ -61,6 +62,8 @@ export default function BankPage() {
   const bankConfigTab = (searchParams.get("bankConfigTab") || "general") as BankConfigTab;
   const observationsEnabled = features?.observations ?? false;
   const bankConfigEnabled = features?.bank_config_api ?? false;
+  const auditLogEnabled = features?.audit_log ?? false;
+  const llmTraceEnabled = features?.llm_trace ?? false;
 
   // Bank actions state
   const [showDeleteDialog, setShowDeleteDialog] = useState(false);
@@ -331,6 +334,11 @@ export default function BankPage() {
                       }`}
                     >
                       {t("auditLogs")}
+                      {!auditLogEnabled && (
+                        <span className="ml-2 text-xs px-1.5 py-0.5 rounded bg-muted text-muted-foreground">
+                          Off
+                        </span>
+                      )}
                       {bankConfigTab === "audit-logs" && (
                         <div className="absolute bottom-0 left-0 right-0 h-0.5 bg-primary" />
                       )}
@@ -344,6 +352,11 @@ export default function BankPage() {
                       }`}
                     >
                       {t("llmRequests")}
+                      {!llmTraceEnabled && (
+                        <span className="ml-2 text-xs px-1.5 py-0.5 rounded bg-muted text-muted-foreground">
+                          Off
+                        </span>
+                      )}
                       {bankConfigTab === "llm-requests" && (
                         <div className="absolute bottom-0 left-0 right-0 h-0.5 bg-primary" />
                       )}
@@ -378,22 +391,46 @@ export default function BankPage() {
                       <WebhooksView />
                     </div>
                   )}
-                  {bankConfigTab === "audit-logs" && (
-                    <div>
-                      <p className="text-sm text-muted-foreground mb-4">
-                        {t("auditLogsDescription")}
-                      </p>
-                      <AuditLogsView />
-                    </div>
-                  )}
-                  {bankConfigTab === "llm-requests" && (
-                    <div>
-                      <p className="text-sm text-muted-foreground mb-4">
-                        {t("llmRequestsDescription")}
-                      </p>
-                      <LLMRequestsView />
-                    </div>
-                  )}
+                  {bankConfigTab === "audit-logs" &&
+                    (auditLogEnabled ? (
+                      <div>
+                        <p className="text-sm text-muted-foreground mb-4">
+                          {t("auditLogsDescription")}
+                        </p>
+                        <AuditLogsView />
+                      </div>
+                    ) : (
+                      <FeatureNotEnabled
+                        title={t("auditLogsNotEnabled")}
+                        description={t.rich("auditLogsDisabledMessage", {
+                          envVar: () => (
+                            <code className="px-1 py-0.5 bg-muted rounded text-xs">
+                              HINDSIGHT_API_AUDIT_LOG_ENABLED=true
+                            </code>
+                          ),
+                        })}
+                      />
+                    ))}
+                  {bankConfigTab === "llm-requests" &&
+                    (llmTraceEnabled ? (
+                      <div>
+                        <p className="text-sm text-muted-foreground mb-4">
+                          {t("llmRequestsDescription")}
+                        </p>
+                        <LLMRequestsView />
+                      </div>
+                    ) : (
+                      <FeatureNotEnabled
+                        title={t("llmRequestsNotEnabled")}
+                        description={t.rich("llmRequestsDisabledMessage", {
+                          envVar: () => (
+                            <code className="px-1 py-0.5 bg-muted rounded text-xs">
+                              HINDSIGHT_API_LLM_TRACE_ENABLED=true
+                            </code>
+                          ),
+                        })}
+                      />
+                    ))}
                 </div>
               </div>
             )}
@@ -510,37 +547,16 @@ export default function BankPage() {
                         <DataView key="observations" factType="observation" />
                       </div>
                     ) : (
-                      <div className="flex flex-col items-center justify-center py-16 text-center">
-                        <div className="text-muted-foreground mb-2">
-                          <svg
-                            xmlns="http://www.w3.org/2000/svg"
-                            width="48"
-                            height="48"
-                            viewBox="0 0 24 24"
-                            fill="none"
-                            stroke="currentColor"
-                            strokeWidth="1.5"
-                            strokeLinecap="round"
-                            strokeLinejoin="round"
-                          >
-                            <path d="M12 2a10 10 0 1 0 10 10A10 10 0 0 0 12 2Z" />
-                            <path d="M12 8v4" />
-                            <path d="M12 16h.01" />
-                          </svg>
-                        </div>
-                        <h3 className="text-lg font-semibold text-foreground mb-1">
-                          {t("observationsNotEnabled")}
-                        </h3>
-                        <p className="text-sm text-muted-foreground max-w-md">
-                          {t.rich("observationsDisabledMessage", {
-                            envVar: () => (
-                              <code className="px-1 py-0.5 bg-muted rounded text-xs">
-                                HINDSIGHT_API_ENABLE_OBSERVATIONS=true
-                              </code>
-                            ),
-                          })}
-                        </p>
-                      </div>
+                      <FeatureNotEnabled
+                        title={t("observationsNotEnabled")}
+                        description={t.rich("observationsDisabledMessage", {
+                          envVar: () => (
+                            <code className="px-1 py-0.5 bg-muted rounded text-xs">
+                              HINDSIGHT_API_ENABLE_OBSERVATIONS=true
+                            </code>
+                          ),
+                        })}
+                      />
                     ))}
                   {subTab === "mental-models" && (
                     <div>