yiffspace-fixers 0.0.1 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2deff4a56be5dc1abebbb670f5a509a11f4d257cc2daba5033b9bb87820fb6d4
-  data.tar.gz: b19fd72d813bc38fa5a915c876eb4e47f7c15fcf0d051ef3851a7afd9fe3eded
+  metadata.gz: 63554d772d4091d0d869023320f37c67b15f7e210ca23ef72ac99f2aaaad2fd3
+  data.tar.gz: f6466feaf844911fd1ddcdedebe1c4d9f55ff225fbce907bf1f37defc3258662
 SHA512:
-  metadata.gz: d1225851f6be5102fe8598353e347e1a4cbd2f771c2139e8156c850fb50d8a6c136c51a6da3ef0d8dd9f23fdc4586dbc375d129b32461163fb9bdf7b2182ffa9
-  data.tar.gz: 9c1e989b7c8f83314de5b30d6ab7e5b9499611b6bcc5569b9459156fd2761d52dcc8f391c330205c2db29d50f4bb9802209b26f6dd39d4491025d63113bd206d
+  metadata.gz: 06e9bfd1da3f639d35d8d14f773a6d8d947486c8b2601aa8ac1bed4fc0746efe8f7e4f85a4d334516db6017d6c2fc3046f0c7069dc5145716ed33b1b413d49b4
+  data.tar.gz: 2b6bde7b6c07121e45cf4f4869ce0274898272fda784d653206551ab378418cb93bb2add97c3a11601b604106a99075e941598282805691cc9b189acc68edf8c

data/CHANGELOG.md

@@ -1,3 +1,17 @@
+## Unreleased
+
+- Fixes run through `YiffSpace::FixTracker.run!` (so every `fixes:*` task) can call
+  `requires_migration!` unqualified, via `YiffSpace::Fixers::Script` - each fix is now loaded
+  wrapped in its own module including it, rather than straight into the top level, so it isn't
+  defined anywhere else in the app. Constants a fix defines now land in that wrapper too, instead
+  of on `Object`. `required_migration_for` reads the unqualified form as well.
+- Fixed `lib/yiffspace/fixers.rb` never actually defining `YiffSpace::Fixers` -
+  it set up the `for_gem_extension` Zeitwerk loader but skipped the namespace
+  module the convention requires the root file to define, so referencing
+  `YiffSpace::Fixers` (even indirectly, e.g. `YiffSpace::Fixers::Engine`)
+  after the gem's initial `require` raised `Zeitwerk::NameError: expected
+  file ... to define constant YiffSpace::Fixers, but didn't`.
+
 ## 0.0.1
 
 - Initial release, extracted from the `yiffspace` gem's `YiffSpace::FixTracker`/

data/README.md

@@ -32,11 +32,12 @@ $ bin/rails fixes:migrate
 A fix can require that a migration has already been applied, and a migration can require that a
 fix has already been applied - useful when one depends on schema or data the other provides.
 `bin/rails fixes:migrate_all` runs both `db/migrate` and `db/fixes` together, in whichever order
-those requirements demand:
+those requirements demand. `requires_migration!` is available unqualified inside fixes run by the
+`fixes:*` tasks (`YiffSpace::FixTracker.requires_migration!` works too, and anywhere else):
 
 ```ruby
 # db/fixes/12_backfill_widget_type.rb
-YiffSpace::FixTracker.requires_migration!("20260822004454")
+requires_migration!("20260822004454")
 
 # db/migrate/20260901000000_remove_legacy_widget_column.rb
 class RemoveLegacyWidgetColumn < ActiveRecord::Migration[8.1]

data/lib/yiffspace/fix_tracker.rb

@@ -11,12 +11,14 @@ module YiffSpace
     # A fix script calls this as its first statement to declare that a migration must already be
     # applied before it can run, e.g.:
     #
-    #   YiffSpace::FixTracker.requires_migration!("20260822004454")
+    #   requires_migration!("20260822004454")
+    #
+    # (unqualified via YiffSpace::Fixers::Script, or as YiffSpace::FixTracker.requires_migration!)
     #
     # Written as a literal string call like that (not built dynamically) so YiffSpace::MigrationSync
     # can also read it back out of the file with #required_migration_for, without loading/running
     # the fix, to order `fixes:migrate_all` correctly.
-    REQUIRES_MIGRATION_PATTERN = /^\s*YiffSpace::FixTracker\.requires_migration!\(\s*["']([^"']+)["']\s*\)/
+    REQUIRES_MIGRATION_PATTERN = /^\s*(?:YiffSpace::FixTracker\.)?requires_migration!\(\s*["']([^"']+)["']\s*\)/
 
     module_function
 
@@ -108,7 +110,8 @@ module YiffSpace
 
     def run!(path)
       name = File.basename(path, ".rb")
-      load(File.expand_path(path))
+      # A fresh wrapper per fix, so any constants a script defines don't carry over to the next one.
+      load(File.expand_path(path), Module.new.include(Fixers::Script))
       record!(name)
       name
     end

data/lib/yiffspace/fixers/script.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module YiffSpace
+  module Fixers
+    # Helpers available unqualified at the top level of a db/fixes/*.rb script when it's run
+    # through YiffSpace::FixTracker.run! - the script is loaded wrapped in a module including
+    # this, so they never leak onto Object for the rest of the app.
+    module Script
+      delegate(:requires_migration!, to: "YiffSpace::FixTracker")
+    end
+  end
+end

data/lib/yiffspace/fixers/version.rb

@@ -2,6 +2,6 @@
 
 module YiffSpace
   module Fixers
-    VERSION = "0.0.1"
+    VERSION = "0.0.3"
   end
 end

data/lib/yiffspace/fixers.rb

@@ -13,6 +13,9 @@ loader.setup
 require_relative("fixers/engine") if defined?(Rails)
 
 module YiffSpace
+  module Fixers
+  end
+
   class Configuration
     # Named `--steps` presets for `bin/rails generate yiffspace:fixer`. See
     # YiffSpace::Configuration::FixerTemplates.

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: yiffspace-fixers
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.3
 platform: ruby
 authors:
 - Donovan_DMC
 bindir: bin
 cert_chain: []
-date: 2026-09-04 00:00:00.000000000 Z
+date: 2026-10-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -75,6 +75,7 @@ files:
 - lib/yiffspace/fixers.rb
 - lib/yiffspace/fixers/engine.rb
 - lib/yiffspace/fixers/requires_fix.rb
+- lib/yiffspace/fixers/script.rb
 - lib/yiffspace/fixers/version.rb
 - lib/yiffspace/migration_sync.rb
 homepage: https://yiff.space