yamine 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04f00c2029bf4cda65d187225d251fc594bacf538092e6a56d9c5a03f9aa47fd
-  data.tar.gz: 963f1c74d64ddb94199d42325f6d8631a5dd2d618119554a4e1a72849394c5bc
+  metadata.gz: 46a9c0208ca1d5581c3c25d45b60370a6e37c7dd4fcf721bda7a5573fc1c70db
+  data.tar.gz: 0b534671965987794b3872969dcc3065dfca5532a401f12701e531a0779e7820
 SHA512:
-  metadata.gz: 2e02efba8d7c16767b0c4b8908d295f4c095f7c32f5ef90885e1ecf0a50aa68d61cb9e0c780d3729f703fcc14fc282eed46430d4efce631762dbb832ddecf408
-  data.tar.gz: 6d36c287b907fcfba07da2db3ecf1904ff4b396fb2bf8df277e0f9dfb23515e9245d574fb0bed85a034cd9f41fae14834842c722c3ac298e9f0fafedb1d30b45
+  metadata.gz: 339e64fabec8dab3c019bfdec8ec8f9f558dc4700b647fd462c4898e4f8796f93c1b65078d9ff171c338f38c70d055a8185b1d21bb547c63831bd816c5b7c6de
+  data.tar.gz: a71d56eef8327103f46a050307bb01095810ccd70085f3728a430617d3230cf567ffe3e9a04047428d6988d63efa7dd6965546bbb5e689f270991f8d4cc55d6d

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## [0.8.1] — 2026-09-10
+
+### Fixed
+
+- **An already-correct `/etc/hosts` no longer reports a failure.**
+  `Hosts.sync` rewrote the file unconditionally, so on a normal machine
+  (where `/etc/hosts` is root-owned) it raised `Errno::EACCES`, returned
+  false, and made `yamine start` print "could not write /etc/hosts (try
+  sudo yamine hosts sync)" on every boot — pointing at an elevated write
+  for a file that already had exactly the right block. `synced?` existed
+  for precisely this guard but `sync` never consulted it. `sync` is now a
+  no-op returning true when the block already matches, and the setup step
+  reports "already lists N hostname(s)" instead of implying a failure.
+  Chrome/Firefox/Edge resolve `*.localhost` natively anyway; the file only
+  matters for Safari and custom TLDs.
+
+
 ## [0.8.0] — 2026-09-10
 
 ### Fixed

data/lib/yamine/cli/system.rb

@@ -220,6 +220,8 @@ module Yamine
         hostnames = ctx.store.load_routes.map { |r| r["hostname"] }
         if hostnames.empty?
           puts "    No routes registered yet — hosts sync will happen on the next boot."
+        elsif Yamine::Hosts.synced?(hostnames)
+          puts "    /etc/hosts already lists #{hostnames.length} hostname(s)."
         elsif Yamine::Hosts.sync(hostnames)
           puts "    Synced #{hostnames.length} hostname(s) to /etc/hosts."
         else
@@ -758,8 +760,12 @@ module Yamine
           end
         end
 
-        # 3. Hosts (best-effort: only needed for Safari; warn, don't fail)
-        unless Hosts.sync(ctx.store.load_routes.map { |r| r["hostname"] })
+        # 3. Hosts (best-effort: only needed for Safari and custom TLDs —
+        # Chrome/Firefox/Edge resolve *.localhost natively). Already-synced
+        # is the common case and must stay silent: warning there sent
+        # people to a sudo write for a file that needed nothing.
+        hostnames = ctx.store.load_routes.map { |r| r["hostname"] }
+        unless hostnames.empty? || Yamine::Hosts.synced?(hostnames) || Hosts.sync(hostnames)
           warn "Warning: could not write /etc/hosts (try sudo yamine hosts sync)."
         end
       end

data/lib/yamine/hosts.rb

@@ -41,7 +41,20 @@ module Yamine
       names
     end
 
+    # Bring the managed block to exactly these hostnames. Idempotent: when
+    # the block already matches, this is a no-op returning true.
+    #
+    # The short-circuit is load-bearing, not an optimization. /etc/hosts is
+    # root-owned on a normal machine, so a rewrite fails without sudo — and
+    # `setup` and every boot's workstation check run this. Without the
+    # guard, a machine whose hosts file was already correct got a
+    # "could not write /etc/hosts (try sudo yamine hosts sync)" warning on
+    # every run, sending people to an elevated write for a file that
+    # needed nothing. (Chrome/Firefox/Edge resolve *.localhost natively;
+    # /etc/hosts only matters for Safari and custom TLDs.)
     def sync(hostnames, path = PATH)
+      return true if synced?(hostnames, path)
+
       content = read(path)
       block = managed_block(hostnames)
       if content.include?(BEGIN_MARKER)

data/lib/yamine/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Yamine
-  VERSION = "0.8.0"
+  VERSION = "0.8.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: yamine
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Kaka Ruto