Skip to content

Ship the AutoFill extension and time-sensitive entitlement - #177

Merged
jjalangtry merged 2 commits into
mainfrom
staging
Jul 30, 2026
Merged

Ship the AutoFill extension and time-sensitive entitlement#177
jjalangtry merged 2 commits into
mainfrom
staging

Conversation

@jjalangtry

@jjalangtry jjalangtry commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Portal work is done, so both entitlements and the Lab86MailAutoFill target are back on.

The extension needed its own App IDs — it was falling back to the wildcard profile, which cannot carry an app group. Created io.lab86.mail.autofill and io.lab86.mail.dev.autofill via the ASC API.

Verified on the Mac build box: the build signs, and the embedded appex carries group.io.lab86.mail and declares ProvidesOneTimeCodes. First successful compile and sign of this target.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • New Features
    • Enabled iOS time-sensitive notifications.
    • Added support for sharing data between the app and its AutoFill extension.
    • Enabled the Mail AutoFill extension for iOS with its required build and configuration settings.

…ment

The developer-portal work is done, so the two entitlements and the
Lab86MailAutoFill target come back on.

Provisioning needed more than the app's own App ID. The extension was falling
back to the wildcard "iOS Team Provisioning Profile: *", which cannot carry an
app group, because no App ID existed for its bundle identifier. Created
io.lab86.mail.autofill and io.lab86.mail.dev.autofill through the App Store
Connect API with the App Groups capability; assigning the group itself stayed
manual, as it was for the app.

Verified on the Mac build box rather than assumed: the build signs, and the
embedded Lab86MailAutoFill.appex carries com.apple.security.application-groups
with group.io.lab86.mail and declares ProvidesOneTimeCodes. That is the first
time this target has been compiled and signed at all.

Note that the bundleIdCapabilities GET returns "(none)" for these App IDs even
when the capabilities are demonstrably present, so it is not a usable check —
a signing build is.
@jjalangtry
jjalangtry merged commit cc9cdd9 into main Jul 30, 2026
3 of 9 checks passed
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9a11ad29-3b8f-4768-a5cd-8c61ceb041ee

📥 Commits

Reviewing files that changed from the base of the PR and between cea0b35 and 70945e1.

📒 Files selected for processing (2)
  • apps/ios/Lab86Mail/Resources/Lab86Mail.entitlements
  • apps/ios/project.yml

📝 Walkthrough

Walkthrough

The iOS configuration activates time-sensitive notifications and the shared app group, enables the Lab86MailAutoFill app-extension target, and declares it as a dependency of the main Lab86Mail target.

Changes

iOS AutoFill integration

Layer / File(s) Summary
Activate shared app entitlements
apps/ios/Lab86Mail/Resources/Lab86Mail.entitlements
Enables time-sensitive notifications and the group.io.lab86.mail application group.
Enable AutoFill target
apps/ios/project.yml
Enables the Lab86MailAutoFill app-extension configuration and wires it as a dependency of Lab86Mail.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch staging

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

✅ Coverage Report

Metric Value
Current coverage 85.10% (42099/49469)
Baseline coverage 85.10% (42099/49469)
Result PASS

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