fix: make /standalone entry work with require() (#2247)

* chore: 🤖 build cjs output for backward compatibility

This commit adds cjs output to rollup config.

* chore: 🤖 add mapping for standalone.cjs

* chore: 🤖 include standalone.cjs as package source

* chore: 🤖 migrate .js files to .mjs

* chore: 🤖 disable ESM as default

* refactor: 💡 delete unnecessary export

* chore: 🤖 revert build config

* chore: 🤖 add require to exports

Without this require, vscode-prettier can't resolve php plugin

* refactor: 💡 fix link in comments

* refactor: 💡 match both .js and .mjs for jest target

* Remove `require`

* Add a comment

---------

Co-authored-by: fisker Cheung <lionkay@gmail.com>

Author: shufo

.eslintrc.yml

@@ -47,9 +47,9 @@ overrides:
   - files: "**/*.cjs"
     parserOptions:
       sourceType: "script"
-  - files: "tests/**/*.js"
+  - files: "tests/**/*.mjs"
     globals:
       run_spec: true
-  - files: "build/*.js"
+  - files: "build/*.mjs"
     parserOptions:
       sourceType: module

build/rollup.config.js build/rollup.config.mjsbuild/rollup.config.js renamed to build/rollup.config.mjs

@@ -12,7 +12,7 @@ import terser from "@rollup/plugin-terser";
 const getPath = (file) => url.fileURLToPath(new URL(file, import.meta.url));
 
 export default {
-  input: getPath("../src/index.js"),
+  input: getPath("../src/index.mjs"),
   output: {
     file: "standalone.js",
     format: "umd",
@@ -32,10 +32,10 @@ export default {
     }),
     commonjs(),
     alias({
-      entries: [{ find: "assert", replacement: getPath("./shims/assert.js") }],
+      entries: [{ find: "assert", replacement: getPath("./shims/assert.mjs") }],
     }),
     inject({
-      Buffer: getPath("./shims/buffer.js"),
+      Buffer: getPath("./shims/buffer.mjs"),
     }),
     replace({
       preventAssignment: true,

build/shims/assert.js build/shims/assert.mjsbuild/shims/assert.js renamed to build/shims/assert.mjs

@@ -6,14 +6,15 @@ const RUN_STANDALONE_TESTS = Boolean(process.env.RUN_STANDALONE_TESTS);
 export default {
   // collectCoverage: ENABLE_COVERAGE,
   collectCoverageFrom: [
-    "<rootDir>/src/**/*.js",
+    "<rootDir>/src/**/*.mjs",
     "!<rootDir>/node_modules/",
     "!<rootDir>/tests_config/",
   ],
   runner: "jest-light-runner",
   transform: {},
-  setupFiles: ["<rootDir>/tests_config/run_spec.js"],
-  testRegex: "jsfmt\\.spec\\.js$|__tests__/.*\\.js$",
+  setupFiles: ["<rootDir>/tests_config/run_spec.mjs"],
+  // Matches `.js` file to prevent files use `.js` extension by mistake, https://github.com/prettier/plugin-php/pull/2247#discussion_r1331847801
+  testRegex: "jsfmt\\.spec\\.m?js$|__tests__/.*\\.m?js$",
   snapshotSerializers: ["jest-snapshot-serializer-raw"],
   globals: {
     STANDALONE: RUN_STANDALONE_TESTS,

build/shims/buffer.js build/shims/buffer.mjsbuild/shims/buffer.js renamed to build/shims/buffer.mjs

@@ -5,7 +5,6 @@
   "repository": "prettier/prettier-php",
   "author": "Lucas Azzola <@azz>",
   "license": "MIT",
-  "type": "module",
   "unpkg": "./standalone.js",
   "browser": "./standalone.js",
   "exports": {
@@ -65,8 +64,8 @@
     "test:node": "jest",
     "test:standalone": "yarn run build && cross-env RUN_STANDALONE_TESTS=true yarn jest",
     "prepublishOnly": "yarn test",
-    "prettier": "prettier --plugin=src/index.js --parser=php",
-    "build": "rollup --config build/rollup.config.js",
+    "prettier": "prettier --plugin=src/index.mjs --parser=php",
+    "build": "rollup --config build/rollup.config.mjs",
     "debug": "node --inspect-brk node_modules/.bin/jest --runInBand"
   },
   "c8": {

jest.config.js jest.config.mjsjest.config.js renamed to jest.config.mjs

@@ -1,4 +1,4 @@
-import { printNumber, normalizeMagicMethodName } from "./util.js";
+import { printNumber, normalizeMagicMethodName } from "./util.mjs";
 
 const ignoredProperties = new Set([
   "loc",

package.json

@@ -1,6 +1,6 @@
 import { util as prettierUtil, doc } from "prettier";
-import { isLookupNode } from "./util.js";
-import { locStart, locEnd } from "./loc.js";
+import { isLookupNode } from "./util.mjs";
+import { locStart, locEnd } from "./loc.mjs";
 
 const {
   addLeadingComment,
@@ -830,7 +830,7 @@ function printComment(path, options) {
   return options.printer.printComment(path, options);
 }
 
-// https://github.com/prettier/prettier/blob/master/src/main/comments.js#L440
+// https://github.com/prettier/prettier/blob/6bff776efd0951b7812818d02d1cae3fda184737/src/main/comments/print.js#L119
 function printDanglingComments(path, options, sameIndent, filter) {
   const parts = [];
   const node = path.getValue();

src/clean.js src/clean.mjssrc/clean.js renamed to src/clean.mjs

@@ -3,20 +3,20 @@ import {
   LINGUIST_LANGUAGES_PHP,
   LINGUIST_LANGUAGES_HTML_PHP,
 } from "./linguist-languages.cjs";
-import parse from "./parser.js";
-import print from "./printer.js";
-import clean from "./clean.js";
-import options from "./options.js";
+import parse from "./parser.mjs";
+import print from "./printer.mjs";
+import clean from "./clean.mjs";
+import options from "./options.mjs";
 import {
   handleOwnLineComment,
   handleEndOfLineComment,
   handleRemainingComment,
   getCommentChildNodes,
   canAttachComment,
   isBlockComment,
-} from "./comments.js";
-import { hasPragma, insertPragma } from "./pragma.js";
-import { locStart, locEnd } from "./loc.js";
+} from "./comments.mjs";
+import { hasPragma, insertPragma } from "./pragma.mjs";
+import { locStart, locEnd } from "./loc.mjs";
 
 const { join, hardline } = doc.builders;