Skip to content

feat(reports): leftover-map coverage on grouping comparison graphic (v2.63.0) - #845

Draft
seonghobae wants to merge 5 commits into
feat/leftover-map-compare-plot-axis-share-v2620from
feat/leftover-map-compare-plot-coverage-v2630
Draft

feat(reports): leftover-map coverage on grouping comparison graphic (v2.63.0)#845
seonghobae wants to merge 5 commits into
feat/leftover-map-compare-plot-axis-share-v2620from
feat/leftover-map-compare-plot-coverage-v2630

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Caption persisted leftover-map complete-case coverage on the grouping-comparison leftover-map graphic when leftoverMapCoverageCounts returns usable integers (ADR 0306 / v2.63.0). Post coverage remains independent of item/incomplete coverage. No SQL/new columns and no coverage inference from marker/pair counts or other leftover-map statistics.

Live authority

Parent convergence

The old child was authored against #844 c25b0551.... After #844 moved, #845 inherited tests/test_compare_graphic_adr_status.py, preserved its own Amended by ADR 0305 / ADR 0306 text while changing ADR 0304 Accepted → Proposed, then recorded current #844 as the second parent in 8d37896a.... The branch moved only by non-force fast-forward; no valid product delta was dropped.

Historical 464-test/oxlint/py_compile evidence predates this exact head and is not transferred. Exact-head required/frontend/browser/a11y and PostgreSQL/k6 evidence plus independent approval remain outstanding. Any direct descendant must be reconstructed against this head before its predecessor evidence can transfer.

Product invariants

  • Missing, non-integer, negative-used, non-positive-scored, or used-greater-than-scored post coverage omits only the comparison graphic coverage caption.
  • 0 of M remains valid when stored counts satisfy the contract.
  • Axis-share display remains independent.
  • This increment does not add item or incomplete coverage captions.

Keep Draft until exact-head evidence and independent approval exist. No self-approval, gate bypass, force push, or destructive rebase.

…v2.63.0)

Caption leftover-map complete-case coverage on the grouping comparison leftover-map graphic from already-named leftover_map_coverage (ADR 0306). Distinct accessible name Leftover map comparison graphic coverage. Omit item/incomplete notes on that comparison plot. Never invent a leftover score or a theta.
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-31 04:40 KST. Writes work as seonghobae. Head 7c026df (v2.63.0 / ADR 0306) stacked on #844 @ 95538f9.

Caption leftover-map complete-case coverage on the grouping comparison leftover-map graphic from already-named leftover_map_coverage. Distinct accessible name Leftover map comparison graphic coverage. Omit leftover-map item coverage and leftover-map incomplete coverage on that comparison plot this increment. No SQL; no new columns. Never invent a leftover score or a theta.

Independent APPROVE is required. Do not self-approve. Do not squash-merge onto the unprotected leftover base. Issues #79 and #87 stay open. #96 is already closed.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

Copy link
Copy Markdown
Contributor Author

Cycle write probe 2026-08-31: leftover-map comparison graphic coverage (#845 @ 7c026df) remains stacked for independent APPROVE. Next buyer gap is leftover-map item complete-case coverage on that comparison graphic (distinct from strip Leftover map comparison item coverage). Do not self-approve. Synthetic fixtures only.

1 similar comment

Copy link
Copy Markdown
Contributor Author

Cycle write probe 2026-08-31: leftover-map comparison graphic coverage (#845 @ 7c026df) remains stacked for independent APPROVE. Next buyer gap is leftover-map item complete-case coverage on that comparison graphic (distinct from strip Leftover map comparison item coverage). Do not self-approve. Synthetic fixtures only.

Copy link
Copy Markdown
Contributor Author

Cycle retry (2026-08-31): GitHub write probe on leftover-map comparison-graphic stack. Independent APPROVE still required; this comment is not a review. Next increment is ADR 0307 / v2.64.0 leftover-map item complete-case coverage on the grouping comparison leftover-map graphic (distinct aria-label "Leftover map comparison graphic item coverage"). Do not squash-merge this stack onto unprotected bases. Leave open.

Copy link
Copy Markdown
Contributor Author

Stacked next increment opened as leftover-map item complete-case coverage on the grouping comparison leftover-map graphic (ADR 0307 / v2.64.0) on feat/leftover-map-compare-plot-item-coverage-v2640 @ 4779764. Distinct accessible name Leftover map comparison graphic item coverage. Leave #845 and the stacked increment open for independent review. Do not squash-merge. This comment is not a review.

@seonghobae
seonghobae marked this pull request as draft September 4, 2026 08:07
Adopt the conflict-free GitHub test-merge tree for exact parent #844 c25b055 while preserving existing #845 product head 7c026df as first-parent ancestry. No force push and no product delta is dropped.
seonghobae added a commit that referenced this pull request Sep 6, 2026
Adopt the conflict-free GitHub test-merge tree for exact parent #845 4e82102 while preserving existing #846 product head 4779764 as first-parent ancestry. No force push and no product delta is dropped.
Preserve the #845 ADR 0306/v2.63.0 coverage delta while adopting #8440882cc9, including inherited ADR 0304 Proposed governance evidence. No force push and no product delta is dropped.
seonghobae added a commit that referenced this pull request Sep 6, 2026
Adopt the verified conflict-free GitHub merge tree for exact parent #8458d37896 and child 63b3e7d. The tree preserves ADR 0307/v2.64.0 and inherits ADR 0304 Proposed plus its regression. No force push and no valid delta is dropped.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant