Skip to content

Track Sonar coverage-wiring gap in dev-notes - #5

Merged
rdryfoos merged 2 commits into
mainfrom
cursor/track-sonar-coverage-gap-7716
Jul 31, 2026
Merged

Track Sonar coverage-wiring gap in dev-notes#5
rdryfoos merged 2 commits into
mainfrom
cursor/track-sonar-coverage-gap-7716

Conversation

@rdryfoos

@rdryfoos rdryfoos commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Summary

Documents the standing SonarCloud Quality Gate failure (0.0% Coverage on New Code) as known craft debt in specs/001-mvp/dev-notes.md Platform readiness — CI does not yet publish XCTest/xccov into Sonar.

Why

Interview / review honesty: the red SonarCloud Code Analysis check is explained and tracked in-repo, not mistaken for a Gate 2 or product failure.

Traces

Craft / platform backlog only (no PRD AC). Related context: Phase E Sonar CI (T089).

Test plan

  • Docs-only change
  • Confirm Platform readiness row reads clearly on GitHub after merge
Open in Web Open in Cursor 

Summary by CodeRabbit

  • Documentation
    • Added notes explaining missing SonarCloud coverage reporting in CI.
    • Clarified that the 80% “Coverage on New Code” quality requirement remains unmet until Swift test coverage is published.

cursoragent and others added 2 commits July 31, 2026 01:54
Document that SonarCloud QG fails on new-code coverage because CI does not upload XCTest/xccov yet — known craft debt, not a Gate 2 failure.

Co-authored-by: Rik Dryfoos <rdryfoos@users.noreply.github.com>
Co-authored-by: Rik Dryfoos <rdryfoos@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b3972e7c-5bfe-4999-a909-f1a393bb2299

📥 Commits

Reviewing files that changed from the base of the PR and between 8d74c68 and fe5789c.

📒 Files selected for processing (1)
  • specs/001-mvp/dev-notes.md

📝 Walkthrough

Walkthrough

The development notes document that CI does not publish Swift test coverage to SonarCloud. They record the resulting 0.0% coverage report and clarify that the 80% “Coverage on New Code” threshold remains unmet outside Phase E scope.

Changes

SonarCloud coverage readiness

Layer / File(s) Summary
Coverage readiness documentation
specs/001-mvp/dev-notes.md
The notes document missing XCTest/xccov or lcov coverage publication to SonarCloud, the resulting 0.0% quality-gate coverage, and the unmet 80% threshold. They distinguish this issue from Phase E scope.

Estimated code review effort: 1 (Trivial) | ~2 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/track-sonar-coverage-gap-7716

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@rdryfoos
rdryfoos marked this pull request as ready for review July 31, 2026 01:57
@rdryfoos
rdryfoos merged commit e63e5b6 into main Jul 31, 2026
5 of 6 checks passed
@rdryfoos
rdryfoos deleted the cursor/track-sonar-coverage-gap-7716 branch July 31, 2026 14:52
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.

2 participants