dsherret / ts-morph

TypeScript Compiler API wrapper for static analysis and programmatic code changes.
https://ts-morph.com
MIT License
4.9k stars 194 forks source link

Single-line comment with AddPropertyAssignment caused an extra comma #1570

Open linzebingo opened 3 weeks ago

linzebingo commented 3 weeks ago

Describe the bug

Version: 23.0.0

Single-line comment with AddPropertyAssignment caused an extra comma

To Reproduce

const { Project } = require('ts-morph')

const sourceCode = `
export default {
  slave: {},
  // mako: {}
}
`
function main() {
  const project = new Project()
  const sourceFile = project.createSourceFile('index.ts', sourceCode)

  const defaultExportSymbol = sourceFile.getDefaultExportSymbol()

  if (defaultExportSymbol) {
    const declaration = defaultExportSymbol.getDeclarations()[0]
    const objectLiteralExpression = declaration.getExpression()
    if (objectLiteralExpression) {
      const publicPathProp = objectLiteralExpression.getProperty('publicPath')

      if (publicPathProp)
        publicPathProp.remove()

      objectLiteralExpression.addPropertyAssignment({
        name: 'publicPath',
        initializer: `"/"`,
      })
    }
  }

  console.log(sourceFile.getText())
}

main()

Expected behavior

Expect:

 export default {
    slave: {},
    // mako: {}
      publicPath: "/"
  }

Actual:

 export default {
    slave: {},,
    // mako: {}
     publicPath: "/"
  }