plan_my_stuff 0.8.0 → 0.10.0

data/lib/plan_my_stuff/issue_extractions/waiting.rb

@@ -0,0 +1,148 @@
+# frozen_string_literal: true
+
+module PlanMyStuff
+  module IssueExtractions
+    module Waiting
+      # Marks the issue as waiting on an end-user reply. Sets +metadata.waiting_on_user_at+ to now, (re)computes
+      # +metadata.next_reminder_at+, and adds the configured +waiting_on_user_label+ to the issue. Called from
+      # +Comment.create!+ when a support user posts a comment with +waiting_on_reply: true+, and from the
+      # +Issues::WaitingsController+ toggle.
+      #
+      # @param user [Object, nil] actor for the label notification event
+      #
+      # @return [self]
+      #
+      def enter_waiting_on_user!(user: nil)
+        now = Time.now.utc
+        label = PlanMyStuff.configuration.waiting_on_user_label
+
+        PlanMyStuff::Label.ensure!(repo: repo, name: label)
+        PlanMyStuff::Label.add!(issue: self, labels: [label], user: user) if labels.exclude?(label)
+
+        self.class.update!(
+          number: number,
+          repo: repo,
+          metadata: {
+            waiting_on_user_at: PlanMyStuff.format_time(now),
+            next_reminder_at: format_next_reminder_at(from: now),
+          },
+        )
+        reload
+      end
+
+      # Clears the waiting-on-user state: removes the label, clears +metadata.waiting_on_user_at+, and clears
+      # +metadata.next_reminder_at+ unless a waiting-on-approval timer is still active. No-ops if the issue is not
+      # currently waiting on a user reply.
+      #
+      # @return [self]
+      #
+      def clear_waiting_on_user!
+        label = PlanMyStuff.configuration.waiting_on_user_label
+        return self if metadata.waiting_on_user_at.nil? && labels.exclude?(label)
+
+        PlanMyStuff::Label.remove!(issue: self, labels: [label]) if labels.include?(label)
+
+        self.class.update!(
+          number: number,
+          repo: repo,
+          metadata: {
+            waiting_on_user_at: nil,
+            next_reminder_at:
+              metadata.waiting_on_approval_at ? PlanMyStuff.format_time(metadata.next_reminder_at) : nil,
+          },
+        )
+        reload
+      end
+
+      # Reopens an issue that was auto-closed by the inactivity sweep, clears +metadata.closed_by_inactivity+, and
+      # emits +plan_my_stuff.issue.reopened_by_reply+ carrying the reopening comment. Does not emit the regular
+      # +issue.reopened+ event \- subscribers that specifically care about this flow subscribe to the dedicated event.
+      #
+      # @param comment [PlanMyStuff::Comment] the reopening comment
+      # @param user [Object, nil] actor for the notification event
+      #
+      # @return [self]
+      #
+      def reopen_by_reply!(comment:, user: nil)
+        inactive_label = PlanMyStuff.configuration.user_inactive_label
+        PlanMyStuff::Label.remove!(issue: self, labels: [inactive_label]) if labels.include?(inactive_label)
+
+        self.class.update!(
+          number: number,
+          repo: repo,
+          state: :open,
+          metadata: { closed_by_inactivity: false },
+        )
+        reload
+
+        PlanMyStuff::Notifications.instrument(
+          'issue.reopened_by_reply',
+          self,
+          user: user,
+          comment: comment,
+        )
+        self
+      end
+
+      private
+
+        # Formats the next reminder time as an ISO 8601 UTC string, using per-issue +metadata.reminder_days+ when set
+        # or +config.reminder_days+ otherwise. Returns +nil+ when the effective schedule is empty.
+        #
+        # @param from [Time] baseline timestamp
+        #
+        # @return [String, nil]
+        #
+        def format_next_reminder_at(from:)
+          days = metadata.reminder_days.presence || PlanMyStuff.configuration.reminder_days
+          return if days.empty?
+
+          PlanMyStuff.format_time(from + days.first.days)
+        end
+
+        # When an issue is transitioning from open to closed, strips both waiting labels from the outgoing labels
+        # array and clears the waiting-related timestamps on +metadata+ so a single save writes both state change and
+        # cleanup. No-op for any other transition.
+        #
+        # @param attrs [Hash] the kwargs hash being assembled for +Issue.update!+; mutated in place
+        #
+        # @return [void]
+        #
+        def clear_waiting_state_on_close(attrs)
+          return unless state_changed?
+          return unless state_was == 'open'
+          return unless state == 'closed'
+
+          return if metadata.waiting_on_user_at.blank? && metadata.waiting_on_approval_at.blank?
+
+          waiting_labels = [
+            PlanMyStuff.configuration.waiting_on_user_label,
+            PlanMyStuff.configuration.waiting_on_approval_label,
+          ]
+          attrs[:labels] = Array.wrap(attrs[:labels]) - waiting_labels
+
+          metadata.waiting_on_user_at = nil
+          metadata.waiting_on_approval_at = nil
+          metadata.next_reminder_at = nil
+        end
+
+        # When an inactivity-closed issue is being reopened, strips the +user_inactive_label+ from the outgoing labels
+        # and clears +metadata.closed_by_inactivity+ so the save writes both. No-op for any other transition or for
+        # reopens of non-inactive closes.
+        #
+        # @param attrs [Hash] the kwargs hash being assembled for +Issue.update!+; mutated in place
+        #
+        # @return [void]
+        #
+        def clear_inactivity_state_on_reopen(attrs)
+          return unless state_changed?
+          return unless state_was == 'closed'
+          return unless state == 'open'
+          return unless metadata.closed_by_inactivity
+
+          attrs[:labels] = Array.wrap(attrs[:labels]) - [PlanMyStuff.configuration.user_inactive_label]
+          metadata.closed_by_inactivity = false
+        end
+    end
+  end
+end

