Add detailed landing page development plan in docs/frontend_landing_plan.md: - Complete landing page structure (Hero, Problem/Solution, Features, Demo, CTA) - Design guidelines from downloaded skills (typography, color, motion, composition) - Security considerations (XSS prevention, input sanitization, CSP) - Performance targets (LCP <2.5s, bundle <150KB, Lighthouse >90) - Responsiveness and accessibility requirements (WCAG 2.1 AA) - Success KPIs and monitoring setup - 3-week development timeline with daily tasks - Definition of Done checklist Download 10+ frontend/UI/UX skills via universal-skills-manager: - frontend-ui-ux: UI/UX design without mockups - frontend-design-guidelines: Production-grade interface guidelines - frontend-developer: React best practices (40+ rules) - frontend-engineer: Next.js 14 App Router patterns - ui-ux-master: Comprehensive design systems and accessibility - ui-ux-systems-designer: Information architecture and interaction - ui-ux-design-user-experience: Platform-specific guidelines - Plus additional reference materials and validation scripts Configure universal-skills MCP with SkillsMP API key for curated skill access. Safety first: All skills validated before installation, no project code modified. Refs: Universal Skills Manager (github:jacob-bd/universal-skills-manager) Next: Begin Sprint 3 landing page development
46 lines
1.0 KiB
Markdown
46 lines
1.0 KiB
Markdown
---
|
|
title: Hoist RegExp Creation
|
|
impact: LOW-MEDIUM
|
|
impactDescription: avoids recreation
|
|
tags: javascript, regexp, optimization, memoization
|
|
---
|
|
|
|
## Hoist RegExp Creation
|
|
|
|
Don't create RegExp inside render. Hoist to module scope or memoize with `useMemo()`.
|
|
|
|
**Incorrect (new RegExp every render):**
|
|
|
|
```tsx
|
|
function Highlighter({ text, query }: Props) {
|
|
const regex = new RegExp(`(${query})`, 'gi')
|
|
const parts = text.split(regex)
|
|
return <>{parts.map((part, i) => ...)}</>
|
|
}
|
|
```
|
|
|
|
**Correct (memoize or hoist):**
|
|
|
|
```tsx
|
|
const EMAIL_REGEX = /^[^\s@]+@[^\s@]+\.[^\s@]+$/
|
|
|
|
function Highlighter({ text, query }: Props) {
|
|
const regex = useMemo(
|
|
() => new RegExp(`(${escapeRegex(query)})`, 'gi'),
|
|
[query]
|
|
)
|
|
const parts = text.split(regex)
|
|
return <>{parts.map((part, i) => ...)}</>
|
|
}
|
|
```
|
|
|
|
**Warning (global regex has mutable state):**
|
|
|
|
Global regex (`/g`) has mutable `lastIndex` state:
|
|
|
|
```typescript
|
|
const regex = /foo/g
|
|
regex.test('foo') // true, lastIndex = 3
|
|
regex.test('foo') // false, lastIndex = 0
|
|
```
|