[feat][gov] Update repo to reflect v.2.9.0 of the API #52
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
simplyluke
commented
Nov 7, 2025
| values = [ | ||
| { | ||
| environment_id = "production" | ||
| environment_id = "a1b2c3d4-e5f6-4a7b-8c9d-0e1f2a3b4c5d" # prod |
Contributor
Author
There was a problem hiding this comment.
Quality of life improvement here. IDs for envs are always UUIDs, not names.
simplyluke
commented
Nov 7, 2025
simplyluke
commented
Nov 7, 2025
| func newHTTPRecorder(t *testing.T) *recorder.Recorder { | ||
| t.Helper() | ||
|
|
||
| // Create an HTTP client with TLS verification disabled for recording. |
Contributor
Author
There was a problem hiding this comment.
I had to make this change to record the tests, at least with an agent.
simplyluke
commented
Nov 7, 2025
|
|
||
| One of these is for the imprecise conversion of zod discriminated unions (used internally) to openapi - instead of `oneOf` schemas, `anyOf` is more accurate for nested object types. | ||
|
|
||
| ### Known OpenAPI Spec Corrections |
Contributor
Author
There was a problem hiding this comment.
Added a lot of context on how I/Sonnet were able to get this to compile and work for 2.9.0 which should be helpful going forward for future updates. Effectively everything below this in the PR is either re-recorded tests or generated code.
simplyluke
changed the title
simplyluke
added a commit
that referenced
this pull request
Nov 13, 2025
notes in readme all tests passing, working on a local server with real examples docs lint fixes add comment on folder deletion recursion notes on reviewing better handling for config variables as secret update docs lint lint add resources resource docs fix test lint lint lint docs update wip on resource configs Bump golang.org/x/crypto from 0.23.0 to 0.35.0 (#53) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.23.0 to 0.35.0. - [Commits](golang/crypto@v0.23.0...v0.35.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-version: 0.35.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> [feat][gov] Update repo to reflect v.2.9.0 of the API (#52) * update repo to reflect v.2.9.0 of the api * fixes to cursor slop * notes in readme * all tests passing, working on a local server with real examples * docs * lint fixes * add comment on folder deletion recursion * notes on reviewing * better handling for config variables as secret * update docs * lint * lint [feat][gov] Add ability to manage resources (#55) * update repo to reflect v.2.9.0 of the api * fixes to cursor slop * notes in readme * all tests passing, working on a local server with real examples * docs * lint fixes * add comment on folder deletion recursion * notes on reviewing * better handling for config variables as secret * update docs * lint * lint * add resources resource * docs * fix test * lint * lint * lint * docs update * more accurate docs probable fixes more linting
simplyluke
added a commit
that referenced
this pull request
Nov 14, 2025
* update repo to reflect v.2.9.0 of the api * working notes in readme all tests passing, working on a local server with real examples docs lint fixes add comment on folder deletion recursion notes on reviewing better handling for config variables as secret update docs lint lint add resources resource docs fix test lint lint lint docs update wip on resource configs Bump golang.org/x/crypto from 0.23.0 to 0.35.0 (#53) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.23.0 to 0.35.0. - [Commits](golang/crypto@v0.23.0...v0.35.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-version: 0.35.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> [feat][gov] Update repo to reflect v.2.9.0 of the API (#52) * update repo to reflect v.2.9.0 of the api * fixes to cursor slop * notes in readme * all tests passing, working on a local server with real examples * docs * lint fixes * add comment on folder deletion recursion * notes on reviewing * better handling for config variables as secret * update docs * lint * lint [feat][gov] Add ability to manage resources (#55) * update repo to reflect v.2.9.0 of the api * fixes to cursor slop * notes in readme * all tests passing, working on a local server with real examples * docs * lint fixes * add comment on folder deletion recursion * notes on reviewing * better handling for config variables as secret * update docs * lint * lint * add resources resource * docs * fix test * lint * lint * lint * docs update * more accurate docs probable fixes more linting * docs * more linting * mention ignoring options in docs * tf file
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
My apologies for the size of this PR. It's almost entirely generated code and I'm aiming to comment on the PR such that reviewers can more easily identify key changes.
This PR updates our OpenAPI spec to v2.9.0 of the retool API, which is current. It generates our SDK code and updates the provider to leverage these updates to bring existing resources/data sources up to date with the current API and its capabilities.
As a follow up I'll be adding some more resources such that more of Retool can be managed via terraform.
This PR should be backwards compatible, all new fields are optional, but given the breadth of the change I'm marking this as our 1.0 release.
The two files that have changed a lot are our SSO and Source Control implementations. These end up much better organized/typed, and the migration to 2.9.0 with autogen code ended up breaking some of the prior typing.
Note that much of the code here was autogenerated for the SDK, and outside of that LLMs were leveraged heavily. I've validated the outputs to the best of my ability (read: I have not manually reviewed all autogenerated SDK code, but have gone through everything an LLM touched)
Tests
I've done the following to validate the changes: