plan_my_stuff 0.8.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6799008cb353423175bdd3132e0fa5f14981b3710378c3b0886a8e9a2394b610
-  data.tar.gz: 75b8336ec9aabd1d77a2642de77c9e46310dda3da49e6abd97b997927e906d05
+  metadata.gz: 6457257ef431e923519943cd227ffc70488b023467253873e20f52c47fd75f52
+  data.tar.gz: b5a16da4dc6c6c3ce45b307879f5146720f2b878809b7a071b7cc9e33986d210
 SHA512:
-  metadata.gz: fdb5b8790905ff4f529496ad2a5c63b13a59a58ce524a8df79b20b0efc536854bc986f0e2fe7896b77bfe9b0a15598b6212a83991d46c9925e4dbf9eecab4871
-  data.tar.gz: b5c150efa84a7ed3143935216d0ac52ae9c8fa7d99cb45b6add6552c13f3b66fad2f604e1aa8871cb05fc69b3aa16e650bce14a60ad1490d0650af3a3e646d91
+  metadata.gz: efee5f594d9bd014b0ce575781790768a8d51277010d9ad9902ac11beea06c17d39232d65a77532768326eccece3ebcbe3bdb18c464341c963eda9b12b1d6075
+  data.tar.gz: ef94bc70d08874c2347e243d0f7967ca7bca2e631cda775494a714df6501d22fe5011a09db805dbb53f5fe8397eb32b8bdf19bbf8a235da39d3f5dd4810c011a

data/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## 0.9.0
+
+### 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
+
+### Added
+
+- `CONFIGURATION.md` is now available bundled with the gem
+
+### Changed
+
+- `config.repos` is now assignable as a whole hash (`config.repos = { atlas: 'Org/Atlas', autofill: 'Org/atlas-autofill-moz' }`) in addition to the existing `config.repos[:key] = '...'` form
+- `Issue.create!` / `Issue.update!` `issue_type:` kwarg now accepts the Symbol nicknames (`'bug'`, `'feature'`, `'it_issue'`, `'other'`, `'performance'`, `'question'`, `'task'`) as Strings too, resolving them to the same canonical name
+
 ## 0.8.0
 
 ### Breaking

data/CONFIGURATION.md

@@ -0,0 +1,351 @@
+# Configuration
+
+Every PlanMyStuff option lives on `PlanMyStuff::Configuration`. The install generator drops a fully-commented copy at
+`config/initializers/plan_my_stuff.rb` — this file documents the same options grouped by concern. Defaults shown apply
+when the option is left unset.
+
+```ruby
+PMS.configure do |config|
+  # ...
+end
+```
+
+`PMS` is an alias for `PlanMyStuff`; consuming apps can use either.
+
+## Authentication (required)
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `access_token` | `String` | — | GitHub PAT with `repo` and `project` scopes. Required. |
+| `import_access_token` | `String, nil` | `nil` | Classic PAT (requires `repo` scope) for the Issues Import API. Fine-grained tokens are not supported by that endpoint. |
+| `organization` | `String` | — | GitHub organization name. Required. |
+
+Both `access_token` and `organization` are validated by `config.validate!`; missing or blank values raise `PlanMyStuff::ConfigurationError`.
+
+> [!NOTE]
+> `import_access_token` is only required if you choose to use `PMS::Issue.import!`/`PMS::Issue.check_import!`
+
+```ruby
+config.access_token = Rails.application.credentials.dig(:plan_my_stuff, :github_token)
+config.import_access_token = Rails.application.credentials.dig(:plan_my_stuff, :github_import_token)
+config.organization = 'YourOrg'
+```
+
+## Repositories
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `repos` | `Hash{Symbol => String}` | `{}` | Named repo configs mapping a key to an `Org/Repo` string. |
+| `default_repo` | `Symbol, nil` | `nil` | Repo key used when callers omit the `repo:` param. |
+
+```ruby
+config.repos = { element: 'YourOrg/Element', underwriter: 'YourOrg/Underwriter' }
+config.default_repo = :element
+```
+
+`repos` can be mutated via `config.repos[:key] = '...'` or set via `config.repos = { key: 'MyOrg/MyRepo' }`
+
+## Projects
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `default_project_number` | `Integer, nil` | `nil` | Default Projects V2 number for `add_to_project: true`. |
+| `testing_template_project_number` | `Integer, nil` | `nil` | Project to clone in `TestingProject.create!` instead of bootstrapping fields. |
+
+```ruby
+config.default_project_number = 14
+config.testing_template_project_number = 42
+```
+
+## App identity
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `app_name` | `String, nil` | `nil` | Stored in metadata so subscribers can attribute writes. |
+| `issues_url_prefix` | `String, nil` | `nil` | Prefix for `Issue#user_link`; the gem appends the issue number. |
+
+```ruby
+config.app_name = 'MyApp'
+
+url_options = Rails.application.config.action_mailer.default_url_options
+config.issues_url_prefix = "#{url_options[:protocol] || 'http'}://#{url_options[:host]}/issues"
+```
+
+## User integration
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `user_class` | `String` | `'User'` | Consuming app's user model class name, constantized for lookups. |
+| `display_name_method` | `Symbol` | `:to_s` | Method called on a user to get the display name for comment headers. |
+| `user_id_method` | `Symbol` | `:id` | Method called on a user to extract the app-side user ID. |
+| `support_method` | `Symbol, Proc` | `:support?` | Method name on the user, or a proc receiving the user, returning whether they're support staff. |
+| `github_login_for` | `Hash{Object => String}` | `{}` | Maps app user id (from `user_id_method`) to GitHub login. Powers the Take UI. |
+
+`support_method` may be a method name on the user object or a proc that receives the user and returns boolean.
+`github_login_for` maps app user id (whatever `user_id_method` returns) to GitHub login and powers the Take UI.
+
+```ruby
+config.user_class = 'User'
+config.display_name_method = :full_name
+config.user_id_method = :id
+config.support_method = :support?
+# or: config.support_method = -> (user) { user.role.in?(%w[support admin]) }
+config.github_login_for = {
+  1 => 'some_username',
+  2 => 'octocat',
+}
+```
+
+## Engine authentication
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `authenticate_with` | `Proc` | `nil` | Block executed as a `before_action` on every engine controller. |
+
+```ruby
+config.authenticate_with do
+  redirect_to main_app.login_path unless current_user
+end
+```
+
+## Markdown rendering
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `markdown_renderer` | `Symbol` | `:commonmarker` | Which markdown gem to use: `:commonmarker`, `:redcarpet`, or `nil` (raw HTML-escaped). |
+| `markdown_options` | `Hash` | `{}` | Default options passed to the renderer. Per-call options merge on top of these. |
+
+Set `markdown_renderer` to `:commonmarker`, `:redcarpet`, or `nil` (raw HTML-escaped). The chosen gem must be in your
+Gemfile.
+
+```ruby
+config.markdown_renderer = :commonmarker
+config.markdown_options = { render: { hardbreaks: true } }
+```
+
+## Notification actor
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `current_user` | `Proc, nil` | `nil` | Fallback actor for notification events when `user:` is not passed. |
+
+```ruby
+config.current_user = -> { Current.user }
+```
+
+## Controller rescue
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `controller_rescue` | `Proc, nil` | `nil` | Receives the rescued exception. Forward to your monitoring service. |
+
+```ruby
+config.controller_rescue = -> (error) { MonitoringService.notice_error(error) }
+```
+
+## Custom fields
+
+App-defined fields stored in metadata. Keys are field names; values are hashes with `:type` and `:required`.
+Supported types: `:string`, `:integer`, `:boolean`, `:array`, `:hash`.
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `custom_fields` | `Hash{Symbol => Hash}` | `{}` | Shared field definitions across all contexts. |
+| `issue_custom_fields` | `Hash{Symbol => Hash}` | `{}` | Issue-only definitions; merged on top of shared. |
+| `comment_custom_fields` | `Hash{Symbol => Hash}` | `{}` | Comment-only definitions; merged on top of shared. |
+| `project_custom_fields` | `Hash{Symbol => Hash}` | `{}` | Project-only definitions; merged on top of shared. |
+| `testing_custom_fields` | `Hash{Symbol => Hash}` | `{}` | Testing-project-only definitions; merged on top of shared. |
+
+Context-specific config wins on key conflicts.
+
+```ruby
+config.custom_fields = {
+  notification_recipients: { type: :array, required: true },
+}
+config.issue_custom_fields = {
+  ticket_type: { type: :string, required: true },
+}
+config.comment_custom_fields = {
+  internal_note: { type: :boolean },
+}
+config.project_custom_fields = {
+  team: { type: :string },
+}
+config.testing_custom_fields = {
+  test_plan_url: { type: :string },
+}
+```
+
+## Issue types
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `issue_types` | `Hash{String => String}` | `{}` | Maps the gem's canonical issue type names to your org's display names. |
+
+Maps the gem's canonical type names (`'Bug'`, `'Feature'`, `'IT Issue / Hardware'`, `'Other'`, `'Performance'`,
+`'Question'`, `'Task'`) to whatever your org uses. Missing keys pass through unchanged.
+
+```ruby
+config.issue_types = {
+  'Bug' => 'User Bug',
+  'Feature' => 'Enhancement',
+}
+```
+
+## Release pipeline
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `pipeline_enabled` | `Boolean` | `true` | Whether the release pipeline feature is enabled. |
+| `pipeline_project_number` | `Integer, nil` | `nil` | Projects V2 number for the pipeline board. Falls back to `default_project_number`. |
+| `webhook_secret` | `String, nil` | `nil` | HMAC secret for GitHub webhook signature verification. Required when webhook routes are mounted. |
+| `pipeline_statuses` | `Hash{String => String}` | `{}` | Display aliases for canonical pipeline status names. |
+| `pipeline_testing_field_name` | `String` | `'Testing'` | Display name for the pipeline project's `Testing` single-select field. |
+| `pipeline_testing_values` | `Hash{Symbol => String}` | `{ active: 'Testing', inactive: 'Not testing' }` | Display labels for the canonical `:active`/`:inactive` testing options. |
+| `pipeline_completion_purge_enabled` | `Boolean` | `true` | Whether the sweep removes aged-out `Completed` items from the pipeline. |
+| `pipeline_completion_ttl_hours` | `Integer` | `24` | Hours after a `Completed` item's last update at which the sweep removes it. |
+| `main_branch` | `String` | `'main'` | Branch PRs merge into for the "Ready for Release" transition. |
+| `production_branch` | `String` | `'production'` | Branch PRs mereg into for the "Release in Progress" transition. |
+
+`pipeline_statuses` aliases the canonical status names (`'Submitted'`, `'Started'`, `'In Review'`, `'Testing'`,
+`'Ready for Release'`, `'Release in Progress'`, `'Completed'`) for display only. The constants remain the internal
+identifiers.
+
+`pipeline_completion_*` controls the sweep that removes aged-out `Completed` items from the pipeline. `webhook_secret`
+is required when webhook routes are mounted.
+
+```ruby
+config.pipeline_enabled = true
+config.pipeline_project_number = 14
+config.webhook_secret = Rails.application.credentials.dig(:plan_my_stuff, :webhook_secret)
+config.pipeline_statuses = {
+  'Submitted' => 'Triaged',
+  'Completed' => 'Done',
+}
+config.pipeline_testing_field_name = 'Testing'
+config.pipeline_testing_values = { active: 'Testing', inactive: 'Not testing' }
+config.pipeline_completion_purge_enabled = true
+config.pipeline_completion_ttl_hours = 24
+config.main_branch = 'main'
+config.production_branch = 'production'
+```
+
+## Follow-up reminders
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `reminders_enabled` | `Boolean` | `true` | Whether the reminders sweep performs any work. |
+| `reminder_days` | `Array<Integer>` | `[1, 3, 7, 10, 14, 18]` | Days-since-waiting at which reminder events fire. |
+| `inactivity_close_days` | `Integer` | `30` | Days of inactivity after which the sweep auto-closes a waiting issue. |
+| `waiting_on_user_label` | `String` | `'waiting-on-user'` | Label flagging issues waiting on an end-user reply. |
+| `waiting_on_approval_label` | `String` | `'waiting-on-approval'` | Label flagging issues waiting on pending approvals. |
+| `user_inactive_label` | `String` | `'user-inactive'` | Label applied to issues auto-closed by the inactivity sweep; removed when an issue is auto-reopened via a user reply. |
+
+Per-issue reminder override: `issue.metadata.reminder_days = [...]`.
+
+```ruby
+config.reminders_enabled = true
+config.reminder_days = [1, 3, 7, 10, 14, 18]
+config.inactivity_close_days = 30
+config.waiting_on_user_label = 'waiting-on-user'
+config.waiting_on_approval_label = 'waiting-on-approval'
+config.user_inactive_label = 'user-inactive'
+```
+
+## Auto-archiving
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `archiving_enabled` | `Boolean` | `true` | Whether the archive sweep performs any work. |
+| `archive_closed_after_days` | `Integer` | `90` | Days after `closed_at` at which a non-inactive-closed issue becomes an archive candidate. |
+| `archived_label` | `String` | `'archived'` | Label added to archived issues; also used by the sweep as a skip marker. |
+
+The archive sweep piggybacks `RemindersSweepJob`. Inactivity-closed and non-PMS issues are excluded.
+
+```ruby
+config.archiving_enabled = true
+config.archive_closed_after_days = 90
+config.archived_label = 'archived'
+```
+
+## AWS webhook
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `sns_topic_arn` | `String, nil` | `nil` | Expected SNS topic ARN for AWS webhook validation. |
+| `aws_service_identifier` | `String, nil` | `nil` | Suffix matched against ECS event resource ARNs (e.g. `'my-app-production-2-web-server'`). |
+| `production_commit_sha` | `String, nil` | `nil` | Prefix-matched against issue metadata `commit_sha` on `SERVICE_DEPLOYMENT_COMPLETED` events. |
+| `process_aws_webhooks` | `Boolean` | `Rails.env.production?` | Whether to process incoming AWS webhook events. |
+| `sns_verifier_class` | `Class` | `Aws::SNS::MessageVerifier` (when defined) | Class instantiated per request for SNS signature verification. Must respond to `authenticate!(raw_body)`. |
+| `sns_verifier_error` | `Class` | `Aws::SNS::MessageVerifier::VerificationError` (when defined) | Exception class rescued during SNS signature verification. |
+
+`production_commit_sha` is prefix-matched against issue metadata `commit_sha` on `SERVICE_DEPLOYMENT_COMPLETED` events.
+`sns_verifier_class` must respond to `authenticate!(raw_body)`.
+
+```ruby
+config.sns_topic_arn = 'arn:aws:sns:us-east-1:123456:ecs-deploy-topic'
+config.aws_service_identifier = 'myapp-production-web'
+config.production_commit_sha = Rails.configuration.x.image_tag
+config.process_aws_webhooks = Rails.env.production?
+config.sns_verifier_class = Aws::SNS::MessageVerifier
+config.sns_verifier_error = Aws::SNS::MessageVerifier::VerificationError
+```
+
+## Caching
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `cache_enabled` | `Boolean` | `true` | ETag-based HTTP caching of GitHub reads via `Rails.cache`. |
+| `cache_version` | `String, nil` | `nil` | Opaque string baked into every PMS cache key; bump to invalidate. |
+
+```ruby
+config.cache_enabled = true
+config.cache_version = Rails.configuration.x.image_tag
+```
+
+## Route mounting
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `mount_groups` | `Hash{Symbol => Boolean}` | `{ webhooks: true, issues: true, projects: true }` | Per-group route mounting toggles. Set a key to `false` to skip mounting that group. |
+
+```ruby
+config.mount_groups = { webhooks: true, issues: true, projects: true }
+```
+
+## Controller overrides
+
+| Option | Type | Default | Description |
+|---|---|---|---|
+| `controllers` | `Hash{Symbol => String}` | `{}` | Per-route controller overrides. Keys are controllable route symbols; values are fully-qualified controller paths. |
+
+Per-route controller overrides. Keys are the controllable route symbols defined in
+`PlanMyStuff::Configuration::DEFAULT_CONTROLLERS`; values are fully-qualified controller paths. Unset keys fall back to
+the gem default.
+
+```ruby
+config.controllers[:issues] = 'my_app/issues'
+```
+
+Controllable keys (with gem defaults):
+
+| Key | Default |
+|---|---|
+| `:issues` | `plan_my_stuff/issues` |
+| `:comments` | `plan_my_stuff/comments` |
+| `:labels` | `plan_my_stuff/labels` |
+| `:projects` | `plan_my_stuff/projects` |
+| `:project_items` | `plan_my_stuff/project_items` |
+| `:testing_projects` | `plan_my_stuff/testing_projects` |
+| `:testing_project_items` | `plan_my_stuff/testing_project_items` |
+| `:'issues/closures'` | `plan_my_stuff/issues/closures` |
+| `:'issues/viewers'` | `plan_my_stuff/issues/viewers` |
+| `:'issues/takes'` | `plan_my_stuff/issues/takes` |
+| `:'issues/waitings'` | `plan_my_stuff/issues/waitings` |
+| `:'issues/links'` | `plan_my_stuff/issues/links` |
+| `:'issues/approvals'` | `plan_my_stuff/issues/approvals` |
+| `:'project_items/statuses'` | `plan_my_stuff/project_items/statuses` |
+| `:'project_items/assignments'` | `plan_my_stuff/project_items/assignments` |
+| `:'testing_project_items/results'` | `plan_my_stuff/testing_project_items/results` |
+| `:'webhooks/github'` | `plan_my_stuff/webhooks/github` |
+| `:'webhooks/aws'` | `plan_my_stuff/webhooks/aws` |

data/app/views/plan_my_stuff/issues/show.html.erb

@@ -21,7 +21,7 @@
     <%= button_to('Take', plan_my_stuff.issue_take_path(@issue.number, repo: @issue.repo.full_name), method: :post) %>
   <% end %>
   <% if @support_user && @pipeline_enabled && @current_user_login.present? && @issue.assignees.include?(@current_user_login) %>
-    <%= button_to('Release', plan_my_stuff.issue_take_path(@issue.number, repo: @issue.repo.full_name), method: :delete) %>
+    <%= button_to('Unassign', plan_my_stuff.issue_take_path(@issue.number, repo: @issue.repo.full_name), method: :delete) %>
   <% end %>
   <% if @support_user %>
     <%= link_to('Start Testing Project', plan_my_stuff.new_testing_project_path(subject_url: @issue.html_url)) %>

data/app/views/plan_my_stuff/partials/_flash.html.erb

@@ -1,4 +1,3 @@
-<%# locals: () %>
 <% if flash[:error].present? %>
   <p role="alert" style="color: red;"><%= flash[:error] %></p>
 <% end %>

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

@@ -106,19 +106,8 @@ PlanMyStuff.configure do |config|
   #     "#{url_options[:protocol] || 'http'}://#{url_options[:host]}/issues"
 
   # --------------------------------------------------------------------------
-  # Request gateway
+  # Notification actor
   # --------------------------------------------------------------------------
-  # Proc returning boolean, or nil (always send). When it returns false the
-  # request is deferred to a background job instead of hitting GitHub.
-  # config.should_send_request = -> { !MaintenanceMode.active? }
-
-  # Map of action type to job class name for deferred requests.
-  # config.job_classes = {
-  #   create_ticket:  'PmsCreateTicketJob',
-  #   post_comment:   'PmsPostCommentJob',
-  #   update_status:  'PmsUpdateStatusJob'
-  # }
-
   # Fallback actor for notification events (plan_my_stuff.*) when a caller
   # does not pass an explicit user: kwarg. Proc/lambda called at event time.
   # config.current_user = -> { Current.user }

data/lib/plan_my_stuff/base_project.rb

@@ -406,7 +406,7 @@ module PlanMyStuff
     #
     def status_field
       status_field!
-    rescue
+    rescue PlanMyStuff::APIError
       nil
     end
 

data/lib/plan_my_stuff/base_project_item.rb

@@ -73,6 +73,7 @@ module PlanMyStuff
           repo: item_hash[:repo],
           state: item_hash[:state],
           status: item_hash[:status],
+          updated_at: item_hash[:updated_at],
           field_values: item_hash[:field_values] || {},
           project: project,
         )

data/lib/plan_my_stuff/comment.rb

@@ -32,6 +32,8 @@ module PlanMyStuff
     class << self
       # Creates a comment on a GitHub issue with PMS metadata and a visible header.
       #
+      # @raise [PlanMyStuff::LockedIssueError] if the parent issue is locked
+      #
       # @param issue [PlanMyStuff::Issue] parent issue
       # @param body [String]
       # @param user [Object, Integer] user object or user_id
@@ -41,8 +43,6 @@ module PlanMyStuff
       # @param waiting_on_reply [Boolean] when true and the author is a support user, marks the issue as waiting on
       #   an end-user reply. Ignored for non-support authors.
       #
-      # @raise [PlanMyStuff::LockedIssueError] if the parent issue is locked
-      #
       # @return [PlanMyStuff::Comment]
       #
       def create!(
@@ -279,7 +279,9 @@ module PlanMyStuff
         )
         hydrate_from_comment(created)
       else
-        update!(user: user, body: body)
+        update_attrs = { user: user, body: body }
+        update_attrs[:visibility] = visibility if visibility_changed?
+        update!(**update_attrs)
       end
 
       self

data/lib/plan_my_stuff/configuration.rb

@@ -84,19 +84,6 @@ module PlanMyStuff
     #
     attr_accessor :markdown_options
 
-    # Proc returning boolean, or nil (always send). When it returns false the request is deferred to a background job
-    # instead of hitting GitHub.
-    #
-    # @return [Proc, nil]
-    #
-    attr_accessor :should_send_request
-
-    # Map of action type to job class name for deferred requests. Keys: :create_ticket, :post_comment, :update_status.
-    #
-    # @return [Hash{Symbol => String}]
-    #
-    attr_accessor :job_classes
-
     # Fallback actor for notification events when a caller does not pass +user:+. Set to a proc/lambda that returns the
     # current request user.
     #
@@ -344,11 +331,12 @@ module PlanMyStuff
     #
     attr_accessor :sns_verifier_error
 
-    # Named repo configs. Set via config.repos[:element] = 'BrandsInsurance/Element'.
+    # Named repo configs. Set via config.repos[:element] = 'BrandsInsurance/Element', or assign a whole hash with
+    # config.repos = { element: 'BrandsInsurance/Element', underwriter: 'BrandsInsurance/Underwriter' }.
     #
     # @return [Hash{Symbol => String}]
     #
-    attr_reader :repos
+    attr_accessor :repos
 
     # @return [Configuration]
     def initialize
@@ -359,7 +347,6 @@ module PlanMyStuff
       @support_method = :support?
       @markdown_renderer = :commonmarker
       @markdown_options = {}
-      @job_classes = {}
       @custom_fields = {}
       @issue_custom_fields = {}
       @comment_custom_fields = {}

data/lib/plan_my_stuff/issue.rb

@@ -363,7 +363,11 @@ module PlanMyStuff
           canonical =
             case value
             when String
-              value
+              begin
+                resolve_issue_type!(value.to_sym)
+              rescue ArgumentError
+                value
+              end
             when Symbol
               ISSUE_TYPE_NICKNAMES[value] || raise(
                 ArgumentError,

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 = 9
     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.9.0
 platform: ruby
 authors:
 - Brands Insurance
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-30 00:00:00.000000000 Z
+date: 2026-05-01 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