Add support for Asana V1 URLs (#36)

Task/Issue URL: https://app.asana.com/1/137249556945/project/1201899738287924/task/1209664522647706

Description:
This change adds support for Asana V1 URLs.

Author: ayoy

lib/fastlane/plugin/ddg_apple_automation/helper/asana_helper.rb

@@ -15,7 +15,12 @@ class AsanaHelper # rubocop:disable Metrics/ClassLength
       ASANA_API_URL = "https://app.asana.com/api/1.0"
       ASANA_TASK_URL_TEMPLATE = "https://app.asana.com/0/0/%s/f"
       ASANA_TAG_URL_TEMPLATE = "https://app.asana.com/0/%s/list"
-      ASANA_TASK_URL_REGEX = %r{https://app.asana.com/[0-9]/[0-9]+/([0-9]+)(:/f)?}
+
+      # https://app.asana.com/0/<project_id>/<task_id>
+      ASANA_V0_TASK_URL_REGEX = %r{https://app.asana.com/0/[0-9]+/([0-9]+)(?:/f)?}
+      # https://app.asana.com/<url_format_version>/<workspace_id>/<object_name>/<object_id>/<subobject_name>/<subobject_id>
+      ASANA_V1_TASK_URL_REGEX = %r{https://app.asana.com/1/[0-9]+(?:/[0-9a-z/]*)?/task/([0-9]+)(:?/[0-9a-z/]*)?(?:\?focus=true)?}
+
       ASANA_WORKSPACE_ID = "137249556945"
 
       IOS_HOTFIX_TASK_TEMPLATE_ID = "1209242676101485"
@@ -53,14 +58,14 @@ def self.asana_tag_url(tag_id)
       end
 
       def self.extract_asana_task_id(task_url, set_gha_output: true)
-        if (match = task_url.match(ASANA_TASK_URL_REGEX))
+        if (match = task_url.match(ASANA_V0_TASK_URL_REGEX)) || (match = task_url.match(ASANA_V1_TASK_URL_REGEX))
           task_id = match[1]
           if set_gha_output
             Helper::GitHubActionsHelper.set_output("asana_task_id", task_id)
           end
           task_id
         else
-          UI.user_error!("URL has incorrect format (attempted to match #{ASANA_TASK_URL_REGEX})")
+          UI.user_error!("URL has incorrect format (attempted to match #{ASANA_V0_TASK_URL_REGEX} or #{ASANA_V1_TASK_URL_REGEX})")
         end
       end
 
@@ -416,7 +421,7 @@ def self.get_task_ids_from_git_log(from_ref, to_ref = "HEAD")
 
         git_log
           .gsub("\n", " ")
-          .scan(%r{\bTask/Issue URL:\s*https://app\.asana\.com[/0-9f]+\b})
+          .scan(%r{\bTask/Issue URL:\s*https://app\.asana\.com[/0-9a-z]+\b})
           .map { |task_line| task_line.gsub(/.*(https.*)/, '\1') }
           .map { |task_url| extract_asana_task_id(task_url, set_gha_output: false) }
           .uniq

lib/fastlane/plugin/ddg_apple_automation/version.rb

@@ -1,5 +1,5 @@
 module Fastlane
   module DdgAppleAutomation
-    VERSION = "2.0.2"
+    VERSION = "2.1.0"
   end
 end

spec/asana_helper_spec.rb

@@ -17,24 +17,44 @@ def asana_task_url(task_id)
   end
 
   describe "#extract_asana_task_id" do
-    it "extracts task ID" do
-      expect(extract_asana_task_id("https://app.asana.com/0/0/0")).to eq("0")
-    end
+    context "v0 task URL" do
+      it "extracts task ID" do
+        expect(extract_asana_task_id("https://app.asana.com/0/0/0")).to eq("0")
+      end
 
-    it "extracts task ID when project ID is non-zero" do
-      expect(extract_asana_task_id("https://app.asana.com/0/753241/9999")).to eq("9999")
-    end
+      it "extracts task ID when project ID is non-zero" do
+        expect(extract_asana_task_id("https://app.asana.com/0/753241/9999")).to eq("9999")
+      end
 
-    it "extracts task ID when first digit is non-zero" do
-      expect(extract_asana_task_id("https://app.asana.com/4/753241/9999")).to eq("9999")
-    end
+      it "extracts long task ID" do
+        expect(extract_asana_task_id("https://app.asana.com/0/0/12837864576817392")).to eq("12837864576817392")
+      end
 
-    it "extracts long task ID" do
-      expect(extract_asana_task_id("https://app.asana.com/0/0/12837864576817392")).to eq("12837864576817392")
+      it "extracts task ID from a URL in focused mode" do
+        expect(extract_asana_task_id("https://app.asana.com/0/0/1234/f")).to eq("1234")
+      end
     end
 
-    it "extracts task ID from a URL with a trailing /f" do
-      expect(extract_asana_task_id("https://app.asana.com/0/0/1234/f")).to eq("1234")
+    context "v1 task URL" do
+      it "extracts task ID" do
+        expect(extract_asana_task_id("https://app.asana.com/1/1234/project/5678/task/9999")).to eq("9999")
+      end
+
+      it "extracts task ID when subtask is present" do
+        expect(extract_asana_task_id("https://app.asana.com/1/1234/project/5678/task/9999/subtask/0001")).to eq("9999")
+      end
+
+      it "extracts task ID when project is missing in the URL" do
+        expect(extract_asana_task_id("https://app.asana.com/1/1234/task/9999")).to eq("9999")
+      end
+
+      it "extracts long task ID" do
+        expect(extract_asana_task_id("https://app.asana.com/1/1234/task/12837864576817392")).to eq("12837864576817392")
+      end
+
+      it "extracts task ID from a URL in focused mode" do
+        expect(extract_asana_task_id("https://app.asana.com/1/1234/project/5678/task/9999?focused=true")).to eq("9999")
+      end
     end
 
     it "sets GHA output" do
@@ -46,7 +66,7 @@ def asana_task_url(task_id)
 
     it "fails when garbage is passed" do
       expect(Fastlane::UI).to receive(:user_error!)
-        .with("URL has incorrect format (attempted to match #{Fastlane::Helper::AsanaHelper::ASANA_TASK_URL_REGEX})")
+        .with("URL has incorrect format (attempted to match #{Fastlane::Helper::AsanaHelper::ASANA_V0_TASK_URL_REGEX} or #{Fastlane::Helper::AsanaHelper::ASANA_V1_TASK_URL_REGEX})")
 
       extract_asana_task_id("not a URL")
     end
@@ -556,6 +576,30 @@ def sanitize_asana_html_notes(content)
 #{'    '}
     Task/Issue URL: https://app.asana.com/0/1202406491309510/1208589738926535/f
 
+commit 7202ff2597d21db57fd6dc9a295e11991c81b3e7
+
+    Hide continue setup cards after 1 week (#3471)
+#{'    '}
+    Task/Issue URL: https://app.asana.com/1/1552213/task/1208589738999999
+
+commit 7202ff2597d21db57fd6dc9a295e11991c81b3e7
+
+    Hide continue setup cards after 1 week (#3471)
+#{'    '}
+    Task/Issue URL: https://app.asana.com/1/1552213/task/1208589738888888?focus=true
+
+commit 7202ff2597d21db57fd6dc9a295e11991c81b3e7
+
+    Hide continue setup cards after 1 week (#3471)
+#{'    '}
+    Task/Issue URL: https://app.asana.com/1/1552213/project/123/task/1208589738777777?focus=true
+
+commit 7202ff2597d21db57fd6dc9a295e11991c81b3e7
+
+    Hide continue setup cards after 1 week (#3471)
+#{'    '}
+    Task/Issue URL: https://app.asana.com/1/1552213/project/123/task/1208589738666666/subtask/527?focus=true
+
 commit e83fd007c0bdf054658068a79f5b7ea45d846468
 
     Receive privacy config updates in AddressBarModel on main thread (#3574)
@@ -593,7 +637,15 @@ def sanitize_asana_html_notes(content)
       allow(Fastlane::Helper::AsanaHelper).to receive(:`).with("git log v1.0.0..HEAD -- ./ ../BrowserServicesKit/").and_return(git_log)
 
       task_ids = Fastlane::Helper::AsanaHelper.get_task_ids_from_git_log("v1.0.0")
-      expect(task_ids).to eq(["1208700893044577", "1208589738926535", "1208804405760977"])
+      expect(task_ids).to eq([
+                               "1208700893044577",
+                               "1208589738926535",
+                               "1208589738999999",
+                               "1208589738888888",
+                               "1208589738777777",
+                               "1208589738666666",
+                               "1208804405760977"
+                             ])
     end
 
     it "returns an empty array if no task IDs are found" do