Skip to content

feat: [CODE-5421]: subscribe GitHub check_run/status via HookEvents.Check#358

Open
anurag-harness wants to merge 2 commits into
masterfrom
CODE-5421-checkhook
Open

feat: [CODE-5421]: subscribe GitHub check_run/status via HookEvents.Check#358
anurag-harness wants to merge 2 commits into
masterfrom
CODE-5421-checkhook

Conversation

@anurag-harness

Copy link
Copy Markdown

No description provided.

@CLAassistant

CLAassistant commented Jun 10, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@anurag-harness anurag-harness changed the title Code 5421 checkhook feat: [CODE-5421]: subscribe GitHub check_run/status via HookEvents.Check Jun 10, 2026
anurag-harness and others added 2 commits June 11, 2026 20:50
… parsing

Introduce scm.CheckHook, a provider-agnostic CI-check webhook carrying a
batch of scm.Check entries (name, status, conclusion, target URL, sha,
timestamps, optional PR). The batch shape accommodates providers that
report multiple checks per event (e.g. GitLab pipeline jobs) while
GitHub/Bitbucket populate a single entry.

GitHub driver now parses check_run and status events (previously
ErrUnknownEvent) into CheckHook. Status carries the normalized
ExecutionStatus while Conclusion preserves the raw provider outcome so
richer GitHub conclusions (neutral, timed_out, ...) are not lost.

Adds HookEvents.Check for subscription and fixture/golden tests for both
events.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…heck

Wire the new HookEvents.Check flag through convertFromHookEvents so
CreateHook registers both check_run and status GitHub events when a
caller subscribes to checks.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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