Scope CI to affected checks and add Claude Feishu review - #357
Merged
Merged
Conversation
Contributor
|
Found 2 test failures on Blacksmith runners: Failures
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI workflow and dependency changes currently trigger unrelated database, browser and native jobs. Refine the existing centralized CI planner so known workflows, shared Actions and Go/Node inputs select only their consumers. Each job keeps its plan condition and the final gate still rejects unexpected skips or missing results. Unknown paths, planner/orchestration changes, main pushes and releases retain full checks.
Also add
.github/workflows/ci-review.yml: a standalone Claude Code Action using MiniMax to summarize completed CI and review changes against repository rules, then deliver a fixed Chinese Feishu Card 2.0 notification. Collection and delivery are inline in the workflow; model tools are disabled and reviewed code is never executed. ConfigureMINIMAX_API_KEY;FEISHU_WEBHOOK_URLis already set, andFEISHU_WEBHOOK_SECRETis optional.Validation:
make check-ci check-docs check-names, actionlint 1.7.12 andgit diff --checkpassed.This PR changes the planner itself, so its own CI intentionally runs the full gate before the narrower rules are used by subsequent PRs.