plan_my_stuff 0.21.0 → 0.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0d33c652f54d9d078276d12baac63134e8bba4064185f61626c66fd924ea958a
-  data.tar.gz: 54f5c69130890fa5ae807d642cd8d89d2f0d74c4d12b638ed892d8f9b2df22d8
+  metadata.gz: d44fbaa2722319a0d586bf8f947222aa76f74462b5516f516f34b87b89ff6211
+  data.tar.gz: 05eaab50eaca5856a19cd68f796b1b4170cb750a6bb8c78566c003a576707122
 SHA512:
-  metadata.gz: 36d2a9a1624279d818d6c1a16fcfd51cd8e047e4f02d51044673fecb0ef3433b4a953290eb51deb868b9a8959fe365abf4dcb0645883466941047c6915c070d3
-  data.tar.gz: fbadf2e2e8337a7014e148e898291de7ef4453729332f2c8d2609942487ebec7359a97a687e44340edac1a0d7c8e36fdeddcd3cc90c8fe30a108f76f7ba74a49
+  metadata.gz: afbbeca592468025940d1b98eae3c93da09ca79a5406afe5d500f93e8d1803c59d8ced137c81b1e05297678de17920681756983cd0b2a4c253900897f4d5d9b5
+  data.tar.gz: 95ca844953334f3d514c1b530cac5837307b3b735975c18256ab09ad408e1e4aea1f964de8d43ffe4c6b74bdf17284f93d8f114f7e47c4ecb37484023278b28e

