feat(new-nav): add Terraform creation flow#2596
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## new-navigation #2596 +/- ##
=================================================
Coverage ? 44.47%
=================================================
Files ? 1073
Lines ? 22156
Branches ? 6493
=================================================
Hits ? 9854
Misses ? 10538
Partials ? 1764
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds a new Terraform service creation funnel to the “new-nav” (TanStack Router) console so users can create Terraform services via a multi-step flow.
Changes:
- Introduces a Terraform creation flow (general → configuration → variables → summary) under
/service/create/terraform. - Wires the Terraform card in the “Service New” page to the new creation-flow route.
- Exposes Terraform configuration/variables settings components from
service-settingsand adds console-v5 routes + route tree updates.
Reviewed changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| libs/pages/application/src/lib/feature/page-settings-terraform-variables/page-settings-terraform-variables.tsx | Removes an unused React import in the Terraform variables settings page. |
| libs/domains/services/feature/src/lib/service-new/service-new.tsx | Updates the Terraform service card link to use the new creation-flow path builder. |
| libs/domains/service-terraform/feature/src/lib/terraform-creation-flow/terraform-creation-flow.tsx | Adds the Terraform funnel wrapper with shared RHF state + step metadata. |
| libs/domains/service-terraform/feature/src/lib/terraform-creation-flow/step-general/step-general.tsx | Adds step 1 (general/source) for Terraform creation. |
| libs/domains/service-terraform/feature/src/lib/terraform-creation-flow/step-configuration/step-configuration.tsx | Adds step 2 (Terraform configuration) and navigation controls. |
| libs/domains/service-terraform/feature/src/lib/terraform-creation-flow/step-variables/step-variables.tsx | Adds step 3 (Terraform variables) and navigation controls. |
| libs/domains/service-terraform/feature/src/lib/terraform-creation-flow/step-summary/step-summary.tsx | Adds step 4 (summary) including create + create-and-plan submission. |
| libs/domains/service-terraform/feature/src/lib/terraform-creation-flow/index.ts | Barrel export for the Terraform creation-flow components. |
| libs/domains/service-terraform/feature/src/lib/hooks/use-terraform-create-context/use-terraform-create-context.tsx | Adds a dedicated context/hook for Terraform creation state. |
| libs/domains/service-terraform/feature/src/index.ts | Exports the Terraform creation-flow from the package entrypoint. |
| libs/domains/service-settings/feature/src/lib/terraform-variables-settings/terraform-variables-settings.tsx | Minor text color token update. |
| libs/domains/service-settings/feature/src/index.ts | Exports Terraform settings components (configuration + variables). |
| apps/console-v5/src/routes/_authenticated/organization/route.tsx | Adds Terraform creation route to the bypass-layout allowlist. |
| apps/console-v5/src/routes/_authenticated/organization/$organizationId/project/$projectId/environment/$environmentId/service/create/terraform/route.tsx | Adds the Terraform creation-flow parent route (Outlet + provider). |
| apps/console-v5/src/routes/_authenticated/organization/$organizationId/project/$projectId/environment/$environmentId/service/create/terraform/index.tsx | Adds index redirect to the Terraform “general” step. |
| apps/console-v5/src/routes/_authenticated/organization/$organizationId/project/$projectId/environment/$environmentId/service/create/terraform/general.tsx | Adds route for step 1. |
| apps/console-v5/src/routes/_authenticated/organization/$organizationId/project/$projectId/environment/$environmentId/service/create/terraform/terraform-configuration.tsx | Adds route for step 2. |
| apps/console-v5/src/routes/_authenticated/organization/$organizationId/project/$projectId/environment/$environmentId/service/create/terraform/input-variables.tsx | Adds route for step 3. |
| apps/console-v5/src/routes/_authenticated/organization/$organizationId/project/$projectId/environment/$environmentId/service/create/terraform/summary.tsx | Adds route for step 4. |
| apps/console-v5/src/routeTree.gen.ts | Regenerates route tree to include the new Terraform routes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Some feedbacks:
Let's putrounded-mdon the Backend configuration and Execution credentials cards
Command should be vertically centered here
Can't retrieve Qovery variables from here (maybe because of localhost?)
Default execution timeout has also gone from 3600 seconds to 60, not sure if it's intended or not!
Here file path should probably be hidden if we don't have any value (was on the old console also)
There was a problem hiding this comment.
Thanks @rmnbrd !
- I think the GitHub Copilot comment is relevant for circular dependencies !
- The link is missing from these cards (this screen is just an example, there will probably be more) and needs to be updated
- If
File pathsis empty, we need to hide it from the summary
6b330d8 to
4124294
Compare
|
@rmnbrd Did not see it before but we can just have surface-neutral-component when the line is selected instead of bg-surface-neutral-componentActive |


Summary
PR adding the Terraform service creation flow
Screenshots / Recordings
📺 https://www.loom.com/share/d82fe4dc16fb4d83b3d4e84a7678e239