Skip to content

[codex] support Swift 6.4 coverage report paths#61

Merged
steipete merged 1 commit into
mainfrom
codex/fix-swiftpm-coverage-path
Jun 30, 2026
Merged

[codex] support Swift 6.4 coverage report paths#61
steipete merged 1 commit into
mainfrom
codex/fix-swiftpm-coverage-path

Conversation

@steipete

Copy link
Copy Markdown
Collaborator

Summary

  • find SwiftPM's coverage JSON by its stable codecov/remindctl.json suffix
  • accept both legacy lowercase debug and current Swift 6.4 Products/Debug output layouts

Root cause

scripts/check-coverage.sh hardcoded a lowercase debug path component. Swift 6.4's current build system writes the same report under .build/coverage/out/Products/Debug/codecov, so the 90% gate failed after otherwise successful tests.

Validation

  • bash -n scripts/check-coverage.sh
  • scripts/check-coverage.sh — 92.5% (515/557), 90% minimum
  • git diff --check

@clawsweeper

clawsweeper Bot commented Jun 30, 2026

Copy link
Copy Markdown

ClawSweeper status: review started.

I am starting a fresh review of this pull request: [codex] support Swift 6.4 coverage report paths This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@steipete
steipete merged commit 316149f into main Jun 30, 2026
3 checks passed
@steipete
steipete deleted the codex/fix-swiftpm-coverage-path branch June 30, 2026 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant