engineering-memory 1.11.28 → 1.11.30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "engineering-memory",
-  "version": "1.11.28",
+  "version": "1.11.30",
   "description": "Installs the Engineering Memory skill and its local MCP bridge. Sign in after installing; your organization and project are resolved from your account.",
   "license": "UNLICENSED",
   "type": "module",

package/runtime/build.json

@@ -1,3 +1,3 @@
 {
-  "gitHead": "7ca14a854db07f8fda3823f303e8a7a6accbebff"
+  "gitHead": "36588e4f0edd74e5ca4f7ebb8c8f77a41a2af925"
 }

package/runtime/dist/src/config.js

@@ -71,6 +71,18 @@ export const endpoints = {
     taskDeliveryDeliver: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/delivery/deliver`,
     taskDeliveryCancel: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/delivery/cancel`,
     taskReviewRequest: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review-request`,
+    reviewQueue: (projectId) => `/projects/${projectId}/reviews`,
+    review: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review`,
+    reviewRounds: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review/rounds`,
+    reviewRoundResult: (projectId, roundId) => `/projects/${projectId}/review-rounds/${roundId}/result`,
+    reviewRoundRules: (projectId, roundId) => `/projects/${projectId}/review-rounds/${roundId}/rules`,
+    reviewFindingRejection: (projectId, findingId) => `/projects/${projectId}/review-findings/${findingId}/rejection`,
+    reviewPass: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review/pass`,
+    reviewSkip: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review/skip`,
+    reviewExemption: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review/exemption`,
+    reviewRoundComment: (projectId, roundId) => `/projects/${projectId}/review-rounds/${roundId}/comment`,
+    reviewComment: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review/comment`,
+    reviewProviderFailure: (projectId, taskId) => `/projects/${projectId}/tasks/${taskId}/review/provider-failure`,
     projectSetup: '/projects/setup',
     projectList: '/projects',
     projectResolve: '/projects/resolve',

package/runtime/dist/src/git/git-inspector.js

@@ -429,6 +429,19 @@ export class GitInspector {
         const result = await this.runner.run('git', ['merge-base', '--is-ancestor', ancestor, descendant], { cwd: repoRoot });
         return result.exitCode === 0;
     }
+    async mergeBase(repoRoot, left, right) {
+        return await this.gitValue(repoRoot, ['merge-base', left, right]);
+    }
+    async changedFiles(repoRoot, base, head) {
+        const result = await this.runner.run('git', ['diff', '--name-status', '-z', '--no-renames', base, head, '--'], { cwd: repoRoot });
+        if (result.exitCode !== 0)
+            throw new Error(`Git diff failed: ${result.stderr.trim()}`);
+        const tokens = result.stdout.split('\0').filter(Boolean);
+        const files = [];
+        for (let index = 0; index + 1 < tokens.length; index += 2)
+            files.push({ status: tokens[index], path: normalizeGitPath(tokens[index + 1]) });
+        return files;
+    }
     async remoteRefContaining(repoRoot, commit) {
         return await this.gitValue(repoRoot, [
             'for-each-ref',

package/runtime/dist/src/localization/catalogue.generated.js

@@ -35,6 +35,10 @@ export const bundledTexts = {
         'delivery.commit_push_draft_pr.label': 'Commit, push and draft PR/MR',
         'delivery.commit_push_pr.description': 'Push the branch and open a request ready for review.',
         'delivery.commit_push_pr.label': 'Commit, push and PR/MR',
+        'delivery.commit_push_pr_unreviewed.description': 'The PR/MR opens ready for the team; no code review is recorded.',
+        'delivery.commit_push_pr_unreviewed.label': 'Commit, push and PR/MR without review',
+        'delivery.commit_push_review_pr.description': 'The PR/MR opens as a draft and becomes ready once the code review ends.',
+        'delivery.commit_push_review_pr.label': 'Commit, push and PR/MR, reviewed first',
         'delivery.context': 'The task is verified and closed. This choice determines where the code goes; approval to publish memory does not authorize Git delivery.',
         'delivery.defer.description': 'Keep the task closed and preserve its files without Git delivery.',
         'delivery.defer.label': 'Keep it for now',
@@ -42,6 +46,9 @@ export const bundledTexts = {
         'delivery.example': 'Commit keeps work on the local branch. PR/MR pushes that branch and opens it for team review.',
         'delivery.message': 'How should we deliver these changes?',
         'delivery.recorded': 'Already answered in Engineering Memory',
+        'delivery.reviewFix': 'These changes fix the review findings of {task}, so they go onto its open PR/MR branch; no new PR/MR is opened.',
+        'delivery.reviewOptional': 'Code review is optional in this project: choose a reviewed PR/MR or one without review.',
+        'delivery.reviewRequired': 'This project reviews code before a PR/MR is ready: the PR/MR opens as a draft and becomes ready after the review.',
         'forgetWorktree.context': 'A checkout that cannot be read is still counted against the worktree limit although nothing can use it; forgetting it frees that slot. No file in the folder is touched or deleted, and the branch stays.',
         'forgetWorktree.example': 'After this, {task} no longer appears in the worktree list and the {folder} folder stays on disk exactly as it is.',
         'forgetWorktree.forget': 'Stop tracking it; leave the files alone',
@@ -263,6 +270,47 @@ export const bundledTexts = {
         'reviewBatch.untitled': 'untitled {kind}',
         'reviewBatch.unversioned': 'Written without a source inspection: {titles}. Tasks that read memory by source version will not see them until an inspection retains them.',
         'reviewBatch.why': 'Why',
+        'reviewComment.exempted': 'Engineering Memory code review of {task}: exempted. Reason: {reason}',
+        'reviewComment.findingsOf': 'Findings of round {round}:',
+        'reviewComment.fixed': 'fixed in {commit}',
+        'reviewComment.open': 'still open',
+        'reviewComment.passed': 'Engineering Memory code review of {task}: passed after {rounds, plural, one {# round} other {# rounds}}',
+        'reviewComment.rejected': 'rejected: {reason}',
+        'reviewComment.round': 'Engineering Memory code review of {task}, round {round}: {count, plural, one {# finding} other {# findings}}',
+        'reviewComment.rule': 'rule {rule}',
+        'reviewComment.skipped': 'Engineering Memory code review of {task}: skipped',
+        'reviewConclude.confirm.description': 'The review ends as described above.',
+        'reviewConclude.confirm.label': 'Yes, end it',
+        'reviewConclude.defer.description': 'The review stays open; nothing is recorded.',
+        'reviewConclude.defer.label': 'Not now',
+        'reviewConclude.example': 'After this, the PR/MR can be made ready for the team.',
+        'reviewConclude.exempted.context': 'Review is required here, so only an owner or maintainer can exempt a delivery, with a written reason that stays with the review.',
+        'reviewConclude.exempted.message': 'Exempt {task} from review?',
+        'reviewConclude.passed.context': 'The fix was pushed and no finding is open. Without a new round, the last round’s result stands.',
+        'reviewConclude.passed.message': 'End the review of {task} as passed?',
+        'reviewConclude.reasonTitle': 'Reason',
+        'reviewConclude.skipped.context': 'Code review is not required in this project, so it can be skipped. Engineering Memory records that it was skipped and by whom.',
+        'reviewConclude.skipped.message': 'Skip the review of {task}?',
+        'reviewPolicy.context': 'The setting applies to the whole project from now on; work already delivered keeps its review state. On GitHub, Engineering Memory records the setting but GitHub does not enforce it.',
+        'reviewPolicy.current': 'Current setting',
+        'reviewPolicy.example': 'With Required, a finished task’s PR/MR opens as a draft, an agent reviews it against the project’s rules, and it becomes ready once each finding is fixed or rejected with a reason.',
+        'reviewPolicy.keep.description': 'Nothing changes.',
+        'reviewPolicy.keep.label': 'Leave it as it is',
+        'reviewPolicy.message': 'Should code be reviewed before a PR/MR is ready?',
+        'reviewPolicy.none.description': 'Deliveries work as they do today, with no code review step.',
+        'reviewPolicy.none.label': 'No review',
+        'reviewPolicy.optional.description': 'The delivery question offers a reviewed PR/MR or one without review; a review can be skipped at any time.',
+        'reviewPolicy.optional.label': 'Optional',
+        'reviewPolicy.required.description': 'Every PR/MR is reviewed before it is ready; only an owner or maintainer can exempt one, with a written reason.',
+        'reviewPolicy.required.label': 'Required',
+        'reviewReject.context': 'A rejected finding needs a written reason. The reason stays with the review, where the team sees it.',
+        'reviewReject.example': 'For example: “This rule is for screens, and this file is a test helper.”',
+        'reviewReject.keep.description': 'The finding stays open.',
+        'reviewReject.keep.label': 'Keep the finding',
+        'reviewReject.message': 'Reject this review finding?',
+        'reviewReject.reasonTitle': 'Reason',
+        'reviewReject.reject.description': 'The finding no longer holds the review back.',
+        'reviewReject.reject.label': 'Reject with this reason',
         'reworkPlan.approve.description': 'The work items are created now, exactly as listed. Implementing each phase still starts separately.',
         'reworkPlan.approve.label': 'Create these work items',
         'reworkPlan.branch': 'branch',
@@ -357,6 +405,7 @@ export const bundledTexts = {
         'taskStart.base.other': 'Another branch on origin',
         'taskStart.base.otherDetail': 'Enter it below.',
         'taskStart.continuePlan': "Continue {branch}, the branch of this work item's confirmed plan",
+        'taskStart.continueReview': 'Continue {branch}, the branch of the PR/MR of {task} under review',
         'taskStart.example': 'A worktree keeps this task in its own folder.',
         'taskStart.keepContext': 'Continue on the current branch.',
         'taskStart.location.busy': 'Task {task} is still running in another client in this folder, so the new branch cannot start here. Once it is paused there, this folder can be chosen too.',
@@ -461,6 +510,10 @@ export const bundledTexts = {
         'delivery.commit_push_draft_pr.label': 'Commit, push ve taslak PR/MR',
         'delivery.commit_push_pr.description': 'Dalı gönder ve incelemeye hazır bir istek aç.',
         'delivery.commit_push_pr.label': 'Commit, push ve PR/MR',
+        'delivery.commit_push_pr_unreviewed.description': 'PR/MR ekibe hazır açılır; kod incelemesi kaydedilmez.',
+        'delivery.commit_push_pr_unreviewed.label': 'Commit, push ve incelemesiz PR/MR',
+        'delivery.commit_push_review_pr.description': 'PR/MR taslak açılır; kod incelemesi bitince hazıra alınır.',
+        'delivery.commit_push_review_pr.label': 'Commit, push ve PR/MR (önce incelenir)',
         'delivery.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.',
         'delivery.defer.description': 'Görev kapalı kalır; dosyalar korunur, Git teslimi yapılmaz.',
         'delivery.defer.label': 'Şimdilik beklet',
@@ -468,6 +521,9 @@ export const bundledTexts = {
         'delivery.example': 'Commit yerel dalda kalır. PR/MR seçersen değişiklikler uzak dala gönderilir ve ekip incelemesine açılır.',
         'delivery.message': 'Değişiklikleri nasıl teslim edelim?',
         'delivery.recorded': 'Engineering Memory’de verilmiş cevap',
+        'delivery.reviewFix': 'Bu değişiklikler {task} incelemesindeki bulguları düzeltir; açık PR/MR dalına gider, yeni PR/MR açılmaz.',
+        'delivery.reviewOptional': 'Bu projede kod incelemesi isteğe bağlı: incelenen ya da incelemesiz bir PR/MR seçebilirsin.',
+        'delivery.reviewRequired': 'Bu projede PR/MR hazır olmadan önce kod incelenir: PR/MR taslak açılır, inceleme bitince hazıra alınır.',
         'forgetWorktree.context': 'Okunamayan bir çalışma kopyası hiçbir işe yaramadığı hâlde worktree sınırına sayılmaya devam eder; takipten çıkarılırsa o yer boşalır. Klasördeki hiçbir dosyaya dokunulmaz, hiçbiri silinmez; dal da kalır.',
         'forgetWorktree.example': 'Bundan sonra {task} worktree listesinde görünmez, {folder} klasörü ise diskte olduğu gibi kalır.',
         'forgetWorktree.forget': 'Takip etmeyi bırak; dosyalara dokunma',
@@ -689,6 +745,47 @@ export const bundledTexts = {
         'reviewBatch.untitled': 'başlıksız {kind}',
         'reviewBatch.unversioned': 'Kaynak incelemesi yapılmadan yazılanlar: {titles}. Hafızayı kaynak sürümüne göre okuyan görevler, bir inceleme bunları saklayana kadar görmez.',
         'reviewBatch.why': 'Nedeni',
+        'reviewComment.exempted': '{task} için Engineering Memory kod incelemesi: muaf tutuldu. Gerekçe: {reason}',
+        'reviewComment.findingsOf': 'Tur {round} bulguları:',
+        'reviewComment.fixed': '{commit} ile düzeltildi',
+        'reviewComment.open': 'hâlâ açık',
+        'reviewComment.passed': '{task} için Engineering Memory kod incelemesi: {rounds, plural, one {# turda} other {# turda}} geçti',
+        'reviewComment.rejected': 'reddedildi: {reason}',
+        'reviewComment.round': '{task} için Engineering Memory kod incelemesi, tur {round}: {count, plural, one {# bulgu} other {# bulgu}}',
+        'reviewComment.rule': 'kural {rule}',
+        'reviewComment.skipped': '{task} için Engineering Memory kod incelemesi: atlandı',
+        'reviewConclude.confirm.description': 'İnceleme yukarıda anlatıldığı gibi sonuçlanır.',
+        'reviewConclude.confirm.label': 'Evet, bitir',
+        'reviewConclude.defer.description': 'İnceleme açık kalır; hiçbir şey kaydedilmez.',
+        'reviewConclude.defer.label': 'Şimdi değil',
+        'reviewConclude.example': 'Bundan sonra PR/MR ekip için hazıra alınabilir.',
+        'reviewConclude.exempted.context': 'Burada inceleme zorunlu; bu yüzden yalnızca sahip ya da bakımcı, incelemeyle birlikte kalacak yazılı bir gerekçeyle muaf tutabilir.',
+        'reviewConclude.exempted.message': '{task} incelemeden muaf tutulsun mu?',
+        'reviewConclude.passed.context': 'Düzeltme gönderildi ve açık bulgu yok. Yeni tur yapılmazsa son turun sonucu geçerli olur.',
+        'reviewConclude.passed.message': '{task} incelemesi geçti olarak bitsin mi?',
+        'reviewConclude.reasonTitle': 'Gerekçe',
+        'reviewConclude.skipped.context': 'Bu projede kod incelemesi zorunlu değil, atlanabilir. Engineering Memory atlandığını ve kimin atladığını kaydeder.',
+        'reviewConclude.skipped.message': '{task} incelemesi atlansın mı?',
+        'reviewPolicy.context': 'Ayar bundan sonra tüm projede geçerli olur; önceden teslim edilen işlerin inceleme durumu değişmez. GitHub’da ayarı Engineering Memory kaydeder, GitHub zorlamaz.',
+        'reviewPolicy.current': 'Şu anki ayar',
+        'reviewPolicy.example': 'Zorunlu seçilirse biten görevin PR/MR’ı taslak açılır, bir ajan projenin kurallarına göre inceler; her bulgu düzeltilince ya da gerekçeyle reddedilince hazıra alınır.',
+        'reviewPolicy.keep.description': 'Hiçbir şey değişmez.',
+        'reviewPolicy.keep.label': 'Olduğu gibi kalsın',
+        'reviewPolicy.message': 'PR/MR hazır olmadan önce kod incelensin mi?',
+        'reviewPolicy.none.description': 'Teslimler bugünkü gibi olur; kod incelemesi adımı yoktur.',
+        'reviewPolicy.none.label': 'İnceleme yok',
+        'reviewPolicy.optional.description': 'Teslim sorusu incelenen ya da incelemesiz PR/MR sunar; inceleme her an atlanabilir.',
+        'reviewPolicy.optional.label': 'İsteğe bağlı',
+        'reviewPolicy.required.description': 'Her PR/MR hazır olmadan önce incelenir; yalnızca sahip ya da bakımcı yazılı gerekçeyle muaf tutabilir.',
+        'reviewPolicy.required.label': 'Zorunlu',
+        'reviewReject.context': 'Reddedilen bulgu için yazılı gerekçe gerekir. Gerekçe incelemeyle birlikte kalır ve ekip görür.',
+        'reviewReject.example': 'Örneğin: “Bu kural ekranlar için; bu dosya bir test yardımcısı.”',
+        'reviewReject.keep.description': 'Bulgu açık kalır.',
+        'reviewReject.keep.label': 'Bulgu kalsın',
+        'reviewReject.message': 'Bu inceleme bulgusu reddedilsin mi?',
+        'reviewReject.reasonTitle': 'Gerekçe',
+        'reviewReject.reject.description': 'Bulgu artık incelemeyi bekletmez.',
+        'reviewReject.reject.label': 'Bu gerekçeyle reddet',
         'reworkPlan.approve.description': 'İş kalemleri listelendiği gibi şimdi oluşturulur. Aşamaların kodlanması yine ayrı ayrı başlar.',
         'reworkPlan.approve.label': 'Bu iş kalemlerini oluştur',
         'reworkPlan.branch': 'dal',
@@ -783,6 +880,7 @@ export const bundledTexts = {
         'taskStart.base.other': 'origin üzerinde başka dal',
         'taskStart.base.otherDetail': 'Alttaki alana yaz.',
         'taskStart.continuePlan': '{branch} dalında devam: bu iş kaleminin onaylı planındaki dal',
+        'taskStart.continueReview': '{branch} dalında devam: {task} için incelenen PR/MR’ın dalı',
         'taskStart.example': 'Ayrı worktree, işi ayrı klasörde tutar.',
         'taskStart.keepContext': 'Mevcut dalda devam edilecek.',
         'taskStart.location.busy': '{task} bu klasörde başka bir oturumda hâlâ çalışıyor; yeni branch burada açılamaz. O oturumda duraklatılırsa bu klasör de seçilebilir.',