Skip to content

feat(oci-devops): add native integration - #7548

Open
BillLeoutsakosvl346 wants to merge 1 commit into
feat/oci-foundationfrom
feat/oci-devops
Open

feat(oci-devops): add native integration#7548
BillLeoutsakosvl346 wants to merge 1 commit into
feat/oci-foundationfrom
feat/oci-devops

Conversation

@BillLeoutsakosvl346

@BillLeoutsakosvl346 BillLeoutsakosvl346 commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

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

  • New feature
  • Documentation (generated integration artifacts)

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.yml workflow 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 at ee955895d25684a09077b4efac972895adb3f9f2. 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

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 6, 2026 1:21am UTC

Request Review

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@greptile-apps

greptile-apps Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds 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.

  • Registers 69 OCI DevOps actions and 13 selectors.
  • Validates discriminated OCI request and response schemas while excluding sensitive response fields.
  • Supports conditional mutations through retry tokens and ETags.
  • The changes since the previous review are formatting-only and introduce no new behavioral findings.

Confidence Score: 5/5

The 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.

Important Files Changed

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]
Loading

Reviews (6): Last reviewed commit: "feat(oci-devops): add native integration" | Re-trigger Greptile

Comment thread apps/sim/lib/internal/oci-devops/schema.ts Outdated
Comment thread apps/sim/lib/internal/oci-devops/operations.ts
Comment thread apps/sim/lib/internal/oci-devops/schema.ts Outdated
Comment thread apps/sim/tools/oci_devops/index.ts Outdated
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile Please review the current head 41f35f1. This revision applies the remaining GitHub formatting suggestions; the integration scope and provider contracts are unchanged.

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile Please review the latest head. The only change since 41f35f1 is the final schema-helper line wrap requested by GitHub lint. Independent validation has agreed there are no unresolved integration findings.

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