@lvce-editor/eslint-plugin-rpc 18.12.0 → 18.13.0

package/dist/index.js

@@ -1,11 +1,13 @@
 const rpcRegistryModule = '@lvce-editor/rpc-registry';
-const rendererWorkerName = 'RendererWorker';
+const isWorkerName = name => {
+  return typeof name === 'string' && (/(?:Worker|Process)$/.test(name) || name === 'ExtensionHost' || name === 'ProcessExplorer');
+};
 const meta$1 = {
   docs: {
-    description: 'Disallow destructuring RendererWorker to preserve tree shaking'
+    description: 'Disallow destructuring RPC registry workers to preserve tree shaking'
   },
   messages: {
-    noRendererWorkerDestructuring: 'Do not destructure `RendererWorker`; call its methods directly so they can be tree-shaken.'
+    noRpcRegistryDestructuring: 'Do not destructure RPC registry workers; access their methods directly so they can be tree-shaken.'
   },
   type: 'problem'
 };
@@ -23,7 +25,7 @@ const findVariable = (scope, name) => {
 const isImportFromRpcRegistry = definition => {
   return definition.type === 'ImportBinding' && definition.parent?.source?.value === rpcRegistryModule;
 };
-const isRendererWorkerImport = variable => {
+const isWorkerImport = variable => {
   return variable?.defs.some(definition => {
     if (!isImportFromRpcRegistry(definition) || definition.node?.type !== 'ImportSpecifier') {
       return false;
@@ -31,7 +33,7 @@ const isRendererWorkerImport = variable => {
     const {
       imported
     } = definition.node;
-    return imported?.name === rendererWorkerName || imported?.value === rendererWorkerName;
+    return isWorkerName(imported?.name ?? imported?.value);
   });
 };
 const isRpcRegistryNamespaceImport = variable => {
@@ -45,45 +47,45 @@ const unwrapExpression = node => {
   }
   return node;
 };
-const isRendererWorkerProperty = node => {
+const isWorkerProperty = node => {
   if (node.computed) {
-    return node.property?.type === 'Literal' && node.property.value === rendererWorkerName;
+    return node.property?.type === 'Literal' && isWorkerName(node.property.value);
   }
-  return node.property?.type === 'Identifier' && node.property.name === rendererWorkerName;
+  return node.property?.type === 'Identifier' && isWorkerName(node.property.name);
 };
-const isRendererWorkerExpression = (context, expression, node) => {
+const isWorkerExpression = (context, expression, node) => {
   const unwrappedExpression = unwrapExpression(expression);
   const scope = context.sourceCode.getScope(node);
   if (unwrappedExpression?.type === 'Identifier') {
-    return isRendererWorkerImport(findVariable(scope, unwrappedExpression.name));
+    return isWorkerImport(findVariable(scope, unwrappedExpression.name));
   }
-  if (unwrappedExpression?.type !== 'MemberExpression' || unwrappedExpression.object?.type !== 'Identifier' || !isRendererWorkerProperty(unwrappedExpression)) {
+  if (unwrappedExpression?.type !== 'MemberExpression' || unwrappedExpression.object?.type !== 'Identifier' || !isWorkerProperty(unwrappedExpression)) {
     return false;
   }
   return isRpcRegistryNamespaceImport(findVariable(scope, unwrappedExpression.object.name));
 };
 const reportObjectPattern = (context, pattern) => {
   context.report({
-    messageId: 'noRendererWorkerDestructuring',
+    messageId: 'noRpcRegistryDestructuring',
     node: pattern
   });
 };
 const create$1 = context => {
   return {
     AssignmentExpression(node) {
-      if (node.left.type === 'ObjectPattern' && isRendererWorkerExpression(context, node.right, node)) {
+      if (node.left.type === 'ObjectPattern' && isWorkerExpression(context, node.right, node)) {
         reportObjectPattern(context, node.left);
       }
     },
     VariableDeclarator(node) {
-      if (node.id.type === 'ObjectPattern' && isRendererWorkerExpression(context, node.init, node)) {
+      if (node.id.type === 'ObjectPattern' && isWorkerExpression(context, node.init, node)) {
         reportObjectPattern(context, node.id);
       }
     }
   };
 };
 
-const noRendererWorkerDestructuring = {
+const noRpcRegistryDestructuring = {
   __proto__: null,
   create: create$1,
   meta: meta$1
@@ -137,7 +139,8 @@ const plugin = {
     version: '0.0.1'
   },
   rules: {
-    'no-renderer-worker-destructuring': noRendererWorkerDestructuring,
+    'no-renderer-worker-destructuring': noRpcRegistryDestructuring,
+    'no-rpc-registry-destructuring': noRpcRegistryDestructuring,
     'prefer-using-mock-rpc': preferUsingMockRpc
   }
 };
@@ -147,7 +150,7 @@ const recommended = [{
     rpc: plugin
   },
   rules: {
-    'rpc/no-renderer-worker-destructuring': 'error',
+    'rpc/no-rpc-registry-destructuring': 'error',
     'rpc/prefer-using-mock-rpc': 'error'
   }
 }];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/eslint-plugin-rpc",
-  "version": "18.12.0",
+  "version": "18.13.0",
   "main": "dist/index.js",
   "repository": {
     "type": "git",