logstruct 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 647c97ba4368b67d39333204ca35a5161b0905dda2c8ef72d01e15d58665d367
-  data.tar.gz: e86e688d516ef584c5f8994ecb0739599c09595a021ebc949f494e6d631f349d
+  metadata.gz: b257a68c1c28d1475a75a22d5114ad27325c6da9c327edb4eba4a035a2ec3de0
+  data.tar.gz: f32b4ea82a198a0a7bae0b6454411ddd28ddf92ae09d96948d02074399cc242c
 SHA512:
-  metadata.gz: 6709f7868f43c348891761749400bb93c762e3df6f2d7093dab12f51c50a2ce33e26bfd94dab009093a1c4207f746735764c56ce20768394a9b8e81cce5d346d
-  data.tar.gz: 7ab05d71a64c08cd4ad747c7684d3b75553e2740c0539cefa2cab2f906f2c5023dd8696bb78129790472896f83008addde117f9e41229d563ad9003a76911fd9
+  metadata.gz: 53ded423b2a206e38da2799efa3a5a8687a68acc86da9270548bd1e08e722cf2862604733f413e71e88d9d8d54ae637e1b7316b893fd1ee4a81e0e0f42fd8efc
+  data.tar.gz: 5fde1dfdb59100833384f59126d64b0cd5466abcc4304278f1312c3bca2fad6f0461b16f47720c142600c6f8e059faf27f032b235d9c34e400fc08738fb29382

data/CHANGELOG.md

@@ -7,6 +7,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.4.0] - 2026-11-06
+
+### Changed
+
+- **Breaking**: Requires `sorbet-runtime` 0.6.12698 or later, the first version with `T::Module`, which `LogStruct::SemanticLogger::Logger`'s signature uses.
+- **Breaking**: Log structs mark the props that implement their interfaces with `override: true`, which current versions of Sorbet require. A custom log struct needs the same on its `level`, `timestamp` and `additional_data` props, with the interfaces included above them.
+- **Development**: Update the locked gems and the docs and tooling packages, including Sorbet 0.6, Tapioca 0.20 and RuboCop 1.91. CI tests Rails 7.2.4, 8.0.5.2 and 8.1.4.1.
+- **Development**: `task ci` runs `jscpd`. GitHub CI runs the dependency audits, and a release waits for them.
+
+### Fixed
+
+- **Development**: Update the locked `crass` dependency to 1.0.8, which fixes GHSA-pq4g-8rxf-4g29, and move `glob` and `minimatch` in the development tools to patched versions.
+- **Generation**: An interrupted run leaves the generated files as they were, because they're written to a staging directory and moved into place.
+
 ## [0.3.0] - 2026-10-23
 
 ### Changed

data/lib/log_struct/concerns/configuration.rb

@@ -12,7 +12,7 @@ module LogStruct
       module ClassMethods
         extend T::Sig
 
-        CONSOLE_COMMAND_ARGS = T.let(["console", "c"].freeze, T::Array[String])
+        CONSOLE_COMMAND_ARGS = ["console", "c"].freeze
         EMPTY_ARGV = T.let([].freeze, T::Array[String])
         RAILS_PATTERNS_LABEL = "Rails filter_parameters"
         RAILS_REGEXPS_LABEL = "Rails filter_parameters (Regexps)"
@@ -237,7 +237,7 @@ module LogStruct
         # there or from the field's value down, as it would in Rails.
         sig { params(patterns: T::Array[T.untyped]).returns(ConfigStruct::PathFilterMatcher) }
         def build_rails_path_matcher(patterns)
-          regexps = patterns.select { |entry| entry.is_a?(Regexp) }
+          regexps = patterns.grep(Regexp)
           strings = patterns.reject { |entry| entry.is_a?(Regexp) }.map { |entry| Regexp.escape(entry.to_s) }
           regexps << Regexp.new(strings.join("|"), true) if strings.any?
           callable = Kernel.lambda do |path, _value, _root|

data/lib/log_struct/config_struct/filters.rb

@@ -62,7 +62,7 @@ module LogStruct
     class LazyRoot
       extend T::Sig
 
-      UNAVAILABLE = T.let(Object.new.freeze, Object)
+      UNAVAILABLE = Object.new.freeze
 
       sig { params(value: T.untyped).void }
       def initialize(value)
@@ -187,10 +187,7 @@ module LogStruct
       # - api_key, secret_key, private_key, access_key, encryption_key, signing_key, master_key: X-Api-Key
       # - signature, sig, salt and certificate as the last word: X-Amz-Signature (but not signature_field_id)
       # Matching whole words avoids false positives like "keyboard", "author" or "cron_key".
-      DEFAULT_SENSITIVE_KEY_PATTERN = T.let(
-        /(?:\A|_)(?:passwords?|passwd|passphrases?|passcodes?|pwd|tokens?|secrets?|auth|authorization|authentication|creds?|credentials?|cookies?|sessions?|jwts?|bearer|otps?|apikeys?|cvv|cvc|ssn|social_security(?:_numbers?)?|credit_cards?|card_numbers?|crypt|crypted)(?:_|\z)|(?:\A|_)(?:api|secret|private|access|encryption|signing|master)_keys?(?:_|\z)|(?:\A|_)(?:signature|sig|salt|certificate)s?\z/,
-        Regexp
-      )
+      DEFAULT_SENSITIVE_KEY_PATTERN = /(?:\A|_)(?:passwords?|passwd|passphrases?|passcodes?|pwd|tokens?|secrets?|auth|authorization|authentication|creds?|credentials?|cookies?|sessions?|jwts?|bearer|otps?|apikeys?|cvv|cvc|ssn|social_security(?:_numbers?)?|credit_cards?|card_numbers?|crypt|crypted)(?:_|\z)|(?:\A|_)(?:api|secret|private|access|encryption|signing|master)_keys?(?:_|\z)|(?:\A|_)(?:signature|sig|salt|certificate)s?\z/
 
       # Matchers for keys that aren't listed by name in filter_keys. Each matcher receives the key (String)
       # and optional value, returning true when the pair should be filtered. Includes a regex matcher for

data/lib/log_struct/enums/level.rb

@@ -32,7 +32,7 @@ module LogStruct
     end
 
     # Convert a string or integer severity to a Level
-    sig { params(severity: T.any(String, Symbol, Integer, NilClass)).returns(Level) }
+    sig { params(severity: T.nilable(T.any(String, Symbol, Integer))).returns(Level) }
     def self.from_severity(severity)
       return Unknown if severity.nil?
       return from_severity_int(severity) if severity.is_a?(Integer)

data/lib/log_struct/integrations/action_mailer/event_logging.rb

@@ -9,6 +9,7 @@ module LogStruct
         extend ActiveSupport::Concern
         extend T::Sig
         extend T::Helpers
+
         requires_ancestor { ::ActionMailer::Base }
         requires_ancestor { ErrorHandling }
 

data/lib/log_struct/integrations/action_mailer/metadata_collection.rb

@@ -7,6 +7,7 @@ module LogStruct
       # Handles collection of metadata for email logging
       module MetadataCollection
         extend T::Sig
+
         # Add message-specific metadata to log data
         sig { params(mailer: T.untyped, log_data: T::Hash[Symbol, T.untyped]).void }
         def self.add_message_metadata(mailer, log_data)

data/lib/log_struct/integrations/active_record.rb

@@ -43,7 +43,7 @@ module LogStruct
 
       # Track subscription state keyed to the current Notifications.notifier instance
       State = ::Struct.new(:subscribed, :notifier_id)
-      STATE = T.let(State.new(false, nil), State)
+      STATE = State.new(false, nil)
 
       # Set up SQL query logging integration
       sig { override.params(config: LogStruct::Configuration).returns(T.nilable(T::Boolean)) }

data/lib/log_struct/integrations/ahoy.rb

@@ -20,7 +20,7 @@ module LogStruct
 
             # Takes whatever Ahoy's track takes and passes it on as given, so Ahoy's defaults still apply
             sig { params(name: T.untyped, args: T.untyped).returns(T.untyped) }
