feat(oci-devops): add native integration - #7548
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryAdds a native OCI DevOps integration covering repository administration, build and deployment workflows, typed OCI resources, server-backed selectors, bounded status reads, generated tool metadata, and documentation.
Confidence Score: 5/5The PR appears safe to merge because no new actionable issue was found and all previous findings are resolved, fixed, or withdrawn. The latest changes only reformat existing schema expressions, remove whitespace, and reorder type-only imports without changing execution or validation behavior. The repository-count and duplicate-export findings were fixed, while the work-request query-key finding was withdrawn after the documented OCI contract was confirmed; the remaining resolved provider-bound finding does not affect merge safety.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/internal/oci-devops/schema.ts | Defines typed OCI DevOps input and allowlisted output schemas; recent changes only apply lint formatting. |
| apps/sim/lib/internal/oci-devops/operations.ts | Implements request construction and bounded OCI DevOps operation handling. |
| apps/sim/lib/internal/oci-devops/execute-tool.ts | Connects registered native tools to authorized OCI DevOps execution. |
| apps/sim/lib/selectors/server/providers/oci-devops.ts | Implements parent-bound OCI DevOps selectors; the latest edit only removes blank whitespace. |
| apps/sim/blocks/blocks/oci_devops.ts | Defines the OCI DevOps block and its supported actions. |
| apps/sim/tools/oci_devops/index.ts | Exports the OCI DevOps tool set using the established alias pattern. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
B[OCI DevOps block] --> T[Native tool definitions]
T --> E[Internal operation executor]
S[Server-backed selectors] --> E
E --> A[OCI DevOps REST API]
A --> V[Allowlisted response schemas]
V --> O[Bounded workflow outputs]
Reviews (6): Last reviewed commit: "feat(oci-devops): add native integration" | Re-trigger Greptile
661201b to
5bb3fd5
Compare
5bb3fd5 to
e7b7d86
Compare
e7b7d86 to
1c14c35
Compare
1c14c35 to
41f35f1
Compare
41f35f1 to
ee95589
Compare
Summary
Add a native OCI DevOps block with 69 actions and 13 server-backed selectors for repository administration and browsing, build and deployment pipelines and executions, typed stages, environments, artifact descriptors, connections, native build triggers, and bounded work-request status.
Stacked on #7444 (
feat/oci-foundation); reuses its credential authorization and OCI client. Supports all four build-stage types, 19 deployment-stage types, and four deployment execution modes. Mutations use explicit retry tokens where supported and ETags for conditional changes. Status reads make one bounded request; acceptance does not imply execution success. Outputs omit connection credentials, webhook secrets, argument values, inline artifacts, and unrestricted logs.No Git transport, inbound Sim triggers, external webhook-secret creation, artifact uploads, infrastructure provisioning, or tenant operations. No foundation or general platform changes are included.
Type of Change
Testing
Focused tests cover authorization, request contracts, discriminated schemas, retry/ETag handling, lifecycle states, bounded pages and outputs, selector parent binding, and internal registration. Validation runs exclusively through the existing GitHub
test-build.ymlworkflow dispatched at this branch; the final-revision run passed lint, repository audits, generated-artifact freshness, all-workspace type checking, all test shards, and the application build atee955895d25684a09077b4efac972895adb3f9f2. No local tests, lint, type checks, builds, or tenant verification were performed. The existing metadata, documentation/catalog, deployment-config, and docs-manifest generators were run.Independent integration review of the combined foundation-plus-child behavior is complete with no unresolved child-specific findings. Greptile reports 5/5 for the final revision; all four review threads have individual dispositions and are resolved. Foundation PR change requests: None. Both PRs remain unmerged.
The only additional file beyond the planned registration/generated corrections is the existing selector manifest census test: four expected counts were updated for the 13 new selectors. No shared implementation or foundation repairs are included.
Checklist