fix: disable doubao coding plan selection

[seefs001]

Summary by CodeRabbit

Release Notes

• Bug Fixes
  • Improved handling of deprecated channel configuration with clear deprecation notices
  • Enhanced channel editing state management to properly distinguish between creating new channels and editing existing ones

[coderabbitai]

Walkthrough

Modified EditChannelModal.jsx to handle deprecation of the "Doubao Coding Plan" base URL option. Added a deprecation constant and state tracking to preserve existing deprecated configurations while preventing new channels from adopting the deprecated value. Updated the selector UI to display a tooltip message and conditionally disable the option based on loaded channel state.

Changes

Cohort / File(s)	Summary
Deprecation Handling for Doubao Coding Plan web/src/components/table/channels/modals/EditChannelModal.jsx	Added DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL constant and initialBaseUrlRef to track deprecated base URLs. Updated channel base URL selector to render deprecation tooltip, disable "Doubao Coding Plan" option for new channels, and preserve ability to maintain existing deprecated configurations. Set/cleared ref during channel load and creation cycles.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• feat: support doubao coding plan #2238 — Modifies the same EditChannelModal.jsx file to handle "Doubao Coding Plan" base URL option behavior and UI presentation.

Poem

🐰 A plan deprecated hops away,
Yet old friends hold their sway,
With tooltip whispers, soft and kind,
New paths excluded, not confined,
The modal remembers what came before! 🌟

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: disabling the Doubao Coding Plan selection in the channel modal UI.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

CodeRabbit can generate a title for your PR based on the changes.

Add @coderabbitai placeholder anywhere in the title of your PR and CodeRabbit will replace it with a title based on the changes in the PR. You can change the placeholder by changing the reviews.auto_title_placeholder setting.

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@web/src/components/table/channels/modals/EditChannelModal.jsx`:
- Line 127: The code uses DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL set to
'doubao-coding-plan', but the backend deprecated special-base is the full
resolved URL, so existing channels with the resolved URL aren't recognized as
deprecated; update the check logic that references
DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL (and all occurrences) to accept both
forms: preserve the existing short token for legacy records and also treat the
backend-resolved URL 'https://ark.cn-beijing.volces.com/api/coding/v3' as
equivalent (either by replacing DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL with the
full URL or by adding a secondary constant and updating comparisons in
EditChannelModal.jsx and the other referenced usages to normalize/compare
against both values before deciding to “keep existing”).
- Around line 422-432: The new user-facing strings
(doubaoCodingPlanDeprecationMessage and the label in
doubaoCodingPlanOptionLabel) are hard-coded and bypass i18n; update the
component to use the useTranslation() hook and replace those literals with
t('...') calls (e.g., t('doubaoCodingPlan.deprecationMessage') and
t('doubaoCodingPlan.label')), keeping canKeepDeprecatedDoubaoCodingPlan logic
unchanged; ensure translation keys are added to the locale resource files and
import/use the hook at the top of EditChannelModal.jsx.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f949a33c-7e5f-4d30-bfd1-19a4f08d6bf8

📥 Commits

Reviewing files that changed from the base of the PR and between deff59a and 6c074ef.

📒 Files selected for processing (1)

• web/src/components/table/channels/modals/EditChannelModal.jsx

[coderabbitai]

⚠️ Potential issue | 🟠 Major

Potential mismatch with backend deprecated base URL breaks “keep existing” behavior

The guard compares/stores doubao-coding-plan, but backend special-base config for OpenAI uses https://ark.cn-beijing.volces.com/api/coding/v3 (constant/channel.go:205-208). If loaded channels carry the resolved URL, existing deprecated channels won’t be recognized as keepable.

🔧 Suggested fix

-const DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL = 'doubao-coding-plan';
+const DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL =
+  'https://ark.cn-beijing.volces.com/api/coding/v3';
+const DEPRECATED_DOUBAO_CODING_PLAN_LEGACY_KEY = 'doubao-coding-plan';
...
-const canKeepDeprecatedDoubaoCodingPlan =
-  initialBaseUrlRef.current === DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL;
+const canKeepDeprecatedDoubaoCodingPlan = [
+  DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL,
+  DEPRECATED_DOUBAO_CODING_PLAN_LEGACY_KEY,
+].includes((initialBaseUrlRef.current || '').trim());

Also applies to: 425-425, 963-963, 3081-3083

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@web/src/components/table/channels/modals/EditChannelModal.jsx` at line 127,
The code uses DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL set to
'doubao-coding-plan', but the backend deprecated special-base is the full
resolved URL, so existing channels with the resolved URL aren't recognized as
deprecated; update the check logic that references
DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL (and all occurrences) to accept both
forms: preserve the existing short token for legacy records and also treat the
backend-resolved URL 'https://ark.cn-beijing.volces.com/api/coding/v3' as
equivalent (either by replacing DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL with the
full URL or by adding a secondary constant and updating comparisons in
EditChannelModal.jsx and the other referenced usages to normalize/compare
against both values before deciding to “keep existing”).

