Revert template state model#231
Conversation
This reverts commit 2e62fdb.
✱ Stainless preview builds for hypemanThis PR will update the Edit this comment to update it. It will appear in the SDK's changelogs. ✅ hypeman-go studio · code · diff
✅ hypeman-openapi studio · code · diff
✅ hypeman-typescript studio · code · diff
This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push. |
Monitoring Plan: Remove Template Instance StateThis PR removes the Key risks to watch post-deploy: (1) any existing Key risks:
|
Summary
Tests
Note
Medium Risk
Removes the
Templatelifecycle state and associated API endpoints, which is a breaking change for clients and alters fork/delete/restore behavior for standby instances. Risk is moderate due to lifecycle semantics and generated OpenAPI client/server surface changes.Overview
Reverts the template instance concept by removing the
Templatestate frominstances(state machine, metadata fields, state derivation, and transition tests) and deleting all template-specific logic such as fork-from-template handling and delete/demote fork-count guards.Removes the
POST /instances/{id}/promote-templateandPOST /instances/{id}/demote-templateendpoints end-to-end (API handlers, scopes, OpenAPI schema/enum, generatedlib/oapiclient/server code, andstainless.yaml), and drops the now-obsolete template-focused unit test suite.Reviewed by Cursor Bugbot for commit 3787ab3. Bugbot is set up for automated code reviews on this repo. Configure here.