diff --git a/‎.changeset/20751-services-strings-stage3-state-the-decision.md‎
Lines changed: 15 additions & 0 deletions b/‎.changeset/20751-services-strings-stage3-state-the-decision.md‎
Lines changed: 15 additions & 0 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/approval-service.test.ts‎
Lines changed: 2 additions & 2 deletions b/‎packages/plugins/plugin-approvals/src/approval-service.test.ts‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/approval-service.ts‎
Lines changed: 14 additions & 10 deletions b/‎packages/plugins/plugin-approvals/src/approval-service.ts‎
Lines changed: 14 additions & 10 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/sys-approval-action.object.ts‎
Lines changed: 4 additions & 3 deletions b/‎packages/plugins/plugin-approvals/src/sys-approval-action.object.ts‎
Lines changed: 4 additions & 3 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/sys-approval-delegation.object.ts‎
Lines changed: 1 addition & 1 deletion b/‎packages/plugins/plugin-approvals/src/sys-approval-delegation.object.ts‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎packages/plugins/plugin-approvals/src/team-approver-org-screen.test.ts‎
Lines changed: 2 additions & 2 deletions b/‎packages/plugins/plugin-approvals/src/team-approver-org-screen.test.ts‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/team-member-org-screen.test.ts‎
Lines changed: 2 additions & 2 deletions b/‎packages/plugins/plugin-approvals/src/team-member-org-screen.test.ts‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/translations/en.objects.generated.ts‎
Lines changed: 3 additions & 3 deletions b/‎packages/plugins/plugin-approvals/src/translations/en.objects.generated.ts‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/translations/es-ES.objects.generated.ts‎
Lines changed: 3 additions & 3 deletions b/‎packages/plugins/plugin-approvals/src/translations/es-ES.objects.generated.ts‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎packages/plugins/plugin-approvals/src/translations/es-ES.source-hashes.generated.ts‎
Lines changed: 1 addition & 1 deletion b/‎packages/plugins/plugin-approvals/src/translations/es-ES.source-hashes.generated.ts‎
Lines changed: 1 addition & 1 deletion
@@ -0,0 +1,15 @@
+---
+'@objectstack/service-datasource': patch
+'@objectstack/plugin-approvals': patch
+---
+
+Datasource and approval refusals, warnings, field help and generated-draft comments no longer cite tracker numbers; each one states the decision behind it in words
+
+Clause-②: no
+
+Some strings these two packages show to operators, administrators and flow authors pointed at an issue-tracker number for the reason behind them. The number goes; where the sentence did not already say what was decided, it now does.
+
+- `@objectstack/service-datasource`: the credential-migration refusal says an unbindable key is either an alias spelling from before inline credentials were refused at publish, which no connection builder reads, or turso's `encryptionKey`, which has no secret slot of its own because the one slot carries the `authToken`; the remote-primary-key comment in a generated object draft says a driver's introspection can report only the first column of a composite key, so the list is a lower bound.
+- `@objectstack/plugin-approvals`: the `queue` approver warning says the platform has no ownership queue to expand the type from, that the type is no longer offered for authoring, and to route the step to a team, department or position instead; the live-record warnings say approvers are being resolved against the trigger snapshot instead of the live record they are normally resolved from; the recall refusal's log line names the admin override; the `sys_approval_action` `via_override` help (in every shipped locale) says a platform or organization admin may act on any pending request, so that one nobody in its slate can decide never stays stuck; the cross-organization team, team-member and manager warnings, the expanded-to-nobody warning, the revise-window refusal, the `attachments` help and the `sys_approval_delegation` description drop their citations.
+
+Text only: no status, error code, field, route or control flow moves. A client or log filter that matches the old text (for example a tracker-number suffix) needs the new spelling.
@@ -3162,7 +3162,7 @@ describe('ApprovalService — queue approver is unresolved (#3508)', () => {
     // No queue expansion exists: the slot is the raw `type:value` literal,
     // which matches no real user id — the request routes to nobody.
     expect(req.pending_approvers).toEqual(['queue:q_west']);
-    expect(warnings.some(([msg]) => String(msg).includes("'queue'") && String(msg).includes('#3508'))).toBe(true);
+    expect(warnings.some(([msg]) => String(msg).includes("'queue'") && String(msg).includes('no ownership queue to expand it from'))).toBe(true);
   });
 });
 
@@ -3215,7 +3215,7 @@ describe('ApprovalService — a graph approver that expands to nobody warns (#38
     expect(req.pending_approvers).toEqual([`${type}:${value}`]);
     const hit = warnings.find(([msg]) => String(msg).includes('expanded to nobody'));
     expect(hit, `no warning for ${type}`).toBeTruthy();
-    expect(String(hit[0])).toContain('#3807');
+    expect(String(hit[0])).toContain('cannot advance until someone is added or the approver is re-pointed');
     expect(hit[1]).toMatchObject({ type, value, organizationId: 't1' });
   });
 
 
@@ -1762,7 +1762,9 @@ export class ApprovalService implements IApprovalService {
     // least visible to operators.
     if (type === 'queue') {
       this.logger?.warn?.(
-        `[approvals] approver type 'queue' is not implemented — the slot resolves to nobody (#3508)`,
+        `[approvals] approver type 'queue' is not implemented — the slot resolves to nobody, because the `
+        + `platform has no ownership queue to expand it from (the type is deprecated and no longer offered `
+        + `for authoring). Route this step to a team, department or position instead.`,
         { value: a.value },
       );
     } else if (GRAPH_APPROVER_TYPES.has(type)) {
@@ -1777,7 +1779,7 @@ export class ApprovalService implements IApprovalService {
       // stops being invisible.
       this.logger?.warn?.(
         `[approvals] approver '${type}:${a.value}' expanded to nobody — the slot routes to no one `
-        + `and the request cannot advance until someone is added or the approver is re-pointed (#3807)`,
+        + `and the request cannot advance until someone is added or the approver is re-pointed`,
         { type, value: a.value, organizationId: organizationId ?? null },
       );
     }
@@ -2039,7 +2041,7 @@ export class ApprovalService implements IApprovalService {
     if (!teamOrg) return false;                  // no tenancy fact on this team
     if (teamOrg === requestOrg) return false;    // it is this org's team — route as before
     this.logger?.warn?.(
-      `[approvals] #10230: team '${teamId}' was dropped from the approver slate — `
+      `[approvals] team '${teamId}' was dropped from the approver slate — `
       + `'sys_team.organization_id' is '${teamOrg}', not the request's organization `
       + `'${requestOrg}', so routing this approval to its members would put approval `
       + `authority over the record outside its tenant. Point the approver at a team in `
@@ -2108,7 +2110,7 @@ export class ApprovalService implements IApprovalService {
     if ((rows?.length ?? 0) >= MEMBER_SCREEN_READ_LIMIT) {
       // Possibly truncated ⇒ the evidence is incomplete ⇒ no evidence.
       this.logger?.warn?.(
-        `[approvals] #10547: the membership screen for team '${teamId}' read `
+        `[approvals] the membership screen for team '${teamId}' read `
         + `${rows.length} 'sys_member' rows, at or above its ${MEMBER_SCREEN_READ_LIMIT}-row `
         + `cap, so the result may be truncated. Routing is left unchanged rather than risk `
         + `dropping a member whose proof of membership fell outside the read.`,
@@ -2134,7 +2136,7 @@ export class ApprovalService implements IApprovalService {
     }
     if (dropped.length) {
       this.logger?.warn?.(
-        `[approvals] #10547: ${dropped.length} member(s) of team '${teamId}' were dropped from `
+        `[approvals] ${dropped.length} member(s) of team '${teamId}' were dropped from `
         + `the approver slate — ${dropped.map(d => `'${d.userId}'`).join(', ')} hold membership `
         + `in other organization(s), none of them the request's organization '${requestOrg}', so `
         + `routing this approval to them would put approval authority over the record outside its `
@@ -2469,7 +2471,7 @@ export class ApprovalService implements IApprovalService {
     if (!orgs.length) return false;          // no tenancy fact recorded for this user
     if (orgs.includes(requestOrg)) return false; // he is a member here — route as before
     this.logger?.warn?.(
-      `[approvals] #10153: manager '${managerId}' was dropped from the approver slate — `
+      `[approvals] manager '${managerId}' was dropped from the approver slate — `
       + `'sys_user.manager_id' points across an organization boundary. He holds membership in `
       + `${orgs.length} organization(s), none of them the request's organization '${requestOrg}', `
       + `so routing this approval to him would put approval authority over the record outside its `
@@ -2624,13 +2626,15 @@ export class ApprovalService implements IApprovalService {
       if (live) return live;
       this.logger?.warn?.(
         `[approvals] live record ${object}/${recordId} not found at node entry — `
-        + 'resolving approvers against the trigger snapshot (#3447 fallback).',
+        + 'resolving approvers against the trigger snapshot instead of the live record they are '
+        + 'normally resolved from.',
         { object, recordId },
       );
     } catch (err: any) {
       this.logger?.warn?.(
         `[approvals] live record re-read failed for ${object}/${recordId}: ${err?.message ?? err} — `
-        + 'resolving approvers against the trigger snapshot (#3447 fallback).',
+        + 'resolving approvers against the trigger snapshot instead of the live record they are '
+        + 'normally resolved from.',
       );
     }
     return fallback ?? {};
@@ -3662,7 +3666,7 @@ export class ApprovalService implements IApprovalService {
       // id), kept where a developer reads them and a user never does.
       const developerMessage =
         `[approvals] recall refused: actor '${actorId}' is not the submitter of request `
-        + `'${requestId}' (submitter '${String(raw.submitter_id)}') and holds no #3424 override `
+        + `'${requestId}' (submitter '${String(raw.submitter_id)}') and holds no admin override `
         + `for a '${raw.status}' request (the override reaches pending requests only)`;
       this.logger?.warn?.(developerMessage, {
         request: requestId,
@@ -4238,7 +4242,7 @@ export class ApprovalService implements IApprovalService {
         (targetType === undefined ? ' which the flow does not declare' : ` of type '${targetType || '(untyped)'}'`) +
         `, but the revise window must be an '${APPROVAL_REVISE_NODE_TYPE}' node — that pause continues only ` +
         'through this service (submitter-only, audited, and refusing a colliding pending request), and any ' +
-        `other node type there is resumable by anyone with the run id (amended ADR-0044, #3823). Fix the flow: ` +
+        `other node type there is resumable by anyone with the run id (amended ADR-0044). Fix the flow: ` +
         `set node '${target || '<revise target>'}' to type '${APPROVAL_REVISE_NODE_TYPE}'.`,
       );
     }
 
@@ -162,8 +162,9 @@ export const SysApprovalAction = ObjectSchema.create({
       required: false,
       group: 'Action',
       description:
-        'True when the actor was admitted to this action only by the privileged-override path (#3424) — '
-        + 'they held no slot in the request’s pending-approver slate.',
+        'True when the actor held no slot in the request’s pending-approver slate and was admitted to '
+        + 'this action only by the privileged override, which lets a platform or organization admin act '
+        + 'on any pending request so that one nobody in its slate can decide never stays stuck.',
     }),
 
     // Structured hand-off parties for `action: 'reassign'` (#4365). Before
@@ -190,7 +191,7 @@ export const SysApprovalAction = ObjectSchema.create({
       required: false,
       multiple: true,
       group: 'Action',
-      description: 'Files supporting this action — e.g. a signed contract or evidence (#3266).',
+      description: 'Files supporting this action — e.g. a signed contract or evidence.',
     }),
 
     created_at: Field.datetime({
 
@@ -40,7 +40,7 @@ export const SysApprovalDelegation = ObjectSchema.create({
   // needed — RLS/permission sets are the authz.
   managedBy: 'system-data',
   description:
-    'Self-service out-of-office rule: route this user\'s approver slots to a delegate within a time window (#1322 M1).',
+    'Self-service out-of-office rule: route this user\'s approver slots to a delegate within a time window.',
   // [ADR-0079] The record title is `display_title`, a text formula over the
   // same two columns `titleFormat` names. With no pointer declared, the
   // registry's designate-only pass stamped `nameField: 'id'` (the first
 
@@ -218,9 +218,9 @@ describe('#10230 team approver org screen', () => {
     expect(engine._reads.filter(r => r === 'sys_team')).toEqual([]);
   });
 
-  it('T7 — the drop is loud: the warning names the team, both organizations and the card', async () => {
+  it('T7 — the drop is loud: the warning names the team and both organizations', async () => {
     await svc.openNodeRequest(input([TEAM_B]), CTX_A);
-    const hit = warnings.find(([msg]) => String(msg).includes('#10230'));
+    const hit = warnings.find(([msg]) => String(msg).includes("team 'team_b' was dropped from the approver slate"));
     console.log('[PROBE T7] warning =', hit ? String(hit[0]).slice(0, 90) : 'NONE');
     expect(hit).toBeTruthy();
     expect(hit![1]).toMatchObject({
 
@@ -245,9 +245,9 @@ describe('#10547 team MEMBER org screen', () => {
     expect(req.pending_approvers).toEqual(['u0', 'u1', 'u2', 'u3', 'u4']);
   });
 
-  it('M9 — the drop is loud: the warning names the users, both organizations and the card', async () => {
+  it('M9 — the drop is loud: the warning names the users and both organizations', async () => {
     await svc.openNodeRequest(input([TEAM_A]), CTX_A);
-    const hit = warnings.find(([msg]) => String(msg).includes('#10547'));
+    const hit = warnings.find(([msg]) => String(msg).includes("member(s) of team 'team_a' were dropped from the approver slate"));
     console.log('[PROBE M9] warning =', hit ? String(hit[0]).slice(0, 100) : 'NONE');
     expect(hit).toBeTruthy();
     expect(String(hit![0])).toContain(ORG_A);
 
@@ -260,7 +260,7 @@ export const enObjects: NonNullable<TranslationData['objects']> = {
       },
       via_override: {
         label: "Via Admin Override",
-        help: "True when the actor was admitted to this action only by the privileged-override path (#3424) — they held no slot in the request’s pending-approver slate."
+        help: "True when the actor held no slot in the request’s pending-approver slate and was admitted to this action only by the privileged override, which lets a platform or organization admin act on any pending request so that one nobody in its slate can decide never stays stuck."
       },
       reassign_from: {
         label: "Reassigned From",
@@ -272,7 +272,7 @@ export const enObjects: NonNullable<TranslationData['objects']> = {
       },
       attachments: {
         label: "Attachments",
-        help: "Files supporting this action — e.g. a signed contract or evidence (#3266)."
+        help: "Files supporting this action — e.g. a signed contract or evidence."
       },
       created_at: {
         label: "Created At"
@@ -297,7 +297,7 @@ export const enObjects: NonNullable<TranslationData['objects']> = {
   sys_approval_delegation: {
     label: "Approval Delegation",
     pluralLabel: "Approval Delegations",
-    description: "Self-service out-of-office rule: route this user's approver slots to a delegate within a time window (#1322 M1).",
+    description: "Self-service out-of-office rule: route this user's approver slots to a delegate within a time window.",
     fields: {
       id: {
         label: "Delegation ID"
 
@@ -260,7 +260,7 @@ export const esESObjects: NonNullable<TranslationData['objects']> = {
       },
       via_override: {
         label: "Mediante anulación de administrador",
-        help: "Verdadero cuando el actor fue admitido en esta acción únicamente por la vía de anulación privilegiada (#3424): no ocupaba ningún puesto en la lista de aprobadores pendientes de la solicitud."
+        help: "Verdadero cuando el actor no ocupaba ningún puesto en la lista de aprobadores pendientes de la solicitud y fue admitido en esta acción únicamente por la anulación privilegiada, que permite a un administrador de la plataforma o de la organización actuar sobre cualquier solicitud pendiente para que una que nadie de su lista puede decidir nunca quede bloqueada."
       },
       reassign_from: {
         label: "Reasignado de",
@@ -272,7 +272,7 @@ export const esESObjects: NonNullable<TranslationData['objects']> = {
       },
       attachments: {
         label: "Adjuntos",
-        help: "Archivos que respaldan esta acción, p. ej. un contrato firmado o pruebas (#3266)."
+        help: "Archivos que respaldan esta acción, p. ej. un contrato firmado o pruebas."
       },
       created_at: {
         label: "Creado el"
@@ -297,7 +297,7 @@ export const esESObjects: NonNullable<TranslationData['objects']> = {
   sys_approval_delegation: {
     label: "Approval Delegation",
     pluralLabel: "Approval Delegations",
-    description: "Self-service out-of-office rule: route this user's approver slots to a delegate within a time window (#1322 M1).",
+    description: "Self-service out-of-office rule: route this user's approver slots to a delegate within a time window.",
     fields: {
       id: {
         label: "Delegation ID"
 
@@ -22,7 +22,7 @@ export const esESGeneratedSourceHashes: Readonly<Record<string, string>> = {
   "objects.sys_approval_delegation._views.active.emptyState.message": "1095d61c017b6db2",
   "objects.sys_approval_delegation._views.active.emptyState.title": "799d28e51d88963e",
   "objects.sys_approval_delegation._views.active.label": "eee107cf4e466779",
-  "objects.sys_approval_delegation.description": "432b0435aa15f6b9",
+  "objects.sys_approval_delegation.description": "517790acf02a372d",
   "objects.sys_approval_delegation.fields.created_at.label": "1f02d416befb595b",
   "objects.sys_approval_delegation.fields.delegate_id.help": "9cfc641f7b0b04f9",
   "objects.sys_approval_delegation.fields.delegate_id.label": "afd6d8733dc5bc14",