fix(service-settings): judge the declared format on the trimmed value

Both live readers (the mail transport's address check and the branding
read) trim before use, so surrounding whitespace is not a format miss.

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

Author: claude

.changeset/24882-settings-format-warning.md

@@ -21,6 +21,6 @@ The formats are the platform's existing grammars for the same words:
 - `phone`: digits, spaces and `+ ( ) - .`, at least five characters.
 - `color`: `#rgb`, `#rgba`, `#rrggbb` or `#rrggbbaa`.
 
-Empty values and non-string values are not judged.
+Empty values and non-string values are not judged. Surrounding whitespace is ignored, because the readers trim it before use.
 
 **Unchanged.** Status codes, response bodies, stored values, the `FieldError`s a refused save carries, and which overrides are in force. Under the standing rule 「甲」, refusing these values waits for the next major version.

packages/services/service-settings/src/settings-format-warning.test.ts

@@ -193,6 +193,9 @@ describe('save door — a format miss is accepted and stored, and warned about o
     for (const colour of ['#abc', '#abcd', '#AABBCC', '#aabbccdd']) {
       expect((await put(http, 'branding', { accent_color: colour })).status).toBe(200);
     }
+    // Surrounding whitespace is trimmed by both live readers before use, so it
+    // is not a format miss; inner whitespace still is (the per-type cases above).
+    expect((await put(http, 'company', { primary_contact_email: ' ops@example.com ' })).status).toBe(200);
     expect(warns).toEqual([]);
   });
 

packages/services/service-settings/src/settings-service.ts

@@ -706,7 +706,10 @@ function firstFormatMiss(type: string, value: unknown): FormatMiss | null {
   const declared = DECLARED_VALUE_FORMATS.get(type);
   if (!declared) return null;
   if (typeof value !== 'string' || value.trim() === '') return null;
-  if (declared.re.test(value)) return null;
+  // Judged trimmed: both live readers trim before use (`plugin-email`'s
+  // address check, `RuntimeConfigPlugin`'s branding read), so surrounding
+  // whitespace is not a format any reader trips over. Inner whitespace is.
+  if (declared.re.test(value.trim())) return null;
   return {
     type,
     code: declared.code,