Repository navigation
Release v2.7.0: Accounts layout, SYSTEM meters, Mods and Anti-AFK fixes - #25
Conversation
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f11930dcdd
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| document.addEventListener('click',event=>{ | ||
| const shortcut=event.target.closest('[data-manager-goto]'); | ||
| if(shortcut){const next=shortcut.dataset.managerGoto;if(!tabs.some(([key])=>key===next))return;pane=next;document.querySelector('.nav-item[data-tab="accounts"]').click();if(data)render();refresh();return;} |
There was a problem hiding this comment.
After opening Sessions or Appearance through the new sidebar shortcuts, clicking Accounts does not return to the account table. The shortcut changes pane, but the Accounts button only calls switchView('accounts'); neither that function nor the manager's view observer resets pane, so refresh renders the previous subpane again. Handle explicit Accounts navigation separately from the synthetic click used by these shortcuts so users can return to their accounts through the sidebar.
Useful? React with 👍 / 👎.
| const active = this.json('active.json'); | ||
| if (!active) throw new Error('No active mod to restore.'); | ||
| const original = decode(active.original,active.originalHash); | ||
| const destination = this.destination(active.version,active.target), currentHash = hash(readAsset(destination)); |
There was a problem hiding this comment.
If a Roblox update or old-version cleanup removes the installation containing an active mod, destination() throws before the active record can be cleared. Every subsequent preview is then blocked by active.json, and the active library entry cannot be removed either, leaving Mods unusable even for a newly detected installation. This can also occur through the repository's ManagerCleanupService.plan(), which does not protect modded versions. Provide an explicit archive/discard recovery path for a missing installation while retaining the original backup rather than requiring a successful restoration to a directory that no longer exists.
Useful? React with 👍 / 👎.
Replaces Home with the compact Accounts landing page, retains DragonStrap branding/purple accents, and adds live sidebar SYSTEM meters, account-policy shortcuts and direct session navigation. Includes the reviewed local Mods media-replacement preview, Statistics cleanup and Anti-AFK retry/input fixes.
Release documentation distinguishes fixture verification from remaining live Mods and per-game Anti-AFK acceptance. Historical v2.6.1 VirusTotal reports remain scoped to their exact files; v2.7.0 is unsigned and has no verified new scan report.
Validation: 259 tests, actual isolated app startup/system IPC, native-input compilation and fake-provider scenarios, multi-size desktop workflows. Packaging uses pinned official Electron/installer archives with SHA-256 checks; corrupt installer archives are rejected before extraction. Source must be committed before packaging, and checksums list only the current release artifacts.