plan_my_stuff 1.1.1 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c0f31d5e29ef0651a4df50d4072a36aad245cd679a5180d1216fc44e271b6a4e
-  data.tar.gz: 6296f1f8a9aad8f09da7bb2f2b5c579b716f6a82c3f8f93f2369487c759e254b
+  metadata.gz: bacd3474e480b36db577d70c215ad0bd440c6b2384319644047ca18f25385324
+  data.tar.gz: 68f011246630794f39094af3a6d09287fc58c35d68082bedb724a0e9c7cd264a
 SHA512:
-  metadata.gz: 4a297c9e689b711e111d90f810f0e21aa484f089c309ea13e7ae8223798416aacd3ba11d81c18f4a3cd9d57574d00c782d4e24aaf295d686f0a76a23b3ed852f
-  data.tar.gz: ca412ab09db968c32247c1d41072f66663b316bd2285e5455c149c6f88c9158a31de8a46533b41a42f94258ac0212f5e854667d1cc4c9c00de7311d490a426c5
+  metadata.gz: ae377f254278d231a437f6ab882c739f372bda399927c8adb1e1df7fa0858a648bf7d7a4c604acddc0471d6eb16b99113e21d811c71fbe374b6e7bf609e99cc6
+  data.tar.gz: '0498c86327fc8423021e1f5be08409303f4ba12f328cdf7df705f8b3b6d68456ac885c20befc38d50ef12512a42dd48a589ee8df59b84791ccf835f4e77a509b'

data/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## 1.2.0
+
+### Added
+
+- Add support for using IO when uploading files
+
+### Changed
+
+- Fix docs in `PlanMyStuff::Issue.update!` for `:issue_type` to allow passing of a `Symbol`
+
+## 1.1.2
+
+### Fixed
+
+- Incorrect default for `config.eager_load_controllers_on_boot` (should be `false` not `true`)
+- `Configuration#repo_nickname_for` no longer produces a space for underscored `config.repos` keys (e.g. `:some_name`
+  now resolves to `"SomeName"` instead of `"Some Name"`), which previously broke URL routing
+
 ## 1.1.1
 
 ### Fixed
@@ -414,7 +432,7 @@ Official release.
 
 ### Documented
 
-- `designs/g9/` documents the design pivot from inline-base64 (body
+- Issue #9 documents the design pivot from inline-base64 (body
   size capped) through sibling-repo (rejected) to the chosen
   side-branch approach.
 
@@ -487,7 +505,7 @@ Official release.
 
 ### Breaking
 
-- `config.should_send_request` and `config.job_classes` accessors — declared but never wired up. The request gateway that would honor them is deferred (see `requirements/09_request_gateway.md` and `designs/init/gem_mvp_deferred_notes.md`); they will return when the gateway lands
+- `config.should_send_request` and `config.job_classes` accessors — declared but never wired up. The request gateway that would honor them is deferred (tracked in issue #12); they will return when the gateway lands
 
 ### Added
 
@@ -574,12 +592,6 @@ Official release.
 - Items at `Ready for Release`, `Release in Progress`, or `Completed` are no longer removed when an unassign webhook fires
 - Pending-approvals guard runs before `ProjectItem.create!` in the assignment flow so the webhook no longer leaves orphan items when `Pipeline.take!`'s approval guard would otherwise raise
 
-### Documented
-
-- `designs/pipeline_cleanup/plan.md` — full design + bug-to-fix mapping
-- `requirements/08_release_pipeline.md` updated to reflect the new flow
-- `requirements/tasks.md` — T-PC-001 added under Phase 14
-
 ## 0.5.1
 
 ### Fixed

data/README.md

@@ -713,9 +713,6 @@ On a successful AWS deployment the gem completes each matching item individually
 (release-notes mailer, Slack summary, changelog generator) get one clean "deployment finished" signal
 without debouncing the per-item events.
 
-See `designs/release_cycle/plan.md` for the full design, including the `projects_v2_item` path that fires
-`Pipeline.take!` when a human drags an item to Started on github.com.
-
 ### "Take" button
 
 The mounted UI exposes a **Take** button on pipeline issues that calls `Pipeline.take!` and assigns the current user. Your app's user-id -> GitHub login mapping is provided by `config.github_login_for` (a hash keyed by user id).

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

@@ -26,8 +26,8 @@ PlanMyStuff.configure do |config|
 
   # Human-readable prefix used as the `Issue#to_param` so URLs read
   # `/issues/Element-1234` instead of `/issues/1234`. Missing keys fall back
-  # to `key.to_s.titleize`; only repos whose key doesn't titleize cleanly
-  # need an entry.
+  # to `key.to_s.titleize.delete(' ')`; only repos whose key doesn't titleize
+  # cleanly need an entry.
   # config.repo_nicknames = { safety: 'Compliance' }
 
   # --------------------------------------------------------------------------
@@ -222,7 +222,7 @@ PlanMyStuff.configure do |config|
   # the constant. Opt in if the host app probes engine controllers via
   # `defined?` in dev mode.
   #
-  # config.eager_load_controllers_on_boot = true
+  # config.eager_load_controllers_on_boot = false
 
   # --------------------------------------------------------------------------
   # Release pipeline

data/lib/plan_my_stuff/attachment_uploader.rb

@@ -30,8 +30,9 @@ module PlanMyStuff
       # @param repo [PlanMyStuff::Repo] source issues repo (used to namespace the path)
       # @param issue_number [Integer]
       # @param files [Array] mix of already-uploaded +PlanMyStuff::Attachment+ instances, uploaded-file objects
-      #   responding to +#path+ and +#original_filename+ (e.g. Rails +ActionDispatch::Http::UploadedFile+), and
-      #   String/Pathname paths to local files
+      #   responding to +#path+ and +#original_filename+ (e.g. Rails +ActionDispatch::Http::UploadedFile+),
+      #   String/Pathname paths to local files, and +{filename:, io:}+ Hashes wrapping an already in-memory
+      #   readable (e.g. an S3 SDK response body)
       #
       # @return [Array<PlanMyStuff::Attachment>]
       #
@@ -83,11 +84,30 @@ module PlanMyStuff
             [leaf_filename(entry.original_filename), File.binread(entry.path)]
           elsif entry.is_a?(String) || entry.is_a?(Pathname)
             [File.basename(entry), File.binread(entry)]
+          elsif entry.is_a?(Hash) && entry[:io].present?
+            [leaf_filename(entry[:filename]), read_io(entry[:io])]
           else
             raise(ArgumentError, "Unsupported attachment entry: #{entry.inspect}")
           end
         end
 
+        # Rewinds +io+ first (if possible) so callers who already read from it for another purpose (e.g.
+        # content-type sniffing) still get the full bytes.
+        #
+        # @raise [ArgumentError] If the IO object doesn't respond to `:read`
+        #
+        # @param io [#read]
+        #
+        # @return [String] raw bytes
+        #
+        def read_io(io)
+          raise(ArgumentError, 'Attachment IO must respond to `#read`') unless io.respond_to?(:read)
+
+          io.rewind if io.respond_to?(:rewind)
+
+          io.read
+        end
+
         # Strips directory components (both POSIX and Windows separators) from a browser-supplied filename so
         # values like +"../../secret.txt"+ or +"C:\\fakepath\\photo.jpg"+ cannot leak into stored metadata or
         # later download paths.

data/lib/plan_my_stuff/configuration.rb

@@ -367,8 +367,8 @@ module PlanMyStuff
     attr_accessor :repos
 
     # Human-readable repo names used as the +to_param+ prefix on +PlanMyStuff::Issue+ instances. Symbol-keyed
-    # against +repos+ -- missing keys fall back to +key.to_s.titleize+, so only entries that diverge from a simple
-    # +titleize+ of the key (e.g. +:safety+ -> +"Compliance"+) need to be listed.
+    # against +repos+ -- missing keys fall back to +key.to_s.titleize.delete(' ')+, so only entries that diverge
+    # from a simple +titleize+ of the key (e.g. +:safety+ -> +"Compliance"+) need to be listed.
     #
     # @return [Hash{Symbol => String}]
     #
@@ -520,14 +520,14 @@ module PlanMyStuff
     end
 
     # Human-readable nickname for a repo key, used as the +to_param+ prefix on +PlanMyStuff::Issue+ instances. Falls
-    # back to +key.to_s.titleize+ when no explicit entry exists in +repo_nicknames+.
+    # back to +key.to_s.titleize.delete(' ')+ when no explicit entry exists in +repo_nicknames+.
     #
     # @param key [Symbol, String]
     #
     # @return [String]
     #
     def repo_nickname_for(key)
-      repo_nicknames[key&.to_sym] || key.to_s.titleize
+      repo_nicknames[key&.to_sym] || key.to_s.titleize.delete(' ')
     end
 
     private

data/lib/plan_my_stuff/issue.rb

@@ -209,9 +209,9 @@ module PlanMyStuff
       # @param labels [Array<String>, nil]
       # @param state [Symbol, nil] :open or :closed
       # @param assignees [Array<String>, String, nil] GitHub logins
-      # @param issue_type [String, nil] GitHub issue type name. Pass a String to set, +nil+ to clear, or omit the
-      #   kwarg to leave the current type untouched. (+nil+-vs-omitted is differentiated by the private
-      #   +ISSUE_TYPE_UNCHANGED+ sentinel.)
+      # @param issue_type [String, Symbol, nil] GitHub issue type name. Pass a String to set, +nil+ to clear, or omit
+      #   the kwarg to leave the current type untouched. (+nil+-vs-omitted is differentiated by the private
+      #   +ISSUE_TYPE_UNCHANGED+ sentinel.). A `Symbol` will map shortcut names to the proper GitHub name.
       # @param issue_fields [Hash{String,Symbol => Object,nil}, nil] GitHub Issue Field values to apply after the
       #   PATCH (or instead of it, when no other attrs are provided). Delegates to +#set_issue_fields!+, so the same
       #   coercion rules and +IssueFieldsNotEnabledError+ behavior apply. +nil+ or an empty hash is a no-op.

data/lib/plan_my_stuff/version.rb

@@ -3,8 +3,8 @@
 module PlanMyStuff
   module VERSION
     MAJOR = 1
-    MINOR = 1
-    TINY = 1
+    MINOR = 2
+    TINY = 0
 
     # 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.1.1
+  version: 1.2.0
 platform: ruby
 authors:
 - Brands Insurance
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-06-19 00:00:00.000000000 Z
+date: 2026-07-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails