Skip to content

ci: run compat tests daily at 2 AM UTC and on SDK-code PRs - #2

Merged
naji247 merged 1 commit into
mainfrom
ci/compat-triggers
Nov 3, 2026
Merged

naji247 merged 1 commit into
mainfrom
ci/compat-triggers

Conversation

@naji247

@naji247 naji247 commented Nov 3, 2026

Copy link
Copy Markdown
Member

Summary

.github/workflows/mcp-compat.yml:

 on:
-  pull_request:
+  pull_request:
+    paths:
+      - "lib/**"
+      - "agentcat.gemspec"
+      - "Gemfile"
+      - "Gemfile.lock"
+      - "Rakefile"
+      - "test/**"
+      - "!test/e2e/**"
+      - "scripts/mcp_versions.rb"
+      - "scripts/release.rb"
+      - ".github/workflows/mcp-compat.yml"
   push:
     branches: [main]
   schedule:
-    - cron: "23 4 * * *" # nightly
+    - cron: "0 2 * * *" # nightly, 02:00 UTC
   workflow_dispatch:

No job gating: the workflow holds only compat jobs (discover, compat, report). The header comment now names the 02:00 UTC time and the PR filter.

SDK-code paths: lib/**, agentcat.gemspec, Gemfile, Gemfile.lock, Rakefile, test/** except test/e2e/** (the compat job runs bundle exec rake, which excludes e2e), scripts/mcp_versions.rb (called by discover, tested by the suite), scripts/release.rb (loaded by test/test_release.rb), .github/workflows/mcp-compat.yml.

Evidence

  • Before: pull_request (all PRs), push to main, schedule 23 4 * * *, workflow_dispatch.
    After: pull_request with the paths above, push to main, schedule 0 2 * * *, workflow_dispatch. yaml.safe_load passes. actionlint is not installed here and was not run.

Merge Danger

Door: two-way

Blast Radius: CI

main has no branch protection and no rulesets ([] and 404), so no check is required and a path filter cannot block a merge. A PR that changes only README, docs, CHANGELOG, examples, sorbet config, or other workflows now skips the compat matrix; ci.yml still runs the unit suite on every PR.

@naji247
naji247 merged commit a53ca4c into main Nov 3, 2026
57 checks passed
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.

2 participants