add Windsurf support by Coooolfan · Pull Request #583 · steipete/CodexBar

Coooolfan · 2026-03-21T13:48:26Z

Read cached plan info from Windsurf's local SQLite database (state.vscdb) and surface daily/weekly quota usage.

Read cached plan info from Windsurf's local SQLite database (state.vscdb) and surface daily/weekly quota usage. - Decode WindsurfCachedPlanInfo with plan name, timestamps, message/flow-action counts, and daily/weekly quota percentages - Convert quota data to UsageSnapshot with proper nil handling: missing quotaUsage or individual percentage fields yield nil RateWindow instead of fabricating 0% usage - Support both .auto and .cli source modes (consistent with other localProbe providers) - Register provider across app, CLI, widget, and cost scanner - Add 10 tests covering JSON decoding, snapshot conversion, edge cases, and error handling

Copilot

Pull request overview

Adds a new Windsurf provider to CodexBar by reading cached plan/quota data from Windsurf’s local VS Code-style SQLite DB (state.vscdb) and surfacing daily/weekly usage in the app UI.

Changes:

Introduces a new Windsurf provider descriptor + implementation and registers it in the provider registries.
Implements WindsurfStatusProbe to read and decode windsurf.settings.cachedPlanInfo from state.vscdb, converting it into UsageSnapshot.
Updates UI/widget display name + color mappings and adds a Windsurf provider icon asset; adds decoding/conversion unit tests.

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
Tests/CodexBarTests/WindsurfStatusProbeTests.swift	Adds unit tests for JSON decoding and snapshot conversion; includes one probe error-case test.
Sources/CodexBarWidget/CodexBarWidgetViews.swift	Adds Windsurf label and widget color mapping.
Sources/CodexBarWidget/CodexBarWidgetProvider.swift	Explicitly marks Windsurf as not supported in widgets (enum mapping).
Sources/CodexBarCore/Vendored/CostUsage/CostUsageScanner.swift	Ensures Windsurf returns an empty cost report (unsupported).
Sources/CodexBarCore/Providers/Windsurf/WindsurfStatusProbe.swift	New SQLite-backed probe + model types + conversion to `UsageSnapshot`.
Sources/CodexBarCore/Providers/Windsurf/WindsurfProviderDescriptor.swift	Registers the Windsurf provider descriptor and local fetch strategy.
Sources/CodexBarCore/Providers/Providers.swift	Adds `.windsurf` to `UsageProvider` and `IconStyle`.
Sources/CodexBarCore/Providers/ProviderDescriptor.swift	Registers Windsurf in `ProviderDescriptorRegistry`.
Sources/CodexBarCLI/TokenAccountCLI.swift	Excludes Windsurf from token-account CLI snapshot mapping (returns nil).
Sources/CodexBar/UsageStore.swift	Adds Windsurf to debug-log switch as unimplemented.
Sources/CodexBar/Resources/ProviderIcon-windsurf.svg	Adds Windsurf provider icon asset.
Sources/CodexBar/Providers/Windsurf/WindsurfProviderImplementation.swift	Registers Windsurf provider implementation in the app target.
Sources/CodexBar/Providers/Shared/ProviderImplementationRegistry.swift	Adds Windsurf to provider implementation registry switch.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-21T13:53:10Z

Sources/CodexBarCore/Providers/Windsurf/WindsurfStatusProbe.swift

+import Foundation
+import SQLite3


WindsurfStatusProbe imports SQLite3 and hardcodes a macOS-only path, but the core module is compiled by the Linux test target as well. Other probes that rely on macOS-only frameworks wrap implementations in #if os(macOS) and provide an #else stub (see CursorStatusProbe.swift). Consider applying the same pattern here so non-macOS builds don’t fail to compile/link (and so the provider can report notSupported cleanly).

Suggested change

import Foundation

import SQLite3

import Foundation

#if canImport(SQLite3)

import SQLite3

#endif

Copilot · 2026-03-21T13:53:10Z

Sources/CodexBarCore/Providers/Windsurf/WindsurfStatusProbe.swift

+        guard let cString = sqlite3_column_text(stmt, 0) else {
+            throw WindsurfStatusProbeError.noData
+        }
+
+        let jsonString = String(cString: cString)
+        guard let jsonData = jsonString.data(using: .utf8) else {
+            throw WindsurfStatusProbeError.parseFailed("Invalid UTF-8 encoding")
+        }


The VSCode/Windsurf state.vscdb schema typically stores ItemTable.value as a BLOB (often UTF-16LE) rather than a UTF-8 TEXT column. Using sqlite3_column_text + .utf8 decoding will fail or silently truncate in those cases. It would be more robust to branch on sqlite3_column_type and decode TEXT vs BLOB (similar to FactoryLocalStorageImporter.decodeSQLiteValue), then feed the resulting string into JSON decoding.

Copilot · 2026-03-21T13:53:10Z

Sources/CodexBarCore/Providers/Windsurf/WindsurfStatusProbe.swift

