refactor(auto-fork): deduplicate code for better maintainability

Code quality improvements:
- Extract _record_fork() helper to eliminate 3 duplicated code blocks
- Remove dead code (redundant validation in detect_unforkable_repos)
- Update all docstrings to reflect GitLab support
- Update SKILL.md: remove outdated "GitLab skipped" notices
- Update README.md: add GL_USER_NAME env var, fix outdated docs

All tests pass (24/24).

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: catastrophe-brandon

.claude/skills/auto-fork/README.md

@@ -1,14 +1,13 @@
 # Auto-Fork Skill
 
-Automatically fork repos and update configuration. Detects repos in `project-repos.json` without forks, creates forks under bot's GitHub account, and updates the config file with the new fork URLs.
+Automatically fork repos and update configuration. Detects repos in `project-repos.json` without forks, creates forks under bot's GitHub or GitLab account, and updates the config file with the new fork URLs.
 
 ## Features
 
 - Scans `project-repos.json` for repos needing forks
-- Creates forks using `gh repo fork` (GitHub only)
+- Creates forks using `gh repo fork` (GitHub) or `glab repo fork` (GitLab)
 - Updates config file with fork URLs
 - Commits changes to new branch
-- GitLab repos logged and skipped (manual forking required)
 
 ## Usage
 
@@ -33,6 +32,7 @@ python3 auto_fork.py --dry-run
 Environment variables (auto-provided by bot runtime):
 
 - `GH_USER_NAME` — bot GitHub username
+- `GL_USER_NAME` — bot GitLab username
 - `BOT_CONFIG_PATH` — config directory (default `rehor-config`)
 - `BOT_INSTANCE_ID` — optional instance ID (affects branch name)
 
@@ -69,14 +69,14 @@ uv run ruff check .
 
 1. **detect_unforkable_repos** - Scans `project-repos.json`:
    - Identifies repos with `upstream` field
-   - Checks if `url` matches `GH_USER_NAME`
+   - Checks if `url` matches bot account (`GH_USER_NAME` for GitHub, `GL_USER_NAME` for GitLab)
    - Skips repos already forked
-   - Skips GitLab repos (GitHub only)
 
 2. **fork_repos** - Creates forks:
-   - Uses `gh repo fork --clone=false`
+   - GitHub: Uses `gh repo fork --clone=false`
+   - GitLab: Uses `glab repo fork --clone=false --hostname gitlab.cee.redhat.com`
    - Handles already-forked repos gracefully
-   - Generates fork URLs: `https://github.com/{bot-username}/{repo-name}.git`
+   - Generates fork URLs with bot username
 
 3. **update_and_commit** - Updates config:
    - Updates `url` field in `project-repos.json`

.claude/skills/auto-fork/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: auto-fork
 description: >
-  Auto-fork repos + update config. Scans project-repos.json for missing forks → forks via gh → updates config → commits.
+  Auto-fork repos + update config. Scans project-repos.json for missing forks → forks via gh/glab → updates config → commits.
 when_to_use: >
   Triage when repo needs fork or setting up instances. Triggers: "fork repo", "auto fork", "setup fork", "missing fork".
 user-invocable: true
@@ -21,13 +21,11 @@ python3 .claude/skills/auto-fork/auto_fork.py 2>&1
 The script executes these operations:
 
 1. **detect_unforkable_repos** - Scan project-repos.json for repos needing forks
-2. **fork_repos** - Create forks using `gh repo fork` for GitHub repos
+2. **fork_repos** - Create forks using `gh repo fork` (GitHub) or `glab repo fork` (GitLab)
 3. **update_and_commit** - Update project-repos.json with new fork URLs and commit changes
 
 After this completes successfully, use the **push-and-pr** skill to push and create PR.
 
-GitLab repos are skipped with a logged notice (manual forking required).
-
 ## Configuration
 
 Env vars (auto-provided by runtime):
@@ -52,4 +50,4 @@ Repos w/ `upstream` but `url` not matching bot account → need fork.
 ## Error Handling
 
 Fail-fast: if any fork operation fails, stops and reports error.
-Idempotent: safe to re-run if forks already exist (gh repo fork handles gracefully).
+Idempotent: safe to re-run if forks already exist (gh/glab repo fork handle gracefully).

.claude/skills/auto-fork/auto_fork.py

@@ -2,17 +2,16 @@
 """
 Auto-fork workflow for repos in project-repos.json.
 
-Detects repos without forks, creates forks under bot's GitHub account,
+Detects repos without forks, creates forks under bot's GitHub or GitLab account,
 and updates project-repos.json locally. After committing changes, use
 push-and-pr skill to create the PR.
 
 Operations:
 1. detect_unforkable_repos - scan for repos needing forks
-2. fork_repos - create forks using gh repo fork
+2. fork_repos - create forks using gh repo fork (GitHub) or glab repo fork (GitLab)
 3. update_and_commit - update project-repos.json and commit changes
 
 After this script completes, use push-and-pr skill to create the PR.
-GitLab repos are skipped with logged notice.
 """
 
 import argparse
@@ -135,19 +134,11 @@ def detect_unforkable_repos(self) -> OperationResult:
         A repo needs a fork if:
         - It has an 'upstream' field
         - Its 'url' field doesn't match bot's account pattern
-        - It's a GitHub repo (GitLab skipped for now)
+        - Supports both GitHub and GitLab repos
 
         Returns:
             OperationResult with list of repos needing forks
         """
-        if not self.bot_username:
-            error_msg = "BOT_GITHUB_USERNAME env var not set"
-            logger.error(error_msg)
-            return OperationResult(
-                operation="detect_unforkable_repos",
-                status=OperationStatus.FAILED,
-                message=error_msg,
-            )
 
         if not self.project_repos_path.exists():
             error_msg = f"project-repos.json not found at {self.project_repos_path}"
@@ -239,6 +230,21 @@ def _get_fork_url(self, repo_name: str, host: str) -> str:
             return f"https://{GITLAB_HOST}/{self.gl_username}/{repo_name}.git"
         return f"https://github.com/{self.bot_username}/{repo_name}.git"
 
+    def _record_fork(self, repo_name: str, host: str) -> str:
+        """
+        Record successful fork and return fork URL.
+
+        Args:
+            repo_name: Name of the repository
+            host: Host type ("github" or "gitlab")
+
+        Returns:
+            Fork URL
+        """
+        fork_url = self._get_fork_url(repo_name, host)
+        self.forked_repos[repo_name] = fork_url
+        return fork_url
+
     def fork_repos(self) -> OperationResult:
         """
         Create forks for detected repos using gh or glab.
@@ -264,8 +270,7 @@ def fork_repos(self) -> OperationResult:
             logger.info(f"Forking {path} ({repo.host})...")
 
             if self.dry_run:
-                fork_url = self._get_fork_url(repo.name, repo.host)
-                self.forked_repos[repo.name] = fork_url
+                fork_url = self._record_fork(repo.name, repo.host)
                 logger.info(f"[DRY RUN] Would fork {path} to {fork_url}")
                 continue
 
@@ -297,16 +302,14 @@ def fork_repos(self) -> OperationResult:
                     # Check if already forked (not an error)
                     if "already exists" in result.stderr.lower() or "already forked" in result.stderr.lower():
                         logger.info(f"{path} already forked")
-                        fork_url = self._get_fork_url(repo.name, repo.host)
-                        self.forked_repos[repo.name] = fork_url
+                        self._record_fork(repo.name, repo.host)
                     else:
                         error_msg = f"Failed to fork {path}: {result.stderr}"
                         logger.error(error_msg)
                         failed.append(repo.name)
                         continue
                 else:
-                    fork_url = self._get_fork_url(repo.name, repo.host)
-                    self.forked_repos[repo.name] = fork_url
+                    fork_url = self._record_fork(repo.name, repo.host)
                     logger.info(f"Forked {path} to {fork_url}")
 
             except Exception as e: