engineering-memory 1.11.19 → 1.11.21

package/runtime/dist/src/mcp/delivery-tools.js

@@ -1,8 +1,12 @@
 import * as z from 'zod/v4';
 import { languageTag } from '../runtime/questionnaire-store.js';
+import { texts } from '../runtime/texts.js';
 import { sha256, stableStringify } from '../utilities/hash.js';
 import { answerChoice, answerData, askQuestionnaire } from './questionnaire-tools.js';
-const option = z.tuple([z.string().trim().min(1).max(100), z.string().trim().min(1).max(240)]);
+const option = z.strictObject({
+    label: z.string().trim().min(1).max(100),
+    description: z.string().trim().min(1).max(240),
+});
 const copySchema = z.strictObject({
     message: z.string().trim().min(1).max(240),
     context: z.string().trim().min(1).max(500),
@@ -18,47 +22,6 @@ const copySchema = z.strictObject({
     baseTitle: z.string().trim().min(1).max(100),
     baseWrite: option,
 });
-const copies = {
-    tr: {
-        message: 'Değişiklikleri nasıl teslim edelim?',
-        context: 'Görev doğrulandı ve kapandı. Bu seçim kodun nereye gönderileceğini belirler; hafıza değişikliklerini yayımlama onayı bunun yerine geçmez.',
-        example: 'Commit yerel dalda kalır. PR/MR seçersen değişiklikler uzak dala gönderilir ve ekip incelemesine açılır.',
-        destination: 'Dal / uzak depo',
-        commit: ['Yalnızca commit', 'Değişiklikleri bu yerel dalda kaydet.'],
-        commit_push: ['Commit ve push', 'Bu dalı belirtilen uzak depoya gönder.'],
-        commit_push_draft_pr: [
-            'Commit, push ve taslak PR/MR',
-            'Dalı gönder; henüz incelemeye hazır olmayan bir taslak aç.',
-        ],
-        commit_push_pr: ['Commit, push ve PR/MR', 'Dalı gönder ve incelemeye hazır bir istek aç.'],
-        defer: ['Şimdilik beklet', 'Görev kapalı kalır; dosyalar korunur, Git teslimi yapılmaz.'],
-        baseMessage: 'PR/MR hangi dala açılsın?',
-        baseContext: 'Hedef dalı açıkça belirt. Depodaki varsayılan dalı kendiliğimizden seçmeyiz.',
-        baseTitle: 'Hedef dal',
-        baseWrite: ['Bu dala aç', 'Yazdığın dalı hedef al; birleştirme ayrı bir karardır.'],
-    },
-    en: {
-        message: 'How should we deliver these changes?',
-        context: 'The task is verified and closed. This choice determines where the code goes; approval to publish memory does not authorize Git delivery.',
-        example: 'Commit keeps work on the local branch. PR/MR pushes that branch and opens it for team review.',
-        destination: 'Branch / remote repository',
-        commit: ['Commit only', 'Save the changes on this local branch.'],
-        commit_push: ['Commit and push', 'Push this branch to the named remote repository.'],
-        commit_push_draft_pr: [
-            'Commit, push and draft PR/MR',
-            'Push the branch and open a draft that is not ready for review.',
-        ],
-        commit_push_pr: [
-            'Commit, push and PR/MR',
-            'Push the branch and open a request ready for review.',
-        ],
-        defer: ['Keep it for now', 'Keep the task closed and preserve its files without Git delivery.'],
-        baseMessage: 'Which branch should the PR/MR target?',
-        baseContext: 'Specify the target branch explicitly. The remote default is not selected automatically.',
-        baseTitle: 'Target branch',
-        baseWrite: ['Use this branch', 'Target the branch you enter; merging is a separate decision.'],
-    },
-};
 const closedSchema = z.object({
     closed: z.literal(true),
     taskVersion: z.number().int().positive(),
@@ -79,7 +42,7 @@ const closedSchema = z.object({
 });
 export function registerDeliveryTools(server, service) {
     server.registerTool('task.close', {
-        description: 'Close the verified task, then open its durable mode-governed Git delivery selector in the same call. Memory publication is separate. Cancellation/feedback leave delivery pending; retry identical input to resume. PR/MR is one review-request concept, not an instruction to merge. No Git action is performed by this tool. Read-only tasks need no delivery. Use complete copy outside Turkish/English. If the user already explicitly chose Git delivery in their own message, deliveryInstruction preserves that choice with its exact relevant excerpt; it is an agent-reported instruction, never a native answer or a substitute for missing user consent. Do not use it for agent decisions or memory-publication approval.',
+        description: 'Close the verified task, then open its durable mode-governed Git delivery selector in the same call. Memory publication is separate. Cancellation/feedback leave delivery pending; retry identical input to resume. PR/MR is one review-request concept, not an instruction to merge. No Git action is performed by this tool. Read-only tasks need no delivery. Supply complete copy when the product text catalogue lacks the conversation language. If the user already explicitly chose Git delivery in their own message, deliveryInstruction preserves that choice with its exact relevant excerpt; it is an agent-reported instruction, never a native answer or a substitute for missing user consent. Do not use it for agent decisions or memory-publication approval.',
         inputSchema: z.object({
             repoRoot: z.string().min(1).optional(),
             taskId: z.string().min(1),
@@ -103,8 +66,8 @@ export function registerDeliveryTools(server, service) {
         }),
     }, async (input, context) => {
         const language = await service.language(input.language);
-        const primary = language?.split('-')[0];
-        const copy = input.copy ?? (primary === 'tr' || primary === 'en' ? copies[primary] : undefined);
+        const copy = input.copy ??
+            (language ? copySchema.safeParse(texts('delivery', language)).data : undefined);
         if (!copy)
             return output({
                 ok: false,
@@ -191,7 +154,11 @@ export function registerDeliveryTools(server, service) {
                     destination.branch +
                     (canPush ? ' → ' + destination.remote + ' (' + destination.pushUrl + ')' : ''),
                 example: copy.example,
-                options: ids.map((id) => ({ id, label: copy[id][0], description: copy[id][1] })),
+                options: ids.map((id) => ({
+                    id,
+                    label: copy[id].label,
+                    description: copy[id].description,
+                })),
                 binding,
             }, context, [], owner);
         const choice = instruction?.choice ?? (form ? answerChoice(form) : null);
@@ -232,8 +199,8 @@ export function registerDeliveryTools(server, service) {
                 context: copy.baseContext,
                 example: copy.example,
                 options: [
-                    { id: 'write', label: copy.baseWrite[0], description: copy.baseWrite[1] },
-                    { id: 'defer', label: copy.defer[0], description: copy.defer[1] },
+                    { id: 'write', label: copy.baseWrite.label, description: copy.baseWrite.description },
+                    { id: 'defer', label: copy.defer.label, description: copy.defer.description },
                 ],
                 textField: { title: copy.baseTitle, maxLength: 240, requiredForChoice: 'write' },
                 binding: { ...binding, choice },