refactor(ambient-ui): narrow CredentialManageSheet prop to non-nullable

Per amber's suggestion: credential prop narrowed from
DomainCredential | null to DomainCredential. All call sites
already conditionally mount the sheet, so the null guard is
redundant. This makes the contract explicit and prevents
future regression of the unbounded useRoleBindings query.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: jsell-rh

components/ambient-ui/src/app/(dashboard)/credentials/_components/credential-manage-sheet.tsx

@@ -37,7 +37,7 @@ export function CredentialManageSheet({
   onOpenChange,
   onNavigateToMatrix,
 }: {
-  credential: DomainCredential | null
+  credential: DomainCredential
   open: boolean
   onOpenChange: (open: boolean) => void
   onNavigateToMatrix?: (credentialName: string) => void
@@ -48,17 +48,13 @@ export function CredentialManageSheet({
   const updateCredential = useUpdateCredential()
   const deleteCredential = useDeleteCredential()
 
-  const { data: liveCredential } = useCredential(credential?.id ?? '')
+  const { data: liveCredential } = useCredential(credential.id)
   const resolved = liveCredential ?? credential
 
   const { data: bindingsData } = useRoleBindings(
-    credential
-      ? { search: `credential_id = '${credential.id}'` }
-      : undefined,
+    { search: `credential_id = '${credential.id}'` },
   )
 
-  if (!credential || !resolved) return null
-
   const providerMeta = getProviderMeta(resolved.provider)
   const category = getCategoryForProvider(resolved.provider)
   const bindingCount = bindingsData?.items.length ?? 0

components/ambient-ui/src/app/(dashboard)/credentials/_components/credential-table.tsx

@@ -208,14 +208,16 @@ export function CredentialTable({
         </Table>
       </div>
 
-      <CredentialManageSheet
-        credential={selectedCredential}
-        open={selectedCredential !== null}
-        onOpenChange={(open) => {
-          if (!open) setSelectedCredential(null)
-        }}
-        onNavigateToMatrix={onNavigateToMatrix}
-      />
+      {selectedCredential && (
+        <CredentialManageSheet
+          credential={selectedCredential}
+          open
+          onOpenChange={(open) => {
+            if (!open) setSelectedCredential(null)
+          }}
+          onNavigateToMatrix={onNavigateToMatrix}
+        />
+      )}
     </>
   )
 }