fix(sdk): honor base_url in bailian runtime readiness and sync placeholder

Author: chenanran555

packages/sdk/src/internal/provider-config.ts

@@ -42,7 +42,14 @@ export function resolveActiveProvider(): ProviderConfigProvider {
 /** Whether the current process env has all required fields for the active provider. */
 export function areRuntimeCredentialsReady(): boolean {
 	const provider = resolveActiveProvider();
-	return AGENTS_PROVIDER_FIELDS[provider].every((field) => process.env[field.key]?.trim());
+	return AGENTS_PROVIDER_FIELDS[provider].every((field) => {
+		if (process.env[field.key]?.trim()) return true;
+		// bailian derives its endpoint from either workspace_id or base_url, so a
+		// configured BAILIAN_BASE_URL satisfies the workspace_id slot (mirrors the
+		// provider config schema's "at least one" rule).
+		if (field.key === "BAILIAN_WORKSPACE_ID") return Boolean(process.env.BAILIAN_BASE_URL?.trim());
+		return false;
+	});
 }
 
 function isProvider(value: string): value is ProviderConfigProvider {

packages/sdk/src/internal/providers/registry.ts

@@ -91,13 +91,14 @@ export function buildProviders(
  * Environment variable mappings for each provider.
  * Used to construct a provider adapter without a agents.yaml config file.
  */
-const PROVIDER_ENV_VARS: Record<string, Record<string, { env: string[]; required: boolean }>> = {
+const PROVIDER_ENV_VARS: Record<string, Record<string, { env: string[]; required: boolean; placeholder?: boolean }>> = {
 	bailian: {
 		api_key: { env: ["DASHSCOPE_API_KEY", "BAILIAN_API_KEY"], required: true },
 		// Neither workspace_id nor base_url is required on its own; the config schema
 		// enforces "at least one" so a host can supply just base_url (BAILIAN_BASE_URL).
+		// base_url is the preferred placeholder emitted by `agents sync`.
 		workspace_id: { env: ["BAILIAN_WORKSPACE_ID"], required: false },
-		base_url: { env: ["BAILIAN_BASE_URL"], required: false },
+		base_url: { env: ["BAILIAN_BASE_URL"], required: false, placeholder: true },
 	},
 	qoder: {
 		api_key: { env: ["QODER_PAT", "QODER_API_KEY"], required: true },
@@ -115,15 +116,17 @@ const PROVIDER_ENV_VARS: Record<string, Record<string, { env: string[]; required
 
 /**
  * Build a `providers` config block for a single provider using `${ENV}`
- * placeholders for its required fields. Used by `agents sync` to emit a providers
- * block without leaking resolved secrets when the original file is unavailable.
+ * placeholders. Emits fields that are either required for env resolution or
+ * explicitly marked as the provider's preferred placeholder (e.g. bailian's
+ * base_url). Used by `agents sync` to emit a providers block without leaking
+ * resolved secrets when the original file is unavailable.
  */
 export function placeholderProviderConfig(providerName: string): Record<string, string> {
 	const envMap = PROVIDER_ENV_VARS[providerName];
 	if (!envMap) return {};
 	const out: Record<string, string> = {};
-	for (const [field, { env, required }] of Object.entries(envMap)) {
-		if (required && env[0]) out[field] = `\${${env[0]}}`;
+	for (const [field, { env, required, placeholder }] of Object.entries(envMap)) {
+		if ((required || placeholder) && env[0]) out[field] = `\${${env[0]}}`;
 	}
 	return out;
 }

packages/sdk/tests/unit/provider-config.test.ts

@@ -126,6 +126,29 @@ describe("provider-config bootstrap", () => {
 		}
 	});
 
+	test("areRuntimeCredentialsReady accepts BAILIAN_BASE_URL in place of workspace_id", () => {
+		const prevProvider = process.env.AGENTS_PROVIDER;
+		const prevKey = process.env.DASHSCOPE_API_KEY;
+		const prevWs = process.env.BAILIAN_WORKSPACE_ID;
+		const prevBaseUrl = process.env.BAILIAN_BASE_URL;
+		process.env.AGENTS_PROVIDER = "bailian";
+		process.env.DASHSCOPE_API_KEY = "sk-x";
+		delete process.env.BAILIAN_WORKSPACE_ID;
+		process.env.BAILIAN_BASE_URL = "https://ws.cn-beijing.maas.aliyuncs.com/api/v1/agentstudio";
+		try {
+			expect(areRuntimeCredentialsReady()).toBe(true);
+		} finally {
+			if (prevProvider === undefined) delete process.env.AGENTS_PROVIDER;
+			else process.env.AGENTS_PROVIDER = prevProvider;
+			if (prevKey === undefined) delete process.env.DASHSCOPE_API_KEY;
+			else process.env.DASHSCOPE_API_KEY = prevKey;
+			if (prevWs === undefined) delete process.env.BAILIAN_WORKSPACE_ID;
+			else process.env.BAILIAN_WORKSPACE_ID = prevWs;
+			if (prevBaseUrl === undefined) delete process.env.BAILIAN_BASE_URL;
+			else process.env.BAILIAN_BASE_URL = prevBaseUrl;
+		}
+	});
+
 	test("applyProviderConfigToEnv respects force flag", () => {
 		const prevProvider = process.env.AGENTS_PROVIDER;
 		const prevKey = process.env.ARK_API_KEY;

packages/sdk/tests/unit/provider-registry.test.ts

@@ -0,0 +1,28 @@
+import { expect, test } from "bun:test";
+import { placeholderProviderConfig } from "../../src/internal/providers/registry.ts";
+
+// Placeholder values are literal "${ENV_VAR}" tokens; assert their shape with a
+// regex instead of literal strings so biome's noTemplateCurlyInString rule (which
+// guards against accidental template placeholders) stays happy.
+const PLACEHOLDER_RE = /^\$\{[A-Z_]+\}$/;
+
+// `agents sync` falls back to these placeholders when the source providers block
+// is unavailable. bailian must emit base_url (its preferred field) so the emitted
+// config is valid without a workspace_id.
+test("placeholderProviderConfig emits api_key + base_url for bailian, not workspace_id", () => {
+	const block = placeholderProviderConfig("bailian");
+	expect(Object.keys(block).sort()).toEqual(["api_key", "base_url"]);
+	expect(block.workspace_id).toBeUndefined();
+	for (const value of Object.values(block)) {
+		expect(value).toMatch(PLACEHOLDER_RE);
+	}
+});
+
+test("placeholderProviderConfig emits only required fields for other providers", () => {
+	expect(Object.keys(placeholderProviderConfig("claude"))).toEqual(["api_key"]);
+	expect(Object.keys(placeholderProviderConfig("ark"))).toEqual(["api_key"]);
+});
+
+test("placeholderProviderConfig returns empty for an unknown provider", () => {
+	expect(placeholderProviderConfig("nope")).toEqual({});
+});