agentweaver 0.1.5 → 0.1.6

package/Dockerfile.codex

@@ -0,0 +1,55 @@
+FROM golang:1.25.5-bookworm
+
+ARG GOLANGCI_LINT_VERSION=v2.7.2
+ARG MOCKGEN_VERSION=v1.6.0
+ARG SWAG_VERSION=latest
+ARG PROTOC_GEN_GO_VERSION=latest
+ARG PROTOC_GEN_GO_GRPC_VERSION=latest
+
+RUN apt-get update \
+    && apt-get install -y --no-install-recommends \
+        ca-certificates \
+        nodejs \
+        npm \
+        curl \
+        jq \
+        less \
+        file \
+        make \
+        procps \
+        ripgrep \
+        git \
+        openssh-client \
+        docker.io \
+        protobuf-compiler \
+        unzip \
+        zip \
+        findutils \
+    && update-ca-certificates \
+    && rm -rf /var/lib/apt/lists/*
+
+RUN if ! getent group 1000 >/dev/null; then groupadd -g 1000 codex; fi \
+    && if ! getent passwd 1000 >/dev/null; then useradd -u 1000 -g 1000 -d /codex-home/home -M -s /bin/bash codex; fi
+
+RUN npm install -g @openai/codex@latest \
+    && npm cache clean --force
+
+RUN GOBIN=/usr/local/bin go install github.com/golangci/golangci-lint/v2/cmd/golangci-lint@${GOLANGCI_LINT_VERSION} \
+    && GOBIN=/usr/local/bin go install github.com/golang/mock/mockgen@${MOCKGEN_VERSION} \
+    && GOBIN=/usr/local/bin go install github.com/swaggo/swag/cmd/swag@${SWAG_VERSION} \
+    && GOBIN=/usr/local/bin go install google.golang.org/protobuf/cmd/protoc-gen-go@${PROTOC_GEN_GO_VERSION} \
+    && GOBIN=/usr/local/bin go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@${PROTOC_GEN_GO_GRPC_VERSION} \
+    && ln -sf /usr/local/go/bin/go /usr/bin/go \
+    && ln -sf /usr/local/go/bin/gofmt /usr/bin/gofmt
+
+ENV PATH="/usr/local/go/bin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
+
+COPY verify_build.sh /usr/local/bin/verify_build.sh
+COPY run_tests.sh /usr/local/bin/run_tests.sh
+COPY run_linter.sh /usr/local/bin/run_linter.sh
+RUN chmod +x /usr/local/bin/verify_build.sh /usr/local/bin/run_tests.sh /usr/local/bin/run_linter.sh
+
+WORKDIR /workspace
+
+ENTRYPOINT ["codex"]
+CMD ["--dangerously-bypass-approvals-and-sandbox"]

package/README.md

@@ -1,19 +1,20 @@
 # AgentWeaver
 
-`AgentWeaver` is a TypeScript/Node.js CLI for engineering workflows around Jira, Codex, and Claude.
+`AgentWeaver` is a TypeScript/Node.js CLI for engineering workflows around Jira, GitLab review artifacts, Codex, and Claude.
 
 It orchestrates a flow like:
 
-`plan -> implement -> test -> review -> review-fix -> test`
+`plan -> implement -> run-linter-loop -> run-tests-loop -> review -> review-fix`
 
 The package is designed to run as an npm CLI and includes an interactive terminal UI built on `neo-blessed`.
 
 ## What It Does
 
 - Fetches a Jira issue by key or browse URL
+- Fetches GitLab merge request review comments into reusable markdown and JSON artifacts
 - Generates workflow artifacts such as design, implementation plan, QA plan, bug analysis, reviews, and summaries
 - Machine-readable JSON artifacts are stored under `.agentweaver-<TASK>/.artifacts/` and act as the source of truth between workflow steps; Markdown artifacts remain for human inspection
-- Runs workflow stages like `bug-analyze`, `bug-fix`, `mr-description`, `plan`, `task-describe`, `implement`, `review`, `review-fix`, `test`, and `auto`
+- Runs workflow stages like `bug-analyze`, `bug-fix`, `mr-description`, `plan`, `task-describe`, `implement`, `review`, `review-fix`, `run-tests-loop`, `run-linter-loop`, and `auto`
 - Persists compact `auto` pipeline state on disk so runs can resume without storing large agent outputs
 - Uses Docker runtime services for isolated Codex execution and build verification
 
@@ -22,9 +23,9 @@ The package is designed to run as an npm CLI and includes an interactive termina
 The CLI now uses an executor + node + declarative flow architecture.
 
 - `src/index.ts` remains the CLI entrypoint and high-level orchestration layer
-- `src/executors/` contains first-class executors for external actions such as Jira fetch, local Codex, Docker-based build verification, Claude, Claude summaries, and process execution
+- `src/executors/` contains first-class executors for external actions such as Jira fetch, GitLab review fetch, local Codex, Docker-based build verification, Claude, Claude summaries, and process execution
 - `src/pipeline/nodes/` contains reusable runtime nodes built on top of executors
-- `src/pipeline/flow-specs/` contains declarative JSON flow specs for `preflight`, `bug-analyze`, `bug-fix`, `mr-description`, `plan`, `task-describe`, `implement`, `review`, `review-fix`, `test`, `test-fix`, `test-linter-fix`, `run-tests-loop`, `run-linter-loop`, and `auto`
+- `src/pipeline/flow-specs/` contains declarative JSON flow specs for `preflight`, `bug-analyze`, `bug-fix`, `gitlab-review`, `mr-description`, `plan`, `task-describe`, `implement`, `review`, `review-fix`, `run-tests-loop`, `run-linter-loop`, and `auto`
 - `src/runtime/` contains shared runtime services such as command resolution, Docker runtime environment setup, and subprocess execution
 
 This keeps command handlers focused on choosing a flow and providing parameters instead of assembling prompts and subprocess wiring inline.
@@ -86,6 +87,7 @@ Required:
 Common optional variables:
 
 - `JIRA_BASE_URL` — required when you pass only an issue key like `DEMO-123`
+- `GITLAB_TOKEN` — personal access token for `gitlab-review`
 - `AGENTWEAVER_HOME` — path to the AgentWeaver installation directory
 - `DOCKER_COMPOSE_BIN` — override compose command, for example `docker compose`
 - `CODEX_BIN` — override `codex` executable path
@@ -117,6 +119,7 @@ Direct CLI usage:
 agentweaver plan DEMO-3288
 agentweaver bug-analyze DEMO-3288
 agentweaver bug-fix DEMO-3288
+agentweaver gitlab-review DEMO-3288
 agentweaver mr-description DEMO-3288
 agentweaver task-describe DEMO-3288
 agentweaver implement DEMO-3288
@@ -132,6 +135,7 @@ From source checkout:
 node dist/index.js plan DEMO-3288
 node dist/index.js bug-analyze DEMO-3288
 node dist/index.js bug-fix DEMO-3288
+node dist/index.js gitlab-review DEMO-3288
 node dist/index.js mr-description DEMO-3288
 node dist/index.js task-describe DEMO-3288
 node dist/index.js auto DEMO-3288
@@ -189,7 +193,7 @@ Activity pane behavior:
 
 ## Docker Runtime
 
-Docker is used as an isolated execution environment for Codex and build/test verification.
+Docker is used as an isolated execution environment for Codex-related runtime scenarios that still require container orchestration.
 
 Main services:
 

package/dist/artifacts.js

@@ -89,6 +89,15 @@ export function mrDescriptionFile(taskKey) {
 export function mrDescriptionJsonFile(taskKey) {
     return taskArtifactsFile(taskKey, `mr-description-${taskKey}.json`);
 }
+export function gitlabReviewFile(taskKey) {
+    return taskWorkspaceFile(taskKey, `gitlab-review-${taskKey}.md`);
+}
+export function gitlabReviewJsonFile(taskKey) {
+    return taskArtifactsFile(taskKey, `gitlab-review-${taskKey}.json`);
+}
+export function gitlabReviewInputJsonFile(taskKey) {
+    return taskArtifactsFile(taskKey, `gitlab-review-input-${taskKey}.json`);
+}
 export function autoStateFile(taskKey) {
     return taskArtifactsFile(taskKey, `.agentweaver-state-${taskKey}.json`);
 }

package/dist/executors/configs/fetch-gitlab-review-config.js

@@ -0,0 +1,3 @@
+export const fetchGitLabReviewExecutorDefaultConfig = {
+    authEnvVar: "GITLAB_TOKEN",
+};

package/dist/executors/fetch-gitlab-review-executor.js

@@ -0,0 +1,25 @@
+import { fetchGitLabReviewExecutorDefaultConfig } from "./configs/fetch-gitlab-review-config.js";
+import { buildGitLabReviewFetchTarget, fetchGitLabReview } from "../gitlab.js";
+export const fetchGitLabReviewExecutor = {
+    kind: "fetch-gitlab-review",
+    version: 1,
+    defaultConfig: fetchGitLabReviewExecutorDefaultConfig,
+    async execute(context, input) {
+        const target = buildGitLabReviewFetchTarget(input.mergeRequestUrl);
+        if (context.verbose) {
+            context.ui.writeStdout(`GitLab MR URL: ${target.mergeRequestUrl}\n`);
+            context.ui.writeStdout(`GitLab project path: ${target.projectPath}\n`);
+            context.ui.writeStdout(`GitLab merge request IID: ${target.mergeRequestIid}\n`);
+            context.ui.writeStdout(`GitLab discussions API URL: ${target.discussionsApiUrl}\n`);
+            context.ui.writeStdout(`Saving GitLab review markdown to: ${input.outputFile}\n`);
+            context.ui.writeStdout(`Saving GitLab review JSON to: ${input.outputJsonFile}\n`);
+        }
+        const artifact = await fetchGitLabReview(input.mergeRequestUrl, input.outputFile, input.outputJsonFile);
+        return {
+            outputFile: input.outputFile,
+            outputJsonFile: input.outputJsonFile,
+            mergeRequestUrl: artifact.merge_request_url,
+            commentsCount: artifact.comments.length,
+        };
+    },
+};

package/dist/gitlab.js

@@ -0,0 +1,153 @@
+import { mkdirSync } from "node:fs";
+import { writeFile } from "node:fs/promises";
+import path from "node:path";
+import { TaskRunnerError } from "./errors.js";
+const MERGE_REQUEST_PATH_RE = /^(?<projectPath>.+?)\/-\/merge_requests\/(?<iid>\d+)(?:\/.*)?$/;
+function normalizeUrl(value) {
+    return value.trim().replace(/\/+$/, "");
+}
+function normalizeProjectPath(value) {
+    return value.replace(/^\/+/, "").replace(/\/+$/, "");
+}
+export function parseGitLabMergeRequestUrl(mergeRequestUrl) {
+    let parsed;
+    try {
+        parsed = new URL(normalizeUrl(mergeRequestUrl));
+    }
+    catch {
+        throw new TaskRunnerError("Expected GitLab merge request URL like https://gitlab.example.com/group/project/-/merge_requests/123");
+    }
+    const match = MERGE_REQUEST_PATH_RE.exec(parsed.pathname);
+    const projectPath = normalizeProjectPath(match?.groups?.projectPath ?? "");
+    const iidRaw = match?.groups?.iid;
+    if (!projectPath || !iidRaw) {
+        throw new TaskRunnerError("Expected GitLab merge request URL like https://gitlab.example.com/group/project/-/merge_requests/123");
+    }
+    return {
+        apiBaseUrl: `${parsed.protocol}//${parsed.host}/api/v4`,
+        mergeRequestUrl: `${parsed.protocol}//${parsed.host}${parsed.pathname}`,
+        projectPath,
+        mergeRequestIid: Number.parseInt(iidRaw, 10),
+    };
+}
+export function buildGitLabReviewFetchTarget(mergeRequestUrl) {
+    const mergeRequestRef = parseGitLabMergeRequestUrl(mergeRequestUrl);
+    return {
+        ...mergeRequestRef,
+        discussionsApiUrl: `${mergeRequestRef.apiBaseUrl}/projects/${encodeURIComponent(mergeRequestRef.projectPath)}/merge_requests/${mergeRequestRef.mergeRequestIid}/discussions`,
+    };
+}
+async function fetchDiscussionPage(target, page, token) {
+    const apiUrl = `${target.discussionsApiUrl}?per_page=100&page=${page}`;
+    const response = await fetch(apiUrl, {
+        headers: {
+            "PRIVATE-TOKEN": token,
+            Accept: "application/json",
+        },
+    });
+    if (!response.ok) {
+        throw new TaskRunnerError([
+            `Failed to fetch GitLab merge request discussions: HTTP ${response.status}`,
+            `MR URL: ${target.mergeRequestUrl}`,
+            `GitLab project path: ${target.projectPath}`,
+            `GitLab merge request IID: ${target.mergeRequestIid}`,
+            `GitLab discussions API URL: ${apiUrl}`,
+        ].join("\n"));
+    }
+    const nextPageHeader = response.headers.get("x-next-page");
+    const nextPage = nextPageHeader && nextPageHeader.trim().length > 0 ? Number.parseInt(nextPageHeader, 10) : null;
+    const discussions = (await response.json());
+    return { discussions, nextPage: Number.isNaN(nextPage ?? Number.NaN) ? null : nextPage };
+}
+async function fetchMergeRequestDiscussions(target, token) {
+    const discussions = [];
+    let page = 1;
+    while (true) {
+        const chunk = await fetchDiscussionPage(target, page, token);
+        discussions.push(...chunk.discussions);
+        if (!chunk.nextPage) {
+            return discussions;
+        }
+        page = chunk.nextPage;
+    }
+}
+function normalizeDiscussionNotes(discussions) {
+    return discussions.flatMap((discussion) => {
+        const discussionId = String(discussion.id ?? "");
+        if (!discussionId) {
+            return [];
+        }
+        return (discussion.notes ?? [])
+            .filter((note) => typeof note.body === "string" && note.body.trim().length > 0)
+            .filter((note) => note.system !== true)
+            .map((note) => ({
+            id: String(note.id ?? `${discussionId}-${note.created_at ?? "unknown"}`),
+            discussion_id: discussionId,
+            body: note.body?.trim() ?? "",
+            author: note.author?.username?.trim() || note.author?.name?.trim() || "unknown",
+            created_at: note.created_at ?? new Date(0).toISOString(),
+            system: Boolean(note.system),
+            resolvable: Boolean(note.resolvable),
+            resolved: Boolean(note.resolved),
+            file_path: note.position?.new_path ?? note.position?.old_path ?? null,
+            new_line: typeof note.position?.new_line === "number" ? note.position.new_line : null,
+            old_line: typeof note.position?.old_line === "number" ? note.position.old_line : null,
+        }));
+    });
+}
+function buildGitLabReviewMarkdown(artifact) {
+    const lines = [
+        "# GitLab Review",
+        "",
+        `- MR: ${artifact.merge_request_url}`,
+        `- Project: ${artifact.project_path}`,
+        `- IID: ${artifact.merge_request_iid}`,
+        `- Fetched at: ${artifact.fetched_at}`,
+        `- Comments: ${artifact.comments.length}`,
+        "",
+    ];
+    if (artifact.comments.length === 0) {
+        lines.push("Код-ревью комментариев не найдено.");
+        return lines.join("\n");
+    }
+    artifact.comments.forEach((comment, index) => {
+        lines.push(`## Comment ${index + 1}`);
+        lines.push(`- Author: ${comment.author}`);
+        lines.push(`- Created at: ${comment.created_at}`);
+        lines.push(`- Discussion: ${comment.discussion_id}`);
+        if (comment.file_path) {
+            const location = [comment.file_path, comment.new_line ?? comment.old_line].filter((item) => item !== null).join(":");
+            lines.push(`- Location: ${location}`);
+        }
+        if (comment.resolvable) {
+            lines.push(`- Resolved: ${comment.resolved ? "yes" : "no"}`);
+        }
+        lines.push("");
+        lines.push(comment.body);
+        lines.push("");
+    });
+    return lines.join("\n");
+}
+export async function fetchGitLabReview(mergeRequestUrl, outputFile, outputJsonFile) {
+    const token = process.env.GITLAB_TOKEN?.trim();
+    if (!token) {
+        throw new TaskRunnerError("GITLAB_TOKEN is required for gitlab-review flow.");
+    }
+    const target = buildGitLabReviewFetchTarget(mergeRequestUrl);
+    const discussions = await fetchMergeRequestDiscussions(target, token);
+    const comments = normalizeDiscussionNotes(discussions);
+    const fetchedAt = new Date().toISOString();
+    const artifact = {
+        summary: comments.length > 0 ? `Fetched ${comments.length} GitLab review comments.` : "No GitLab review comments found.",
+        merge_request_url: target.mergeRequestUrl,
+        project_path: target.projectPath,
+        merge_request_iid: target.mergeRequestIid,
+        fetched_at: fetchedAt,
+        comments,
+    };
+    mkdirSync(path.dirname(outputFile), { recursive: true });
+    mkdirSync(path.dirname(outputJsonFile), { recursive: true });
+    await writeFile(outputJsonFile, `${JSON.stringify(artifact, null, 2)}\n`, "utf8");
+    await writeFile(outputFile, `${buildGitLabReviewMarkdown(artifact)}\n`, "utf8");
+    return artifact;
+}

package/dist/index.js

@@ -3,7 +3,7 @@ import { existsSync, readdirSync, readFileSync, rmSync, writeFileSync } from "no
 import path from "node:path";
 import process from "node:process";
 import { fileURLToPath } from "node:url";
-import { REVIEW_FILE_RE, REVIEW_REPLY_FILE_RE, autoStateFile, bugAnalyzeArtifacts, bugAnalyzeJsonFile, bugFixDesignJsonFile, bugFixPlanJsonFile, designJsonFile, ensureTaskWorkspaceDir, jiraTaskFile, planJsonFile, planArtifacts, qaJsonFile, readyToMergeFile, requireArtifacts, reviewReplyJsonFile, reviewFixSelectionJsonFile, reviewJsonFile, taskWorkspaceDir, taskSummaryFile, } from "./artifacts.js";
+import { REVIEW_FILE_RE, REVIEW_REPLY_FILE_RE, autoStateFile, bugAnalyzeArtifacts, bugAnalyzeJsonFile, bugFixDesignJsonFile, bugFixPlanJsonFile, designJsonFile, ensureTaskWorkspaceDir, gitlabReviewFile, gitlabReviewJsonFile, jiraTaskFile, planJsonFile, planArtifacts, qaJsonFile, readyToMergeFile, requireArtifacts, reviewReplyJsonFile, reviewFixSelectionJsonFile, reviewJsonFile, taskWorkspaceDir, taskSummaryFile, } from "./artifacts.js";
 import { TaskRunnerError } from "./errors.js";
 import { buildJiraApiUrl, buildJiraBrowseUrl, extractIssueKey, requireJiraTaskFile } from "./jira.js";
 import { validateStructuredArtifacts } from "./structured-artifacts.js";
@@ -14,7 +14,7 @@ import { loadDeclarativeFlow } from "./pipeline/declarative-flows.js";
 import { findPhaseById, runExpandedPhase } from "./pipeline/declarative-flow-runner.js";
 import { runPreflightFlow } from "./pipeline/flows/preflight-flow.js";
 import { resolveCmd, resolveDockerComposeCmd } from "./runtime/command-resolution.js";
-import { defaultDockerComposeFile, dockerRuntimeEnv } from "./runtime/docker-runtime.js";
+import { agentweaverHome, defaultDockerComposeFile, dockerRuntimeEnv } from "./runtime/docker-runtime.js";
 import { runCommand } from "./runtime/process-runner.js";
 import { InteractiveUi } from "./interactive-ui.js";
 import { bye, printError, printInfo, printPanel, printSummary, setFlowExecutionState, stripAnsi, } from "./tui.js";
@@ -22,15 +22,13 @@ import { requestUserInputInTerminal } from "./user-input.js";
 const COMMANDS = [
     "bug-analyze",
     "bug-fix",
+    "gitlab-review",
     "mr-description",
     "plan",
     "task-describe",
     "implement",
     "review",
     "review-fix",
-    "test",
-    "test-fix",
-    "test-linter-fix",
     "run-tests-loop",
     "run-linter-loop",
     "auto",
@@ -81,6 +79,7 @@ function usage() {
     return `Usage:
   agentweaver <jira-browse-url|jira-issue-key>
   agentweaver --force <jira-browse-url|jira-issue-key>
+  agentweaver gitlab-review <jira-browse-url|jira-issue-key>
   agentweaver bug-analyze [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
   agentweaver bug-fix [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
   agentweaver mr-description [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
@@ -89,9 +88,6 @@ function usage() {
   agentweaver implement [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
   agentweaver review [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
   agentweaver review-fix [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
-  agentweaver test [--dry] [--verbose] <jira-browse-url|jira-issue-key>
-  agentweaver test-fix [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
-  agentweaver test-linter-fix [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
   agentweaver run-tests-loop [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
   agentweaver run-linter-loop [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
   agentweaver auto [--dry] [--verbose] [--prompt <text>] <jira-browse-url|jira-issue-key>
@@ -116,6 +112,7 @@ Required environment variables:
 
 Optional environment variables:
   JIRA_BASE_URL
+  GITLAB_TOKEN
   AGENTWEAVER_HOME
   DOCKER_COMPOSE_BIN
   CODEX_BIN
@@ -398,6 +395,7 @@ function latestReviewReplyIteration(taskKey) {
 function buildConfig(command, jiraRef, options = {}) {
     const jiraIssueKey = extractIssueKey(jiraRef);
     ensureTaskWorkspaceDir(jiraIssueKey);
+    const homeDir = agentweaverHome(PACKAGE_ROOT);
     return {
         command,
         jiraRef,
@@ -412,6 +410,8 @@ function buildConfig(command, jiraRef, options = {}) {
         jiraBrowseUrl: buildJiraBrowseUrl(jiraRef),
         jiraApiUrl: buildJiraApiUrl(jiraRef),
         jiraTaskFile: jiraTaskFile(jiraIssueKey),
+        runTestsScript: path.join(homeDir, "run_tests.sh"),
+        runLinterScript: path.join(homeDir, "run_linter.sh"),
     };
 }
 function checkPrerequisites(config) {
@@ -428,26 +428,18 @@ function checkPrerequisites(config) {
     if (config.command === "review") {
         resolveCmd("claude", "CLAUDE_BIN");
     }
-    if (["implement", "review-fix", "test", "run-tests-loop", "run-linter-loop"].includes(config.command)) {
-        resolveDockerComposeCmd();
-        if (!existsSync(config.dockerComposeFile)) {
-            throw new TaskRunnerError(`docker-compose file not found: ${config.dockerComposeFile}`);
-        }
-    }
 }
 function checkAutoPrerequisites(config) {
     resolveCmd("codex", "CODEX_BIN");
     resolveCmd("claude", "CLAUDE_BIN");
-    resolveDockerComposeCmd();
-    if (!existsSync(config.dockerComposeFile)) {
-        throw new TaskRunnerError(`docker-compose file not found: ${config.dockerComposeFile}`);
-    }
 }
 function autoFlowParams(config) {
     return {
         jiraApiUrl: config.jiraApiUrl,
         taskKey: config.taskKey,
         dockerComposeFile: config.dockerComposeFile,
+        runTestsScript: config.runTestsScript,
+        runLinterScript: config.runLinterScript,
         extraPrompt: config.extraPrompt,
         reviewFixPoints: config.reviewFixPoints,
     };
@@ -455,6 +447,7 @@ function autoFlowParams(config) {
 const FLOW_DESCRIPTIONS = {
     auto: "Полный пайплайн задачи: планирование, реализация, проверки, ревью, ответы на ревью и повторные итерации до готовности к merge.",
     "bug-analyze": "Анализирует баг по Jira и создаёт структурированные артефакты: гипотезу причины, дизайн исправления и план работ.",
+    "gitlab-review": "Запрашивает GitLab MR URL через user-input, загружает комментарии код-ревью по API и сохраняет markdown плюс structured JSON artifact.",
     "bug-fix": "Берёт результаты bug-analyze как source of truth и реализует исправление бага в коде.",
     "mr-description": "Готовит краткое intent-описание для merge request на основе задачи и текущих изменений.",
     plan: "Загружает задачу из Jira и создаёт дизайн, план реализации и QA-план в structured JSON и markdown.",
@@ -462,11 +455,8 @@ const FLOW_DESCRIPTIONS = {
     implement: "Реализует задачу по утверждённым design/plan артефактам и при необходимости запускает post-verify сборки.",
     review: "Запускает Claude-код-ревью текущих изменений, валидирует structured findings, затем готовит ответ на замечания через Codex.",
     "review-fix": "Исправляет замечания после review-reply, обновляет код и прогоняет обязательные проверки после правок.",
-    test: "Запускает verify/build-проверку в контейнере и показывает результат с краткой сводкой при падении.",
-    "test-fix": "Прогоняет тесты, исправляет найденные проблемы и готовит код к следующей успешной проверке.",
-    "test-linter-fix": "Прогоняет линтер и генерацию, затем исправляет замечания для чистого прогона.",
-    "run-tests-loop": "Циклически запускает `./run_tests.sh`, анализирует последнюю ошибку и правит код до успешного прохождения или исчерпания попыток.",
-    "run-linter-loop": "Циклически запускает `./run_linter.sh`, исправляет проблемы линтера или генерации и повторяет попытки до успеха.",
+    "run-tests-loop": "Циклически запускает `./run_tests.sh` локально, анализирует последнюю ошибку и правит код до успешного прохождения или исчерпания попыток.",
+    "run-linter-loop": "Циклически запускает `./run_linter.sh` локально, исправляет проблемы линтера или генерации и повторяет попытки до успеха.",
 };
 function flowDescription(id) {
     return FLOW_DESCRIPTIONS[id] ?? "Описание для этого flow пока не задано.";
@@ -506,15 +496,13 @@ function interactiveFlowDefinitions() {
         autoFlowDefinition(),
         declarativeFlowDefinition("bug-analyze", "bug-analyze", "bug-analyze.json"),
         declarativeFlowDefinition("bug-fix", "bug-fix", "bug-fix.json"),
+        declarativeFlowDefinition("gitlab-review", "gitlab-review", "gitlab-review.json"),
         declarativeFlowDefinition("mr-description", "mr-description", "mr-description.json"),
         declarativeFlowDefinition("plan", "plan", "plan.json"),
         declarativeFlowDefinition("task-describe", "task-describe", "task-describe.json"),
         declarativeFlowDefinition("implement", "implement", "implement.json"),
         declarativeFlowDefinition("review", "review", "review.json"),
         declarativeFlowDefinition("review-fix", "review-fix", "review-fix.json"),
-        declarativeFlowDefinition("test", "test", "test.json"),
-        declarativeFlowDefinition("test-fix", "test-fix", "test-fix.json"),
-        declarativeFlowDefinition("test-linter-fix", "test-linter-fix", "test-linter-fix.json"),
         declarativeFlowDefinition("run-tests-loop", "run-tests-loop", "run-tests-loop.json"),
         declarativeFlowDefinition("run-linter-loop", "run-linter-loop", "run-linter-loop.json"),
     ];
@@ -684,6 +672,23 @@ async function executeCommand(config, runFollowupVerify = true, requestUserInput
         }, requestUserInput);
         return false;
     }
+    if (config.command === "gitlab-review") {
+        requireJiraTaskFile(config.jiraTaskFile);
+        validateStructuredArtifacts([
+            { path: designJsonFile(config.taskKey), schemaId: "implementation-design/v1" },
+            { path: planJsonFile(config.taskKey), schemaId: "implementation-plan/v1" },
+        ], "GitLab-review mode requires valid structured plan artifacts from the planning phase.");
+        const iteration = nextReviewIterationForTask(config.taskKey);
+        await runDeclarativeFlowBySpecFile("gitlab-review.json", config, {
+            taskKey: config.taskKey,
+            iteration,
+            extraPrompt: config.extraPrompt,
+        }, requestUserInput);
+        if (!config.dryRun) {
+            printSummary("GitLab Review", `Artifacts:\n${gitlabReviewFile(config.taskKey)}\n${gitlabReviewJsonFile(config.taskKey)}`);
+        }
+        return false;
+    }
     if (config.command === "bug-fix") {
         requireJiraTaskFile(config.jiraTaskFile);
         requireArtifacts(bugAnalyzeArtifacts(config.taskKey), "Bug-fix mode requires bug-analyze artifacts from the bug analysis phase.");
@@ -722,24 +727,10 @@ async function executeCommand(config, runFollowupVerify = true, requestUserInput
             { path: planJsonFile(config.taskKey), schemaId: "implementation-plan/v1" },
             { path: qaJsonFile(config.taskKey), schemaId: "qa-plan/v1" },
         ], "Implement mode requires valid structured plan artifacts from the planning phase.");
-        try {
-            await runDeclarativeFlowBySpecFile("implement.json", config, {
-                taskKey: config.taskKey,
-                dockerComposeFile: config.dockerComposeFile,
-                extraPrompt: config.extraPrompt,
-                runFollowupVerify,
-            }, requestUserInput);
-        }
-        catch (error) {
-            if (!config.dryRun) {
-                const output = String(error.output ?? "");
-                if (output.trim()) {
-                    printError("Build verification failed");
-                    printSummary("Build Failure Summary", await summarizeBuildFailure(output));
-                }
-            }
-            throw error;
-        }
+        await runDeclarativeFlowBySpecFile("implement.json", config, {
+            taskKey: config.taskKey,
+            extraPrompt: config.extraPrompt,
+        }, requestUserInput);
         return false;
     }
     if (config.command === "review") {
@@ -766,61 +757,20 @@ async function executeCommand(config, runFollowupVerify = true, requestUserInput
             { path: reviewJsonFile(config.taskKey, latestIteration), schemaId: "review-findings/v1" },
             { path: reviewReplyJsonFile(config.taskKey, latestIteration), schemaId: "review-reply/v1" },
         ], "Review-fix mode requires valid structured review artifacts.");
-        try {
-            await runDeclarativeFlowBySpecFile("review-fix.json", config, {
-                taskKey: config.taskKey,
-                dockerComposeFile: config.dockerComposeFile,
-                latestIteration,
-                reviewFixSelectionJsonFile: reviewFixSelectionJsonFile(config.taskKey, latestIteration),
-                runFollowupVerify,
-                extraPrompt: config.extraPrompt,
-                reviewFixPoints: config.reviewFixPoints,
-            }, requestUserInput);
-        }
-        catch (error) {
-            if (!config.dryRun) {
-                const output = String(error.output ?? "");
-                if (output.trim()) {
-                    printError("Build verification failed");
-                    printSummary("Build Failure Summary", await summarizeBuildFailure(output));
-                }
-            }
-            throw error;
-        }
-        return false;
-    }
-    if (config.command === "test") {
-        requireJiraTaskFile(config.jiraTaskFile);
-        try {
-            await runDeclarativeFlowBySpecFile("test.json", config, {
-                taskKey: config.taskKey,
-                dockerComposeFile: config.dockerComposeFile,
-            }, requestUserInput);
-        }
-        catch (error) {
-            if (!config.dryRun) {
-                const output = String(error.output ?? "");
-                if (output.trim()) {
-                    printError("Build verification failed");
-                    printSummary("Build Failure Summary", await summarizeBuildFailure(output));
-                }
-            }
-            throw error;
-        }
-        return false;
-    }
-    if (config.command === "test-fix" || config.command === "test-linter-fix") {
-        requireJiraTaskFile(config.jiraTaskFile);
-        await runDeclarativeFlowBySpecFile(config.command === "test-fix" ? "test-fix.json" : "test-linter-fix.json", config, {
+        await runDeclarativeFlowBySpecFile("review-fix.json", config, {
             taskKey: config.taskKey,
+            latestIteration,
+            reviewFixSelectionJsonFile: reviewFixSelectionJsonFile(config.taskKey, latestIteration),
             extraPrompt: config.extraPrompt,
+            reviewFixPoints: config.reviewFixPoints,
         }, requestUserInput);
         return false;
     }
     if (config.command === "run-tests-loop" || config.command === "run-linter-loop") {
         await runDeclarativeFlowBySpecFile(config.command === "run-tests-loop" ? "run-tests-loop.json" : "run-linter-loop.json", config, {
             taskKey: config.taskKey,
-            dockerComposeFile: config.dockerComposeFile,
+            runTestsScript: config.runTestsScript,
+            runLinterScript: config.runLinterScript,
             extraPrompt: config.extraPrompt,
         }, requestUserInput);
         return false;

package/dist/pipeline/flow-specs/auto.json

@@ -193,7 +193,7 @@
             "fileName": { "const": "run-linter-loop.json" },
             "labelText": { "const": "Running run-linter-loop after implement" },
             "taskKey": { "ref": "params.taskKey" },
-            "dockerComposeFile": { "ref": "params.dockerComposeFile" },
+            "runLinterScript": { "ref": "params.runLinterScript" },
             "extraPrompt": { "ref": "params.extraPrompt" }
           }
         },
@@ -204,7 +204,7 @@
             "fileName": { "const": "run-tests-loop.json" },
             "labelText": { "const": "Running run-tests-loop after implement" },
             "taskKey": { "ref": "params.taskKey" },
-            "dockerComposeFile": { "ref": "params.dockerComposeFile" },
+            "runTestsScript": { "ref": "params.runTestsScript" },
             "extraPrompt": { "ref": "params.extraPrompt" }
           }
         }