Add TaskRowComponent extensibility to Strata SDK tasks index (#304)

## Changes

> What was added, updated, or removed in this PR.

- Introduce Strata::Tasks::TaskRowComponent, following the
CaseRowComponent pattern: configurable columns and headers, sortable
cell data attributes, and row_classes for host styling.
- Strata::TasksController#index now renders strata/tasks/index
explicitly and exposes tasks_index_locals (task_row_component_class
defaults to the base component) so applications can inject subclasses
and pass row options without forking the template.
- Add strata.tasks.index.actions_label, update the staff generator to
stop emitting a redundant tasks/index wrapper, refresh
implementing-tasks-views documentation, and remove the dummy app shim
view now that the controller owns the render path.

## Context

> Background context, more in-depth details of the implementation, and
anything else you'd like to call out or ask reviewers.

- The tasks index table is owned by the engine; host apps customize
behavior by subclassing Strata::TasksController and overriding
tasks_index_locals (e.g. super.merge(task_row_component_class:,
task_row_component_options:)). That mirrors how cases use a configurable
row component, without requiring a host app/views/tasks/index.html.erb.
- For reviewers: Any app that still ships a local tasks/index wrapper
should drop it and use tasks_index_locals + a TaskRowComponent subclass
instead.

## Testing

> Provide evidence that the code works as expected. Explain what was
done for testing and the results of the test plan. Include screenshots,
[GIF demos](https://www.cockos.com/licecap/), shell commands or output
to help show the changes working as expected. ProTip: you can drag and
drop or paste images into this textbox.

- Component & generator: bundle exec rspec
spec/components/strata/tasks/task_row_component_spec.rb
spec/lib/generators/strata/generators/staff_generator_spec.rb — passing.
- Controller: bundle exec rspec
spec/controllers/strata/tasks_controller_spec.rb — passing; one example
remains pending (pre-existing skip around routing).
- Optional manual check: load the tasks index in the dummy app and
confirm the table, filters, and "Pick next task" behave as before; add a
screenshot or GIF if your team wants UI proof.

Author: jgavin-nava

app/components/strata/tasks/task_row_component.html.erb

@@ -0,0 +1,5 @@
+<% self.class.columns.each do |column| %>
+  <%= tag.td(**cell_html_options(column)) do %>
+    <%= send(column) || "—" %>
+  <% end %>
+<% end %>

app/components/strata/tasks/task_row_component.rb

@@ -0,0 +1,87 @@
+# frozen_string_literal: true
+
+module Strata
+  module Tasks
+    # TaskRowComponent renders one row of the tasks index table. Host apps can subclass
+    # and override {.columns}, {.header_translation_for}, {.header_options_for}, and
+    # column instance methods to add or reorder columns.
+    #
+    # @example Default usage (from +strata/tasks/index+)
+    #   <%= render TaskRowComponent.new(task: task) %>
+    #
+    class TaskRowComponent < ViewComponent::Base
+      def initialize(task:, **)
+        @task = task
+      end
+
+      def self.columns
+        %i[due_date type case_id created_date]
+      end
+
+      def self.headers
+        columns.map { |column| header_translation_for(column) }
+      end
+
+      def self.header_translation_for(column)
+        I18n.t("strata.tasks.index.columns.col_#{column}")
+      end
+
+      def self.header_options_for(column)
+        return { aria_sort: "descending" } if column == :created_date
+
+        {}
+      end
+
+      def row_classes
+        nil
+      end
+
+      protected
+
+      def due_date
+        helpers.local_en_us(@task.due_on)
+      end
+
+      def type
+        link_to I18n.t("tasks.types.#{@task.type.underscore}"),
+                helpers.url_for(only_path: true, action: :show, id: @task.id.to_s)
+      end
+
+      def case_id
+        @task.case_id
+      end
+
+      def created_date
+        helpers.local_en_us(@task.created_at.to_date)
+      end
+
+      def sort_value_for(column)
+        case column.to_sym
+        when :due_date
+          helpers.time_since_epoch(@task.due_on)
+        when :type
+          @task.type
+        when :case_id
+          @task.case_id
+        when :created_date
+          helpers.time_since_epoch(@task.created_at)
+        else
+          nil
+        end
+      end
+
+      def cell_classes(_column)
+        nil
+      end
+
+      def cell_html_options(column)
+        attrs = {}
+        sv = sort_value_for(column)
+        attrs[:data] = { sort_value: sv } unless sv.nil?
+        cc = cell_classes(column)
+        attrs[:class] = cc if cc.present?
+        attrs
+      end
+    end
+  end
+end

app/controllers/strata/tasks_controller.rb

@@ -4,8 +4,6 @@ module Strata
   # Controller for managing Strata::Task records. Handles listing, filtering, showing, and updating tasks.
   # This controller helps a parent application manage tasks by not forcing the parent application to implement the same functionality.
   class TasksController < ::StaffController
-    helper DateHelper
-
     before_action :set_task, only: %i[ show update ]
     before_action :set_case, only: %i[ show update ]
     before_action :set_application_form, only: %i[ show update]
@@ -15,6 +13,7 @@ def index
       @task_types = Strata::Task.distinct(:type).unscope(:order).pluck(:type) # Postgres does not support using `order` with `distinct`, thus we have to unscope `order` here.
       @tasks = filter_tasks
       @unassigned_tasks = Strata::Task.incomplete.unassigned
+      render "strata/tasks/index", locals: tasks_index_locals
     end
 
     def show
@@ -44,6 +43,16 @@ def pick_up_next_task
 
     protected
 
+    def tasks_index_locals
+      {
+        tasks: @tasks,
+        task_types: @task_types,
+        unassigned_tasks: @unassigned_tasks,
+        task_row_component_class: Strata::Tasks::TaskRowComponent,
+        task_row_component_options: {}
+      }
+    end
+
     def set_task
       @task = Strata::Task.find(params[:id]) if params[:id].present?
     end

app/helpers/strata/application_helper.rb

@@ -7,6 +7,8 @@ module Strata
   # @see Strata::FormBuilder for more information about available form helpers
   #
   module ApplicationHelper
+    include DateHelper
+
     def strata_form_with(model: false, scope: nil, url: nil, format: nil, **options, &block)
       options[:builder] = Strata::FormBuilder
       args = { scope: scope, url: url, format: format, **options }

app/views/strata/tasks/index.html.erb

@@ -3,6 +3,8 @@
   # @param tasks [Array<Strata::Task>] Array of tasks to display
   # @param unassigned_tasks [Array<Strata::Task>] Array of unassigned tasks for button state
   # @param task_types [Array<String>] Array of task types for filtering
+  # @param task_row_component_class [Class] ViewComponent subclass of {Strata::Tasks::TaskRowComponent}
+  # @param task_row_component_options [Hash] Extra keyword arguments passed to each row component
   #
   # @example Task types array
   #   ["Strata::Task", "Strata::OtherTask", "MyCustomTask"]
@@ -13,6 +15,8 @@
   #   - strata.tasks.index.tabs.completed: Label for the completed tasks tab
   #
  %>
+<% task_row_component_class = local_assigns.fetch(:task_row_component_class, Strata::Tasks::TaskRowComponent) %>
+<% task_row_component_options = local_assigns.fetch(:task_row_component_options, {}) %>
 <!-- Main content -->
 <div class="grid-container">
   <h1 class="usa-heading-xl"><%= t("strata.tasks.index.title") %></h1>
@@ -31,7 +35,7 @@
       <%= render partial: 'strata/tasks/due_date_filter' %>
     </div>
     <div class="grid-col-12 tablet:grid-col-3">
-      <label class="usa-label" for="next-task-button" style="margin-bottom: 0.5rem">Actions:</label>
+      <label class="usa-label" for="next-task-button" style="margin-bottom: 0.5rem"><%= t("strata.tasks.index.actions_label") %></label>
       <%= button_to t("strata.tasks.actions.pick_next"), url_for(only_path: true, action: :pick_up_next_task),
                       id: "next-task-button",
                       disabled: !unassigned_tasks&.any?,
@@ -44,19 +48,16 @@
     <!-- Tasks -->
     <%= render Strata::US::TableComponent.new(striped: true, sticky_header: true, sortable: true, classes: "width-full") do |table| %>
       <% table.with_caption do %><span class="usa-sr-only"><%= t("strata.tasks.index.title") %></span><% end %>
-      <% table.with_header(sortable: true) { t("strata.tasks.index.columns.col_due_date") } %>
-      <% table.with_header(sortable: true) { t("strata.tasks.index.columns.col_type") } %>
-      <% table.with_header(sortable: true) { t("strata.tasks.index.columns.col_case_id") } %>
-      <% table.with_header(sortable: true, aria_sort: "descending") { t("strata.tasks.index.columns.col_created_date") } %>
+      <% task_row_component_class.columns.each do |column| %>
+        <% table.with_header(sortable: true, **task_row_component_class.header_options_for(column)) do %>
+          <%= task_row_component_class.header_translation_for(column) %>
+        <% end %>
+      <% end %>
 
       <% tasks.each do |task| %>
-        <% table.with_row do |row| %>
-          <% row.with_cell(sort_value: time_since_epoch(task.due_on)) { local_en_us(task.due_on) } %>
-          <% row.with_cell(sort_value: task.type) do %>
-            <%= link_to t("tasks.types.#{task.type.underscore}"), url_for(only_path: true, action: :show, id: task.id.to_s) %>
-          <% end %>
-          <% row.with_cell(sort_value: task.case_id) { task.case_id } %>
-          <% row.with_cell(sort_value: time_since_epoch(task.created_at)) { local_en_us(task.created_at.to_date) } %>
+        <% row_component = task_row_component_class.new(task: task, **task_row_component_options) %>
+        <% table.with_row(classes: row_component.row_classes) do %>
+          <%= render row_component %>
         <% end %>
       <% end %>
     <% end %>

config/locales/strata/en.yml

@@ -69,6 +69,7 @@ en:
         tasks: "Tasks"
       index:
         title: "Tasks"
+        actions_label: "Actions:"
         tabs:
           assigned: "Assigned"
           completed: "Completed"

docs/implementing-tasks-views.md

@@ -1,39 +1,40 @@
 # Implementing Task Views
 
-This guide explains how to implement and customize the task views in the Strata engine, with a focus on the `index.html.erb` template.
+This guide explains how to implement and customize the task views in the Strata engine, with a focus on the tasks index.
 
 ## Quick Start Implementation
 
-To implement the tasks index view in your application, create an `index.html.erb` file in your views directory with the following content:
+`Strata::TasksController#index` renders the engine template `strata/tasks/index` and passes locals from `tasks_index_locals`. Host applications normally **do not** add `app/views/tasks/index.html.erb`; override `tasks_index_locals` (or `index`) in your `TasksController` subclass to customize behavior.
 
-```erb
-<%= render template: 'strata/tasks/index', locals: {
-  tasks: @tasks,
-  task_types: @task_types,
-  unassigned_tasks: @unassigned_tasks
-} %>
-```
+### Default locals
+
+The engine supplies:
 
-### Required Local Variables
+- `tasks`, `task_types`, `unassigned_tasks` (from instance variables)
+- `task_row_component_class` (defaults to `Strata::Tasks::TaskRowComponent`)
+- `task_row_component_options` (defaults to `{}`)
 
-The view requires the following local variables to be passed:
+### Custom task columns
 
-- `tasks`: An array of `Strata::Task` objects to display in the list
-- `task_types`: An array of available task types for filtering
-- `unassigned_tasks`: An array of `Strata::Task` objects used to determine the state of the "Pick Next Task" button
+Subclass `Strata::Tasks::TaskRowComponent` and pass your class in `tasks_index_locals`, following the same pattern as `Strata::Cases::CaseRowComponent` for the cases index.
 
-### Controller Setup
+### Controller setup
 
-In your controller, ensure you have the following instance variables set:
+Override `tasks_index_locals` when you need extra row options or a custom row component:
 
 ```ruby
-def index
-  @tasks = Strata::Task.where(assignee_id: current_user.id) # or your task retrieval logic
-  @task_types = ["MyCustomTask", "OtherTask", Strata::Task.name]
-  @unassigned_tasks = Strata::Task.where(assignee_id: nil) # or your unassigned tasks logic
+protected
+
+def tasks_index_locals
+  super.merge(
+    task_row_component_class: MyApp::TaskRowComponent,
+    task_row_component_options: { my_data: @my_data }
+  )
 end
 ```
 
+The default `Strata::TasksController#index` sets `@task_types`, `@tasks`, and `@unassigned_tasks` before rendering.
+
 ## Internationalization (i18n)
 
 The view uses several translation keys that you can override in your application:

lib/generators/strata/staff/USAGE

@@ -14,7 +14,6 @@ This will create:
 
     Views:
         app/views/staff/index.html.erb - Staff dashboard index page
-        app/views/tasks/index.html.erb - Tasks listing page that renders strata/tasks/index template
         app/views/tasks/show.html.erb - Task detail page that renders strata/tasks/show template
 
     Tests:
@@ -31,6 +30,6 @@ This will create:
 Notes:
     - The generated staff controller includes TODO comments for implementing authentication and authorization
     - The tasks controller includes TODO comments for setting up case and application form relationships
-    - The generated views render existing strata/tasks templates with appropriate locals
+    - Strata::TasksController renders the tasks index from the engine; override tasks_index_locals to customize row components
     - You'll need to customize the case_classes method in the staff controller to match your application's case models
     - The generated spec file includes TODO placeholders for implementing comprehensive tests

lib/generators/strata/staff/staff_generator.rb

@@ -20,7 +20,6 @@ def create_tasks_controller
 
       def create_views
         template "staff_index.html.erb", "app/views/staff/index.html.erb"
-        template "tasks_index.html.erb", "app/views/tasks/index.html.erb"
         template "task_show.html.erb", "app/views/tasks/show.html.erb"
       end