-            def track(name, *args)
+            def track(name, *args) # rubocop:disable Sorbet/BlockMethodDefinition -- it's prepended and calls implicit super, which define_method can't do
               result = super
               begin
                 LogStruct.info(

data/lib/log_struct/integrations/carrierwave.rb

@@ -35,6 +35,7 @@ module LogStruct
       module LoggingMethods
         extend T::Sig
         extend T::Helpers
+
         requires_ancestor { ::CarrierWave::Uploader::Base }
 
         sig { params(args: T.untyped).returns(T.untyped) }

data/lib/log_struct/integrations/dotenv.rb

@@ -17,11 +17,12 @@ module LogStruct
     module Dotenv
       extend T::Sig
       extend IntegrationInterface
+
       @original_logger_setter = T.let(nil, T.nilable(UnboundMethod))
 
       # Internal state holder to avoid duplicate subscriptions in a Sorbet-friendly way
       State = ::Struct.new(:subscribed)
-      STATE = T.let(State.new(false), State)
+      STATE = State.new(false)
 
       sig { override.params(config: LogStruct::Configuration).returns(T.nilable(T::Boolean)) }
       def self.setup(config)

data/lib/log_struct/integrations/good_job.rb

@@ -157,7 +157,8 @@ module LogStruct
         return true unless defined?(::GoodJob::Notifier::CONNECTION_ERRORS)
 
         ::GoodJob::Notifier::CONNECTION_ERRORS.any? do |error_name|
-          error_class = error_name.safe_constantize
+          # CONNECTION_ERRORS holds class names, because an app might not have loaded the database adapter they belong to
+          error_class = error_name.safe_constantize # rubocop:disable Sorbet/ConstantsFromStrings
           error_class.is_a?(Class) && exception.is_a?(error_class)
         end
       end

data/lib/log_struct/integrations/host_authorization.rb

@@ -12,14 +12,11 @@ module LogStruct
       extend T::Sig
       extend IntegrationInterface
 
-      RESPONSE_HTML = T.let(
-        "<html><head><title>Blocked Host</title></head><body>" \
+      RESPONSE_HTML = "<html><head><title>Blocked Host</title></head><body>" \
         "<h1>Blocked Host</h1>" \
         "<p>This host is not permitted to access this application.</p>" \
         "<p>If you are the administrator, check your configuration.</p>" \
-        "</body></html>",
-        String
-      )
+        "</body></html>"
       RESPONSE_HEADERS = T.let(
         {
           "Content-Type" => "text/html",
@@ -27,7 +24,7 @@ module LogStruct
         }.freeze,
         T::Hash[String, String]
       )
-      FORBIDDEN_STATUS = T.let(403, Integer)
+      FORBIDDEN_STATUS = 403
 
       # Set up host authorization logging
       sig { override.params(config: LogStruct::Configuration).returns(T.nilable(T::Boolean)) }

data/lib/log_struct/integrations/lograge.rb

@@ -14,28 +14,25 @@ module LogStruct
     module Lograge
       extend IntegrationInterface
 
-      LOGRAGE_KNOWN_KEYS = T.let(
-        [
-          :method,
-          :path,
-          :format,
-          :controller,
-          :action,
-          :status,
-          :duration,
-          :view,
-          :db,
-          :params,
-          :request_id,
-          :source_ip,
-          :user_agent,
-          :referer,
-          :host,
-          :content_type,
-          :accept
-        ].freeze,
-        T::Array[Symbol]
-      )
+      LOGRAGE_KNOWN_KEYS = [
+        :method,
+        :path,
+        :format,
+        :controller,
+        :action,
+        :status,
+        :duration,
+        :view,
+        :db,
+        :params,
+        :request_id,
+        :source_ip,
+        :user_agent,
+        :referer,
+        :host,
+        :content_type,
+        :accept
+      ].freeze
 
       # With config.exceptions_app = routes, Rails renders an error page by sending the request through the app again
       # as GET /<status>, and keeps the request's own method and path in these headers

data/lib/log_struct/integrations/puma.rb

@@ -196,7 +196,7 @@ module LogStruct
             return true
           end
 
-          if (m = l.match(/^(?:\*\s*)?Puma version: (\S+)(?:.*"([^\"]+)")?/))
+          if (m = l.match(/^(?:\*\s*)?Puma version: (\S+)(?:.*"([^"]+)")?/))
             T.cast(STATE[:start_info], T::Hash[Symbol, T.untyped])[:puma_version] = m[1]
             if m[2]
               T.cast(STATE[:start_info], T::Hash[Symbol, T.untyped])[:puma_codename] = m[2]

data/lib/log_struct/integrations/rack_error_handler/error_lines.rb

@@ -12,10 +12,7 @@ module LogStruct
       module ErrorLines
         extend T::Sig
 
-        CSRF_ERROR_NAMES = T.let(
-          ["ActionController::InvalidAuthenticityToken", "ActionController::InvalidCrossOriginRequest"].freeze,
-          T::Array[String]
-        )
+        CSRF_ERROR_NAMES = ["ActionController::InvalidAuthenticityToken", "ActionController::InvalidCrossOriginRequest"].freeze
 
         sig { returns(T::Boolean) }
         def self.enabled?

data/lib/log_struct/integrations/shrine.rb

@@ -14,7 +14,7 @@ module LogStruct
       extend T::Sig
       extend IntegrationInterface
 
-      SHRINE_EVENTS = T.let(%i[upload exists download delete metadata open].freeze, T::Array[Symbol])
+      SHRINE_EVENTS = %i[upload exists download delete metadata open].freeze
 
       # The file name in a Content-Disposition (attachment; filename="..."; filename*=UTF-8''...). A quoted name
       # can hold a semicolon, and whatever follows a quote (malformed or not) goes too.

data/lib/log_struct/log/action_mailer/delivered.rb

@@ -23,11 +23,15 @@ module LogStruct
       class Delivered < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+        include LogStruct::Log::Interfaces::AdditionalDataField
+
         # Shared/common fields
-        const :source, Source::Mailer, default: Source::Mailer
-        const :event, Event, default: Event::Delivered
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Mailer, default: Source::Mailer, override: true
+        const :event, Event, default: Event::Delivered, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :to, T.nilable(T::Array[String]), default: nil
         const :from, T.nilable(String), default: nil
@@ -38,8 +42,7 @@ module LogStruct
         const :attachment_count, T.nilable(Integer), default: nil
 
         # Additional data
-        include LogStruct::Log::Interfaces::AdditionalDataField
-        const :additional_data, T.nilable(T::Hash[T.any(String, Symbol), T.untyped]), default: nil
+        const :additional_data, T.nilable(T::Hash[T.any(String, Symbol), T.untyped]), default: nil, override: true
         include LogStruct::Log::Shared::MergeAdditionalDataFields
 
         FIELD_KEYS = T.let(
@@ -53,7 +56,6 @@ module LogStruct
         end
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/action_mailer/delivery.rb

@@ -23,11 +23,15 @@ module LogStruct
       class Delivery < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+        include LogStruct::Log::Interfaces::AdditionalDataField
+
         # Shared/common fields
-        const :source, Source::Mailer, default: Source::Mailer
-        const :event, Event, default: Event::Delivery
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Mailer, default: Source::Mailer, override: true
+        const :event, Event, default: Event::Delivery, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :to, T.nilable(T::Array[String]), default: nil
         const :from, T.nilable(String), default: nil
@@ -38,8 +42,7 @@ module LogStruct
         const :attachment_count, T.nilable(Integer), default: nil
 
         # Additional data
-        include LogStruct::Log::Interfaces::AdditionalDataField
-        const :additional_data, T.nilable(T::Hash[T.any(String, Symbol), T.untyped]), default: nil
+        const :additional_data, T.nilable(T::Hash[T.any(String, Symbol), T.untyped]), default: nil, override: true
         include LogStruct::Log::Shared::MergeAdditionalDataFields
 
         FIELD_KEYS = T.let(
@@ -53,7 +56,6 @@ module LogStruct
         end
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/action_mailer/error.rb

@@ -23,11 +23,15 @@ module LogStruct
       class Error < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+        include LogStruct::Log::Interfaces::AdditionalDataField
+
         # Shared/common fields
-        const :source, Source::Mailer, default: Source::Mailer
-        const :event, Event, default: Event::Error
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Mailer, default: Source::Mailer, override: true
+        const :event, Event, default: Event::Error, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :to, T.nilable(T::Array[String]), default: nil
         const :from, T.nilable(String), default: nil
@@ -43,8 +47,7 @@ module LogStruct
         const :backtrace, T.nilable(T::Array[String]), default: nil
 
         # Additional data
-        include LogStruct::Log::Interfaces::AdditionalDataField
-        const :additional_data, T.nilable(T::Hash[T.any(String, Symbol), T.untyped]), default: nil
+        const :additional_data, T.nilable(T::Hash[T.any(String, Symbol), T.untyped]), default: nil, override: true
         include LogStruct::Log::Shared::MergeAdditionalDataFields
 
         FIELD_KEYS = T.let(
@@ -58,7 +61,6 @@ module LogStruct
         end
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/action_mailer.rb

@@ -15,6 +15,7 @@ module LogStruct
     class ActionMailer
       class BaseFields < T::Struct
         extend T::Sig
+
         const :to, T.nilable(T::Array[String]), default: nil
         const :from, T.nilable(String), default: nil
         const :subject, T.nilable(String), default: nil

data/lib/log_struct/log/active_job/discard.rb

@@ -23,11 +23,14 @@ module LogStruct
       class Discard < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::Discard
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::Discard, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -41,7 +44,6 @@ module LogStruct
         const :message, String
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job/enqueue.rb

@@ -23,11 +23,14 @@ module LogStruct
       class Enqueue < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::Enqueue
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::Enqueue, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -37,7 +40,6 @@ module LogStruct
         const :provider_job_id, T.nilable(String), default: nil
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job/error.rb

@@ -23,11 +23,14 @@ module LogStruct
       class Error < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::Error
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::Error, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -42,7 +45,6 @@ module LogStruct
         const :backtrace, T.nilable(T::Array[String]), default: nil
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job/finish.rb

@@ -23,11 +23,14 @@ module LogStruct
       class Finish < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::Finish
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::Finish, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -41,7 +44,6 @@ module LogStruct
         const :finished_at, Time
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job/retry.rb

@@ -23,11 +23,14 @@ module LogStruct
       class Retry < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::Retry
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::Retry, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -42,7 +45,6 @@ module LogStruct
         const :wait_ms, T.nilable(Float), default: nil
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job/retry_stopped.rb

@@ -23,11 +23,14 @@ module LogStruct
       class RetryStopped < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::RetryStopped
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::RetryStopped, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -41,7 +44,6 @@ module LogStruct
         const :message, String
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job/schedule.rb

@@ -23,11 +23,14 @@ module LogStruct
       class Schedule < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::Schedule
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::Schedule, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -40,7 +43,6 @@ module LogStruct
         const :scheduled_at, Time
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job/start.rb

@@ -23,11 +23,14 @@ module LogStruct
       class Start < T::Struct
         extend T::Sig
 
+        # A prop marked `override` needs the method it implements to exist already
+        include LogStruct::Log::Interfaces::CommonFields
+
         # Shared/common fields
-        const :source, Source::Job, default: Source::Job
-        const :event, Event, default: Event::Start
-        const :timestamp, Time, factory: -> { Time.now }
-        const :level, Level, default: Level::Info
+        const :source, Source::Job, default: Source::Job, override: true
+        const :event, Event, default: Event::Start, override: true
+        const :timestamp, Time, factory: -> { Time.now }, override: true
+        const :level, Level, default: Level::Info, override: true
 
         const :job_id, String
         const :job_class, String
@@ -41,7 +44,6 @@ module LogStruct
         const :attempt, T.nilable(Integer), default: nil
 
         # Serialize shared fields
-        include LogStruct::Log::Interfaces::CommonFields
         include LogStruct::Log::Shared::SerializeCommon
 
         sig { returns(T::Hash[LogStruct::LogField, T.untyped]) }

data/lib/log_struct/log/active_job.rb

@@ -20,6 +20,7 @@ module LogStruct
     class ActiveJob
       class BaseFields < T::Struct
         extend T::Sig
+
         const :job_id, String
         const :job_class, String
         const :queue_name, T.nilable(Symbol), default: nil