[coderabbitai]

⚠️ Potential issue | 🟠 Major

New tooltip/label text bypasses i18n pipeline

Line 422-432 introduces user-facing strings outside t(...). This won’t be localized and violates frontend i18n conventions for this path.

🌐 Suggested fix

-  const doubaoCodingPlanDeprecationMessage =
-    'Doubao Coding Plan 不再允许新增。根据火山方舟文档，Coding 套餐额度仅适用于 AI Coding 产品内调用，不适用于单独 API 调用；在非 AI Coding 产品中使用对应的 Base URL 和 API Key 可能被视为违规，并可能导致订阅停用或账号封禁。';
+  const doubaoCodingPlanDeprecationMessage = t(
+    'Doubao Coding Plan 不再允许新增。根据火山方舟文档，Coding 套餐额度仅适用于 AI Coding 产品内调用，不适用于单独 API 调用；在非 AI Coding 产品中使用对应的 Base URL 和 API Key 可能被视为违规，并可能导致订阅停用或账号封禁。',
+  );
...
-        <span>Doubao Coding Plan</span>
+        <span>{t('Doubao Coding Plan')}</span>

As per coding guidelines: “Frontend i18n: Use useTranslation() hook and call t('中文key') in components.”

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	const doubaoCodingPlanDeprecationMessage =
	'Doubao Coding Plan 不再允许新增。根据火山方舟文档，Coding 套餐额度仅适用于 AI Coding 产品内调用，不适用于单独 API 调用；在非 AI Coding 产品中使用对应的 Base URL 和 API Key 可能被视为违规，并可能导致订阅停用或账号封禁。';
	const canKeepDeprecatedDoubaoCodingPlan =
	initialBaseUrlRef.current === DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL;
	const doubaoCodingPlanOptionLabel = (
	<Tooltip content={doubaoCodingPlanDeprecationMessage} position='left'>
	<span className='inline-flex items-center gap-2'>
	<span>Doubao Coding Plan</span>
	</span>
	</Tooltip>
	);
	const doubaoCodingPlanDeprecationMessage = t(
	'Doubao Coding Plan 不再允许新增。根据火山方舟文档，Coding 套餐额度仅适用于 AI Coding 产品内调用，不适用于单独 API 调用；在非 AI Coding 产品中使用对应的 Base URL 和 API Key 可能被视为违规，并可能导致订阅停用或账号封禁。',
	);
	const canKeepDeprecatedDoubaoCodingPlan =
	initialBaseUrlRef.current === DEPRECATED_DOUBAO_CODING_PLAN_BASE_URL;
	const doubaoCodingPlanOptionLabel = (
	<Tooltip content={doubaoCodingPlanDeprecationMessage} position='left'>
	<span className='inline-flex items-center gap-2'>
	<span>{t('Doubao Coding Plan')}</span>
	</span>
	</Tooltip>
	);

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@web/src/components/table/channels/modals/EditChannelModal.jsx` around lines
422 - 432, The new user-facing strings (doubaoCodingPlanDeprecationMessage and
the label in doubaoCodingPlanOptionLabel) are hard-coded and bypass i18n; update
the component to use the useTranslation() hook and replace those literals with
t('...') calls (e.g., t('doubaoCodingPlan.deprecationMessage') and
t('doubaoCodingPlan.label')), keeping canKeepDeprecatedDoubaoCodingPlan logic
unchanged; ensure translation keys are added to the locale resource files and
import/use the hook at the top of EditChannelModal.jsx.