ryanashcraft

name	quality-check
description	Use when reviewing a plan or implementation approach to identify hacks, shortcuts, or inelegant solutions that sacrifice long-term codebase health.

Review this plan for anything that could be more elegant — hacks, unnecessary remnants, things that could be simplified, or patterns that diverge from how similar things are already done. The bar is: changes should improve or meet the existing quality and architecture of the codebase, not add tech debt.

If a question can be answered by exploring the codebase, explore the codebase instead.

ryanashcraft

# Xcode Project Makefile
# Example of building Xcode projects with proper settings and clean output

.DEFAULT_GOAL := help

# Configuration
PROJECT_PATH = MyApp/MyApp.xcodeproj
SCHEME = MyApp
CONFIG = Debug
DERIVED_DATA = build

ryanashcraft

#!/bin/bash

# xcodebuild-filter - Simplify xcodebuild output
# Usage: xcodebuild [args] | ./bin/xcodebuild-filter [--errors-only]

# Configuration
ERRORS_ONLY=false

# Parse arguments
for arg in "$@"; do

ryanashcraft

import SwiftUI

@available(watchOS 10.0, *)
@available(iOS 17.0, *)
private struct ContentMarginsFollowReadableWidthModifier: ViewModifier {
    static let readableWidth: Double = 672
    static let minInsetLength: Double = 16

    @ViewBuilder
    func body(content: Content) -> some View {

ryanashcraft

import SwiftUI

@main
struct WindowPresentationFunApp: App {
    @State var appState = AppState()

    var body: some Scene {
        WindowGroup {
            RootView(appState: appState)
                .onAppear {

ryanashcraft

import SwiftUI

private struct SafeAreaInsetsFollowReadableWidthModifier: ViewModifier {
    static let readableWidth: Double = 672

    @ViewBuilder
    func body(content: Content) -> some View {
        GeometryReader { geometry in
            let insetLength = max(0, geometry.size.width - Self.readableWidth) / 2

ryanashcraft

import Foundation

private extension Bundle {
    private static let packageName = "my-package"
    private static let moduleName = "MyModule"

    static var swiftUIPreviewsCompatibleModule: Bundle {
        final class CurrentBundleFinder {}

        let isPreview = ProcessInfo.processInfo.environment["XCODE_RUNNING_FOR_PREVIEWS"] == "1"

ryanashcraft

import SwiftUI

// Works around a critical issue observed on iPads with a trackpad cursor.
//
// With highPriorityGesture, other interactive content on the screen can become unresponsive after
// triggering the gesture with an iPad cursor. It's as if the gesture seems to never relinquish its
// grasp on the entire UI, until you interact with the screen directly with a tap or trigger a
// context menu.
//
// This view modifier and extension works around this by forcing the gesture to be "cancelled" by

ryanashcraft

/// An iOS style TabView that doesn't reset it's childrens navigation stacks when tabs are switched.
public struct UIKitTabView: View {
    private var viewControllers: [UIHostingController<AnyView>]
    private var selectedIndex: Binding<Int>?
    @State private var fallbackSelectedIndex: Int = 0
    
    public init(selectedIndex: Binding<Int>? = nil, @TabBuilder _ views: () -> [Tab]) {
        self.viewControllers = views().map {
            let host = UIHostingController(rootView: $0.view)
            host.tabBarItem = $0.barItem

ryanashcraft

import SwiftUI

// Models

enum Lyric {
    case line(String)
    case pause(TimeInterval)
}

class ScrollToModel: ObservableObject {