chore: remove isString, isNumber

.eslintrc.json

@@ -22,6 +22,22 @@
       "rules": {
         "react/jsx-key": "off"
       }
+    },
+    {
+      "files": ["**/*.{js,jsx,ts,tsx}"],
+      "rules": {
+        "no-restricted-syntax": [
+          "error",
+          {
+            "selector": "CallExpression[callee.name='isNumber']",
+            "message": "use native typeof === 'number' check"
+          },
+          {
+            "selector": "CallExpression[callee.name='isString']",
+            "message": "use native typeof === 'string' check"
+          }
+        ]
+      }
     }
   ],
   "parser": "@typescript-eslint/parser",

src/handler/expand.ts

@@ -10,7 +10,7 @@ import cssColorParse from 'parse-css-color'
 
 import CssDimension from '../vendor/parse-css-dimension/index.js'
 import parseTransformOrigin from '../transform-origin.js'
-import { isString, lengthToNumber, v } from '../utils.js'
+import { lengthToNumber, v } from '../utils.js'
 
 // https://react-cn.github.io/react/tips/style-props-value-px.html
 const optOutPx = new Set([
@@ -221,6 +221,7 @@ function getErrorHint(name: string) {
 }
 
 const RGB_SLASH = /rgb\((\d+)\s+(\d+)\s+(\d+)\s*\/\s*([\.\d]+)\)/
+
 function normalizeColor(value: string | object) {
   if (typeof value === 'string') {
     if (RGB_SLASH.test(value.trim())) {
@@ -429,7 +430,7 @@ function preprocess(
   value: string | number,
   currentColor: string
 ): string | number {
-  if (isString(value)) {
+  if (typeof value === 'string') {
     value = convertCurrentColorToActualValue(value, currentColor)
   }
 

src/text.ts

@@ -12,9 +12,7 @@ import {
   buildXMLString,
   splitByBreakOpportunities,
   isUndefined,
-  isString,
   lengthToNumber,
-  isNumber,
 } from './utils.js'
 import buildText, { container } from './builder/text.js'
 import { buildDropShadow } from './builder/shadow.js'
@@ -146,9 +144,10 @@ export default async function* buildTextNodes(
     return measureGraphemeArray(segment(text, 'grapheme'))
   }
 
-  const tabWidth = isString(tabSize)
-    ? lengthToNumber(tabSize, fontSize as number, 1, parentStyle)
-    : measureGrapheme(Space) * (tabSize as number)
+  const tabWidth =
+    typeof tabSize === 'string'
+      ? lengthToNumber(tabSize, fontSize as number, 1, parentStyle)
+      : measureGrapheme(Space) * (tabSize as number)
 
   const calc = (
     text: string,
@@ -846,7 +845,7 @@ function processTextOverflow(
     textOverflow === 'ellipsis' &&
     display === '-webkit-box' &&
     WebkitBoxOrient === 'vertical' &&
-    isNumber(WebkitLineClamp) &&
+    typeof WebkitLineClamp === 'number' &&
     WebkitLineClamp > 0
   ) {
     return [WebkitLineClamp, HorizontalEllipsis]

src/utils.ts

@@ -262,14 +262,6 @@ export function toString(x: unknown): string {
   return Object.prototype.toString.call(x)
 }
 
-export function isString(x: unknown): x is string {
-  return typeof x === 'string'
-}
-
-export function isNumber(x: unknown): x is number {
-  return typeof x === 'number'
-}
-
 export function isUndefined(x: unknown): x is undefined {
   return toString(x) === '[object Undefined]'
 }