+        guard sqlite3_step(stmt) == SQLITE_ROW else {
+            throw WindsurfStatusProbeError.noData


sqlite3_step can return error codes (e.g., SQLITE_BUSY, SQLITE_ERROR) that are currently mapped to .noData, which would mislead users and hide the underlying SQLite error. Consider switching on the step result and throwing .sqliteFailed(sqlite3_errmsg(db)) for non-SQLITE_ROW/SQLITE_DONE cases, reserving .noData for SQLITE_DONE (no matching row).

Suggested change

guard sqlite3_step(stmt) == SQLITE_ROW else {

throw WindsurfStatusProbeError.noData

let stepResult = sqlite3_step(stmt)

switch stepResult {

case SQLITE_ROW:

break

case SQLITE_DONE:

throw WindsurfStatusProbeError.noData

default:

let message = db.flatMap { String(cString: sqlite3_errmsg($0)) }

?? "unknown error (sqlite3_step returned code \(stepResult))"

throw WindsurfStatusProbeError.sqliteFailed(message)

Copilot · 2026-03-21T13:53:11Z

Tests/CodexBarTests/WindsurfStatusProbeTests.swift

+    @Test
+    func `probe throws dbNotFound for missing file`() {
+        let probe = WindsurfStatusProbe(dbPath: "/nonexistent/path/state.vscdb")
+
+        #expect(throws: WindsurfStatusProbeError.self) {
+            _ = try probe.fetch()
+        }


The probe is only tested for the missing-file case right now. Given the new SQLite parsing logic in WindsurfStatusProbe.fetch(), it would be valuable to add a test that creates a temporary SQLite DB with an ItemTable row for windsurf.settings.cachedPlanInfo and asserts fetch() succeeds (ideally covering both TEXT and BLOB/UTF-16 encodings).

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ba15392f0c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-03-21T13:58:32Z

Sources/CodexBarCore/Providers/Windsurf/WindsurfStatusProbe.swift

+        guard sqlite3_step(stmt) == SQLITE_ROW else {
+            throw WindsurfStatusProbeError.noData


Handle non-row sqlite3_step statuses explicitly

If sqlite3_step returns SQLITE_BUSY, SQLITE_ERROR, etc. while state.vscdb is being updated by Windsurf, this guard converts it into .noData. That makes refreshes report the misleading “Sign in to Windsurf first” path instead of the real SQLite failure, and because windsurf.local has no fallback the provider just looks unauthenticated. Please distinguish SQLITE_DONE from actual SQLite errors here and surface sqlite3_errmsg for the latter.

Useful? React with 👍 / 👎.

- Distinguish SQLITE_DONE (no data) from SQLITE_BUSY/ERROR (actual SQLite failure) instead of treating all non-ROW as noData - Decode column value by type (TEXT vs BLOB with UTF-16LE/UTF-8 fallback) to handle VSCode-style BLOB storage - Wrap SQLite-dependent code in #if os(macOS) with a stub for non-macOS builds

Coooolfan · 2026-03-22T05:39:26Z

used it for a day. there is a clear delay fetching the value. may need a different way to get the quota...

Add web API support for the Windsurf provider via Firebase token extraction from browser IndexedDB, token refresh, and ConnectRPC GetPlanStatus API. Implement full settings UI with Usage source (Auto/Web/Local) and Cookie source (Auto/Manual/Off) pickers, manual token input supporting both refresh and access tokens, and proper Keychain access gating.

Coooolfan · 2026-03-22T08:02:13Z

now adds a web-based usage fetching option for the Windsurf provider (via Firebase tokens from browser IndexedDB). It respects the Keychain toggle in Advanced settings.
also, you can manually paste their own token if preferred. Docs for how to grab the token are included as well.

Copilot AI review requested due to automatic review settings March 21, 2026 13:48

Copilot started reviewing on behalf of Coooolfan March 21, 2026 13:48 View session

Copilot AI reviewed Mar 21, 2026

View reviewed changes

chatgpt-codex-connector bot reviewed Mar 21, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

add Windsurf support#583

add Windsurf support#583
Coooolfan wants to merge 3 commits intosteipete:mainfrom
Coooolfan:feat/windsurf-provider

Coooolfan commented Mar 21, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 21, 2026

Uh oh!

Copilot AI Mar 21, 2026

Uh oh!

Copilot AI Mar 21, 2026

Uh oh!

Copilot AI Mar 21, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Mar 21, 2026

Uh oh!

Coooolfan commented Mar 22, 2026

Uh oh!

Coooolfan commented Mar 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		guard sqlite3_step(stmt) == SQLITE_ROW else {
		throw WindsurfStatusProbeError.noData

-        guard sqlite3_step(stmt) == SQLITE_ROW else {
-            throw WindsurfStatusProbeError.noData
+        let stepResult = sqlite3_step(stmt)
+        switch stepResult {
+        case SQLITE_ROW:
+            break
+        case SQLITE_DONE:
+            throw WindsurfStatusProbeError.noData
+        default:
+            let message = db.flatMap { String(cString: sqlite3_errmsg($0)) }
+                ?? "unknown error (sqlite3_step returned code \(stepResult))"
+            throw WindsurfStatusProbeError.sqliteFailed(message)

Conversation

Coooolfan commented Mar 21, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 21, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 21, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Mar 21, 2026

Choose a reason for hiding this comment

Uh oh!

Coooolfan commented Mar 22, 2026

Uh oh!

Coooolfan commented Mar 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants