Refactor initial setup process: Consolidate setup file copying and directory creation into the InitialSetupUseCase, updating return values to include skipped files for better logging.

Author: efraespada

build/cli/index.js

@@ -46418,7 +46418,6 @@ const constants_1 = __nccwpck_require__(8593);
 const chalk_1 = __importDefault(__nccwpck_require__(7037));
 const boxen_1 = __importDefault(__nccwpck_require__(4506));
 const queue_utils_1 = __nccwpck_require__(9800);
-const setup_files_1 = __nccwpck_require__(1666);
 async function mainRun(execution) {
     const results = [];
     await execution.setup();
@@ -46459,19 +46458,6 @@ async function mainRun(execution) {
             title: constants_1.TITLE,
             titleAlignment: 'center'
         }));
-        if (execution.isSingleAction && execution.singleAction.currentSingleAction === constants_1.ACTIONS.INITIAL_SETUP) {
-            const cwd = process.cwd();
-            (0, logger_1.logInfo)('📁 Ensuring .github and .github/workflows exist...');
-            (0, setup_files_1.ensureGitHubDirs)(cwd);
-            (0, logger_1.logInfo)('📋 Copying setup files from setup/ to .github/ (skipping existing)...');
-            const copied = (0, setup_files_1.copySetupFiles)(cwd);
-            if (copied > 0) {
-                (0, logger_1.logInfo)(`✅ Copied ${copied} file(s).`);
-            }
-            else {
-                (0, logger_1.logInfo)('ℹ️  No setup/ folder found or all files already exist; nothing to copy.');
-            }
-        }
     }
     try {
         if (execution.isSingleAction) {
@@ -52873,6 +52859,7 @@ const issue_repository_1 = __nccwpck_require__(57);
 const project_repository_1 = __nccwpck_require__(7917);
 const result_1 = __nccwpck_require__(7305);
 const logger_1 = __nccwpck_require__(8836);
+const setup_files_1 = __nccwpck_require__(1666);
 class InitialSetupUseCase {
     constructor() {
         this.taskId = 'InitialSetupUseCase';
@@ -52883,6 +52870,11 @@ class InitialSetupUseCase {
         const steps = [];
         const errors = [];
         try {
+            // 0. Setup files (.github/workflows, pull_request_template.md, .env)
+            (0, logger_1.logInfo)('📋 Ensuring .github and copying setup files...');
+            (0, setup_files_1.ensureGitHubDirs)(process.cwd());
+            const filesResult = (0, setup_files_1.copySetupFiles)(process.cwd());
+            steps.push(`✅ Setup files: ${filesResult.copied} copied, ${filesResult.skipped} already existed`);
             // 1. Verificar acceso a GitHub con Personal Access Token
             (0, logger_1.logInfo)('🔐 Checking GitHub access...');
             const githubAccessResult = await this.verifyGitHubAccess(param);
@@ -57994,13 +57986,14 @@ function ensureGitHubDirs(cwd) {
  * Skips files that already exist at destination (no overwrite).
  * Logs each file copied or skipped. No-op if setup/ does not exist.
  * @param cwd - Repo root
- * @returns Number of files copied
+ * @returns { copied, skipped }
  */
 function copySetupFiles(cwd) {
     const setupDir = path.join(cwd, 'setup');
     if (!fs.existsSync(setupDir))
-        return 0;
+        return { copied: 0, skipped: 0 };
     let copied = 0;
+    let skipped = 0;
     const workflowsSrc = path.join(setupDir, 'workflows');
     const workflowsDst = path.join(cwd, '.github', 'workflows');
     if (fs.existsSync(workflowsSrc)) {
@@ -58011,6 +58004,7 @@ function copySetupFiles(cwd) {
             if (fs.statSync(src).isFile()) {
                 if (fs.existsSync(dst)) {
                     (0, logger_1.logInfo)(`  ⏭️  .github/workflows/${f} already exists; skipping.`);
+                    skipped += 1;
                 }
                 else {
                     fs.copyFileSync(src, dst);
@@ -58025,6 +58019,7 @@ function copySetupFiles(cwd) {
     if (fs.existsSync(prTemplateSrc)) {
         if (fs.existsSync(prTemplateDst)) {
             (0, logger_1.logInfo)('  ⏭️  .github/pull_request_template.md already exists; skipping.');
+            skipped += 1;
         }
         else {
             fs.copyFileSync(prTemplateSrc, prTemplateDst);
@@ -58037,14 +58032,15 @@ function copySetupFiles(cwd) {
     if (fs.existsSync(envSrc) && fs.statSync(envSrc).isFile()) {
         if (fs.existsSync(envDst)) {
             (0, logger_1.logInfo)('  ⏭️  .env already exists; skipping.');
+            skipped += 1;
         }
         else {
             fs.copyFileSync(envSrc, envDst);
             (0, logger_1.logInfo)('  ✅ Copied setup/.env → .env');
             copied += 1;
         }
     }
-    return copied;
+    return { copied, skipped };
 }
 
 

build/cli/src/utils/setup_files.d.ts

@@ -8,6 +8,9 @@ export declare function ensureGitHubDirs(cwd: string): void;
  * Skips files that already exist at destination (no overwrite).
  * Logs each file copied or skipped. No-op if setup/ does not exist.
  * @param cwd - Repo root
- * @returns Number of files copied
+ * @returns { copied, skipped }
  */
-export declare function copySetupFiles(cwd: string): number;
+export declare function copySetupFiles(cwd: string): {
+    copied: number;
+    skipped: number;
+};

build/github_action/index.js

@@ -41936,7 +41936,6 @@ const constants_1 = __nccwpck_require__(8593);
 const chalk_1 = __importDefault(__nccwpck_require__(7037));
 const boxen_1 = __importDefault(__nccwpck_require__(4506));
 const queue_utils_1 = __nccwpck_require__(9800);
-const setup_files_1 = __nccwpck_require__(1666);
 async function mainRun(execution) {
     const results = [];
     await execution.setup();
@@ -41977,19 +41976,6 @@ async function mainRun(execution) {
             title: constants_1.TITLE,
             titleAlignment: 'center'
         }));
-        if (execution.isSingleAction && execution.singleAction.currentSingleAction === constants_1.ACTIONS.INITIAL_SETUP) {
-            const cwd = process.cwd();
-            (0, logger_1.logInfo)('📁 Ensuring .github and .github/workflows exist...');
-            (0, setup_files_1.ensureGitHubDirs)(cwd);
-            (0, logger_1.logInfo)('📋 Copying setup files from setup/ to .github/ (skipping existing)...');
-            const copied = (0, setup_files_1.copySetupFiles)(cwd);
-            if (copied > 0) {
-                (0, logger_1.logInfo)(`✅ Copied ${copied} file(s).`);
-            }
-            else {
-                (0, logger_1.logInfo)('ℹ️  No setup/ folder found or all files already exist; nothing to copy.');
-            }
-        }
     }
     try {
         if (execution.isSingleAction) {
@@ -47952,6 +47938,7 @@ const issue_repository_1 = __nccwpck_require__(57);
 const project_repository_1 = __nccwpck_require__(7917);
 const result_1 = __nccwpck_require__(7305);
 const logger_1 = __nccwpck_require__(8836);
+const setup_files_1 = __nccwpck_require__(1666);
 class InitialSetupUseCase {
     constructor() {
         this.taskId = 'InitialSetupUseCase';
@@ -47962,6 +47949,11 @@ class InitialSetupUseCase {
         const steps = [];
         const errors = [];
         try {
+            // 0. Setup files (.github/workflows, pull_request_template.md, .env)
+            (0, logger_1.logInfo)('📋 Ensuring .github and copying setup files...');
+            (0, setup_files_1.ensureGitHubDirs)(process.cwd());
+            const filesResult = (0, setup_files_1.copySetupFiles)(process.cwd());
+            steps.push(`✅ Setup files: ${filesResult.copied} copied, ${filesResult.skipped} already existed`);
             // 1. Verificar acceso a GitHub con Personal Access Token
             (0, logger_1.logInfo)('🔐 Checking GitHub access...');
             const githubAccessResult = await this.verifyGitHubAccess(param);
@@ -53469,13 +53461,14 @@ function ensureGitHubDirs(cwd) {
  * Skips files that already exist at destination (no overwrite).
  * Logs each file copied or skipped. No-op if setup/ does not exist.
  * @param cwd - Repo root
- * @returns Number of files copied
+ * @returns { copied, skipped }
  */
 function copySetupFiles(cwd) {
     const setupDir = path.join(cwd, 'setup');
     if (!fs.existsSync(setupDir))
-        return 0;
+        return { copied: 0, skipped: 0 };
     let copied = 0;
+    let skipped = 0;
     const workflowsSrc = path.join(setupDir, 'workflows');
     const workflowsDst = path.join(cwd, '.github', 'workflows');
     if (fs.existsSync(workflowsSrc)) {
@@ -53486,6 +53479,7 @@ function copySetupFiles(cwd) {
             if (fs.statSync(src).isFile()) {
                 if (fs.existsSync(dst)) {
                     (0, logger_1.logInfo)(`  ⏭️  .github/workflows/${f} already exists; skipping.`);
+                    skipped += 1;
                 }
                 else {
                     fs.copyFileSync(src, dst);
@@ -53500,6 +53494,7 @@ function copySetupFiles(cwd) {
     if (fs.existsSync(prTemplateSrc)) {
         if (fs.existsSync(prTemplateDst)) {
             (0, logger_1.logInfo)('  ⏭️  .github/pull_request_template.md already exists; skipping.');
+            skipped += 1;
         }
         else {
             fs.copyFileSync(prTemplateSrc, prTemplateDst);
@@ -53512,14 +53507,15 @@ function copySetupFiles(cwd) {
     if (fs.existsSync(envSrc) && fs.statSync(envSrc).isFile()) {
         if (fs.existsSync(envDst)) {
             (0, logger_1.logInfo)('  ⏭️  .env already exists; skipping.');
+            skipped += 1;
         }
         else {
             fs.copyFileSync(envSrc, envDst);
             (0, logger_1.logInfo)('  ✅ Copied setup/.env → .env');
             copied += 1;
         }
     }
-    return copied;
+    return { copied, skipped };
 }
 
 

build/github_action/src/data/repository/branch_repository.d.ts

@@ -33,7 +33,7 @@ export declare class BranchRepository {
         totalCommits: number;
         files: {
             filename: string;
-            status: "modified" | "added" | "removed" | "renamed" | "copied" | "changed" | "unchanged";
+            status: "added" | "removed" | "modified" | "renamed" | "copied" | "changed" | "unchanged";
             additions: number;
             deletions: number;
             changes: number;

build/github_action/src/utils/setup_files.d.ts

@@ -8,6 +8,9 @@ export declare function ensureGitHubDirs(cwd: string): void;
  * Skips files that already exist at destination (no overwrite).
  * Logs each file copied or skipped. No-op if setup/ does not exist.
  * @param cwd - Repo root
- * @returns Number of files copied
+ * @returns { copied, skipped }
  */
-export declare function copySetupFiles(cwd: string): number;
+export declare function copySetupFiles(cwd: string): {
+    copied: number;
+    skipped: number;
+};

src/actions/common_action.ts

@@ -8,11 +8,10 @@ import { PullRequestReviewCommentUseCase } from '../usecase/pull_request_review_
 import { PullRequestUseCase } from '../usecase/pull_request_use_case';
 import { SingleActionUseCase } from '../usecase/single_action_use_case';
 import { logError, logInfo } from '../utils/logger';
-import { ACTIONS, TITLE } from '../utils/constants';
+import { TITLE } from '../utils/constants';
 import chalk from 'chalk';
 import boxen from 'boxen';
 import { waitForPreviousRuns } from '../utils/queue_utils';
-import { copySetupFiles, ensureGitHubDirs } from '../utils/setup_files';
 
 export async function mainRun(execution: Execution): Promise<Result[]> {
     const results: Result[] = []
@@ -64,18 +63,6 @@ export async function mainRun(execution: Execution): Promise<Result[]> {
                 }
             )
         );
-        if (execution.isSingleAction && execution.singleAction.currentSingleAction === ACTIONS.INITIAL_SETUP) {
-            const cwd = process.cwd();
-            logInfo('📁 Ensuring .github and .github/workflows exist...');
-            ensureGitHubDirs(cwd);
-            logInfo('📋 Copying setup files from setup/ to .github/ (skipping existing)...');
-            const copied = copySetupFiles(cwd);
-            if (copied > 0) {
-                logInfo(`✅ Copied ${copied} file(s).`);
-            } else {
-                logInfo('ℹ️  No setup/ folder found or all files already exist; nothing to copy.');
-            }
-        }
     }
 
     try {

src/usecase/actions/initial_setup_use_case.ts

@@ -4,6 +4,7 @@ import { ProjectRepository } from "../../data/repository/project_repository";
 import { Result } from "../../data/model/result";
 import { ParamUseCase } from "../base/param_usecase";
 import { logError, logInfo } from "../../utils/logger";
+import { copySetupFiles, ensureGitHubDirs } from "../../utils/setup_files";
 
 export class InitialSetupUseCase implements ParamUseCase<Execution, Result[]> {
     taskId: string = 'InitialSetupUseCase';
@@ -16,6 +17,12 @@ export class InitialSetupUseCase implements ParamUseCase<Execution, Result[]> {
         const errors: string[] = [];
 
         try {
+            // 0. Setup files (.github/workflows, pull_request_template.md, .env)
+            logInfo('📋 Ensuring .github and copying setup files...');
+            ensureGitHubDirs(process.cwd());
+            const filesResult = copySetupFiles(process.cwd());
+            steps.push(`✅ Setup files: ${filesResult.copied} copied, ${filesResult.skipped} already existed`);
+
             // 1. Verificar acceso a GitHub con Personal Access Token
             logInfo('🔐 Checking GitHub access...');
             const githubAccessResult = await this.verifyGitHubAccess(param);

src/utils/setup_files.ts

@@ -24,13 +24,14 @@ export function ensureGitHubDirs(cwd: string): void {
  * Skips files that already exist at destination (no overwrite).
  * Logs each file copied or skipped. No-op if setup/ does not exist.
  * @param cwd - Repo root
- * @returns Number of files copied
+ * @returns { copied, skipped }
  */
-export function copySetupFiles(cwd: string): number {
+export function copySetupFiles(cwd: string): { copied: number; skipped: number } {
   const setupDir = path.join(cwd, 'setup');
-  if (!fs.existsSync(setupDir)) return 0;
+  if (!fs.existsSync(setupDir)) return { copied: 0, skipped: 0 };
 
   let copied = 0;
+  let skipped = 0;
   const workflowsSrc = path.join(setupDir, 'workflows');
   const workflowsDst = path.join(cwd, '.github', 'workflows');
   if (fs.existsSync(workflowsSrc)) {
@@ -41,6 +42,7 @@ export function copySetupFiles(cwd: string): number {
       if (fs.statSync(src).isFile()) {
         if (fs.existsSync(dst)) {
           logInfo(`  ⏭️  .github/workflows/${f} already exists; skipping.`);
+          skipped += 1;
         } else {
           fs.copyFileSync(src, dst);
           logInfo(`  ✅ Copied setup/workflows/${f} → .github/workflows/${f}`);
@@ -54,6 +56,7 @@ export function copySetupFiles(cwd: string): number {
   if (fs.existsSync(prTemplateSrc)) {
     if (fs.existsSync(prTemplateDst)) {
       logInfo('  ⏭️  .github/pull_request_template.md already exists; skipping.');
+      skipped += 1;
     } else {
       fs.copyFileSync(prTemplateSrc, prTemplateDst);
       logInfo('  ✅ Copied setup/pull_request_template.md → .github/pull_request_template.md');
@@ -65,11 +68,12 @@ export function copySetupFiles(cwd: string): number {
   if (fs.existsSync(envSrc) && fs.statSync(envSrc).isFile()) {
     if (fs.existsSync(envDst)) {
       logInfo('  ⏭️  .env already exists; skipping.');
+      skipped += 1;
     } else {
       fs.copyFileSync(envSrc, envDst);
       logInfo('  ✅ Copied setup/.env → .env');
       copied += 1;
     }
   }
-  return copied;
+  return { copied, skipped };
 }