Skip to content

chore(ios): remove dead code from the app targets - #280

Merged
schiste merged 4 commits into
mainfrom
agent/remove-dead-code-from-the-ios-app-target
Oct 23, 2026
Merged

schiste merged 4 commits into
mainfrom
agent/remove-dead-code-from-the-ios-app-target

Conversation

@schiste

@schiste schiste commented Oct 23, 2026

Copy link
Copy Markdown
Member

Summary

Remove dead code from the iOS app targets

Changes

Checklist

  • pnpm quality:prepush passes, or pnpm quality:prepush:full for high-impact changes
  • cargo test --workspace passes in apps/chau7-next (if macOS app changes)
  • swift test --package-path apps/chau7-ios/Chau7Core passes (if Swift changes)
  • New tests added for new behavior
  • Documentation updated (README, CHANGELOG.md if user-facing)

Test plan

Commits:

  • f6f0c93 chore(ios): remove dead code from the app targets

Opened by aethyme broker push as a draft.

Delete RemoteActivityCard.swift, which is never instantiated, and the
all-comment RemoteCrypto.swift stub, whose header asked for its own
deletion, along with their project references. Also remove symbols with
no callers in the app, widget or tests: isEncryptedSession,
clearMacNonce, BackgroundKeepalive.isActive, cellSize(for:),
drawMilliseconds (always 0), DiagnosticsLog.trace and measure,
ApprovalRequest title/subtitle/isProtectedRemoteAction, the viewport's
unread displayState, ProtectedRemoteSend.tabTitle, and three unused
fillFoldedRun parameters.

xcodebuild test, Chau7RemoteApp scheme, iPhone 17 Pro simulator: 125
tests, 0 failures; plutil -lint project.pbxproj OK.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@schiste
schiste marked this pull request as ready for review October 23, 2026 15:43
@schiste
schiste merged commit de02565 into main Oct 23, 2026
1 check passed
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