feat(no-inspect): add no-inspect rule

Author: mikededo

packages/eslint-plugin-svelte/docs/rules/no-inspect.md

@@ -0,0 +1,37 @@
+# (svelte/no-inspect)
+
+> description
+
+## :book: Rule Details
+
+This rule reports ???.
+
+<ESLintCodeBlock>
+
+<!--eslint-skip-->
+
+```svelte
+<script>
+  /* eslint svelte/no-inspect: "error" */
+</script>
+
+<!-- ✓ GOOD -->
+
+<!-- ✗ BAD -->
+```
+
+</ESLintCodeBlock>
+
+## :wrench: Options
+
+```json
+{
+  "svelte/no-inspect": ["error", {}]
+}
+```
+
+-
+
+## :books: Further Reading
+
+-

packages/eslint-plugin-svelte/src/rules/no-inspect.ts

@@ -0,0 +1,30 @@
+import type { TSESTree } from '@typescript-eslint/types';
+
+import { createRule } from '../utils';
+
+export default createRule('no-inspect', {
+	meta: {
+		docs: {
+			description: '',
+			category: 'Best Practices',
+			recommended: true,
+			default: 'warn'
+		},
+		schema: [],
+		messages: {
+			unexpected: 'Do not use $inspect directive'
+		},
+		type: 'suggestion'
+	},
+	create(context) {
+		return {
+			Identifier(node: TSESTree.Identifier) {
+				if (node.name !== '$inspect') {
+					return;
+				}
+
+				context.report({ messageId: 'unexpected', node });
+			}
+		};
+	}
+});

packages/eslint-plugin-svelte/src/utils/rules.ts

@@ -32,6 +32,7 @@ import noIgnoredUnsubscribe from '../rules/no-ignored-unsubscribe';
 import noImmutableReactiveStatements from '../rules/no-immutable-reactive-statements';
 import noInlineStyles from '../rules/no-inline-styles';
 import noInnerDeclarations from '../rules/no-inner-declarations';
+import noInspect from '../rules/no-inspect';
 import noNotFunctionHandler from '../rules/no-not-function-handler';
 import noObjectInTextMustaches from '../rules/no-object-in-text-mustaches';
 import noReactiveFunctions from '../rules/no-reactive-functions';
@@ -97,6 +98,7 @@ export const rules = [
 	noImmutableReactiveStatements,
 	noInlineStyles,
 	noInnerDeclarations,
+	noInspect,
 	noNotFunctionHandler,
 	noObjectInTextMustaches,
 	noReactiveFunctions,

packages/eslint-plugin-svelte/tests/fixtures/rules/no-inspect/invalid/test01-errors.yaml

@@ -0,0 +1,12 @@
+- message: Do not use $inspect directive
+  line: 2
+  column: 3
+  suggestions: null
+- message: Do not use $inspect directive
+  line: 5
+  column: 13
+  suggestions: null
+- message: Do not use $inspect directive
+  line: 8
+  column: 5
+  suggestions: null

packages/eslint-plugin-svelte/tests/fixtures/rules/no-inspect/invalid/test01-input.svelte

@@ -0,0 +1,10 @@
+<script>
+  $inspect(1);
+  $state(0);
+
+  const a = $inspect(1);
+
+  const _ = () => {
+    $inspect(1);
+  }
+</script>

packages/eslint-plugin-svelte/tests/fixtures/rules/no-inspect/valid/test01-input.svelte

@@ -0,0 +1,3 @@
+<script>
+  const _ = $state(1);
+</script>

packages/eslint-plugin-svelte/tests/src/rules/no-inspect.ts

@@ -0,0 +1,12 @@
+import { RuleTester } from '../../utils/eslint-compat';
+import rule from '../../../src/rules/no-inspect';
+import { loadTestCases } from '../../utils/utils';
+
+const tester = new RuleTester({
+	languageOptions: {
+		ecmaVersion: 2020,
+		sourceType: 'module'
+	}
+});
+
+tester.run('no-inspect', rule as any, loadTestCases('no-inspect'));