Merge pull request #172 from rostilos/1.5.6-rc

1.5.6 rc

Author: rostilos

README.md

@@ -34,6 +34,7 @@ CodeCrow supports multiple version control systems. The AI analysis engine is th
 | `/ask <question>` |    ✅     |   ✅   |   ✅   |
 | `/analyze`        |    ✅     |   ✅   |   ✅   |
 | `/summarize`      |    ✅     |   ✅   |   ✅   |
+| `/qa-doc`         |    ✅     |   ✅   |   ✅   |
 
 ### Dashboard & Issue Management
 
@@ -50,6 +51,8 @@ These features are platform-independent and available through the CodeCrow web U
 | Project Analytics           | Aggregated severity breakdown, analysis history, and branch health             |
 | AI Model Selection          | Choose your LLM provider and model (OpenRouter, Anthropic, Google, OpenAI)     |
 | Workspace & Team Management | Roles (Owner, Admin, Member, Viewer), member invites, ownership transfer       |
+| Task Management (Jira)      | Connect Jira Cloud to link PRs with tasks for QA documentation                 |
+| QA Auto-Documentation       | AI-generated QA docs posted as Jira comments after each analysis               |
 | Two-Factor Authentication   | TOTP-based 2FA for sensitive operations                                        |
 
 ### Setup Methods
@@ -105,7 +108,8 @@ The RAG pipeline (codebase indexing for context-aware reviews) provides enhanced
 - **Context-Aware Reviews**: Powered by a custom RAG (Retrieval-Augmented Generation) pipeline using Qdrant vector storage.
 - **Incremental Analysis**: Only scans changed code to keep feedback fast and cost-efficient.
 - **Multi-Tenant Architecture**: Securely manage multiple teams and projects from a single dashboard.
-- **Interactive Commands**: Command CodeCrow directly from PR comments using `/ask`, `/analyze`, and `/summarize`.
+- **Interactive Commands**: Command CodeCrow directly from PR comments using `/ask`, `/analyze`, `/summarize`, and `/qa-doc`.
+- **QA Auto-Documentation**: Automatically generate QA testing documentation from PR analysis and post it to linked Jira tickets. Task IDs are auto-detected from branch names, PR titles, or PR descriptions — or you can specify one explicitly with `/qa-doc PROJ-123`.
 - **Issue Lifecycle**: Automatic tracking of resolved vs. open issues across analyses with deterministic and AI-based reconciliation.
 - **Bring Your Own Model**: Connect your preferred LLM provider — OpenRouter, Anthropic, Google, or OpenAI.
 

deployment/docker-compose.yml

@@ -24,6 +24,26 @@ services:
       retries: 5
       start_period: 30s
 
+  pgadmin:
+    image: dpage/pgadmin4:9
+    container_name: codecrow-pgadmin
+    environment:
+      PGADMIN_DEFAULT_EMAIL: ${PGADMIN_DEFAULT_EMAIL:-pgadmin@localhost}
+      PGADMIN_DEFAULT_PASSWORD: ${PGADMIN_DEFAULT_PASSWORD:?PGADMIN_DEFAULT_PASSWORD must be set in .env}
+      PGADMIN_CONFIG_MASTER_PASSWORD_REQUIRED: "True"
+      PGADMIN_DISABLE_POSTFIX: "True"
+      PGADMIN_LISTEN_PORT: 80
+    ports:
+      - "127.0.0.1:5050:80"
+    depends_on:
+      postgres:
+        condition: service_healthy
+    volumes:
+      - pgadmin_data:/var/lib/pgadmin
+    networks:
+      - codecrow-network
+    restart: unless-stopped
+
   redis:
     image: redis:7-alpine
     container_name: codecrow-redis
@@ -207,8 +227,10 @@ services:
       CODECROW_INTERNAL_SECRET: ${INTERNAL_API_SECRET:?INTERNAL_API_SECRET must be set in .env}
       QDRANT_API_KEY: ${QDRANT_API_KEY:?QDRANT_API_KEY must be set in .env}
       REDIS_URL: redis://redis:6379/1
+      #UVICORN_WORKERS: 1
     ports:
       - "127.0.0.1:8001:8001"
+      #- "127.0.0.1:5678:5678"
     depends_on:
       redis:
         condition: service_healthy
@@ -220,6 +242,7 @@ services:
         condition: service_healthy
     networks:
       - codecrow-network
+    #command: ["python", "-m", "debugpy", "--listen", "0.0.0.0:5678", "main.py"]
     volumes:
       - source_code_tmp:/tmp
       - rag_logs:/app/logs
@@ -274,6 +297,9 @@ volumes:
   qdrant_data:
     name: qdrant_data
     driver: local
+  pgadmin_data:
+    name: pgadmin_data
+    driver: local
   web_logs:
     name: web_logs
     driver: local

frontend

@@ -149,15 +149,21 @@ public static int mapLineNumber(int oldLine, List<int[]> hunks, String fileDiff)
      * Walk a hunk body line-by-line to map an old line number to its new position.
      * <p>
      * Context lines advance both counters; {@code -} lines advance only old;
-     * {@code +} lines advance only new.  If the target old line was deleted,
-     * returns the original line number unchanged.
+     * {@code +} lines advance only new.
+     * <p>
+     * If the target old line was <b>deleted</b>, returns the new-file line number
+     * of the next surviving (context or added) line after the deletion block.
+     * This anchors the issue to the nearest relevant context in the new file
+     * rather than leaving it at a stale old position.
+     * Falls back to {@code hunkNewStart} if no surviving line is found.
      */
     static int mapLineInsideHunk(int targetOldLine, int hunkOldStart,
                                  int hunkNewStart, String fileDiff) {
         String[] lines = fileDiff.split("\\r?\\n");
         int oldCursor = hunkOldStart;
         int newCursor = hunkNewStart;
         boolean inTargetHunk = false;
+        boolean targetWasDeleted = false;
 
         for (String line : lines) {
             Matcher hm = HUNK_HEADER.matcher(line);
@@ -178,19 +184,39 @@ static int mapLineInsideHunk(int targetOldLine, int hunkOldStart,
 
             if (line.startsWith("-")) {
                 if (oldCursor == targetOldLine) {
-                    return targetOldLine;
+                    // Target line was deleted — don't return the old number.
+                    // Continue scanning to find the next surviving line
+                    // (context or "+" line) and return its new position.
+                    targetWasDeleted = true;
                 }
                 oldCursor++;
             } else if (line.startsWith("+")) {
+                if (targetWasDeleted) {
+                    // First added/surviving line after the deletion — anchor here
+                    return newCursor;
+                }
                 newCursor++;
             } else {
+                // Context line (unchanged)
+                if (targetWasDeleted) {
+                    // First surviving context line after the deletion — anchor here
+                    return newCursor;
+                }
                 if (oldCursor == targetOldLine) {
                     return newCursor;
                 }
                 oldCursor++;
                 newCursor++;
             }
         }
+
+        // If we flagged the line as deleted but found no surviving line after it
+        // (e.g., deletion was at the very end of the hunk), return the current
+        // new-cursor position which points to the end of the hunk in the new file.
+        if (targetWasDeleted) {
+            return newCursor > hunkNewStart ? newCursor - 1 : hunkNewStart;
+        }
+
         return targetOldLine;
     }
 }

java-ecosystem/libs/analysis-engine/src/main/java/org/rostilos/codecrow/analysisengine/util/DiffParsingUtils.java

@@ -140,4 +140,12 @@
 
     opens org.rostilos.codecrow.core.model.reconcile
             to org.hibernate.orm.core, spring.beans, spring.context, spring.core;
+
+    // Task-management integration (QA Auto-Documentation)
+    exports org.rostilos.codecrow.core.model.taskmanagement;
+    exports org.rostilos.codecrow.core.persistence.repository.taskmanagement;
+    exports org.rostilos.codecrow.core.dto.taskmanagement;
+
+    opens org.rostilos.codecrow.core.model.taskmanagement
+            to org.hibernate.orm.core, spring.beans, spring.context, spring.core;
 }

java-ecosystem/libs/core/src/main/java/module-info.java

@@ -5,6 +5,7 @@
 import org.rostilos.codecrow.core.model.project.config.CommentCommandsConfig;
 import org.rostilos.codecrow.core.model.project.config.ProjectConfig;
 import org.rostilos.codecrow.core.model.project.config.ProjectRulesConfig;
+import org.rostilos.codecrow.core.model.project.config.QaAutoDocConfig;
 import org.rostilos.codecrow.core.model.project.config.RagConfig;
 import org.rostilos.codecrow.core.model.vcs.VcsConnection;
 import org.rostilos.codecrow.core.model.vcs.VcsRepoInfo;
@@ -36,7 +37,8 @@ public record ProjectDTO(
         Long qualityGateId,
         Integer maxAnalysisTokenLimit,
         Boolean useMcpTools,
-        ProjectRulesConfigDTO projectRulesConfig) {
+        ProjectRulesConfigDTO projectRulesConfig,
+        QaAutoDocConfigDTO qaAutoDocConfig) {
     public static ProjectDTO fromProject(Project project) {
         Long vcsConnectionId = null;
         String vcsConnectionType = null;
@@ -139,6 +141,12 @@ public static ProjectDTO fromProject(Project project) {
             projectRulesConfigDTO = ProjectRulesConfigDTO.fromConfig(config.projectRules());
         }
 
+        // Get QA auto-doc config
+        QaAutoDocConfigDTO qaAutoDocConfigDTO = null;
+        if (config != null && config.qaAutoDoc() != null) {
+            qaAutoDocConfigDTO = QaAutoDocConfigDTO.fromConfig(config.qaAutoDoc());
+        }
+
         return new ProjectDTO(
                 project.getId(),
                 project.getName(),
@@ -164,7 +172,8 @@ public static ProjectDTO fromProject(Project project) {
                 project.getQualityGate() != null ? project.getQualityGate().getId() : null,
                 maxAnalysisTokenLimit,
                 useMcpTools,
-                projectRulesConfigDTO);
+                projectRulesConfigDTO,
+                qaAutoDocConfigDTO);
     }
 
     public record DefaultBranchStats(
@@ -252,4 +261,30 @@ public record CustomRuleDTO(
             boolean enabled,
             int priority
     ) {}
+
+    /**
+     * DTO for QA auto-documentation configuration.
+     */
+    public record QaAutoDocConfigDTO(
+            boolean enabled,
+            Long taskManagementConnectionId,
+            String taskIdPattern,
+            String taskIdSource,
+            String templateMode,
+            String customTemplate
+    ) {
+        public static QaAutoDocConfigDTO fromConfig(QaAutoDocConfig config) {
+            if (config == null) {
+                return new QaAutoDocConfigDTO(false, null, null, null, null, null);
+            }
+            return new QaAutoDocConfigDTO(
+                    config.enabled(),
+                    config.taskManagementConnectionId(),
+                    config.taskIdPattern(),
+                    config.taskIdSource() != null ? config.taskIdSource().name() : null,
+                    config.templateMode() != null ? config.templateMode().name() : null,
+                    config.customTemplate()
+            );
+        }
+    }
 }

java-ecosystem/libs/core/src/main/java/org/rostilos/codecrow/core/dto/project/ProjectDTO.java

@@ -0,0 +1,29 @@
+package org.rostilos.codecrow.core.dto.taskmanagement;
+
+import jakarta.validation.constraints.NotNull;
+import jakarta.validation.constraints.Size;
+
+/**
+ * Request DTO for updating QA auto-documentation configuration on a project.
+ */
+public record QaAutoDocConfigRequest(
+        @NotNull(message = "Enabled flag is required")
+        Boolean enabled,
+
+        Long taskManagementConnectionId,
+
+        /** Regex pattern for extracting task ID from PR metadata */
+        @Size(max = 256, message = "Task ID pattern must be at most 256 characters")
+        String taskIdPattern,
+
+        /** Where to look for the task ID: BRANCH_NAME, PR_TITLE, PR_DESCRIPTION */
+        String taskIdSource,
+
+        /** Template mode: RAW, BASE, CUSTOM */
+        String templateMode,
+
+        /** Custom template text (only for CUSTOM mode, max 5000 chars) */
+        @Size(max = 5000, message = "Custom template must be at most 5000 characters")
+        String customTemplate
+) {
+}

java-ecosystem/libs/core/src/main/java/org/rostilos/codecrow/core/dto/taskmanagement/QaAutoDocConfigRequest.java

@@ -0,0 +1,35 @@
+package org.rostilos.codecrow.core.dto.taskmanagement;
+
+import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.NotNull;
+import jakarta.validation.constraints.Size;
+
+/**
+ * Request DTO for creating or updating a task management connection.
+ * <p>
+ * Provider-specific credential requirements are validated at the service layer,
+ * since different providers need different credential fields:
+ * <ul>
+ *   <li>Jira Cloud: {@code email} + {@code apiToken}</li>
+ *   <li>Jira Data Center (future): {@code apiToken} only (Personal Access Token)</li>
+ * </ul>
+ */
+public record TaskManagementConnectionRequest(
+        @NotBlank(message = "Connection name is required")
+        @Size(max = 256, message = "Connection name must be at most 256 characters")
+        String connectionName,
+
+        @NotNull(message = "Provider type is required")
+        String providerType,
+
+        @NotBlank(message = "Base URL is required")
+        @Size(max = 512, message = "Base URL must be at most 512 characters")
+        String baseUrl,
+
+        /** Required for Jira Cloud; may be null for other providers. */
+        String email,
+
+        /** API token (Jira Cloud) or Personal Access Token (Jira Data Center). */
+        String apiToken
+) {
+}

java-ecosystem/libs/core/src/main/java/org/rostilos/codecrow/core/dto/taskmanagement/TaskManagementConnectionRequest.java

@@ -0,0 +1,20 @@
+package org.rostilos.codecrow.core.dto.taskmanagement;
+
+import java.time.OffsetDateTime;
+
+/**
+ * Response DTO for task management connection details.
+ * Excludes sensitive credential data.
+ */
+public record TaskManagementConnectionResponse(
+        Long id,
+        String connectionName,
+        String providerType,
+        String status,
+        String baseUrl,
+        /** Masked email (e.g. "j***@example.com") — never returns raw credentials */
+        String maskedEmail,
+        OffsetDateTime createdAt,
+        OffsetDateTime updatedAt
+) {
+}

java-ecosystem/libs/core/src/main/java/org/rostilos/codecrow/core/dto/taskmanagement/TaskManagementConnectionResponse.java

@@ -13,6 +13,7 @@ public enum JobType {
     ASK_COMMAND,
     ANALYZE_COMMAND,
     REVIEW_COMMAND,
+    QA_DOC_COMMAND,
     // Ignored comment events (not CodeCrow commands)
     IGNORED_COMMENT
 }