feat(x2a): add collapsible details to the project list (#2253)

* feat(x2a): add collapsible details to the project list

Signed-off-by: Marek Libra <marek.libra@gmail.com>

* review

---------

Signed-off-by: Marek Libra <marek.libra@gmail.com>

Author: mareklibra

workspaces/x2a/.changeset/floppy-seals-take.md

@@ -0,0 +1,8 @@
+---
+'@red-hat-developer-hub/backstage-plugin-scaffolder-backend-module-x2a': patch
+'@red-hat-developer-hub/backstage-plugin-x2a-backend': patch
+'@red-hat-developer-hub/backstage-plugin-x2a-common': patch
+'@red-hat-developer-hub/backstage-plugin-x2a': patch
+---
+
+Adding collapsible row detail to the Project List.

workspaces/x2a/plugins/x2a-backend/src/router.ts

@@ -395,6 +395,7 @@ export async function createRouter({
     const lastAnalyzeJobsOfModules = await Promise.all(
       modules.map(module =>
         x2aDatabase.listJobs({
+          projectId,
           moduleId: module.id,
           phase: 'analyze',
           lastJobOnly: true,
@@ -404,6 +405,7 @@ export async function createRouter({
     const lastMigrateJobsOfModules = await Promise.all(
       modules.map(module =>
         x2aDatabase.listJobs({
+          projectId,
           moduleId: module.id,
           phase: 'migrate',
           lastJobOnly: true,
@@ -413,6 +415,7 @@ export async function createRouter({
     const lastPublishJobsOfModules = await Promise.all(
       modules.map(module =>
         x2aDatabase.listJobs({
+          projectId,
           moduleId: module.id,
           phase: 'publish',
           lastJobOnly: true,
@@ -492,15 +495,16 @@ export async function createRouter({
     async (req: express.Request, res: express.Response) => {
       const endpoint = 'POST /projects/:projectId/modules/:moduleId/run';
       const { projectId, moduleId } = req.params;
-      logger.info(
-        `${endpoint} request received: projectId=${projectId}, moduleId=${moduleId}`,
-      );
 
       // Validate request body
       const runModuleRequestSchema = z.object({
         phase: z.enum(['analyze', 'migrate', 'publish']),
-        sourceRepoToken: z.string(),
-        targetRepoToken: z.string(),
+        sourceRepoAuth: z.object({
+          token: z.string(),
+        }),
+        targetRepoAuth: z.object({
+          token: z.string(),
+        }),
         aapCredentials: z
           .object({
             url: z.string(),
@@ -521,8 +525,8 @@ export async function createRouter({
       }
       const {
         phase,
-        sourceRepoToken,
-        targetRepoToken,
+        sourceRepoAuth,
+        targetRepoAuth,
         aapCredentials,
         userPrompt,
       } = parsedBody.data;
@@ -575,12 +579,12 @@ export async function createRouter({
         sourceRepo: {
           url: project.sourceRepoUrl,
           branch: project.sourceRepoBranch,
-          token: sourceRepoToken,
+          token: sourceRepoAuth.token,
         },
         targetRepo: {
           url: project.targetRepoUrl,
           branch: project.targetRepoBranch,
-          token: targetRepoToken,
+          token: targetRepoAuth.token,
         },
         aapCredentials,
         userPrompt,
@@ -646,6 +650,7 @@ export async function createRouter({
 
     // Get latest job for module filtered by requested phase
     const jobs = await x2aDatabase.listJobs({
+      projectId,
       moduleId,
       phase,
     });

workspaces/x2a/plugins/x2a-backend/src/schema/openapi.yaml

@@ -297,8 +297,8 @@ paths:
                   description: Optional user prompt for customizing the migration
               required:
                 - phase
-                - sourceRepo
-                - targetRepo
+                - sourceRepoAuth
+                - targetRepoAuth
       responses:
         '200':
           description: Migration job created successfully
@@ -394,6 +394,9 @@ components:
         createdBy:
           type: string
           description: The user who created the project (Backstage user reference)
+        migrationPlan:
+          $ref: '#/components/schemas/Artifact'
+          description: Project migration plan artifact (by init phase)
       required:
         - id
         - name
@@ -486,14 +489,21 @@ components:
           items:
             $ref: '#/components/schemas/Artifact'
 
+    ArtifactType:
+      type: string
+      enum:
+        - migration_plan
+        - module_migration_plan
+        - migrated_sources
+
     Artifact:
       type: object
       properties:
         id:
           type: string
           description: UUID for the artifact
         type:
-          type: string
+          $ref: '#/components/schemas/ArtifactType'
           description: Type of the artifact
         value:
           type: string

workspaces/x2a/plugins/x2a-backend/src/schema/openapi/generated/models/Artifact.model.ts

@@ -17,6 +17,7 @@
 // ******************************************************************
 // * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY. *
 // ******************************************************************
+import { ArtifactType } from '../models/ArtifactType.model';
 
 /**
  * @public
@@ -26,10 +27,7 @@ export interface Artifact {
    * UUID for the artifact
    */
   id: string;
-  /**
-   * Type of the artifact
-   */
-  type: string;
+  type: ArtifactType;
   /**
    * Value of the artifact
    */

workspaces/x2a/plugins/x2a-backend/src/schema/openapi/generated/models/ArtifactType.model.ts

@@ -0,0 +1,27 @@
+/*
+ * Copyright Red Hat, Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+// ******************************************************************
+// * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY. *
+// ******************************************************************
+
+/**
+ * @public
+ */
+export type ArtifactType =
+  | 'migration_plan'
+  | 'module_migration_plan'
+  | 'migrated_sources';

workspaces/x2a/plugins/x2a-backend/src/schema/openapi/generated/models/Project.model.ts

@@ -17,6 +17,7 @@
 // ******************************************************************
 // * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY. *
 // ******************************************************************
+import { Artifact } from '../models/Artifact.model';
 
 /**
  * @public
@@ -62,4 +63,5 @@ export interface Project {
    * The user who created the project (Backstage user reference)
    */
   createdBy: string;
+  migrationPlan?: Artifact;
 }

workspaces/x2a/plugins/x2a-backend/src/schema/openapi/generated/models/ProjectsProjectIdModulesModuleIdRunPostRequest.model.ts

@@ -26,8 +26,8 @@ import { ModulePhase } from '../models/ModulePhase.model';
  */
 export interface ProjectsProjectIdModulesModuleIdRunPostRequest {
   phase: ModulePhase;
-  sourceRepoAuth?: GitRepoAuth;
-  targetRepoAuth?: GitRepoAuth;
+  sourceRepoAuth: GitRepoAuth;
+  targetRepoAuth: GitRepoAuth;
   aapCredentials?: AAPCredentials;
   /**
    * Optional user prompt for customizing the migration

workspaces/x2a/plugins/x2a-backend/src/schema/openapi/generated/models/index.ts

@@ -16,6 +16,7 @@
 
 export * from '../models/AAPCredentials.model';
 export * from '../models/Artifact.model';
+export * from '../models/ArtifactType.model';
 export * from '../models/GitRepoAuth.model';
 export * from '../models/Job.model';
 export * from '../models/JobStatusEnum.model';

workspaces/x2a/plugins/x2a-backend/src/schema/openapi/generated/router.ts

@@ -452,8 +452,8 @@ export const spec = {
                 },
                 "required": [
                   "phase",
-                  "sourceRepo",
-                  "targetRepo"
+                  "sourceRepoAuth",
+                  "targetRepoAuth"
                 ]
               }
             }
@@ -597,6 +597,10 @@ export const spec = {
           "createdBy": {
             "type": "string",
             "description": "The user who created the project (Backstage user reference)"
+          },
+          "migrationPlan": {
+            "$ref": "#/components/schemas/Artifact",
+            "description": "Project migration plan artifact (by init phase)"
           }
         },
         "required": [
@@ -713,6 +717,14 @@ export const spec = {
           }
         }
       },
+      "ArtifactType": {
+        "type": "string",
+        "enum": [
+          "migration_plan",
+          "module_migration_plan",
+          "migrated_sources"
+        ]
+      },
       "Artifact": {
         "type": "object",
         "properties": {
@@ -721,7 +733,7 @@ export const spec = {
             "description": "UUID for the artifact"
           },
           "type": {
-            "type": "string",
+            "$ref": "#/components/schemas/ArtifactType",
             "description": "Type of the artifact"
           },
           "value": {