ruby_everywhere 0.12.0 → 0.14.0

data/lib/everywhere/commands/preview.rb

@@ -65,7 +65,7 @@ module Everywhere
           @url = lan_url(port)
         else
           Shellout.ensure_tool!("cloudflared",
-                                "preview shares your app through a Cloudflare tunnel — " \
+                                "preview shares your app through a Cloudflare tunnel. " \
                                 "install it with `brew install cloudflared` (or use --lan)")
           validate_tunnel_options!(tunnel_name, tunnel_host)
           @tunnel_name = tunnel_name
@@ -98,10 +98,10 @@ module Everywhere
       rescue Everywhere::Error
         config = Everywhere::Config.load(Dir.pwd)
         if config.remote?
-          UI.die!("this app runs in remote mode — #{config.remote_url} is already online, share that " \
+          UI.die!("this app runs in remote mode; #{config.remote_url} is already online, share that " \
                   "(preview tunnels a local dev server)")
         end
-        UI.die!("no supported framework here (Rails, Hanami, Sinatra) — nothing to preview")
+        UI.die!("no supported framework here (Rails, Hanami, Sinatra), so nothing to preview")
       end
 
       def validate_tunnel_options!(name, host)
@@ -120,7 +120,7 @@ module Everywhere
 
       def lan_ip
         Socket.ip_address_list.find { |a| a.ipv4_private? }&.ip_address ||
-          UI.die!("couldn't find a LAN address — is this machine on a network? (drop --lan to tunnel instead)")
+          UI.die!("couldn't find a LAN address. Is this machine on a network? (drop --lan to tunnel instead)")
       end
 
       def start_tunnel
@@ -146,7 +146,7 @@ module Everywhere
         begin
           Timeout.timeout(TUNNEL_BOOT_TIMEOUT) { found.pop }
         rescue Timeout::Error
-          UI.die!("cloudflared never reported a tunnel URL after #{TUNNEL_BOOT_TIMEOUT}s — " \
+          UI.die!("cloudflared never reported a tunnel URL after #{TUNNEL_BOOT_TIMEOUT}s. " \
                   "check your network, or share on the local network with --lan")
         end
       end
@@ -170,11 +170,11 @@ module Everywhere
         service = rule&.fetch("service", nil).to_s
         if rule.nil? || service.start_with?("http_status:")
           UI.warn("your cloudflared config's ingress has no rule for #{@tunnel_host}, so requests will get " \
-                  "the catch-all#{" (#{service})" unless service.empty?} — add " \
+                  "the catch-all#{" (#{service})" unless service.empty?}. Add " \
                   "{hostname: #{@tunnel_host}, service: http://localhost:#{@port}} above it")
         elsif (svc_port = service[/:(\d+)\z/, 1]) && Integer(svc_port) != @port
           UI.warn("your cloudflared config's ingress routes #{@tunnel_host} to #{service}, but this preview " \
-                  "runs on port #{@port} — point that rule at the preview, or match it with --port #{svc_port}")
+                  "runs on port #{@port}. Point that rule at the preview, or match it with --port #{svc_port}")
         end
       end
 
@@ -220,7 +220,7 @@ module Everywhere
       def warn_if_loopback_only
         TCPSocket.new(URI.parse(@url).host, Integer(@port)).close
       rescue SystemCallError
-        UI.warn("the dev server only listens on 127.0.0.1, so phones can't reach #{@url} — " \
+        UI.warn("the dev server only listens on 127.0.0.1, so phones can't reach #{@url}. " \
                 "make bin/dev bind 0.0.0.0 (rackup --host 0.0.0.0), or drop --lan and tunnel instead")
       end
 
@@ -231,7 +231,7 @@ module Everywhere
         # here: that process has no EVERYWHERE_JUMP_TOKEN, so Jump could never
         # pair with it.
         if port_open?(port)
-          UI.die!("something is already running on port #{port} — preview must start the server itself " \
+          UI.die!("something is already running on port #{port}, and preview must start the server itself " \
                   "(the Jump pairing token rides in on its environment); stop that server or use --port")
         end
 
@@ -277,7 +277,7 @@ module Everywhere
         UI.step("preview of #{UI.bold(@config.name)} is live → #{UI.cyan(@url)}")
         show_qr
         Console.print("  #{UI.dim("Manual entry in Jump:")} #{@url}  #{UI.dim("token:")} #{@token}\n")
-        Console.print(UI.dim("  The link and QR die with this session — Ctrl-C to stop.\n"))
+        Console.print(UI.dim("  The link and QR die with this session. Ctrl-C to stop.\n"))
         UI.note("waiting for devices…", marker: "⌁", color: :cyan)
         @dock.show(footer_facts)
       end
@@ -321,7 +321,7 @@ module Everywhere
           end.join + "\e[0m"
         end.join("\n") << "\e[0m\n"
       rescue LoadError, StandardError => e
-        UI.warn("couldn't render the QR (#{e.class}) — use the link below")
+        UI.warn("couldn't render the QR (#{e.class}); use the link below")
         ""
       end
 
@@ -345,12 +345,12 @@ module Everywhere
           break if pause == :quit
 
           web = @children.synchronize { @pids[:web] }
-          UI.die!("the dev server exited — scroll up for its output") unless ChildProcesses.alive?(web)
+          UI.die!("the dev server exited. Scroll up for its output") unless ChildProcesses.alive?(web)
 
           tunnel = @children.synchronize { @pids[:tunnel] }
           next if tunnel.nil? || ChildProcesses.alive?(tunnel)
 
-          UI.warn("the tunnel dropped — starting a fresh one")
+          UI.warn("the tunnel dropped, starting a fresh one")
           stop_child(:tunnel)
           @url = start_tunnel
           announce
@@ -384,7 +384,7 @@ module Everywhere
         wait_for_port_close
         @config = Everywhere::Config.load(@framework.root)
         start_server(@port.to_s)
-        UI.note("server is back — same URL, same token", marker: "⌁", color: :cyan)
+        UI.note("server is back, same URL, same token", marker: "⌁", color: :cyan)
       end
 
       # stop_child has reaped the process, but the kernel can hold its listener

data/lib/everywhere/commands/publish.rb

@@ -60,7 +60,7 @@ module Everywhere
         # RequestTimeExpired means the bucket rejected our SigV4 timestamp; the
         # only realistic local cause is a wrong clock, and the raw message
         # doesn't say so.
-        hint = e.message.include?("RequestTimeExpired") ? " — your system clock may be off by more than 15 minutes" : ""
+        hint = e.message.include?("RequestTimeExpired") ? " (your system clock may be off by more than 15 minutes)" : ""
         UI.die!("#{e.message}#{hint}")
       end
 
@@ -106,12 +106,12 @@ module Everywhere
 
         if (head = s3.head_object(key))
           if head[:size] == size && !force
-            UI.step("#{filename} already published (same size) — skipping upload")
+            UI.step("#{filename} already published (same size), skipping upload")
             s3.put_object("#{key}.minisig", signature, content_type: "text/plain",
                           cache_control: UpdateManifest::IMMUTABLE_CACHE)
             return
           elsif !force
-            UI.die!("#{key} already exists with different content (#{head[:size]} vs #{size} bytes) — " \
+            UI.die!("#{key} already exists with different content (#{head[:size]} vs #{size} bytes). " \
                     "bump app.version, or pass --force if you really mean to replace a published artifact")
           end
         end
@@ -157,32 +157,32 @@ module Everywhere
 
         filename = File.basename(entry.fetch("url"))
         key = UpdateManifest.artifact_key(filename, **loc)
-        UI.die!("#{key} is gone from the bucket — can't roll back to it") unless s3.head_object(key)
+        UI.die!("#{key} is gone from the bucket, so can't roll back to it") unless s3.head_object(key)
 
         s3.put_object(UpdateManifest.latest_key(**loc), UpdateManifest.to_json(entry),
                       content_type: "application/json", cache_control: UpdateManifest::MUTABLE_CACHE)
         maintain_alias!(filename, version, loc) if latest_alias
 
         UI.ok("latest.json → #{version}")
-        UI.warn("apps still on newer #{entry["app"] ? entry.dig("app", "name") : "builds"} won't downgrade — " \
+        UI.warn("apps still on newer #{entry["app"] ? entry.dig("app", "name") : "builds"} won't downgrade. " \
                 "ship a higher-versioned release to move everyone off the bad build")
       end
 
       # ---- pieces --------------------------------------------------------------
 
       def preflight!
-        UI.die!("no updates.url in everywhere.yml — add an updates: section (see `every updates keygen`)") unless @config.updates_url
-        UI.die!("no updates.public_key in everywhere.yml — run `every updates keygen`") unless @config.updates_public_key
+        UI.die!("no updates.url in everywhere.yml. Add an updates: section (see `every updates keygen`)") unless @config.updates_url
+        UI.die!("no updates.public_key in everywhere.yml. Run `every updates keygen`") unless @config.updates_public_key
         updates_errors = @config.updates_errors
         UI.die!("updates: in config/everywhere.yml:\n#{updates_errors.join("\n")}") unless updates_errors.empty?
         UI.die!("no updates.s3.bucket in everywhere.yml") if s3_config["bucket"].to_s.empty?
-        UI.die!("missing bucket credentials — set AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY") if
+        UI.die!("missing bucket credentials. Set AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY") if
           ENV["AWS_ACCESS_KEY_ID"].to_s.empty? || ENV["AWS_SECRET_ACCESS_KEY"].to_s.empty?
       end
 
       def receipt_path(given)
         path = given ? File.expand_path(given, @root) : File.join(@root, "dist", "release.json")
-        UI.die!("no receipt at #{UI.short_path(path)} — run `every release` first") unless File.exist?(path)
+        UI.die!("no receipt at #{UI.short_path(path)}. Run `every release` first") unless File.exist?(path)
         path
       end
 
@@ -193,7 +193,7 @@ module Everywhere
       end
 
       def verify_artifact!(receipt, receipt_path)
-        filename = receipt.dig("artifact", "filename") or UI.die!("receipt has no artifact record — re-run `every release`")
+        filename = receipt.dig("artifact", "filename") or UI.die!("receipt has no artifact record. Re-run `every release`")
         path = File.join(File.dirname(receipt_path), filename)
         UI.die!("artifact #{UI.short_path(path)} not found next to the receipt") unless File.exist?(path)
 
@@ -219,14 +219,14 @@ module Everywhere
           File.write("#{path}.minisig", signature)
           Minisign.verify_file(path, signature: signature, public_key: @config.updates_public_key)
         rescue Minisign::Error
-          UI.die!("signature doesn't verify against updates.public_key in everywhere.yml — " \
+          UI.die!("signature doesn't verify against updates.public_key in everywhere.yml. " \
                   "the key you're signing with isn't the one shipped apps trust")
         end
         signature
       rescue Minisign::PassphraseError
-        UI.die!("wrong or missing key passphrase — set EVERYWHERE_SIGNING_KEY_PASSPHRASE")
+        UI.die!("wrong or missing key passphrase. Set EVERYWHERE_SIGNING_KEY_PASSPHRASE")
       rescue Minisign::Error => e
-        UI.die!("couldn't sign with that key — #{e.message}")
+        UI.die!("couldn't sign with that key: #{e.message}")
       end
 
       def signing_key(key_option)
@@ -241,7 +241,7 @@ module Everywhere
         return ENV["EVERYWHERE_SIGNING_KEY"] if ENV["EVERYWHERE_SIGNING_KEY"]
 
         default = File.join(Paths.cache_dir, "keys", "#{@config.bundle_id}.key")
-        UI.die!("no signing key found — run `every updates keygen` (looked for #{UI.short_path(default)})") unless File.exist?(default)
+        UI.die!("no signing key found. Run `every updates keygen` (looked for #{UI.short_path(default)})") unless File.exist?(default)
         File.read(default)
       end
 

data/lib/everywhere/commands/release.rb

@@ -33,7 +33,7 @@ module Everywhere
 
       option :root,       default: ".",              desc: "App root directory"
       option :target,     default: Build::DEFAULT_TARGET, desc: "Build target (os-arch)"
-      option :channel,    default: "direct",         desc: "Distribution channel (direct | mac_app_store | ...)"
+      option :channel,    default: "direct",         desc: "Distribution channel (#{Build::CHANNELS.map { |os, chs| "#{os}: #{chs.join("|")}" }.join(" · ")})"
       option :ruby,       default: nil,              desc: "Ruby version to package (default: build.ruby or 4.0.6)"
       option :entry,      default: "native_boot.rb", desc: "Entry point script relative to root"
       option :app_name,   default: nil,              desc: "Bundle name (default from config)"
@@ -50,7 +50,7 @@ module Everywhere
         ruby ||= config.build_ruby || "4.0.6"
         os = Everywhere::Config.os_of(target)
 
-        preflight!(target, os)
+        preflight!(target, os, channel)
 
         framework = config.remote? ? nil : Framework.detect(root_path)
         dist_dir = File.expand_path("dist", root_path)
@@ -82,18 +82,22 @@ module Everywhere
 
       private
 
-      def preflight!(target, os)
+      def preflight!(target, os, channel)
         case os
         when "macos"
           UI.die!("`every release --target #{target}` runs on macOS only") unless Host.darwin?
           UI.die!("signing engine missing at #{NOTARIZE}") unless File.exist?(NOTARIZE)
         when "ios"
-          UI.die!("iOS releases need Xcode — `every release --target #{target}` runs on macOS only") unless Host.darwin?
+          UI.die!("iOS releases need Xcode; `every release --target #{target}` runs on macOS only") unless Host.darwin?
         when "android"
           nil # Gradle signs and builds anywhere the Android SDK does — Linux included.
         else
-          UI.die!("no release builder for #{target} — macos-*, ios-* and android-* are supported")
+          UI.die!("no release builder for #{target}: macos-*, ios-* and android-* are supported")
         end
+
+        # After the os check: the channel table is keyed by os, so an unknown
+        # target has to die on the missing builder, not a nil channel list.
+        Build.check_channel!(os, channel)
       end
 
 
@@ -112,7 +116,7 @@ module Everywhere
       # then read the signing facts back off the bundle.
       def notarize!(dist_dir, root_path, env_file)
         app_dir = Dir[File.join(dist_dir, "*.app")].max_by { |d| File.mtime(d) }
-        UI.die!("no .app found in #{dist_dir} — run without --skip-build first") unless app_dir
+        UI.die!("no .app found in #{dist_dir}. Run without --skip-build first") unless app_dir
 
         env_file ||= discover_env_file(root_path)
         env = env_file ? { "ENV_FILE" => env_file } : {}
@@ -122,7 +126,7 @@ module Everywhere
         # with, and "which file did that come from" is the first thing you ask
         # when a build signs as the wrong team.
         UI.detail("signing env: #{UI.short_path(File.expand_path(env_file))}") if env_file
-        UI.warn("no signing env found — set APPLE_* or pass --env-file (notarize.sh will fail without it)") unless env_file || ENV["APPLE_SIGNING_IDENTITY"]
+        UI.warn("no signing env found. Set APPLE_* or pass --env-file (notarize.sh will fail without it)") unless env_file || ENV["APPLE_SIGNING_IDENTITY"]
         log = "#{app_dir}.notarize.log"
         Shellout.run_logged!(env, ["/bin/bash", NOTARIZE, app_dir], log: log, filter: LogFilter.for(:notarize))
 
@@ -139,7 +143,7 @@ module Everywhere
         found = patterns.filter_map { |p| Dir[File.join(dist_dir, p)].max_by { |f| File.mtime(f) } }.first
         return found if found
 
-        UI.die!("no #{patterns.map { |p| p.delete("*") }.join(" or ")} found in #{dist_dir} — " \
+        UI.die!("no #{patterns.map { |p| p.delete("*") }.join(" or ")} found in #{dist_dir}. " \
                 "run without --skip-build first")
       end
 
@@ -248,7 +252,7 @@ module Everywhere
 
       def summarize(out, artifact, signing, root_path)
         UI.phase("Release ready 🎉")
-        UI.success("#{UI.rel_path(artifact, root_path)} #{UI.dim("(#{(File.size(artifact) / 1024.0 / 1024.0).round}MB — send this to your friends)")}")
+        UI.success("#{UI.rel_path(artifact, root_path)} #{UI.dim("(#{(File.size(artifact) / 1024.0 / 1024.0).round}MB, send this to your friends)")}")
         UI.substep("signed by #{UI.bold(signing["signed_by"] || "?")}")
         UI.substep(signing_line(signing["platform"] || {}, signing))
         UI.substep("receipt #{UI.dim(UI.rel_path(out, root_path))}")

data/lib/everywhere/commands/updates_keygen.rb

@@ -26,7 +26,7 @@ module Everywhere
         pub_path = File.join(keys_dir, "#{bundle_id}.pub")
 
         if File.exist?(sk_path) && !force
-          UI.die!("a signing key for #{bundle_id} already exists at #{UI.short_path(sk_path)} — " \
+          UI.die!("a signing key for #{bundle_id} already exists at #{UI.short_path(sk_path)}. " \
                   "pass --force to overwrite it (shipped apps signed with the old key will REFUSE updates signed with a new one)")
         end
 
@@ -44,7 +44,7 @@ module Everywhere
 
         UI.ok("secret key #{UI.short_path(sk_path)} #{UI.dim("(key id #{pair.key_id_hex})")}")
         if secret.empty?
-          UI.warn("this secret key is stored UNENCRYPTED — anyone who can read " \
+          UI.warn("this secret key is stored UNENCRYPTED. Anyone who can read " \
                   "#{UI.short_path(sk_path)} can sign updates your users will install")
         end
         UI.ok("public key #{UI.short_path(pub_path)}")
@@ -57,7 +57,7 @@ module Everywhere
             public_key: "#{Minisign.public_key_base64(pair.public_key)}"
 
         YAML
-        UI.substep("back up the secret key somewhere safe — losing it strands every shipped app on its current version")
+        UI.substep("back up the secret key somewhere safe: losing it strands every shipped app on its current version")
       end
 
       private
@@ -85,7 +85,7 @@ module Everywhere
         pass = $stdin.getpass("Passphrase: ")
         confirm = $stdin.getpass("Passphrase (again): ")
         UI.die!("passphrases didn't match") unless pass == confirm
-        UI.die!("empty passphrase — rerun without --passphrase for an unencrypted key") if pass.empty?
+        UI.die!("empty passphrase. Rerun without --passphrase for an unencrypted key") if pass.empty?
         pass
       end
     end

data/lib/everywhere/config/app.rb

@@ -30,7 +30,7 @@ module Everywhere
           value = section["bundle_id"].to_s
           next if value.empty? || (value.match?(BUNDLE_ID) && !value.include?(".."))
 
-          "#{at}.bundle_id #{value.inspect} is not a bundle id — reverse-DNS, " \
+          "#{at}.bundle_id #{value.inspect} is not a bundle id. Use reverse-DNS: " \
             "letters/digits/./- only (like com.example.app)"
         end + android_target_sdk_errors
       end
@@ -69,11 +69,11 @@ module Everywhere
         tested = AndroidSdk::TARGET_SDK
 
         if value.nil?
-          ["platforms.android.target_sdk #{raw.inspect} is not an API level — " \
+          ["platforms.android.target_sdk #{raw.inspect} is not an API level: " \
            "a whole number like #{tested}"]
         elsif value < tested
           ["platforms.android.target_sdk #{value} is below the shell's tested level " \
-           "(#{tested}) — this option only raises it, to ship ahead of a Play deadline"]
+           "(#{tested}); this option only raises it, to ship ahead of a Play deadline"]
         else
           []
         end

data/lib/everywhere/config/mobile.rb

@@ -144,11 +144,11 @@ module Everywhere
       def permission_errors(os)
         permissions.flat_map do |name, usage|
           unless MOBILE_PERMISSIONS.include?(name)
-            next ["unknown permission #{name.inspect} — supported: #{MOBILE_PERMISSIONS.join(", ")}"]
+            next ["unknown permission #{name.inspect}. Supported: #{MOBILE_PERMISSIONS.join(", ")}"]
           end
 
           if os == "ios" && IOS_USAGE_KEYS.key?(name) && usage["ios"].to_s.strip.empty?
-            next ["#{name} needs a usage string — the sentence iOS shows when asking. " \
+            next ["#{name} needs a usage string: the sentence iOS shows when asking. " \
                   "In config/everywhere.yml:\n  permissions:\n    #{name}: \"Why the app needs #{name}.\""]
           end
 

data/lib/everywhere/config/native_desktop.rb

@@ -86,7 +86,7 @@ module Everywhere
           errors << "native.desktop.commands must be a list of function names"
         else
           errors += native_desktop_commands.reject { |name| name.match?(RUST_FN_NAME) }.map do |name|
-            "native.desktop.commands: #{name.inspect} is not a Rust function name — " \
+            "native.desktop.commands: #{name.inspect} is not a Rust function name. " \
               "snake_case (letters, digits, _), matching the fn in native/desktop/"
           end
           duplicates = native_desktop_commands.tally.select { |_name, count| count > 1 }.keys
@@ -161,7 +161,7 @@ module Everywhere
         return [] if version.empty?
         return [] if version.match?(CRATE_VERSION)
 
-        ["#{at} #{version.inspect} is not a cargo version requirement — like \"1.2\", \"^1.2.3\" or \"~0.4\""]
+        ["#{at} #{version.inspect} is not a cargo version requirement. Try \"1.2\", \"^1.2.3\" or \"~0.4\""]
       end
     end
   end

data/lib/everywhere/config/native_mobile.rb

@@ -110,7 +110,7 @@ module Everywhere
           if present.empty?
             errors << "#{at} needs one version requirement (#{PACKAGE_REQUIREMENT_KEYS.join(" / ")})"
           elsif present.length > 1
-            errors << "#{at} sets conflicting requirements (#{present.join(", ")}) — pick one"
+            errors << "#{at} sets conflicting requirements (#{present.join(", ")}); pick one"
           elsif %w[from exact].include?(present.first) && !entry[present.first].to_s.strip.match?(PACKAGE_VERSION)
             errors << "#{at}.#{present.first} #{entry[present.first].to_s.inspect} must be a version like \"1.2.0\""
           end
@@ -212,7 +212,7 @@ module Everywhere
           coordinate = entry.strip
           next if coordinate.match?(MAVEN_COORDINATE)
 
-          "#{at} #{coordinate.inspect} is not a Maven coordinate — " \
+          "#{at} #{coordinate.inspect} is not a Maven coordinate. " \
             "\"group:artifact:version\" (letters, digits, . _ -)"
         end
       end
@@ -269,7 +269,7 @@ module Everywhere
       def native_android_icon_font_errors
         return [] if ANDROID_ICON_FONTS.key?(native_android_icon_font)
 
-        ["native.android.icon_font #{native_android_icon_font.inspect} is not a known font — " \
+        ["native.android.icon_font #{native_android_icon_font.inspect} is not a known font. " \
          "one of: #{ANDROID_ICON_FONTS.keys.join(", ")}"]
       end
 

data/lib/everywhere/config/updates.rb

@@ -39,11 +39,21 @@ module Everywhere
         return [] if uri.scheme == "https"
         return [] if uri.scheme == "http" && uri.host.to_s.delete("[]").match?(UPDATES_LOOPBACK)
 
-        ["updates.url #{url.inspect} must be https — the shell disables the updater otherwise"]
+        ["updates.url #{url.inspect} must be https, or the shell disables the updater"]
       rescue URI::Error => e
         ["updates.url #{url.inspect} is not a URL (#{e.message})"]
       end
 
+      # Not an error — no `updates:` at all is a perfectly good app — but a feed
+      # URL without a key drops #updates_shell_hash to nil, so the app ships
+      # looking configured with the updater off.
+      def updates_warnings
+        return [] unless updates_url && updates_public_key.to_s.empty?
+
+        ["updates.url is set but updates.public_key is missing, so the updater will be disabled " \
+         "in this build; run `every updates keygen`"]
+      end
+
       # The client subset for the shell; nil (and thus absent from
       # everywhere.json) until both url and public_key are configured — an
       # unsigned update feed is not a thing.

data/lib/everywhere/database.rb

@@ -31,7 +31,7 @@ module Everywhere
       elsif defined?(::Sequel)
         prepare_sequel(root)
       else
-        log "no ActiveRecord/Sequel detected — skipping database prepare"
+        log "no ActiveRecord/Sequel detected, skipping database prepare"
       end
     rescue StandardError => e
       warn "[everywhere] database prepare failed: #{e.class}: #{e.message}"
@@ -48,7 +48,7 @@ module Everywhere
       end
 
       paths = migration_paths(root)
-      return log("no migrations found — skipping") if paths.empty?
+      return log("no migrations found, skipping") if paths.empty?
 
       # Opening the connection creates the sqlite file if it doesn't exist yet.
       ::ActiveRecord::Base.connection
@@ -67,7 +67,7 @@ module Everywhere
       return log("no Sequel database handle (set Everywhere.config.sequel_db)") unless db
 
       paths = migration_paths(root)
-      return log("no migrations found — skipping") if paths.empty?
+      return log("no migrations found, skipping") if paths.empty?
 
       ::Sequel.extension :migration
       paths.each { |dir| ::Sequel::Migrator.run(db, dir) }

data/lib/everywhere/desktop_assets.rb

@@ -86,8 +86,8 @@ module Everywhere
         files.group_by { |file| file[:name] }.each do |name, group|
           dirs = group.map { |file| file[:dir] }.uniq
           if dirs.length > 1
-            errors << "#{name}: same asset name in #{dirs.map { |dir| label(dir) }.join(" and ")} — " \
-                      "names are global (the shell reads one flat folder), so rename one"
+            errors << "#{name}: same asset name in #{dirs.map { |dir| label(dir) }.join(" and ")}. " \
+                      "Names are global (the shell reads one flat folder), so rename one"
           end
           group.group_by { |file| [file[:scale], file[:dark]] }.each_value do |clash|
             next if clash.length < 2
@@ -104,10 +104,10 @@ module Everywhere
       def unsupported_error(path, source)
         relative = relative(path, source)
         if VECTOR.include?(File.extname(path).downcase)
-          "#{relative}: the desktop shell can't rasterize vectors — export a PNG " \
+          "#{relative}: the desktop shell can't rasterize vectors. Export a PNG " \
             "(at @2x, so it stays sharp on a Retina display)"
         else
-          "#{relative}: unsupported image type — use #{EXTENSIONS.join(", ")}"
+          "#{relative}: unsupported image type. Use #{EXTENSIONS.join(", ")}"
         end
       end
 
@@ -116,7 +116,7 @@ module Everywhere
         name, scale, dark = parse(File.basename(path, File.extname(path)))
 
         unless name.match?(VALID_NAME)
-          errors << "#{relative}: #{name.inspect} isn't a usable asset name — use letters, digits, _, - and ."
+          errors << "#{relative}: #{name.inspect} isn't a usable asset name. Use letters, digits, _, - and ."
           return nil
         end
 

data/lib/everywhere/desktop_dev_app.rb

@@ -106,7 +106,7 @@ module Everywhere
       dest
     rescue StandardError => e
       # A Dock icon is never worth failing a dev session over.
-      UI.warn("couldn't prepare the Dock icon (#{e.class}) — using the default")
+      UI.warn("couldn't prepare the Dock icon (#{e.class}), using the default")
       ""
     end
 

data/lib/everywhere/emulator.rb

@@ -113,7 +113,7 @@ module Everywhere
       deadline = Time.now + timeout
       bounded_wait(adb_argv(nil, "wait-for-device"), timeout: timeout)
       serial = booted_serial or
-        UI.die!("emulator #{name} never registered with adb — try booting it from Android " \
+        UI.die!("emulator #{name} never registered with adb. Try booting it from Android " \
                 "Studio's Device Manager to see what it says")
       wait_until_booted(serial, timeout: [deadline - Time.now, POLL_INTERVAL].max)
       serial
@@ -187,7 +187,7 @@ module Everywhere
       return if status&.success? && !out.include?("Failure [")
 
       if application_id && out.include?("INSTALL_FAILED_UPDATE_INCOMPATIBLE")
-        UI.warn "#{application_id} is installed with a different signing key — reinstalling " \
+        UI.warn "#{application_id} is installed with a different signing key, reinstalling " \
                 "from scratch #{UI.dim("(its app data is dropped)")}"
         uninstall(serial, application_id)
         out, status = capture_install(serial, apk)
@@ -227,7 +227,7 @@ module Everywhere
           *adb_argv(serial, "shell", "am", "start", "-n", retry_component)
         )
         if started?(retry_out, retry_status)
-          UI.warn "launched #{resolved} — #{activity} is stale, update Emulator::DEFAULT_ACTIVITY"
+          UI.warn "launched #{resolved}; #{activity} is stale, update Emulator::DEFAULT_ACTIVITY"
           return
         end
       end
@@ -320,12 +320,12 @@ module Everywhere
 
       rewritten = rewrite_loopback(url)
       if emulator?(serial) && rewritten != url
-        UI.warn "adb reverse failed — pointing the shell at #{UI.cyan(rewritten)} instead " \
+        UI.warn "adb reverse failed, pointing the shell at #{UI.cyan(rewritten)} instead " \
                 "#{UI.dim("(#{LOOPBACK_ALIAS} is the emulator's alias for this machine)")}"
         return rewritten
       end
 
-      UI.warn "adb reverse failed and #{serial || "this target"} isn't an emulator — " \
+      UI.warn "adb reverse failed and #{serial || "this target"} isn't an emulator. " \
               "#{UI.cyan(url)} will resolve to the device itself; pass a LAN address " \
               "(--dev-url http://<this machine's IP>:#{port}) with the dev server bound to 0.0.0.0"
       url

data/lib/everywhere/log_filter.rb

@@ -188,7 +188,7 @@ module Everywhere
       when /\Aw: /
         return "    #{UI.yellow(short)}" if line.include?("/extensions/")
 
-        once(:shell_warning) { UI.note_line("Kotlin warnings from the shell template — see the full log") }
+        once(:shell_warning) { UI.note_line("Kotlin warnings from the shell template; see the full log") }
       when /\ABUILD SUCCESSFUL(?: in (.+))?/
         UI.note_line("build succeeded#{Regexp.last_match(1) && " (#{Regexp.last_match(1)})"}",
                      marker: "✓", color: :green)

data/lib/everywhere/omniauth.rb

@@ -39,7 +39,7 @@ module Everywhere
       # re-running replaces the wrapper, never stacks it.
       def protect!(entry_window: ENTRY_WINDOW)
         unless defined?(::OmniAuth) && ::OmniAuth.respond_to?(:config)
-          raise "Everywhere::OmniAuth.protect! needs OmniAuth loaded first — " \
+          raise "Everywhere::OmniAuth.protect! needs OmniAuth loaded first. " \
                 "call it below your OmniAuth/Devise setup (config/initializers/omniauth.rb)"
         end
 

data/lib/everywhere/paths.rb

@@ -91,7 +91,7 @@ module Everywhere
     def cache_dir
       File.expand_path(ENV["RUBYEVERYWHERE_HOME"] || "~/.rubyeverywhere")
     rescue ArgumentError
-      UI.die!("couldn't resolve your home directory — set HOME or RUBYEVERYWHERE_HOME")
+      UI.die!("couldn't resolve your home directory. Set HOME or RUBYEVERYWHERE_HOME")
     end
 
     # The stamped copy of the iOS template for one app. Persistent (not a

data/lib/everywhere/platform/credentials.rb

@@ -32,7 +32,7 @@ module Everywhere
       def self.read_or_warn(path)
         JSON.parse(File.read(path))
       rescue JSON::ParserError, SystemCallError, IOError => e
-        Everywhere::UI.warn("ignoring unreadable credentials at #{path} (#{e.class}) — " \
+        Everywhere::UI.warn("ignoring unreadable credentials at #{path} (#{e.class}). " \
                             "delete it and run `every platform login`")
         {}
       end

data/lib/everywhere/request_context.rb

@@ -80,7 +80,7 @@ module Everywhere
     # message can name it.
     def tab(name)
       unless config && platform
-        raise Error, "RequestContext#tab needs the app config — it's available " \
+        raise Error, "RequestContext#tab needs the app config. It's available " \
                      "inside an Everywhere.tabs block, not to a standalone context"
       end
 
@@ -90,7 +90,7 @@ module Everywhere
               declared.find { |tab| tab["title"].to_s.casecmp?(wanted) }
       return found if found
 
-      raise Error, "no tab named #{wanted.inspect} in config/everywhere.yml — " \
+      raise Error, "no tab named #{wanted.inspect} in config/everywhere.yml. " \
                    "declared: #{declared.map { |tab| tab["title"] }.inspect}"
     end
   end

data/lib/everywhere/shellout.rb

@@ -41,7 +41,7 @@ module Everywhere
     def run!(env, *cmd, chdir: Dir.pwd)
       success = system(child_env(env), *cmd, chdir: chdir)
       # nil means the command never ran (missing binary), false that it failed.
-      UI.die!("#{cmd.first} not found — is it installed and on your PATH?") if success.nil?
+      UI.die!("#{cmd.first} not found. Is it installed and on your PATH?") if success.nil?
       UI.die!("command failed: #{cmd.join(" ")}") unless success
     end
 
@@ -58,7 +58,7 @@ module Everywhere
     end
 
     def ensure_tool!(name, hint)
-      tool?(name) or UI.die!("#{name} not found — #{hint}")
+      tool?(name) or UI.die!("#{name} not found. #{hint}")
     end
 
     # quiet also detaches stdin. A quiet command is by definition one that has

data/lib/everywhere/simulator.rb

@@ -25,7 +25,7 @@ module Everywhere
       end
 
       device = devices.reverse.find { |d| d["name"].to_s.match?(/\AiPhone/) } || devices.last
-      UI.die!("no iOS Simulator devices — open Xcode once to install a Simulator runtime") unless device
+      UI.die!("no iOS Simulator devices. Open Xcode once to install a Simulator runtime") unless device
 
       UI.step("booting simulator #{UI.bold(device["name"])}")
       Shellout.run?("xcrun", "simctl", "boot", device["udid"], quiet: true) # races `open` starting it; harmless