Upgrade TypeScript

[lucacavallaro]

No description provided.

[dx-release-internal]

⚠️ Missing Nx Version Plan

Nx Release reports that this PR has touched projects without version plan coverage.

Run pnpm nx release plan and add the generated version plans.

Nx Plan Check result:

NX   Touched projects based on changed files

  - dx-metrics-import
  - @pagopa/opex-dashboard
  - dx-metrics
  - @pagopa/dx-mcpserver
  - docs
  - @pagopa/dx-cli
  - @pagopa/dx-metrics-core
  - @pagopa/azure-tracing
  - @pagopa/dx-savemoney
  - @pagopa-dx/terraform-plan-storage-download
  - @pagopa-dx/terraform-plan-storage-upload
  - run-dx-task
  - pr-comment-action

NOTE: You can adjust your "versionPlans.ignorePatternsForPlanCheck" config to stop certain files from resulting in projects being classed as touched for the purposes of this command.


 NX   Project "renew-tls-certificate" has pending bumps in version plan(s)

  - "patch" in version-plan-1785250247375.md


 NX   Project "@pagopa/docker-build-push" has pending bumps in version plan(s)

  - "patch" in version-plan-1785250247375.md


 NX   Project "@pagopa/azure-tracing" has pending bumps in version plan(s)

  - "patch" in version-plan-1785514869108.md



 NX   Touched projects missing version plans

The following touched projects do not feature in any version plan files:
  - dx-metrics-import
  - @pagopa/opex-dashboard
  - dx-metrics
  - @pagopa/dx-mcpserver
  - docs
  - @pagopa/dx-cli
  - @pagopa/dx-metrics-core
  - @pagopa/dx-savemoney
  - @pagopa-dx/terraform-plan-storage-download
  - @pagopa-dx/terraform-plan-storage-upload
  - run-dx-task
  - pr-comment-action

Please use `nx release plan` to generate missing version plans, or adjust your "versionPlans.ignorePatternsForPlanCheck" config stop certain files from affecting the projects for the purposes of this command.

Run with --verbose to see the full list of changed files used for the touched projects logic.

This comment is managed automatically by Nx Release. Version plan guide

[github-actions]

Code Review Report

Terraform Plan: infra/policy/dev - ✅ Success

Plan: 2 to add, 0 to change, 0 to destroy.

Note

Full plan output is not included in this comment.
See the workflow run logs or downloaded Terraform plan report artifacts for the complete output.

Terraform Plan: infra/resources/dev - ❌ Failed

Warning

Warning: Reference to undefined provider

on aws.tf line 12, in module "mcp_server":
12: aws.us_east_1 = aws.us-east-1

There is no explicit declaration for local provider name "aws.us_east_1" in
module.mcp_server, so Terraform is assuming you mean to pass a configuration
for "hashicorp/aws".

If you also control the child module, add a required_providers entry named
"aws.us_east_1" with the source address "hashicorp/aws".
::error::Terraform exited with code 1.

Caution

Error: retrieving Policy for Service (Subscription: "35e6e3b2-4388-470e-a1b9-ad3bc34326d1"
Resource Group Name: "dx-d-itn-ai-rg-01"
Service Name: "dx-d-itn-ai-apim-01"): unexpected status 422 (422 Unprocessable Entity) with error: ManagementApiRequestFailed: Failed to connect to management endpoint dx-d-itn-ai-apim-01.management.azure-api.net:3443 for a service deployed in a Virtual Network. Make sure to follow guidance at https://aka.ms/apim-vnet-common-issues for Inbound connectivity to Management endpoint. Check 'ApiManagement Control Plane - inbound' connectivity at https://aka.ms/apimnetworkstatus.

Caution

Error: retrieving Backend (Subscription: "35e6e3b2-4388-470e-a1b9-ad3bc34326d1"
Resource Group Name: "dx-d-itn-ai-rg-01"
Service Name: "dx-d-itn-ai-apim-01"
Backend: "foundry"): unexpected status 422 (422 Unprocessable Entity) with error: ManagementApiRequestFailed: Failed to connect to management endpoint dx-d-itn-ai-apim-01.management.azure-api.net:3443 for a service deployed in a Virtual Network. Make sure to follow guidance at https://aka.ms/apim-vnet-common-issues for Inbound connectivity to Management endpoint. Check 'ApiManagement Control Plane - inbound' connectivity at https://aka.ms/apimnetworkstatus.

Caution

Error: retrieving Logger (Subscription: "35e6e3b2-4388-470e-a1b9-ad3bc34326d1"
Resource Group Name: "dx-d-itn-ai-rg-01"
Service Name: "dx-d-itn-ai-apim-01"
Logger: "dx-d-itn-ai-apim-01-logger"): unexpected status 422 (422 Unprocessable Entity) with error: ManagementApiRequestFailed: Failed to connect to management endpoint dx-d-itn-ai-apim-01.management.azure-api.net:3443 for a service deployed in a Virtual Network. Make sure to follow guidance at https://aka.ms/apim-vnet-common-issues for Inbound connectivity to Management endpoint. Check 'ApiManagement Control Plane - inbound' connectivity at https://aka.ms/apimnetworkstatus.

Caution

Error: retrieving Api (Subscription: "35e6e3b2-4388-470e-a1b9-ad3bc34326d1"
Resource Group Name: "dx-d-itn-ai-rg-01"
Service Name: "dx-d-itn-ai-apim-01"
Api: "ai-foundry;rev=1"): unexpected status 422 (422 Unprocessable Entity) with error: ManagementApiRequestFailed: Failed to connect to management endpoint dx-d-itn-ai-apim-01.management.azure-api.net:3443 for a service deployed in a Virtual Network. Make sure to follow guidance at https://aka.ms/apim-vnet-common-issues for Inbound connectivity to Management endpoint. Check 'ApiManagement Control Plane - inbound' connectivity at https://aka.ms/apimnetworkstatus.

Caution

Error: retrieving Operation (Subscription: "35e6e3b2-4388-470e-a1b9-ad3bc34326d1"
Resource Group Name: "dx-d-itn-ai-rg-01"
Service Name: "dx-d-itn-ai-apim-01"
Api: "ai-foundry"
Operation: "create-response"): unexpected status 422 (422 Unprocessable Entity) with error: ManagementApiRequestFailed: Failed to connect to management endpoint dx-d-itn-ai-apim-01.management.azure-api.net:3443 for a service deployed in a Virtual Network. Make sure to follow guidance at https://aka.ms/apim-vnet-common-issues for Inbound connectivity to Management endpoint. Check 'ApiManagement Control Plane - inbound' connectivity at https://aka.ms/apimnetworkstatus.

Caution

Error: retrieving Api Version Set (Subscription: "35e6e3b2-4388-470e-a1b9-ad3bc34326d1"
Resource Group Name: "dx-d-itn-ai-rg-01"
Service Name: "dx-d-itn-ai-apim-01"
Version Set: "ai-foundry"): unexpected status 422 (422 Unprocessable Entity) with error: ManagementApiRequestFailed: Failed to connect to management endpoint dx-d-itn-ai-apim-01.management.azure-api.net:3443 for a service deployed in a Virtual Network. Make sure to follow guidance at https://aka.ms/apim-vnet-common-issues for Inbound connectivity to Management endpoint. Check 'ApiManagement Control Plane - inbound' connectivity at https://aka.ms/apimnetworkstatus.

Plan: 0 to add, 1 to change, 6 to destroy.

Note

Full plan output is not included in this comment.
See the workflow run logs or downloaded Terraform plan report artifacts for the complete output.

Terraform Plan: infra/resources/prod - ✅ Success

Warning

Warning: Argument is deprecated

with module.container_app_infra.azurerm_monitor_diagnostic_setting.container_app_environment,
on .terraform/modules/container_app_infra/monitoring.tf line 3, in resource "azurerm_monitor_diagnostic_setting" "container_app_environment":
3: resource "azurerm_monitor_diagnostic_setting" "container_app_environment" {

metric has been deprecated in favour of the enabled_metric property and
will be removed in v5.0 of the AzureRM provider

Plan: 0 to add, 7 to change, 0 to destroy.

Note

Full plan output is not included in this comment.
See the workflow run logs or downloaded Terraform plan report artifacts for the complete output.

Terraform Plan: infra/core/dev - ✅ Success

Plan: 0 to add, 44 to change, 0 to destroy.

Note

Full plan output is not included in this comment.
See the workflow run logs or downloaded Terraform plan report artifacts for the complete output.

Terraform Plan: infra/core/prod - ✅ Success

Plan: 0 to add, 2 to change, 0 to destroy.

Note

Full plan output is not included in this comment.
See the workflow run logs or downloaded Terraform plan report artifacts for the complete output.

Terraform Plan: infra/bootstrapper/dev - ✅ Success

Warning

Warning: Argument is deprecated

with module.azure.github_actions_environment_secret.integration_tests_client_id[0],
on ../_modules/azure/custom_environments.tf line 16, in resource "github_actions_environment_secret" "integration_tests_client_id":
16: plaintext_value = azurerm_user_assigned_identity.integration_tests[0].client_id

Use value.

(and 26 more similar warnings elsewhere)

Warning

Warning: Redundant ignore_changes element

on ../_modules/azure/secrets.tf line 1, in resource "github_actions_secret" "codecov_token":
1: resource "github_actions_secret" "codecov_token" {

Adding an attribute name to ignore_changes tells Terraform to ignore future
changes to the argument in configuration after the object has been created,
retaining the value originally configured.

The attribute remote_updated_at is decided by the provider alone and
therefore there can be no configured value to compare with. Including this
attribute in ignore_changes has no effect. Remove the attribute from
ignore_changes to quiet this warning.

(and one more similar warning elsewhere)

Plan: 3 to add, 14 to change, 14 to destroy.

Note

Full plan output is not included in this comment.
See the workflow run logs or downloaded Terraform plan report artifacts for the complete output.

---

Triggered by @lucacavallaro, commit: 91909efee675f7cf76317691f5bd3944106e1e8d.

[github-actions]

Azure Static Web Apps: Your stage site is ready! Visit it here: https://proud-mud-0405c6703-2029.westeurope.1.azurestaticapps.net

[kin0992]

issue: I think this is out of scope.

The PR should handle only the TS upgrade, not changing the bundler (which should be in a separate PR)

[lucacavallaro]

I will check that, If I remember correctly, tsup does not work with latest typescript version due to compiler changes.

[kin0992]

question: why not moving directly to version 7?

[lucacavallaro]

@kin0992 version 7 is a complete rewrite in Go, that can break our toolchain and IDE-integration. Version 6.x is a "migration release" meant to prepare the codebase to be ported on version 7 (which shares the same config file and same deprecations).

I prefer to do a gradual upgrade, in order to avoid breaking our tools.

[kin0992]

question: is this true? is tsup not working with version 6?

[kin0992]

note: it looks to me that tsup continues to work with TS 6: https://github.com/pagopa/dx/actions/runs/30647189843/job/91211379188?pr=2029#step:9:980