fix: emit end of file comments (#1491)

Co-authored-by: Cheng Liu <[email protected]>

Author: chengcyber

internal/parser/parser.go

@@ -345,7 +345,6 @@ func (p *Parser) parseSourceFileWorker() *ast.SourceFile {
 	if len(p.reparseList) > 0 {
 		statements = append(statements, p.reparseList...)
 		p.reparseList = nil
-		end = p.nodePos()
 	}
 	node := p.factory.NewSourceFile(p.opts, p.sourceText, p.newNodeList(core.NewTextRange(pos, end), statements), eof)
 	p.finishNode(node, pos)

testdata/baselines/reference/compiler/emitEndOfFileJSDocComments.js

@@ -0,0 +1,13 @@
+//// [tests/cases/compiler/emitEndOfFileJSDocComments.ts] ////
+
+//// [emitEndOfFileJSDocComments.js]
+/** @typedef {number} A */
+var unrelated;
+/** @typedef {number} B */
+
+//// [emitEndOfFileJSDocComments.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+/** @typedef {number} A */
+var unrelated;
+/** @typedef {number} B */ 

testdata/baselines/reference/compiler/emitEndOfFileJSDocComments.symbols

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/emitEndOfFileJSDocComments.ts] ////
+
+=== emitEndOfFileJSDocComments.js ===
+/** @typedef {number} A */
+var unrelated;
+>unrelated : Symbol(unrelated, Decl(emitEndOfFileJSDocComments.js, 1, 3))
+
+/** @typedef {number} B */

testdata/baselines/reference/compiler/emitEndOfFileJSDocComments.types

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/emitEndOfFileJSDocComments.ts] ////
+
+=== emitEndOfFileJSDocComments.js ===
+/** @typedef {number} A */
+var unrelated;
+>unrelated : any
+
+/** @typedef {number} B */

testdata/baselines/reference/compiler/emitEndOfFileJSDocComments2.js

@@ -0,0 +1,42 @@
+//// [tests/cases/compiler/emitEndOfFileJSDocComments2.ts] ////
+
+//// [emitEndOfFileJSDocComments2.js]
+/** @typedef {number} A */
+
+/**
+ * JSDoc comment for function
+ * @param {string} param - A string parameter
+ * @returns {number} The length of the string
+ */
+function test(param) {
+	// Comment inside function
+	return param.length;
+	/** @typedef {number} B2 */
+}
+
+// Single line comment
+/** @typedef {number} C */
+/**
+ * Multiple line comment
+ */
+
+
+//// [emitEndOfFileJSDocComments2.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+/** @typedef {number} A */
+/**
+ * JSDoc comment for function
+ * @param {string} param - A string parameter
+ * @returns {number} The length of the string
+ */
+function test(param) {
+    // Comment inside function
+    return param.length;
+    /** @typedef {number} B2 */
+}
+// Single line comment
+/** @typedef {number} C */
+/**
+ * Multiple line comment
+ */

testdata/baselines/reference/compiler/emitEndOfFileJSDocComments2.symbols

@@ -0,0 +1,29 @@
+//// [tests/cases/compiler/emitEndOfFileJSDocComments2.ts] ////
+
+=== emitEndOfFileJSDocComments2.js ===
+/** @typedef {number} A */
+
+/**
+ * JSDoc comment for function
+ * @param {string} param - A string parameter
+ * @returns {number} The length of the string
+ */
+function test(param) {
+>test : Symbol(test, Decl(emitEndOfFileJSDocComments2.js, 0, 0))
+>param : Symbol(param, Decl(emitEndOfFileJSDocComments2.js, 7, 14))
+
+	// Comment inside function
+	return param.length;
+>param.length : Symbol(length, Decl(lib.es5.d.ts, --, --))
+>param : Symbol(param, Decl(emitEndOfFileJSDocComments2.js, 7, 14))
+>length : Symbol(length, Decl(lib.es5.d.ts, --, --))
+
+	/** @typedef {number} B2 */
+}
+
+// Single line comment
+/** @typedef {number} C */
+/**
+ * Multiple line comment
+ */
+

testdata/baselines/reference/compiler/emitEndOfFileJSDocComments2.types

@@ -0,0 +1,29 @@
+//// [tests/cases/compiler/emitEndOfFileJSDocComments2.ts] ////
+
+=== emitEndOfFileJSDocComments2.js ===
+/** @typedef {number} A */
+
+/**
+ * JSDoc comment for function
+ * @param {string} param - A string parameter
+ * @returns {number} The length of the string
+ */
+function test(param) {
+>test : (param: string) => number
+>param : string
+
+	// Comment inside function
+	return param.length;
+>param.length : number
+>param : string
+>length : number
+
+	/** @typedef {number} B2 */
+}
+
+// Single line comment
+/** @typedef {number} C */
+/**
+ * Multiple line comment
+ */
+

testdata/baselines/reference/conformance/jsDeclarationsDefault2.js

@@ -17,6 +17,9 @@ const _default = class {
 };
 exports._default = _default;
 exports.default = 12;
+/**
+ * @typedef {string | number} default
+ */
 
 
 //// [index1.d.ts]
@@ -26,6 +29,9 @@ export declare const _default: {
 declare const _default_1: number;
 export default _default_1;
 export type default = string | number;
+/**
+ * @typedef {string | number} default
+ */
 
 
 //// [DtsFileErrors]
@@ -58,4 +64,7 @@ out/index1.d.ts(6,32): error TS2693: 'number' only refers to a type, but is bein
 !!! error TS2693: 'string' only refers to a type, but is being used as a value here.
                                    ~~~~~~
 !!! error TS2693: 'number' only refers to a type, but is being used as a value here.
+    /**
+     * @typedef {string | number} default
+     */
 

testdata/baselines/reference/submodule/compiler/reuseTypeAnnotationImportTypeInGlobalThisTypeArgument.js

@@ -35,6 +35,10 @@ export const blah = handleParamGovernance({});
 //// [types.d.ts]
 export {};
 export type ParamStateRecord = Record<Keyword, ParamValueTyped>;
+/**
+ * @typedef {Record<Keyword, ParamValueTyped>} ParamStateRecord a Record containing
+ *   keyword pairs with descriptions of parameters under governance.
+ */
 //// [index.d.ts]
 export declare const blah: {
     publicMixin: {

testdata/baselines/reference/submodule/compiler/reuseTypeAnnotationImportTypeInGlobalThisTypeArgument.js.diff

@@ -4,12 +4,14 @@
 
 
  //// [types.d.ts]
--/**
-- * a Record containing
-- *   keyword pairs with descriptions of parameters under governance.
-- */
 +export {};
- export type ParamStateRecord = Record<Keyword, ParamValueTyped>;
++export type ParamStateRecord = Record<Keyword, ParamValueTyped>;
+ /**
+- * a Record containing
++ * @typedef {Record<Keyword, ParamValueTyped>} ParamStateRecord a Record containing
+  *   keyword pairs with descriptions of parameters under governance.
+  */
+-export type ParamStateRecord = Record<Keyword, ParamValueTyped>;
  //// [index.d.ts]
 -export const blah: {
 +export declare const blah: {