fix: permit appropriate computed member expressions and prototype access

Author: brettz9

__tests__/spec-only.js

@@ -9,7 +9,10 @@ ruleTester.run('spec-only', rule, {
     'Promise.resolve()',
     'Promise.reject()',
     'Promise.all()',
+    'Promise["all"]',
+    'Promise[method];',
     'Promise.race()',
+    'var ctch = Promise.prototype.catch',
     'Promise.withResolvers()',
     'new Promise(function (resolve, reject) {})',
     'SomeClass.resolve()',
@@ -53,5 +56,13 @@ ruleTester.run('spec-only', rule, {
       `,
       errors: [{ message: "Avoid using non-standard 'Promise.done'" }],
     },
+    {
+      code: `var done = Promise.prototype.done`,
+      errors: [{ message: "Avoid using non-standard 'Promise.prototype'" }],
+    },
+    {
+      code: `Promise["done"];`,
+      errors: [{ message: "Avoid using non-standard 'Promise.done'" }],
+    },
   ],
 })

rules/lib/is-promise.js

@@ -29,7 +29,7 @@ function isPromise(expression) {
       expression.callee.type === 'MemberExpression' &&
       expression.callee.object.type === 'Identifier' &&
       expression.callee.object.name === 'Promise' &&
-      PROMISE_STATICS[expression.callee.property.name] &&
+      PROMISE_STATICS.has(expression.callee.property.name) &&
       expression.callee.property.name !== 'withResolvers')
   )
 }

rules/lib/promise-statics.js

@@ -1,11 +1,11 @@
 'use strict'
 
-module.exports = {
-  all: true,
-  allSettled: true,
-  any: true,
-  race: true,
-  reject: true,
-  resolve: true,
-  withResolvers: true,
-}
+module.exports = new Set([
+  'all',
+  'allSettled',
+  'any',
+  'race',
+  'reject',
+  'resolve',
+  'withResolvers',
+])

rules/no-new-statics.js

@@ -22,7 +22,7 @@ module.exports = {
         if (
           node.callee.type === 'MemberExpression' &&
           node.callee.object.name === 'Promise' &&
-          PROMISE_STATICS[node.callee.property.name]
+          PROMISE_STATICS.has(node.callee.property.name)
         ) {
           context.report({
             node,

rules/spec-only.js

@@ -3,6 +3,8 @@
 const PROMISE_STATICS = require('./lib/promise-statics')
 const getDocsUrl = require('./lib/get-docs-url')
 
+const PROMISE_INSTANCE_METHODS = new Set(['then', 'catch', 'finally'])
+
 module.exports = {
   meta: {
     type: 'problem',
@@ -35,14 +37,19 @@ module.exports = {
       MemberExpression(node) {
         if (
           node.object.type === 'Identifier' &&
+          (!node.computed || node.property.type === 'Literal') &&
           node.object.name === 'Promise' &&
-          !(node.property.name in PROMISE_STATICS) &&
+          ((node.property.name && !PROMISE_STATICS.has(node.property.name)) ||
+            (node.property.value &&
+              !PROMISE_STATICS.has(node.property.value))) &&
+          (node.property.name !== 'prototype' ||
+            !PROMISE_INSTANCE_METHODS.has(node?.parent?.property?.name)) &&
           !allowedMethods.includes(node.property.name)
         ) {
           context.report({
             node,
             messageId: 'avoidNonStandard',
-            data: { name: node.property.name },
+            data: { name: node.property.name ?? node.property.value },
           })
         }
       },