From 0700ff676f02b5fc50b3f7aace7a855a9bf54d32 Mon Sep 17 00:00:00 2001
From: automated-signal <37887102+automated-signal@users.noreply.github.com>
Date: Mon, 20 Jul 2026 12:32:43 -0500
Subject: [PATCH] Always show contact name in Safety Number warning

Co-authored-by: Scott Nonnenberg <scott@signal.org>
---
 _locales/en/messages.json                     |  2 +-
 .../SafetyNumberNotification.dom.stories.tsx  | 22 ++-----------------
 .../SafetyNumberNotification.dom.tsx          | 16 +++++---------
 .../conversation/Timeline.dom.stories.tsx     |  1 -
 ts/state/selectors/message.preload.ts         |  3 +--
 ts/test-mock/messaging/relink_test.node.ts    |  2 +-
 .../messaging/safety_number_test.node.ts      |  6 ++---
 ts/test-mock/pnp/pni_change_test.node.ts      |  4 ++--
 8 files changed, 15 insertions(+), 41 deletions(-)

diff --git a/_locales/en/messages.json b/_locales/en/messages.json
index 226d034124..566152b2dc 100644
--- a/_locales/en/messages.json
+++ b/_locales/en/messages.json
@@ -3178,7 +3178,7 @@
   },
   "icu:safetyNumberChangedGroup": {
     "messageformat": "Safety Number with {name} has changed",
-    "description": "A notification shown in a group conversation when a contact reinstalls, showing the contact name"
+    "description": "A notification shown in a group and 1:1 conversations when a contact reinstalls, showing the contact name"
   },
   "icu:ConversationDetails__viewSafetyNumber": {
     "messageformat": "View Safety Number",
diff --git a/ts/components/conversation/SafetyNumberNotification.dom.stories.tsx b/ts/components/conversation/SafetyNumberNotification.dom.stories.tsx
index 1d4b04f664..d37c6a9c62 100644
--- a/ts/components/conversation/SafetyNumberNotification.dom.stories.tsx
+++ b/ts/components/conversation/SafetyNumberNotification.dom.stories.tsx
@@ -17,22 +17,17 @@ const createContact = (props: Partial<ContactType>): ContactType => ({
 
 export default {
   title: 'Components/Conversation/SafetyNumberNotification',
-  argTypes: {
-    isGroup: { control: { type: 'boolean' } },
-  },
   args: {
     i18n,
     contact: {} as ContactType,
-    isGroup: false,
     toggleSafetyNumberModal: action('toggleSafetyNumberModal'),
   },
 } satisfies Meta<Props>;
 
-export function GroupConversation(args: Props): JSX.Element {
+export function Default(args: Props): JSX.Element {
   return (
     <SafetyNumberNotification
       {...args}
-      isGroup
       contact={createContact({
         title: 'Mr. Fire',
       })}
@@ -40,23 +35,10 @@ export function GroupConversation(args: Props): JSX.Element {
   );
 }
 
-export function DirectConversation(args: Props): JSX.Element {
+export function LongName(args: Props): JSX.Element {
   return (
     <SafetyNumberNotification
       {...args}
-      isGroup
-      contact={createContact({
-        title: 'Mr. Fire',
-      })}
-    />
-  );
-}
-
-export function LongNameInGroup(args: Props): JSX.Element {
-  return (
-    <SafetyNumberNotification
-      {...args}
-      isGroup
       contact={createContact({
         title: '🐈‍⬛🍕🎂'.repeat(50),
       })}
diff --git a/ts/components/conversation/SafetyNumberNotification.dom.tsx b/ts/components/conversation/SafetyNumberNotification.dom.tsx
index 2e0fac1558..7587868f02 100644
--- a/ts/components/conversation/SafetyNumberNotification.dom.tsx
+++ b/ts/components/conversation/SafetyNumberNotification.dom.tsx
@@ -15,7 +15,6 @@ export type ContactType = {
 };
 
 export type PropsData = {
-  isGroup: boolean;
   contact: ContactType;
 };
 
@@ -31,7 +30,6 @@ export type Props = PropsData & PropsHousekeeping & PropsActions;
 
 export function SafetyNumberNotification({
   contact,
-  isGroup,
   i18n,
   toggleSafetyNumberModal,
 }: Props): JSX.Element {
@@ -50,15 +48,11 @@ export function SafetyNumberNotification({
     <SystemMessage
       icon="safety-number"
       contents={
-        isGroup ? (
-          <I18n
-            id="icu:safetyNumberChangedGroup"
-            components={{ name }}
-            i18n={i18n}
-          />
-        ) : (
-          <I18n id="icu:safetyNumberChanged" i18n={i18n} />
-        )
+        <I18n
+          id="icu:safetyNumberChangedGroup"
+          components={{ name }}
+          i18n={i18n}
+        />
       }
       button={
         <Button
diff --git a/ts/components/conversation/Timeline.dom.stories.tsx b/ts/components/conversation/Timeline.dom.stories.tsx
index 95dbd6ec5b..dc22e6c883 100644
--- a/ts/components/conversation/Timeline.dom.stories.tsx
+++ b/ts/components/conversation/Timeline.dom.stories.tsx
@@ -131,7 +131,6 @@ const items: Record<string, TimelineItemType> = {
         id: '+1202555000',
         title: 'Mr. Fire',
       },
-      isGroup: true,
     },
     timestamp: Date.now(),
   },
diff --git a/ts/state/selectors/message.preload.ts b/ts/state/selectors/message.preload.ts
index f51e139afb..3043d9bc9d 100644
--- a/ts/state/selectors/message.preload.ts
+++ b/ts/state/selectors/message.preload.ts
@@ -1546,10 +1546,9 @@ function getPropsForSafetyNumberNotification(
     );
   }
 
-  const contact = identifier ? conversationSelector(identifier) : conversation;
+  const contact = isGroup ? conversationSelector(identifier) : conversation;
 
   return {
-    isGroup,
     contact,
   };
 }
diff --git a/ts/test-mock/messaging/relink_test.node.ts b/ts/test-mock/messaging/relink_test.node.ts
index 9195a7f9b0..32cff97231 100644
--- a/ts/test-mock/messaging/relink_test.node.ts
+++ b/ts/test-mock/messaging/relink_test.node.ts
@@ -147,7 +147,7 @@ describe('messaging/relink', function (this: Mocha.Suite) {
         )
         .click();
 
-      await expectSystemMessages(window, [/Safety Number has changed/]);
+      await expectSystemMessages(window, [/Safety Number with Alice/]);
     }
   });
 });
diff --git a/ts/test-mock/messaging/safety_number_test.node.ts b/ts/test-mock/messaging/safety_number_test.node.ts
index 73e12556fb..3938019afc 100644
--- a/ts/test-mock/messaging/safety_number_test.node.ts
+++ b/ts/test-mock/messaging/safety_number_test.node.ts
@@ -113,7 +113,7 @@ describe('safety number', function (this: Mocha.Suite) {
     await changeIdentityKey();
 
     await expectSystemMessages(window, [
-      /Safety Number has changed/, // Bob's key from storage service
+      /Safety Number with Alice/, // Alice's key from storage service
     ]);
 
     debug('Sending message');
@@ -126,8 +126,8 @@ describe('safety number', function (this: Mocha.Suite) {
     await dialog.locator(`"${alice.profileName}"`).waitFor();
 
     await expectSystemMessages(window, [
-      /Safety Number has changed/, // Bob's key from storage service
-      /Safety Number has changed/, // Fixed Alice's key from backend
+      /Safety Number with/, // One is a fixed Alice's key from backend
+      /Safety Number with/, // Other is Bob's key from storage service
     ]);
 
     debug('Confirming send');
diff --git a/ts/test-mock/pnp/pni_change_test.node.ts b/ts/test-mock/pnp/pni_change_test.node.ts
index 5d0fc161dc..c10e345a83 100644
--- a/ts/test-mock/pnp/pni_change_test.node.ts
+++ b/ts/test-mock/pnp/pni_change_test.node.ts
@@ -285,7 +285,7 @@ describe('pnp/PNI Change', function (this: Mocha.Suite) {
       // Two notifications - the safety number change and PhoneNumberDiscovery
       await expectSystemMessages(window, [
         /.* belongs to ContactA/,
-        /Safety Number has changed/,
+        /Safety Number with ContactA/,
       ]);
     }
   });
@@ -423,7 +423,7 @@ describe('pnp/PNI Change', function (this: Mocha.Suite) {
       // Three notifications - accepted, the safety number change and PhoneNumberDiscovery
       await expectSystemMessages(window, [
         /.* belongs to ContactA/,
-        /Safety Number has changed/,
+        /Safety Number with ContactA/,
       ]);
     }
   });