Skip to content

Move check ci procedure to new /check-ci skill#3952

Open
cataphract wants to merge 1 commit into
masterfrom
glopes/check-ci-skill
Open

Move check ci procedure to new /check-ci skill#3952
cataphract wants to merge 1 commit into
masterfrom
glopes/check-ci-skill

Conversation

@cataphract
Copy link
Copy Markdown
Contributor

Description

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@cataphract cataphract requested a review from a team as a code owner June 4, 2026 09:30
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 42108a28b1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .claude/skills/check-ci/SKILL.md Outdated
results with ci-watch, investigate failures, and report. Use when the user
asks to check, watch, or monitor CI, or to see whether a pipeline passed.
argument-hint: "[--commit <ref> | --pipeline <id>] [--list-jobs]"
allowed-tools: Bash Read Grep Glob Agent TaskCreate TaskUpdate TaskStop
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Auto-approve the required completion notification tool

When /check-ci is used for unattended CI monitoring in the default permission mode, the final notification step can stall on an approval prompt because the skill explicitly requires mcp__speak_when_done__speak after ci-watch exits but this allowlist only pre-approves Bash/read/search/task tools. Include the mcp__speak_when_done__speak MCP tool here so the skill can actually speak the completion/failure result without waiting for user input at the end of a long run.

Useful? React with 👍 / 👎.

@datadog-official
Copy link
Copy Markdown

datadog-official Bot commented Jun 4, 2026

Pipelines  Tests

Fix all issues with BitsAI

⚠️ Warnings

🚦 42 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-php | ASAN test_c with multiple observers: [8.4]   View in Datadog   GitLab

See error Fatal error: Uncaught Exception: wait for replay timeout in /go/src/github.com/DataDog/apm-reliability/dd-trace-php/tmp/build_extension/tests/ext/includes/request_replayer.inc:100

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [7.1]   View in Datadog   GitLab

See error Failed to propagate distributed tracing headers due to connection reset by peer error from sidecar process.

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [7.4]   View in Datadog   GitLab

See error Test failure: Crashtracker collects all threads when collect_all_threads is enabled at tmp/build_extension/tests/ext/crashtracker_collect_all_threads.phpt

View all 42 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🔄 Datadog auto-retried 1 job - 1 passed on retry View in Datadog

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 54.15% (+0.00%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 758fa7b | Docs | Datadog PR Page | Give us feedback!

@cataphract cataphract force-pushed the glopes/check-ci-skill branch from 42108a2 to 78a30ec Compare June 4, 2026 09:45
@cataphract cataphract force-pushed the glopes/check-ci-skill branch from 78a30ec to 758fa7b Compare June 4, 2026 09:49
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