reviewer comments

.eslintrc.js

@@ -18,12 +18,6 @@ module.exports = {
         allowSingleLine: true,
       },
     ],
-    camelcase: [
-      'error',
-      {
-        properties: 'never',
-      },
-    ],
     'capitalized-comments': 'off',
     'class-methods-use-this': 'off',
     'comma-dangle': 'off',
@@ -267,7 +261,18 @@ module.exports = {
         '@typescript-eslint/triple-slash-reference': 0,
         '@typescript-eslint/ban-ts-comment': 'off',
         '@typescript-eslint/no-empty-function': 0,
-        '@typescript-eslint/camelcase': 0,
+        '@typescript-eslint/naming-convention': [
+          'error',
+          {
+            selector: 'default',
+            format: ['camelCase', 'UPPER_CASE', 'PascalCase'],
+            leadingUnderscore: 'allow',
+          },
+          {
+            selector: 'property',
+            format: null,
+          },
+        ],
         '@typescript-eslint/no-explicit-any': 0,
         '@typescript-eslint/explicit-function-return-type': 'error',
         '@typescript-eslint/no-var-requires': 0,

src/ResourceNamespace.ts

@@ -16,7 +16,7 @@ function ResourceNamespace(
 
 export function resourceNamespace(
   namespace: string,
-  resources: Record<string, any>
+  resources: Record<string, (...args: any[]) => void>
 ): (stripe: StripeObject) => StripeResourceNamespaceObject {
   return function(stripe: StripeObject): StripeResourceNamespaceObject {
     return new (ResourceNamespace as any)(stripe, resources);

src/Types.d.ts

@@ -240,14 +240,3 @@ type UserProvidedConfig = {
   telemetry?: boolean;
   appInfo?: AppInfo;
 };
-type OAuthAuthorizeUrlParams = {
-  response_type?: 'code';
-  client_id?: string;
-  scope?: string;
-};
-type OAuthAuthorizeUrlOptions = {
-  express?: boolean;
-};
-type OAuthDeauthorizeParams = {
-  client_id?: string;
-};

src/resources/Accounts.ts

@@ -12,7 +12,7 @@ export const Accounts = StripeResource.extend({
     fullPath: '/v1/accounts',
   }),
 
-  retrieve(id: string | null | undefined, ...args: any[]) {
+  retrieve(id: string, ...args: any[]) {
     // No longer allow an api key to be passed as the first string to this function due to ambiguity between
     // old account ids and api keys. To request the account for an api key, send null as the id
     if (typeof id === 'string') {

src/resources/OAuth.ts

@@ -3,6 +3,18 @@
 import {StripeResource} from '../StripeResource';
 import {stringifyRequestData} from '../utils';
 
+type OAuthAuthorizeUrlParams = {
+  response_type?: 'code';
+  client_id?: string;
+  scope?: string;
+};
+type OAuthAuthorizeUrlOptions = {
+  express?: boolean;
+};
+type OAuthDeauthorizeParams = {
+  client_id?: string;
+};
+
 const stripeMethod = StripeResource.method;
 
 const oAuthHost = 'connect.stripe.com';