ruby_everywhere 0.12.0 → 0.14.0

data/lib/everywhere/builders/android.rb

@@ -173,7 +173,7 @@ module Everywhere
         @version_code = version_code
 
         unless dev || @config.remote?
-          UI.die!("Android apps are remote mode only for now — set remote.url in config/everywhere.yml " \
+          UI.die!("Android apps are remote mode only for now. Set remote.url in config/everywhere.yml " \
                   "(the shell wraps your deployed app; nothing is packaged locally)")
         end
 
@@ -184,7 +184,7 @@ module Everywhere
         # package name. Caught here so the answer is a config key, not a
         # Gradle stacktrace.
         unless application_id.match?(APPLICATION_ID)
-          UI.die!("#{application_id.inspect} can't be an Android applicationId — it needs two or " \
+          UI.die!("#{application_id.inspect} can't be an Android applicationId. It needs two or " \
                   "more dot-separated segments of letters, digits and _, each starting with a " \
                   "letter. Set platforms.android.bundle_id in config/everywhere.yml")
         end
@@ -208,12 +208,12 @@ module Everywhere
 
       def check_format!(dev:, format:)
         unless FORMATS.key?(format)
-          UI.die!("#{format.inspect} isn't an Android package format — :apk (installable) or :aab " \
+          UI.die!("#{format.inspect} isn't an Android package format. Use :apk (installable) or :aab " \
                   "(what Google Play accepts)")
         end
         return unless format == :aab && dev
 
-        UI.die!("an .aab can't be installed on a device or an emulator — it's an upload format Play " \
+        UI.die!("an .aab can't be installed on a device or an emulator. It's an upload format Play " \
                 "re-signs into per-device APKs. `every dev` builds an APK; ask for the bundle when " \
                 "you're publishing")
       end
@@ -231,7 +231,7 @@ module Everywhere
         missing = REQUIRED_SIGNING_ENV.reject { |name| env_value(name) }
         unless missing.empty?
           UI.die!("EVERY_ANDROID_KEYSTORE is set but #{missing.join(" and ")} " \
-                  "#{missing.one? ? "is" : "are"} not — signing needs the keystore, its password " \
+                  "#{missing.one? ? "is" : "are"} not. Signing needs the keystore, its password " \
                   "and the alias of the key inside it (EVERY_ANDROID_KEY_PASSWORD falls back to " \
                   "the keystore password)")
         end
@@ -316,7 +316,7 @@ module Everywhere
         xml = <<~XML
           <?xml version="1.0" encoding="utf-8"?>
           <!-- Written by `every build` because config/everywhere.yml sets
-               remote.instances: true — previewed apps arrive as plain http on
+               remote.instances: true, and previewed apps arrive as plain http on
                LAN addresses that cannot be enumerated at build time. Do not
                edit; the template's strict default returns if instances is
                turned off. -->
@@ -344,7 +344,7 @@ module Everywhere
       # opted in, so the shell's own tested level is the default.
       def write_properties(work)
         File.write(File.join(work, "app", "everywhere.properties"), <<~PROPERTIES)
-          # Written by `every build --target #{@target}` — do not edit; changes
+          # Written by `every build --target #{@target}`. Do not edit; changes
           # belong in config/everywhere.yml.
           everywhere.applicationId=#{properties_value(application_id)}
           everywhere.versionName=#{properties_value(@config.version(target: @target))}
@@ -365,7 +365,7 @@ module Everywhere
         override = env_value(VERSION_CODE_ENV) or return derived_version_code
 
         unless override.match?(/\A[1-9]\d*\z/)
-          UI.die!("#{VERSION_CODE_ENV}=#{override.inspect} — a versionCode is a positive integer, " \
+          UI.die!("#{VERSION_CODE_ENV}=#{override.inspect}: a versionCode is a positive integer, " \
                   "and Play only accepts an upload whose code is higher than the last one's")
         end
         override.to_i
@@ -431,7 +431,7 @@ module Everywhere
       def write_launcher_icons(work)
         source = @config.icon
         unless source && File.exist?(source)
-          UI.warn "no app icon (set app.icon or add icon.png) — shipping the placeholder"
+          UI.warn "no app icon (set app.icon or add icon.png). Shipping the placeholder"
           return
         end
 
@@ -439,7 +439,7 @@ module Everywhere
         res = File.join(work, "app", "src", "main", "res")
         return if Icon.write_android_mipmaps(source, res, background: background)
 
-        UI.warn "couldn't read #{File.basename(source)} — shipping the placeholder icon"
+        UI.warn "couldn't read #{File.basename(source)}. Shipping the placeholder icon"
       end
 
       # App-supplied images: native/android/assets/** → drawable resources, so
@@ -495,12 +495,12 @@ module Everywhere
         return if tabs.length <= BAR_SLOTS
 
         overflow = tabs.drop(BAR_TABS_WITH_MORE).map { |tab| tab["title"] }
-        UI.detail "#{tabs.length} tabs: Android's bar holds #{BAR_TABS_WITH_MORE} plus a \"More\" tab — " \
+        UI.detail "#{tabs.length} tabs: Android's bar holds #{BAR_TABS_WITH_MORE} plus a \"More\" tab. " \
                   "#{overflow.map(&:inspect).join(", ")} open from there (iOS does the same past five)"
 
         return if tabs.length <= CROWDED_TABS
 
-        UI.warn "#{overflow.length} tabs behind \"More\" is a long list to scroll on a phone — " \
+        UI.warn "#{overflow.length} tabs behind \"More\" is a long list to scroll on a phone. " \
                 "consider moving the rarer ones into the app's own navigation"
       end
 
@@ -514,7 +514,7 @@ module Everywhere
       def resolve_icon_font!(template)
         file = @config.native_android_icon_font_file
         @icon_font = nil
-        return unless file # icon_font: none — the app brings its own drawables
+        return unless file # icon_font: none, so the app brings its own drawables
 
         @icon_font = if @config.native_android_icon_font_fetched?
                        fetch_icon_font!(file)
@@ -529,7 +529,7 @@ module Everywhere
         dir = BUNDLED_FONT_DIRS.map { |parts| File.join(template, *parts) }
                                .find { |candidate| File.file?(File.join(candidate, file)) }
         unless dir
-          UI.die!("the Android template has no #{file} — reinstall ruby_everywhere, or set " \
+          UI.die!("the Android template has no #{file}. Reinstall ruby_everywhere, or set " \
                   "native.android.icon_font: symbols in config/everywhere.yml to fetch one instead")
         end
 
@@ -557,7 +557,7 @@ module Everywhere
         self.class.fetch!(icon_font_url(pin["path"], ".codepoints"), codepoints, sha256: pin["codepoints"])
         resolved
       rescue StandardError => e
-        UI.die!("couldn't fetch the #{@config.native_android_icon_font} icon font (#{e.message}) — " \
+        UI.die!("couldn't fetch the #{@config.native_android_icon_font} icon font (#{e.message}). " \
                 "set native.android.icon_font: classic in config/everywhere.yml to use the 2,235-icon " \
                 "set bundled in the gem, which needs no download and works offline")
       end
@@ -630,8 +630,8 @@ module Everywhere
           icon = tab["icon"].to_s
           base = icon.sub(/\.fill\z/, "")
           if base != icon && !fillable
-            "tab #{tab["title"].inspect}: #{icon.inspect} — icon_font: #{@config.native_android_icon_font} " \
-              "has no FILL axis; use #{base.inspect}, or switch to icon_font: symbols"
+            "tab #{tab["title"].inspect}: #{icon.inspect} needs a FILL axis, which icon_font: " \
+              "#{@config.native_android_icon_font} lacks; use #{base.inspect}, or switch to icon_font: symbols"
           elsif !names.include?(base)
             "tab #{tab["title"].inspect}: #{icon.inspect} isn't an icon in #{font["file"]}"
           end
@@ -639,7 +639,7 @@ module Everywhere
         return if problems.empty?
 
         UI.die!("icons.android in config/everywhere.yml (#{names.size} icons in " \
-                "#{font["file"]} — browse them at https://fonts.google.com/icons):\n#{problems.join("\n")}")
+                "#{font["file"]}, browse them at https://fonts.google.com/icons):\n#{problems.join("\n")}")
       end
 
       # "name<space>hex" per line. Parsed in Ruby rather than shelled out to awk
@@ -690,7 +690,7 @@ module Everywhere
                Do not edit; this source set is merged into app/src/main/AndroidManifest.xml.
                Note for whoever edits this header next: XML forbids a double hyphen
                anywhere inside a comment, so the flag spelling of the target must not
-               appear here. aapt2 does not warn — the manifest merger just fails to
+               appear here. aapt2 does not warn: the manifest merger just fails to
                parse the file, minutes into a Gradle run. -->
           <manifest xmlns:android="http://schemas.android.com/apk/res/android">
           #{body.empty? ? "    <!-- nothing declared: no permissions, deep links or sign-in scheme -->" : body}
@@ -731,7 +731,7 @@ module Everywhere
         UI.step("stamping deep links #{UI.dim("(#{domains.join(", ")})")}")
         unless @config.deep_linking_android?
           UI.warn "deep links won't auto-verify until deep_linking.android.package and " \
-                  ".sha256_cert_fingerprints are set — until then they open in the browser"
+                  ".sha256_cert_fingerprints are set. Until then they open in the browser"
         end
 
         domains.map do |host|
@@ -804,7 +804,7 @@ module Everywhere
           UI.step("bundling native extensions #{UI.dim("(#{sources.length} Kotlin files from native/android)")}")
         elsif sources.any?
           UI.warn "native/android has Kotlin files but everywhere.yml declares nothing under " \
-                  "native.android — they compile in, but no components/screens/splash are hooked up"
+                  "native.android. They compile in, but no components/screens/splash are hooked up"
         end
         File.write(File.join(dest, "EverywhereExtensions.kt"), extensions_registry)
       end
@@ -826,7 +826,7 @@ module Everywhere
         return if declared == EXTENSIONS_PACKAGE
 
         found = declared ? "declares package #{declared}" : "declares no package"
-        UI.die!("native/android/#{relative} #{found} — every Kotlin file here compiles into the " \
+        UI.die!("native/android/#{relative} #{found}. Every Kotlin file here compiles into the " \
                 "shell's extension package, so it has to start with:\n\n  package #{EXTENSIONS_PACKAGE}\n")
       end
 
@@ -846,7 +846,7 @@ module Everywhere
         splash = @config.native_android_splash
 
         <<~KOTLIN
-          // Written by `every build` from config/everywhere.yml (native.android) — do
+          // Written by `every build` from config/everywhere.yml (native.android). Do
           // not edit; changes belong in native/android/ and everywhere.yml.
           //
           // A declared component is registered under the name its own class states:
@@ -897,7 +897,7 @@ module Everywhere
         lines = packages.map { |coordinate| "    add(\"implementation\", \"#{coordinate}\")" }
         File.write(File.join(work, "app", "native-packages.gradle.kts"), <<~KOTLIN)
           // Written by `every build` from config/everywhere.yml (native.android.packages)
-          // — do not edit; changes belong in config/everywhere.yml.
+          // Do not edit; changes belong in config/everywhere.yml.
           dependencies {
           #{lines.join("\n")}
           }
@@ -966,7 +966,7 @@ module Everywhere
         UI.die!("Gradle reported success but no #{kind} in #{UI.short_path(dir)}") unless built
 
         if @signing && built == unsigned
-          UI.die!("Gradle wrote #{File.basename(built)} — EVERY_ANDROID_KEYSTORE is set, so the " \
+          UI.die!("Gradle wrote #{File.basename(built)}, but EVERY_ANDROID_KEYSTORE is set, so the " \
                   "#{kind} should have been signed. Check the keystore's alias and passwords")
         end
 

data/lib/everywhere/builders/desktop.rb

@@ -119,7 +119,7 @@ module Everywhere
         path = File.join(work, "Cargo.toml")
         source = File.read(path)
         unless source.include?(CRATES_MARKER)
-          UI.die!("the shell's Cargo.toml has no #{CRATES_MARKER} marker — " \
+          UI.die!("the shell's Cargo.toml has no #{CRATES_MARKER} marker. " \
                   "reinstall ruby_everywhere or pass --shell-dir at your own risk")
         end
 
@@ -186,7 +186,7 @@ module Everywhere
         # declare the file in everywhere.yml, it's to delete it.
         sources = NativeSources.copy!(from: source, into: dest, ext: "rs") do |_path, relative|
           if File.basename(relative) == MODULE_FILE
-            UI.die!("#{File.join(SOURCE_DIR, relative)} is generated by `every build` — " \
+            UI.die!("#{File.join(SOURCE_DIR, relative)} is generated by `every build`. " \
                     "remove it and let the CLI write the module declarations")
           end
         end
@@ -222,7 +222,7 @@ module Everywhere
 
       def child_module(children)
         <<~RUST
-          // GENERATED by `every build` — edits are overwritten.
+          // GENERATED by `every build`. Edits are overwritten.
           // Declares the .rs files this directory holds in native/desktop/.
           #{children.map { |name| "pub mod #{name};" }.join("\n")}
         RUST
@@ -244,7 +244,7 @@ module Everywhere
         <<~RUST
           // GENERATED by `every build` / `every dev` from `native.desktop` in
           // config/everywhere.yml, alongside the app's own .rs files copied from
-          // native/desktop/. Edits are overwritten — see support/desktop/README.md.
+          // native/desktop/. Edits are overwritten; see support/desktop/README.md.
 
           use crate::commands;
 
@@ -256,7 +256,7 @@ module Everywhere
           // is the right way to find out.
           #{glob_imports(paths)}
 
-          // The shell's own commands. App extensions answer over events instead —
+          // The shell's own commands. App extensions answer over events instead,
           // see extension_host.rs for why invoke isn't available to them.
           pub fn handler() -> impl Fn(tauri::ipc::Invoke<tauri::Wry>) -> bool + Send + Sync + 'static {
               tauri::generate_handler![commands::notify_command]
@@ -274,7 +274,7 @@ module Everywhere
           ) -> Result<serde_json::Value, String> {
               match name {
           #{dispatch_arms(commands)}        _ => Err(format!(
-                      "no desktop command named {name:?} — declare it under native.desktop.commands"
+                      "no desktop command named {name:?}. Declare it under native.desktop.commands"
                   )),
               }
           }

data/lib/everywhere/builders/ios.rb

@@ -90,14 +90,14 @@ module Everywhere
 
       def preflight!(dev:, device:)
         unless dev || @config.remote?
-          UI.die!("iOS apps are remote mode only for now — set remote.url in config/everywhere.yml " \
+          UI.die!("iOS apps are remote mode only for now. Set remote.url in config/everywhere.yml " \
                   "(the shell wraps your deployed app; nothing is packaged locally)")
         end
         UI.die!("iOS builds need macOS with Xcode installed") unless Host.darwin?
         return device_preflight! if device
         return if Shellout.run?("xcrun", "--sdk", "iphonesimulator", "--show-sdk-path", quiet: true)
 
-        UI.die!("no iOS Simulator SDK — install full Xcode (not just the Command Line Tools) " \
+        UI.die!("no iOS Simulator SDK. Install full Xcode (not just the Command Line Tools) " \
                 "and run: xcode-select --switch /Applications/Xcode.app")
       end
 
@@ -106,17 +106,17 @@ module Everywhere
       # its own vocabulary.
       def device_preflight!
         unless Shellout.run?("xcrun", "--sdk", "iphoneos", "--show-sdk-path", quiet: true)
-          UI.die!("no iOS device SDK — install full Xcode (not just the Command Line Tools) " \
+          UI.die!("no iOS device SDK. Install full Xcode (not just the Command Line Tools) " \
                   "and run: xcode-select --switch /Applications/Xcode.app")
         end
 
         missing = REQUIRED_SIGNING_ENV.reject { |name, _| env_value(name) }
         unless missing.empty?
           UI.die!("device builds need signing credentials in the environment:\n" \
-                  "#{missing.map { |name, hint| "  #{name} — #{hint}" }.join("\n")}")
+                  "#{missing.map { |name, hint| "  #{name}: #{hint}" }.join("\n")}")
         end
         unless EXPORT_METHODS.include?(export_method)
-          UI.die!("EVERY_IOS_EXPORT_METHOD=#{export_method} isn't an Xcode export method — " \
+          UI.die!("EVERY_IOS_EXPORT_METHOD=#{export_method} isn't an Xcode export method. " \
                   "use one of: #{EXPORT_METHODS.join(", ")}")
         end
         asc_preflight! if store_channel?
@@ -129,12 +129,12 @@ module Everywhere
                          .map { |_field, names| names.first }
         unless missing.empty?
           UI.die!("--channel #{@channel} uploads to App Store Connect, which needs an API key " \
-                  "in the environment — missing: #{missing.join(", ")}")
+                  "in the environment. Missing: #{missing.join(", ")}")
         end
         return if export_method == DEFAULT_EXPORT_METHOD
 
         UI.die!("--channel #{@channel} uploads to App Store Connect, which only accepts " \
-                "#{DEFAULT_EXPORT_METHOD} builds — unset EVERY_IOS_EXPORT_METHOD=#{export_method}")
+                "#{DEFAULT_EXPORT_METHOD} builds. Unset EVERY_IOS_EXPORT_METHOD=#{export_method}")
       end
 
       def asc_env(field) = ASC_ENV.fetch(field).filter_map { |name| env_value(name) }.first
@@ -284,7 +284,7 @@ module Everywhere
           FileUtils.mkdir_p(dest)
           File.write(File.join(dest, "EverywhereExtensions.swift"), extensions_registry)
         elsif sources.any?
-          UI.warn "native/ios has Swift files but everywhere.yml declares nothing under native.ios — " \
+          UI.warn "native/ios has Swift files but everywhere.yml declares nothing under native.ios. " \
                   "they compile in, but no components/screens/splash are hooked up"
         end
       end
@@ -309,7 +309,7 @@ module Everywhere
           end
 
         <<~SWIFT
-          // Written by `every build` from config/everywhere.yml (native.ios) — do
+          // Written by `every build` from config/everywhere.yml (native.ios). Do
           // not edit; changes belong in native/ios/ and everywhere.yml.
           import HotwireNative
           import SwiftUI
@@ -368,7 +368,7 @@ module Everywhere
         <<~SWIFT
           // swift-tools-version:5.9
           // Written by `every build` from config/everywhere.yml (native.ios.packages)
-          // — do not edit; changes belong in config/everywhere.yml.
+          // Do not edit; changes belong in config/everywhere.yml.
           import PackageDescription
 
           let package = Package(
@@ -390,7 +390,7 @@ module Everywhere
       def native_package_exports(packages)
         header = <<~SWIFT
           // Written by `every build` from config/everywhere.yml (native.ios.packages)
-          // — re-exports each pinned product so native/ios code reaches it with a
+          // Re-exports each pinned product so native/ios code reaches it with a
           // single `import NativeExtensions`. Do not edit.
         SWIFT
         modules = packages.flat_map { |pkg| pkg["products"] }.uniq.sort
@@ -489,7 +489,7 @@ module Everywhere
         # and no deep linking gets the setting just the same.
         entitlements = "CODE_SIGN_ENTITLEMENTS = App/App.entitlements\n" if entitlement_keys.any?
         File.write(File.join(work, "App", "App.xcconfig"), <<~XCCONFIG)
-          // Written by `every build --target #{@target}` — do not edit; changes
+          // Written by `every build --target #{@target}`. Do not edit; changes
           // belong in config/everywhere.yml.
           PRODUCT_BUNDLE_IDENTIFIER = #{xcconfig_value(bundle_id)}
           MARKETING_VERSION = #{xcconfig_value(@config.version(target: @target))}
@@ -526,7 +526,7 @@ module Everywhere
       def write_app_icon(work)
         source = @config.icon
         unless source && File.exist?(source)
-          UI.warn "no app icon (set app.icon or add icon.png) — shipping the placeholder"
+          UI.warn "no app icon (set app.icon or add icon.png). Shipping the placeholder"
           return
         end
 
@@ -534,7 +534,7 @@ module Everywhere
         dest = File.join(work, "App", "Assets.xcassets", "AppIcon.appiconset")
         return if Icon.write_ios_appiconset(source, dest, background: background)
 
-        UI.warn "couldn't read #{File.basename(source)} — shipping the placeholder icon"
+        UI.warn "couldn't read #{File.basename(source)}. Shipping the placeholder icon"
       end
 
       # App-supplied images: native/ios/assets/** → imagesets in the template's

data/lib/everywhere/builders/native_sources.rb

@@ -22,7 +22,7 @@ module Everywhere
           relative = file.delete_prefix("#{from}/")
           if reserved && File.basename(file) == reserved
             UI.die!("#{label}/#{relative}: #{reserved} is generated by " \
-                    "`every build` — declare your types under #{label.tr("/", ".")} " \
+                    "`every build`. Declare your types under #{label.tr("/", ".")} " \
                     "in config/everywhere.yml instead")
           end
           yield(file, relative) if block_given?

data/lib/everywhere/child_supervision.rb

@@ -160,7 +160,7 @@ module Everywhere
       deadline = Clock.monotonic + timeout
       until port_open?(port)
         if pid && !ChildProcesses.alive?(pid)
-          UI.die!("the dev server exited before it finished booting — scroll up for its output#{dead_hint}")
+          UI.die!("the dev server exited before it finished booting. Scroll up for its output#{dead_hint}")
         end
         if Clock.monotonic > deadline
           UI.die!("dev server never came up on port #{port} after #{timeout}s#{slow_hint}")

data/lib/everywhere/commands/build.rb

@@ -41,18 +41,40 @@ module Everywhere
 
       ANDROID_FORMATS = %w[apk aab].freeze
 
-      desc "Package the app natively — desktop (tebako press + shell + .app), iOS (--ios) or Android (--android)"
+      # Valid --channel per target os — the pairing the Platform accepts, minus
+      # nothing: macOS has no store channel (every macOS release signs with
+      # Developer ID and notarizes regardless), iOS `direct` exports an .ipa
+      # without the App Store Connect upload, and Android's `play` selects the
+      # .aab path even though the Play upload itself happens platform-side.
+      CHANNELS = {
+        "macos"   => %w[direct],
+        "ios"     => %w[direct testflight app_store],
+        "android" => %w[direct play]
+      }.freeze
+
+      # An os/channel pair nothing implements has to be refused before the
+      # build runs — it used to land in the receipt, where the only thing
+      # reading it (a store upload, a Platform distributor) has no idea what
+      # it means. Unknown os falls through: builder dispatch owns that error.
+      def self.check_channel!(os, channel)
+        allowed = CHANNELS[os] || return
+        return if allowed.include?(channel)
+
+        UI.die!("unsupported --channel #{channel} for #{os}: use one of #{allowed.join(", ")}")
+      end
+
+      desc "Package the app natively: desktop (tebako press + shell + .app), iOS (--ios) or Android (--android)"
 
       option :root,       default: ".",              desc: "App root directory"
       option :output,     default: nil,              desc: "Output binary path (default: dist/<app>)"
       option :ruby,       default: "4.0.6",          desc: "Ruby version to package"
       option :entry,      default: "native_boot.rb", desc: "Entry point script relative to root"
       option :app_name,   default: nil,              desc: "Bundle name (default: capitalized app dir)"
-      option :target,     default: nil,              desc: "Build target (os-arch) — selects any platforms: overrides, default: \"#{DEFAULT_TARGET}\""
+      option :target,     default: nil,              desc: "Build target (os-arch); selects any platforms: overrides, default: \"#{DEFAULT_TARGET}\""
       option :shell_dir,  default: nil,              desc: "Path to the shell's src-tauri directory"
       option :template_dir, default: nil,             desc: "Path to the mobile shell template (iOS/Android)"
       option :format,     default: nil,               desc: "Android artifact format (#{ANDROID_FORMATS.join(" | ")}), default: apk"
-      option :channel,    default: "direct",          desc: "Distribution channel (direct | testflight | app_store | ...)"
+      option :channel,    default: "direct",          desc: "Distribution channel (#{CHANNELS.map { |os, chs| "#{os}: #{chs.join("|")}" }.join(" · ")})"
       option :ios,        type: :boolean, default: false, desc: "Build the iOS shell (shorthand for --target #{IOS_TARGET})"
       option :android,    type: :boolean, default: false, desc: "Build the Android shell (shorthand for --target #{ANDROID_TARGET})"
       option :device,     type: :boolean, default: false, desc: "iOS: archive and export a device .ipa instead of a simulator .app"
@@ -62,6 +84,7 @@ module Everywhere
                app_name: nil, target: nil, shell_dir: nil, template_dir: nil, format: nil,
                channel: "direct", ios: false, android: false, device: false, skip_press: false, **)
         target = resolve_target(target, ios: ios, android: android)
+        self.class.check_channel!(Everywhere::Config.os_of(target), channel)
         root_path = File.expand_path(root)
         config = Everywhere::Config.load(root_path)
 
@@ -70,11 +93,7 @@ module Everywhere
         identity_errors = config.identity_errors
         UI.die!("app: in config/everywhere.yml:\n#{identity_errors.join("\n")}") unless identity_errors.empty?
 
-        # Baked into the shell here, so this is the last moment an unusable
-        # feed URL can still be a message instead of a shipped app that never
-        # updates.
-        updates_errors = config.updates_errors
-        UI.die!("updates: in config/everywhere.yml:\n#{updates_errors.join("\n")}") unless updates_errors.empty?
+        check_updates!(config)
 
         # Builder dispatch on the target's os (build-engine.md §5): macOS is
         # the tebako+tauri path below; the mobile targets are Hotwire Native
@@ -96,7 +115,7 @@ module Everywhere
           # thirty minutes in, at the first codesign.
           UI.die!("desktop builds are macOS-only for now") unless Host.darwin?
         else
-          UI.die!("no builder for target #{target} — macos-*, ios-* and android-* are supported")
+          UI.die!("no builder for target #{target}: macos-*, ios-* and android-* are supported")
         end
 
         # Desktop chrome is validated here rather than in the shell: a typo in
@@ -131,7 +150,7 @@ module Everywhere
         entry_path = File.join(framework.root, entry)
 
         unless File.exist?(entry_path)
-          UI.die!("#{entry} not found in #{framework.root} — the app needs a packaged-boot entry point")
+          UI.die!("#{entry} not found in #{framework.root}: the app needs a packaged-boot entry point")
         end
 
         UI.step("#{UI.bold(framework.name)} app at #{framework.root}")
@@ -150,6 +169,17 @@ module Everywhere
 
       private
 
+      # The `updates:` section is baked into the shell during this build, so
+      # this is the last moment an unusable feed URL — or a missing signing key,
+      # which turns the updater off without failing anything — can still be a
+      # message instead of a shipped app that never updates. Once per build:
+      # every target stamps everywhere.json from the same config.
+      def check_updates!(config)
+        errors = config.updates_errors
+        UI.die!("updates: in config/everywhere.yml:\n#{errors.join("\n")}") unless errors.empty?
+        config.updates_warnings.each { |warning| UI.warn(warning) }
+      end
+
       # `--ios` is the shorthand `every dev` already has; --target stays the
       # general escape hatch. Both given is only an error when they disagree —
       # `--ios --target ios-arm64` is redundant, not wrong.
@@ -168,10 +198,10 @@ module Everywhere
       # `--format aab` on an iOS build is a mistake we name rather than drop.
       def mobile_options(target, device:, format:)
         if Everywhere::Config.os_of(target) == "android"
-          UI.die!("--device is an iOS flag — Android always builds an installable artifact") if device
+          UI.die!("--device is an iOS flag; Android always builds an installable artifact") if device
           { format: android_format(format) }
         else
-          UI.die!("--format is an Android flag (#{ANDROID_FORMATS.join("/")}) — iOS builds a .app or .ipa") if format
+          UI.die!("--format is an Android flag (#{ANDROID_FORMATS.join("/")}); iOS builds a .app or .ipa") if format
           { device: device }
         end
       end
@@ -192,7 +222,7 @@ module Everywhere
       def resolve_target(target, ios:, android:)
         asked = { ios: ios, android: android }.select { |_flag, given| given }.keys
         if asked.length > 1
-          UI.die!("#{asked.map { |f| "--#{f}" }.join(" and ")} are separate builds — run one at a time")
+          UI.die!("#{asked.map { |f| "--#{f}" }.join(" and ")} are separate builds; run one at a time")
         end
 
         flag = asked.first or return target || DEFAULT_TARGET
@@ -200,7 +230,7 @@ module Everywhere
         return shorthand if target.nil?
 
         unless Everywhere::Config.os_of(target) == flag.to_s
-          UI.die!("--#{flag} and --target #{target} disagree — drop one " \
+          UI.die!("--#{flag} and --target #{target} disagree; drop one " \
                   "(--#{flag} means --target #{shorthand})")
         end
         target
@@ -213,7 +243,7 @@ module Everywhere
       def ensure_gems!(framework)
         return if Shellout.run?(bundle_env, "bundle", "check", chdir: framework.root, quiet: true)
 
-        UI.step("installing gems #{UI.dim("(bundle install — first build on this machine)")}")
+        UI.step("installing gems #{UI.dim("(bundle install, first build on this machine)")}")
         Shellout.run!(bundle_env, "bundle", "install", chdir: framework.root)
       end
 
@@ -242,7 +272,7 @@ module Everywhere
         FileUtils.mkdir_p(out_dir)
         log = "#{output_path}.build.log"
 
-        UI.step("rbe-tebako press #{UI.dim("(Ruby #{ruby} — grab a coffee on first run)")}")
+        UI.step("rbe-tebako press #{UI.dim("(Ruby #{ruby}, grab a coffee on first run)")}")
         UI.detail("full build log: #{UI.short_path(log)}")
         # Press into a staging dir OUTSIDE the app root for the same reason
         # (tebako prints a big warning about exactly this). Block form: the
@@ -275,7 +305,7 @@ module Everywhere
       # Rails binary (as "app-server") + Info.plist + icon. An .app bundle is
       # just a directory layout, so we build it by hand — no tauri-cli needed.
       def bundle_app(server_binary, app_name, shell_dir, config, target: DEFAULT_TARGET, dist_dir: nil)
-        Shellout.ensure_tool!("cargo", "the desktop shell is built with Rust — install it from https://rustup.rs")
+        Shellout.ensure_tool!("cargo", "the desktop shell is built with Rust; install it from https://rustup.rs")
 
         # cargo always runs in a STAGED copy of the shell under
         # ~/.rubyeverywhere, never in the gem (or in whatever --shell-dir
@@ -290,7 +320,7 @@ module Everywhere
 
         icon_source = config.icon
         if icon_source && !File.exist?(icon_source)
-          UI.warn "app icon #{icon_source} not found — using placeholder"
+          UI.warn "app icon #{icon_source} not found, using placeholder"
           icon_source = nil
         end
         UI.step("app icon: #{icon_source ? icon_source.sub("#{config.root}/", "") : "placeholder"}")
@@ -307,7 +337,7 @@ module Everywhere
           if Everywhere::Icon.macos_master(icon_source, shaped)
             macos_icon = shaped
           else
-            UI.warn "couldn't read #{File.basename(icon_source)} (unusual PNG flavor) — using it unshaped"
+            UI.warn "couldn't read #{File.basename(icon_source)} (unusual PNG flavor), using it unshaped"
           end
         end
 
@@ -329,7 +359,7 @@ module Everywhere
                       true
                     end
           unless dock_ok
-            UI.warn "couldn't convert #{File.basename(icon_source)} to RGBA (unusual PNG flavor) — Dock icon will use the placeholder after launch"
+            UI.warn "couldn't convert #{File.basename(icon_source)} to RGBA (unusual PNG flavor); the Dock icon will use the placeholder after launch"
           end
           FileUtils.touch(File.join(shell_dir, "build.rs"))
         end
@@ -359,7 +389,7 @@ module Everywhere
             FileUtils.cp(splash, File.join(resources, "splash.html"))
             UI.step("custom splash: #{splash.sub("#{config.root}/", "")}")
           else
-            UI.warn "splash #{splash} not found — using the default splash"
+            UI.warn "splash #{splash} not found, using the default splash"
           end
         end
         icns = make_icns(macos_icon || shell_icon, resources)
@@ -403,7 +433,7 @@ module Everywhere
         out = File.join(resources_dir, "AppIcon.icns")
         Everywhere::Icon.write_icns(png, out) ? "AppIcon" : nil
       rescue StandardError => e
-        UI.warn "icon generation failed (#{e.message}) — bundling without a custom icon"
+        UI.warn "icon generation failed (#{e.message}), bundling without a custom icon"
         nil
       end
 
@@ -450,7 +480,7 @@ module Everywhere
         end
 
         press_warnings = text.scan(/^.*(?:WARNING|Warning):(?!.*bitcode).*$/).size
-        UI.warn "tebako emitted #{press_warnings} warning(s) — details in the build log" if press_warnings.positive?
+        UI.warn "tebako emitted #{press_warnings} warning(s), details in the build log" if press_warnings.positive?
       end
 
       def press_env
@@ -467,7 +497,7 @@ module Everywhere
       # a silently dropped PATH entry.
       def bison_prefix!
         @bison_prefix ||= brew_bison_prefix ||
-                          UI.die!("tebako needs Homebrew bison — brew install bison")
+                          UI.die!("tebako needs Homebrew bison: brew install bison")
       end
 
       def brew_bison_prefix

data/lib/everywhere/commands/clean.rb

@@ -36,7 +36,7 @@ module Everywhere
           FileUtils.rm_rf(dir)
           UI.ok("removed #{dir} #{UI.dim("(#{human_size(bytes)})")}")
         end
-        UI.success("#{human_size(freed)} freed — the next build recompiles the shells cold")
+        UI.success("#{human_size(freed)} freed; the next build recompiles the shells cold")
       end
 
       private