standard_health 0.7.1 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 162374f071e6634d43876ef75f8fa6f50b1ec7bfd6a8d01bd94a91c854151d09
-  data.tar.gz: 58c40844359768a3d905c75575599dc1bf61dab15ebabc372c3e5e56cd4e169d
+  metadata.gz: 3cf58624de0179c4903a4159e227baa82079b2a78e97679122a429b186783918
+  data.tar.gz: 33963ed5e02ec9794a3133e4fbf3444cb443bc821045ec8bde6ecfb4b575f0a0
 SHA512:
-  metadata.gz: a93890d618342991d02699f9a68d0cf432596f4fba5eff7659d4d4c8675d22e5e0bac3d996abed41e02ddd7f53b84c771f21e093a6c6e22b39404ec9ccc7498c
-  data.tar.gz: 13b347704251b47c6a877ce5fb559686d9fee639610109300da1fd989627010a3823f678ab420330a26ac851548647da3ab12405081cd13a2d9024c90d4bb5dc
+  metadata.gz: 846b8334841aeb56c429f8a767f76c1e3e22e849da7f5de261370895cb3d7e735e2ee1fe3a72508637474f0338b34c05848bdda32e359bee783857ca3dd85e79
+  data.tar.gz: b6fefd789b745ec5a20819c46b89d610b226283b09024db1426ba5aacebee6838941f9d9c423b152bc55df2ab09bd341906cc5b15de6e2a7c0c59a5891d291f6

data/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.8.0] - 2026-10-22
+
+### Changed
+
+- **Re-registering a check replaces it.** `register_check` and
+  `register_aggregate_check` replace an existing registration of the same name
+  in place (last writer wins) instead of appending a second one, matching
+  `register_diagnostics_assertion`. A `configure` block that runs twice, e.g.
+  from `to_prepare` on reload, no longer runs every check twice.
+- **`env_spec` aliases are checked at boot.** Assigning `c.env_spec = …` calls
+  the new `EnvSpec#validate!`, which raises `EnvSpec::UnknownModeAlias` naming
+  every `in:` alias that has no `mode_alias`. Before, an undeclared alias raised
+  only on the first audit, which was a production `/diagnostics/env` request
+  (it 500'd one app's doctor tier once). All five estate apps declare every
+  alias they use, so none of them is affected.
+
 ## [0.7.1] - 2026-09-25
 
 ### Fixed

data/README.md

@@ -362,7 +362,7 @@ end
 
 - `nil` (omitted) — entry always applies
 - `Array<String>` — literal mode list (existing behaviour)
-- `Symbol` — resolved against `mode_alias` at audit time. An undeclared Symbol raises `StandardHealth::EnvSpec::UnknownModeAlias`.
+- `Symbol` — resolved against `mode_alias`. Assigning the spec (`c.env_spec = …`) checks every alias, so an undeclared Symbol raises `StandardHealth::EnvSpec::UnknownModeAlias` at boot, naming each unknown alias, rather than on the first `/diagnostics/env` request.
 
 ### `description:` and `consumed_by:`
 
@@ -522,6 +522,10 @@ end
 
 `with_timing` captures `latency_ms` on success and converts any `StandardError` into `{ status: :fail, error: <message> }`.
 
+Registering a name that is already registered replaces that check in place (the
+same for `register_aggregate_check`), so a `configure` block that runs twice,
+e.g. from `to_prepare` on reload, doesn't run any check twice.
+
 **Per-registration options (0.6.0).** Keywords other than `critical:` and
 `timeout:` are forwarded to the check's constructor, and validated against its
 signature at registration — a typo fails at boot, not on every probe:

data/lib/standard_health/configuration.rb

@@ -86,7 +86,16 @@ module StandardHealth
     # An optional `StandardHealth::EnvSpec` instance describing required and
     # recommended environment variables for the host app. Audited via the
     # /diagnostics/env endpoint.
