plan_my_stuff 1.0.0 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3993e1306f27793791833787c546d81a42ed7bdf23d1420a658ec3877acf284b
-  data.tar.gz: 4593de2a00ea81ad22d0d951e1a97eb858a3d26071510c42e2e1aab9b99ae0f8
+  metadata.gz: 0b0b4807143a42fc7a7b667b6023f182e6bfe5601c9302d7044c7e30b4dab5db
+  data.tar.gz: 19c3cb2b4ab0156d01b2ec1a1f52086c1299f3171a2284fa0df995d163b17bb4
 SHA512:
-  metadata.gz: a9e140d9c6371186961903c5251143a63f4bedb3c238fc2d2b69e25ef3263824bdeacf767494c46473a5c7e9560c50b057e9b82136505b3615bcacdc742b3b6c
-  data.tar.gz: f73cb36b19ced07d713ae6e1aee02ab314292b3410d8b14d160e2156ceedd4a202f2fcecff7f67b61d25c93887bbc7e694f4415b89bab1203faa1e8bf7d7566c
+  metadata.gz: 5de35eb3a9b45ba810df2b786081d9cc3988f46048ac94b8df9c65887c71103bc2b4de1b5c4eeb0fa7bb49156c4eb405fdc0fe368764f5b86d7de977dd7f5449
+  data.tar.gz: 268532412c45fbeceeeae220870ec19bddf150644ac23689f680cd4f0fe97bce5161decd56ae85bb72343160abc5ae3cb585e5b55129e38455e604e385013847

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 1.0.2
+
+### Fixed
+
+- `Issue.new` now accepts `issue_fields:` without erroring, staging the values for the next `save!`
+- `Issue#issue_fields` no longer crashes on unpersisted records; it returns the pending values passed to `.new` so a
+  failed `new` form can be re-rendered without losing the user's Issue Field selections
+
+## 1.0.1
+
+### Fixed
+
+- `eager_load_controllers_on_boot` no longer crashes host-app boot with `NameError: uninitialized constant
+  PlanMyStuff::Issues` when `config.eager_load` is `false` (dev/test/console/rake) on zeitwerk 2.6.x, as shipped with
+  Rails 7.1. The engine now requires its controller files directly instead of driving zeitwerk's `eager_load_dir`,
+  which fails to register child autoloads on the gem-defined `PlanMyStuff` namespace on those versions (closes #104).
+
 ## 1.0.0
 
 Official release.

data/lib/plan_my_stuff/engine.rb

@@ -10,14 +10,15 @@ module PlanMyStuff
     config.after_initialize do
       next unless PlanMyStuff.configuration.eager_load_controllers_on_boot
 
+      # Require the controller files directly rather than going through zeitwerk's +eager_load_dir+ /
+      # +eager_load_namespace+. +PlanMyStuff+ is an external namespace pre-defined by the gem's +lib/+, and some
+      # zeitwerk versions (notably 2.6.12, shipped with Rails 7.1) never register child autoloads on it. On those
+      # versions the namespace-aware loaders raise +NameError+ descending into +PlanMyStuff::Issues+ / +::Webhooks+
+      # -- and so does a plain +constantize+. Requiring the files defines the constants directly and works on every
+      # supported zeitwerk version: zeitwerk's +require+ decoration keeps its bookkeeping in sync on versions that do
+      # manage these files, and in eager-loading hosts the files are already loaded so +require+ is a no-op.
       controllers_dir = File.expand_path('../../app/controllers', __dir__)
-      loader = Rails.autoloaders.main
-
-      if loader.respond_to?(:eager_load_dir)
-        loader.eager_load_dir(controllers_dir)
-      else
-        Dir.glob(File.join(controllers_dir, '**/*.rb')).each { |path| require path }
-      end
+      Dir.glob(File.join(controllers_dir, '**/*.rb')).each { |path| require(path) }
     end
   end
 end

data/lib/plan_my_stuff/issue.rb

@@ -808,6 +808,7 @@ module PlanMyStuff
 
     def initialize(**attrs)
       @body_dirty = false
+      @pending_issue_fields = attrs.delete(:issue_fields) if attrs.key?(:issue_fields)
       super
     end
 
@@ -1049,7 +1050,9 @@ module PlanMyStuff
 
     # Returns a hash-like view of GitHub Issue Field values currently set on this issue. Reads on first access and
     # memoizes; +set_issue_fields!+ invalidates the cache. Returns an empty set without making a request when
-    # +config.issue_fields_enabled+ is +false+.
+    # +config.issue_fields_enabled+ is +false+. For unpersisted records, returns the pending values passed to +.new+
+    # (the raw consumer-supplied hash) without making a request, so a failed +new+ form can be re-rendered without
+    # losing the user's input.
     #
     # @return [PlanMyStuff::IssueFieldValueSet]
     #
@@ -1376,6 +1379,10 @@ module PlanMyStuff
       def load_issue_fields!
         return PlanMyStuff::IssueFieldValueSet.new({}) unless PlanMyStuff.configuration.issue_fields_enabled
 
+        # Unpersisted records have no values on GitHub yet. Surface the pending values passed to +.new+ / +update!+
+        # so a consuming app can re-render a failed "new" form without losing the user's Issue Field selections.
+        return PlanMyStuff::IssueFieldValueSet.new((@pending_issue_fields || {}).transform_keys(&:to_s)) if new_record?
+
         data = PlanMyStuff.client.graphql(
           PlanMyStuff::GraphQL::Queries::READ_ISSUE_FIELD_VALUES,
           variables: { owner: repo.organization, name: repo.name, number: number },

data/lib/plan_my_stuff/version.rb

@@ -4,7 +4,7 @@ module PlanMyStuff
   module VERSION
     MAJOR = 1
     MINOR = 0
-    TINY = 0
+    TINY = 2
 
     # Set PRE to nil unless it's a pre-release (beta, rc, etc.)
     PRE = nil

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: plan_my_stuff
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.2
 platform: ruby
 authors:
 - Brands Insurance
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-05-29 00:00:00.000000000 Z
+date: 2026-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails