| name | gluestack-ui-v4:validation |
| description | Validation checklist and anti-patterns for gluestack-ui v4 - use for code review, checking implementation quality, and identifying common mistakes. |
Gluestack UI v4 - Validation & Anti-Patterns
This sub-skill focuses on validating implementations, identifying anti-patterns, and ensuring code quality for gluestack-ui v4.
Validation Checklist
When reviewing code, check for:
Component Usage
Component Props vs className
Styling
Compound Components
Icons
Cross-Platform Compatibility
Performance & Best Practices
Anti-Patterns to Avoid
❌ Don't: Mix React Native and Gluestack Components
import { View, Text } from "react-native";
import { Button } from "@/components/ui/button";
<View>
<Text>Mixed usage</Text>
<Button>Click</Button>
</View>
Why it's bad: Loses theming, accessibility, and cross-platform consistency.
Correct approach:
import { Box } from "@/components/ui/box";
import { Text } from "@/components/ui/text";
import { Button, ButtonText } from "@/components/ui/button";
<Box>
<Text>Consistent usage</Text>
<Button>
<ButtonText>Click</ButtonText>
</Button>
</Box>
❌ Don't: Use Non-Semantic Color Tokens
STRICTLY PROHIBITED - You MUST use ONLY semantic tokens for ALL colors.
<Text className="text-typography-900">Heading</Text>
<Text className="text-typography-700">Body</Text>
<Text className="text-typography-500">Muted</Text>
<Box className="bg-neutral-100">Card</Box>
<Text className="text-neutral-600">Text</Text>
<Box className="border-neutral-300">Border</Box>
<Box className="bg-gray-50">Background</Box>
<Text className="text-gray-900">Content</Text>
<Box className="border-gray-200">Border</Box>
<Box className="bg-blue-600">Primary</Box>
<Text className="text-red-500">Error</Text>
<Box className="border-green-400">Success</Box>
<Box className="bg-[#3b82f6]">Arbitrary</Box>
<Text className="text-[#DC2626]">Error</Text>
<Box style={{ backgroundColor: '#fff' }}>White</Box>
<Text style={{ color: 'red' }}>Error</Text>
<Text className="text-black opacity-70">Muted</Text>
<Box className="bg-blue-600 bg-opacity-90">Transparent</Box>
Why it's bad:
- ❌ Breaks theming and dark mode
- ❌ Creates maintenance debt
- ❌ Violates design system
- ❌ Inconsistent colors across app
Correct approach:
<Text className="text-foreground">Heading</Text>
<Text className="text-foreground">Body</Text>
<Text className="text-muted-foreground">Muted</Text>
<Box className="bg-muted">Card</Box>
<Text className="text-muted-foreground">Text</Text>
<Box className="border-border">Border</Box>
<Box className="bg-background">Background</Box>
<Text className="text-foreground">Content</Text>
<Box className="border-border">Border</Box>
<Box className="bg-primary">Primary</Box>
<Text className="text-destructive">Error</Text>
<Box className="border-primary">Success</Box>
<Text className="text-foreground/70">Muted</Text>
<Box className="bg-primary/90">Transparent</Box>
❌ Don't: Skip Sub-Components
<Button>Click Me</Button>
<Input>
<InputIcon as={MailIcon} />
<InputField />
</Input>
<FormControl>
<Text>Email</Text>
<InputField />
</FormControl>
Why it's bad: Components won't render correctly, breaks styling and accessibility.
Correct approach:
<Button>
<ButtonText>Click Me</ButtonText>
</Button>
<Input>
<InputSlot>
<InputIcon as={MailIcon} />
</InputSlot>
<InputField />
</Input>
<FormControl>
<FormControlLabel>
<FormControlLabelText>Email</FormControlLabelText>
</FormControlLabel>
<Input>
<InputField />
</Input>
</FormControl>
❌ Don't: Use Inline Styles When className Works
<Box style={{ padding: 16, backgroundColor: '#fff' }} />
<Text style={{ fontSize: 18, fontWeight: 'bold' }} />
Why it's bad: Bypasses optimization, breaks theming, harder to maintain.
Correct approach:
<Box className="p-4 bg-background" />
<Text size="lg" bold className="text-foreground" />
❌ Don't: Use Arbitrary Spacing Values
<Box className="p-[13px] m-[27px]" />
<Box style={{ padding: 13, margin: 27 }} />
<VStack className="gap-[15px]" />
Why it's bad: Creates maintenance burden, inconsistent spacing across app.
Correct approach:
<Box className="p-3 m-6" />
<VStack space="lg" />
❌ Don't: Use className for Component Props
<VStack className="gap-4">
<Box>Item</Box>
</VStack>
<Button className="bg-primary px-8 py-2">
<ButtonText>Click</ButtonText>
</Button>
<Heading className="text-2xl font-bold">Title</Heading>
Why it's bad: Loses type safety, harder to maintain, bypasses design system.
Correct approach:
<VStack space="lg">
<Box>Item</Box>
</VStack>
<Button variant="default" size="lg">
<ButtonText>Click</ButtonText>
</Button>
<Heading size="2xl" bold>Title</Heading>
❌ Don't: Import from react-native for Wrapped Components
import { KeyboardAvoidingView, View, Text } from 'react-native';
<KeyboardAvoidingView>
<View>
<Text>Content</Text>
</View>
</KeyboardAvoidingView>
Why it's bad: Breaks cross-platform compatibility, loses theming and accessibility.
Correct approach:
import { KeyboardAvoidingView } from '@/components/ui/keyboard-avoiding-view';
import { Box } from '@/components/ui/box';
import { Text } from '@/components/ui/text';
<KeyboardAvoidingView>
<Box>
<Text>Content</Text>
</Box>
</KeyboardAvoidingView>
❌ Don't: Use ScrollView for Long Lists
<ScrollView>
{items.map((item) => (
<Box key={item.id}>
<Text>{item.name}</Text>
</Box>
))}
</ScrollView>
Why it's bad: No virtualization, all items rendered at once, poor performance.
Correct approach:
<FlatList
data={items}
renderItem={({ item }) => (
<Box>
<Text>{item.name}</Text>
</Box>
)}
keyExtractor={(item) => item.id}
/>
❌ Don't: Use Animated API for Animations
import { Animated } from 'react-native';
const animValue = new Animated.Value(0);
Animated.timing(animValue, {
toValue: 100,
duration: 300,
}).start();
Why it's bad: Runs on JavaScript thread, can cause jank and dropped frames.
Correct approach:
import { useSharedValue, withTiming } from 'react-native-reanimated';
const animValue = useSharedValue(0);
animValue.value = withTiming(100, { duration: 300 });
Common Mistakes Summary
| Mistake | Impact | Correct Approach |
|---|
| Using React Native primitives | Loses theming, accessibility, cross-platform support | Use Gluestack components |
Using typography-*, neutral-*, gray-* tokens | Breaks theming and dark mode | Use ONLY semantic tokens |
Using numbered colors (red-500, blue-600) | Breaks theming and dark mode | Use semantic tokens |
Using opacity utilities (opacity-70) | Inconsistent transparency | Use alpha values (/70, /90) |
| Raw color values | Breaks theming and dark mode | Use semantic tokens |
| Skipping sub-components | Components won't render correctly | Use proper compound components |
| Inline styles for static values | Bypasses optimization, harder to maintain | Use className |
| Arbitrary spacing values | Creates maintenance burden | Use spacing scale |
| className instead of props | Loses type safety | Use component props when available |
| Direct react-native imports | Breaks cross-platform compatibility | Use Gluestack wrappers |
| ScrollView for long lists | Poor performance | Use FlatList |
| Animated API | Janky animations | Use Reanimated worklets |
Critical Issues (Must Fix Immediately)
🔴 Critical: InputIcon Not Wrapped in InputSlot
<Input>
<InputIcon as={MailIcon} />
<InputField />
</Input>
<Input>
<InputSlot>
<InputIcon as={MailIcon} />
</InputSlot>
<InputField />
</Input>
🔴 Critical: Missing ButtonText
<Button>Submit</Button>
<Button>
<ButtonText>Submit</ButtonText>
</Button>
🔴 Critical: Using React Native Instead of Gluestack
import { View, Text } from 'react-native';
import { Box } from '@/components/ui/box';
import { Text } from '@/components/ui/text';
🔴 Critical: Using Non-Semantic Color Tokens
<Text className="text-typography-900">Heading</Text>
<Box className="bg-neutral-100">Card</Box>
<Text className="text-gray-700">Content</Text>
<Box className="bg-blue-600">Primary</Box>
<Text className="text-foreground">Heading</Text>
<Box className="bg-muted">Card</Box>
<Text className="text-foreground">Content</Text>
<Box className="bg-primary">Primary</Box>
Code Review Guidelines
High Priority (Must Have)
- Component consistency - All components use Gluestack wrappers
- Compound components - All sub-components used correctly
- CRITICAL: Semantic tokens ONLY - Zero tolerance for:
typography-*, neutral-*, gray-*, slate-* tokens
- Numbered colors:
red-500, blue-600, green-400
- Arbitrary values:
[#3b82f6], [#DC2626]
- Opacity utilities:
opacity-70, bg-opacity-90
- Must use ONLY:
text-foreground, bg-primary, border-border, etc.
- Component props - Props used instead of className when available
Medium Priority (Should Have)
- TypeScript types - All props and components typed
- Spacing scale - No arbitrary spacing values
- Performance - FlatList for long lists, memoization for expensive components
- Dark mode - Proper dark mode support
Low Priority (Nice to Have)
- Animations - Using Reanimated instead of Animated API
- Code organization - Logical component structure
- Documentation - Comments for complex logic
Escalation Guidance
When a design request cannot be satisfied with existing patterns:
Step 1: Push Back Early
Explain the implications:
- Performance impact
- Maintenance burden
- Breaks theming/dark mode
- Inconsistent with design system
Example:
"Using arbitrary spacing values like p-[13px] creates maintenance issues and breaks consistency. Can we use p-3 (12px) or p-4 (16px) from our spacing scale instead?"
Step 2: Propose Alternatives
Map to existing tokens:
Suggest new semantic tokens:
Step 3: Add to Design System
If truly needed, add token to gluestack-ui-provider/config.ts:
export const config = {
tokens: {
colors: {
success: '#22c55e',
'success-foreground': '#ffffff',
},
},
};
Step 4: Document Exception
If inline style is unavoidable, document why:
<Box style={{ paddingBottom: insets.bottom }}>
{}
</Box>
Quick Validation Script
Use this mental checklist when reviewing code:
- ✅ Gluestack components? (not React Native)
- ✅ Compound components correct? (ButtonText, InputSlot, etc.)
- ✅ CRITICAL: Semantic tokens ONLY?
- ❌ No
typography-*, neutral-*, gray-*, slate-*
- ❌ No numbered colors:
red-500, blue-600
- ❌ No arbitrary values:
[#3b82f6]
- ❌ No opacity utilities:
opacity-70
- ✅ Only semantic:
text-foreground, bg-primary, border-border
- ✅ Component props? (space, size, variant)
- ✅ Spacing scale? (no arbitrary values)
- ✅ TypeScript types? (all props typed)
- ✅ Performance? (FlatList, memoization)
- ✅ Cross-platform? (Gluestack wrappers)
Reference