Skip to content

ci(coderabbit-config): refresh the vendored CodeRabbit schema - #250

Merged
mattmillerai merged 1 commit into
mainfrom
ci/refresh-coderabbit-schema
Sep 8, 2026
Merged

mattmillerai merged 1 commit into
mainfrom
ci/refresh-coderabbit-schema

Conversation

@cloud-code-bot

@cloud-code-bot cloud-code-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Upstream's CodeRabbit config schema has drifted from the copy vendored here, which is what coderabbit-config-validate.yml grades every enrolled repo against.

Added properties (4):

  • knowledge_base.automatic_linking_mode
  • reviews.slop_detection.include_all_authors
  • reviews.tools.vale
  • reviews.tools.vale.enabled

Before merging, check the tightened-cap section above if there is one: a lowered maxLength retroactively invalidates configs nobody has touched, and merging this makes their next PR red. Landing the cleanup alongside is usually the right order.

Warning

The invariant unit suite FAILS against the proposed schema (this run). That suite asserts the exact facts the checker's messages rest on, so this is the signal to read closely, not a flake: either the schema change is one we must adapt the checker and its tests to, or the fetch is not what we think it is. Update those assertions in this PR, deliberately, before merging.

Invariant suite output
...................................................................F...............................
======================================================================
FAIL: test_the_two_halves_partition_the_objects_that_declare_properties (test_check_coderabbit_config.OpennessTest.test_the_two_halves_partition_the_objects_that_declare_properties)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/github-workflows/github-workflows/.github/coderabbit-config/tests/test_check_coderabbit_config.py", line 1060, in test_the_two_halves_partition_the_objects_that_declare_properties
    self.assertEqual(len(omitted), 104)
AssertionError: 105 != 104

----------------------------------------------------------------------
Ran 99 tests in 1.948s

FAILED (failures=1)

Merging this touches .github/coderabbit-config/**, which triggers bump-coderabbit-config-callers.yml and rolls the SHA-pinned caller fleet forward.

Opened by the refresh-coderabbit-schema workflow.

@cloud-code-bot
cloud-code-bot Bot force-pushed the ci/refresh-coderabbit-schema branch from 7315157 to cfb25de Compare September 7, 2026 15:08
@mattmillerai
mattmillerai merged commit c132ae1 into main Sep 8, 2026
3 of 4 checks passed
@mattmillerai
mattmillerai deleted the ci/refresh-coderabbit-schema branch September 8, 2026 04:36
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.

1 participant