Skip to content

Scope CI to affected checks and add Claude Feishu review - #357

Merged
RyanLee-Dev merged 3 commits into
mainfrom
codex/ci-claude-feishu
Oct 1, 2026
Merged

RyanLee-Dev merged 3 commits into
mainfrom
codex/ci-claude-feishu

Conversation

@RyanLee-Dev

@RyanLee-Dev RyanLee-Dev commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

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. Configure MINIMAX_API_KEY; FEISHU_WEBHOOK_URL is already set, and FEISHU_WEBHOOK_SECRET is optional.

Validation:

  • make check-ci check-docs check-names, actionlint 1.7.12 and git diff --check passed.
  • Tests cover precise workflow/dependency selections, actual shared Node Action consumers, every job's plan condition, mixed changes and full-gate fallbacks.
  • Replayed real documentation, installer, Web and Core commit diffs. Replaying the notification-only diff selects just hygiene and lint.
  • Inline notification smoke checks passed; live MiniMax/Feishu delivery remains unverified.

This PR changes the planner itself, so its own CI intentionally runs the full gate before the narrower rules are used by subsequent PRs.

@blacksmith-sh

blacksmith-sh Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Found 2 test failures on Blacksmith runners:

Failures

Test View Logs
github.com/MiniMax-AI/OpenAgentCore/services/core/internal/persistence/postgres/
deploymentpg/TestResetStartEscalateAndComplete
View Logs
github.com/MiniMax-AI/OpenAgentCore/services/core/internal/persistence/postgres/
modelconfigurationpg/TestObservationExcludesOtherSourcesAndHistoricalSessions
View Logs

Fix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@RyanLee-Dev RyanLee-Dev changed the title Add Claude CI review and fixed Feishu card notifications Scope CI to affected checks and add Claude Feishu review Oct 1, 2026
@RyanLee-Dev
RyanLee-Dev merged commit 9b1bfe3 into main Oct 1, 2026
13 of 14 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.

1 participant