@arcgis/eslint-config 4.34.0-next.4 → 4.34.0-next.40

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -22,8 +22,7 @@ const defaultConfig = [
22
22
  languageOptions: {
23
23
  globals: {
24
24
  ...globals.browser,
25
- ...globals.node,
26
- ...globals.jest
25
+ ...globals.node
27
26
  },
28
27
  ecmaVersion: "latest",
29
28
  sourceType: "module"
@@ -539,7 +538,8 @@ const defaultConfig = [
539
538
  "@typescript-eslint/no-unsafe-enum-comparison": "off",
540
539
  // Developer may actually mean to use ||
541
540
  // Also, refactoring this out may break things
542
- "@typescript-eslint/prefer-nullish-coalescing": "off"
541
+ "@typescript-eslint/prefer-nullish-coalescing": "off",
542
+ "@typescript-eslint/no-unnecessary-type-conversion": "warn"
543
543
  //#endregion
544
544
  // TODO: go over https://typescript-eslint.io/linting/troubleshooting/performance-troubleshooting
545
545
  }
@@ -126,7 +126,7 @@ declare global {
126
126
  {
127
127
  group: ["/@lit/context"],
128
128
  importNames: ["ContextProvider", "ContextConsumer", "provide", "consume"],
129
- message: `For lazy-loading compatibility, import the useContextProvider() and the useContextConsumer() controllers from @arcgis/lumina rather than directly calling the Lit's controllers/decorators.`,
129
+ message: `For lazy-loading compatibility, import the useContextProvider() and the useContextConsumer() controllers from @arcgis/lumina/context rather than directly calling the Lit's controllers/decorators. See https://qawebgis.esri.com/components/lumina/state-management#lit-context`,
130
130
  caseSensitive: true
131
131
  },
132
132
  {
@@ -1,5 +1,5 @@
1
1
  import { ESLintUtils } from "@typescript-eslint/utils";
2
- const version = "4.34.0-next.4";
2
+ const version = "4.34.0-next.40";
3
3
  function makeEslintPlugin(pluginName, urlCreator) {
4
4
  const rules = [];
5
5
  const creator = ESLintUtils.RuleCreator(urlCreator);
@@ -1,4 +1,4 @@
1
- import { m as makeEslintPlugin } from "../../makePlugin-Ca089k86.js";
1
+ import { m as makeEslintPlugin } from "../../makePlugin-BWc0voYM.js";
2
2
  import { AST_NODE_TYPES, ESLintUtils, AST_TOKEN_TYPES } from "@typescript-eslint/utils";
3
3
  import ts from "typescript";
4
4
  import { camelToKebab } from "@arcgis/components-utils";
@@ -1277,7 +1277,7 @@ plugin.createRule({
1277
1277
  return;
1278
1278
  }
1279
1279
  if (isCreateElementComponent(node)) {
1280
- return void context.report({
1280
+ return context.report({
1281
1281
  node,
1282
1282
  messageId: "default"
1283
1283
  });
@@ -1,4 +1,4 @@
1
- import { m as makeEslintPlugin } from "../../makePlugin-Ca089k86.js";
1
+ import { m as makeEslintPlugin } from "../../makePlugin-BWc0voYM.js";
2
2
  import { resolve } from "node:path/posix";
3
3
  import { AST_NODE_TYPES } from "@typescript-eslint/utils";
4
4
  const plugin = makeEslintPlugin(
@@ -94,9 +94,10 @@ plugin.createRule({
94
94
  }
95
95
  });
96
96
  const reTouchingJsDoc = /\*\/\s+\/\*\*/gu;
97
- const description$1 = `@arcgis/core imports need to end with .js for better compatibility with @arcgis/core ESM CDN builds.`;
97
+ const description$1 = `@arcgis/core imports need to end with .js for better compatibility with ESM CDN builds for @arcgis/core and other packages.`;
98
+ const packagesToEnforce = ["@arcgis/core/", "@amcharts/amcharts4/", "@amcharts/amcharts5/"];
98
99
  plugin.createRule({
99
- name: "require-js-in-core-import",
100
+ name: "require-js-in-imports",
100
101
  meta: {
101
102
  docs: {
102
103
  description: description$1,
@@ -119,7 +120,7 @@ plugin.createRule({
119
120
  return;
120
121
  }
121
122
  const specifier = node.source.value;
122
- if (typeof specifier !== "string" || !specifier.startsWith("@arcgis/core/") || // Already ends with .js or .json
123
+ if (typeof specifier !== "string" || !packagesToEnforce.some((pkg) => specifier.startsWith(pkg)) || // Already ends with .js or .json
123
124
  specifier.includes(".")) {
124
125
  return;
125
126
  }
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@arcgis/eslint-config",
3
- "version": "4.34.0-next.4",
3
+ "version": "4.34.0-next.40",
4
4
  "description": "ESLint configuration for arcgis-web-components",
5
5
  "type": "module",
6
6
  "main": "index.js",
@@ -20,17 +20,17 @@
20
20
  ],
21
21
  "license": "SEE LICENSE IN LICENSE.md",
22
22
  "dependencies": {
23
- "@arcgis/components-utils": "4.34.0-next.4",
24
- "@eslint/js": "^9.23.0",
23
+ "@arcgis/components-utils": "4.34.0-next.40",
24
+ "@eslint/js": "^9.29.0",
25
25
  "@types/confusing-browser-globals": "^1.0.3",
26
26
  "confusing-browser-globals": "^1.0.11",
27
27
  "eslint-plugin-storybook": "^0.12.0",
28
28
  "globals": "^15.12.0",
29
29
  "tslib": "^2.8.1",
30
30
  "typescript": "~5.8.3",
31
- "typescript-eslint": "^8.28.0"
31
+ "typescript-eslint": "^8.35.0"
32
32
  },
33
33
  "peerDependencies": {
34
- "eslint": "^9.23.0"
34
+ "eslint": "^9.29.0"
35
35
  }
36
36
  }