terser/terser

Strip the final semi-colon?

Open

#1 119 ouverte le 27 nov. 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (8 215 stars) (386 forks)batch import
good first issue

Description

Bug report or Feature request?

terser should not insert ; at the end

Version (complete output of terser -V or specific git commit)

terser 5.10.0

Complete CLI command or minify() options used

The playground default config

{
  module: true,
  compress: {},
  mangle: {},
  output: {},
  parse: {},
  rename: {},
}

terser input

export default {}

terser output or error

export default{};

Expected result

export default{}

Guide contributeur

Stack technique
javascriptnodejs
Domaine
tooling
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75
Direction de recherche
The issue is located in terser's output generation code, likely in `lib/output.js` or similar. The output adds a semicolon after `export default{}` because the code generator treats it as a statement and appends a semicolon. However, for module exports, a trailing semicolon is unnecessary and may cause issues. Investigate the `output` module to see how semicolons are added after statements, and modify the logic to skip the semicolon when the statement is an `export default` and it's the last statement in the module. Also check if there is an existing option to control trailing semicolons.