Repository navigation
refactor: share the app icon and wordmark from Design/brand - #228
Merged
Merged
Conversation
Gets the iPhone app ready for TestFlight. - Bundle ids move from the dev.alex placeholders to com.assemblyai.blurt.ios (app, .keyboard, .core, .tests, .probe), the App Group to group.com.assemblyai.blurt.ios, and the term-pack type to com.assemblyai.blurt.terms. The Darwin notification names, log subsystem, queue labels and scripts follow. The Mac app keeps its dev.alex ids: changing them would orphan Sparkle updates and its granted Accessibility permission. - API-key entry is no longer debug-only. Sign in with AssemblyAI doesn't exist yet, so a TestFlight build had no way to add a key. The setup card and Settings now offer "Paste your key" and "Get a free key" in every build, as the Mac app does. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
App Review 5.1.2(i) requires clearly disclosing where personal data goes, third-party AI included, and getting explicit permission first. The listening window now won't open until the user agrees, in a sheet that names AssemblyAI, lists what is sent (the voice while dictating, the text before the cursor, recent dictations, key terms with contact names) and links the privacy policy. Every path to the microphone goes through startListening(), so one gate covers the button, the keyboard's blurt://start and in-app dictation. Consent is versioned (AIConsent): sending something new means a new version, and everyone is asked again. Settings' About section links the privacy policy too. The debug -BlurtStartListening flag grants consent, since scripted runs can't tap through the sheet. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The iPhone project pulled the app icon and the wordmark straight out of the Mac app's folder, so a Mac-side move or rename would have broken the iPhone build. Both now live in a top-level Design/brand that each project references directly; neither app's folder is a dependency of the other. The Mac project lists them explicitly (it used to get them by including its whole Blurt folder) and its project.pbxproj is regenerated. Both app bundles still carry the icon and logo. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2 tasks done
he-james
requested review from
alexkroman and
nbisht-aai
and removed request for
alexkroman
October 2, 2026 22:39
he-james
marked this pull request as ready for review
October 2, 2026 22:40
nbisht-aai
approved these changes
Oct 14, 2026
he-james
enabled auto-merge (squash)
October 14, 2026 21:15
Resolves App/BlurtiOS/project.yml: keeps the Design/brand asset paths from this branch and main's per-target TARGETED_DEVICE_FAMILY "1", camera purpose string and build 2 (#245). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
he-james
added a commit
that referenced
this pull request
Oct 14, 2026
Resolves App/BlurtiOS/project.yml: keeps this branch's AssemblyAI product link and main's per-target TARGETED_DEVICE_FAMILY "1", camera purpose string, build 2 (#245) and Design/brand paths (#228). main's new read-aloud code (#225) uses HTTPTransport, which this branch moves into the AssemblyAI module, so ReadAloudLLM.swift and the tests that reach it (ReadAloudLLMTests, AssemblyAISpeechSynthesizerTests, AssemblyAITranscriberTests, AsyncStreamFixtures) now import AssemblyAI. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
The iPhone project pulled the app icon and wordmark straight out of the Mac app's folder (
../Blurt/Blurt/...), so a Mac-side rename would break the iPhone build. Both now live in a top-levelDesign/brand/that each project references directly, so neither app depends on the other's folder.AppIcon.iconandblurt-ready-logo.pngmove toDesign/brand/(plus a shortDesign/README.md).Blurt/folder;project.pbxprojregenerated.Stacked on #227.
How it was tested
Mac app builds (Debug, unsigned, as
check.shdoes); the bundle hasAppIcon.icns,Assets.carandblurt-ready-logo.png.scripts/ios-check.shpasses (134/134); the iPhone bundle has its icon and logo, and the wordmark renders in the simulator.prettier, markdownlint, swiftlint clean on touched files.
I read AGENTS.md and this doesn't reintroduce anything deliberately removed
Docs updated (
Design/README.md)🤖 Generated with Claude Code