Skip to content

Windows test rig: run-windows skill and first live check - #53

Merged
Fazorath merged 3 commits into
mainfrom
issue-51-windows-test-rig
Oct 26, 2026
Merged

Fazorath merged 3 commits into
mainfrom
issue-51-windows-test-rig

Conversation

@Fazorath

Copy link
Copy Markdown
Owner

Closes #51

Adds .claude/skills/run-windows/SKILL.md (hard rules first, then launch, checks and report format) and gitignores .claude/worktrees/.

First live run on the Windows PC (character docked, EVE windowed)

Screenshots stay on the PC, outside the repo.

Check Result
App starts from a fresh clone with venv + requirements.txt (throwaway ABYSS_TRACKER_DB) Pass
ESI token refreshes with no further help Pass. Restarted app was already signed in from an earlier SSO sign-in; wallet fetched with no browser step. SSO sign-in itself was not re-run, and Gmail was not opened.
Location and run detection while docked Pass. ESI location read as not in the Abyss; detection on for ~1.5 min, no timer start or run prompt.
Wallet stat matches the client Pass. App showed 2288.49M; client reads 2288 (user-confirmed).
Session, Incursion, History, pocket pages open and screenshotted Pass
Pocket window stays on top beside windowed EVE Pass by window geometry: topmost flag set, above the client in z-order when overlapping. EVE is masked in screenshots, so no combined image.

The release .exe and the active-ship/cargo check (#52) are out of scope. No input was sent to the EVE launcher or client.

@Fazorath Fazorath left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Review against #51's hard rules and acceptance criteria.

Good: every hard rule kept: no input to EVE, character docked, a throwaway ABYSS_TRACKER_DB, screenshots kept on the PC, Gmail not opened. Each acceptance criterion has a pass/fail row. The skill matches the code: EVE_CLIENT_ID is required (sso.py), sign-in is on the Settings page, and python -m unittest discover matches CI.

Fixed in 89fb801:

  1. Check 6 asked for a screenshot of the pocket window over EVE, but EVE is masked in computer-use screenshots and the run passed by window geometry. Step 6 now describes the geometry check, including Unpin clearing the topmost flag.
  2. Check 3 asked for a sensible station/system name, which conflicted with the note to print only in_abyss. It now reports only "not in the Abyss" and keeps location names out of reports and PRs, because the repo is public.
  3. The hard rules now say outright that reading EVE's window (GetWindowRect, IsIconic, its flags) is fine, and that SetWindowPos, SetForegroundWindow, ShowWindow and window messages must never target an EVE window handle.
  4. The SSH line said Arcanum handles headless jobs, but the PC's OpenSSH server is off, so the skill now says to run tests from the desktop terminal.

Not covered by this run: the SSO sign-in step and the Gmail code path. An old stored token proved the refresh. Signing out and running check 2 once would cover the sign-in.

@Fazorath
Fazorath merged commit e973477 into main Oct 26, 2026
7 checks passed
@Fazorath
Fazorath deleted the issue-51-windows-test-rig branch October 26, 2026 17: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.

Windows test rig: launch and screenshot AbyssTracker on the PC with a live EVE character

1 participant