swift-testing-code-review — independently scanned and version-tracked by SaferSkills.
SaferSkills independently audited swift-testing-code-review (Agent Skill) and scored it 100/100 (green). The audit ran 55 deterministic rules across Security, Supply Chain, Maintenance, Transparency, and Community; it found 0 high-severity and 0 lower-severity findings. The full rule-by-rule trace and per-finding evidence are below. Free, methodology-open.
Findings & checks · 0 flagged
Every scanned point with the score it earned and what moved between them.
First recorded scan — no prior version to compare against.
The primary manifest — the file an agent reads to learn what this artifact does.
Complete in order before recording Swift Testing review findings. Stack with review-verification-protocol for universal review rules.
.swift paths under review (or a user-named single file). Pass: Paths captured in working notes or one line: No Swift files in scope — then stop with no findings.import Testing or @Test / #expect / #require / @Suite appears in that file (open or search). Pass: At least one match per critiqued file, or you exclude that file from Swift Testing review with a one-line reason (e.g. XCTest-only).[FILE:LINE] evidence. If you report zero issues, state Protocol applied; no Swift Testing issues (or equivalent) in the review summary.| Issue Type | Reference |
|---|---|
| #expect vs #require, expression capture, error testing | references/expect-macro.md |
| @Test with arguments, traits, zip() pitfalls | references/parameterized.md |
| confirmation, async sequences, completion handlers | references/async-testing.md |
| @Suite, tags, parallel execution, .serialized | references/organization.md |
#expect (not pre-computed booleans)#require used only for preconditions, #expect for assertions(any Error).self)zip() (not Cartesian product)confirmation(expectedCount:)withCheckedContinuation, not confirmation.serialized applied only where necessary (shared resources)@Test functions#expect lose diagnostic context?#require stopping tests prematurely instead of revealing all failures?zip() silently drop test cases due to unequal array lengths?confirmation?~30 seconds. Free. No account. Every finding cites a rule and a line of evidence.