feat(oci): add native Notifications integration - #7545
feat(oci): add native Notifications integration#7545BillLeoutsakosvl346 wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
e410611 to
862ca82
Compare
Greptile SummaryAdds a native OCI Notifications integration backed by the existing OCI API-key service account and signed client.
Confidence Score: 5/5The PR appears safe to merge, with no outstanding correctness, security, or repository-rule violations identified. No new actionable issue remains. The provider-ID correction preserves credential discovery through the shared OCI API-key service-account provider. BillLeoutsakosvl346 resolved the earlier update-description and selector-prerequisite threads after documenting the intentional contracts and accepted shared-framework limitations; the duplicate description finding was conceded by Greptile. BillLeoutsakosvl346 also resolved the retryability thread after explaining that retries occur inside the OCI client and the response marker is not currently consumed by workflow-level retry handling.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/oauth/oauth.ts | Registers OCI Notifications with a distinct service provider ID while continuing to share the OCI API-key service-account credential provider. |
| apps/sim/lib/internal/oci-notifications/operations.ts | Implements signed OCI Notifications topic, subscription, lock, compartment, confirmation, and publication operations. |
| apps/sim/lib/internal/oci-notifications/execute-tool.ts | Validates and dispatches OCI Notifications operations through the shared OCI client. |
| apps/sim/lib/internal/oci-notifications/endpoints.ts | Discovers authenticated publishing and subscription endpoints through GetTopic. |
| apps/sim/blocks/blocks/oci_notifications.ts | Defines the integration block, operation-specific inputs, selectors, parsing, and adapter mappings. |
| apps/sim/lib/selectors/server/providers/oci-notifications.ts | Implements credential-bound topic and subscription selector providers. |
| apps/sim/lib/internal/oci-notifications/schema.ts | Defines request validation contracts for all supported OCI Notifications operations. |
| apps/docs/content/docs/integrations/oci_notifications.mdx | Documents all integration operations, inputs, outputs, endpoint discovery, and publication retry caveats. |
Sequence Diagram
sequenceDiagram
participant W as Sim Workflow
participant B as OCI Notifications Block
participant C as Shared OCI Signed Client
participant T as OCI GetTopic Endpoint
participant N as OCI Notifications Endpoint
W->>B: Invoke selected operation
B->>C: Load OCI API-key credential
alt Topic-native operation
C->>N: Send signed request
else Subscription or publish operation
C->>T: GetTopic
T-->>C: Authenticated API endpoint
C->>N: Send signed request to discovered endpoint
end
N-->>B: Status, request ID, and operation result
B-->>W: Tool response
Reviews (4): Last reviewed commit: "feat(oci): add native Notifications inte..." | Re-trigger Greptile
862ca82 to
f91e1c3
Compare
f91e1c3 to
2b9e913
Compare
Summary
Stack
feat/oci-foundation.3fa59e758f5d282e95978aca7482a91a4f3dcee5.Type of Change
Testing
2b9e91324f1e1b64b04dc296a7ac9feac1915c06: lint/audits, docs sync, workspace type checks, all three test shards, migration consistency, and application build. All 45 focused Notifications tests passed.Checklist