Skip to content

Cursor rules integration with Swift Concurrency skill#1064

Merged
d4r1091 merged 1 commit intodevelopfrom
dc-noticket-integrate-cursor
Feb 27, 2026
Merged

Cursor rules integration with Swift Concurrency skill#1064
d4r1091 merged 1 commit intodevelopfrom
dc-noticket-integrate-cursor

Conversation

@d4r1091
Copy link
Copy Markdown
Member

@d4r1091 d4r1091 commented Feb 24, 2026

Context

It's been a while, I'm experimenting and defining these rules for Cursor, which I've been finding works pretty well lately.
I wanted to share them with you to seeing whether you think we may benefit team-wide.

Main "features" are:

  • SwiftUI-First Approach: Use SwiftUI for new UI with EcosiaThemeable protocol and theme system
  • Localization Standards: All user-facing strings use String.localized(.key) enum pattern from String.swift
  • Ecosia Code Marking: Mark customizations with // Ecosia: comments; use /* Ecosia: */ blocks when swapping Firefox behavior
  • MVVM Architecture: Follow protocol-oriented MVVM with @published, @StateObject, and async/await patterns
  • Analytics at UI Layer: Implement all analytics using Analytics.shared at ViewModel/UI level only
  • Testing Required: Include tests (Given/When/Then structure) for all new code and features
  • Git Standards: Await user confirmation before commits/pushes; split changes into logical commits
  • Error Handling: Use EcosiaLogger with .error level and proper Result type error propagation
  • Naming Conventions: camelCase for variables/functions, PascalCase for types, verb-based methods
  • Accessibility: Support Dark Mode, Dynamic Type, VoiceOver, and Reduce Motion in all UI

It also includes the Swift Concurrency Skill out of the box

@d4r1091 d4r1091 requested a review from a team February 24, 2026 16:40
@d4r1091 d4r1091 changed the title Cursor integration with Swift Concurrency skill Cursor rules integration with Swift Concurrency skill Feb 24, 2026
Copy link
Copy Markdown

@falkorichter falkorichter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

amazing, will try to use this in claude code

@d4r1091
Copy link
Copy Markdown
Member Author

d4r1091 commented Feb 27, 2026

@falkorichter we may need a .claude or a common place to make these rules valid across LLMs 🤔 💭

@d4r1091 d4r1091 merged commit f1b5ea3 into develop Feb 27, 2026
1 check passed
@d4r1091 d4r1091 deleted the dc-noticket-integrate-cursor branch February 27, 2026 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants