fix(service-sms): sms/test names the transport that carried it, not "via log"

With the recipient defaulting to the admin's own number, a deployment that
configured its SMS transport at boot while the namespace's provider sits at
its `log` default now reaches this path from the served form: the live
service delivers, and the answer said "via log".

Claude-Session: https://claude.ai/code/session_011TdZ8GjixNZFLz1xJD9zzK
Co-authored-by: Claude <noreply@anthropic.com>

Author: claude

.changeset/24767-settings-test-send-recipient.md

@@ -11,7 +11,7 @@ The Mail and SMS settings pages offer a Send test button and no recipient input,
 
 With no `to` in the request, the test now goes to the person who pressed the button: `mail/test` sends to the caller's own account email and `sms/test` to the caller's own phone number, each read from the caller's own `sys_user` row. A `to` in the request body still wins, as before. When the caller's account has no address or number, or the account cannot be read, the action refuses and says that nothing was sent. A phone number is on an account only where phone sign-in is enabled, so on other deployments "Send test SMS" still needs a `to` in the request body.
 
-`sms/test` no longer reports a send through the log-only transport (the namespace's default `None (log only)` provider) as a success. It answers `ok: false` with severity `warning`, as `mail/test` already does for its log transport.
+`sms/test` no longer reports a send through the log-only transport (the namespace's default `None (log only)` provider) as a success. It answers `ok: false` with severity `warning`, as `mail/test` already does for its log transport. When the form names no delivering provider and the transport the deployment configured at boot carries the test, the success message names that transport instead of saying the message went "via log".
 
 **Unchanged.**
 

packages/services/service-sms/src/sms-plugin.test.ts

@@ -186,6 +186,34 @@ describe('SmsServicePlugin', () => {
     expect(send).not.toHaveBeenCalled();
     expect(result.message).not.toContain('5550006');
   });
+
+  it('names the deployment\'s transport, not `log`, when the boot-configured one carried the test', async () => {
+    // A deployment that configured Twilio at boot, with the namespace's
+    // `provider` still at its `log` default: the live service delivers, so the
+    // answer must not say the message went "via log".
+    const fetchImpl = vi.fn(async () => new Response(
+      JSON.stringify({ sid: 'SM1', status: 'queued' }),
+      { status: 201, headers: { 'Content-Type': 'application/json' } },
+    ));
+    const harness = fakeCtx({
+      settingsValues: { provider: 'log' },
+      settingsSources: { provider: 'default' },
+    });
+    const plugin = new SmsServicePlugin({
+      provider: 'twilio',
+      providerOptions: { accountSid: 'AC1', authToken: 't', from: '+15005550006', fetchImpl },
+    });
+    await plugin.init(harness.ctx);
+    await plugin.start(harness.ctx);
+    await harness.fireReady();
+
+    const result = await harness.actions.get('sms/test')!({
+      values: { provider: 'log' }, payload: { to: '+15005550009' }, ctx: {},
+    });
+    expect(result).toMatchObject({ ok: true, severity: 'info' });
+    expect(fetchImpl).toHaveBeenCalledTimes(1);
+    expect(result.message).not.toMatch(/via log\b/);
+  });
 });
 
 describe('SmsServicePlugin — authored-value criterion (#5536)', () => {

packages/services/service-sms/src/sms-plugin.ts

@@ -370,10 +370,14 @@ export class SmsServicePlugin implements Plugin {
               if (result.status === 'failed') {
                 return { ok: false, severity: 'error', message: result.error ?? 'Send failed.' };
               }
+              // Name the transport that actually carried it. When the form
+              // names no delivering provider, the live service did — the one
+              // the deployment configured at boot, never `log` (refused above).
+              const via = target === svc ? 'the deployment\'s configured transport' : resolved.provider;
               return {
                 ok: true,
                 severity: 'info',
-                message: `Sent test SMS to ${maskPhoneNumber(to)} via ${resolved.provider} (id=${result.messageId ?? result.id}).`,
+                message: `Sent test SMS to ${maskPhoneNumber(to)} via ${via} (id=${result.messageId ?? result.id}).`,
               };
             } catch (err: any) {
               return { ok: false, severity: 'error', message: err?.message ?? String(err) };