Skip to content

Conversation

@hiroTamada
Copy link
Contributor

@hiroTamada hiroTamada commented Jan 26, 2026

Summary

  • Add builder_instance_id field to the Build API response for debugging purposes
  • This exposes the existing BuilderInstance field that was already stored in build metadata but not returned to clients

Changes

  • openapi.yaml: Add builder_instance_id to Build schema (nullable string)
  • lib/builds/types.go: Add BuilderInstanceID field to domain Build struct
  • lib/builds/storage.go: Update toBuild() to copy BuilderInstanceBuilderInstanceID
  • cmd/api/api/builds.go: Update buildToOAPI() to map the new field

Test plan

  • Run make oapi-generate to regenerate lib/oapi/oapi.go
  • Verify API returns builder_instance_id in build responses
  • Check that existing builds without the field return null

Notes

  • No existing tests for buildToOAPI or toBuild conversion functions
  • The lib/oapi/oapi.go file needs to be regenerated after checkout via make oapi-generate

Note

Adds the builder VM instance ID to build responses and ensures codegen stays in sync.

  • Extend Build schema with builder_instance_id (nullable) in openapi.yaml; regenerate lib/oapi/oapi.go
  • Add BuilderInstanceID to domain Build (lib/builds/types.go) and map from stored BuilderInstance in toBuild() (lib/builds/storage.go)
  • Map field in API response via buildToOAPI() (cmd/api/api/builds.go)
  • CI: update .github/workflows/test.yml to run make oapi-generate before build/tests

Written by Cursor Bugbot for commit f1ef89f. This will update automatically on new commits. Configure here.

@github-actions
Copy link

github-actions bot commented Jan 26, 2026

✱ Stainless preview builds

This PR will update the hypeman SDKs with the following commit message.

feat(builds): expose builder VM instance ID in build response
⚠️ hypeman-typescript studio · code

There was a regression in your SDK.
generate ⚠️build ✅lint ✅test ✅

npm install https://pkg.stainless.com/s/hypeman-typescript/46d5e26a7c2d5f53b4a9d88558d0f96561cf1b7d/dist.tar.gz
⚠️ hypeman-go studio · code

There was a regression in your SDK.
generate ⚠️lint ✅test ✅

go get github.com/stainless-sdks/hypeman-go@7fea1660f3d17d8a35f5d2f6aa352b553785624b
hypeman-cli studio

Code was not generated because there was a fatal error.


This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push.
If you push custom code to the preview branch, re-run this workflow to update the comment.
Last updated: 2026-01-27 20:06:03 UTC

- Add `builder_instance_id` field to Build schema in openapi.yaml
- Regenerate lib/oapi/oapi.go with oapi-codegen
- Add BuilderInstanceID field to domain Build struct
- Update toBuild() to map BuilderInstance -> BuilderInstanceID
- Update buildToOAPI() to include the new field in API responses
- Update CI to regenerate OpenAPI code before building

This exposes the existing BuilderInstance field that was already stored
in build metadata but not returned to clients, useful for debugging.
@hiroTamada hiroTamada force-pushed the feat/add-builder-instance-id-to-build-response branch from 7f437d1 to 4fbb257 Compare January 26, 2026 20:15
@hiroTamada hiroTamada merged commit 0383db5 into main Jan 27, 2026
2 of 3 checks passed
@hiroTamada hiroTamada deleted the feat/add-builder-instance-id-to-build-response branch January 27, 2026 20:05
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.

3 participants