@nextcloud/eslint-config 8.2.0 → 8.3.0-beta.0

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [v8.3.0-beta.0](https://github.com/nextcloud/eslint-config/tree/v8.3.0-beta.0) (2023-05-12)
+
+[Full Changelog](https://github.com/nextcloud/eslint-config/compare/v8.2.1...v8.3.0-beta.0)
+
+**Merged pull requests:**
+- Typescript 5 support
+- Various dependencies upgrades
+
+## [v8.2.1](https://github.com/nextcloud/eslint-config/tree/v8.2.1) (2023-01-30)
+
+[Full Changelog](https://github.com/nextcloud/eslint-config/compare/v8.2.0...v8.2.1)
+
+**Fixed:**
+- Fix resolving packages that use the package `exports` feature by @susnux in https://github.com/nextcloud/eslint-config/pull/452
+
+**Merged pull requests:**
+- Various dependencies upgrades
+
 ## [v8.2.0](https://github.com/nextcloud/eslint-config/tree/v8.2.0) (2023-01-11)
 
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v8.1.5...v8.2.0)
@@ -24,8 +42,7 @@
 
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v8.1.3...v8.1.4)
 
-**Fixed bugs:**
-
+**Fixed:**
 - Fix eslint-plugin-n conflict [\#421](https://github.com/nextcloud/eslint-config/pull/421) ([skjnldsv](https://github.com/skjnldsv))
 
 ## [v8.1.3](https://github.com/nextcloud/eslint-config/tree/v8.1.3) (2022-11-04)
@@ -49,7 +66,6 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v8.0.0...v8.1.2)
 
 **Dependency updates:**
-
 - Use [`eslint-plugin-n`](https://www.npmjs.com/package/eslint-plugin-n) instead of [`eslint-plugin-node`](https://www.npmjs.com/package/eslint-plugin-node)
 
 ## [v8.0.0](https://github.com/nextcloud/eslint-config/tree/v8.0.0) (2022-04-20)
@@ -57,11 +73,9 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v7.0.2...v8.0.0)
 
 **Breaking:**
-
 - Require file extensions on import [\#292](https://github.com/nextcloud/eslint-config/pull/292) ([Pytal](https://github.com/Pytal))
 
 **Dependency updates:**
-
 - Bump eslint from 8.7.0 to 8.8.0 [\#282](https://github.com/nextcloud/eslint-config/pull/282) ([dependabot[bot]](https://github.com/apps/dependabot))
 - Bump eslint from 8.8.0 to 8.9.0 [\#296](https://github.com/nextcloud/eslint-config/pull/296) ([dependabot[bot]](https://github.com/apps/dependabot))
 
@@ -70,11 +84,9 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v7.0.1...v7.0.2)
 
 **Fixed:**
-
 - Fix usual same-line vue attribute behaviour [\#272](https://github.com/nextcloud/eslint-config/pull/272) ([skjnldsv](https://github.com/skjnldsv))
 
 **Dependency updates:**
-
 - Bump eslint from 8.6.0 to 8.7.0 [\#271](https://github.com/nextcloud/eslint-config/pull/271) ([dependabot[bot]](https://github.com/apps/dependabot))
 - Bump @babel/core from 7.16.7 to 7.16.10 [\#273](https://github.com/nextcloud/eslint-config/pull/273) ([dependabot[bot]](https://github.com/apps/dependabot))
 
@@ -83,11 +95,9 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v7.0.0...v7.0.1)
 
 **Fixed bugs:**
-
 - Allow component names to be single-word [\#270](https://github.com/nextcloud/eslint-config/pull/270) ([skjnldsv](https://github.com/skjnldsv))
 
 **Dependency updates:**
-
 - Bump webpack from 5.65.0 to 5.66.0 [\#269](https://github.com/nextcloud/eslint-config/pull/269) ([dependabot[bot]](https://github.com/apps/dependabot))
 - Bump eslint-plugin-vue from 8.2.0 to 8.3.0 [\#268](https://github.com/nextcloud/eslint-config/pull/268) ([dependabot[bot]](https://github.com/apps/dependabot))
 
@@ -96,7 +106,6 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v6.1.2...v7.0.0)
 
 **Merged pull requests:**
-
 - Eslint 8 [\#266](https://github.com/nextcloud/eslint-config/pull/266) ([skjnldsv](https://github.com/skjnldsv))
 - v7.0.0 [\#267](https://github.com/nextcloud/eslint-config/pull/267) ([skjnldsv](https://github.com/skjnldsv))
 ## [v6.1.2](https://github.com/nextcloud/eslint-config/tree/v6.1.2) (2021-12-20)
@@ -104,7 +113,6 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v6.1.1...v6.1.2)
 
 **Fixes:**
-
 - Fix vue custom event name casing [\#249](https://github.com/nextcloud/eslint-config/pull/249) ([Pytal](https://github.com/Pytal))
 
 ## [v6.1.1](https://github.com/nextcloud/eslint-config/tree/v6.1.1) (2021-12-02)
@@ -112,7 +120,6 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v6.1.0...v6.1.1)
 
 **Fixes:**
-
 - Disable tag rule [\#248](https://github.com/nextcloud/eslint-config/pull/248) ([skjnldsv](https://github.com/skjnldsv))
 
 ## [v6.1.0](https://github.com/nextcloud/eslint-config/tree/v6.1.0) (2021-08-11)
@@ -120,7 +127,6 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v6.0.1...v6.1.0)
 
 **Fix dependency:**
-
 - Remove undesired eslint-webpack-plugin ([skjnldsv](https://github.com/skjnldsv))
 
 ## [v6.0.1](https://github.com/nextcloud/eslint-config/tree/v6.0.1) (2021-08-11)
@@ -128,7 +134,6 @@
 [Full Changelog](https://github.com/nextcloud/eslint-config/compare/v6.0.0...v6.0.1)
 
 **Merged pull requests:**
-
 - Fix license header tags spacing [\#180](https://github.com/nextcloud/eslint-config/pull/180) ([skjnldsv](https://github.com/skjnldsv))
 
 ## [v6.0.0](https://github.com/nextcloud/eslint-config/tree/v6.0.0)

package/README.md

@@ -1,7 +1,7 @@
 ## @nextcloud/eslint-config
 
 [![npm last version](https://img.shields.io/npm/v/@nextcloud/eslint-config.svg?style=flat-square)](https://www.npmjs.com/package/@nextcloud/eslint-config)
-[![Lint status](https://img.shields.io/github/actions/workflow/status/nextcloud/eslint-config/lint.yml?label=lint&style=flat-square)](https://github.com/nextcloud/eslint-config/actions/workflows/lint.yml)
+[![Lint status](https://img.shields.io/github/actions/workflow/status/nextcloud/eslint-config/lint-eslint.yml?label=lint&style=flat-square)](https://github.com/nextcloud/eslint-config/actions/workflows/lint.yml)
 [![Dependabot status](https://img.shields.io/badge/Dependabot-enabled-brightgreen.svg?longCache=true&style=flat-square&logo=dependabot)](https://dependabot.com)
 
 

package/index.js

@@ -29,6 +29,9 @@ module.exports = {
 				paths: ['src'],
 				extensions: ['.js', '.vue'],
 			},
+			exports: {
+				conditions: ['import'],
+			},
 		},
 		jsdoc: {
 			tagNamePreference: {
@@ -118,7 +121,7 @@ module.exports = {
 	},
 	overrides: [
 		{
-			files: ['**/*.ts'],
+			files: ['**/*.ts', '**/*.tsx'],
 			extends: [
 				'@vue/eslint-config-typescript/recommended',
 				'plugin:import/typescript',
@@ -145,7 +148,7 @@ module.exports = {
 				'import/resolver': {
 					node: {
 						paths: ['src'],
-						extensions: ['.js', '.ts', '.vue'],
+						extensions: ['.(m|c)?js', '.ts', '.tsx', '.vue'],
 					},
 				},
 			},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nextcloud/eslint-config",
-  "version": "8.2.0",
+  "version": "8.3.0-beta.0",
   "description": "Eslint shared config for nextcloud vue.js apps",
   "main": "index.js",
   "repository": {
@@ -9,7 +9,8 @@
   },
   "scripts": {
     "lint": "eslint -c index.js *.js",
-    "lint:fix": "eslint -c index.js --fix *.js"
+    "lint:fix": "eslint -c index.js --fix *.js",
+    "test": "jest"
   },
   "peerDependencies": {
     "@babel/core": "^7.13.10",
@@ -18,27 +19,32 @@
     "@vue/eslint-config-typescript": "^11.0.2",
     "eslint": "^8.27.0",
     "eslint-config-standard": "^17.0.0",
+    "eslint-import-resolver-exports": "^1.0.0-beta.4",
     "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-jsdoc": "^39.6.2",
+    "eslint-plugin-jsdoc": "^43.0.3",
     "eslint-plugin-n": "^15.5.1",
     "eslint-plugin-promise": "^6.1.1",
     "eslint-plugin-vue": "^9.7.0",
-    "typescript": "^4.9.4",
+    "typescript": "^5.0.2",
     "webpack": "^5.4.0"
   },
   "devDependencies": {
     "@babel/core": "^7.13.10",
     "@babel/eslint-parser": "^7.16.5",
     "@nextcloud/eslint-plugin": "^2.0.0",
+    "@types/jest": "^29.4.0",
     "@vue/eslint-config-typescript": "^11.0.2",
     "eslint": "^8.27.0",
     "eslint-config-standard": "^17.0.0",
+    "eslint-import-resolver-exports": "^1.0.0-beta.4",
     "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-jsdoc": "^39.6.2",
+    "eslint-plugin-jsdoc": "^43.0.3",
     "eslint-plugin-n": "^15.5.1",
     "eslint-plugin-promise": "^6.1.1",
     "eslint-plugin-vue": "^9.7.0",
-    "typescript": "^4.9.4",
+    "jest": "^29.4.1",
+    "ts-jest": "^29.0.5",
+    "typescript": "^5.0.2",
     "webpack": "^5.4.0"
   },
   "keywords": [
@@ -57,5 +63,12 @@
   "engines": {
     "node": "^16.0.0",
     "npm": "^7.0.0 || ^8.0.0"
+  },
+  "jest": {
+    "preset": "ts-jest",
+    "testEnvironment": "node",
+    "setupFilesAfterEnv": [
+      "./tests/setup-jest.ts"
+    ]
   }
 }

package/tests/eslint-config.test.ts

@@ -0,0 +1,31 @@
+import { ESLint } from "eslint"
+import type { Linter } from "eslint"
+import * as path from 'path'
+import * as eslintConfig from '../index.js'
+
+
+const eslint = new ESLint({
+	baseConfig: eslintConfig as unknown as Linter.Config<Linter.RulesRecord>
+})
+
+const lintFile = async (file) => {
+	const real = path.resolve(path.join(__dirname, file))
+	return await eslint.lintFiles(real)
+}
+
+test('some basic issues should fail', async () => {
+	const results = await lintFile('fixtures/example-fail.js')
+	expect(results).toHaveIssueCount(3)
+	expect(results).toHaveIssue('spaced-comment')
+	expect(results).toHaveIssue({ ruleId: 'no-console', line: 3 })
+})
+
+test('TSX is linted', async () => {
+	const ignored = await eslint.isPathIgnored('./fixtures/some.tsx')
+	expect(ignored).toBe(false)
+
+	const results = await lintFile('fixtures/some.tsx')
+	expect(results).toHaveIssue({ruleId: 'jsdoc/check-tag-names', line: 5})
+	expect(results).toHaveIssue({ruleId: '@typescript-eslint/no-unused-vars', line: 7})
+	expect(results).toHaveIssueCount(2)
+})

package/tests/eslint.d.ts

@@ -0,0 +1,11 @@
+export { }
+
+declare global {
+	namespace jest {
+		interface Matchers<R> {
+			toPass(): R;
+			toHaveIssueCount(n: number): R;
+			toHaveIssue(issue: string | {ruleId: string, line?: number}): R;
+		}
+	}
+}

package/tests/fixtures/example-fail.js

@@ -0,0 +1,3 @@
+//should fail because of missing space on this comment
+// and usage of `console` and using a semicolon
+console.log('some');

package/tests/fixtures/some.tsx

@@ -0,0 +1,7 @@
+// TSX
+
+/**
+ * @notExported <- should work
+ * @aaabbbcccddd <- should fail
+ */
+const foo = '' // <- should fail, as unused

package/tests/setup-jest.ts

@@ -0,0 +1,128 @@
+import { ESLint, Linter } from "eslint"
+
+/**
+ * Add some custom matchers for ESLint to jest
+ */
+expect.extend({
+	toPass: assertLintingPassed,
+	toHaveIssueCount: assertHavingNIssues,
+	toHaveIssue: assertHavingIssue
+})
+
+/**
+ * Check if linting a file did not throw any errors or warnings
+ *
+ * @param received Lint result
+ */
+function hasNoIssues(received: ESLint.LintResult) {
+	// dirty type check
+	if (received?.errorCount === undefined) throw new Error('Expected ESLintResult')
+
+	return received.errorCount === 0 && received.warningCount === 0
+}
+
+/**
+ * Check if linting of multiple fils
+ *
+ * @param received 
+ * @returns {}
+ */
+function assertLintingPassed(received: ESLint.LintResult | ESLint.LintResult[]) {
+	// allow single ESLintResult
+	if (!Array.isArray(received)) {
+		received = [received]
+	}
+
+	const errors = [] as {file: string, errors: Linter.LintMessage[]}[]
+	const pass = received.every((result) => {
+		// save issues
+		errors.push({file: result.filePath, errors: result.messages})
+		return hasNoIssues(result)
+	})
+
+	return {
+		pass,
+		message: () => {
+			if (pass) {
+				return 'Expected file to not pass eslint, but got no issues'
+			} else {
+				const errorMessages = errors.map((m) => 
+					`file: ${m.file}\n` + m.errors.map((e) => 'line: ' + e.line + ': ' + (e.ruleId || e.message))
+				)
+				return 'Expected file to pass eslint, got issues:\n' + errorMessages.join('\n')
+			}
+		}
+	}
+}
+
+/**
+ * Count the total amount of issues
+ *
+ * @param received lint result
+ * @returns total amount of issues
+ */
+function countIssues(received: ESLint.LintResult) {
+	return received.errorCount + received.warningCount
+}
+
+/**
+ * Check if exactly the same number of issues is reported
+ *
+ * @param received the lint result
+ * @param expected number of expected issues
+ * @returns jest matcher result
+ */
+function assertHavingNIssues(received: ESLint.LintResult | ESLint.LintResult[], expected: number) {
+	if (!(typeof expected === 'number')) throw new Error('Expected a number as expected value')
+
+	if (!Array.isArray(received)) {
+		received = [received]
+	}
+
+	const issues = received.map(countIssues).reduce((p, c) => p + c)
+	const pass = issues === expected
+
+	return {
+		pass,
+		message: () => pass ? `Expected not to find exactly ${expected} issues.` : `Expected ${expected} issues, found ${issues}`
+	}
+}
+
+/**
+ * Check if result contains the expected issue
+ *
+ * @param received the lint result
+ * @param issue the expected issue
+ * @returns jest matcher result
+ */
+function assertHavingIssue(received: ESLint.LintResult | ESLint.LintResult[], issue: string | {ruleId: string, line?: number}) {
+	if (!Array.isArray(received)) {
+		received = [received]
+	}
+
+	// Should not pass
+	if (assertLintingPassed(received).pass) {
+		return {
+			pass: false,
+			message: () => 'Expected issue, but no linting issues found.'
+		}
+	}
+
+	const name = typeof issue === 'string' ? issue : issue.ruleId
+	const result = received.some((result) => {
+		return result.messages.some((message) => {
+			// ensure name matches
+			if (message.ruleId !== name) return false
+			// if line is requested ignore not matching ones
+			if (typeof issue === 'object' && issue.line !== undefined && issue.line !== message.line) return false
+			// otherwise matched	
+			return true
+		})
+	});
+
+	const onLine = typeof issue === 'string' ? '' : ` on line ${issue.line}`
+	return {
+		pass: result,
+		message: () => result ? `Unexpected error '${name}'${onLine} found.`  : `Expected error '${name}'${onLine} not found.`
+	}
+}