Merge Dev branch to main

[Kief5555]

This pull request introduces significant improvements to the build, packaging, and documentation systems for the project, with a focus on expanding platform support, improving artifact handling, and adding Nix packaging. The changes clarify the build matrix, add new packaging targets (notably Windows ARM64 and iOS), and provide detailed instructions for using the project with Nix. Additionally, the CI workflows have been updated to better handle platform-specific artifacts and auto-update feeds, and the documentation has been updated accordingly.

Build and Packaging Improvements:

• Added Windows ARM64 as a distinct build and release target, with its own packaging instructions and artifact handling; clarified that Windows auto-update remains x64-only for now. Also added iOS (ipa) as a packaging target. (.github/workflows/auto-build.yml, .github/workflows/release.yml, README.md, docs/development.md, opennow-stable/README.md) [1] [2] [3] [4] [5]

• Improved artifact upload steps in CI workflows to use per-platform artifact_paths, ensuring only relevant files are uploaded for each build matrix entry and preventing ARM64 artifacts from overwriting x64 auto-update metadata. (.github/workflows/auto-build.yml, .github/workflows/release.yml) [1] [2]

Nix Packaging Support:

• Added a flake.nix file to provide Nix flake support, enabling easy installation and packaging of the app on NixOS and related systems. Updated documentation with usage examples for both home-manager and system-wide configurations. (flake.nix, README.md) (flake.nixR1-R123, Fb536197L79R79)

Documentation Updates:

• Updated documentation to reflect the new build matrix, clarify packaging formats for each platform, and explain the current status of Windows ARM64 and auto-update feeds. Added a TestFlight badge for iOS beta access. (README.md, docs/development.md, opennow-stable/README.md) [1] [2] [3] [4]

Dependency and Metadata Updates:

• Updated opennow-stable/package-lock.json to version 0.3.6 and made minor dependency metadata adjustments, including the addition of the encoding package. [1] [2]

Other Minor Improvements:

• Adjusted some package metadata fields (such as peer and optional) in the lockfile for accuracy and compatibility. (opennow-stable/package-lock.json) [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15]

[capy-ai]

Added 1 comment

[capy-ai]

[🟡 Medium] [🔵 Bug]

The confirmRemoveAccount callback performs three sequential IPC calls (removeAccount, getSavedAccounts, getAuthSession) without a try/catch, and is invoked via void confirmRemoveAccount() which silently drops rejections. The modal is dismissed at lines 2402–2403 before the async work begins, so if any IPC call fails the user gets no feedback and the UI state becomes inconsistent (modal gone, but account not removed/state not refreshed). Contrast with handleSwitchAccount (line 2358) which wraps the same pattern in try/catch with recovery logic.

// opennow-stable/src/renderer/src/App.tsx
setRemoveAccountConfirmOpen(false);
setAccountToRemove(null);

await window.openNow.removeAccount(targetUserId);
const [accounts, sessionResult] = await Promise.all([
  window.openNow.getSavedAccounts(),
  window.openNow.getAuthSession(),
]);

Wrap the body in a try/catch similar to handleSwitchAccount and surface the error to the user (e.g., via setLoginError).

[capy-ai]

Added 1 comment

[capy-ai]

[🟡 Medium] [🔵 Bug]

Inside DiscordStatusMonitor.sync(), the outer guard at line 368 is if (!currentActivity || currentActivity.appId !== sessionAppId). Within that block, the inner checks at lines 369 and 372 test currentActivity?.appId === sessionAppId, which is always false — either currentActivity is null or its appId differs from sessionAppId. The ternary therefore always falls through to sessionAppId (a numeric string like "12345678") and new Date(), meaning the Discord status will display a raw numeric app ID rather than the game title.

// opennow-stable/src/main/index.ts
if (!currentActivity || currentActivity.appId !== sessionAppId) {
  const title = (currentActivity?.appId === sessionAppId && currentActivity.gameName)
    ? currentActivity.gameName   // ← never reached
    : sessionAppId;              // ← always taken

This matters on cold-start recovery: when the app restarts while a session is active, the monitor clears stale activity and then re-sets it with the numeric app ID. Fix by extracting the conditional or looking up the game name from the session metadata independently of currentActivity.

Suggested change

	const title = (currentActivity?.appId === sessionAppId && currentActivity.gameName)
	const title = currentActivity?.gameName ?? sessionAppId;
	const startTime = currentActivity?.startTimestamp ?? new Date();