Merge pull request #195 from JetBrains/dx-improve

DX: Check prettier by default linter, fix code by pre-commit hook, fix release files

Author: zoobestik

.eslintignore

@@ -1,2 +1,8 @@
-node_modules
-dist
+/out
+/build
+/.idea
+/node_modules
+/dist
+/test-results/
+/playwright-report/
+/playwright/

.eslintrc.js

@@ -6,7 +6,7 @@ module.exports = {
   extends: [
     'eslint:recommended',
     'plugin:@typescript-eslint/recommended',
-    'prettier',
+    'plugin:prettier/recommended',
   ],
   parser: '@typescript-eslint/parser',
   parserOptions: {
@@ -19,5 +19,10 @@ module.exports = {
     sourceType: 'module',
   },
   plugins: ['@typescript-eslint', 'eslint-plugin-import'],
-  rules: {},
+  rules: {
+    quotes: ['error', 'single', {
+      'avoidEscape': true,
+      'allowTemplateLiterals': false
+    }]
+  }
 };

.husky/pre-commit

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+npx lint-staged

.lintstagedrc.json

@@ -0,0 +1,5 @@
+{
+  "*.ts": [
+    "npm run fix"
+  ]
+}

.nvmrc

@@ -0,0 +1 @@
+18

.prettierignore

@@ -1,6 +1,8 @@
-# Ignore everything recursively
-*
-# But not the .ts files
-!*.ts
-# Check subdirectories too
-!*/
+/out
+/build
+/.idea
+/node_modules
+/dist
+/test-results/
+/playwright-report/
+/playwright/

package.json

@@ -11,7 +11,8 @@
   "repository": "JetBrains/kotlin-playground",
   "main": "dist/playground.min.js",
   "files": [
-    "dist"
+    "dist",
+    "!dist/REMOVE_ME.js*"
   ],
   "devDependencies": {
     "@babel/cli": "^7.21.5",
@@ -36,13 +37,16 @@
     "eslint": "^8.47.0",
     "eslint-config-prettier": "^9.0.0",
     "eslint-plugin-import": "^2.28.0",
+    "eslint-plugin-prettier": "^5.0.1",
     "fast-async": "7",
     "fast-deep-equal": "^3.1.3",
     "file-loader": "^6.2.0",
     "flatten": "^1.0.2",
     "github-markdown-css": "^3.0.1",
     "html-webpack-plugin": "^5.5.3",
+    "husky": "^8.0.3",
     "is-empty-object": "^1.1.1",
+    "lint-staged": "^15.0.2",
     "lz-string": "^1.4.4",
     "markdown-it": "^12.3.2",
     "markdown-it-highlightjs": "^3.0.0",
@@ -53,7 +57,7 @@
     "monkberry-loader": "4.0.9",
     "postcss": "^8.4.31",
     "postcss-loader": "^7.3.3",
-    "prettier": "^3.0.2",
+    "prettier": "^3.0.3",
     "query-string": "^6.5.0",
     "sass": "^1.66.0",
     "sass-loader": "^13.3.2",
@@ -77,9 +81,10 @@
     "release:ci": "rm -rf dist && npm run build:all && $NPM_TOKEN=%env.NPM_TOKEN% npm publish",
     "start": "webpack-dev-server --port 9002",
     "lint": "eslint . --ext .ts",
-    "prettier": "prettier --write .",
+    "fix": "eslint --fix --ext .ts .",
     "test": "npm run build:all && npm run test:run",
     "test:run": "playwright test",
-    "test:server": "node tests/utlis/server/index.js"
+    "test:server": "node tests/utlis/server/index.js",
+    "prepare": "husky install"
   }
 }

playwright.config.ts

@@ -3,7 +3,7 @@ import { config as dotenv } from 'dotenv';
 import { defineConfig, devices } from '@playwright/test';
 import { isKeyOfObject } from './src/utils/types';
 
-dotenv({ path: `.env.local`, override: true });
+dotenv({ path: '.env.local', override: true });
 
 const PROJECTS_LIST = {
   DEV: ['Desktop Chrome'],

tests/crosslink.test.ts

@@ -60,11 +60,12 @@ ${'      '}
     });
 
     //language=text
-    const codeWithMark = `fun containsEven(collection: Collection<Int>): Boolean = collection.any {[mark]TODO()[/mark]}`;
+    const codeWithMark =
+      'fun containsEven(collection: Collection<Int>): Boolean = collection.any {[mark]TODO()[/mark]}';
 
     checkLink(generateCrosslink(codeWithMark), {
       //language=kotlin
-      code: `fun containsEven(collection: Collection<Int>): Boolean = collection.any {TODO()}`,
+      code: 'fun containsEven(collection: Collection<Int>): Boolean = collection.any {TODO()}',
     });
   });