Rename checks

lxsmnsyc · May 29, 2023 · 95681d1 · 95681d1
1 parent e7ceb45
commit 95681d1
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 14 deletions.
diff --git a/packages/forgetti/src/core/checks.ts b/packages/forgetti/src/core/checks.ts
@@ -30,7 +30,7 @@ export function isHookOrComponentName(ctx: StateContext, id: t.Identifier): bool
 
 const FORGETTI_SKIP = /^\s*@forgetti skip\s*$/;
 
-export function isNodeShouldBeSkipped(node: t.Node): boolean {
+export function shouldSkipNode(node: t.Node): boolean {
   // Node without leading comments shouldn't be skipped
   if (node.leadingComments) {
     for (let i = 0, len = node.leadingComments.length; i < len; i++) {
@@ -44,7 +44,7 @@ export function isNodeShouldBeSkipped(node: t.Node): boolean {
 
 const FORGETTI_JSX_SKIP = /^\s*@forgetti jsx\s*$/;
 
-export function isJSXShouldBeSkipped(node: t.Node): boolean {
+export function shouldSkipJSX(node: t.Node): boolean {
   // Node without leading comments shouldn't be skipped
   if (node.leadingComments) {
     for (let i = 0, len = node.leadingComments.length; i < len; i++) {
@@ -65,7 +65,7 @@ export function isComponentValid(
     node.type !== 'ArrowFunctionExpression'
     && !!node.id
     && isHookOrComponentName(ctx, node.id)
-    && !isNodeShouldBeSkipped(node)
+    && !shouldSkipNode(node)
   );
 }
 

diff --git a/packages/forgetti/src/core/is-constant.ts b/packages/forgetti/src/core/is-constant.ts
@@ -2,7 +2,7 @@
 /* eslint-disable @typescript-eslint/no-use-before-define */
 import type * as babel from '@babel/core';
 import * as t from '@babel/types';
-import { isNestedExpression, isNodeShouldBeSkipped, isPathValid } from './checks';
+import { isNestedExpression, shouldSkipNode, isPathValid } from './checks';
 import type OptimizerScope from './optimizer-scope';
 import getForeignBindings, { isForeignBinding } from './get-foreign-bindings';
 import type { ComponentNode, StateContext } from './types';
@@ -343,7 +343,7 @@ function uncachedIsConstant(
   instance: OptimizerInstance,
   path: babel.NodePath<t.Expression>,
 ): boolean {
-  if (isNodeShouldBeSkipped(path.node)) {
+  if (shouldSkipNode(path.node)) {
     return false;
   }
   if (isPathValid(path, isNestedExpression)) {

diff --git a/packages/forgetti/src/core/optimize-jsx.ts b/packages/forgetti/src/core/optimize-jsx.ts
@@ -3,7 +3,7 @@ import * as t from '@babel/types';
 import type { ComponentNode, StateContext } from './types';
 import getImportIdentifier from './get-import-identifier';
 import { RUNTIME_MEMO } from './imports';
-import { isJSXShouldBeSkipped, isPathValid } from './checks';
+import { shouldSkipJSX, isPathValid } from './checks';
 import type { ImportDefinition } from './presets';
 
 interface JSXReplacement {
@@ -169,7 +169,7 @@ function transformJSX(
   path: babel.NodePath<t.JSXElement | t.JSXFragment>,
   memoDefinition: ImportDefinition,
 ): void {
-  if (isJSXShouldBeSkipped(path.node)) {
+  if (shouldSkipJSX(path.node)) {
     return;
   }
   const state: State = {

diff --git a/packages/forgetti/src/core/optimizer.ts b/packages/forgetti/src/core/optimizer.ts
@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/no-unnecessary-condition */
 import type * as babel from '@babel/core';
 import * as t from '@babel/types';
-import { isNestedExpression, isNodeShouldBeSkipped, isPathValid } from './checks';
+import { isNestedExpression, shouldSkipNode, isPathValid } from './checks';
 import getForeignBindings, { isForeignBinding } from './get-foreign-bindings';
 import getImportIdentifier from './get-import-identifier';
 import { RUNTIME_EQUALS } from './imports';
@@ -875,7 +875,7 @@ export default class Optimizer {
   optimizeExpression(
     path: babel.NodePath<t.Expression>,
   ): OptimizedExpression {
-    if (isNodeShouldBeSkipped(path.node)) {
+    if (shouldSkipNode(path.node)) {
       return optimizedExpr(path.node, undefined, true);
     }
     if (isPathValid(path, isNestedExpression)) {
@@ -1006,7 +1006,7 @@ export default class Optimizer {
   private optimizeBlock(
     path: babel.NodePath<t.BlockStatement>,
   ): void {
-    if (isNodeShouldBeSkipped(path.node)) {
+    if (shouldSkipNode(path.node)) {
       return;
     }
     const statements = path.get('body');
@@ -1152,7 +1152,7 @@ export default class Optimizer {
     path: babel.NodePath<t.Statement>,
     topBlock = false,
   ): void {
-    if (isNodeShouldBeSkipped(path.node)) {
+    if (shouldSkipNode(path.node)) {
       return;
     }
     switch (path.type) {

diff --git a/packages/forgetti/src/index.ts b/packages/forgetti/src/index.ts
@@ -6,7 +6,7 @@ import {
   isComponentValid,
   getImportSpecifierName,
   isHookOrComponentName,
-  isNodeShouldBeSkipped,
+  shouldSkipNode,
 } from './core/checks';
 import Optimizer from './core/optimizer';
 import type {
@@ -228,10 +228,10 @@ function transformVariableDeclarator(
     path.node.init
     && path.node.id.type === 'Identifier'
     && isHookOrComponentName(ctx, path.node.id)
-    && !isNodeShouldBeSkipped(path.node)
+    && !shouldSkipNode(path.node)
     && (
       path.parent.type === 'VariableDeclaration'
-        ? !isNodeShouldBeSkipped(path.parent)
+        ? !shouldSkipNode(path.parent)
         : true
     )
   ) {