Skip to content

Commit aa8f2a7

Browse files
authored
fix: better debug rules message formatting (#1062)
1 parent 432b0c6 commit aa8f2a7

File tree

13 files changed

+196
-182
lines changed

13 files changed

+196
-182
lines changed

examples/vite-react-dom-app/eslint.config.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@ export default tseslint.config(
5858
},
5959
rules: {
6060
...eslintPluginReactHooks.configs.recommended.rules,
61+
"@eslint-react/debug/jsx": "warn",
6162
},
6263
},
6364
);

packages/plugins/eslint-plugin-react-debug/src/rules/class-component.spec.ts

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import tsx from "dedent";
22

33
import { allValid, ruleTester } from "../../../../../test";
4+
import { stringify } from "../utils";
45
import rule, { RULE_NAME } from "./class-component";
56

67
ruleTester.run(RULE_NAME, rule, {
@@ -16,7 +17,7 @@ ruleTester.run(RULE_NAME, rule, {
1617
errors: [
1718
{
1819
messageId: "classComponent",
19-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
20+
data: { json: stringify({ name: "ParentComponent" }) },
2021
},
2122
],
2223
},
@@ -31,7 +32,7 @@ ruleTester.run(RULE_NAME, rule, {
3132
errors: [
3233
{
3334
messageId: "classComponent",
34-
data: { json: JSON.stringify({ name: "ClassComponent" }) },
35+
data: { json: stringify({ name: "ClassComponent" }) },
3536
},
3637
],
3738
},
@@ -54,7 +55,7 @@ ruleTester.run(RULE_NAME, rule, {
5455
errors: [
5556
{
5657
messageId: "classComponent",
57-
data: { json: JSON.stringify({ name: "UnstableNestedClassComponent" }) },
58+
data: { json: stringify({ name: "UnstableNestedClassComponent" }) },
5859
},
5960
],
6061
},
@@ -79,11 +80,11 @@ ruleTester.run(RULE_NAME, rule, {
7980
errors: [
8081
{
8182
messageId: "classComponent",
82-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
83+
data: { json: stringify({ name: "ParentComponent" }) },
8384
},
8485
{
8586
messageId: "classComponent",
86-
data: { json: JSON.stringify({ name: "UnstableNestedClassComponent" }) },
87+
data: { json: stringify({ name: "UnstableNestedClassComponent" }) },
8788
},
8889
],
8990
},
@@ -108,11 +109,11 @@ ruleTester.run(RULE_NAME, rule, {
108109
errors: [
109110
{
110111
messageId: "classComponent",
111-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
112+
data: { json: stringify({ name: "ParentComponent" }) },
112113
},
113114
{
114115
messageId: "classComponent",
115-
data: { json: JSON.stringify({ name: "UnstableNestedClassComponent" }) },
116+
data: { json: stringify({ name: "UnstableNestedClassComponent" }) },
116117
},
117118
],
118119
},
@@ -135,7 +136,7 @@ ruleTester.run(RULE_NAME, rule, {
135136
errors: [
136137
{
137138
messageId: "classComponent",
138-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
139+
data: { json: stringify({ name: "ParentComponent" }) },
139140
},
140141
],
141142
},
@@ -158,7 +159,7 @@ ruleTester.run(RULE_NAME, rule, {
158159
errors: [
159160
{
160161
messageId: "classComponent",
161-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
162+
data: { json: stringify({ name: "ParentComponent" }) },
162163
},
163164
],
164165
},
@@ -181,7 +182,7 @@ ruleTester.run(RULE_NAME, rule, {
181182
errors: [
182183
{
183184
messageId: "classComponent",
184-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
185+
data: { json: stringify({ name: "ParentComponent" }) },
185186
},
186187
],
187188
},
@@ -204,7 +205,7 @@ ruleTester.run(RULE_NAME, rule, {
204205
errors: [
205206
{
206207
messageId: "classComponent",
207-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
208+
data: { json: stringify({ name: "ParentComponent" }) },
208209
},
209210
],
210211
},
@@ -223,7 +224,7 @@ ruleTester.run(RULE_NAME, rule, {
223224
errors: [
224225
{
225226
messageId: "classComponent",
226-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
227+
data: { json: stringify({ name: "ParentComponent" }) },
227228
},
228229
],
229230
},
@@ -249,7 +250,7 @@ ruleTester.run(RULE_NAME, rule, {
249250
errors: [
250251
{
251252
messageId: "classComponent",
252-
data: { json: JSON.stringify({ name: "ParentComponent" }) },
253+
data: { json: stringify({ name: "ParentComponent" }) },
253254
},
254255
],
255256
},

packages/plugins/eslint-plugin-react-debug/src/rules/class-component.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import type { RuleListener } from "@typescript-eslint/utils/ts-eslint";
33
import type { CamelCase } from "string-ts";
44
import * as ER from "@eslint-react/core";
55

6-
import { createRule } from "../utils";
6+
import { createRule, stringify } from "../utils";
77

88
export const RULE_NAME = "class-component";
99

@@ -41,7 +41,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
4141
messageId: "classComponent",
4242
node: component,
4343
data: {
44-
json: JSON.stringify({ name }),
44+
json: stringify({ name }),
4545
},
4646
});
4747
}

0 commit comments

Comments
 (0)