Merge branch 'main' into bb/ux/message-feed

* main:
  Changeset version bump (#7980)
  chore: add changeset for v3.28.2 (#7979)
  Remove chevrons from chat buttons (#7970)
  Disable Roomote Control on logout (#7976)
  ux: Smaller and more subtle auto-approve UI (#7894)
  feat: add Qwen3 Next 80B A3B models to chutes provider (#7948)
  fix: include API key in Ollama /api/tags requests (#7903)
  fix: make nested git repository warning persistent with path info (#7885)
  Add a little padding to the cloudview (#7954)

Author: brunobergher

CHANGELOG.md

@@ -1,5 +1,19 @@
 # Roo Code Changelog
 
+## [3.28.2] - 2025-09-14
+
+![3.28.2 Release - Auto-approve improvements](/releases/3.28.2-release.png)
+
+- Improve auto-approve UI with smaller and more subtle design (thanks @brunobergher!)
+- Fix: Message queue re-queue loop in Task.ask() causing performance issues (#7861 by @hannesrudolph, PR by @daniel-lxs)
+- Fix: Restrict @-mention parsing to line-start or whitespace boundaries to prevent false triggers (#7875 by @hannesrudolph, PR by @app/roomote)
+- Fix: Make nested git repository warning persistent with path info for better visibility (#7884 by @hannesrudolph, PR by @app/roomote)
+- Fix: Include API key in Ollama /api/tags requests for authenticated instances (#7902 by @ItsOnlyBinary, PR by @app/roomote)
+- Fix: Preserve original first message context during conversation condensing (thanks @daniel-lxs!)
+- Add Qwen3 Next 80B A3B models to chutes provider (thanks @daniel-lxs!)
+- Disable Roomote Control on logout for better security (thanks @cte!)
+- Add padding to the cloudview for better visual spacing (thanks @mrubens!)
+
 ## [3.28.1] - 2025-09-11
 
 ![3.28.1 Release - Kangaroo riding rocket to the clouds](/releases/3.28.1-release.png)

packages/cloud/src/bridge/BridgeOrchestrator.ts

@@ -59,13 +59,28 @@ export class BridgeOrchestrator {
 		return BridgeOrchestrator.instance
 	}
 
-	public static isEnabled(user?: CloudUserInfo | null, remoteControlEnabled?: boolean): boolean {
-		return !!(user?.id && user.extensionBridgeEnabled && remoteControlEnabled)
+	public static isEnabled(user: CloudUserInfo | null, remoteControlEnabled: boolean): boolean {
+		// Always disabled if signed out.
+		if (!user) {
+			return false
+		}
+
+		// Disabled by the user's organization?
+		if (!user.extensionBridgeEnabled) {
+			return false
+		}
+
+		// Disabled by the user?
+		if (!remoteControlEnabled) {
+			return false
+		}
+
+		return true
 	}
 
 	public static async connectOrDisconnect(
-		userInfo: CloudUserInfo | null,
-		remoteControlEnabled: boolean | undefined,
+		userInfo: CloudUserInfo,
+		remoteControlEnabled: boolean,
 		options: BridgeOrchestratorOptions,
 	): Promise<void> {
 		if (BridgeOrchestrator.isEnabled(userInfo, remoteControlEnabled)) {

packages/types/src/providers/chutes.ts

@@ -32,6 +32,8 @@ export type ChutesModelId =
 	| "moonshotai/Kimi-K2-Instruct-75k"
 	| "moonshotai/Kimi-K2-Instruct-0905"
 	| "Qwen/Qwen3-235B-A22B-Thinking-2507"
+	| "Qwen/Qwen3-Next-80B-A3B-Instruct"
+	| "Qwen/Qwen3-Next-80B-A3B-Thinking"
 
 export const chutesDefaultModelId: ChutesModelId = "deepseek-ai/DeepSeek-R1-0528"
 
@@ -308,4 +310,24 @@ export const chutesModels = {
 		outputPrice: 0.31202496,
 		description: "Qwen3 235B A22B Thinking 2507 model with 262K context window.",
 	},
+	"Qwen/Qwen3-Next-80B-A3B-Instruct": {
+		maxTokens: 32768,
+		contextWindow: 131072,
+		supportsImages: false,
+		supportsPromptCache: false,
+		inputPrice: 0,
+		outputPrice: 0,
+		description:
+			"Fast, stable instruction-tuned model optimized for complex tasks, RAG, and tool use without thinking traces.",
+	},
+	"Qwen/Qwen3-Next-80B-A3B-Thinking": {
+		maxTokens: 32768,
+		contextWindow: 131072,
+		supportsImages: false,
+		supportsPromptCache: false,
+		inputPrice: 0,
+		outputPrice: 0,
+		description:
+			"Reasoning-first model with structured thinking traces for multi-step problems, math proofs, and code synthesis.",
+	},
 } as const satisfies Record<string, ModelInfo>

releases/3.28.2-release.png

@@ -0,0 +1,257 @@
+// npx vitest run __tests__/extension.spec.ts
+
+import type * as vscode from "vscode"
+import type { AuthState } from "@roo-code/types"
+
+vi.mock("vscode", () => ({
+	window: {
+		createOutputChannel: vi.fn().mockReturnValue({
+			appendLine: vi.fn(),
+		}),
+		registerWebviewViewProvider: vi.fn(),
+		registerUriHandler: vi.fn(),
+		tabGroups: {
+			onDidChangeTabs: vi.fn(),
+		},
+		onDidChangeActiveTextEditor: vi.fn(),
+	},
+	workspace: {
+		registerTextDocumentContentProvider: vi.fn(),
+		getConfiguration: vi.fn().mockReturnValue({
+			get: vi.fn().mockReturnValue([]),
+		}),
+		createFileSystemWatcher: vi.fn().mockReturnValue({
+			onDidCreate: vi.fn(),
+			onDidChange: vi.fn(),
+			onDidDelete: vi.fn(),
+			dispose: vi.fn(),
+		}),
+		onDidChangeWorkspaceFolders: vi.fn(),
+	},
+	languages: {
+		registerCodeActionsProvider: vi.fn(),
+	},
+	commands: {
+		executeCommand: vi.fn(),
+	},
+	env: {
+		language: "en",
+	},
+	ExtensionMode: {
+		Production: 1,
+	},
+}))
+
+vi.mock("@dotenvx/dotenvx", () => ({
+	config: vi.fn(),
+}))
+
+const mockBridgeOrchestratorDisconnect = vi.fn().mockResolvedValue(undefined)
+
+vi.mock("@roo-code/cloud", () => ({
+	CloudService: {
+		createInstance: vi.fn(),
+		hasInstance: vi.fn().mockReturnValue(true),
+		get instance() {
+			return {
+				off: vi.fn(),
+				on: vi.fn(),
+				getUserInfo: vi.fn().mockReturnValue(null),
+				isTaskSyncEnabled: vi.fn().mockReturnValue(false),
+			}
+		},
+	},
+	BridgeOrchestrator: {
+		disconnect: mockBridgeOrchestratorDisconnect,
+	},
+	getRooCodeApiUrl: vi.fn().mockReturnValue("https://app.roocode.com"),
+}))
+
+vi.mock("@roo-code/telemetry", () => ({
+	TelemetryService: {
+		createInstance: vi.fn().mockReturnValue({
+			register: vi.fn(),
+			setProvider: vi.fn(),
+			shutdown: vi.fn(),
+		}),
+		get instance() {
+			return {
+				register: vi.fn(),
+				setProvider: vi.fn(),
+				shutdown: vi.fn(),
+			}
+		},
+	},
+	PostHogTelemetryClient: vi.fn(),
+}))
+
+vi.mock("../utils/outputChannelLogger", () => ({
+	createOutputChannelLogger: vi.fn().mockReturnValue(vi.fn()),
+	createDualLogger: vi.fn().mockReturnValue(vi.fn()),
+}))
+
+vi.mock("../shared/package", () => ({
+	Package: {
+		name: "test-extension",
+		outputChannel: "Test Output",
+		version: "1.0.0",
+	},
+}))
+
+vi.mock("../shared/language", () => ({
+	formatLanguage: vi.fn().mockReturnValue("en"),
+}))
+
+vi.mock("../core/config/ContextProxy", () => ({
+	ContextProxy: {
+		getInstance: vi.fn().mockResolvedValue({
+			getValue: vi.fn(),
+			setValue: vi.fn(),
+			getValues: vi.fn().mockReturnValue({}),
+			getProviderSettings: vi.fn().mockReturnValue({}),
+		}),
+	},
+}))
+
+vi.mock("../integrations/editor/DiffViewProvider", () => ({
+	DIFF_VIEW_URI_SCHEME: "test-diff-scheme",
+}))
+
+vi.mock("../integrations/terminal/TerminalRegistry", () => ({
+	TerminalRegistry: {
+		initialize: vi.fn(),
+		cleanup: vi.fn(),
+	},
+}))
+
+vi.mock("../services/mcp/McpServerManager", () => ({
+	McpServerManager: {
+		cleanup: vi.fn().mockResolvedValue(undefined),
+		getInstance: vi.fn().mockResolvedValue(null),
+		unregisterProvider: vi.fn(),
+	},
+}))
+
+vi.mock("../services/code-index/manager", () => ({
+	CodeIndexManager: {
+		getInstance: vi.fn().mockReturnValue(null),
+	},
+}))
+
+vi.mock("../services/mdm/MdmService", () => ({
+	MdmService: {
+		createInstance: vi.fn().mockResolvedValue(null),
+	},
+}))
+
+vi.mock("../utils/migrateSettings", () => ({
+	migrateSettings: vi.fn().mockResolvedValue(undefined),
+}))
+
+vi.mock("../utils/autoImportSettings", () => ({
+	autoImportSettings: vi.fn().mockResolvedValue(undefined),
+}))
+
+vi.mock("../extension/api", () => ({
+	API: vi.fn().mockImplementation(() => ({})),
+}))
+
+vi.mock("../activate", () => ({
+	handleUri: vi.fn(),
+	registerCommands: vi.fn(),
+	registerCodeActions: vi.fn(),
+	registerTerminalActions: vi.fn(),
+	CodeActionProvider: vi.fn().mockImplementation(() => ({
+		providedCodeActionKinds: [],
+	})),
+}))
+
+vi.mock("../i18n", () => ({
+	initializeI18n: vi.fn(),
+}))
+
+describe("extension.ts", () => {
+	let mockContext: vscode.ExtensionContext
+	let authStateChangedHandler:
+		| ((data: { state: AuthState; previousState: AuthState }) => void | Promise<void>)
+		| undefined
+
+	beforeEach(() => {
+		vi.clearAllMocks()
+		mockBridgeOrchestratorDisconnect.mockClear()
+
+		mockContext = {
+			extensionPath: "/test/path",
+			globalState: {
+				get: vi.fn().mockReturnValue(undefined),
+				update: vi.fn(),
+			},
+			subscriptions: [],
+		} as unknown as vscode.ExtensionContext
+
+		authStateChangedHandler = undefined
+	})
+
+	test("authStateChangedHandler calls BridgeOrchestrator.disconnect when logged-out event fires", async () => {
+		const { CloudService, BridgeOrchestrator } = await import("@roo-code/cloud")
+
+		// Capture the auth state changed handler.
+		vi.mocked(CloudService.createInstance).mockImplementation(async (_context, _logger, handlers) => {
+			if (handlers?.["auth-state-changed"]) {
+				authStateChangedHandler = handlers["auth-state-changed"]
+			}
+
+			return {
+				off: vi.fn(),
+				on: vi.fn(),
+				telemetryClient: null,
+			} as any
+		})
+
+		// Activate the extension.
+		const { activate } = await import("../extension")
+		await activate(mockContext)
+
+		// Verify handler was registered.
+		expect(authStateChangedHandler).toBeDefined()
+
+		// Trigger logout.
+		await authStateChangedHandler!({
+			state: "logged-out" as AuthState,
+			previousState: "logged-in" as AuthState,
+		})
+
+		// Verify BridgeOrchestrator.disconnect was called
+		expect(mockBridgeOrchestratorDisconnect).toHaveBeenCalled()
+	})
+
+	test("authStateChangedHandler does not call BridgeOrchestrator.disconnect for other states", async () => {
+		const { CloudService } = await import("@roo-code/cloud")
+
+		// Capture the auth state changed handler.
+		vi.mocked(CloudService.createInstance).mockImplementation(async (_context, _logger, handlers) => {
+			if (handlers?.["auth-state-changed"]) {
+				authStateChangedHandler = handlers["auth-state-changed"]
+			}
+
+			return {
+				off: vi.fn(),
+				on: vi.fn(),
+				telemetryClient: null,
+			} as any
+		})
+
+		// Activate the extension.
+		const { activate } = await import("../extension")
+		await activate(mockContext)
+
+		// Trigger login.
+		await authStateChangedHandler!({
+			state: "logged-in" as AuthState,
+			previousState: "logged-out" as AuthState,
+		})
+
+		// Verify BridgeOrchestrator.disconnect was NOT called.
+		expect(mockBridgeOrchestratorDisconnect).not.toHaveBeenCalled()
+	})
+})