Skip to content

feat(review): add review.auto_merge_summary read-only conditions table (#2051) - #3806

Closed
Lang-bt wants to merge 2 commits into
JSONbored:mainfrom
Lang-bt:feat/review-auto-merge-summary-2051-lang-bt
Closed

feat(review): add review.auto_merge_summary read-only conditions table (#2051)#3806
Lang-bt wants to merge 2 commits into
JSONbored:mainfrom
Lang-bt:feat/review-auto-merge-summary-2051-lang-bt

Conversation

@Lang-bt

@Lang-bt Lang-bt commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Closes #2051

Summary

  • Add review.auto_merge_summary (bool, default off) — read-only Auto-merge conditions collapsible in the unified comment.
  • Shows pass/fail for CI green, gate passing, mergeable-clean, and linked-issue from readiness facts already on the comment path; does not change merge/close decisions.
  • Document in .gittensory.yml.example and config/examples/gittensory.full.yml.

Supersedes #3793 (closed after codecov/patch at 94.11%; this push adds branch-coverage tests for the partial bridge path).

Scope

Validation

  • git diff --check
  • npm run typecheck
  • npm run test:coverage -- test/unit/auto-merge-summary-collapsible.test.ts test/unit/focus-manifest.test.ts test/unit/unified-comment-bridge.test.ts test/unit/config-templates.test.ts test/unit/queue.test.ts -t "auto_merge|auto-merge|Auto-merge|parses review.auto_merge"
  • npm audit --audit-level=moderate (0 vulnerabilities)

codecov/patch requires ≥99% on changed src/** lines/branches — added tests for mergeReadiness-omitted path, explicit false knob, warn-state linked issue, and missing mergeStateLabel.

Safety

  • Display-only read-only table; no secrets or disposition changes.
  • Unit + queue integration tests for on/off arms and processor wiring.

UI Evidence

N/A — backend unified-comment rendering only.

Made with Cursor

james3773 and others added 2 commits July 6, 2026 03:39
JSONbored#2051)

Wire the manifest knob through focus-manifest parse/serialize, the unified-comment
bridge collapsible, and the processor publish path with unit + queue integration tests.

Co-authored-by: Cursor <cursoragent@cursor.com>
…ch (JSONbored#2051)

Exercise mergeReadiness-omitted render path, explicit false knob, warn-state
linked-issue row, and missing mergeStateLabel so bridge patch coverage reaches 99%.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Lang-bt
Lang-bt requested a review from JSONbored as a code owner July 6, 2026 11:41
@superagent-security

superagent-security Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@JSONbored JSONbored left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

merge conflicts:

This branch has conflicts that must be resolved
Use the [web editor](https://github.com/JSONbored/gittensory/pull/3806/conflicts) or the command line to resolve conflicts before continuing.

src/review/unified-comment-bridge.ts
src/signals/focus-manifest.ts
test/unit/focus-manifest.test.ts
test/unit/signals-coverage.test.ts

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.

feat(config): add review.auto_merge_summary read-only knobs surface

3 participants