Skip to content

feat(git): carry the PR head SHA on PullRequestChanged - #127

Merged
Pomdapis merged 1 commit into
mainfrom
feat/git-pr-head-sha
Jul 24, 2026
Merged

feat(git): carry the PR head SHA on PullRequestChanged#127
Pomdapis merged 1 commit into
mainfrom
feat/git-pr-head-sha

Conversation

@Pomdapis

Copy link
Copy Markdown
Contributor

Additive surface fix found during the Nexus N1 shim: the neutral GitWebhookEvent.PullRequestChanged dropped the source head commit SHA, which CI/CD consumers (Nexus PipelineTriggerContext.CommitSha) require to build the exact proposed revision — this blocked rerouting the Nexus webhook handler through IGitWebhookIngestor. Adds optional SourceHeadSha (default null, source-compatible); the GitHub adapter populates it from pull_request.head.sha. Two new ingestor tests (present/absent). No tag yet — will ride v1.0.5-preview.3 when the N3 phase pins it.

https://claude.ai/code/session_01VFDn7gCH7vdvT4eoNhsMLr

CI/CD consumers need the source head commit to build/deploy the exact proposed
revision (Nexus PipelineTriggerContext.CommitSha); the neutral event dropped it,
blocking the webhook-handler migration onto the ingestor. Additive optional
positional parameter; the GitHub adapter populates it from pull_request.head.sha.

Claude-Session: https://claude.ai/code/session_01VFDn7gCH7vdvT4eoNhsMLr
@Pomdapis
Pomdapis merged commit 0d91741 into main Jul 24, 2026
5 checks passed
@Pomdapis
Pomdapis deleted the feat/git-pr-head-sha branch July 24, 2026 10:51
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