Skip to content

refactor(sfcompute): use Brev integration API - #137

Merged
drewmalin merged 3 commits into
brevdev:mainfrom
sfcompute:mason/brev-integration-client
Aug 5, 2026
Merged

refactor(sfcompute): use Brev integration API#137
drewmalin merged 3 commits into
brevdev:mainfrom
sfcompute:mason/brev-integration-client

Conversation

@masonsfc

@masonsfc masonsfc commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

SFCompute now exposes a stable Brev-specific API contract through sfcompute/sfcompute#8035. This moves the provider off the generated preview V2 SDK and limits the integration to the resources Brev uses.

  • call /integrations/brev/v1 for instance lifecycle, SSH information, and pool capacity
  • preserve the existing Brev provider interface, stable instance type ID, lifecycle mapping, filtering, tags, and idempotent termination behavior
  • exhaust cursor pagination so instance listing and capacity accounting include every instance
  • remove the unused sfc-go dependency

Validation

  • go run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.6.0 run ./v1/providers/sfcomputev2
  • go test -race ./v1/providers/sfcomputev2
  • go vet ./v1/providers/sfcomputev2
  • go test ./...

@masonsfc
masonsfc force-pushed the mason/brev-integration-client branch 2 times, most recently from 6f05f79 to b6b5015 Compare July 31, 2026 21:41
@masonsfc
masonsfc force-pushed the mason/brev-integration-client branch from b6b5015 to 47627b6 Compare July 31, 2026 21:58
@masonsfc
masonsfc marked this pull request as ready for review August 4, 2026 16:45
@masonsfc
masonsfc requested a review from a team as a code owner August 4, 2026 16:45
@drewmalin
drewmalin merged commit 2f7e07f into brevdev:main Aug 5, 2026
5 of 7 checks passed
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.

2 participants