[PR #288] [MERGED] ci(coderabbit-config): refresh the vendored CodeRabbit schema #289

New issue

Closed

opened 2026-10-26 22:38:07 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-26 22:38:07 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/github-workflows/pull/288
Author: @cloud-code-bot[bot]
Created: 9/14/2026
Status: ✅ Merged
Merged: 9/19/2026
Merged by: @mattmillerai

Base: main ← Head: ci/refresh-coderabbit-schema

📝 Commits (1)

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

📊 Changes

1 file changed (+26 additions, -16 deletions)

View changed files

📝 .github/coderabbit-config/schema.v2.json (+26 -16)

📄 Description

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

⚠️ Tightened maxLength caps — may invalidate existing configs (1):

chat.integrations.jira.issue_template: uncapped → 3000

Added properties (2):

chat.integrations.jira.issue_template
reviews.tools.ruff.config_file

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.............................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

======================================================================
FAIL: test_the_committed_schema_has_the_documented_cap_set (test_schema_drift.VendoredSchemaTest.test_the_committed_schema_has_the_documented_cap_set)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/github-workflows/github-workflows/.github/coderabbit-config/tests/test_schema_drift.py", line 213, in test_the_committed_schema_has_the_documented_cap_set
    self.assertEqual(len(caps), 14)
AssertionError: 15 != 14

----------------------------------------------------------------------
Ran 99 tests in 2.586s

FAILED (failures=2)

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.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/github-workflows/pull/288 **Author:** [@cloud-code-bot[bot]](https://github.com/apps/cloud-code-bot) **Created:** 9/14/2026 **Status:** ✅ Merged **Merged:** 9/19/2026 **Merged by:** [@mattmillerai](https://github.com/mattmillerai) **Base:** `main` ← **Head:** `ci/refresh-coderabbit-schema` --- ### 📝 Commits (1) - [`8719735`](https://github.com/Comfy-Org/github-workflows/commit/871973588114046d4df1c0d2efe9878d99475a51) ci(coderabbit-config): refresh the vendored CodeRabbit schema ### 📊 Changes **1 file changed** (+26 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `.github/coderabbit-config/schema.v2.json` (+26 -16) </details> ### 📄 Description Upstream's CodeRabbit config schema has drifted from the copy vendored here, which is what `coderabbit-config-validate.yml` grades every enrolled repo against. **⚠️ Tightened `maxLength` caps — may invalidate existing configs** (1): - `chat.integrations.jira.issue_template: uncapped → 3000` **Added properties** (2): - `chat.integrations.jira.issue_template` - `reviews.tools.ruff.config_file` **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](https://github.com/Comfy-Org/github-workflows/actions/runs/34860228839)). 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. <details><summary>Invariant suite output</summary> ``` ...................................................................F.............................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 ====================================================================== FAIL: test_the_committed_schema_has_the_documented_cap_set (test_schema_drift.VendoredSchemaTest.test_the_committed_schema_has_the_documented_cap_set) ---------------------------------------------------------------------- Traceback (most recent call last): File "/home/runner/work/github-workflows/github-workflows/.github/coderabbit-config/tests/test_schema_drift.py", line 213, in test_the_committed_schema_has_the_documented_cap_set self.assertEqual(len(caps), 14) AssertionError: 15 != 14 ---------------------------------------------------------------------- Ran 99 tests in 2.586s FAILED (failures=2) ``` </details> 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._ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-26 22:38:07 +00:00