Skip to content

feat: add screensaver extension pack - #30

Merged
keys-i merged 1 commit into
mainfrom
screensaver
Aug 18, 2026
Merged

keys-i merged 1 commit into
mainfrom
screensaver

Conversation

@keys-i

@keys-i keys-i commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Summary

Adds the typed Scripts UI contract for the native screensaver command from keys-i/tools#7. The wizard exposes snapshot/open, all six owned scenes, conditional Git/GIF inputs, deterministic seeds, weather controls, frame/FPS bounds, and plain/JSON output. --night is a checkbox option, so the Textual boolean-select crash path is not reintroduced.

Part of #23

Scope

  • bin/
  • dev/
  • lib/
  • sys/
  • Documentation or repository settings

Testing

Commands run:

bash -n run.sh
bash run.sh --self-test
uv run dev/gui.py --self-test
git diff --check

All passed; GUI self-test covered 16 scripts, layout snapshots, keyboard behavior, and fallbacks. Mandatory fallback review returned VERDICT: PASS.

Tested systems and interpreter versions:

macOS 26.5.2 arm64
Python 3.14 via uv
Textual 8.2.8

Checklist

  • The change is focused and contains no unrelated files.
  • The code is clean and comments explain intent or risk.
  • New or changed behaviour has a repeatable test or documented manual check.
  • Help text and dependencies are current.
  • Destructive behaviour and elevated permissions are clearly disclosed.
  • No credentials, private data, or machine-specific paths are included.

Reviewer Notes

Stacked on science and depends on keys-i/tools#7. The pack contains no implementation or download logic; it needs the unpublished native screensaver binary on PATH. Rollback is a revert of f9506a1.

@keys-i keys-i mentioned this pull request Aug 18, 2026
8 of 11 tasks
@keys-i
keys-i changed the base branch from science to main August 18, 2026 22:49
@keys-i keys-i closed this Aug 18, 2026
@keys-i keys-i reopened this Aug 18, 2026
@keys-i
keys-i marked this pull request as ready for review August 18, 2026 22:51
@keys-i
keys-i merged commit accc5bc into main Aug 18, 2026
25 checks passed
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