-    attr_accessor :env_spec
+    attr_reader :env_spec
+
+    # Validates the spec as it is assigned (`EnvSpec#validate!`), so an
+    # `in:` alias with no `mode_alias` fails the initializer at boot rather
+    # than raising on the first audit, which is a /diagnostics/env request
+    # in production.
+    def env_spec=(spec)
+      spec.validate! if spec.respond_to?(:validate!)
+      @env_spec = spec
+    end
 
     # --- Instrumentation -----------------------------------------------
     #
@@ -270,13 +279,16 @@ module StandardHealth
     #   Validated against the constructor's signature HERE, so a typo fails
     #   at boot instead of turning into an ArgumentError on every probe (which
     #   the aggregator would dutifully report as a failing check forever).
+    #
+    # Re-registering a name replaces that registration in place (last writer
+    # wins), so `configure` can safely re-run, e.g. from `to_prepare` on
+    # reload, without running the check twice.
     def register_check(name, klass, critical: false, timeout: nil, **options)
       validate_check_options!(klass, options)
       registration = Registration.new(
         name: name.to_sym, klass: klass, critical: critical, timeout: timeout, options: options
       )
-      @checks << registration
-      registration
+      upsert_registration(@checks, registration)
     end
 
     # The check set every consumer app registers by hand. Each entry: default
@@ -362,13 +374,14 @@ module StandardHealth
     #
     #   c.register_aggregate_check :solid_cable, StandardHealth::Checks::SolidCable
     #   c.register_aggregate_check :attestation_roots, "AttestationRootsCheck"
+    #
+    # Re-registering a name replaces it in place, as for #register_check.
     def register_aggregate_check(name, klass, critical: false, timeout: nil, **options)
       validate_check_options!(klass, options)
       registration = Registration.new(
         name: name.to_sym, klass: klass, critical: critical, timeout: timeout, options: options
       )
-      @aggregate_checks << registration
-      registration
+      upsert_registration(@aggregate_checks, registration)
     end
 
     DiagnosticsAssertion = Struct.new(:name, :callable)
@@ -440,6 +453,13 @@ module StandardHealth
 
     private
 
+    # Replaces the registration with the same name in place, or appends.
+    def upsert_registration(registry, registration)
+      index = registry.index { |existing| existing.name == registration.name }
+      index ? registry[index] = registration : registry << registration
+      registration
+    end
+
     # Rejects option keys the check's constructor cannot accept. Only
     # inspects the signature — never instantiates — so a check with side
     # effects in `initialize` is not run at boot. A constructor taking

data/lib/standard_health/env_spec.rb

@@ -115,6 +115,21 @@ module StandardHealth
       @mode_aliases[name.to_sym] = Array(modes).map(&:to_s)
     end
 
+    # Resolves every `in:` alias now. `Configuration#env_spec=` calls this,
+    # so an undeclared or misspelt alias fails at boot instead of raising on
+    # the first audit (a /diagnostics/env request, or an EnvSpecAudit row).
+    #
+    # @raise [UnknownModeAlias] naming every unknown alias
+    # @return [self]
+    def validate!
+      unknown = entries.filter_map { |entry| entry.modes if alias_unknown?(entry.modes) }.uniq
+      return self if unknown.empty?
+
+      raise UnknownModeAlias,
+        "Unknown mode alias#{"es" if unknown.size > 1}: #{unknown.map(&:inspect).join(", ")} " \
+        "(declare it with mode_alias)"
+    end
+
     # Group subsequent declarations under a label. Pure metadata —
     # propagated to audit rows as `group:` and otherwise inert. Nested
     # `group` blocks are supported; the innermost label is the one that
@@ -253,6 +268,10 @@ module StandardHealth
       modes.include?(mode_str)
     end
 
+    def alias_unknown?(modes)
+      modes.is_a?(Symbol) && !@mode_aliases.key?(modes)
+    end
+
     def resolve_modes(modes)
       return nil if modes.nil?
       if modes.is_a?(Symbol)

data/lib/standard_health/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: standard_health
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.8.0
 platform: ruby
 authors:
 - Jaryl Sim