plan_my_stuff 0.6.0 → 0.8.0

data/lib/plan_my_stuff/reminders/fire.rb

@@ -100,19 +100,19 @@ module PlanMyStuff
         def pending_approvers
           @issue.pending_approvals.filter_map do |approval|
             PlanMyStuff::UserResolver.resolve(approval.user_id)
-          rescue
+          rescue ActiveRecord::RecordNotFound
             next
           end
         end
 
         # @return [Array<Integer>]
         def effective_reminder_days
-          @issue.metadata.reminder_days.presence || PlanMyStuff.configuration.reminder_days
+          (@issue.metadata.reminder_days.presence || PlanMyStuff.configuration.reminder_days).sort
         end
 
         # Time of the next milestone past +now+, or +nil+ when the last
         # milestone has passed. Returned as a native +Time+ (not ISO
-        # string) so +IssueMetadata#to_h+'s +format_time+ serializes it
+        # string) so +IssueMetadata#to_h+'s +PlanMyStuff.format_time+ serializes it
         # cleanly.
         #
         # @return [Time, nil]

data/lib/plan_my_stuff/reminders/sweep.rb

@@ -23,10 +23,10 @@ module PlanMyStuff
         return unless PlanMyStuff.configuration.reminders_enabled
 
         candidates.each do |issue|
-          if Closer.should_close?(issue, now: @now)
-            Closer.new(issue, now: @now).call
-          elsif Fire.ready?(issue, now: @now)
-            Fire.new(issue, now: @now).call
+          if PlanMyStuff::Reminders::Closer.should_close?(issue, now: @now)
+            PlanMyStuff::Reminders::Closer.new(issue, now: @now).call
+          elsif PlanMyStuff::Reminders::Fire.ready?(issue, now: @now)
+            PlanMyStuff::Reminders::Fire.new(issue, now: @now).call
           end
         end
       end

data/lib/plan_my_stuff/repo.rb

@@ -16,11 +16,15 @@ module PlanMyStuff
     class << self
       # Builds a Repo instance from a Symbol key, full name String, or nil (default).
       #
+      # @raise [PlanMyStuff::ConfigurationError] if repo is not provided and cannot be resolved from config
+      # @raise [ArgumentError] if repo cannot be resolved
+      # @raise [ArgumentError] if repo is invalid format
+      #
       # @param repo [Symbol, String, PlanMyStuff::Repo, nil]
       #
       # @return [PlanMyStuff::Repo]
       #
-      def resolve(repo = nil)
+      def resolve!(repo = nil)
         return repo if repo.is_a?(PlanMyStuff::Repo)
 
         repo ||= PlanMyStuff.configuration.default_repo
@@ -38,10 +42,10 @@ module PlanMyStuff
           full_name = PlanMyStuff.configuration.repos[repo]
           raise(ArgumentError, "Unknown repo key: #{repo.inspect}") if full_name.nil?
 
-          from_full_name(full_name, key: repo)
+          from_full_name!(full_name, key: repo)
         when String
           key = PlanMyStuff.configuration.repos.key(repo)
-          from_full_name(repo, key: key)
+          from_full_name!(repo, key: key)
         else
           raise(ArgumentError, "Cannot resolve repo: #{repo.inspect}")
         end
@@ -49,12 +53,14 @@ module PlanMyStuff
 
       private
 
+        # @raise [ArgumentError] if full_name is not in "Org/Repo" format
+        #
         # @param full_name [String] e.g. "YourOrgName/MyRepository"
         # @param key [Symbol, nil]
         #
         # @return [PlanMyStuff::Repo]
         #
-        def from_full_name(full_name, key: nil)
+        def from_full_name!(full_name, key: nil)
           org, name = full_name.split('/', 2)
 
           raise(ArgumentError, "Invalid repo full_name: #{full_name.inspect}") if name.nil?
@@ -81,8 +87,8 @@ module PlanMyStuff
     # @see #full_name
     alias to_s full_name
 
-    # Enables implicit string coercion so Repo instances behave as
-    # strings when passed to Octokit or compared with String#==.
+    # Enables implicit string coercion so Repo instances behave as strings when passed to Octokit or compared with
+    # String#==.
     #
     # @see #full_name
     alias to_str full_name

data/lib/plan_my_stuff/test_helpers.rb

@@ -306,7 +306,7 @@ module PlanMyStuff
 
         expect(match).to(
           be_truthy,
-          "Expected PMS #{description} with #{filters.inspect}, " \
+          "Expected PlanMyStuff #{description} with #{filters.inspect}, " \
             "but recorded actions were: #{format_actions(type)}",
         )
       end
@@ -335,7 +335,7 @@ module PlanMyStuff
     # @return [void]
     #
     def test_mode!
-      TestHelpers.recorded_actions = []
+      PlanMyStuff::TestHelpers.recorded_actions = []
       return if @_test_mode
 
       @_test_mode_originals = {}
@@ -367,7 +367,7 @@ module PlanMyStuff
 
       @_test_mode_originals = nil
       @_test_mode = false
-      TestHelpers.recorded_actions = []
+      PlanMyStuff::TestHelpers.recorded_actions = []
     end
 
     private
@@ -400,7 +400,7 @@ module PlanMyStuff
       def stub_issue_class_methods!
         issue_mod = PlanMyStuff::Issue
         %i[create! find list update!].each { |m| save_original(issue_mod, m) }
-        %i[add_viewers remove_viewers].each { |m| save_instance_original(issue_mod, m) }
+        %i[add_viewers! remove_viewers!].each { |m| save_instance_original(issue_mod, m) }
 
         issue_mod.define_singleton_method(:create!) do |**params|
           PlanMyStuff::TestHelpers.recorded_actions << {
@@ -415,7 +415,7 @@ module PlanMyStuff
 
           resolved_repo =
             if params[:repo]
-              PlanMyStuff::Repo.resolve(params[:repo]).full_name
+              PlanMyStuff::Repo.resolve!(params[:repo]).full_name
             else
               'TestOrg/TestRepo'
             end
@@ -434,7 +434,7 @@ module PlanMyStuff
             params: { number: number, repo: repo },
           }
 
-          resolved_repo = repo ? PlanMyStuff::Repo.resolve(repo).full_name : 'TestOrg/TestRepo'
+          resolved_repo = repo ? PlanMyStuff::Repo.resolve!(repo).full_name : 'TestOrg/TestRepo'
           PlanMyStuff::TestHelpers.build_issue(number: number, repo: resolved_repo)
         end
 
@@ -456,7 +456,7 @@ module PlanMyStuff
           nil
         end
 
-        issue_mod.define_method(:add_viewers) do |user_ids:, user: nil|
+        issue_mod.define_method(:add_viewers!) do |user_ids:, user: nil|
           PlanMyStuff::TestHelpers.recorded_actions << {
             type: :viewers_added,
             params: { number: number, repo: repo&.to_s, user_ids: Array.wrap(user_ids), user: user },
@@ -465,7 +465,7 @@ module PlanMyStuff
           nil
         end
 
-        issue_mod.define_method(:remove_viewers) do |user_ids:, user: nil|
+        issue_mod.define_method(:remove_viewers!) do |user_ids:, user: nil|
           PlanMyStuff::TestHelpers.recorded_actions << {
             type: :viewers_removed,
             params: { number: number, repo: repo&.to_s, user_ids: Array.wrap(user_ids), user: user },
@@ -548,9 +548,9 @@ module PlanMyStuff
       # @return [void]
       def stub_project_item_class_methods!
         item_mod = PlanMyStuff::ProjectItem
-        %i[move_item create! assign].each { |m| save_original(item_mod, m) }
+        %i[move_item! create! assign!].each { |m| save_original(item_mod, m) }
 
-        item_mod.define_singleton_method(:move_item) do |**params|
+        item_mod.define_singleton_method(:move_item!) do |**params|
           PlanMyStuff::TestHelpers.recorded_actions << {
             type: :item_moved,
             params: {
@@ -589,7 +589,7 @@ module PlanMyStuff
           )
         end
 
-        item_mod.define_singleton_method(:assign) do |**params|
+        item_mod.define_singleton_method(:assign!) do |**params|
           PlanMyStuff::TestHelpers.recorded_actions << {
             type: :item_assigned,
             params: {

data/lib/plan_my_stuff/testing_project.rb

@@ -17,7 +17,7 @@ module PlanMyStuff
     # "Test Status" is intentionally a separate field from GitHub's built-in
     # "Status" (which BaseProject and Pipeline hardcode). Using a parallel
     # field lets the QA sign-off flow own its own options (Passed/Failed)
-    # without colliding with the board's default Status column — a testing
+    # without colliding with the board's default Status column - a testing
     # project pointed at a pipeline-tracked repo won't overwrite either.
     BOOTSTRAP_FIELDS = [
       { name: 'Test Status', data_type: 'SINGLE_SELECT', options: ['Todo', 'In Progress', 'Passed', 'Failed'] },
@@ -100,12 +100,12 @@ module PlanMyStuff
       # Finds a testing project by number. Raises if the project exists but is
       # not a testing project.
       #
+      # @raise [ArgumentError] if the project is not a testing project
+      #
       # @param number [Integer]
       # @param paginate [Symbol] :auto (default) or :cursor
       # @param cursor [String, nil]
       #
-      # @raise [ArgumentError] if the project is not a testing project
-      #
       # @return [PlanMyStuff::TestingProject]
       #
       def find(number, paginate: :auto, cursor: nil)
@@ -142,7 +142,7 @@ module PlanMyStuff
         def create_from_template!(title:, description:, readme:, project_metadata:, template_number:)
           project = clone!(source_number: template_number, title: title)
 
-          serialized_readme = PlanMyStuff::MetadataParser.serialize(project_metadata.to_h, readme)
+          serialized_readme = PlanMyStuff::MetadataParser.serialize!(project_metadata.to_h, readme)
           update_input = { projectId: project.id, readme: serialized_readme }
           update_input[:shortDescription] = description if description.present?
 
@@ -177,7 +177,7 @@ module PlanMyStuff
           project_id = new_project[:id]
           project_number = new_project[:number]
 
-          serialized_readme = PlanMyStuff::MetadataParser.serialize(project_metadata.to_h, readme)
+          serialized_readme = PlanMyStuff::MetadataParser.serialize!(project_metadata.to_h, readme)
           update_input = { projectId: project_id, readme: serialized_readme }
           update_input[:shortDescription] = description if description.present?
 
@@ -199,8 +199,9 @@ module PlanMyStuff
         # @return [void]
         #
         def bootstrap_fields!(project_id, project_number)
-          current = find(project_number)
-          existing_names = current.fields.pluck(:name)
+          org = PlanMyStuff.configuration.organization
+          page = fetch_project_page(org, project_number, nil)
+          existing_names = (page[:raw].dig(:fields, :nodes) || []).pluck(:name)
 
           BOOTSTRAP_FIELDS.each do |field_def|
             next if existing_names.include?(field_def[:name])
@@ -246,14 +247,14 @@ module PlanMyStuff
 
     # Returns the Test Status single-select field definition.
     #
-    # @return [Hash] with :id and :options keys
-    #
     # @raise [PlanMyStuff::APIError] if no Test Status field exists
     #
-    def status_field
+    # @return [Hash] with :id and :options keys
+    #
+    def status_field!
       field = fields.find { |f| f[:name] == 'Test Status' && f[:options] }
 
-      raise(APIError, "No 'Test Status' field found on project ##{number}") unless field
+      raise(PlanMyStuff::APIError, "No 'Test Status' field found on project ##{number}") unless field
 
       field
     end

data/lib/plan_my_stuff/testing_project_item.rb

@@ -25,7 +25,7 @@ module PlanMyStuff
       #
       def create!(issue_or_title, draft: false, body: nil, project_number: nil, user: nil)
         item = super
-        move_item(project_number: item.project.number, item_id: item.id, status: DEFAULT_STATUS)
+        move_item!(project_number: item.project.number, item_id: item.id, status: DEFAULT_STATUS)
         update_single_select_field!(
           project_number: item.project.number,
           item_id: item.id,
@@ -103,7 +103,7 @@ module PlanMyStuff
     # @return [Hash] mutation result
     #
     def update_passed_at!(time)
-      update_field!('Passed At', time.utc.iso8601)
+      update_field!('Passed At', PlanMyStuff.format_time(time))
     end
 
     # Updates the Deadline Miss Reason field on this testing project item.
@@ -124,14 +124,16 @@ module PlanMyStuff
     #
     # No-op when the user has already signed off on this item.
     #
+    # @raise [PlanMyStuff::ValidationError] when user is blank
+    #
     # @param user [Object] PMS user object of the tester signing off
     #
     # @return [void]
     #
     def mark_passed!(user)
-      raise(PMS::ValidationError, 'No user configured for sign-off.') if user.blank?
+      raise(PlanMyStuff::ValidationError, 'No user configured for sign-off.') if user.blank?
 
-      user_id = PMS::UserResolver.user_id(user).to_s
+      user_id = PlanMyStuff::UserResolver.user_id(user).to_s
       current = user_ids_from_field('Passed By')
       return if current.include?(user_id)
 
@@ -158,20 +160,20 @@ module PlanMyStuff
     # Records +user_id+ as having failed this item, writes +result_notes+,
     # and moves the item to the Failed status.
     #
+    # @raise [PlanMyStuff::ValidationError] if result_notes is blank
+    #
     # @param user [Object] PMS user object of the tester failing the item
     # @param result_notes [String] required explanation of the failure
     #
-    # @raise [PlanMyStuff::ValidationError] if result_notes is blank
-    #
     # @return [void]
     #
     def mark_failed!(user, result_notes:)
-      raise(PMS::ValidationError, 'Result notes are required when failing an item.') if result_notes.blank?
-      raise(PMS::ValidationError, 'No user configured for sign-off.') if user.blank?
+      raise(PlanMyStuff::ValidationError, 'Result notes are required when failing an item.') if result_notes.blank?
+      raise(PlanMyStuff::ValidationError, 'No user configured for sign-off.') if user.blank?
 
       update_result_notes!(result_notes)
 
-      user_id = PMS::UserResolver.user_id(user).to_s
+      user_id = PlanMyStuff::UserResolver.user_id(user).to_s
       current = user_ids_from_field('Failed By')
       return if current.include?(user_id)
 

data/lib/plan_my_stuff/testing_project_metadata.rb

@@ -3,7 +3,7 @@
 module PlanMyStuff
   # Metadata for testing projects, stored in the GH Project readme field.
   # Detected at read time by kind: "testing" in the serialized metadata blob.
-  class TestingProjectMetadata < BaseProjectMetadata
+  class TestingProjectMetadata < PlanMyStuff::BaseProjectMetadata
     # @return [Array<String>] PR/issue URLs this testing project covers
     attr_accessor :subject_urls
     # @return [Date, nil] project-level deadline
@@ -86,7 +86,7 @@ module PlanMyStuff
     def to_h
       super.merge(
         subject_urls: subject_urls,
-        due_date: due_date&.iso8601,
+        due_date: PlanMyStuff.format_time(due_date),
         deadline_miss_reason: deadline_miss_reason,
       )
     end

data/lib/plan_my_stuff/version.rb

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

data/lib/plan_my_stuff/webhook_replayer.rb

@@ -167,6 +167,8 @@ module PlanMyStuff
     # a time. Stops on the first non-success so state can be
     # investigated without skipping ahead.
     #
+    # @raise [PlanMyStuff::Error] when the endpoint returns a non-success HTTP status
+    #
     # @return [void]
     #
     def replay_pending(pending, endpoint_url)
@@ -191,14 +193,18 @@ module PlanMyStuff
         )
 
         unless response.is_a?(Net::HTTPSuccess)
-          raise('  ! non-success; stopping so you can investigate')
+          raise(PlanMyStuff::Error, '  ! non-success; stopping so you can investigate')
         end
 
         mark_processed(target[:processed_file], id)
       end
     end
 
+    # @raise [ArgumentError] when scope is :repo but repo is nil or empty
+    # @raise [ArgumentError] when scope is neither :org nor :repo
+    #
     # @return [String]
+    #
     def hooks_base_path(scope:, repo:)
       case scope
       when :org
@@ -212,7 +218,10 @@ module PlanMyStuff
       end
     end
 
+    # @raise [PlanMyStuff::Error] when no webhook matches the given URL
+    #
     # @return [Integer]
+    #
     def resolve_hook_id(scope:, repo:, webhook_url:)
       hooks = gh_get(hooks_base_path(scope: scope, repo: repo), per_page: 100)
       match = hooks.find { |h| h.dig('config', 'url') == webhook_url }
@@ -247,7 +256,10 @@ module PlanMyStuff
       File.open(path, 'a') { |f| f.puts(id) }
     end
 
+    # @raise [PlanMyStuff::Error] when the endpoint returns a non-success HTTP status
+    #
     # @return [void]
+    #
     def replay_each(to_process, deliveries_path, endpoint_url, processed_file, interactive:)
       to_process.each.with_index(1) do |summary, index|
         id = summary['id']
@@ -265,7 +277,7 @@ module PlanMyStuff
         )
 
         unless response.is_a?(Net::HTTPSuccess)
-          raise('  ! non-success; stopping so you can investigate')
+          raise(PlanMyStuff::Error, '  ! non-success; stopping so you can investigate')
         end
 
         mark_processed(processed_file, id)

data/lib/plan_my_stuff.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'date'
+
 require 'active_support/core_ext/array/wrap'
 require 'active_support/core_ext/object/blank'
 
@@ -44,7 +46,7 @@ module PlanMyStuff
   class << self
     # @return [PlanMyStuff::Configuration]
     def configuration
-      @configuration ||= Configuration.new
+      @configuration ||= PlanMyStuff::Configuration.new
     end
 
     # @return [PlanMyStuff::Configuration]
@@ -54,7 +56,12 @@ module PlanMyStuff
 
     # @return [PlanMyStuff::Client]
     def client
-      @client ||= Client.new
+      @client ||= PlanMyStuff::Client.new
+    end
+
+    # @return [PlanMyStuff::Client]
+    def import_client
+      @import_client ||= PlanMyStuff::Client.new(importing: true)
     end
 
     # Returns the appropriate HTTP 422 status symbol for the current Rails version.
@@ -70,13 +77,30 @@ module PlanMyStuff
       end
     end
 
+    # Formats a time-ish value as an ISO 8601 string. +Time+ and
+    # +DateTime+ are normalized to UTC; +Date+ is serialized as-is.
+    #
+    # @param value [Time, DateTime, Date, String, nil]
+    #
+    # @return [String, nil]
+    #
+    def format_time(value)
+      return if value.nil?
+      return value if value.is_a?(String)
+      return value.iso8601 if value.is_a?(Date) && !value.is_a?(DateTime)
+
+      value.utc.iso8601
+    end
+
     # Resets the memoized client and configuration. Useful for testing.
     #
     # @return [void]
     #
     def reset!
       exit_test_mode! if defined?(@_test_mode) && @_test_mode
+      PlanMyStuff::Client.exit_trace_mode!
       @client = nil
+      @import_client = nil
       @configuration = nil
     end
   end

data/lib/tasks/plan_my_stuff.rake

@@ -18,12 +18,16 @@ namespace :plan_my_stuff do
   end
 
   namespace :webhooks do
-    desc 'Create an organization webhook (URL=... [EVENTS=ev1,ev2])'
+    desc 'Create an organization webhook (URL=... [EVENTS=ev1,ev2]). ' \
+      'Raises PlanMyStuff::ConfigurationError if config.webhook_secret is blank.'
     task create_org: :environment do
-      url = ENV.fetch('URL') { raise('URL env var is required') }
-      events = %w[projects_v2 projects_v2_item projects_v2_status_update]
+      url = ENV.fetch('URL') { raise(ArgumentError, 'URL env var is required') }
+      default_events = %w[projects_v2 projects_v2_item projects_v2_status_update]
+      events = ENV['EVENTS'].present? ? ENV['EVENTS'].split(',').map(&:strip).compact_blank : default_events
       config = PlanMyStuff.configuration
-      raise('PlanMyStuff.configuration.webhook_secret is blank') if config.webhook_secret.blank?
+      if config.webhook_secret.blank?
+        raise(PlanMyStuff::ConfigurationError, 'PlanMyStuff.configuration.webhook_secret is blank')
+      end
 
       hook = PlanMyStuff.client.rest(
         :create_org_hook,
@@ -36,16 +40,21 @@ namespace :plan_my_stuff do
       puts("Events: #{events.join(', ')}")
     end
 
-    desc 'Create a repo webhook (REPO=owner/name URL=... [EVENTS=ev1,ev2])'
+    desc 'Create a repo webhook (REPO=owner/name URL=... [EVENTS=ev1,ev2]). ' \
+      'Raises PlanMyStuff::ConfigurationError if config.webhook_secret is blank.'
     task create_repo: :environment do
-      url = ENV.fetch('URL') { raise('URL env var is required') }
+      url = ENV.fetch('URL') { raise(ArgumentError, 'URL env var is required') }
       repo = ENV.fetch('REPO') do
-        PlanMyStuff.client.resolve_repo ||
-          raise('REPO env var is required or configured (e.g. BrandsInsurance/PlanMyStuff)')
+        PlanMyStuff.client.resolve_repo!
+      rescue
+        raise(ArgumentError, 'REPO env var is required or configured (e.g. BrandsInsurance/PlanMyStuff)')
       end
-      events = %w[pull_request issues]
+      default_events = %w[pull_request issues]
+      events = ENV['EVENTS'].present? ? ENV['EVENTS'].split(',').map(&:strip).compact_blank : default_events
       config = PlanMyStuff.configuration
-      raise('PlanMyStuff.configuration.webhook_secret is blank') if config.webhook_secret.blank?
+      if config.webhook_secret.blank?
+        raise(PlanMyStuff::ConfigurationError, 'PlanMyStuff.configuration.webhook_secret is blank')
+      end
 
       hook = PlanMyStuff.client.rest(
         :create_hook,
@@ -64,8 +73,8 @@ namespace :plan_my_stuff do
     task replay: :environment do
       require 'plan_my_stuff/webhook_replayer'
 
-      endpoint_url = ENV.fetch('ENDPOINT_URL') { raise('ENDPOINT_URL env var is required') }
-      webhook_url = ENV.fetch('WEBHOOK_URL') { raise('WEBHOOK_URL env var is required') }
+      endpoint_url = ENV.fetch('ENDPOINT_URL') { raise(ArgumentError, 'ENDPOINT_URL env var is required') }
+      webhook_url = ENV.fetch('WEBHOOK_URL') { raise(ArgumentError, 'WEBHOOK_URL env var is required') }
       scope = ENV.fetch('SCOPE', 'org').to_sym
       repo = ENV.fetch('REPO', nil)
       processed_file =
@@ -85,11 +94,12 @@ namespace :plan_my_stuff do
     end
 
     desc 'Continuously poll org + repo hooks and auto-replay new deliveries ' \
-      '(ENDPOINT_URL=... [ORG_WEBHOOK_URL=...] [REPO_WEBHOOK_URL=... REPO=owner/name] [INTERVAL=15])'
+      '(ENDPOINT_URL=... [ORG_WEBHOOK_URL=...] [REPO_WEBHOOK_URL=... REPO=owner/name] [INTERVAL=15]). ' \
+      'At least one of ORG_WEBHOOK_URL or REPO_WEBHOOK_URL is required (raises ArgumentError if both absent).'
     task listen: :environment do
       require 'plan_my_stuff/webhook_replayer'
 
-      endpoint_url = ENV.fetch('ENDPOINT_URL') { raise('ENDPOINT_URL env var is required') }
+      endpoint_url = ENV.fetch('ENDPOINT_URL') { raise(ArgumentError, 'ENDPOINT_URL env var is required') }
       org_webhook_url = ENV.fetch('ORG_WEBHOOK_URL', nil)
       repo_webhook_url = ENV.fetch('REPO_WEBHOOK_URL', nil)
       repo = ENV.fetch('REPO', nil)
@@ -98,11 +108,11 @@ namespace :plan_my_stuff do
       targets = []
       targets << { scope: :org, webhook_url: org_webhook_url } if org_webhook_url.present?
       if repo_webhook_url.present?
-        raise('REPO env var required when REPO_WEBHOOK_URL is set') if repo.blank?
+        raise(ArgumentError, 'REPO env var required when REPO_WEBHOOK_URL is set') if repo.blank?
 
         targets << { scope: :repo, webhook_url: repo_webhook_url, repo: repo }
       end
-      raise('Set at least one of ORG_WEBHOOK_URL or REPO_WEBHOOK_URL') if targets.empty?
+      raise(ArgumentError, 'Set at least one of ORG_WEBHOOK_URL or REPO_WEBHOOK_URL') if targets.empty?
 
       processed_file_for = -> (target) {
         Rails.root.join('tmp', 'plan_my_stuff', "webhook_listen_#{target[:scope]}_processed.txt").to_s
@@ -121,10 +131,10 @@ namespace :plan_my_stuff do
     task simulate_aws: :environment do
       require 'plan_my_stuff/aws_sns_simulator'
 
-      endpoint_url = ENV.fetch('ENDPOINT_URL') { raise('ENDPOINT_URL env var is required') }
+      endpoint_url = ENV.fetch('ENDPOINT_URL') { raise(ArgumentError, 'ENDPOINT_URL env var is required') }
       event_name = ENV.fetch('EVENT', PlanMyStuff::AwsSnsSimulator::DEFAULT_EVENT)
 
-      PlanMyStuff::AwsSnsSimulator.post(
+      PlanMyStuff::AwsSnsSimulator.post!(
         endpoint_url: endpoint_url,
         event_name: event_name,
       )
@@ -133,7 +143,8 @@ namespace :plan_my_stuff do
 
   namespace :reminders do
     desc 'Enqueue a RemindersSweepJob per configured repo ' \
-      '([REPO=<key>] to target a single repo)'
+      '([REPO=<key>] to target a single repo). ' \
+      'Raises PlanMyStuff::ConfigurationError when no repos are configured (config.repos empty and REPO unset).'
     task sweep: :environment do
       config = PlanMyStuff.configuration
       repo_keys =
@@ -143,7 +154,9 @@ namespace :plan_my_stuff do
           config.repos.keys
         end
 
-      raise('No repos configured (set config.repos or pass REPO=<key>)') if repo_keys.empty?
+      if repo_keys.empty?
+        raise(PlanMyStuff::ConfigurationError, 'No repos configured (set config.repos or pass REPO=<key>)')
+      end
 
       repo_keys.each do |key|
         PlanMyStuff::RemindersSweepJob.requeue(key)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: plan_my_stuff
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Brands Insurance
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-27 00:00:00.000000000 Z
+date: 2026-04-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -86,6 +86,7 @@ files:
 - app/views/plan_my_stuff/issues/partials/_links.html.erb
 - app/views/plan_my_stuff/issues/partials/_viewers.html.erb
 - app/views/plan_my_stuff/issues/show.html.erb
+- app/views/plan_my_stuff/partials/_flash.html.erb
 - app/views/plan_my_stuff/projects/edit.html.erb
 - app/views/plan_my_stuff/projects/index.html.erb
 - app/views/plan_my_stuff/projects/new.html.erb