Architectural Refactoring & Type-Safe GitHub Action (#6)

* chore(config): add ConfigManager with centralized validation and immutable typing

- adds src/config.ts containing the configmanager class.
- implements typed input reading via core.getinput.
- introduces ActionInputs interface with readonly properties to ensure immutability.
- centralizes validation rules (regex, nullability), following clean code and fail fast standards.
- acts as an adapter mapping external inputs (snake_case) to internal camelcase properties.

* refactor(core): transform index.ts into a pure action orchestrator

- removes duplicate logic for validation and input reading.
- uses configmanager to securely retrieve all configurations.
- maintains only the orchestration flow (bootstrap → exec), following srp.
- improves architectural clarity and prepares the module for future extensions.

* build: update compiled artifacts in dist/

- updates dist/index.js to reflect the new modular architecture.
- synchronizes the build after refactoring index.ts and adding config.ts.
- no extra functional changes beyond the typescript output.

* documenting...

Author: rmottanet

README.md

@@ -52,10 +52,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Exec Branch Shift
-        uses: ws2git/branch-shift@v1 
+        uses: ws2git/branch-shift@v1.5
 
         with:
-          github_token: ${{ secrets.YOUR_PAT }} 
+          github-token: ${{ secrets.YOUR_PAT }} 
           owner: ${{ github.event.inputs.owner }}
           repo: ${{ github.event.inputs.repo }}
           branch: ${{ github.event.inputs.branch }}
@@ -67,7 +67,7 @@ jobs:
 
 | Name | Required | Description |
 |---|---|---|
-| `github_token` | Yes | GitHub Token with `repo` scope to authorize the API call. |
+| `github-token` | Yes | GitHub Token with `repo` scope to authorize the API call. |
 | `owner` | Yes | The owner of the repository (e.g., your-organization or your-username). |
 | `repo` | Yes | The name of the repository (e.g., your-repo). |
 | `branch` | Yes | The current name of the branch to be renamed. |

action.yml

@@ -5,7 +5,7 @@ branding:
   icon: 'git-branch'
   color: 'orange'
 inputs:
-  github_token:
+  github-token:
     description: 'GitHub Token'
     required: true
   owner:

dist/index.js

@@ -29922,7 +29922,7 @@ function wrappy (fn, cb) {
 
 /***/ }),
 
-/***/ 9407:
+/***/ 2973:
 /***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
 
 "use strict";
@@ -29961,36 +29961,87 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ConfigManager = void 0;
 const core = __importStar(__nccwpck_require__(7484));
-const github = __importStar(__nccwpck_require__(3228));
-/**
-* Helper function to validate inputs before execution.
-* Throws an error if any validation fails.
-*/
-function validateInputs(inputs) {
-    const { owner, repo, branch, newName, githubToken } = inputs;
-    if (!owner || !repo || !branch || !newName || !githubToken) {
-        throw new Error('All input fields (owner, repo, branch, new_name) are required.');
+class ConfigManager {
+    static getInputs() {
+        const inputs = {
+            owner: core.getInput('owner', { required: true }).trim(),
+            repo: core.getInput('repo', { required: true }).trim(),
+            branch: core.getInput('branch', { required: true }).trim(),
+            newName: core.getInput('new_name', { required: true }).trim(),
+            githubToken: core.getInput('github-token', { required: true }).trim(),
+        };
+        this.validate(inputs);
+        return inputs;
     }
-    const branchRegex = /^[a-zA-Z0-9-_./]+$/;
-    if (!branchRegex.test(newName)) {
-        throw new Error(`The new branch name is invalid: '${newName}'. Allowed characters: a-z, A-Z, 0-9, -, _, ., /`);
+    static validate(inputs) {
+        const { owner, repo, branch, newName, githubToken } = inputs;
+        if (!owner || !repo || !branch || !newName || !githubToken) {
+            throw new Error('All inputs are required and cannot be empty.');
+        }
+        const branchRegex = /^[a-zA-Z0-9-_./]+$/;
+        if (!branchRegex.test(newName)) {
+            throw new Error(`Invalid branch name format: '${newName}'.`);
+        }
     }
 }
+exports.ConfigManager = ConfigManager;
+
+
+/***/ }),
+
+/***/ 9407:
+/***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+const core = __importStar(__nccwpck_require__(7484));
+const github = __importStar(__nccwpck_require__(3228));
+const config_1 = __nccwpck_require__(2973);
 /**
  * Main asynchronous function.
  * @returns {Promise<void>}
  */
 async function run() {
     try {
-        const inputs = {
-            githubToken: core.getInput('github_token', { required: true }).trim(),
-            owner: core.getInput('owner', { required: true }).trim(),
-            repo: core.getInput('repo', { required: true }).trim(),
-            branch: core.getInput('branch', { required: true }).trim(),
-            newName: core.getInput('new_name', { required: true }).trim(),
-        };
-        validateInputs(inputs);
+        core.info('-> Initializing action...');
+        const inputs = config_1.ConfigManager.getInputs();
+        core.info('✅ Inputs validated.');
         const octokit = github.getOctokit(inputs.githubToken);
         core.info(`Starting the renaming of '${inputs.branch}' to '${inputs.newName}'...`);
         await octokit.request('POST /repos/{owner}/{repo}/branches/{branch}/rename', {
@@ -30002,7 +30053,7 @@ async function run() {
                 'X-GitHub-Api-Version': '2022-11-28',
             },
         });
-        core.info(`✅ Branch renamed successfully.`);
+        core.info(`✅ Branch renomeada com sucesso.`);
     }
     catch (error) {
         if (error instanceof Error) {

src/config.ts

@@ -0,0 +1,38 @@
+import * as core from '@actions/core';
+
+export interface ActionInputs {
+  readonly owner: string;
+  readonly repo: string;
+  readonly branch: string;
+  readonly newName: string;
+  readonly githubToken: string;
+}
+
+export class ConfigManager {
+  static getInputs(): ActionInputs {
+    const inputs: ActionInputs = {
+      owner: core.getInput('owner', { required: true }).trim(),
+      repo: core.getInput('repo', { required: true }).trim(),
+      branch: core.getInput('branch', { required: true }).trim(),
+      newName: core.getInput('new_name', { required: true }).trim(),
+      githubToken: core.getInput('github-token', { required: true }).trim(),
+    };
+
+    this.validate(inputs);
+    return inputs;
+  }
+
+  private static validate(inputs: ActionInputs): void {
+    const { owner, repo, branch, newName, githubToken } = inputs;
+
+    if (!owner || !repo || !branch || !newName || !githubToken) {
+      throw new Error('All inputs are required and cannot be empty.');
+    }
+
+    const branchRegex = /^[a-zA-Z0-9-_./]+$/;
+    if (!branchRegex.test(newName)) {
+      throw new Error(`Invalid branch name format: '${newName}'.`);
+    }
+
+  }
+}

src/index.ts

@@ -1,47 +1,17 @@
 import * as core from '@actions/core';
 import * as github from '@actions/github';
-
-interface ActionInputs {
-  readonly githubToken: string;
-  readonly owner: string;
-  readonly repo: string;
-  readonly branch: string;
-  readonly newName: string;
-}
-
-/**
-* Helper function to validate inputs before execution.
-* Throws an error if any validation fails.
-*/
-function validateInputs(inputs: ActionInputs): void {
-  const { owner, repo, branch, newName, githubToken } = inputs;
-
-  if (!owner || !repo || !branch || !newName || !githubToken) {
-    throw new Error('All input fields (owner, repo, branch, new_name) are required.');
-  }
-
-  const branchRegex = /^[a-zA-Z0-9-_./]+$/;
-  if (!branchRegex.test(newName)) {
-    throw new Error(`The new branch name is invalid: '${newName}'. Allowed characters: a-z, A-Z, 0-9, -, _, ., /`);
-  }
-}
-
+import { ConfigManager } from './config';
 
 /**
  * Main asynchronous function.
  * @returns {Promise<void>}
  */
 async function run(): Promise<void> {
   try {
-    const inputs: ActionInputs = {
-      githubToken: core.getInput('github_token', { required: true }).trim(),
-      owner: core.getInput('owner', { required: true }).trim(),
-      repo: core.getInput('repo', { required: true }).trim(),
-      branch: core.getInput('branch', { required: true }).trim(),
-      newName: core.getInput('new_name', { required: true }).trim(),
-    };
-    
-    validateInputs(inputs);
+    core.info('-> Initializing action...');
+
+    const inputs = ConfigManager.getInputs();
+    core.info('✅ Inputs validated.');
 
     const octokit = github.getOctokit(inputs.githubToken);
 
@@ -57,7 +27,7 @@ async function run(): Promise<void> {
       },
     });
 
-    core.info(`✅ Branch renamed successfully.`);
+    core.info(`✅ Branch renomeada com sucesso.`);
 
   } catch (error: unknown) {
     if (error instanceof Error) {