fix: handle issues.edited webhook to update ticket title and description

- Add handler for issues.edited events to update ticket metadata
- Fix pull_request.edited to also update title (was only updating description)
- Add comprehensive tests for both issue and PR metadata updates
- Fixes issue where renaming a GitHub issue wouldn't update the bounty title

This ensures that when users rename issues or PRs on GitHub, the corresponding
tickets in Algora are updated with the new title and description.

Author: zcesur

lib/algora_web/controllers/webhooks/github_controller.ex

@@ -274,6 +274,9 @@ defmodule AlgoraWeb.Webhooks.GithubController do
     end
   end
 
+  defp process_event(%Webhook{event_action: "issues.edited"} = webhook, _commands),
+    do: handle_ticket_metadata_change(webhook)
+
   defp process_event(%Webhook{event_action: event_action, payload: payload}, commands)
        when event_action in ["pull_request.opened", "pull_request.reopened", "pull_request.edited"] do
     source =
@@ -285,7 +288,10 @@ defmodule AlgoraWeb.Webhooks.GithubController do
 
     if source do
       source
-      |> change(%{description: payload["pull_request"]["body"]})
+      |> change(%{
+        title: payload["pull_request"]["title"],
+        description: payload["pull_request"]["body"]
+      })
       |> Repo.update()
     end
 
@@ -699,6 +705,30 @@ defmodule AlgoraWeb.Webhooks.GithubController do
     end
   end
 
+  defp handle_ticket_metadata_change(%Webhook{payload: payload} = webhook) do
+    github_ticket = get_github_ticket(webhook)
+
+    case Workspace.get_ticket(
+           payload["repository"]["owner"]["login"],
+           payload["repository"]["name"],
+           github_ticket["number"]
+         ) do
+      nil ->
+        :ok
+
+      ticket ->
+        case ticket
+             |> change(%{
+               title: github_ticket["title"],
+               description: github_ticket["body"]
+             })
+             |> Repo.update() do
+          {:ok, _} -> :ok
+          {:error, reason} -> {:error, reason}
+        end
+    end
+  end
+
   defp alert(%Webhook{event_action: event_action} = webhook, {:error, error}) do
     message =
       case get_github_ticket(webhook) do

test/algora_web/controllers/webhooks/github_controller_test.exs

@@ -256,6 +256,53 @@ defmodule AlgoraWeb.Webhooks.GithubControllerTest do
       # This test should now PASS with the fix - visibility is preserved
       assert final_bounty.visibility == original_visibility
     end
+
+    test "updates ticket title and description on issues.edited", ctx do
+      issue_number = :rand.uniform(1000)
+      updated_title = "Updated Issue Title"
+      updated_body = "Updated issue description"
+
+      # First create the issue (title will be set by GitHub mock to "title #{number}")
+      process_scenario!(ctx, [
+        %{
+          event_action: "issues.opened",
+          user_type: :repo_admin,
+          body: "/bounty $100",
+          params: %{
+            "issue" => %{
+              "number" => issue_number,
+              "state" => "open"
+            }
+          }
+        }
+      ])
+
+      ticket = Repo.get_by!(Ticket, number: issue_number)
+      # This comes from GitHub mock
+      assert ticket.title == "title #{issue_number}"
+      # This comes from GitHub mock
+      assert ticket.description == "body #{issue_number}"
+
+      # Now edit the issue title and description
+      process_scenario!(ctx, [
+        %{
+          event_action: "issues.edited",
+          user_type: :repo_admin,
+          params: %{
+            "issue" => %{
+              "number" => issue_number,
+              "state" => "open",
+              "title" => updated_title,
+              "body" => updated_body
+            }
+          }
+        }
+      ])
+
+      ticket = Repo.get_by!(Ticket, number: issue_number)
+      assert ticket.title == updated_title
+      assert ticket.description == updated_body
+    end
   end
 
   describe "create tips" do
@@ -1578,6 +1625,7 @@ defmodule AlgoraWeb.Webhooks.GithubControllerTest do
             "id" => 123,
             "number" => 123,
             "state" => "open",
+            "title" => "Default Issue Title",
             "body" => mock_body(ctx[:body]),
             "user" => mock_user(ctx[:author])
           }
@@ -1597,6 +1645,7 @@ defmodule AlgoraWeb.Webhooks.GithubControllerTest do
             "id" => 123,
             "number" => 123,
             "state" => "open",
+            "title" => "Default PR Title",
             "body" => mock_body(ctx[:body]),
             "user" => mock_user(ctx[:author]),
             "merged_at" => nil