data/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## 0.22.0
+
+### Added
+
+- `Issue#add_to_priority_list!(priority:)` and `Issue#remove_from_priority_list!` - first-class setters that
+  mirror the `#priority_list?` / `#priority_list_priority` readers. Each writes both `Priority List` and
+  `Priority List Priority` in a single mutation so the two fields cannot drift apart (closes #80).
+
+### Changed
+
+- `IssueMetadata#priority_list` / `#priority_list_priority` deprecation message now points callers at the new
+  `Issue#add_to_priority_list!` / `#remove_from_priority_list!` pair instead of raw `set_issue_fields!`.
+
+## 0.21.1
+
+### Added
+
+- `config.eager_load_controllers_on_boot` (default `false`). When `true`, the engine eager-loads
+  `app/controllers/` in `after_initialize` so consuming apps with `eager_load = false` (dev mode) see
+  `PlanMyStuff::*Controller` constants without having to reference them first. Opt in if the host app
+  probes engine controllers via `defined?`, e.g. `defined?(PlanMyStuff::Issues::TakesController)` would
+  otherwise return `nil` until something triggered Zeitwerk autoload. Falls back to `Kernel#require` on
+  Zeitwerk < 2.6.2 (which lacks `eager_load_dir`).
+
 ## 0.21.0
 
 ### Breaking

data/CONFIGURATION.md

@@ -49,6 +49,19 @@ config.repo_nicknames = { safety: 'Compliance' } # :element -> "Element", :under
 `Issue#to_param` then returns `"Element-1234"` / `"Compliance-567"`, encoding both repo and number in a single
 URL segment so `youtrack_issue_path(@issue)` works without a `repo:` query param.
 
+## Attachments
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `attachment_repo` | `String` | `'pms-attachments'` | Bare repo (under `organization`) for uploaded attachments. |
+
+The repo must already exist; the uploader does not create it. Attachments commit onto
+`config.main_branch` under `<repo_key_or_name>/issue-<number>/<uuid>.<ext>`.
+
+```ruby
+config.attachment_repo = 'pms-attachments'
+```
+
 ## Projects
 
 | Option | Type | Default | Description |
@@ -337,6 +350,20 @@ config.cache_version = Rails.configuration.x.image_tag
 config.mount_groups = { webhooks: true, issues: true, projects: true }
 ```
 
+## Boot behavior
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `eager_load_controllers_on_boot` | `Boolean` | `false` | Eager-load engine controllers in `after_initialize`. |
+
+Opt in if the host app probes engine controllers via `defined?` in dev mode. When `true`, the
+engine walks `app/controllers` once on boot so `defined?(PlanMyStuff::SomeController)` resolves
+without first referencing the constant.
+
+```ruby
+config.eager_load_controllers_on_boot = true
+```
+
 ## Controller overrides
 
 | Option | Type | Default | Description |

data/lib/generators/plan_my_stuff/install/templates/initializer.rb

@@ -30,6 +30,15 @@ PlanMyStuff.configure do |config|
   # need an entry.
   # config.repo_nicknames = { safety: 'Compliance' }
 
+  # --------------------------------------------------------------------------
+  # Attachments
+  # --------------------------------------------------------------------------
+  # Bare repo name (under `organization`) that stores uploaded attachment
+  # binaries. The repo must already exist; the uploader does not create it.
+  # Attachments commit onto `config.main_branch` under
+  # `<repo_key_or_name>/issue-<number>/<uuid>.<ext>`.
+  # config.attachment_repo = 'pms-attachments'
+
   # --------------------------------------------------------------------------
   # Projects
   # --------------------------------------------------------------------------
@@ -184,6 +193,16 @@ PlanMyStuff.configure do |config|
   #
   # config.issue_fields_enabled = false
 
+  # --------------------------------------------------------------------------
+  # Boot behavior
+  # --------------------------------------------------------------------------
+  # Eager-load the engine's controllers in after_initialize so that
+  # `defined?(PlanMyStuff::SomeController)` resolves without first referencing
+  # the constant. Opt in if the host app probes engine controllers via
+  # `defined?` in dev mode.
+  #
+  # config.eager_load_controllers_on_boot = true
+
   # --------------------------------------------------------------------------
   # Release pipeline
   # --------------------------------------------------------------------------

data/lib/plan_my_stuff/configuration.rb

@@ -373,6 +373,15 @@ module PlanMyStuff
     #
     attr_accessor :issue_fields_enabled
 
+    # Whether to eager-load the engine's controllers on host boot. Defaults to +false+ (opt-in). When +true+, the engine
+    # walks +app/controllers+ during +after_initialize+ so +defined?(PlanMyStuff::SomeController)+ resolves without
+    # first referencing the constant. Enable in host apps that rely on +defined?+ probes against engine controllers in
+    # dev mode.
+    #
+    # @return [Boolean]
+    #
+    attr_accessor :eager_load_controllers_on_boot
+
     # @return [Configuration]
     def initialize
       @repos = {}
@@ -413,6 +422,7 @@ module PlanMyStuff
       @pipeline_completion_purge_enabled = true
       @pipeline_completion_ttl_hours = 24
       @issue_fields_enabled = true
+      @eager_load_controllers_on_boot = false
       @process_aws_webhooks = Rails.env.production?
       @sns_verifier_class = ::Aws::SNS::MessageVerifier if defined?(::Aws::SNS::MessageVerifier)
       @sns_verifier_error =

data/lib/plan_my_stuff/engine.rb

@@ -3,5 +3,21 @@
 module PlanMyStuff
   class Engine < ::Rails::Engine
     isolate_namespace PlanMyStuff
+
+    # Opt-in (via +config.eager_load_controllers_on_boot+): eager-load the engine's controllers regardless of the host
+    # app's eager_load setting. Without this, `defined?(PlanMyStuff::SomeController)` returns nil in host-app dev mode
+    # until something explicitly references the constant, since `defined?` does not trigger Zeitwerk autoload.
+    config.after_initialize do
+      next unless PlanMyStuff.configuration.eager_load_controllers_on_boot
+
+      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')).sort.each { |path| require path }
+      end
+    end
   end
 end

data/lib/plan_my_stuff/issue.rb

@@ -842,6 +842,35 @@ module PlanMyStuff
       issue_fields['Priority List Priority']
     end
 
+    # Adds this issue to the Priority List at the given priority (or re-prioritizes if already listed). Sets
+    # +Priority List+ and +Priority List Priority+ together in a single +setIssueFieldValue+ mutation so the two
+    # fields never drift out of sync.
+    #
+    # @raise [PlanMyStuff::IssueFieldsNotEnabledError] when +config.issue_fields_enabled+ is +false+
+    #
+    # @param priority [Integer]
+    #
+    # @return [self]
+    #
+    def add_to_priority_list!(priority:)
+      raise(ArgumentError, 'Priority must be an integer') unless priority.is_a?(Integer)
+
+      set_issue_fields!('Priority List' => 'Yes', 'Priority List Priority' => priority)
+    end
+
+    alias update_priority_list_priority! add_to_priority_list!
+
+    # Removes this issue from the Priority List. Clears both +Priority List+ and +Priority List Priority+ in a single
+    # +setIssueFieldValue+ mutation so the two fields never drift out of sync.
+    #
+    # @raise [PlanMyStuff::IssueFieldsNotEnabledError] when +config.issue_fields_enabled+ is +false+
+    #
+    # @return [self]
+    #
+    def remove_from_priority_list!
+      set_issue_fields!('Priority List' => nil, 'Priority List Priority' => nil)
+    end
+
     # Bulk-updates GitHub Issue Field values in a single +setIssueFieldValue+ mutation. Each key is the field display
     # name; values are coerced to the right input fragment based on the field's type. Passing +nil+ as a value clears
     # that field.

data/lib/plan_my_stuff/issue_metadata.rb

@@ -5,7 +5,8 @@ module PlanMyStuff
     PRIORITY_LIST_METADATA_DEPRECATION =
       'PlanMyStuff: IssueMetadata#priority_list / #priority_list_priority are deprecated. priority_list moved to ' \
         "GitHub Issue Fields ('Priority List' single_select, 'Priority List Priority' number) in 0.20.0. Reads " \
-        'now come from issue_fields; legacy writes forward to set_issue_fields! on save. The metadata accessors ' \
+        'now come from Issue#priority_list? / #priority_list_priority; for writes, use ' \
+        'Issue#add_to_priority_list!(priority: N) and Issue#remove_from_priority_list!. The metadata accessors ' \
         'will be removed in 1.0.0.'
 
     # @return [Time, nil] first support action timestamp, nil until set
@@ -201,7 +202,7 @@ module PlanMyStuff
       @priority_list
     end
 
-    # @deprecated Use +Issue#update!(issue_fields: { 'Priority List' => 'Yes' })+. Removed in 1.0.0.
+    # @deprecated Use +Issue#add_to_priority_list!(priority: N)+ / +Issue#remove_from_priority_list!+. Removed in 1.0.0.
     #
     # @param value [Object]
     #
@@ -231,7 +232,7 @@ module PlanMyStuff
       @priority_list_priority
     end
 
-    # @deprecated Use +Issue#update!(issue_fields: { 'Priority List Priority' => N })+. Removed in 1.0.0.
+    # @deprecated Use +Issue#add_to_priority_list!(priority: N)+ / +Issue#remove_from_priority_list!+. Removed in 1.0.0.
     #
     # @param value [Object]
     #

data/lib/plan_my_stuff/version.rb

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

metadata

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