data/lib/plan_my_stuff/label.rb

@@ -48,7 +48,7 @@ module PlanMyStuff
         client = PlanMyStuff.client
         client.rest(:label, repo, name)
       rescue PlanMyStuff::APIError => e
-        raise unless e.status == 404
+        raise(e) unless e.status == 404
 
         create_label!(client, repo, name, color, description)
       end
@@ -62,11 +62,11 @@ module PlanMyStuff
       #
       # @return [Boolean]
       #
-      def exists?(repo:, name:)
+      def exists?(repo:, name:, do_raise: true)
         PlanMyStuff.client.rest(:label, repo, name)
         true
       rescue PlanMyStuff::APIError => e
-        raise unless e.status == 404
+        raise(e) if do_raise && e.status != 404
 
         false
       end
@@ -115,7 +115,7 @@ module PlanMyStuff
 
           client.rest(:add_label, repo, name, color, **options)
         rescue PlanMyStuff::APIError => e
-          raise unless e.status == 422
+          raise(e) unless e.status == 422
         end
 
         # Hydrates a Label from a GitHub API response.

data/lib/plan_my_stuff/version.rb

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

data/lib/tasks/plan_my_stuff.rake

@@ -46,7 +46,7 @@ namespace :plan_my_stuff do
       url = ENV.fetch('URL') { raise(ArgumentError, 'URL env var is required') }
       repo = ENV.fetch('REPO') do
         PlanMyStuff.client.resolve_repo!
-      rescue
+      rescue PlanMyStuff::ConfigurationError, ArgumentError
         raise(ArgumentError, 'REPO env var is required or configured (e.g. BrandsInsurance/PlanMyStuff)')
       end
       default_events = %w[pull_request issues]
@@ -94,7 +94,7 @@ 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=30]). ' \
       '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'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: plan_my_stuff
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.10.0
 platform: ruby
 authors:
 - Brands Insurance
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-30 00:00:00.000000000 Z
+date: 2026-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -52,6 +52,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - CHANGELOG.md
+- CONFIGURATION.md
 - LICENSE
 - README.md
 - app/controllers/plan_my_stuff/application_controller.rb
@@ -112,6 +113,7 @@ files:
 - lib/plan_my_stuff/aws_sns_simulator.rb
 - lib/plan_my_stuff/base_metadata.rb
 - lib/plan_my_stuff/base_project.rb
+- lib/plan_my_stuff/base_project_extractions/graphql_hydration.rb
 - lib/plan_my_stuff/base_project_item.rb
 - lib/plan_my_stuff/base_project_metadata.rb
 - lib/plan_my_stuff/cache.rb
@@ -124,6 +126,10 @@ files:
 - lib/plan_my_stuff/errors.rb
 - lib/plan_my_stuff/graphql/queries.rb
 - lib/plan_my_stuff/issue.rb
+- lib/plan_my_stuff/issue_extractions/approvals.rb
+- lib/plan_my_stuff/issue_extractions/links.rb
+- lib/plan_my_stuff/issue_extractions/viewers.rb
+- lib/plan_my_stuff/issue_extractions/waiting.rb
 - lib/plan_my_stuff/issue_metadata.rb
 - lib/plan_my_stuff/label.rb
 - lib/plan_my_stuff/link.rb