fix: Pretty-printer: keep the comment after opening an object scope indented

Author: prantlf

lib/printer.js

@@ -126,8 +126,10 @@
           }
 
           var token = skipWhitespace()
-          if (token && token.type === 'comment') {
-            if (foundLineBreak || needsLineBreak) {
+          // If line break followed the previous token, leave the comment
+          // to be handled by the next usual token processing.
+          if (!foundLineBreak && token && token.type === 'comment') {
+            if (needsLineBreak) {
               // If the previous non-whitespace token was ended by a line
               // break, retain it. Print the comment after the line break too.
               if (!addedLineBreak) {

test/print.js

@@ -115,4 +115,30 @@ addTest('keep comment locations', function () {
   // // test`
 })
 
+addTest('keep comment after opening an object scope indented', function () {
+  // `{
+  // // String parameter
+  // "key": 'value',
+  // }`
+  var tokens = [
+    { type: 'symbol', raw: '{', value: '{' },
+    { type: 'whitespace', raw: '\n' },
+    { type: 'comment', raw: '// String parameter' },
+    { type: 'whitespace', raw: '\n' },
+    { type: 'literal', raw: '"key"', value: 'key' },
+    { type: 'symbol', raw: ':', value: ':' },
+    { type: 'whitespace', raw: ' ' },
+    { type: 'literal', raw: '\'value\'', value: 'value' },
+    { type: 'symbol', raw: ',', value: ',' },
+    { type: 'whitespace', raw: '\n' },
+    { type: 'symbol', raw: '}', value: '}' }
+  ]
+  var output = print(tokens, { indent: '  ' })
+  assert.equal(output, '{\n  // String parameter\n  "key": \'value\',\n  \n}')
+  // `{
+  // // String parameter
+  // "key": 'value',
+  // }`
+})
+
 if (require.main === module) { require('test').run(exports) }