Skip to content

chore(coverage): raise the unit coverage floor to 99/99/99/99 and fix the documented values - #500

Merged
mrbobbytables merged 1 commit into
mainfrom
quality/test-unit-coverage-floor
Oct 18, 2026
Merged

mrbobbytables merged 1 commit into
mainfrom
quality/test-unit-coverage-floor

Conversation

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Test Improvement

Files claimed: vitest.config.mjs (coverage.thresholds), docs/contributing.md (line 48). Test configuration and its documentation only — no src/, __tests__/ or dist/ change. Disjoint from #497 (src/plugins/approve.ts, src/issueComment/meow.ts) and #498 (__tests__/bundle/tideOptionalFields.test.ts).

Verification (local, this branch, main @ 1062ce7)

Related Issue

Closes #499


Filed by quality agent (hold-gated mode). Human review required.

— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88

…the documented values

main reaches 100 / 99.7 / 100 / 100 (lines / branches / functions /
statements) while vitest.config.mjs gated build-test at 95 / 93 / 98 / 95,
leaving ~120 lines of slack before a regression fails. Set the floor just
under the current figure, the same rule as #202 and #484, and correct
docs/contributing.md, which still listed the pre-#202 values 85/83/91/85.

Closes #499

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: quality <quality@hive.kubestellar.io>
@hivecommons-hive
hivecommons-hive Bot requested a review from jpmcb as a code owner October 17, 2026 08:33
@hivecommons-hive hivecommons-hive Bot added the hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 17, 2026
@hivecommons-hive

Copy link
Copy Markdown
Contributor Author

Important

Held for human review by the hive's ACMM level gate.

This PR was opened by the "quality" agent while Hive policy required a human checkpoint for that agent. Agent PRs are held at ACMM L3–L5; an outreach PR that targets a repository this hive does not manage is held at every level because it publishes project-facing communication into someone else's project.

Hive will keep the hold label until a human removes it. Operators can make a deliberate one-off release during an ACMM level change with release_level_holds=true, but level changes never release this hold automatically.

@github-actions

Copy link
Copy Markdown
Contributor

Please add a kind label with /kind feature, /kind failing-test or /kind cleanup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold Indicates that a PR should not merge because someone has issued a /hold command. needs-kind

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[quality] unit coverage gate in vitest.config.mjs (95/93/98/95) sits ~5 points under main's floor; docs/contributing.md:48 still documents 85/83/91/85

1 participant