`.swiftlint.yml` had `included: Brew`, so `swiftlint lint --strict` skipped
the app targets entirely; it now lists `Homebrew` and `HomebrewUpgradeHelper`.
`pr_build_test.yml` filtered on `Brew/**` and `Brew.entitlements`, so an
app-only change could skip Build and Test. No entitlements file exists
anywhere in the tree, so that entry is dropped rather than repointed.
BrewUILint's glob reached `Homebrew` but not `HomebrewUpgradeHelper`, leaving
2 files unlinted; CI, `scripts/pre-commit` and AGENTS.md now agree on
`find Homebrew HomebrewUpgradeHelper Sources`.
CONVENTIONS.md and two Cursor rules pointed at `Brew/Models/HomebrewPackageReference.swift`
and `Brew/PreviewSupport/AppPreviewSupport.swift`, neither of which exists under
any prefix; they now point at `Sources/BrewCore/Models/HomebrewPackageID.swift`
and `Sources/BrewRepositoryInterfaces/PreviewSupport/PreviewSupport.swift`.
The BrewUILint step scanned 'Brew/', which no longer exists after the
source folder was renamed to 'Homebrew/', so the hook errored on every
commit. Point the whole-tree scan at 'Homebrew/'.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
BrewUILint only scanned Brew/ (now the thin app shell), so its rules — notably
NonisolatedExtensionRule, which guards the cross-package MainActor-default
extension trap the split made possible — no longer reached Sources/. Point CI
(and the AGENTS.md parity command) at `find Brew Sources`, and add a pre-commit
step that builds + runs BrewUILint over the whole tree on every commit (it must
see all files in one pass so the rule knows every `nonisolated` type). Tests/
stay excluded. Also annotate two fixture extensions on BrewCore types
`nonisolated` per that rule.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Run the custom linter on Brew sources in swift_quality, document local parity in AGENTS.md, align package identity cursor guidance, and fix pre-commit when no files are partially staged.
Co-authored-by: Cursor <cursoragent@cursor.com>
SurefireStudios