From 684c1ab60026da5400aab3634ad41e1b176b2dd0 Mon Sep 17 00:00:00 2001
From: jialin <jialinkuang@126.com>
Date: Thu, 16 Apr 2026 15:03:18 +0800
Subject: [PATCH] fix: apikey model access

---
 src/components/seal-form/wrapper/index.tsx    |   1 +
 src/pages/_components/select-panel/index.tsx  |  15 +-
 .../add-apikey-modal/allow-models.tsx         | 142 ++++++++++--------
 .../components/add-apikey-modal/form.tsx      |  16 +-
 .../components/add-apikey-modal/index.tsx     |   7 +-
 src/pages/api-keys/config/index.ts            |   4 +-
 src/pages/api-keys/hooks/use-keys-columns.tsx |  52 ++++---
 7 files changed, 132 insertions(+), 105 deletions(-)

diff --git a/src/components/seal-form/wrapper/index.tsx b/src/components/seal-form/wrapper/index.tsx
index b19d5472..309a8ba8 100644
--- a/src/components/seal-form/wrapper/index.tsx
+++ b/src/components/seal-form/wrapper/index.tsx
@@ -170,6 +170,7 @@ const Extra = styled.div`
 
 const AddAfter = styled.div`
   position: relative;
+  border-radius: 0 var(--ant-border-radius) var(--ant-border-radius) 0;
   color: var(--ant-color-text-tertiary);
   font-size: var(--font-size-base);
   padding-inline: calc(var(--ant-padding-sm) - 1px);
diff --git a/src/pages/_components/select-panel/index.tsx b/src/pages/_components/select-panel/index.tsx
index b13b4e20..85c17065 100644
--- a/src/pages/_components/select-panel/index.tsx
+++ b/src/pages/_components/select-panel/index.tsx
@@ -37,6 +37,12 @@ interface SelectPanelProps {
   selectedKeys: string[];
   notFoundContent?: React.ReactNode;
   onSelectChange: (selectedKeys: string[]) => void;
+  styles?: {
+    container?: React.CSSProperties;
+    left?: React.CSSProperties;
+    right?: React.CSSProperties;
+    header?: React.CSSProperties;
+  };
 }
 
 const SelectPanel: React.FC<SelectPanelProps> = ({
@@ -46,6 +52,7 @@ const SelectPanel: React.FC<SelectPanelProps> = ({
   selectedKeys,
   searchPlaceholder,
   notFoundContent,
+  styles,
   onSelectChange
 }) => {
   const intl = useIntl();
@@ -150,9 +157,13 @@ const SelectPanel: React.FC<SelectPanelProps> = ({
   };
 
   return (
-    <PanelWrapper $maxHeight={height} $leftWidth={leftWidth}>
+    <PanelWrapper
+      $maxHeight={height}
+      $leftWidth={leftWidth}
+      style={styles?.container}
+    >
       <Left>
-        <Header>
+        <Header style={styles?.header}>
           <span
             style={{
               display: 'flex',
diff --git a/src/pages/api-keys/components/add-apikey-modal/allow-models.tsx b/src/pages/api-keys/components/add-apikey-modal/allow-models.tsx
index f04aca3b..6a805c45 100644
--- a/src/pages/api-keys/components/add-apikey-modal/allow-models.tsx
+++ b/src/pages/api-keys/components/add-apikey-modal/allow-models.tsx
@@ -4,10 +4,11 @@ import useAppUtils from '@/hooks/use-app-utils';
 import SelectPanel from '@/pages/_components/select-panel';
 import { queryMyModels } from '@/pages/llmodels/apis';
 import { useIntl } from '@umijs/max';
-import { Divider, Form, Radio } from 'antd';
+import { Checkbox, Divider, Form, Radio } from 'antd';
 import { useEffect, useState } from 'react';
 import styled from 'styled-components';
-import { ListItem } from '../../config/types';
+import { accessScopeOptions } from '../../config';
+import { FormData, ListItem } from '../../config/types';
 
 const Label = styled.div`
   font-weight: 500;
@@ -29,6 +30,7 @@ const AllowModelsForm: React.FC<{
     form
   ) as string[];
   const allowedType = Form.useWatch('allowed_type', form);
+  const scope = Form.useWatch('scope', form) as string[];
   const [modelList, setModelList] = useState<{ key: string; title: string }[]>(
     []
   );
@@ -56,11 +58,10 @@ const AllowModelsForm: React.FC<{
   const handleAllowTypeChange = (e: any) => {
     const value = e.target.value;
     onValuesChange?.({ allowed_type: value }, form.getFieldsValue());
-    if (value === 'management') {
-      form.setFieldsValue({ scope: ['management'] });
-    } else {
-      form.setFieldsValue({ scope: ['inference'] });
-    }
+  };
+
+  const handleOnScopeChange = (checkedValues: any) => {
+    console.log('checkedValues', form.getFieldValue('allowed_type'));
   };
 
   useEffect(() => {
@@ -71,61 +72,82 @@ const AllowModelsForm: React.FC<{
     <div>
       <Divider></Divider>
       <Label>{intl.formatMessage({ id: 'apikeys.access.permissions' })}</Label>
-      <Form.Item
-        name="allowed_type"
-        initialValue="all"
-        style={{ marginBottom: 8 }}
-      >
-        <Radio.Group
-          onChange={handleAllowTypeChange}
-          options={[
-            {
-              label: intl.formatMessage({
-                id: 'apikeys.accessScope.management'
-              }),
-              value: 'management'
-            },
-            {
-              label: intl.formatMessage({ id: 'apikeys.models.all' }),
-              value: 'all'
-            },
-            {
-              label: intl.formatMessage({ id: 'apikeys.models.selected' }),
-              value: 'custom'
-            }
-          ]}
-        ></Radio.Group>
+      <Form.Item<FormData> name="scope" style={{ marginBottom: 8 }}>
+        <Checkbox.Group
+          options={accessScopeOptions.map((item) => ({
+            label: intl.formatMessage({ id: item.label }),
+            value: item.value
+          }))}
+          onChange={handleOnScopeChange}
+        ></Checkbox.Group>
       </Form.Item>
-      <Form.Item
-        name="allowed_model_names"
-        hidden={allowedType === 'all' || allowedType === 'management'}
-        rules={[
-          {
-            required: allowedType === 'custom',
-            message: getRuleMessage(
-              'select',
-              intl.formatMessage({ id: 'models.table.models' })
-            )
-          }
-        ]}
-      >
-        <SelectPanel
-          height={300}
-          searchPlaceholder={intl.formatMessage({ id: 'common.filter.name' })}
-          options={modelList}
-          selectedKeys={allowedModelNames || []}
-          notFoundContent={intl.formatMessage({
-            id: 'apikeys.models.noModelsFound'
-          })}
-          onSelectChange={(selectedKeys) => {
-            form.setFieldsValue({ allowed_model_names: selectedKeys });
-            onValuesChange?.(
-              { allowed_model_names: selectedKeys },
-              form.getFieldsValue()
-            );
+      {scope?.includes('inference') && (
+        <div
+          style={{
+            padding: '12px',
+            backgroundColor: 'var(--ant-color-fill-quaternary)',
+            borderRadius: 4
           }}
-        />
-      </Form.Item>
+        >
+          <Form.Item name="allowed_type" noStyle>
+            <Radio.Group
+              onChange={handleAllowTypeChange}
+              options={[
+                {
+                  label: intl.formatMessage({ id: 'apikeys.models.all' }),
+                  value: 'all'
+                },
+                {
+                  label: intl.formatMessage({ id: 'apikeys.models.selected' }),
+                  value: 'custom'
+                }
+              ]}
+            ></Radio.Group>
+          </Form.Item>
+          {allowedType === 'custom' && (
+            <Form.Item
+              name="allowed_model_names"
+              style={{ marginBottom: 0, marginTop: 12 }}
+              rules={[
+                {
+                  required: allowedType === 'custom',
+                  message: getRuleMessage(
+                    'select',
+                    intl.formatMessage({ id: 'models.table.models' })
+                  )
+                }
+              ]}
+            >
+              <SelectPanel
+                height={300}
+                styles={{
+                  container: {
+                    backgroundColor: 'var(--ant-color-bg-container) !important'
+                  },
+                  header: {
+                    backgroundColor: 'var(--ant-color-bg-container) !important'
+                  }
+                }}
+                searchPlaceholder={intl.formatMessage({
+                  id: 'common.filter.name'
+                })}
+                options={modelList}
+                selectedKeys={allowedModelNames || []}
+                notFoundContent={intl.formatMessage({
+                  id: 'apikeys.models.noModelsFound'
+                })}
+                onSelectChange={(selectedKeys) => {
+                  form.setFieldsValue({ allowed_model_names: selectedKeys });
+                  onValuesChange?.(
+                    { allowed_model_names: selectedKeys },
+                    form.getFieldsValue()
+                  );
+                }}
+              />
+            </Form.Item>
+          )}
+        </div>
+      )}
     </div>
   );
 };
diff --git a/src/pages/api-keys/components/add-apikey-modal/form.tsx b/src/pages/api-keys/components/add-apikey-modal/form.tsx
index 8b78d26c..b6c3a05f 100644
--- a/src/pages/api-keys/components/add-apikey-modal/form.tsx
+++ b/src/pages/api-keys/components/add-apikey-modal/form.tsx
@@ -6,7 +6,7 @@ import { PageActionType } from '@/config/types';
 import { useIntl } from '@umijs/max';
 import { Form } from 'antd';
 import React from 'react';
-import { accessScopeOptions, expirationOptions } from '../../config';
+import { expirationOptions } from '../../config';
 import { FormData, ListItem } from '../../config/types';
 import AllowModelsForm from './allow-models';
 
@@ -115,19 +115,7 @@ const APIKeyForm: React.FC<{
           )}
         </>
       )}
-      <Form.Item<FormData>
-        name="scope"
-        hidden
-        normalize={(value) => (value ? [value] : [])}
-        getValueProps={(value) => ({
-          value: Array.isArray(value) ? value[0] : value
-        })}
-      >
-        <SealSelect
-          options={accessScopeOptions}
-          label={intl.formatMessage({ id: 'models.table.accessScope' })}
-        ></SealSelect>
-      </Form.Item>
+
       <AllowModelsForm
         currentData={currentData}
         action={action}
diff --git a/src/pages/api-keys/components/add-apikey-modal/index.tsx b/src/pages/api-keys/components/add-apikey-modal/index.tsx
index 28020bd6..06806198 100644
--- a/src/pages/api-keys/components/add-apikey-modal/index.tsx
+++ b/src/pages/api-keys/components/add-apikey-modal/index.tsx
@@ -118,7 +118,7 @@ const AddModal: React.FC<AddModalProps> = ({
         ..._.omit(formdata, ['allowed_type']),
         allowed_model_names:
           formdata.allowed_type === 'all' ||
-          formdata.allowed_type === 'management'
+          !formdata.scope?.includes('inference')
             ? []
             : formdata.allowed_model_names || []
       };
@@ -186,9 +186,7 @@ const AddModal: React.FC<AddModalProps> = ({
         scope: currentData.scope,
         allowed_type: currentData.allowed_model_names?.length
           ? 'custom'
-          : currentData.scope?.includes('management')
-            ? 'management'
-            : 'all',
+          : 'all',
         expires_in: parseExpireValue(currentData as ListItem),
         allowed_model_names: currentData.allowed_model_names || []
       });
@@ -272,6 +270,7 @@ const AddModal: React.FC<AddModalProps> = ({
           onFinish={handleOnOk}
           preserve={false}
           initialValues={{
+            allowed_type: 'all',
             scope: ['inference']
           }}
         >
diff --git a/src/pages/api-keys/config/index.ts b/src/pages/api-keys/config/index.ts
index f45a66b3..626c9756 100644
--- a/src/pages/api-keys/config/index.ts
+++ b/src/pages/api-keys/config/index.ts
@@ -27,13 +27,13 @@ export const expirationOptions = [
 
 export const accessScopeOptions = [
   {
-    label: 'management',
+    label: 'apikeys.accessScope.management',
     value: 'management',
     description: 'v2/',
     locale: true
   },
   {
-    label: 'inference',
+    label: 'apikeys.table.bindModels',
     value: 'inference',
     description: 'v1/',
     locale: true
diff --git a/src/pages/api-keys/hooks/use-keys-columns.tsx b/src/pages/api-keys/hooks/use-keys-columns.tsx
index f08e0cfd..ccd486fd 100644
--- a/src/pages/api-keys/hooks/use-keys-columns.tsx
+++ b/src/pages/api-keys/hooks/use-keys-columns.tsx
@@ -8,7 +8,6 @@ import { Tag } from 'antd';
 import { ColumnsType } from 'antd/lib/table';
 import dayjs from 'dayjs';
 import { useMemo } from 'react';
-import { accessScopeOptions } from '../config';
 import { ListItem } from '../config/types';
 
 interface ColumnsHookProps {
@@ -16,7 +15,7 @@ interface ColumnsHookProps {
   sortOrder: string[];
 }
 
-const actionList: Global.ActionItem[] = [
+const actionList: Global.ActionItem<string>[] = [
   {
     label: 'common.button.edit',
     key: 'edit',
@@ -36,20 +35,6 @@ const useModelsColumns = ({
 }: ColumnsHookProps): ColumnsType<ListItem> => {
   const intl = useIntl();
 
-  const renderAPIs = (record: ListItem) => {
-    const option = accessScopeOptions.find(
-      (item) => item.value === record.scope?.[0]
-    );
-    return (
-      <span className="flex-center gap-4">
-        {intl.formatMessage({
-          id: option?.label || ''
-        })}
-        {option?.description && <span>[{option?.description}]</span>}
-      </span>
-    );
-  };
-
   return useMemo(() => {
     return [
       {
@@ -112,13 +97,34 @@ const useModelsColumns = ({
           showTitle: false
         },
         render: (text: string[], record: ListItem) => (
-          <AutoTooltip ghost>
-            {record.scope?.[0] === 'management'
-              ? intl.formatMessage({ id: 'apikeys.accessScope.management' })
-              : record.allowed_model_names?.length
-                ? record.allowed_model_names.join(', ')
-                : intl.formatMessage({ id: 'apikeys.models.all' })}
-          </AutoTooltip>
+          <div className="flex-column gap-4">
+            {record.scope?.includes('management') && (
+              <AutoTooltip ghost>
+                {intl.formatMessage({ id: 'apikeys.accessScope.management' })}
+              </AutoTooltip>
+            )}
+            {record.scope?.includes('inference') && (
+              <div
+                style={{
+                  border: '1px solid var(--ant-color-split)',
+                  color: 'var(--ant-color-text-tertiary)',
+                  backgroundColor: 'var(--ant-color-fill-quaternary)',
+                  borderRadius: 4,
+                  paddingInline: 8,
+                  flexGrow: 0,
+                  maxWidth: '100%',
+                  width: 'max-content',
+                  display: 'flex'
+                }}
+              >
+                <AutoTooltip ghost>
+                  {record.allowed_model_names?.length
+                    ? record.allowed_model_names.join(', ')
+                    : intl.formatMessage({ id: 'apikeys.models.all' })}
+                </AutoTooltip>
+              </div>
+            )}
+          </div>
         )
       },
       {