Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 18, 2025

The codebase still shipped duplicated AutoHotkey v1 scripts despite v2 equivalents. v1 copies were kept for BO6 macros, Minecraft helpers, GUI launchers, ControllerQuit, Powerplan, and Keys.ahk.

  • BO6 macros: Made AHK_v2/Black_ops_6/bo6-afk.ahk reusable (mode registry + optional autostart/hotkey toggle) and replaced all v1 BO6 scripts with thin v2 wrappers that call the shared implementation.
  • App launchers/utilities: Replaced v1 Minecraft, ControllerQuit, Powerplan, GUI_PC/GUI_Laptop/GUI_Shared, WM, WM2 with v2 stubs that simply dispatch to their existing v2 counterparts.
  • Keys.ahk migration: Ported the hotkey suite to v2 syntax, preserving snapping, media, file-date rename, and documentation UI; removed unused vars and hardened shell selection iteration.

Example wrapper (v1 BO6 replaced):

#Requires AutoHotkey v2.0
g_registerDefaultHotkeys := false
#Include %A_ScriptDir%\..\..\AHK_v2\Black_ops_6\bo6-afk.ahk

F6::StartMode("bank_basic")
F7::{ StopAll(), ExitApp() }
Original prompt

Find and refactor duplicated code. Refactor/remove the old autohotkey v1 code, migrate anything that isnt in v2 yet to v2


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Ven0m0 <82972344+Ven0m0@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor duplicated code and migrate to AutoHotkey v2 Refactor duplicated AHK v1 scripts to v2 wrappers and migrate Keys.ahk Dec 18, 2025
Copilot AI requested a review from Ven0m0 December 18, 2025 10:47
@Ven0m0 Ven0m0 marked this pull request as ready for review December 18, 2025 11:05
@Ven0m0 Ven0m0 merged commit ac4a473 into main Dec 18, 2025
1 check failed
@Ven0m0 Ven0m0 deleted the copilot/refactor-duplicate-code branch December 18, 2025 11:05
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.

2 participants