oh-my-customcode 0.1.2 → 0.1.4

package/templates/agents/sw-engineer/frontend/vercel-agent/refs/web-design/index.yaml

@@ -0,0 +1,20 @@
+# Web Design Guide
+
+metadata:
+  name: web-design
+  description: Web design best practices and accessibility reference
+
+source:
+  type: external
+  origin: github
+  url: https://github.com/vercel-labs/agent-skills
+  related_skill: web-design-guidelines
+
+documents:
+  - name: accessibility
+    title: Accessibility Best Practices
+    path: ./accessibility.md
+
+  - name: performance
+    title: Performance Guidelines
+    path: ./performance.md

package/templates/agents/sw-engineer/frontend/vercel-agent/refs/web-design/performance.md

@@ -0,0 +1,102 @@
+# Performance Guidelines
+
+> Reference guide for react-best-practices skill
+
+## Core Web Vitals
+
+### LCP (Largest Contentful Paint)
+```
+Good: < 2.5s
+Needs improvement: 2.5s - 4s
+Poor: > 4s
+```
+
+### FID (First Input Delay)
+```
+Good: < 100ms
+Needs improvement: 100ms - 300ms
+Poor: > 300ms
+```
+
+### CLS (Cumulative Layout Shift)
+```
+Good: < 0.1
+Needs improvement: 0.1 - 0.25
+Poor: > 0.25
+```
+
+## Optimization Strategies
+
+### JavaScript
+```
+- Code splitting
+- Tree shaking
+- Lazy loading
+- Minimize main thread work
+```
+
+### Images
+```
+- Modern formats (WebP, AVIF)
+- Responsive images
+- Lazy loading
+- Proper sizing
+```
+
+### CSS
+```
+- Critical CSS inline
+- Defer non-critical
+- Remove unused
+- Minimize specificity
+```
+
+### Fonts
+```
+- font-display: swap
+- Preload critical fonts
+- Subset fonts
+- System font fallback
+```
+
+## Next.js Specific
+
+### Server Components
+```tsx
+// Default: Server Component (no directive)
+export default function Page() {
+  return <div>Server rendered</div>
+}
+```
+
+### Client Components
+```tsx
+'use client'
+// Only when needed for interactivity
+export default function Button() {
+  return <button onClick={...}>Click</button>
+}
+```
+
+### Data Fetching
+```tsx
+// Server: fetch in component
+async function Page() {
+  const data = await fetch(...)
+  return <div>{data}</div>
+}
+
+// Client: use SWR/React Query
+'use client'
+function Component() {
+  const { data } = useSWR('/api/data', fetcher)
+  return <div>{data}</div>
+}
+```
+
+## Measurement Tools
+
+- Lighthouse
+- WebPageTest
+- Chrome DevTools Performance
+- Next.js Analytics

package/templates/agents/sw-engineer/frontend/vercel-agent/refs/web-design-guidelines/SKILL.md

@@ -0,0 +1,117 @@
+# Web Design Guidelines
+
+> **Source**: vercel-labs/agent-skills
+> **Rules**: 100+ rules across 11 categories
+
+## When to Use
+
+- UI code review
+- Accessibility audit
+- Design consistency check
+- UX best practices verification
+
+## Command Triggers
+
+- "Review UI"
+- "Check accessibility"
+- "Design audit"
+- "Verify best practices"
+
+## Categories
+
+### ARIA Labels
+```
+- All interactive elements have labels
+- Proper role attributes
+- aria-live for dynamic content
+- Descriptive aria-describedby
+```
+
+### Focus States
+```
+- Visible focus indicators
+- Logical focus order
+- Focus trapping in modals
+- Skip links for navigation
+```
+
+### Form Validation
+```
+- Inline error messages
+- Accessible error announcements
+- Clear input requirements
+- Proper label associations
+```
+
+### Animations
+```
+- Respect prefers-reduced-motion
+- Meaningful animations only
+- Performance-optimized transforms
+- No animation on load
+```
+
+### Typography
+```
+- Readable font sizes (16px min)
+- Proper line height (1.5+)
+- Sufficient color contrast
+- Responsive text scaling
+```
+
+### Images
+```
+- Alt text for all images
+- Decorative images marked
+- Responsive images
+- Lazy loading implemented
+```
+
+### Performance
+```
+- Minimize layout shifts
+- Optimize largest contentful paint
+- Reduce time to interactive
+- Efficient CSS selectors
+```
+
+### Navigation
+```
+- Keyboard accessible
+- Clear current state
+- Consistent patterns
+- Breadcrumbs where needed
+```
+
+### Dark Mode
+```
+- System preference detection
+- Smooth transitions
+- Consistent color tokens
+- Image adjustments
+```
+
+### Touch Interaction
+```
+- Minimum 44px touch targets
+- Appropriate spacing
+- Gesture alternatives
+- Haptic feedback consideration
+```
+
+### i18n
+```
+- RTL support
+- Text expansion room
+- Locale-aware formatting
+- Translation-ready strings
+```
+
+## Execution Flow
+
+```
+1. Scan UI components
+2. Check against category rules
+3. Report violations
+4. Suggest fixes
+```

package/templates/agents/sw-engineer/frontend/vercel-agent/refs/web-design-guidelines/index.yaml

@@ -0,0 +1,34 @@
+# Web Design Guidelines Skill
+
+metadata:
+  name: web-design-guidelines
+  category: development
+  description: UI code review with 100+ rules for accessibility, performance, and UX
+
+source:
+  type: external
+  origin: github
+  url: https://github.com/vercel-labs/agent-skills
+  path: skills/web-design-guidelines
+  version: "1.0.0"
+  last_updated: "2025-01-22"
+
+triggers:
+  - "UI review"
+  - "accessibility check"
+  - "design audit"
+  - "UX review"
+  - "a11y check"
+
+categories:
+  - aria-labels
+  - focus-states
+  - form-validation
+  - animations
+  - typography
+  - images
+  - performance
+  - navigation
+  - dark-mode
+  - touch-interaction
+  - i18n

package/templates/agents/sw-engineer/language/golang-expert/refs/go-best-practices/SKILL.md

@@ -0,0 +1,202 @@
+# Go Best Practices Skill
+
+> **Category**: Development
+> **Source**: Internal (based on Effective Go)
+
+## Purpose
+
+Apply idiomatic Go patterns and best practices from official Go documentation.
+
+## Rules
+
+### 1. Formatting
+
+```yaml
+rules:
+  - Use gofmt for all code formatting
+  - Indentation uses tabs, not spaces
+  - No line length limit, but break long lines sensibly
+  - Let gofmt handle alignment
+```
+
+### 2. Naming Conventions
+
+```yaml
+package_names:
+  - Short, concise, lowercase, single-word
+  - No underscores or mixedCaps
+  - Name is basename of source directory
+  - Example: bufio, not bufIO or buf_io
+
+getters:
+  - No "Get" prefix for getters
+  - obj.Owner() not obj.GetOwner()
+  - Setter can use "Set": obj.SetOwner(user)
+
+interface_names:
+  - One-method interfaces: method name + "er"
+  - Reader, Writer, Formatter, CloseNotifier
+  - Avoid stealing standard names unless same signature
+
+mixedCaps:
+  - Use MixedCaps or mixedCaps, never underscores
+  - Exported: MixedCaps (uppercase first)
+  - Unexported: mixedCaps (lowercase first)
+```
+
+### 3. Control Structures
+
+```yaml
+if_statements:
+  - Accept initialization statement
+  - Prefer: if err := file.Chmod(0664); err != nil
+  - Omit else when if ends with break/continue/return
+  - Avoid unnecessary else
+
+for_loops:
+  - Go's only loop construct
+  - for init; condition; post { } - like C for
+  - for condition { } - like C while
+  - for { } - infinite loop
+  - Use range for strings, slices, arrays, maps, channels
+
+switch:
+  - Cases don't fall through by default
+  - Use fallthrough keyword if needed
+  - Multiple cases: case '0', '1', '2'
+  - Type switch: switch v := x.(type)
+```
+
+### 4. Functions
+
+```yaml
+multiple_returns:
+  - Return multiple values for result + error
+  - func (file *File) Write(b []byte) (n int, err error)
+
+named_results:
+  - Document return values with names
+  - Can simplify code but use judiciously
+  - Unnamed returns OK for short functions
+
+defer:
+  - Executes when function returns
+  - LIFO order for multiple defers
+  - Use for cleanup: unlock mutexes, close files
+  - Arguments evaluated when defer executes, not when called
+```
+
+### 5. Data
+
+```yaml
+new_vs_make:
+  - new(T): allocates zeroed storage, returns *T
+  - make(T, args): creates slices, maps, channels only
+  - make returns initialized (not zeroed) value of type T
+
+slices:
+  - Backed by arrays, can grow
+  - append() to add elements
+  - copy() for safe duplication
+  - Slice of slice shares underlying array
+
+maps:
+  - Reference type, nil until initialized
+  - make(map[KeyType]ValueType)
+  - Comma ok idiom: val, ok := map[key]
+  - delete(map, key) to remove
+```
+
+### 6. Methods
+
+```yaml
+pointer_vs_value_receivers:
+  - Pointer receiver: can modify, no copy overhead
+  - Value receiver: safe from modification
+  - If any method needs pointer, all should use pointer
+  - Rule: values immutable, pointers mutable
+```
+
+### 7. Interfaces
+
+```yaml
+design:
+  - Interfaces define behavior, not data
+  - Small interfaces (1-3 methods) preferred
+  - Accept interfaces, return structs
+  - If type implements interface, it satisfies implicitly
+
+common_interfaces:
+  - io.Reader: Read(p []byte) (n int, err error)
+  - io.Writer: Write(p []byte) (n int, err error)
+  - fmt.Stringer: String() string
+  - error: Error() string
+```
+
+### 8. Concurrency
+
+```yaml
+goroutines:
+  - Lightweight, multiplexed onto OS threads
+  - go function() to start
+  - Don't communicate by sharing memory
+  - Share memory by communicating
+
+channels:
+  - Primary synchronization mechanism
+  - ch := make(chan int) - unbuffered
+  - ch := make(chan int, 100) - buffered
+  - Send: ch <- v
+  - Receive: v := <-ch
+  - Close: close(ch)
+
+patterns:
+  - Don't leak goroutines - ensure they exit
+  - Use context for cancellation
+  - select for multiple channel operations
+  - sync.WaitGroup for goroutine coordination
+```
+
+### 9. Error Handling
+
+```yaml
+principles:
+  - Errors are values, handle them explicitly
+  - Check errors immediately after call
+  - Return errors, don't panic
+  - Add context when propagating: fmt.Errorf("op failed: %w", err)
+
+panic_recover:
+  - Panic for unrecoverable errors only
+  - Library functions should return errors, not panic
+  - Recover only in deferred functions
+  - Convert internal panics to errors at package boundary
+```
+
+### 10. Project Structure
+
+```yaml
+packages:
+  - One package per directory
+  - Package name matches directory name
+  - main package for executables
+  - Avoid circular imports
+
+files:
+  - doc.go for package documentation
+  - _test.go suffix for test files
+  - Group related types and functions
+```
+
+## Application
+
+When writing or reviewing Go code:
+
+1. **Always** run `gofmt` or `goimports`
+2. **Always** handle returned errors
+3. **Prefer** composition over inheritance
+4. **Prefer** small interfaces
+5. **Prefer** channels for goroutine communication
+6. **Avoid** global state
+7. **Avoid** init() when possible
+8. **Document** exported identifiers

package/templates/agents/sw-engineer/language/golang-expert/refs/go-best-practices/index.yaml

@@ -0,0 +1,25 @@
+# Go Best Practices Skill
+
+metadata:
+  name: go-best-practices
+  category: development
+  description: Idiomatic Go patterns from Effective Go
+
+source:
+  type: internal
+  reference: https://go.dev/doc/effective_go
+
+provides:
+  - Go code formatting standards
+  - Naming conventions
+  - Control structure patterns
+  - Function design patterns
+  - Data structure usage
+  - Method receiver guidelines
+  - Interface design
+  - Concurrency patterns
+  - Error handling patterns
+  - Project structure guidelines
+
+used_by:
+  - golang-expert