@f5xc-salesdemos/xcsh 18.87.3 → 18.89.0

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [18.88.0] - 2026-05-31
+
+### Changed
+
+- Salesforce tools extracted to marketplace plugin: sf_setup, sf_query, sf_org_display, and sf_pipeline_report are now available as an installable plugin (`@f5xc-salesdemos/xcsh-salesforce`) via the Extension API instead of built-in tools. Install with `xcsh plugin install salesforce`. Context discovery, pipeline reporting, and container-adapted authentication are preserved with full feature parity. ([#1059](https://github.com/f5xc-salesdemos/xcsh/issues/1059))
+
 ## [18.75.0] - 2026-05-23
 
 ### Added

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/xcsh",
-	"version": "18.87.3",
+	"version": "18.89.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/f5xc-salesdemos/xcsh",
 	"author": "Can Boluk",
@@ -50,12 +50,12 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.16.1",
 		"@mozilla/readability": "^0.6",
-		"@f5xc-salesdemos/xcsh-stats": "18.87.3",
-		"@f5xc-salesdemos/pi-agent-core": "18.87.3",
-		"@f5xc-salesdemos/pi-ai": "18.87.3",
-		"@f5xc-salesdemos/pi-natives": "18.87.3",
-		"@f5xc-salesdemos/pi-tui": "18.87.3",
-		"@f5xc-salesdemos/pi-utils": "18.87.3",
+		"@f5xc-salesdemos/xcsh-stats": "18.89.0",
+		"@f5xc-salesdemos/pi-agent-core": "18.89.0",
+		"@f5xc-salesdemos/pi-ai": "18.89.0",
+		"@f5xc-salesdemos/pi-natives": "18.89.0",
+		"@f5xc-salesdemos/pi-tui": "18.89.0",
+		"@f5xc-salesdemos/pi-utils": "18.89.0",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/config/settings-schema.ts

@@ -1360,17 +1360,6 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
-	"salesforce.enabled": {
-		type: "boolean",
-		default: true,
-		ui: {
-			tab: "tools",
-			label: "Salesforce CLI",
-			description:
-				"Enable sf_* tools for Salesforce org management, SOQL queries, and pipeline reporting via sf CLI",
-		},
-	},
-
 	"web_search.enabled": {
 		type: "boolean",
 		default: true,

package/src/extensibility/extensions/loader.ts

@@ -28,6 +28,7 @@ import type {
 	LoadExtensionsResult,
 	MessageRenderer,
 	RegisteredCommand,
+	ServiceStatusContribution,
 	ToolDefinition,
 } from "./types";
 
@@ -176,6 +177,10 @@ class ConcreteExtensionAPI implements ExtensionAPI, IExtensionRuntime {
 		this.extension.messageRenderers.set(customType, renderer as MessageRenderer);
 	}
 
+	registerServiceStatus(contribution: ServiceStatusContribution): void {
+		this.extension.serviceStatuses.set(contribution.name, contribution);
+	}
+
 	getFlag(name: string): boolean | string | undefined {
 		if (!this.extension.flags.has(name)) return undefined;
 		return this.runtime.flagValues.get(name);
@@ -257,6 +262,7 @@ function createExtension(extensionPath: string, resolvedPath: string): Extension
 		commands: new Map(),
 		flags: new Map(),
 		shortcuts: new Map(),
+		serviceStatuses: new Map(),
 	};
 }
 

package/src/extensibility/extensions/runner.ts

@@ -36,6 +36,7 @@ import type {
 	RegisteredTool,
 	ResourcesDiscoverEvent,
 	ResourcesDiscoverResult,
+	ServiceStatusContribution,
 	SessionBeforeBranchResult,
 	SessionBeforeCompactResult,
 	SessionBeforeSwitchResult,
@@ -251,6 +252,17 @@ export class ExtensionRunner {
 		return tools;
 	}
 
+	/** Get all registered service status contributions from all extensions. */
+	getAllRegisteredServiceStatuses(): ServiceStatusContribution[] {
+		const statuses: ServiceStatusContribution[] = [];
+		for (const ext of this.extensions) {
+			for (const contribution of ext.serviceStatuses.values()) {
+				statuses.push(contribution);
+			}
+		}
+		return statuses;
+	}
+
 	getFlags(): Map<string, ExtensionFlag> {
 		const allFlags = new Map<string, ExtensionFlag>();
 		for (const ext of this.extensions) {

package/src/extensibility/extensions/types.ts

@@ -933,6 +933,19 @@ export interface RegisteredCommand {
 	handler: (args: string, ctx: ExtensionCommandContext) => Promise<void>;
 }
 
+// ============================================================================
+// Service Status Contributions
+// ============================================================================
+
+export interface ServiceStatusContribution {
+	name: string;
+	check: () => Promise<{ state: "connected" | "unauthenticated" | "unavailable"; hint?: string }>;
+	fix?: {
+		prompt: string;
+		command: string[];
+	};
+}
+
 // ============================================================================
 // Extension API
 // ============================================================================
@@ -1064,6 +1077,9 @@ export interface ExtensionAPI {
 	/** Register a custom renderer for CustomMessageEntry. */
 	registerMessageRenderer<T = unknown>(customType: string, renderer: MessageRenderer<T>): void;
 
+	/** Register a service status check for the welcome screen. */
+	registerServiceStatus(contribution: ServiceStatusContribution): void;
+
 	// =========================================================================
 	// Actions
 	// =========================================================================
@@ -1351,6 +1367,7 @@ export interface Extension {
 	commands: Map<string, RegisteredCommand>;
 	flags: Map<string, ExtensionFlag>;
 	shortcuts: Map<KeyId, ExtensionShortcut>;
+	serviceStatuses: Map<string, ServiceStatusContribution>;
 }
 
 /** Result of loading extensions. */

package/src/internal-urls/build-info-runtime.ts

@@ -209,7 +209,6 @@ export function renderAboutDoc(info: RuntimeBuildInfo, context: ContextStatus |
 		"SSH remote execution, image generation and analysis.",
 		"",
 		"SE specialization: F5 XC API integration (xcsh_api, api-catalog, api-spec),",
-		"Salesforce pipeline intelligence (sf_query, xcsh://salesforce),",
 		"F5 XC federated product docs (llms.txt hierarchy),",
 		"user/computer profiling (xcsh://user, xcsh://computer),",
 		"SE-specific subagents (deal-analyst, status-operator, cli-operator, github-ops).",

package/src/internal-urls/build-info.generated.ts

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "18.87.3",
-	"commit": "13a382a1bfb96241f9e1710c1bba113f7e3ea0f5",
-	"shortCommit": "13a382a",
+	"version": "18.89.0",
+	"commit": "e4462e8d079ea1a120c1f9253c79f78eb4d01e65",
+	"shortCommit": "e4462e8",
 	"branch": "main",
-	"tag": "v18.87.3",
-	"commitDate": "2026-05-29T17:31:47Z",
-	"buildDate": "2026-05-30T00:23:51.633Z",
+	"tag": "v18.89.0",
+	"commitDate": "2026-05-31T23:55:12Z",
+	"buildDate": "2026-06-01T00:31:02.396Z",
 	"dirty": true,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/13a382a1bfb96241f9e1710c1bba113f7e3ea0f5",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.87.3"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/e4462e8d079ea1a120c1f9253c79f78eb4d01e65",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.89.0"
 };

package/src/internal-urls/index.ts

@@ -36,7 +36,6 @@ export * from "./parse";
 export * from "./profile-collectors";
 export * from "./router";
 export * from "./rule-protocol";
-export * from "./salesforce-context";
 export * from "./skill-protocol";
 export * from "./terraform-resolve";
 export type * from "./terraform-types";

package/src/internal-urls/profile-collectors.ts

@@ -14,109 +14,6 @@ export interface ProfileCollector {
 	collect(): Promise<Partial<UserProfile>>;
 }
 
-// ---------------------------------------------------------------------------
-// Salesforce
-// ---------------------------------------------------------------------------
-
-const salesforceCollector: ProfileCollector = {
-	id: "salesforce",
-	name: "Salesforce",
-
-	async available(): Promise<boolean> {
-		if (!$which("sf")) return false;
-		try {
-			const proc = await $`sf org display --json`.quiet().nothrow();
-			if (proc.exitCode !== 0) return false;
-			const parsed = JSON.parse(proc.stdout.toString()) as Record<string, unknown>;
-			const result = parsed.result as Record<string, unknown> | undefined;
-			return typeof result?.username === "string" && result.username.length > 0;
-		} catch {
-			return false;
-		}
-	},
-
-	async collect(): Promise<Partial<UserProfile>> {
-		try {
-			// Get username
-			const orgProc = await $`sf org display --json`.quiet().nothrow();
-			if (orgProc.exitCode !== 0) return {};
-			const orgData = JSON.parse(orgProc.stdout.toString()) as Record<string, unknown>;
-			const orgResult = orgData.result as Record<string, unknown> | undefined;
-			const username = orgResult?.username as string | undefined;
-			if (!username) return {};
-
-			// Build and run SOQL
-			const soql = `SELECT Id, Username, FirstName, LastName, Email, Title, Department, Division, CompanyName, AboutMe, ManagerId, Manager.Name, Manager.Email, UserRole.Name, Profile.Name, Street, City, State, PostalCode, Country, Phone, MobilePhone FROM User WHERE Username = '${username}'`;
-			const queryProc = await $`sf data query --query ${soql} --json`.quiet().nothrow();
-			if (queryProc.exitCode !== 0) return {};
-
-			const queryData = JSON.parse(queryProc.stdout.toString()) as Record<string, unknown>;
-			const queryResult = queryData.result as Record<string, unknown> | undefined;
-			const records = queryResult?.records as Record<string, unknown>[] | undefined;
-			const rec = records?.[0];
-			if (!rec) return {};
-
-			// Map fields
-			const profile: Partial<UserProfile> = {};
-
-			if (rec.FirstName) profile.givenName = rec.FirstName as string;
-			if (rec.LastName) profile.familyName = rec.LastName as string;
-			if (rec.Email) profile.email = rec.Email as string;
-
-			const phone = (rec.Phone || rec.MobilePhone) as string | undefined;
-			if (phone) profile.telephone = phone;
-
-			if (rec.Title) profile.jobTitle = rec.Title as string;
-			if (rec.Department) profile.department = rec.Department as string;
-			if (rec.Division) profile.division = rec.Division as string;
-
-			const companyName = (rec.CompanyName as string) || "F5";
-			profile.worksFor = { name: companyName };
-
-			// Manager
-			const mgr = rec.Manager as Record<string, unknown> | undefined;
-			if (mgr) {
-				const mgrName = mgr.Name as string | undefined;
-				const mgrEmail = mgr.Email as string | undefined;
-				if (mgrName || mgrEmail) {
-					profile.manager = {};
-					if (mgrName) {
-						const parts = mgrName.split(" ");
-						profile.manager.givenName = parts[0];
-						if (parts.length > 1) profile.manager.familyName = parts.slice(1).join(" ");
-					}
-					if (mgrEmail) profile.manager.email = mgrEmail;
-				}
-			}
-
-			// Address
-			const street = rec.Street as string | undefined;
-			const city = rec.City as string | undefined;
-			const state = rec.State as string | undefined;
-			const postalCode = rec.PostalCode as string | undefined;
-			const country = rec.Country as string | undefined;
-			if (street || city || state || postalCode || country) {
-				profile.address = {};
-				if (street) profile.address.streetAddress = street;
-				if (city) profile.address.addressLocality = city;
-				if (state) profile.address.addressRegion = state;
-				if (postalCode) profile.address.postalCode = postalCode;
-				if (country) profile.address.addressCountry = country;
-			}
-
-			// Identifiers
-			if (rec.Id) {
-				profile.identifiers = { salesforceId: rec.Id as string };
-			}
-
-			return profile;
-		} catch (err: unknown) {
-			logger.debug("salesforce collector failed", { error: err });
-			return {};
-		}
-	},
-};
-
 // ---------------------------------------------------------------------------
 // GitHub
 // ---------------------------------------------------------------------------
@@ -225,4 +122,4 @@ const systemCollector: ProfileCollector = {
 // Registry
 // ---------------------------------------------------------------------------
 
-export const PROFILE_COLLECTORS: readonly ProfileCollector[] = [salesforceCollector, githubCollector, systemCollector];
+export const PROFILE_COLLECTORS: readonly ProfileCollector[] = [githubCollector, systemCollector];

package/src/internal-urls/user-profile.ts

@@ -45,27 +45,27 @@ export interface UserProfile {
 	description?: string;
 	image?: string;
 	sameAs?: string[];
-	identifiers?: { github?: string; twitter?: string; salesforceId?: string };
-	/** User-authored: short role label, e.g. 'SE', 'AE', 'CSM', 'SA'. Set manually; not derived from Salesforce. */
+	identifiers?: { github?: string; twitter?: string };
+	/** User-authored: short role label, e.g. 'SE', 'AE', 'CSM', 'SA'. Set manually. */
 	role?: string;
 	/**
 	 * User-authored: confirmed partner (AE/SE counterpart, CSM, etc.).
-	 * Set manually in user-profile.json. Survives Salesforce re-seeds.
+	 * Set manually in user-profile.json.
 	 */
 	partner?: {
-		/** Salesforce User Id — used to scope pipeline queries */
+		/** Partner user ID — used to scope pipeline queries */
 		id?: string;
 		name: string;
 		title?: string;
 		/** Short role label, e.g. 'AE', 'SE', 'CSM' */
 		role?: string;
 	};
-	/** User-authored: primary territory names. Exact Salesforce field values. Scopes pipeline reports. */
+	/** User-authored: primary territory names. Scopes pipeline reports. */
 	territories?: string[];
 	/** User-authored: quarterly quota target in dollars. Used for coverage ratio calculations. */
 	quota?: number;
 	observations?: UserProfileObservation[];
-	sources?: { salesforce?: string; github?: string; system?: string; conversation?: string };
+	sources?: { github?: string; system?: string; conversation?: string };
 	updatedAt?: string;
 }
 
@@ -164,9 +164,7 @@ export function renderProfileMarkdown(profile: UserProfile): string {
 
 	const isEmpty = !profile.givenName && !profile.familyName && !profile.email && !profile.jobTitle;
 	if (isEmpty) {
-		sections.push(
-			"No profile data yet. Use `xcsh://user?seed=true` to populate from Salesforce, GitHub, and system sources.\n",
-		);
+		sections.push("No profile data yet. Use `xcsh://user?seed=true` to populate from GitHub and system sources.\n");
 		sections.push("Profile facts can also be added progressively during conversation.\n");
 		return sections.join("\n");
 	}
@@ -267,7 +265,6 @@ export function renderProfileMarkdown(profile: UserProfile): string {
 	if (profile.description) onlineLines.push(`- **Bio:** ${profile.description}`);
 	if (profile.identifiers?.github) onlineLines.push(`- **GitHub:** ${profile.identifiers.github}`);
 	if (profile.identifiers?.twitter) onlineLines.push(`- **Twitter/X:** ${profile.identifiers.twitter}`);
-	if (profile.identifiers?.salesforceId) onlineLines.push(`- **Salesforce ID:** ${profile.identifiers.salesforceId}`);
 	if (profile.sameAs && profile.sameAs.length > 0) {
 		for (const link of profile.sameAs) {
 			onlineLines.push(`- **Profile:** ${link}`);
@@ -293,7 +290,6 @@ export function renderProfileMarkdown(profile: UserProfile): string {
 		sections.push("\n---\n");
 		sections.push("**Sources:**");
 		const srcLines: string[] = [];
-		if (profile.sources.salesforce) srcLines.push(`Salesforce: ${profile.sources.salesforce}`);
 		if (profile.sources.github) srcLines.push(`GitHub: ${profile.sources.github}`);
 		if (profile.sources.system) srcLines.push(`System: ${profile.sources.system}`);
 		if (profile.sources.conversation) srcLines.push(`Conversation: ${profile.sources.conversation}`);

package/src/internal-urls/xcsh-protocol.ts

@@ -21,6 +21,9 @@
  * - xcsh://terraform/{category}/{resource} - Self-contained resource doc
  * - xcsh://user - Human user profile
  * - xcsh://user?seed=true - Seed profile from sources and render
+ *
+ * Note: Salesforce context (xcsh://salesforce) has been extracted to the
+ * salesforce plugin. See packages/salesforce/ for the standalone implementation.
  */
 import * as path from "node:path";
 import { logger } from "@f5xc-salesdemos/pi-utils";
@@ -37,7 +40,6 @@ import type {
 import { getRuntimeBuildInfo, type RuntimeBuildInfo, renderAboutDoc } from "./build-info-runtime";
 import { loadComputerProfile, renderComputerProfileMarkdown, seedComputerProfile } from "./computer-profile";
 import { EMBEDDED_DOC_FILENAMES, EMBEDDED_DOCS } from "./docs-index.generated";
-import { loadSalesforceContext, renderSalesforceContextMarkdown, seedSalesforceContext } from "./salesforce-context";
 import { createTerraformResolver, type TerraformResolver } from "./terraform-resolve";
 import type { TerraformIndex } from "./terraform-types";
 import type { InternalResource, InternalUrl, ProtocolHandler } from "./types";
@@ -51,8 +53,6 @@ const BRANDING_HOST = "branding";
 const TERRAFORM_HOST = "terraform";
 const USER_ROUTE = "user";
 const COMPUTER_ROUTE = "computer";
-const SALESFORCE_ROUTE = "salesforce";
-
 const EMPTY_INDEX: ApiSpecIndex = { version: "unavailable", timestamp: "", domains: [] };
 const EMPTY_CATALOG_INDEX: ApiCatalogIndex = {
 	version: "unavailable",
@@ -311,10 +311,6 @@ export class InternalDocsProtocolHandler implements ProtocolHandler {
 			return this.#resolveComputerProfile(url);
 		}
 
-		if (host === SALESFORCE_ROUTE) {
-			return this.#resolveSalesforceContext(url);
-		}
-
 		const pathname = url.rawPathname ?? url.pathname;
 		const filename = host ? (pathname && pathname !== "/" ? host + pathname : host) : "";
 
@@ -357,22 +353,6 @@ export class InternalDocsProtocolHandler implements ProtocolHandler {
 		};
 	}
 
-	async #resolveSalesforceContext(url: InternalUrl): Promise<InternalResource> {
-		const params = new URLSearchParams(url.search);
-		const shouldRefresh = params.get("refresh") === "true";
-
-		const ctx = shouldRefresh ? await seedSalesforceContext() : await loadSalesforceContext();
-		const content = renderSalesforceContextMarkdown(ctx);
-
-		return {
-			url: url.href,
-			content,
-			contentType: "text/markdown",
-			size: Buffer.byteLength(content, "utf-8"),
-			sourcePath: `xcsh://${SALESFORCE_ROUTE}`,
-		};
-	}
-
 	async #listDocs(url: InternalUrl): Promise<InternalResource> {
 		if (EMBEDDED_DOC_FILENAMES.length === 0) {
 			throw new Error("No documentation files found");
@@ -387,7 +367,6 @@ export class InternalDocsProtocolHandler implements ProtocolHandler {
 		const brandingEntry = `- [${BRANDING_HOST}](${SCHEME_PREFIX}${BRANDING_HOST}) — F5 XC branding and legacy name mapping (v${branding.version})`;
 		const userEntry = `- [${USER_ROUTE}](${SCHEME_PREFIX}${USER_ROUTE}) — human user profile`;
 		const computerEntry = `- [${COMPUTER_ROUTE}](${SCHEME_PREFIX}${COMPUTER_ROUTE}) — machine hardware and environment profile`;
-		const salesforceEntry = `- [${SALESFORCE_ROUTE}](${SCHEME_PREFIX}${SALESFORCE_ROUTE}) — Salesforce pipeline context and team discovery`;
 		const tf = loadTerraformIndex();
 		const terraformEntry = `- [${TERRAFORM_HOST}/](${SCHEME_PREFIX}${TERRAFORM_HOST}/) — F5 XC Terraform provider (${Object.keys(tf.resources).length} resources, v${tf.version})`;
 		const listing = [
@@ -398,10 +377,9 @@ export class InternalDocsProtocolHandler implements ProtocolHandler {
 			terraformEntry,
 			userEntry,
 			computerEntry,
-			salesforceEntry,
 			...EMBEDDED_DOC_FILENAMES.map(f => `- [${f}](${SCHEME_PREFIX}${f})`),
 		].join("\n");
-		const totalCount = EMBEDDED_DOC_FILENAMES.length + 8;
+		const totalCount = EMBEDDED_DOC_FILENAMES.length + 7;
 		const content = `# Documentation\n\n${totalCount} files available:\n\n${listing}\n`;
 
 		return {

package/src/modes/components/welcome-checks.ts

@@ -234,104 +234,6 @@ export async function checkGitLabStatus(cwd: string): Promise<WelcomeGitLabStatu
 	}
 }
 
-export type SalesforceCheckState = "connected" | "auth_error" | "session_expired" | "not_configured";
-
-export interface WelcomeSalesforceStatus {
-	state: SalesforceCheckState;
-	username?: string;
-	orgAlias?: string;
-	instanceUrl?: string;
-}
-
-/** Idempotent startup check: sf installed -> org list -> default org -> display status. */
-export async function checkSalesforceStatus(_cwd: string): Promise<WelcomeSalesforceStatus | undefined> {
-	try {
-		if (!$which("sf")) return undefined;
-
-		// Suppress telemetry consent nag (idempotent)
-		await $`sf config set disable-telemetry true --global`.quiet().nothrow();
-
-		// Step 1: Get org list
-		const listResult = await $`sf org list --json`.quiet().nothrow();
-		if (listResult.exitCode !== 0) return { state: "auth_error" };
-
-		let listData: {
-			result?: {
-				nonScratchOrgs?: unknown[];
-				sandboxes?: unknown[];
-				scratchOrgs?: unknown[];
-				devHubs?: unknown[];
-				other?: unknown[];
-			};
-		};
-		try {
-			listData = JSON.parse(listResult.text());
-		} catch {
-			return { state: "auth_error" };
-		}
-
-		const r = listData.result ?? {};
-		const seen = new Set<string>();
-		const allRawOrgs = (
-			[
-				...(r.nonScratchOrgs ?? []),
-				...(r.sandboxes ?? []),
-				...(r.scratchOrgs ?? []),
-				...(r.devHubs ?? []),
-				...(r.other ?? []),
-			] as Record<string, unknown>[]
-		).filter(org => {
-			const id = String(org.orgId ?? org.orgid ?? "");
-			if (!id || seen.has(id)) return false;
-			seen.add(id);
-			return true;
-		});
-
-		if (allRawOrgs.length === 0) return { state: "auth_error" };
-
-		// Step 2: Find default org (normalize raw CLI fields)
-		const defaultRaw = allRawOrgs.find(
-			org =>
-				(typeof org.defaultMarker === "string" && org.defaultMarker.includes("(U)")) ||
-				org.isDefaultUsername === true,
-		);
-
-		if (!defaultRaw) {
-			return { state: "not_configured", username: allRawOrgs[0]?.username as string | undefined };
-		}
-
-		const alias = (defaultRaw.alias ?? defaultRaw.username) as string;
-
-		// Step 3: Display org details
-		const displayResult = await $`sf org display --target-org ${alias} --json`.quiet().nothrow();
-		if (displayResult.exitCode !== 0) {
-			return { state: "session_expired", username: defaultRaw.username as string | undefined, orgAlias: alias };
-		}
-
-		let displayData: { result?: Record<string, unknown> };
-		try {
-			displayData = JSON.parse(displayResult.text());
-		} catch {
-			return { state: "session_expired", username: defaultRaw.username as string | undefined, orgAlias: alias };
-		}
-
-		const result = displayData.result;
-		if (!result || result.connectedStatus !== "Connected") {
-			return { state: "session_expired", username: defaultRaw.username as string | undefined, orgAlias: alias };
-		}
-
-		return {
-			state: "connected",
-			username: result.username as string | undefined,
-			orgAlias: alias,
-			instanceUrl: result.instanceUrl as string | undefined,
-		};
-	} catch (err) {
-		logger.warn("Salesforce startup check failed", { error: String(err) });
-		return { state: "auth_error" };
-	}
-}
-
 export type GitHubCheckState = "connected" | "auth_error";
 
 export interface WelcomeGitHubStatus {
@@ -391,20 +293,6 @@ export function mapGitLabStatus(status: WelcomeGitLabStatus | undefined): Servic
 	}
 }
 
-export function mapSalesforceStatus(status: WelcomeSalesforceStatus | undefined): ServiceStatus {
-	if (!status) return { name: "Salesforce", state: "unavailable", hint: "not installed" };
-	switch (status.state) {
-		case "connected":
-			return { name: "Salesforce", state: "connected" };
-		case "session_expired":
-			return { name: "Salesforce", state: "unauthenticated", hint: "session expired, run: sf org login web" };
-		case "not_configured":
-			return { name: "Salesforce", state: "unauthenticated", hint: "run: sf org login web --set-default" };
-		default:
-			return { name: "Salesforce", state: "unauthenticated", hint: "run: sf org login web" };
-	}
-}
-
 export function mapGitHubStatus(status: WelcomeGitHubStatus | undefined): ServiceStatus {
 	if (!status) return { name: "GitHub", state: "unavailable", hint: "not installed" };
 	switch (status.state) {
@@ -609,7 +497,6 @@ export function getFixableServices(statuses: {
 	gcloud: WelcomeGcloudStatus | undefined;
 	github: WelcomeGitHubStatus | undefined;
 	gitlab: WelcomeGitLabStatus | undefined;
-	salesforce: WelcomeSalesforceStatus | undefined;
 }): FixableService[] {
 	const fixable: FixableService[] = [];
 
@@ -629,14 +516,6 @@ export function getFixableServices(statuses: {
 			recheck: async () => mapGitHubStatus(await checkGitHubStatus()),
 		});
 	}
-	if (statuses.salesforce?.state === "session_expired") {
-		fixable.push({
-			name: "Salesforce",
-			prompt: "Salesforce session expired",
-			command: ["sf", "org", "login", "web"],
-			recheck: async () => mapSalesforceStatus(await checkSalesforceStatus(getProjectDir())),
-		});
-	}
 	if (statuses.azure?.state === "auth_error") {
 		fixable.push({
 			name: "Azure",