Skip to content

Add org ID to capability request metadata#1953

Merged
prashantkumar1982 merged 1 commit intomainfrom
codex/request-metadata-org-id
Apr 1, 2026
Merged

Add org ID to capability request metadata#1953
prashantkumar1982 merged 1 commit intomainfrom
codex/request-metadata-org-id

Conversation

@prashantkumar1982
Copy link
Copy Markdown
Contributor

@prashantkumar1982 prashantkumar1982 commented Apr 1, 2026

Summary

We need this so workflows, when calling Vault DON, can provide both OrgID and WorkflowOwner in the request.

The simplest seamless place to carry that data is capabilities.RequestMetadata, since the workflow engine already constructs it for capability calls, and downstream consumers like confidential-compute can read OrgID from there without needing their own org-resolution path.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

👋 prashantkumar1982, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

⚠️ API Diff Results - github.com/smartcontractkit/chainlink-common

⚠️ Breaking Changes (14)

pkg/capabilities (7)
  • AttributedSignature — 🗑️ Removed

  • ErrResponsePayloadNotAvailable — 🗑️ Removed

  • Execute — Type changed:

func(
  context.Context, 
  CapabilityRequest, 
  I, 
  C, 
  func(context.Context, 
  RequestMetadata, 
  I, 
  C) (O, 
  ResponseMetadata, 
  - *OCRAttestation, 
  error)
)
(CapabilityResponse, error)
  • ExtractMeteringFromMetadata — 🗑️ Removed

  • OCRAttestation — 🗑️ Removed

  • ResponseToReportData — 🗑️ Removed

  • SetResponse — Type changed:

func(
  *CapabilityResponse, 
  bool, 
  - google.golang.org/protobuf/proto.Message, 
  - *OCRAttestation
  + google.golang.org/protobuf/proto.Message
)
error
pkg/capabilities.CapabilityResponse (1)
  • OCRAttestation — 🗑️ Removed
pkg/capabilities.ResponseAndMetadata (1)
  • OCRAttestation — 🗑️ Removed
pkg/capabilities/pb (2)
  • AttributedSignature — 🗑️ Removed

  • OCRAttestation — 🗑️ Removed

pkg/capabilities/pb.(*CapabilityResponse) (1)
  • GetOcrAttestation — 🗑️ Removed
pkg/capabilities/pb.CapabilityResponse (1)
  • OcrAttestation — 🗑️ Removed
pkg/settings/cresettings.Workflows (1)
  • FeatureChainCapabilityHashBasedOCRActivePeriod — 🗑️ Removed

✅ Compatible Changes (3)

pkg/capabilities.RequestMetadata (1)
  • OrgID — ➕ Added
pkg/capabilities/pb.(*RequestMetadata) (1)
  • GetOrgId — ➕ Added
pkg/capabilities/pb.RequestMetadata (1)
  • OrgId — ➕ Added

📄 View full apidiff report

@prashantkumar1982 prashantkumar1982 added this pull request to the merge queue Apr 1, 2026
Merged via the queue into main with commit 720567e Apr 1, 2026
31 of 33 checks passed
@prashantkumar1982 prashantkumar1982 deleted the codex/request-metadata-org-id branch April 1, 2026 19:04
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.

4 participants