refactor: rename ACCOUNT_SETTING_TAB.PROVIDER to MODEL_PROVIDER

Rename the constant for clarity and consistency with the new sandbox-provider tab naming convention. Update all references across the codebase to use the new constant name.
2026-01-14 06:07:33 +08:00 · 2026-01-13 15:07:04 +08:00 · 2026-01-13 15:07:04 +08:00 · ffc39b0235
commit ffc39b0235
parent f72f58dbc4
12 changed files with 19 additions and 18 deletions
--- a/web/app/components/app/configuration/dataset-config/settings-modal/index.spec.tsx
+++ b/web/app/components/app/configuration/dataset-config/settings-modal/index.spec.tsx
@ -360,7 +360,7 @@ describe('SettingsModal', () => {
      await user.click(screen.getByText('datasetSettings.form.embeddingModelTipLink'))

      // Assert
-      expect(mockSetShowAccountSettingModal).toHaveBeenCalledWith({ payload: ACCOUNT_SETTING_TAB.PROVIDER })
+      expect(mockSetShowAccountSettingModal).toHaveBeenCalledWith({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })
    })
  })

--- a/web/app/components/app/configuration/dataset-config/settings-modal/index.tsx
+++ b/web/app/components/app/configuration/dataset-config/settings-modal/index.tsx
@ -282,7 +282,7 @@ const SettingsModal: FC<SettingsModalProps> = ({
              </div>
              <div className="mt-2 w-full text-xs leading-6 text-text-tertiary">
                {t('form.embeddingModelTip', { ns: 'datasetSettings' })}
-                <span className="cursor-pointer text-text-accent" onClick={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.PROVIDER })}>{t('form.embeddingModelTipLink', { ns: 'datasetSettings' })}</span>
+                <span className="cursor-pointer text-text-accent" onClick={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })}>{t('form.embeddingModelTipLink', { ns: 'datasetSettings' })}</span>
              </div>
            </div>
          </div>
--- a/web/app/components/app/configuration/index.tsx
+++ b/web/app/components/app/configuration/index.tsx
@ -577,7 +577,7 @@ const Configuration: FC = () => {
      // old dataset struct
      if (modelConfig.agent_mode?.tools?.find(({ dataset }: any) => dataset?.enabled))
        datasets = modelConfig.agent_mode?.tools.filter(({ dataset }: any) => dataset?.enabled)
-        // new dataset struct
+      // new dataset struct
      else if (modelConfig.dataset_configs.datasets?.datasets?.length > 0)
        datasets = modelConfig.dataset_configs?.datasets?.datasets

@ -1033,7 +1033,7 @@ const Configuration: FC = () => {
                  <div className="flex grow flex-col rounded-tl-2xl border-l-[0.5px] border-t-[0.5px] border-components-panel-border bg-chatbot-bg ">
                    <Debug
                      isAPIKeySet={isAPIKeySet}
-                      onSetting={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.PROVIDER })}
+                      onSetting={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })}
                      inputs={inputs}
                      modelParameterParams={{
                        setModel: setModel as any,
@ -1054,7 +1054,7 @@ const Configuration: FC = () => {
              content={t('trailUseGPT4Info.description', { ns: 'appDebug' })}
              isShow={showUseGPT4Confirm}
              onConfirm={() => {
-                setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.PROVIDER })
+                setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })
                setShowUseGPT4Confirm(false)
              }}
              onCancel={() => setShowUseGPT4Confirm(false)}
@ -1086,7 +1086,7 @@ const Configuration: FC = () => {
            <Drawer showClose isOpen={isShowDebugPanel} onClose={hideDebugPanel} mask footer={null}>
              <Debug
                isAPIKeySet={isAPIKeySet}
-                onSetting={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.PROVIDER })}
+                onSetting={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })}
                inputs={inputs}
                modelParameterParams={{
                  setModel: setModel as any,
--- a/web/app/components/app/overview/apikey-info-panel/cloud.spec.tsx
+++ b/web/app/components/app/overview/apikey-info-panel/cloud.spec.tsx
@ -81,7 +81,7 @@ describe('APIKeyInfoPanel - Cloud Edition', () => {
      interactions.clickMainButton()

      expect(mockSetShowAccountSettingModal).toHaveBeenCalledWith({
-        payload: ACCOUNT_SETTING_TAB.PROVIDER,
+        payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER,
      })
    })

--- a/web/app/components/app/overview/apikey-info-panel/index.spec.tsx
+++ b/web/app/components/app/overview/apikey-info-panel/index.spec.tsx
@ -94,7 +94,7 @@ describe('APIKeyInfoPanel - Community Edition', () => {
      interactions.clickMainButton()

      expect(mockSetShowAccountSettingModal).toHaveBeenCalledWith({
-        payload: ACCOUNT_SETTING_TAB.PROVIDER,
+        payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER,
      })
    })

--- a/web/app/components/app/overview/apikey-info-panel/index.tsx
+++ b/web/app/components/app/overview/apikey-info-panel/index.tsx
@ -49,7 +49,7 @@ const APIKeyInfoPanel: FC = () => {
      <Button
        variant="primary"
        className="mt-2 space-x-2"
-        onClick={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.PROVIDER })}
+        onClick={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })}
      >
        <div className="text-sm font-medium">{t('apiKeyInfo.setAPIBtn', { ns: 'appOverview' })}</div>
        <LinkExternal02 className="h-4 w-4" />
