plan_my_stuff 0.1.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b39dc4e08c17c126407b4b96f698d4718b77f9485f251af2f7a1c642e58a1b8
-  data.tar.gz: 335dbb910692194950e6ac9bcc904d302b67ddf34d4cf0e21611dd3bf36d675d
+  metadata.gz: 9b5b0d6a9e0582853ebf435a48b49fabb8dc9bd1227bddc6f2c3467d74872c80
+  data.tar.gz: 3e1df8e562940c9c1435c2edc4b88c7b85c2c2c8309b95702f8f6d3f978bdf19
 SHA512:
-  metadata.gz: 0f9b3a98fd6085704eafe6bf19a90bca60010f7004c28913b6356d96cef8ac08145f82e12a389fde0adb40c2ffa4eb7aca1aa3ea9f2065ed3549a28f6eacffdd
-  data.tar.gz: 303c6bec1248313856b9dad67234a77344ac1e9ea66e2c90361e498524c9f46f9da142a5c278f2866806be1437fc701097699c67c86890e02757c31632a657cc
+  metadata.gz: fdc842d462d7e04c163560c475bd6519e04b4b5f57b2684be5137ed32ff0bc9e257d9f09e7ab1cda06e6106065e38e4a1c80f0fea96d812841141cc14cbed4a6
+  data.tar.gz: 9e4954b597a281e4c68e1ea9ecbe2748efbcac31062f7fd3b925b887dc0616015df27df8c0122fc4165621227f5eefa13ead1ce0583cbe9dcd377ecaf0e47e23

data/CHANGELOG.md

@@ -0,0 +1,27 @@
+# Changelog
+
+## 0.2.0
+
+### Added
+
+- Validate `config.custom_fields` on `create!`/`update!` actions
+
+### Changes
+
+- CodeRabbit improvements
+
+## 0.1.2
+
+### Fixed
+
+- Issues' links were looking for the issue in the default repo with the same number
+
+## 0.1.1
+
+### Fixed
+
+- Wrong changelog included in publish
+
+## 0.1.0
+
+- Initial Setup

data/app/controllers/plan_my_stuff/application_controller.rb

@@ -70,7 +70,10 @@ module PlanMyStuff
       def parse_viewer_ids(ids_string)
         return [] if ids_string.blank?
 
-        ids_string.split(',').filter_map { |id| id.strip.presence&.to_i }
+        ids_string.split(',').filter_map do |id|
+          token = id.strip
+          token.to_i if token.match?(/\A\d+\z/)
+        end
       end
   end
 end

data/app/controllers/plan_my_stuff/comments_controller.rb

@@ -4,7 +4,7 @@ module PlanMyStuff
   class CommentsController < ApplicationController
     # POST /issues/:issue_id/comments
     def create
-      @issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo]&.to_sym)
+      @issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo])
 
       PMS::Comment.create!(
         issue: @issue,
@@ -14,25 +14,33 @@ module PlanMyStuff
       )
 
       flash[:success] = 'Comment was successfully created.'
-      redirect_to(plan_my_stuff.issue_path(@issue.number))
+      redirect_to(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
     end
 
     # GET /issues/:issue_id/comments/:id/edit
     def edit
       load_comment
       return unless @comment
+      return redirect_to_issue if issue_body_comment?
 
       @support_user = support_user?
-      return redirect_to_unauthorized(plan_my_stuff.issue_path(@issue.number)) unless can_edit?(@comment)
+      return if can_edit?(@comment)
+
+      redirect_to_unauthorized(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
     end
 
     # PATCH/PUT /issues/:issue_id/comments/:id
     def update
       load_comment
       return unless @comment
+      return redirect_to_issue if issue_body_comment?
 
       @support_user = support_user?
-      return redirect_to_unauthorized(plan_my_stuff.issue_path(@issue.number)) unless can_edit?(@comment)
+      unless can_edit?(@comment)
+        redirect_to_unauthorized(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
+
+        return
+      end
 
       update_attrs = { body: comment_params[:body] }
       update_attrs[:visibility] = comment_params[:visibility].to_sym if @support_user && comment_params[:visibility]
@@ -40,7 +48,7 @@ module PlanMyStuff
       @comment.update!(**update_attrs)
 
       flash[:success] = 'Comment was successfully updated.'
-      redirect_to(plan_my_stuff.issue_path(@issue.number))
+      redirect_to(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
     rescue PMS::StaleObjectError
       flash.now[:error] = 'Comment was modified by someone else. Please review the latest changes and try again.'
       render(:edit, status: PMS.unprocessable_status)
@@ -58,7 +66,7 @@ module PlanMyStuff
       # @return [void]
       #
       def load_comment
-        @issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo]&.to_sym)
+        @issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo])
         @comment = PMS::Comment.find(params[:id].to_i, issue: @issue)
       end
 
@@ -78,5 +86,15 @@ module PlanMyStuff
 
         comment.metadata.created_by == PMS::UserResolver.user_id(user)
       end
+
+      # @return [Boolean]
+      def issue_body_comment?
+        @comment.metadata.issue_body?
+      end
+
+      # @return [void]
+      def redirect_to_issue
+        redirect_to(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
+      end
   end
 end

data/app/controllers/plan_my_stuff/issues_controller.rb

@@ -8,7 +8,7 @@ module PlanMyStuff
       @per_page = (params[:per_page] || 25).to_i
       @state = (params[:state] || 'open').to_sym
       @labels = params[:labels].present? ? Array.wrap(params[:labels]) : []
-      @repo = params[:repo]&.to_sym
+      @repo = params[:repo]
 
       @issues = PMS::Issue.list(
         repo: @repo,
@@ -35,7 +35,7 @@ module PlanMyStuff
       )
 
       flash[:success] = 'Issue was successfully created.'
-      redirect_to(plan_my_stuff.issue_path(@issue.number))
+      redirect_to(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
     rescue PMS::ValidationError => e
       @issue = PMS::Issue.new(title: issue_params[:title], body: issue_params[:body])
       @support_user = support_user?
@@ -45,7 +45,7 @@ module PlanMyStuff
 
     # GET /issues/:id
     def show
-      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo]&.to_sym)
+      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo])
       @comments = filter_visible_comments(@issue.comments)
       @support_user = support_user?
       @current_user_id = pms_current_user.present? ? PMS::UserResolver.user_id(pms_current_user) : nil
@@ -53,13 +53,13 @@ module PlanMyStuff
 
     # GET /issues/:id/edit
     def edit
-      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo]&.to_sym)
+      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo])
       @support_user = support_user?
     end
 
     # PATCH/PUT /issues/:id
     def update
-      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo]&.to_sym)
+      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo])
 
       @issue.update!(
         title: issue_params[:title],
@@ -68,7 +68,7 @@ module PlanMyStuff
       )
 
       flash[:success] = 'Issue was successfully updated.'
-      redirect_to(plan_my_stuff.issue_path(@issue.number))
+      redirect_to(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
     rescue PMS::StaleObjectError
       @support_user = support_user?
       flash.now[:error] = 'Issue was modified by someone else. Please review the latest changes and try again.'
@@ -77,48 +77,54 @@ module PlanMyStuff
 
     # PATCH /issues/:id/close
     def close
-      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo]&.to_sym)
+      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo])
       @issue.update!(state: :closed)
 
       flash[:success] = 'Issue was successfully closed.'
-      redirect_to(plan_my_stuff.issue_path(@issue.number))
+      redirect_to(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
     end
 
     # PATCH /issues/:id/reopen
     def reopen
-      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo]&.to_sym)
+      @issue = PMS::Issue.find(params[:id].to_i, repo: params[:repo])
       @issue.update!(state: :open)
 
       flash[:success] = 'Issue was successfully reopened.'
-      redirect_to(plan_my_stuff.issue_path(@issue.number))
+      redirect_to(plan_my_stuff.issue_path(@issue.number, repo: @issue.repo.full_name))
     end
 
     # POST /issues/:id/add_viewers
     def add_viewers
-      return redirect_to_unauthorized(plan_my_stuff.issue_path(params[:id])) unless support_user?
+      unless support_user?
+        redirect_to_unauthorized(plan_my_stuff.issue_path(params[:id], repo: params[:repo]))
+        return
+      end
 
       viewer_ids = parse_viewer_ids(params[:viewer_ids])
       if viewer_ids.blank?
         flash[:error] = 'No valid viewer IDs provided.'
-        redirect_to(plan_my_stuff.edit_issue_path(params[:id]))
+        redirect_to(plan_my_stuff.edit_issue_path(params[:id], repo: params[:repo]))
         return
       end
 
-      PMS::Issue.add_viewers(number: params[:id].to_i, user_ids: viewer_ids, repo: params[:repo]&.to_sym)
+      PMS::Issue.add_viewers(number: params[:id].to_i, user_ids: viewer_ids, repo: params[:repo])
 
       flash[:success] = 'Viewers were successfully added.'
-      redirect_to(plan_my_stuff.edit_issue_path(params[:id]))
+      redirect_to(plan_my_stuff.edit_issue_path(params[:id], repo: params[:repo]))
     end
 
     # DELETE /issues/:id/remove_viewer
     def remove_viewer
-      return redirect_to_unauthorized(plan_my_stuff.issue_path(params[:id])) unless support_user?
+      unless support_user?
+        redirect_to_unauthorized(plan_my_stuff.issue_path(params[:id], repo: params[:repo]))
+        return
+      end
 
       viewer_id = params[:viewer_id].to_i
-      PMS::Issue.remove_viewers(number: params[:id].to_i, user_ids: [viewer_id], repo: params[:repo]&.to_sym)
+      PMS::Issue.remove_viewers(number: params[:id].to_i, user_ids: [viewer_id], repo: params[:repo])
 
       flash[:success] = 'Viewer was successfully removed.'
-      redirect_to(plan_my_stuff.edit_issue_path(params[:id]))
+      redirect_to(plan_my_stuff.edit_issue_path(params[:id], repo: params[:repo]))
     end
 
     private

data/app/controllers/plan_my_stuff/labels_controller.rb

@@ -7,24 +7,24 @@ module PlanMyStuff
       labels = parse_labels(params[:label_name])
       if labels.blank?
         flash[:error] = 'Label name is required.'
-        redirect_to(plan_my_stuff.issue_path(params[:issue_id]))
+        redirect_to(plan_my_stuff.issue_path(params[:issue_id], repo: params[:repo]))
         return
       end
 
-      issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo]&.to_sym)
+      issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo])
       PMS::Label.add(issue: issue, labels: labels)
 
       flash[:success] = 'Label was successfully added.'
-      redirect_to(plan_my_stuff.issue_path(issue.number))
+      redirect_to(plan_my_stuff.issue_path(issue.number, repo: issue.repo.full_name))
     end
 
     # DELETE /issues/:issue_id/labels/:name
     def remove_from_issue
-      issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo]&.to_sym)
+      issue = PMS::Issue.find(params[:issue_id].to_i, repo: params[:repo])
       PMS::Label.remove(issue: issue, labels: [params[:name]])
 
       flash[:success] = 'Label was successfully removed.'
-      redirect_to(plan_my_stuff.issue_path(issue.number))
+      redirect_to(plan_my_stuff.issue_path(issue.number, repo: issue.repo.full_name))
     end
   end
 end

data/app/controllers/plan_my_stuff/project_items_controller.rb

@@ -50,6 +50,12 @@ module PlanMyStuff
 
     # PATCH /projects/:project_id/items/:id/unassign
     def unassign
+      if params[:username].blank?
+        flash[:error] = 'Username is required to unassign.'
+        redirect_to(plan_my_stuff.project_path(params[:project_id]))
+        return
+      end
+
       item = find_project_item
       current_assignees = item.field_values['Assignees'] || []
       remaining = current_assignees - [params[:username]]

data/app/controllers/plan_my_stuff/projects_controller.rb

@@ -7,11 +7,65 @@ module PlanMyStuff
       @projects = PMS::Project.list.reject(&:closed)
     end
 
+    # GET /projects/new
+    def new
+      @project = PMS::Project.new
+    end
+
+    # POST /projects
+    def create
+      @project = PMS::Project.create!(
+        title: project_params[:title],
+        readme: project_params[:readme] || '',
+        description: project_params[:description],
+        user: pms_current_user,
+      )
+
+      flash[:success] = 'Project was successfully created.'
+      redirect_to(plan_my_stuff.project_path(@project.number))
+    rescue PMS::ValidationError => e
+      @project = PMS::Project.new(
+        title: project_params[:title],
+        readme: project_params[:readme],
+      )
+      flash.now[:error] = e.message
+      render(:new, status: PMS.unprocessable_status)
+    end
+
     # GET /projects/:id
     def show
       @project = PMS::Project.find(params[:id].to_i)
       @statuses = @project.statuses.pluck(:name)
       @items_by_status = @project.items.group_by(&:status)
     end
+
+    # GET /projects/:id/edit
+    def edit
+      @project = PMS::Project.find(params[:id].to_i)
+    end
+
+    # PATCH/PUT /projects/:id
+    def update
+      @project = PMS::Project.find(params[:id].to_i)
+
+      @project.update!(
+        title: project_params[:title],
+        readme: project_params[:readme],
+        description: project_params[:description],
+      )
+
+      flash[:success] = 'Project was successfully updated.'
+      redirect_to(plan_my_stuff.project_path(@project.number))
+    rescue PMS::StaleObjectError
+      flash.now[:error] = 'Project was modified by someone else. Please review the latest changes and try again.'
+      render(:edit, status: PMS.unprocessable_status)
+    end
+
+    private
+
+      # @return [ActionController::Parameters]
+      def project_params
+        params.require(:project).permit(:title, :readme, :description)
+      end
   end
 end

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

@@ -14,7 +14,7 @@
       <% @issues.each do |issue| %>
         <tr>
           <td><%= issue.number %></td>
-          <td><%= link_to(issue.title, plan_my_stuff.issue_path(issue.number)) %></td>
+          <td><%= link_to(issue.title, plan_my_stuff.issue_path(issue.number, repo: issue.repo.full_name)) %></td>
           <td><%= issue.state %></td>
           <td><%= issue.labels.join(', ') %></td>
         </tr>
@@ -24,14 +24,14 @@
 
   <nav>
     <% if @page > 1 %>
-      <%= link_to('Previous', plan_my_stuff.issues_path(page: @page - 1, state: @state, labels: @labels)) %>
+      <%= link_to('Previous', plan_my_stuff.issues_path(page: @page - 1, state: @state, labels: @labels, repo: @repo)) %>
     <% end %>
     <% if @issues.size == @per_page %>
-      <%= link_to('Next', plan_my_stuff.issues_path(page: @page + 1, state: @state, labels: @labels)) %>
+      <%= link_to('Next', plan_my_stuff.issues_path(page: @page + 1, state: @state, labels: @labels, repo: @repo)) %>
     <% end %>
   </nav>
 <% else %>
   <p>No issues found.</p>
 <% end %>
 <br>
-<%= link_to "New Issue", plan_my_stuff.new_issue_path %>
+<%= link_to "New Issue", plan_my_stuff.new_issue_path(repo: @repo) %>

data/app/views/plan_my_stuff/issues/partials/_form.html.erb

@@ -1,9 +1,13 @@
-<% persisted = issue.persisted? %>
-<%= form_with(
-  url: persisted ? plan_my_stuff.issue_path(issue.number) : plan_my_stuff.issues_path,
-  method: persisted ? :patch : :post,
-  scope: :issue,
-) do |form| %>
+<%
+  persisted = issue.persisted?
+  url =
+    if persisted
+      plan_my_stuff.issue_path(issue.number, repo: issue.repo&.full_name)
+    else
+      plan_my_stuff.issues_path(repo: issue.repo&.full_name)
+    end
+%>
+<%= form_with(url: url, method: persisted ? :patch : :post, scope: :issue) do |form| %>
   <div>
     <%= form.label(:title, 'Title') %>
     <%= form.text_field(:title, value: issue.title) %>

data/app/views/plan_my_stuff/issues/partials/_viewers.html.erb

@@ -9,7 +9,7 @@
         <%=
           button_to(
             'Remove',
-            plan_my_stuff.remove_viewer_issue_path(issue.number, viewer_id: viewer_id),
+            plan_my_stuff.remove_viewer_issue_path(issue.number, viewer_id: viewer_id, repo: issue.repo.full_name),
             method: :delete
           )
         %>
@@ -20,7 +20,7 @@
   <p>No viewers added.</p>
 <% end %>
 
-<%= form_with(url: plan_my_stuff.add_viewers_issue_path(issue.number), method: :post) do |form| %>
+<%= form_with(url: plan_my_stuff.add_viewers_issue_path(issue.number, repo: issue.repo.full_name), method: :post) do |form| %>
   <div>
     <%= form.label(:viewer_ids, 'Add viewer IDs (comma-separated)') %>
     <%= form.text_field(:viewer_ids) %>

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

@@ -1,11 +1,11 @@
 <h1><%= @issue.title %> <small>#<%= @issue.number %></small></h1>
 
 <p>
-  <%= link_to('Edit', plan_my_stuff.edit_issue_path(@issue.number)) %>
+  <%= link_to('Edit', plan_my_stuff.edit_issue_path(@issue.number, repo: @issue.repo.full_name)) %>
   <% if @issue.state == 'open' %>
-    <%= button_to('Close Issue', plan_my_stuff.close_issue_path(@issue.number), method: :patch) %>
+    <%= button_to('Close Issue', plan_my_stuff.close_issue_path(@issue.number, repo: @issue.repo.full_name), method: :patch) %>
   <% else %>
-    <%= button_to('Reopen Issue', plan_my_stuff.reopen_issue_path(@issue.number), method: :patch) %>
+    <%= button_to('Reopen Issue', plan_my_stuff.reopen_issue_path(@issue.number, repo: @issue.repo.full_name), method: :patch) %>
   <% end %>
 </p>
 
@@ -35,7 +35,7 @@
       <%= PlanMyStuff::Markdown.render(comment.body || '').html_safe %>
       <% if comment.pms_comment? && (@support_user || comment.metadata.created_by == @current_user_id) %>
         <p>
-          <%= link_to('Edit', plan_my_stuff.edit_issue_comment_path(@issue.number, comment.id)) %>
+          <%= link_to('Edit', plan_my_stuff.edit_issue_comment_path(@issue.number, comment.id, repo: @issue.repo.full_name)) %>
         </p>
       <% end %>
     </div>

data/app/views/plan_my_stuff/projects/edit.html.erb

@@ -0,0 +1,7 @@
+<h1>Edit Project #<%= @project.number %></h1>
+
+<% if flash[:error].present? %>
+  <p style="color: red;"><%= flash[:error] %></p>
+<% end %>
+
+<%= render({ partial: 'plan_my_stuff/projects/partials/form', locals: { project: @project } }) %>

data/app/views/plan_my_stuff/projects/index.html.erb

@@ -11,3 +11,5 @@
 <% else %>
   <p>No projects found.</p>
 <% end %>
+
+<p><%= link_to('New Project', plan_my_stuff.new_project_path) %></p>

data/app/views/plan_my_stuff/projects/new.html.erb

@@ -0,0 +1,7 @@
+<h1>New Project</h1>
+
+<% if flash[:error].present? %>
+  <p style="color: red;"><%= flash[:error] %></p>
+<% end %>
+
+<%= render({ partial: 'plan_my_stuff/projects/partials/form', locals: { project: @project } }) %>

data/app/views/plan_my_stuff/projects/partials/_form.html.erb

@@ -0,0 +1,29 @@
+<%
+  persisted = project.persisted?
+  url =
+    if persisted
+      plan_my_stuff.project_path(project.number)
+    else
+      plan_my_stuff.projects_path
+    end
+%>
+<%= form_with(url: url, method: persisted ? :patch : :post, scope: :project) do |form| %>
+  <div>
+    <%= form.label(:title, 'Title') %>
+    <%= form.text_field(:title, value: project.title, required: true) %>
+  </div>
+
+  <div>
+    <%= form.label(:description, 'Description') %>
+    <%= form.text_field(:description, value: project.description) %>
+  </div>
+
+  <div>
+    <%= form.label(:readme, 'Readme') %>
+    <%= form.text_area(:readme, rows: 8, value: project.readme) %>
+  </div>
+
+  <div>
+    <%= form.submit(persisted ? 'Update Project' : 'Create Project') %>
+  </div>
+<% end %>

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

@@ -1,5 +1,7 @@
 <h1><%= @project.title %></h1>
 
+<p><%= link_to('Edit', plan_my_stuff.edit_project_path(@project.number)) %></p>
+
 <% if @statuses.any? %>
   <table>
     <thead>
@@ -16,12 +18,12 @@
             <% items = @items_by_status[status] || [] %>
             <% items.each do |item| %>
               <div style="border: 1px solid black; margin: 1em">
-                <% if item.number.present? %>
-                  <strong><%= link_to(item.title, plan_my_stuff.issue_path(item.number)) %></strong>
+                <% unless item.draft? %>
+                  <strong><%= link_to(item.title, plan_my_stuff.issue_path(item.number, repo: item.repo.full_name)) %></strong>
                 <% else %>
                   <strong><%= item.title %></strong>
                 <% end %>
-                <% if item.number.present? %>
+                <% unless item.draft? %>
                   <small>#<%= item.number %></small>
                 <% end %>
 
@@ -46,7 +48,7 @@
                             url: plan_my_stuff.unassign_project_item_path(@project.number, item.id),
                             method: :patch,
                             local: true,
-                            style: 'display: inline',
+                            html: { style: 'display: inline' },
                           ) do |form|
                         %>
                           <%= form.hidden_field(:username, value: username) %>

data/config/routes.rb

@@ -13,7 +13,7 @@ PlanMyStuff::Engine.routes.draw do
     delete 'labels/:name', to: 'labels#remove_from_issue', as: :remove_label
   end
 
-  resources :projects, only: %i[index show] do
+  resources :projects, except: %i[destroy] do
     resources :items, only: %i[create], controller: 'project_items' do
       member do
         patch :move

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

@@ -100,7 +100,16 @@ PMS.configure do |config|
   # Supported types: :string, :integer, :boolean, :array, :hash
   # required: true means the key must be present (value can be nil, [], etc.)
   #
+  # Shared fields (available on both issues and comments):
   config.custom_fields = {
-    notification_recipients: { type: :array, required: true },
+    # notification_recipients: { type: :array, required: true },
   }
+
+  # Issue-only fields (merged on top of shared, context wins on conflicts):
+  # config.issue_custom_fields = {
+  #   ticket_type: { type: :string, required: true },
+  # }
+
+  # Comment-only fields (merged on top of shared, context wins on conflicts):
+  # config.comment_custom_fields = {}
 end

data/lib/plan_my_stuff/application_record.rb

@@ -8,6 +8,19 @@ module PlanMyStuff
   class ApplicationRecord
     include ActiveModel::Model
 
+    class << self
+      # Reads a field from an object that may respond to method calls or hash access.
+      #
+      # @param obj [Object]
+      # @param field [Symbol]
+      #
+      # @return [Object]
+      #
+      def read_field(obj, field)
+        obj.respond_to?(field) ? obj.public_send(field) : obj[field]
+      end
+    end
+
     def initialize(**)
       super
       @persisted = false
@@ -33,7 +46,30 @@ module PlanMyStuff
       # @return [Object]
       #
       def read_field(obj, field)
-        obj.respond_to?(field) ? obj.public_send(field) : obj[field]
+        self.class.read_field(obj, field)
+      end
+
+      # Reads a field from an object, returning nil if the field does not exist.
+      #
+      # @param obj [Object]
+      # @param field [Symbol]
+      #
+      # @return [Object, nil]
+      #
+      def safe_read_field(obj, field)
+        read_field(obj, field)
+      rescue NameError
+        nil
+      end
+
+      # @return [Time, nil]
+      def parse_github_time(value)
+        return if value.nil?
+        return value.utc if value.is_a?(Time)
+
+        Time.parse(value.to_s).utc
+      rescue ArgumentError
+        nil
       end
   end
 end