Skip to content

Integrations v2 | Declarative Provider Registration & Runtime#2236

Merged
matoszz merged 78 commits intomainfrom
feat-moreintegrations
Apr 1, 2026
Merged

Integrations v2 | Declarative Provider Registration & Runtime#2236
matoszz merged 78 commits intomainfrom
feat-moreintegrations

Conversation

@matoszz
Copy link
Copy Markdown
Member

@matoszz matoszz commented Mar 30, 2026

This rewrite replaces a flat package (where each provider was a monolithic implementation wired into a shared activation service and ingest pipeline) with a declarative provider registration architecture. Each provider is now a self-contained package that registers its credentials, clients, operations, mappings, and webhooks through a builder pattern. The new runtime package handles the full integration lifecycle (credential resolution, client caching, operation dispatch, webhook routing) uniformly for all definitions. The old scattered concerns and somewhat naive approach has been wholesale replaced by a significantly more advanced, structured, and extensible framework. Identity for credentials, operations, and webhook events is now derived from go types rather than hand-assigned strings, making collisions a compile-time error instead of a runtime surprise - coupled with the declarative provider setup, we now have uniform definitions with collision-free guarantees.

You can see detailed information surrounding the new package architecture (and some rationale on the decisions around it) in the README.md in the package.

@theopenlane-bender
Copy link
Copy Markdown

🔧 Configuration Changes Detected

This PR contains changes that will affect the Helm chart configuration. A draft infrastructure PR has been automatically created to preview these changes:

📋 Draft PR: https://github.com/theopenlane/openlane-infra/pull/817

Changes Preview:

✅ Updated ConfigMap template

  • 🔄 Merged Helm values.yaml
  • 🔐 External secrets configuration updated
  • ✅ Updated ConfigMap template

The draft infrastructure PR will be closed automatically after this core PR is merged.

@matoszz matoszz force-pushed the feat-moreintegrations branch from 9775209 to b873714 Compare April 1, 2026 13:03
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

@matoszz matoszz merged commit 074764b into main Apr 1, 2026
25 checks passed
@matoszz matoszz deleted the feat-moreintegrations branch April 1, 2026 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants