feature #59789 [Notifier] [Bluesky] Return the record CID as additional info (javiereguiluz)

nicolas-grekas · nicolas-grekas · commit 3ef4f4e1821d · 2025-02-18T09:13:30.000+01:00
This PR was squashed before being merged into the 7.3 branch. Discussion ---------- [Notifier] [Bluesky] Return the record CID as additional info | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Issues | - | License | MIT Some Bluesky API operations (like replies and quotes) require "strong references", which is the combination of the `uri` and the `cid` (See https://docs.bsky.app/docs/advanced-guides/posts#replies-quote-posts-and-embeds). After the changes made in #59742, we should also return the `cid` as additional info. Commits ------- 85a073e [Notifier] [Bluesky] Return the record CID as additional info
diff --git a/src/Symfony/Component/Notifier/Bridge/Bluesky/BlueskyTransport.php b/src/Symfony/Component/Notifier/Bridge/Bluesky/BlueskyTransport.php
@@ -124,7 +124,8 @@ protected function doSend(MessageInterface $message): SentMessage
 
         if (200 === $statusCode) {
             $content = $response->toArray();
-            $sentMessage = new SentMessage($message, (string) $this);
+
+            $sentMessage = new SentMessage($message, (string) $this, ['cid' => $content['cid']]);
             $sentMessage->setMessageId($content['uri']);
 
             return $sentMessage;
diff --git a/src/Symfony/Component/Notifier/Bridge/Bluesky/CHANGELOG.md b/src/Symfony/Component/Notifier/Bridge/Bluesky/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGELOG
 ---
 
  * Add option to attach a website preview card
+ * Add `cid` info into returned `SentMessage`
 
 7.2
 ---
diff --git a/src/Symfony/Component/Notifier/Bridge/Bluesky/Tests/BlueskyTransportTest.php b/src/Symfony/Component/Notifier/Bridge/Bluesky/Tests/BlueskyTransportTest.php
@@ -338,6 +338,7 @@ public function testReturnedMessageId()
         $message = $transport->send(new ChatMessage('Hello!'));
 
         $this->assertSame($recordUri, $message->getMessageId());
+        $this->assertSame($recordCid, $message->getInfo('cid'));
     }
 
     public static function sendMessageWithEmbedDataProvider(): iterable
diff --git a/src/Symfony/Component/Notifier/Bridge/Bluesky/composer.json b/src/Symfony/Component/Notifier/Bridge/Bluesky/composer.json
@@ -24,7 +24,7 @@
         "psr/log": "^1|^2|^3",
         "symfony/clock": "^6.4|^7.0",
         "symfony/http-client": "^6.4|^7.0",
-        "symfony/notifier": "^7.2",
+        "symfony/notifier": "^7.3",
         "symfony/string": "^6.4|^7.0"
     },
     "require-dev": {

Original file line number	Diff line number	Diff line change
`@@ -338,6 +338,7 @@ public function testReturnedMessageId()`
`338`	`338`	`$message = $transport->send(new ChatMessage('Hello!'));`
`339`	`339`
`340`	`340`	`$this->assertSame($recordUri, $message->getMessageId());`
	`341`	`+ $this->assertSame($recordCid, $message->getInfo('cid'));`
`341`	`342`	`}`
`342`	`343`
`343`	`344`	`public static function sendMessageWithEmbedDataProvider(): iterable`