Skip to content

CI: select Swift 6.2 toolchain for CodeQL#41787

Merged
BunsDev merged 3 commits intomainfrom
codex/codeql-swift-6-2
Mar 10, 2026
Merged

CI: select Swift 6.2 toolchain for CodeQL#41787
BunsDev merged 3 commits intomainfrom
codex/codeql-swift-6-2

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Mar 10, 2026

CodeQL’s Swift job was using the runner default toolchain, which exposed Swift 6.1 and failed against packages requiring Swift tools 6.2.

Change Summary

  • This updates .github/workflows/codeql.yml to select /Applications/Xcode_26.1.app prior to installing Swift utilities.
  • Then it logs the active Xcode/Swift versions so the runner configuration is explicit.

@openclaw-barnacle openclaw-barnacle bot added size: XS maintainer Maintainer-authored PR labels Mar 10, 2026
@BunsDev BunsDev marked this pull request as ready for review March 10, 2026 05:31
@BunsDev BunsDev self-assigned this Mar 10, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 10, 2026

Greptile Summary

This PR correctly fixes the CodeQL Swift job by explicitly selecting Xcode 26.1 (which provides Swift 6.2) before installing Swift build utilities, replacing the runner's default toolchain (Swift 6.1). The change directly resolves the stated problem: packages requiring Swift 6.2 no longer fail against the older 6.1 toolchain.

Changes reviewed:

  • .github/workflows/codeql.yml: Lines 97–100 now select /Applications/Xcode_26.1.app via sudo xcode-select -s and log the active Xcode/Swift versions for transparency.

The fix is sound and immediately functional.

Confidence Score: 5/5

  • This PR is safe to merge. It directly resolves the stated Swift 6.1/6.2 toolchain mismatch with a straightforward, correct fix.
  • The change correctly and immediately solves the described problem. The logic is sound: explicitly select Xcode 26.1 before installing Swift utilities, then log versions for transparency. No functional bugs or defects were identified in the implementation.
  • No files require special attention

Last reviewed commit: 1949e5c

@BunsDev BunsDev force-pushed the codex/codeql-swift-6-2 branch from 212ff1a to 8abc6c1 Compare March 10, 2026 06:22
@BunsDev BunsDev merged commit 5296147 into main Mar 10, 2026
13 checks passed
@BunsDev BunsDev deleted the codex/codeql-swift-6-2 branch March 10, 2026 06:22
@BunsDev
Copy link
Copy Markdown
Member Author

BunsDev commented Mar 10, 2026

Merged via squash.

Thanks @BunsDev!

mukhtharcm pushed a commit to hnykda/openclaw that referenced this pull request Mar 10, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
mrosmarin added a commit to mrosmarin/openclaw that referenced this pull request Mar 10, 2026
* main: (43 commits)
  docs: add openclaw#42173 to CHANGELOG — strip leaked model control tokens (openclaw#42216)
  Agents: align onPayload callback and OAuth imports
  docs: add Tengji (George) Zhang to maintainer table (openclaw#42190)
  fix: strip leaked model control tokens from user-facing text (openclaw#42173)
  Changelog: add unreleased March 9 entries
  chore: add .dev-state to .gitignore (openclaw#41848)
  fix(agents): avoid duplicate same-provider cooldown probes in fallback runs (openclaw#41711)
  fix(mattermost): preserve markdown formatting and native tables (openclaw#18655)
  feat(acp): add resumeSessionId to sessions_spawn for ACP session resume (openclaw#41847)
  ACPX: bump bundled acpx to 0.1.16 (openclaw#41975)
  mattermost: fix DM media upload for unprefixed user IDs (openclaw#29925)
  fix(msteams): use General channel conversation ID as team key for Bot Framework compatibility (openclaw#41838)
  fix(mattermost): read replyTo param in plugin handleAction send (openclaw#41176)
  fix(sandbox): pass real workspace to sessions_spawn when workspaceAccess is ro (openclaw#40757)
  fix(ui): replace Manual RPC text input with sorted method dropdown (openclaw#14967)
  CI: select Swift 6.2 toolchain for CodeQL (openclaw#41787)
  fix(agents): forward memory flush write path (openclaw#41761)
  fix(telegram): move network fallback to resolver-scoped dispatchers (openclaw#40740)
  fix(security): harden replaceMarkers() to catch space/underscore boundary marker variants (openclaw#35983)
  fix(web-search): recover OpenRouter Perplexity citations from message annotations (openclaw#40881)
  ...
Moshiii pushed a commit to Moshiii/openclaw that referenced this pull request Mar 11, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
Moshiii pushed a commit to Moshiii/openclaw that referenced this pull request Mar 11, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
frankekn pushed a commit to MoerAI/openclaw that referenced this pull request Mar 11, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
frankekn pushed a commit to Effet/openclaw that referenced this pull request Mar 11, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
frankekn pushed a commit to ImLukeF/openclaw that referenced this pull request Mar 11, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
dominicnunez pushed a commit to dominicnunez/openclaw that referenced this pull request Mar 11, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
dhoman pushed a commit to dhoman/chrono-claw that referenced this pull request Mar 11, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
ahelpercn pushed a commit to ahelpercn/openclaw that referenced this pull request Mar 12, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
Ruijie-Ysp pushed a commit to Ruijie-Ysp/clawdbot that referenced this pull request Mar 12, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
qipyle pushed a commit to qipyle/openclaw that referenced this pull request Mar 12, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
senw-developers pushed a commit to senw-developers/va-openclaw that referenced this pull request Mar 17, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
V-Gutierrez pushed a commit to V-Gutierrez/openclaw-vendor that referenced this pull request Mar 17, 2026
Merged via squash.

Prepared head SHA: 8abc6c1
Co-authored-by: BunsDev <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant