agentweaver 0.1.5 → 0.1.6

package/dist/pipeline/nodes/gitlab-review-artifacts-node.js

@@ -0,0 +1,105 @@
+import { writeFileSync } from "node:fs";
+import { readFileSync } from "node:fs";
+import { TaskRunnerError } from "../../errors.js";
+function commentLine(comment) {
+    if (typeof comment.new_line === "number") {
+        return comment.new_line;
+    }
+    if (typeof comment.old_line === "number") {
+        return comment.old_line;
+    }
+    return null;
+}
+function commentLocation(comment) {
+    const filePath = typeof comment.file_path === "string" ? comment.file_path.trim() : "";
+    const line = commentLine(comment);
+    if (!filePath) {
+        return "general";
+    }
+    return line === null ? filePath : `${filePath}:${line}`;
+}
+function toReviewFinding(comment, index) {
+    const body = typeof comment.body === "string" ? comment.body.trim() : "";
+    if (!body) {
+        return null;
+    }
+    const author = typeof comment.author === "string" ? comment.author.trim() : "unknown";
+    const location = commentLocation(comment);
+    const preview = body.replace(/\s+/g, " ");
+    const titlePreview = preview.length > 72 ? `${preview.slice(0, 69)}...` : preview;
+    const discussionId = typeof comment.discussion_id === "string" ? comment.discussion_id.trim() : "";
+    const createdAt = typeof comment.created_at === "string" ? comment.created_at.trim() : "";
+    return {
+        severity: "medium",
+        title: `GitLab comment ${index + 1} | ${location} | ${author} | ${titlePreview}`,
+        description: [
+            `Location: ${location}`,
+            `Author: ${author}`,
+            discussionId ? `Discussion: ${discussionId}` : "",
+            createdAt ? `Created at: ${createdAt}` : "",
+            "",
+            body,
+        ]
+            .filter((line) => line.length > 0)
+            .join("\n"),
+    };
+}
+function renderReviewMarkdown(artifact) {
+    const lines = [
+        "# Review",
+        "",
+        artifact.summary,
+        "",
+        `Ready to merge: ${artifact.ready_to_merge ? "yes" : "no"}`,
+        "",
+    ];
+    if (artifact.findings.length === 0) {
+        lines.push("Замечаний не найдено.");
+        return lines.join("\n");
+    }
+    artifact.findings.forEach((finding, index) => {
+        lines.push(`## Finding ${index + 1}`);
+        lines.push(`- Severity: ${finding.severity}`);
+        lines.push(`- Title: ${finding.title}`);
+        lines.push("");
+        lines.push(finding.description);
+        lines.push("");
+    });
+    return lines.join("\n");
+}
+export const gitlabReviewArtifactsNode = {
+    kind: "gitlab-review-artifacts",
+    version: 1,
+    async run(_context, params) {
+        let parsed;
+        try {
+            parsed = JSON.parse(readFileSync(params.gitlabReviewJsonFile, "utf8"));
+        }
+        catch (error) {
+            throw new TaskRunnerError(`Failed to read GitLab review artifact from ${params.gitlabReviewJsonFile}: ${error.message}`);
+        }
+        const gitlabReview = parsed;
+        const findings = (Array.isArray(gitlabReview.comments) ? gitlabReview.comments : [])
+            .map((comment, index) => toReviewFinding(comment, index))
+            .filter((finding) => finding !== null);
+        const artifact = {
+            summary: findings.length > 0
+                ? `Импортировано ${findings.length} комментариев код-ревью из GitLab.`
+                : "Комментарии код-ревью в GitLab не найдены.",
+            ready_to_merge: findings.length === 0,
+            findings,
+        };
+        writeFileSync(params.reviewJsonFile, `${JSON.stringify(artifact, null, 2)}\n`, "utf8");
+        writeFileSync(params.reviewFile, `${renderReviewMarkdown(artifact)}\n`, "utf8");
+        return {
+            value: {
+                findingsCount: findings.length,
+                readyToMerge: artifact.ready_to_merge,
+            },
+            outputs: [
+                { kind: "artifact", path: params.reviewFile, required: true },
+                { kind: "artifact", path: params.reviewJsonFile, required: true },
+            ],
+        };
+    },
+};

package/dist/pipeline/nodes/local-script-check-node.js

@@ -0,0 +1,81 @@
+import { TaskRunnerError } from "../../errors.js";
+import { printInfo } from "../../tui.js";
+function parseStructuredResult(output, commandLabel) {
+    const lines = output
+        .split(/\r?\n/)
+        .map((line) => line.replace(/\u001b\[[0-9;]*m/g, "").trim())
+        .filter(Boolean);
+    for (let index = lines.length - 1; index >= 0; index -= 1) {
+        const line = lines[index];
+        if (!line) {
+            continue;
+        }
+        const candidates = [];
+        if (line.startsWith("{") && line.endsWith("}")) {
+            candidates.push(line);
+        }
+        const firstBrace = line.indexOf("{");
+        const lastBrace = line.lastIndexOf("}");
+        if (firstBrace >= 0 && lastBrace > firstBrace) {
+            const slice = line.slice(firstBrace, lastBrace + 1).trim();
+            if (slice && !candidates.includes(slice)) {
+                candidates.push(slice);
+            }
+        }
+        for (const rawJson of candidates) {
+            let parsed;
+            try {
+                parsed = JSON.parse(rawJson);
+            }
+            catch {
+                continue;
+            }
+            if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
+                continue;
+            }
+            const candidate = parsed;
+            if (typeof candidate.ok !== "boolean" ||
+                typeof candidate.kind !== "string" ||
+                typeof candidate.stage !== "string" ||
+                typeof candidate.exitCode !== "number" ||
+                typeof candidate.summary !== "string" ||
+                typeof candidate.command !== "string") {
+                continue;
+            }
+            const details = candidate.details;
+            if (details !== undefined && (!details || typeof details !== "object" || Array.isArray(details))) {
+                continue;
+            }
+            return {
+                ok: candidate.ok,
+                kind: candidate.kind,
+                stage: candidate.stage,
+                exitCode: candidate.exitCode,
+                summary: candidate.summary,
+                command: candidate.command,
+                details: details ?? {},
+            };
+        }
+    }
+    throw new TaskRunnerError(`Structured result is missing or invalid in output of '${commandLabel}'.`);
+}
+export const localScriptCheckNode = {
+    kind: "local-script-check",
+    version: 1,
+    async run(context, params) {
+        printInfo(params.labelText);
+        const output = await context.runtime.runCommand(params.argv, {
+            dryRun: context.dryRun,
+            verbose: context.verbose,
+            label: params.argv.join(" "),
+            printFailureOutput: true,
+            env: { ...context.env },
+        });
+        return {
+            value: {
+                output,
+                parsed: parseStructuredResult(output, params.argv.join(" ")),
+            },
+        };
+    },
+};

package/dist/pipeline/nodes/review-findings-form-node.js

@@ -6,20 +6,20 @@ export const reviewFindingsFormNode = {
     async run(_context, params) {
         let parsed;
         try {
-            parsed = JSON.parse(readFileSync(params.reviewJsonFile, "utf8"));
+            parsed = JSON.parse(readFileSync(params.reviewReplyJsonFile, "utf8"));
         }
         catch (error) {
-            throw new TaskRunnerError(`Failed to read review findings from ${params.reviewJsonFile}: ${error.message}`);
+            throw new TaskRunnerError(`Failed to read review reply from ${params.reviewReplyJsonFile}: ${error.message}`);
         }
-        const review = parsed;
-        const findings = Array.isArray(review.findings) ? review.findings : [];
-        const selectableFindings = findings
-            .map((finding) => ({
-            severity: typeof finding.severity === "string" ? finding.severity.trim() : "",
-            title: typeof finding.title === "string" ? finding.title.trim() : "",
-            description: typeof finding.description === "string" ? finding.description.trim() : "",
+        const reviewReply = parsed;
+        const responses = Array.isArray(reviewReply.responses) ? reviewReply.responses : [];
+        const selectableFindings = responses
+            .map((response) => ({
+            findingTitle: typeof response.finding_title === "string" ? response.finding_title.trim() : "",
+            disposition: typeof response.disposition === "string" ? response.disposition.trim() : "",
+            action: typeof response.action === "string" ? response.action.trim() : "",
         }))
-            .filter((finding) => finding.title.length > 0);
+            .filter((response) => response.findingTitle.length > 0);
         const fields = [
             {
                 id: "apply_all",
@@ -35,10 +35,10 @@ export const reviewFindingsFormNode = {
                 type: "multi-select",
                 label: "Какие findings исправить сейчас",
                 help: "Space переключает пункт. Если apply_all=false, выберите хотя бы один finding.",
-                options: selectableFindings.map((finding) => ({
-                    value: finding.title,
-                    label: `[${finding.severity || "info"}] ${finding.title}`,
-                    ...(finding.description ? { description: finding.description } : {}),
+                options: selectableFindings.map((response) => ({
+                    value: response.findingTitle,
+                    label: `${response.findingTitle} | ${response.disposition || "-"}`,
+                    ...(response.action ? { description: response.action } : {}),
                 })),
                 default: [],
             });

package/dist/pipeline/prompt-registry.js

@@ -1,4 +1,4 @@
-import { BUG_ANALYZE_PROMPT_TEMPLATE, BUG_FIX_PROMPT_TEMPLATE, IMPLEMENT_PROMPT_TEMPLATE, MR_DESCRIPTION_PROMPT_TEMPLATE, PLAN_PROMPT_TEMPLATE, REVIEW_FIX_PROMPT_TEMPLATE, REVIEW_PROMPT_TEMPLATE, REVIEW_REPLY_PROMPT_TEMPLATE, REVIEW_REPLY_SUMMARY_PROMPT_TEMPLATE, REVIEW_SUMMARY_PROMPT_TEMPLATE, RUN_LINTER_LOOP_FIX_PROMPT_TEMPLATE, RUN_TESTS_LOOP_FIX_PROMPT_TEMPLATE, TASK_SUMMARY_PROMPT_TEMPLATE, TEST_FIX_PROMPT_TEMPLATE, TEST_LINTER_FIX_PROMPT_TEMPLATE, } from "../prompts.js";
+import { BUG_ANALYZE_PROMPT_TEMPLATE, BUG_FIX_PROMPT_TEMPLATE, IMPLEMENT_PROMPT_TEMPLATE, MR_DESCRIPTION_PROMPT_TEMPLATE, PLAN_PROMPT_TEMPLATE, REVIEW_FIX_PROMPT_TEMPLATE, REVIEW_PROMPT_TEMPLATE, REVIEW_REPLY_PROMPT_TEMPLATE, REVIEW_REPLY_SUMMARY_PROMPT_TEMPLATE, REVIEW_SUMMARY_PROMPT_TEMPLATE, RUN_LINTER_LOOP_FIX_PROMPT_TEMPLATE, RUN_TESTS_LOOP_FIX_PROMPT_TEMPLATE, TASK_SUMMARY_PROMPT_TEMPLATE, } from "../prompts.js";
 const promptTemplates = {
     "bug-analyze": BUG_ANALYZE_PROMPT_TEMPLATE,
     "bug-fix": BUG_FIX_PROMPT_TEMPLATE,
@@ -13,8 +13,6 @@ const promptTemplates = {
     "run-linter-loop-fix": RUN_LINTER_LOOP_FIX_PROMPT_TEMPLATE,
     "run-tests-loop-fix": RUN_TESTS_LOOP_FIX_PROMPT_TEMPLATE,
     "task-summary": TASK_SUMMARY_PROMPT_TEMPLATE,
-    "test-fix": TEST_FIX_PROMPT_TEMPLATE,
-    "test-linter-fix": TEST_LINTER_FIX_PROMPT_TEMPLATE,
 };
 export function isPromptTemplateRef(value) {
     return value in promptTemplates;

package/dist/pipeline/registry.js

@@ -2,12 +2,14 @@ import { commandCheckExecutor } from "../executors/command-check-executor.js";
 import { claudeExecutor } from "../executors/claude-executor.js";
 import { codexDockerExecutor } from "../executors/codex-docker-executor.js";
 import { codexLocalExecutor } from "../executors/codex-local-executor.js";
+import { fetchGitLabReviewExecutor } from "../executors/fetch-gitlab-review-executor.js";
 import { jiraFetchExecutor } from "../executors/jira-fetch-executor.js";
 import { processExecutor } from "../executors/process-executor.js";
 import { verifyBuildExecutor } from "../executors/verify-build-executor.js";
 const builtInExecutors = {
     process: processExecutor,
     "command-check": commandCheckExecutor,
+    "fetch-gitlab-review": fetchGitLabReviewExecutor,
     "jira-fetch": jiraFetchExecutor,
     "codex-local": codexLocalExecutor,
     "codex-docker": codexDockerExecutor,

package/dist/pipeline/value-resolver.js

@@ -1,5 +1,5 @@
 import { existsSync } from "node:fs";
-import { artifactFile, bugAnalyzeArtifacts, bugAnalyzeFile, bugAnalyzeJsonFile, bugFixDesignFile, bugFixDesignJsonFile, bugFixPlanFile, bugFixPlanJsonFile, designFile, designJsonFile, jiraDescriptionFile, jiraDescriptionJsonFile, jiraTaskFile, mrDescriptionFile, mrDescriptionJsonFile, planArtifacts, planFile, planJsonFile, qaFile, qaJsonFile, readyToMergeFile, reviewFile, reviewFixFile, reviewFixJsonFile, reviewJsonFile, reviewReplyFile, reviewReplyJsonFile, taskSummaryFile, taskSummaryJsonFile, } from "../artifacts.js";
+import { artifactFile, bugAnalyzeArtifacts, bugAnalyzeFile, bugAnalyzeJsonFile, bugFixDesignFile, bugFixDesignJsonFile, bugFixPlanFile, bugFixPlanJsonFile, designFile, designJsonFile, gitlabReviewFile, gitlabReviewInputJsonFile, gitlabReviewJsonFile, jiraDescriptionFile, jiraDescriptionJsonFile, jiraTaskFile, mrDescriptionFile, mrDescriptionJsonFile, planArtifacts, planFile, planJsonFile, qaFile, qaJsonFile, readyToMergeFile, reviewFile, reviewFixFile, reviewFixJsonFile, reviewJsonFile, reviewReplyFile, reviewReplyJsonFile, taskSummaryFile, taskSummaryJsonFile, } from "../artifacts.js";
 import { TaskRunnerError } from "../errors.js";
 import { formatTemplate } from "../prompts.js";
 function readStepRef(segments, context, originalPath) {
@@ -88,6 +88,12 @@ function resolveArtifact(spec, context) {
             return designFile(taskKey);
         case "design-json-file":
             return designJsonFile(taskKey);
+        case "gitlab-review-file":
+            return gitlabReviewFile(taskKey);
+        case "gitlab-review-input-json-file":
+            return gitlabReviewInputJsonFile(taskKey);
+        case "gitlab-review-json-file":
+            return gitlabReviewJsonFile(taskKey);
         case "jira-description-file":
             return jiraDescriptionFile(taskKey);
         case "jira-description-json-file":

package/dist/prompts.js

@@ -56,8 +56,6 @@ export const REVIEW_FIX_PROMPT_TEMPLATE = "Используй только ст
 export const TASK_SUMMARY_PROMPT_TEMPLATE = "Посмотри в {jira_task_file}. " +
     "Сделай краткое резюме задачи, на 1-2 абзаца. " +
     "Сначала запиши source-of-truth JSON в {task_summary_json_file} в виде объекта { summary: string }, затем markdown-версию в {task_summary_file}.";
-export const TEST_FIX_PROMPT_TEMPLATE = "Прогони тесты, исправь ошибки.";
-export const TEST_LINTER_FIX_PROMPT_TEMPLATE = "Прогони линтер, исправь замечания.";
 export const RUN_TESTS_LOOP_FIX_PROMPT_TEMPLATE = "Запусти ./run_tests.sh, проанализируй последнюю ошибку проверки, исправь код и подготовь изменения так, чтобы следующий прогон run_tests.sh прошёл успешно.";
 export const RUN_LINTER_LOOP_FIX_PROMPT_TEMPLATE = "Запусти ./run_linter.sh, проанализируй последнюю ошибку линтера или генерации, исправь код и подготовь изменения так, чтобы следующий прогон run_linter.sh прошёл успешно.";
 export const AUTO_REVIEW_FIX_EXTRA_PROMPT = "Исправлять только блокеры, критикалы и важные";

package/dist/structured-artifacts.js

@@ -54,6 +54,11 @@ function validateBriefText(value, path, issues) {
     }
     expectNonEmptyString(value.summary, `${path}.summary`, issues);
 }
+function expectNumber(value, path, issues) {
+    if (typeof value !== "number" || Number.isNaN(value)) {
+        issues.push(`${path} must be a number`);
+    }
+}
 function implementationDesignSchema() {
     return {
         id: "implementation-design/v1",
@@ -204,6 +209,33 @@ function reviewFixReportSchema() {
         },
     };
 }
+function gitlabReviewSchema() {
+    return {
+        id: "gitlab-review/v1",
+        validate({ path, value }) {
+            const issues = [];
+            if (!expectObject(value, path, issues)) {
+                return issues;
+            }
+            expectNonEmptyString(value.summary, `${path}.summary`, issues);
+            expectNonEmptyString(value.merge_request_url, `${path}.merge_request_url`, issues);
+            expectNonEmptyString(value.project_path, `${path}.project_path`, issues);
+            expectNumber(value.merge_request_iid, `${path}.merge_request_iid`, issues);
+            expectNonEmptyString(value.fetched_at, `${path}.fetched_at`, issues);
+            expectObjectArray(value.comments, `${path}.comments`, issues, (item, itemPath, currentIssues) => {
+                expectNonEmptyString(item.id, `${itemPath}.id`, currentIssues);
+                expectNonEmptyString(item.discussion_id, `${itemPath}.discussion_id`, currentIssues);
+                expectNonEmptyString(item.body, `${itemPath}.body`, currentIssues);
+                expectNonEmptyString(item.author, `${itemPath}.author`, currentIssues);
+                expectNonEmptyString(item.created_at, `${itemPath}.created_at`, currentIssues);
+                if (item.file_path !== undefined && item.file_path !== null) {
+                    expectNonEmptyString(item.file_path, `${itemPath}.file_path`, currentIssues);
+                }
+            }, true);
+            return issues;
+        },
+    };
+}
 function userInputSchema() {
     return {
         id: "user-input/v1",
@@ -223,6 +255,7 @@ const schemas = {
     "bug-analysis/v1": bugAnalysisSchema(),
     "bug-fix-design/v1": bugFixDesignSchema(),
     "bug-fix-plan/v1": bugFixPlanSchema(),
+    "gitlab-review/v1": gitlabReviewSchema(),
     "implementation-design/v1": implementationDesignSchema(),
     "implementation-plan/v1": implementationPlanSchema(),
     "jira-description/v1": { id: "jira-description/v1", validate: ({ path, value }) => {