style(projects): optimize code style

Author: Ohh-889

ErrorBoundary.tsx

@@ -34,7 +34,7 @@ const ErrorPage: FC<Props> = ({ error, resetErrorBoundary }) => {
   // 可以在这里根据不同的业务逻辑处理错误或者上报给日志服务
   const hook = HookSupportChecker()
 
-  console.error(error, 999);
+  console.error(error);
 
   return (
     <div className="size-full min-h-520px flex-col-center gap-16px overflow-hidden">

build/plugins/auto-import.ts

@@ -6,7 +6,7 @@ export function setupAutoImport(viteEnv: Env.ImportMeta) {
   return AutoImport({
     imports: ['react', 'react-router-dom', 'react-i18next', 'ahooks', { from: 'react', imports: ['FC'], type: true }],
     include: [/\.[tj]sx?$/],
-    dirs: ['src/hooks/**','src/components/**'],
+    dirs: ['src/hooks/**', 'src/components/**'],
     dts: 'src/types/auto-imports.d.ts',
     resolvers: [
       IconsResolver({
@@ -20,11 +20,10 @@ export function setupAutoImport(viteEnv: Env.ImportMeta) {
   });
 }
 
-
-function autoImportAntd(componentName:string)
-{
+function autoImportAntd(componentName: string) {
   const pattern = /^A[A-Z]/;
-  if (pattern.test(componentName)) {
-    return { name: componentName.slice(1), from: 'antd' }
+  if (!pattern.test(componentName)) {
+    return { name: componentName.slice(1), from: 'antd' };
   }
+  return null;
 }

build/plugins/router.ts

@@ -9,7 +9,7 @@ export function setupElegantRouter() {
       base: 'src/layouts/base-layout/index.tsx',
       blank: 'src/layouts/blank-layout/index.tsx'
     },
-    log:false,
+    log: false,
     customRoutes: {
       names: [
         'exception_403',

eslint.config.js

@@ -8,12 +8,12 @@ export default defineConfig(
       singleAttributePerLine: true,
       trailingCommas: 'none'
     },
-    ignores: ['src/layouts/modules/global-tab/index.tsx']
+    ignores: ['src/layouts/modules/global-tab/index.tsx', 'ErrorBoundary.tsx']
   },
   {
     rules: {
       'no-underscore-dangle': 'off',
-      'react/jsx-no-undef': [true, { allowGlobals: true }]
+      'react/jsx-no-undef': 'off'
     }
   }
 );

packages/simple-router/src/hooks/useRouter/index.ts

@@ -1,6 +1,6 @@
 import { createContext, useContext } from 'react';
 
-import Router from '../../router'
+import type Router from '../../router';
 
 export const RouterContext = createContext<Router>({} as Router);
 export function useRouter() {

packages/simple-router/src/matcher/index.ts

@@ -64,7 +64,6 @@ class CreateRouterMatcher {
         const parentPath = parent.record.path as string;
         const connectingSlash = parentPath[parentPath.length - 1] === '/' ? '' : '/';
         normalizedRecord.path = parent.record.path + (path && connectingSlash + path);
-
       }
 
       // create the object beforehand, so it can be passed to children
@@ -204,7 +203,7 @@ class CreateRouterMatcher {
    * @returns An array of route names.
    */
   getAllRouteNames() {
-    return Array.from(this.matcherMap.keys())
+    return Array.from(this.matcherMap.keys());
   }
 
   /**
@@ -213,7 +212,6 @@ class CreateRouterMatcher {
    * @param matcher - The matcher object to insert.
    */
   insertMatcher(matcher: RouteRecordRaw) {
-
     if (matcher.record.path === '*') {
       this.matchers.unshift(matcher);
     } else {

packages/simple-router/src/matcher/pathMatcher.ts

@@ -5,7 +5,7 @@ export function createRouteRecordMatcher(record: RouteRecordNormalized, parent?:
     parent,
     // these needs to be populated by the parent
     children: [],
-    name:record.name
+    name: record.name
   };
 
   if (parent) {

packages/simple-router/src/matcher/shared.ts

@@ -41,16 +41,16 @@ export function generatePath(pathTemplate: string, params: { [key: string]: stri
  * @returns the normalized version
  */
 export function normalizeRouteRecord(record: ElegantConstRoute): RouteRecordNormalized {
-
   return {
-    redirect: record.redirect||record.children&&record.children[0].path,
+    redirect: record.redirect || (record.children && record.children[0].path),
     path: record.path || '',
     name: record.name,
     meta: record.meta || {},
-    children: record.children?.map(child => {
-child.redirect=child.redirect||child.children&&child.children[0].path
-      return child
-    }) || [],
+    children:
+      record.children?.map(child => {
+        child.redirect ||= child.children && child.children[0].path;
+        return child;
+      }) || [],
     component: record.component
   };
 }

packages/simple-router/src/matcher/types.ts

@@ -19,5 +19,5 @@ export interface RouteRecordRaw {
   record: RouteRecordNormalized;
   children: RouteRecordRaw[];
   parent: RouteRecordRaw | undefined;
-  name?:string
+  name?: string;
 }

packages/simple-router/src/router.tsx

@@ -5,12 +5,11 @@ import type { ElegantConstRoute } from '@ohh-889/react-auto-route';
 import React from 'react';
 import type { RouteLocationNamedRaw, RouteLocationNormalizedLoaded, RouterOptions } from './types';
 import CreateRouterMatcher from './matcher';
-import type { RouteRecordNormalized } from './matcher/types';
+import type { RouteRecordNormalized, RouteRecordRaw } from './matcher/types';
 import { START_LOCATION_NORMALIZED } from './types';
 import { RouterContext } from './hooks/useRouter';
 import { RouteContext } from './hooks/useRoute';
 import { warn } from './warning';
-import type { RouteRecordRaw } from './matcher/types'
 
 const historyCreatorMap: Record<
   'hash' | 'history' | 'memory',
@@ -90,33 +89,30 @@ class CreateRouter {
     });
 
     // Update react-router's routes
-    this.#changeRoutes()
+    this.#changeRoutes();
   }
 
   #changeRoutes() {
     this.reactRouter._internalSetRoutes([...this.initReactRoutes, ...this.reactRoutes]);
   }
 
   removeRoute(name: string) {
-    const matched = this.matcher.getRecordMatcher(name)
-    if (!matched) return
+    const matched = this.matcher.getRecordMatcher(name);
+    if (!matched) return;
     if (matched.parent) {
+      const parentNames = findParentNames(matched.parent);
+      let routes = this.reactRoutes;
 
-      const parentNames = findParentNames(matched.parent)
-      let routes = this.reactRoutes
-
-      parentNames.forEach(name => {
-        const finalRoute = routes.find(route => route.id === name)
-        if (finalRoute && finalRoute.children) routes = finalRoute.children
-      })
-      removeElement(routes, matched.name)
-
+      parentNames.forEach(parentName => {
+        const finalRoute = routes.find(route => route.id === parentName);
+        if (finalRoute && finalRoute.children) routes = finalRoute.children;
+      });
+      removeElement(routes, matched.name);
     } else {
-      this.reactRoutes = this.reactRoutes.filter(route => route.id !== matched.record.name)
+      this.reactRoutes = this.reactRoutes.filter(route => route.id !== matched.record.name);
     }
-    this.#changeRoutes()
+    this.#changeRoutes();
     this.matcher.removeRoute(name);
-
   }
 
   #onBeforeRouteChange = (
@@ -142,22 +138,20 @@ class CreateRouter {
     if (to.redirect) {
       if (to.redirect.startsWith('/')) {
         if (to.redirect === this.currentRoute.fullPath) {
-          return true
+          return true;
         }
       } else {
-        const finalRoute = to.matched[to.matched.length - 1]
-
+        const finalRoute = to.matched[to.matched.length - 1];
 
-        const finalPath = getFullPath(finalRoute)
+        const finalPath = getFullPath(finalRoute);
 
-        if (finalPath === this.currentRoute.fullPath) return true
+        if (finalPath === this.currentRoute.fullPath) return true;
       }
     }
 
     return beforeEach(to, this.currentRoute, this.#next);
   };
 
-
   #next(param?: boolean | string | Location | RouteLocationNamedRaw) {
     if (!param) return false;
     if (typeof param === 'string') {
@@ -319,26 +313,21 @@ function cleanParams(params: Record<string, any> | undefined): Record<string, an
   return Object.fromEntries(Object.entries(params).filter(([_, value]) => value !== null));
 }
 
-
-
-
-
 function findParentNames(matched: RouteRecordRaw | undefined): (string | undefined)[] {
-  const parentNames: (string | undefined)[] = []
+  const parentNames: (string | undefined)[] = [];
 
   function helper(current: RouteRecordRaw | undefined) {
     if (current?.parent) {
-      helper(current.parent)
+      helper(current.parent);
     }
-    parentNames.push(current?.name)
+    parentNames.push(current?.name);
   }
 
-  helper(matched)
+  helper(matched);
 
-  return parentNames
+  return parentNames;
 }
 
-
 function removeElement(arr: RouteObject[], name: string | undefined) {
   const index = arr.findIndex(route => route.id === name);
   if (index !== -1) {
@@ -347,15 +336,13 @@ function removeElement(arr: RouteObject[], name: string | undefined) {
   return arr;
 }
 
-
-function getFullPath(route: RouteRecordNormalized | ElegantConstRoute ): string {
+function getFullPath(route: RouteRecordNormalized | ElegantConstRoute): string {
   // 如果当前 route 存在并且有 children
   if (route && route.redirect && route.children && route.children.length > 0) {
     // 获取第一个子路由
-    const firstChild = route.children.find(child => child.path === route.redirect)
+    const firstChild = route.children.find(child => child.path === route.redirect);
     // 递归调用，继续拼接子路由的 path
-    if (firstChild)
-      return `${route.path}/${getFullPath(firstChild)}`;
+    if (firstChild) return `${route.path}/${getFullPath(firstChild)}`;
   }
   // 如果没有 children，返回当前 route 的 path
   return route.path;