Skip to content

[JS] Remove the last remaining legacy backend code#6186

Draft
Sergej Jaskiewicz (broadwaylamb) wants to merge 2 commits into
masterfrom
rr/broadwaylamb/js-delete-mode-dead-code
Draft

[JS] Remove the last remaining legacy backend code#6186
Sergej Jaskiewicz (broadwaylamb) wants to merge 2 commits into
masterfrom
rr/broadwaylamb/js-delete-mode-dead-code

Conversation

@broadwaylamb

Copy link
Copy Markdown
Contributor

RIP

@kotlin-safemerge

kotlin-safemerge Bot commented Jun 9, 2026

Copy link
Copy Markdown

Code Owners

RuleOwnersApproval
**.​gradle.​kts, /​repo/​@sellmair
kotlin-build-infrastructure

UNASSIGNED
/​analysis/​, /​compiler/​psi/​
kotlin-analysis-api

UNASSIGNED
/​build-​common/​
build-tools-api
kotlin-build-tools

UNASSIGNED
/​compiler/​daemon/​
build-tools-api

UNASSIGNED
/​compiler/​test-​infrastructure-​utils/​
kotlin-frontend

UNASSIGNED
*
kotlin

UNASSIGNED
/​js/​
kotlin-js

UNASSIGNED

@kodee-bot

kodee-bot Bot commented Jun 9, 2026

Copy link
Copy Markdown

This merge request potentially modifies the stubs format, but doesn't update KotlinStubVersions.kt.
Please verify whether a stub version bump is needed in compiler/psi/psi-api/src/org/jetbrains/kotlin/psi/stubs/KotlinStubVersions.kt.
If you are unsure, please consult with the Analysis API team (#kotlin-analysis-api in Slack).

@broadwaylamb Sergej Jaskiewicz (broadwaylamb) force-pushed the rr/broadwaylamb/js-delete-mode-dead-code branch 2 times, most recently from b476411 to 9ec7a98 Compare June 9, 2026 13:09
It was an artifact from the pre-IR backend era that's been dead for
a while.

^KT-73067 Fixed
It only contained the logic for legacy JS metadata serialization,
which we haven't been using for quite a while.

JS AST protobuf was unused too. It used to be used in JS incremental
compilation, but we migrated off it long ago (see `JsIrAstSerializer`)

^KT-70223 Fixed
@broadwaylamb Sergej Jaskiewicz (broadwaylamb) force-pushed the rr/broadwaylamb/js-delete-mode-dead-code branch from 9ec7a98 to 9605685 Compare June 9, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant