feat(upload): Enable file uploads inside threads (#188)

* Adding threadts to allow posting to a thread

* CS fix

* tested uploading in thread

* Update phpunit.xml.dist

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update tests/WritingTest.php

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update src/Client.php

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Michael Tiel <michael.tiel@onthatass.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: 3 people

phpunit.xml.dist

@@ -10,7 +10,10 @@
     <php>
         <ini name="error_reporting" value="-1" />
         <!--<server name="SLACK_TOKEN" value="edit_me" />-->
+        <!-- should be the channel ID, not name -->
         <!--<server name="SLACK_TEST_CHANNEL" value="edit_me" />-->
+        <!-- set this to test uploads in a thread -->
+        <!--<server name="SLACK_TEST_THREAD_TS" value="edit_me" />-->
     </php>
 
     <testsuites>

src/Client.php

@@ -99,6 +99,7 @@ public function iterate(string $method, array $arguments): iterable
      *                                    ]
      * @param string      $channelId      The Slack channel ID to upload files to (e.g., 'C12345678').
      * @param string|null $initialComment optional comment to add with the upload
+     * @param string|null $threadTs       optional thread_ts (Slack thread timestamp) to attach the upload to a thread
      *
      * @throws \RuntimeException        if upload or Slack API interaction fails
      * @throws \JsonException
@@ -114,6 +115,7 @@ public function filesUploadV2(
         array $files,
         string $channelId,
         ?string $initialComment = null,
+        ?string $threadTs = null,
     ): FilesCompleteUploadExternalPostResponsedefault|FilesCompleteUploadExternalPostResponse200|null {
         $filesPayload = [];
 
@@ -161,6 +163,7 @@ public function filesUploadV2(
                 [
                     'files' => json_encode($filesPayload, \JSON_THROW_ON_ERROR),
                     'channel_id' => $channelId,
+                    'thread_ts' => $threadTs,
                     'initial_comment' => $initialComment,
                 ]
             )

tests/WritingTest.php

@@ -174,6 +174,7 @@ public function testItCanFileGetUploadUrlExternal(): void
         $filePath = __DIR__ . '/resources/' . $fileName;
         $fileStream = Stream::create(fopen($filePath, 'r'));
         $fileSize = $fileStream->getSize();
+        $threadTs = $_SERVER['SLACK_TEST_THREAD_TS'] ?? null;
 
         $response = $client->filesGetUploadUrlExternal(
             [
@@ -191,7 +192,7 @@ public function testItCanFileGetUploadUrlExternal(): void
 
         $uploadResponse = $client->filesUploadV2([
             ['path' => $filePath, 'title' => 'Test image', 'alt_text' => 'Test image'],
-        ], $_SERVER['SLACK_TEST_CHANNEL'], 'Uploaded with the 2025 API.');
+        ], $_SERVER['SLACK_TEST_CHANNEL'], 'Uploaded with the 2025 API.', $threadTs);
 
         self::assertTrue($uploadResponse->getOk());
     }