fix: fix sonarqube

Author: wkylin

src/mock/permission.ts

@@ -128,24 +128,20 @@ export const mockRoles: Role[] = [
 /**
  * 固定的测试账号（四个角色对应四个账号）
  */
-export const testAccounts: Record<string, { password: string; role: string; name: string }> = {
+export const testAccounts: Record<string, { role: string; name: string }> = {
   '[email protected]': {
-    password: '123456',
     role: 'super_admin',
     name: '超级管理员',
   },
   '[email protected]': {
-    password: '123456',
     role: 'admin',
     name: '管理员',
   },
   '[email protected]': {
-    password: '123456',
     role: 'business_user',
     name: '业务员',
   },
   '[email protected]': {
-    password: '123456',
     role: 'user',
     name: '普通用户',
   },
@@ -304,47 +300,45 @@ const safeJsonParse = <T>(text: string | null): T | null => {
   }
 }
 
+const safeGetStorageItem = (key: string): string | null => {
+  try {
+    return globalThis?.localStorage?.getItem(key) ?? null
+  } catch {
+    return null
+  }
+}
+
 const tryGetManualRolePermission = (): UserPermission | null => {
-  const storedRoleCode = localStorage.getItem('user_role')
-  console.log('📝 手动设置的角色:', storedRoleCode)
+  const storedRoleCode = safeGetStorageItem('user_role')
   if (storedRoleCode && mockUserPermissions[storedRoleCode]) {
-    console.log('✅ 使用手动设置的角色:', storedRoleCode)
     return { ...mockUserPermissions[storedRoleCode] }
   }
   return null
 }
 
 const tryGetGithubUserPermission = (): UserPermission | null => {
-  const githubUser = localStorage.getItem('github_user')
+  const githubUser = safeGetStorageItem('github_user')
   const user = safeJsonParse<{ email?: string }>(githubUser)
   if (!user) return null
 
-  console.log('🔍 检测到 GitHub 用户:', user)
-  if (user.email === '[email protected]' || user.email) {
-    console.log('✅ GitHub 用户登录，授予超级管理员权限')
+  if (user.email) {
     return { ...mockUserPermissions['super_admin'] }
   }
   return null
 }
 
 const tryGetTokenRolePermission = (): UserPermission | null => {
-  const tokenData = localStorage.getItem('token')
-  console.log('🎫 Token 数据:', tokenData)
+  const tokenData = safeGetStorageItem('token')
   if (!tokenData) return null
 
   const tokenObj = safeJsonParse<{ token?: string }>(tokenData)
   const email = tokenObj?.token ?? tokenData
-  console.log('📧 解析出的邮箱:', email)
-  console.log('🔎 查找账号:', email, '在', Object.keys(testAccounts))
 
   const account = testAccounts[email]
   if (!account) {
-    console.log('❌ 未找到匹配的测试账号')
     return null
   }
 
-  console.log('✅ 找到账号，角色:', account.role)
-  console.log('📋 返回权限数据:', mockUserPermissions[account.role])
   return { ...mockUserPermissions[account.role] }
 }
 
@@ -354,8 +348,6 @@ const tryGetTokenRolePermission = (): UserPermission | null => {
 export const mockGetUserPermissions = async (_userId?: string, _roleCode?: string): Promise<UserPermission> => {
   await new Promise((resolve) => setTimeout(resolve, 100))
 
-  console.log('🔍 开始获取用户权限...')
-
   // 1. 优先使用手动设置的角色（用于测试切换）
   const manual = tryGetManualRolePermission()
   if (manual) return manual
@@ -369,7 +361,6 @@ export const mockGetUserPermissions = async (_userId?: string, _roleCode?: strin
   if (token) return token
 
   // 4. 默认返回普通用户权限
-  console.log('⚠️ 使用默认权限（普通用户）')
   return { ...mockUserPermissions['user'] }
 }
 

src/pages/demo/index.jsx

@@ -1,5 +1,4 @@
-import React, { useState, useEffect } from 'react'
-import { Table } from 'antd'
+import React from 'react'
 import { Smartphone, Monitor, Rocket } from 'lucide-react'
 import PageContainer from '@stateless/PageContainer'
 import FixTabPanel from '@stateless/FixTabPanel'
@@ -28,7 +27,6 @@ import HePng from '@assets/images/he.png'
 import SongPng from '@assets/images/song.png'
 import XuePng from '@assets/images/xue.png'
 import OneTimePasscode from '@stateless/OneTimePasscode'
-import styles from './index.module.less'
 
 const companies = [SpringPng, HePng, SongPng, XuePng]
 
@@ -85,68 +83,20 @@ const fixTabsData = [
   },
 ]
 
-const tabs = [
-  {
-    title: 'Product',
-    value: 'product',
-    content: (
-      <div className="relative h-full w-full overflow-hidden rounded-2xl bg-linear-to-br from-purple-700 to-violet-900 p-10 text-xl font-bold text-white md:text-4xl">
-        <p>Product Tab</p>
-      </div>
-    ),
-  },
-  {
-    title: 'Services',
-    value: 'services',
-    content: (
-      <div className="relative h-full w-full overflow-hidden rounded-2xl bg-linear-to-br from-purple-700 to-violet-900 p-10 text-xl font-bold text-white md:text-4xl">
-        <p>Services tab</p>
-      </div>
-    ),
-  },
-  {
-    title: 'Playground',
-    value: 'playground',
-    content: (
-      <div className="relative h-full w-full overflow-hidden rounded-2xl bg-linear-to-br from-purple-700 to-violet-900 p-10 text-xl font-bold text-white md:text-4xl">
-        <p>Playground tab</p>
-      </div>
-    ),
-  },
-  {
-    title: 'Content',
-    value: 'content',
-    content: (
-      <div className="relative h-full w-full overflow-hidden rounded-2xl bg-linear-to-br from-purple-700 to-violet-900 p-10 text-xl font-bold text-white md:text-4xl">
-        <p>Content tab</p>
-      </div>
-    ),
-  },
-  {
-    title: 'Random',
-    value: 'random',
-    content: (
-      <div className="relative h-full w-full overflow-hidden rounded-2xl bg-linear-to-br from-purple-700 to-violet-900 p-10 text-xl font-bold text-white md:text-4xl">
-        <p>Random tab</p>
-      </div>
-    ),
-  },
-]
-
 const customCommandMap = {
   npm: 'npm run shadcn add button',
   yarn: 'yarn shadcn add button',
   pnpm: 'pnpm dlx shadcn@latest add button',
   bun: 'bun x shadcn@latest add button',
 }
 
-const dateDifference = (date1, date2) => {
-  const d1 = new Date(date1)
-  const d2 = new Date(date2)
-  const diffTime = Math.abs(d2 - d1)
-  const diffDays = Math.ceil(diffTime / (1000 * 60 * 60 * 24))
-  return diffDays
-}
+const animatedListItems = Array.from({ length: 10 }, (_, index) => ({
+  id: `animated-${index + 1}`,
+}))
+
+const stickyCardItems = Array.from({ length: 4 }, (_, index) => ({
+  id: `sticky-${index + 1}`,
+}))
 
 const tagCardList = [
   {
@@ -315,7 +265,7 @@ const ProDemo = () => {
   return (
     <FixTabPanel>
       <PageContainer>
-        <OneTimePasscode length={6} variant="compact" onComplete={(code) => console.log('OTP Code:', code)} />
+        <OneTimePasscode length={6} variant="compact" onComplete={() => {}} />
         <MusicPlayer />
         <TransferHistory />
         <FixCarousel />
@@ -338,17 +288,13 @@ const ProDemo = () => {
         <LogoSlider companies={companies} />
         <section style={{ height: 240, overflow: 'hidden', margin: 20 }}>
           <AnimatedList>
-            {Array.from({ length: 10 }, () => ({
-              id: Math.random(),
-            }))
-              .flat()
-              .map((item, index) => (
-                <div key={item?.id} className="flex flex-col items-center justify-center gap-4">
-                  <div className="flex items-center justify-center gap-4">
-                    <div className="h-16 w-100 rounded-full bg-gradient-to-br from-purple-500 to-blue-500" />
-                  </div>
+            {animatedListItems.map((item) => (
+              <div key={item.id} className="flex flex-col items-center justify-center gap-4">
+                <div className="flex items-center justify-center gap-4">
+                  <div className="h-16 w-100 rounded-full bg-gradient-to-br from-purple-500 to-blue-500" />
                 </div>
-              ))}
+              </div>
+            ))}
           </AnimatedList>
         </section>
 
@@ -374,7 +320,7 @@ const ProDemo = () => {
           ]}
           className="my-4 rounded-xl"
         />
-        <StickyCard cards={[...Array.from({ length: 4 }, () => ({ id: Math.random() }))]} />
+        <StickyCard cards={stickyCardItems} />
         <section style={{ marginBottom: '15px' }}>
           <OrgChart />
         </section>

src/pages/layout/primaryNav/index.jsx

@@ -1,4 +1,5 @@
 import React from 'react'
+import PropTypes from 'prop-types'
 import { Button, Space } from 'antd'
 import { DashboardOutlined, ProjectOutlined, HomeOutlined } from '@ant-design/icons'
 import useSafeNavigate from '@app-hooks/useSafeNavigate'
@@ -8,15 +9,46 @@ export const usePrimaryNavItems = () => {
   const { redirectTo } = useSafeNavigate()
   const { routes } = usePermission()
 
+  const matchWildcard = (text, pattern) => {
+    if (pattern === '*') return true
+    if (!pattern.includes('*')) return text === pattern
+
+    const parts = pattern.split('*').filter((p) => p.length > 0)
+    if (parts.length === 0) return true
+
+    let pos = 0
+    const startsWithWildcard = pattern.startsWith('*')
+    const endsWithWildcard = pattern.endsWith('*')
+
+    if (!startsWithWildcard) {
+      const first = parts[0]
+      if (!text.startsWith(first)) return false
+      pos = first.length
+    }
+
+    for (let i = startsWithWildcard ? 0 : 1; i < parts.length; i += 1) {
+      const part = parts[i]
+      const idx = text.indexOf(part, pos)
+      if (idx === -1) return false
+      pos = idx + part.length
+    }
+
+    if (!endsWithWildcard) {
+      const last = parts[parts.length - 1]
+      return text.endsWith(last)
+    }
+
+    return true
+  }
+
   const hasAccess = (path) => {
     if (!routes || routes.length === 0) return false
     if (path === '/') return true
     return routes.some((route) => {
       if (route === path) return true
       if (path.startsWith(route + '/')) return true
       if (route.includes('*')) {
-        const pattern = route.replace('*', '.*')
-        return new RegExp(`^${pattern}$`).test(path)
+        return matchWildcard(path, route)
       }
       return false
     })
@@ -63,4 +95,8 @@ const PrimaryNav = ({ layout = '' }) => {
   )
 }
 
+PrimaryNav.propTypes = {
+  layout: PropTypes.string,
+}
+
 export default PrimaryNav

src/pages/layout/proContent/breadcrumb/util.js

@@ -46,8 +46,8 @@ export const findRouteChain = (routes, pathname) => {
       if (route.key === pathToCheck || route.path === pathToCheck) return true
 
       // 动态路由匹配
-      if (route.key && route.key.includes(':')) {
-        const pattern = route.key.replace(/:[^/]+/g, '[^/]+')
+      if (route?.key?.includes(':')) {
+        const pattern = route?.key.replace(/:[^/]+/g, '[^/]+')
         const regex = new RegExp(`^${pattern}$`)
         return regex.test(pathToCheck)
       }

src/pages/layout/proContent/index.jsx

@@ -1,10 +1,7 @@
-import React, { useState, useEffect, useRef } from 'react'
-import { Layout, FloatButton, theme, Space } from 'antd'
+import React, { useState, useEffect } from 'react'
+import { Layout, theme, Space } from 'antd'
 import { useLocation } from 'react-router-dom'
-import { VerticalAlignTopOutlined } from '@ant-design/icons'
 import { getKeyName } from '@utils/publicFn'
-import { useProTabContext } from '@app-hooks/proTabsContext'
-import ClockFace from '@stateless/ClockFace'
 import ProBreadcrumb from './breadcrumb'
 import ProTabs from '../proTabs'
 import styles from './index.module.less'
@@ -13,7 +10,6 @@ const { Content, Header, Footer } = Layout
 
 const ProContent = () => {
   const [tabActiveKey, setTabActiveKey] = useState('home')
-  const { activeKey, panes } = useProTabContext()
   const [panesItem, setPanesItem] = useState({
     title: '',
     content: null,
@@ -22,8 +18,6 @@ const ProContent = () => {
     path: '',
     i18nKey: '',
   })
-
-  const pathRef = useRef('')
   const { pathname, search } = useLocation()
   const {
     token: { colorBgContainer, colorBgLayout },
@@ -32,19 +26,16 @@ const ProContent = () => {
     // pass full path (including search) so getKeyName can consider query params
     const full = search ? pathname + search : pathname
     const { tabKey, title, element, i18nKey } = getKeyName(full)
-    const newPath = full
-    pathRef.current = newPath
 
     setPanesItem({
       title,
       content: element,
       key: tabKey,
       closable: tabKey !== '/',
-      path: newPath,
+      path: full,
       i18nKey,
     })
     setTabActiveKey(tabKey)
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [pathname, search])
 
   return (