plan_my_stuff 0.8.0 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6799008cb353423175bdd3132e0fa5f14981b3710378c3b0886a8e9a2394b610
-  data.tar.gz: 75b8336ec9aabd1d77a2642de77c9e46310dda3da49e6abd97b997927e906d05
+  metadata.gz: 3d74ab84a6dc7a2e69a33df3e7c1babc51e8e0e9813b388b541b5021c7a61169
+  data.tar.gz: 492a5c2be1f50cb1fe899d101dee7157a779ff649c292e0075fbe89cc5f8cd55
 SHA512:
-  metadata.gz: fdb5b8790905ff4f529496ad2a5c63b13a59a58ce524a8df79b20b0efc536854bc986f0e2fe7896b77bfe9b0a15598b6212a83991d46c9925e4dbf9eecab4871
-  data.tar.gz: b5c150efa84a7ed3143935216d0ac52ae9c8fa7d99cb45b6add6552c13f3b66fad2f604e1aa8871cb05fc69b3aa16e650bce14a60ad1490d0650af3a3e646d91
+  metadata.gz: 2838c1763d069dc702b99e3d89591b8bc9bec52c2d686ccef1e85ceefa3d9d1fbfe90b0b1875d6d200ec6a88236d5e2619d6bf2151e73b075dfca7b693e2ef0c
+  data.tar.gz: c42b1d88a9fbbc07c1c8f7db547af43a9b3417858e0a542c6a0cc1ae2addf76fb44630a9675b5ab6f1999e26330822fdf5500245cbedb08946e3da5029086f93

data/CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## 0.10.0
+
+### Changed
+
+- `Issue` slimmed from 1791 to 911 lines by extracting feature clusters into per-feature modules under a sibling
+  `PlanMyStuff::IssueExtractions::*` namespace, included into `Issue`. Public API unchanged (`issue.approve!`,
+  `issue.add_related!`, `issue.enter_waiting_on_user!`, `issue.add_viewers!`, etc. still resolve to the same methods).
+  - `PlanMyStuff::IssueExtractions::Approvals` - `lib/plan_my_stuff/issue_extractions/approvals.rb`
+  - `PlanMyStuff::IssueExtractions::Links` - `lib/plan_my_stuff/issue_extractions/links.rb`
+  - `PlanMyStuff::IssueExtractions::Waiting` - `lib/plan_my_stuff/issue_extractions/waiting.rb`
+  - `PlanMyStuff::IssueExtractions::Viewers` - `lib/plan_my_stuff/issue_extractions/viewers.rb`
+- `BaseProject` slimmed from 661 to 504 lines by extracting GraphQL hydration helpers into
+  `PlanMyStuff::BaseProjectExtractions::GraphqlHydration` (`lib/plan_my_stuff/base_project_extractions/graphql_hydration.rb`),
+  included into `BaseProject`'s singleton class. `BaseProject.find` / `BaseProject.list` remain public entry points.
+- Specs for `Issue` features now live alongside the modules at `spec/plan_my_stuff/issue_extractions/<feature>_spec.rb`.
+
+## 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

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative 'base_project_extractions/graphql_hydration'
+
 module PlanMyStuff
   # Shared base for GitHub Projects V2 wrappers. Holds attribute definitions, generic find/list/update machinery,
   # hydration, and instance helpers. Concrete subclasses (Project, TestingProject) add their own +create!+ behavior
@@ -43,6 +45,8 @@ module PlanMyStuff
     attribute :has_next_page
 
     class << self
+      include PlanMyStuff::BaseProjectExtractions::GraphqlHydration
+
       # Generic find - returns whichever concrete project type is at the given number, dispatching on metadata kind.
       # Subclasses may override to apply filtering (e.g. Project raises for testing projects by default).
       #
@@ -180,47 +184,6 @@ module PlanMyStuff
 
       private
 
-        # Builds a summary Project from a list query node. Dispatches to TestingProject when the readme metadata has
-        # kind: "testing".
-        #
-        # @param node [Hash]
-        #
-        # @return [PlanMyStuff::BaseProject]
-        #
-        def build_summary(node)
-          raw_readme = node[:readme] || ''
-          parsed_meta = PlanMyStuff::MetadataParser.parse(raw_readme)
-          klass = dispatch_project_class(parsed_meta[:metadata])
-          project = klass.new
-          project.__send__(:hydrate_summary, node, raw_readme: raw_readme, parsed_meta: parsed_meta)
-          project
-        end
-
-        # Builds a detailed Project from a find query response. Dispatches to TestingProject when the readme metadata
-        # has kind: "testing".
-        #
-        # @param graphql_project [Hash]
-        # @param items [Array<Hash>]
-        # @param next_cursor [String, nil]
-        # @param has_next_page [Boolean, nil]
-        #
-        # @return [PlanMyStuff::BaseProject]
-        #
-        def build_detail(graphql_project, items:, next_cursor: nil, has_next_page: nil)
-          raw_readme = graphql_project[:readme] || ''
-          parsed_meta = PlanMyStuff::MetadataParser.parse(raw_readme)
-          klass = dispatch_project_class(parsed_meta[:metadata])
-          project = klass.new
-          project.__send__(
-            :hydrate_detail,
-            graphql_project,
-            items: items,
-            next_cursor: next_cursor,
-            has_next_page: has_next_page,
-          )
-          project
-        end
-
         # Returns the appropriate project class based on the metadata kind field. Always dispatches to a concrete
         # subclass (never BaseProject itself).
         #
@@ -234,140 +197,6 @@ module PlanMyStuff
           PlanMyStuff::Project
         end
 
-        # @param org [String]
-        # @param number [Integer]
-        #
-        # @return [PlanMyStuff::BaseProject]
-        #
-        def find_auto_paginated(org, number)
-          all_items = []
-          cursor = nil
-          raw_project = nil
-          page = nil
-
-          loop do
-            page = fetch_project_page(org, number, cursor)
-            raw_project ||= page[:raw]
-            all_items.concat(page[:items])
-
-            break if !page[:has_next_page] || all_items.length >= MAX_AUTO_PAGINATE_ITEMS
-
-            cursor = page[:next_cursor]
-          end
-
-          build_detail(
-            raw_project,
-            items: all_items,
-            next_cursor: page[:next_cursor],
-            has_next_page: page[:has_next_page],
-          )
-        end
-
-        # @param org [String]
-        # @param number [Integer]
-        # @param cursor [String, nil]
-        #
-        # @return [PlanMyStuff::BaseProject]
-        #
-        def find_with_cursor(org, number, cursor:)
-          page = fetch_project_page(org, number, cursor)
-          build_detail(
-            page[:raw],
-            items: page[:items],
-            next_cursor: page[:next_cursor],
-            has_next_page: page[:has_next_page],
-          )
-        end
-
-        # Fetches a single page of project data. Returns a lightweight hash for pagination loop consumption (not a
-        # Project instance).
-        #
-        # @param org [String]
-        # @param number [Integer]
-        # @param cursor [String, nil]
-        #
-        # @return [Hash] with :raw, :items, :next_cursor, :has_next_page
-        #
-        def fetch_project_page(org, number, cursor)
-          variables = { org: org, number: number }
-          variables[:cursor] = cursor if cursor
-
-          data = PlanMyStuff.client.graphql(
-            PlanMyStuff::GraphQL::Queries::FIND_PROJECT,
-            variables: variables,
-          )
-
-          raw_project = data.dig(:organization, :projectV2)
-          page_info = raw_project.dig(:items, :pageInfo) || {}
-          items_data = raw_project.dig(:items, :nodes) || []
-
-          {
-            raw: raw_project,
-            items: items_data.map { |item| parse_project_item(item) },
-            next_cursor: page_info[:endCursor],
-            has_next_page: page_info[:hasNextPage],
-          }
-        end
-
-        # @param item [Hash] raw GraphQL project item node
-        #
-        # @return [Hash]
-        #
-        def parse_project_item(item)
-          content = item[:content] || {}
-          field_values = item.dig(:fieldValues, :nodes) || []
-          repo_name = content.dig(:repository, :nameWithOwner)
-
-          {
-            id: item[:id],
-            type: item[:type],
-            content_node_id: content[:id],
-            title: content[:title],
-            body: content[:body],
-            number: content[:number],
-            url: content[:url],
-            state: content[:state],
-            repo: repo_name.present? ? PlanMyStuff::Repo.resolve!(repo_name) : nil,
-            status: extract_item_status(field_values),
-            field_values: parse_field_values(field_values),
-            updated_at: item[:updatedAt],
-            github_response: item,
-          }
-        end
-
-        # @param field_values [Array<Hash>]
-        #
-        # @return [String, nil]
-        #
-        def extract_item_status(field_values)
-          status_value = field_values.find { |fv| fv.dig(:field, :name) == 'Status' }
-
-          status_value&.dig(:name)
-        end
-
-        # @param field_values [Array<Hash>]
-        #
-        # @return [Hash]
-        #
-        def parse_field_values(field_values)
-          result = {}
-
-          field_values.each do |fv|
-            field_name = fv.dig(:field, :name)
-            next unless field_name
-
-            value = fv[:name] || fv[:text]
-            users_node = fv[:users]
-            if users_node
-              value = (users_node[:nodes] || []).map { |u| u[:login] }
-            end
-
-            result[field_name] = value
-          end
-
-          result
-        end
-
         # Resolves a project number to its node ID.
         #
         # @param org [String]
@@ -406,7 +235,7 @@ module PlanMyStuff
     #
     def status_field
       status_field!
-    rescue
+    rescue PlanMyStuff::APIError
       nil
     end