--- a/web/app/components/base/features/new-feature-panel/moderation/moderation-setting-modal.tsx
+++ b/web/app/components/base/features/new-feature-panel/moderation/moderation-setting-modal.tsx
@ -50,7 +50,7 @@ const ModerationSettingModal: FC<ModerationSettingModalProps> = ({
  const { setShowAccountSettingModal } = useModalContext()
  const handleOpenSettingsModal = () => {
    setShowAccountSettingModal({
-      payload: ACCOUNT_SETTING_TAB.PROVIDER,
+      payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER,
      onCancelCallback: () => {
        refetchModelProviders()
      },
@ -281,7 +281,7 @@ const ModerationSettingModal: FC<ModerationSettingModalProps> = ({
                >
                  &nbsp;
                  {t('settings.provider', { ns: 'common' })}
-&nbsp;
+                  &nbsp;
                </span>
                {t('feature.moderation.modal.openaiNotConfig.after', { ns: 'appDebug' })}
              </div>
--- a/web/app/components/datasets/create/index.tsx
+++ b/web/app/components/datasets/create/index.tsx
@ -145,7 +145,7 @@ const DatasetUpdateForm = ({ datasetId }: DatasetUpdateFormProps) => {
              {(step === 2 && (!datasetId || (datasetId && !!datasetDetail))) && (
                <StepTwo
                  isAPIKeySet={!!embeddingsDefaultModel}
-                  onSetting={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.PROVIDER })}
+                  onSetting={() => setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })}
                  indexingType={datasetDetail?.indexing_technique}
                  datasetId={datasetId}
                  dataSourceType={dataSourceType}
--- a/web/app/components/datasets/documents/detail/settings/document-settings.tsx
+++ b/web/app/components/datasets/documents/detail/settings/document-settings.tsx
@ -19,6 +19,7 @@ import AppUnavailable from '@/app/components/base/app-unavailable'
 import Loading from '@/app/components/base/loading'
 import StepTwo from '@/app/components/datasets/create/step-two'
 import AccountSetting from '@/app/components/header/account-setting'
+import { ACCOUNT_SETTING_TAB } from '@/app/components/header/account-setting/constants'
 import { ModelTypeEnum } from '@/app/components/header/account-setting/model-provider-page/declarations'
 import { useDefaultModel } from '@/app/components/header/account-setting/model-provider-page/hooks'
 import DatasetDetailContext from '@/context/dataset-detail'
@ -155,7 +156,7 @@ const DocumentSettings = ({ datasetId, documentId }: DocumentSettingsProps) => {
      </div>
      {isShowSetAPIKey && (
        <AccountSetting
-          activeTab="provider"
+          activeTab={ACCOUNT_SETTING_TAB.MODEL_PROVIDER}
          onCancel={async () => {
            hideSetAPIkey()
          }}
--- a/web/app/components/header/account-setting/constants.ts
+++ b/web/app/components/header/account-setting/constants.ts
@ -2,7 +2,7 @@ export const ACCOUNT_SETTING_MODAL_ACTION = 'showSettings'

 export const ACCOUNT_SETTING_TAB = {
  SANDBOX_PROVIDER: 'sandbox-provider',
-  PROVIDER: 'provider',
+  MODEL_PROVIDER: 'model-provider',
  MEMBERS: 'members',
  BILLING: 'billing',
  DATA_SOURCE: 'data-source',
--- a/web/app/components/header/account-setting/index.tsx
+++ b/web/app/components/header/account-setting/index.tsx
@ -77,7 +77,7 @@ export default function AccountSetting({

    const items: GroupItem[] = [
      {
-        key: ACCOUNT_SETTING_TAB.PROVIDER,
+        key: ACCOUNT_SETTING_TAB.MODEL_PROVIDER,
        name: t('settings.provider', { ns: 'common' }),
        icon: <RiBrain2Line className={iconClassName} />,
        activeIcon: <RiBrain2Fill className={iconClassName} />,
@ -234,7 +234,7 @@ export default function AccountSetting({
                  <div className="system-sm-regular mt-1 text-text-tertiary">{activeItem?.description}</div>
                )}
              </div>
-              {activeItem?.key === 'provider' && (
+              {activeItem?.key === ACCOUNT_SETTING_TAB.MODEL_PROVIDER && (
                <div className="flex grow justify-end">
                  <Input
                    showLeftIcon
@ -247,7 +247,7 @@ export default function AccountSetting({
              )}
            </div>
            <div className="px-4 pt-2 sm:px-8">
-              {activeMenu === 'provider' && <ModelProviderPage searchText={searchValue} />}
+              {activeMenu === ACCOUNT_SETTING_TAB.MODEL_PROVIDER && <ModelProviderPage searchText={searchValue} />}
              {activeMenu === 'sandbox-provider' && <SandboxProviderPage />}
              {activeMenu === 'members' && <MembersPage />}
              {activeMenu === 'billing' && <BillingPage />}
--- a/web/app/components/header/account-setting/model-provider-page/model-selector/popup.tsx
+++ b/web/app/components/header/account-setting/model-provider-page/model-selector/popup.tsx
@ -134,7 +134,7 @@ const Popup: FC<PopupProps> = ({
        className="sticky bottom-0 flex cursor-pointer items-center rounded-b-lg border-t border-divider-subtle bg-components-panel-bg px-4 py-2 text-text-accent-light-mode-only"
        onClick={() => {
          onHide()
-          setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.PROVIDER })
+          setShowAccountSettingModal({ payload: ACCOUNT_SETTING_TAB.MODEL_PROVIDER })
        }}
      >
        <span className="system-xs-medium">{t('model.settingsLink', { ns: 'common' })}</span>