From 9447c0811bf15f6416863e575c8e36599eb55a2d Mon Sep 17 00:00:00 2001 From: luciferlive112116 <291889058+luciferlive112116@users.noreply.github.com> Date: Mon, 6 Jul 2026 12:59:06 +0800 Subject: [PATCH] docs(config): document shipped review.auto_review knobs in gittensory.full.yml Co-authored-by: Cursor --- .gittensory.yml.example | 21 ++++++++++++++++++--- config/examples/gittensory.full.yml | 21 ++++++++++++++++++--- test/unit/config-templates.test.ts | 8 ++++++++ 3 files changed, 44 insertions(+), 6 deletions(-) diff --git a/.gittensory.yml.example b/.gittensory.yml.example index 7c34ba44c6..ea54610708 100644 --- a/.gittensory.yml.example +++ b/.gittensory.yml.example @@ -337,7 +337,7 @@ gate: # deterministic gate policy above. Omit the block to keep the byte-identical # defaults. review: - # Deterministic AI review eligibility filters (`review.auto_review`, #1954 / #2038–#2041). Each knob quietly + # Deterministic AI review eligibility filters (`review.auto_review`, #1954 / #2038–#2065). Each knob quietly # skips the advisory AI review for matching PRs — never a gate failure. When the Orb review check is enabled, # skipped PRs complete as "skipped" with a human-readable reason (see `evaluateAutoReviewSkipReason` in # `src/signals/focus-manifest.ts` and the public-surface `SKIP_SUMMARY.ignored_author` mapping in @@ -372,8 +372,23 @@ review: # Default: null / 0 (re-review every sync, byte-identical). # auto_pause_after_reviewed_commits: 3 - # Planned eligibility knobs (tracked separately — not parsed yet; omit until shipped): - # skip_labels, skip_docs_only, max_added_lines, max_files (#2062–#2065). + # Case-insensitive PR label names that skip AI review (e.g. maintenance or do-not-review markers). + # Default: [] (no label-based skip). (#2062) + # skip_labels: + # - do-not-review + # - wip + + # Bool | null. Default: null (docs-only PRs are reviewed as today). When true, PRs whose every + # changed file classifies as documentation skip AI review. Empty changed-file list is NOT docs-only. (#2063) + # skip_docs_only: true + + # Non-negative integer. Skip AI review when total added lines across the PR diff exceed this cap. + # Default: 0 (no cap — reviewed regardless of size). (#2065) + # max_added_lines: 500 + + # Non-negative integer. Skip AI review when changed-file count exceeds this cap. + # Default: 0 (no cap). (#2065) + # max_files: 25 # Planned display toggle (#2069): when enabled, the unified review comment includes a # `review effort: N/5 (~M min)` line derived from the per-PR effort estimate. diff --git a/config/examples/gittensory.full.yml b/config/examples/gittensory.full.yml index 3743e257f0..e287bab71b 100644 --- a/config/examples/gittensory.full.yml +++ b/config/examples/gittensory.full.yml @@ -350,7 +350,7 @@ gate: # deterministic gate policy above. Omit the block to keep the byte-identical # defaults. review: - # Deterministic AI review eligibility filters (`review.auto_review`, #1954 / #2038–#2041). Each knob quietly + # Deterministic AI review eligibility filters (`review.auto_review`, #1954 / #2038–#2065). Each knob quietly # skips the advisory AI review for matching PRs — never a gate failure. When the Orb review check is enabled, # skipped PRs complete as "skipped" with a human-readable reason (see `evaluateAutoReviewSkipReason` in # `src/signals/focus-manifest.ts` and the public-surface `SKIP_SUMMARY.ignored_author` mapping in @@ -385,8 +385,23 @@ review: # Default: null / 0 (re-review every sync, byte-identical). # auto_pause_after_reviewed_commits: 3 - # Planned eligibility knobs (tracked separately — not parsed yet; omit until shipped): - # skip_labels, skip_docs_only, max_added_lines, max_files (#2062–#2065). + # Case-insensitive PR label names that skip AI review (e.g. maintenance or do-not-review markers). + # Default: [] (no label-based skip). (#2062) + # skip_labels: + # - do-not-review + # - wip + + # Bool | null. Default: null (docs-only PRs are reviewed as today). When true, PRs whose every + # changed file classifies as documentation skip AI review. Empty changed-file list is NOT docs-only. (#2063) + # skip_docs_only: true + + # Non-negative integer. Skip AI review when total added lines across the PR diff exceed this cap. + # Default: 0 (no cap — reviewed regardless of size). (#2065) + # max_added_lines: 500 + + # Non-negative integer. Skip AI review when changed-file count exceeds this cap. + # Default: 0 (no cap). (#2065) + # max_files: 25 # Planned display toggle (#2069): when enabled, the unified review comment includes a # `review effort: N/5 (~M min)` line derived from the per-PR effort estimate. diff --git a/test/unit/config-templates.test.ts b/test/unit/config-templates.test.ts index 2325ac2178..eb0fe7ce2c 100644 --- a/test/unit/config-templates.test.ts +++ b/test/unit/config-templates.test.ts @@ -48,6 +48,14 @@ describe("config/examples review templates (#1682)", () => { expect(manifest.features.rag).toBeNull(); }); + it("documents every shipped review.auto_review eligibility knob in gittensory.full.yml (#2055)", () => { + const full = readConfigExample("gittensory.full.yml"); + for (const field of ["skip_labels", "skip_docs_only", "max_added_lines", "max_files"]) { + expect(full, `missing auto_review field ${field}`).toMatch(new RegExp(`# ${field}:`)); + } + expect(full).not.toMatch(/not parsed yet/); + }); + it("parses gittensory.minimal.yml with zero warnings and enables no agent actions", () => { const manifest = parseFocusManifestContent(readConfigExample("gittensory.minimal.yml"), "repo_file"); expect(manifest.warnings).toEqual([]);