Skip to content

Conversation

philprime
Copy link
Member

Summary

Removes the unused SwiftUI Introspect dependency from the project.

Changes Made

  • Verified no usage: Confirmed SwiftUI Introspect is not used anywhere in the codebase
  • Removed dependencies: Cleaned up project.pbxproj and Package.resolved
  • Updated licenses: Regenerated license files to remove SwiftUI Introspect entries
  • Build verification: All targets compile successfully without the dependency

Why This Change?

The SwiftUI Introspect package was added as a dependency but is not actually used anywhere in the codebase. The project uses modern SwiftUI patterns like @FocusState for TextField management instead of UIKit introspection.

Removing unused dependencies:

  • Reduces build time and app size
  • Simplifies dependency management
  • Eliminates potential security vulnerabilities from unused code
  • Improves project maintainability

Testing

  • All targets build successfully (make build-ios)
  • No linting violations (0 violations across 115 files)
  • License generation works correctly
  • Package resolution updated automatically

- Remove SwiftUI Introspect package from Xcode project dependencies
- Update Package.resolved to exclude SwiftUI Introspect
- Regenerate license files to remove SwiftUI Introspect entries
- Verify all targets build successfully without the dependency

The SwiftUI Introspect package was not being used anywhere in the codebase.
Modern SwiftUI patterns like @focusstate are used instead for TextField management.
@philprime philprime self-assigned this Sep 30, 2025
@Copilot Copilot AI review requested due to automatic review settings September 30, 2025 09:18
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the unused SwiftUI Introspect dependency from the project to improve maintainability and reduce build complexity. The dependency was verified as unused throughout the codebase.

Key changes:

  • Removed SwiftUI Introspect package references from Xcode project configuration
  • Cleaned up license files to remove SwiftUI Introspect entries
  • Updated package resolution to reflect the dependency removal

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
swiftui-introspect.plist Removed license file for SwiftUI Introspect dependency
Licenses.plist Removed SwiftUI Introspect entry from license index
Licenses.latest_result.txt Updated license generation result to exclude SwiftUI Introspect
Package.resolved Updated package resolution hash and removed SwiftUI Introspect pin
project.pbxproj Removed all SwiftUI Introspect references from Xcode project configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@philprime philprime enabled auto-merge (squash) September 30, 2025 09:18
@philprime philprime merged commit 2951fa9 into main Sep 30, 2025
4 checks passed
@philprime philprime deleted the chore/remove-swiftui-introspect branch September 30, 2025 09:25
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.

1 participant