Skip to content

chore(NA): upgrade typescript into v5.1.6#186437

Merged
mistic merged 68 commits intoelastic:mainfrom
mistic:upgrade-ts-v5.1
Jul 29, 2024
Merged

chore(NA): upgrade typescript into v5.1.6#186437
mistic merged 68 commits intoelastic:mainfrom
mistic:upgrade-ts-v5.1

Conversation

@mistic
Copy link
Copy Markdown
Contributor

@mistic mistic commented Jun 19, 2024 •
edited by mbondyra
Loading

This PR bumps the Typescript version used on Kibana into v5.1.6. The full set of changes can be found here and here.

In order to make assumptions over the codebase and all the type of errors I choose to mark the failures with @ts-expect-error and let each team decide how to handle it.

There is a list below with the files (and teams those belong to) where the annotations were added. If each team could provide help of fixing the ones under their domain it would be fantastic. It can be done in this PR or in subsequent ones.

Here it goes the list for follow up:

@elastic/kibana-core

@elastic/kibana-data-discovery

@elastic/obs-knowledge-team

@elastic/kibana-esql

@elastic/kibana-management

@elastic/kibana-security

@elastic/appex-sharedux

@elastic/kibana-visualizations

@elastic/kibana-presentation

@elastic/response-ops

@elastic/ml-ui

@elastic/search-kibana

@elastic/fleet

@elastic/kibana-gis

@elastic/stack-monitoring

@elastic/obs-ux-infra_services-team

@elastic/obs-ai-assistant

@elastic/search-kibana

@elastic/kibana-security

@elastic/security-solution

@elastic/security-threat-hunting-investigations

@elastic/obs-ux-management-team

@mistic mistic changed the title upgrade typescript into v4.9.5 upgrade typescript into v5.1.6 Jun 19, 2024
@mistic mistic changed the title upgrade typescript into v5.1.6 chore(NA): upgrade typescript into v5.1.6 Jun 19, 2024
@mistic
Copy link
Copy Markdown
Contributor Author

mistic commented Jun 19, 2024

@elastic/kibana-core I believe @kbn/telemetry-tools would have to be fixed to work with TS v5, could you give me an help here please?

@afharo
Copy link
Copy Markdown
Member

afharo commented Jun 19, 2024

@mistic ack! I'll take a look today

@afharo
Copy link
Copy Markdown
Member

afharo commented Jun 19, 2024

My first look shows that the new version of TS changed the module resolution APIs. We need to adapt this section of the code for that:

kibana/packages/kbn-telemetry-tools/src/tools/utils.ts

Lines 149 to 156 in e5fa891

const resolvedModule = (originalSource as any).resolvedModules.get(importedModuleName);
if (!resolvedModule) {
throw new Error(
`Import for [${importedModuleName}] in [${originalSource.fileName}] could not be resolved by TypeScript`
);
}
const resolvedModuleSourceFile = program.getSourceFile(resolvedModule.resolvedFileName);

Unit tests in that package already highlight that, so it should be "easier" to iterate.

@mistic
Copy link
Copy Markdown
Contributor Author

mistic commented Jun 19, 2024

@afharo do you think something like what I've here it's okay?

@afharo
Copy link
Copy Markdown
Member

afharo commented Jun 19, 2024

@afharo do you think something like what I've here it's okay?

That's exactly what I was looking for! You are a real TS guru!

@mistic
Copy link
Copy Markdown
Contributor Author

mistic commented Jun 19, 2024

@elastic/ml-ui I think the schema extractor would have to also be updated to work properly on TS v5. I've skipped for now the jest tests that were failing here. I made some experiments but the unit tests keep getting different outputs from what you have there now and I didn't want to make more assumptions. Could you please take a look on it?

@elastic elastic deleted a comment from kibana-ci Jul 3, 2024
@elastic elastic deleted a comment from elasticmachine Jul 3, 2024
@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Jul 29, 2024 •
edited
Loading

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
aiops 557.8KB 557.8KB +16.0B
cases 503.3KB 503.3KB +8.0B
data 48.0KB 48.0KB +7.0B
ml 4.6MB 4.6MB -24.0B
securitySolution 20.4MB 20.4MB +167.0B
stackConnectors 554.6KB 554.7KB +149.0B
transform 632.6KB 632.6KB +8.0B
triggersActionsUi 1.6MB 1.6MB +96.0B
visDefaultEditor 142.3KB 142.3KB +8.0B
visTypeHeatmap 9.8KB 9.8KB +2.0B
total +437.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-srcJs 3.3MB 3.3MB +16.0B
stackConnectors 53.7KB 53.7KB +8.0B
total +24.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@ymao1 ymao1 mentioned this pull request Jul 30, 2024
Open
@tonyghiani tonyghiani mentioned this pull request Aug 5, 2024
Closed
@mistic mistic mentioned this pull request Aug 30, 2024
Merged
@kibanamachine kibanamachine mentioned this pull request Oct 25, 2024
Merged
@kibanamachine kibanamachine mentioned this pull request Nov 22, 2024
Merged
1 task
@kibanamachine kibanamachine mentioned this pull request Dec 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alvarezmelissa87 alvarezmelissa87 alvarezmelissa87 left review comments

@nickofthyme nickofthyme nickofthyme approved these changes

@darnautov darnautov darnautov approved these changes

@lukasolson lukasolson lukasolson approved these changes

@afharo afharo afharo approved these changes

@sabarasaba sabarasaba sabarasaba approved these changes

@adcoelho adcoelho adcoelho approved these changes

@jbudz jbudz jbudz approved these changes

@hop-dev hop-dev hop-dev approved these changes

@shahzad31 shahzad31 shahzad31 approved these changes

@nreese nreese nreese approved these changes

@patrykkopycinski patrykkopycinski patrykkopycinski approved these changes

@gergoabraham gergoabraham gergoabraham approved these changes

@dhurley14 dhurley14 dhurley14 approved these changes

@maximpn maximpn maximpn approved these changes

@maryam-saeidi maryam-saeidi maryam-saeidi approved these changes

@stratoula stratoula stratoula approved these changes

@michaelolo24 michaelolo24 michaelolo24 approved these changes

@MadameSheema MadameSheema MadameSheema approved these changes

@semd semd semd approved these changes

@kc13greiner kc13greiner kc13greiner approved these changes

@smith smith smith approved these changes

@banderror banderror Awaiting requested review from banderror

+2 more reviewers

@dgieselaar dgieselaar dgieselaar approved these changes

@rshen91 rshen91 rshen91 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

apm:review backport:skip This PR does not require backporting backported chore ci:project-deploy-observability Create an Observability project Feature:ExpressionLanguage Interpreter expression language (aka canvas pipeline) release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. Team:Fleet Team label for Observability Data Collection Fleet team Team:Obs AI Assistant Observability AI Assistant Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. Team:Operations Kibana-Operations Team v8.15.1 v8.16.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.