Skip to content

Conversation

@MerwelLabs
Copy link

Adds autorun and sync helper scripts for local webui maintenance and repo sync. Small utility files only.

Copilot AI review requested due to automatic review settings September 28, 2025 00:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds utility scripts for automating local webui maintenance and repository synchronization. The changes introduce PowerShell scripts and batch files to handle automatic startup of a text-generation-webui server and synchronization of MerwelLabs GitHub repositories.

  • Autorun functionality for text-generation-webui with minimized startup
  • Repository synchronization scripts with logging and error handling
  • Installation script for Windows startup integration

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
sync_merwel_repos.ps1 Enhanced repo sync script with comprehensive logging and log rotation
start_webui_minimized.ps1 PowerShell script to launch webui server minimized using venv
install_autostart.bat Batch script to create Windows startup shortcut
clone_merwel_repos.ps1 Basic repository cloning/updating script without logging
AUTORUN_README.md Documentation for autorun setup and usage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@echo off
REM Creates a shortcut in the current user's Startup folder to run the web UI minimized at login.
set SCRIPT_PATH=%~dp0start_webui_minimized.ps1
necho Creating startup shortcut...
Copy link

Copilot AI Sep 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a typo in the echo command - 'necho' should be 'echo'.

Suggested change
necho Creating startup shortcut...
echo Creating startup shortcut...

Copilot uses AI. Check for mistakes.
#>

$dest = 'D:\OneDrive_Merwel\OneDrive\Github'
$gh = 'C:\Program Files\GitHub CLI\gh.exe'
Copy link

Copilot AI Sep 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hard-coded paths reduce portability. Consider using environment variables or configuration parameters to make these paths configurable.

Suggested change
$gh = 'C:\Program Files\GitHub CLI\gh.exe'
# Allow override of GitHub CLI path via environment variable GH_PATH
$gh = if ($env:GH_PATH) { $env:GH_PATH } else { 'C:\Program Files\GitHub CLI\gh.exe' }

Copilot uses AI. Check for mistakes.
Comment on lines +2 to +3
$dest = 'D:\OneDrive_Merwel\OneDrive\Github'
$gh = 'C:\Program Files\GitHub CLI\gh.exe'
Copy link

Copilot AI Sep 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hard-coded paths are duplicated across scripts. Consider centralizing these configuration values in a shared configuration file or variables to avoid maintenance issues.

Copilot uses AI. Check for mistakes.
# Start text-generation-webui in the project's venv, minimized.
# Created by automation. To edit, update this file in the project folder.

$ProjectPath = 'D:\AI_ML_Development\text-generation-webui'
Copy link

Copilot AI Sep 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hard-coded project path reduces portability. Consider making this configurable through environment variables or command-line parameters.

Suggested change
$ProjectPath = 'D:\AI_ML_Development\text-generation-webui'
# Allow project path to be set via environment variable or command-line argument.
if ($env:TEXTGEN_PROJECT_PATH) {
$ProjectPath = $env:TEXTGEN_PROJECT_PATH
} elseif ($args.Count -ge 1) {
$ProjectPath = $args[0]
} else {
$ProjectPath = 'D:\AI_ML_Development\text-generation-webui'
}

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant