From b1c2950b1ed22c65aac7b543c911766874d38a05 Mon Sep 17 00:00:00 2001
From: hsiegeln <37154749+hsiegeln@users.noreply.github.com>
Date: Thu, 26 Mar 2026 12:55:19 +0100
Subject: [PATCH] fix: add id field to AppConfigPage DataTable rows

DataTable requires rows with an { id: string } constraint. Map
ApplicationConfig to ConfigRow adding id from the application field.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 ui/src/pages/Admin/AppConfigPage.tsx | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/ui/src/pages/Admin/AppConfigPage.tsx b/ui/src/pages/Admin/AppConfigPage.tsx
index 04b5360f..70284410 100644
--- a/ui/src/pages/Admin/AppConfigPage.tsx
+++ b/ui/src/pages/Admin/AppConfigPage.tsx
@@ -6,6 +6,8 @@ import { useAllApplicationConfigs, useUpdateApplicationConfig } from '../../api/
 import type { ApplicationConfig } from '../../api/queries/commands';
 import styles from './AppConfigPage.module.css';
 
+type ConfigRow = ApplicationConfig & { id: string };
+
 function timeAgo(iso?: string): string {
   if (!iso) return '\u2014';
   const diff = Date.now() - new Date(iso).getTime();
@@ -45,7 +47,7 @@ export default function AppConfigPage() {
     });
   }, [updateConfig, toast]);
 
-  const columns: Column<ApplicationConfig>[] = useMemo(() => [
+  const columns: Column<ConfigRow>[] = useMemo(() => [
     {
       key: '_inspect',
       header: '',
@@ -162,9 +164,9 @@ export default function AppConfigPage() {
 
   return (
     <div>
-      <DataTable<ApplicationConfig>
+      <DataTable<ConfigRow>
         columns={columns}
-        data={configs ?? []}
+        data={(configs ?? []).map(c => ({ ...c, id: c.application }))}
         pageSize={50}
       />
     </div>