Super+N logs out instantly with no confirmation #60

Merged

Breadway merged 1 commit from fix/bos-42-confirm-session-exit into main

2026-09-30 10:58:32 +08:00

Breadway commented

2026-09-30 02:37:18 +08:00

Owner

Verified with JSON parsing, the Lua keybind loader stub, and git diff --check. A full ISO build was not run.

The separate bos.breadway.dev keybind table still needs a reviewable change. Its local source has no Git remote, and no bos-site Forgejo repository is available. This PR covers only the reviewable BOS repository portion.

Move the Hyprland exit action to SUPER+Shift+Escape, and update the README shortcut table and Unreleased changelog. The in-session breadhelp cheatsheet reads the same binds.json source, so it reflects the new shortcut automatically. Verified with JSON parsing, the Lua keybind loader stub, and git diff --check. A full ISO build was not run. The separate bos.breadway.dev keybind table still needs a reviewable change. Its local source has no Git remote, and no bos-site Forgejo repository is available. This PR covers only the reviewable BOS repository portion.

Breadway added 1 commit

2026-09-30 02:37:18 +08:00

fix: require deliberate session exit shortcut ed628f22b8

Breadway referenced this pull request

2026-09-30 02:37:37 +08:00

Super+N logs out instantly with no confirmation #42

Breadway commented

2026-09-30 03:41:25 +08:00

Author

Owner

Existing pull request: #60. This card tracks the partial BOS-42 implementation and remains TODO to match BOS-42. BOS-42 is still blocked on the separate bos.breadway.dev keybind table because no reviewable bos-site repository is available.

Existing pull request: https://git.breadway.dev/Breadway/bos/pulls/60. This card tracks the partial BOS-42 implementation and remains TODO to match BOS-42. BOS-42 is still blocked on the separate bos.breadway.dev keybind table because no reviewable bos-site repository is available.

Breadway commented

2026-09-30 10:55:00 +08:00

Author

Owner

Existing PR #60 covers the reviewable BOS repository scope for this card: https://git.breadway.dev/Breadway/bos/pulls/60. It moves the Hyprland exit binding to SUPER+Shift+Escape and updates the README shortcut table and Unreleased changelog. I verified the PR branch binds.json through Forgejo: it parses and contains one exit action on SUPER+Shift+Escape. The PR records the Lua loader stub and git diff checks; no CI is configured. I could not create a separate isolated clone here because the SSH proxy config has unsafe ownership and direct HTTPS DNS resolution failed. The separate bos.breadway.dev keybind table remains outside this PR, as noted in the issue body.

Existing PR #60 covers the reviewable BOS repository scope for this card: [https://git.breadway.dev/Breadway/bos/pulls/60](https://git.breadway.dev/Breadway/bos/pulls/60). It moves the Hyprland exit binding to SUPER+Shift+Escape and updates the README shortcut table and Unreleased changelog. I verified the PR branch binds.json through Forgejo: it parses and contains one exit action on SUPER+Shift+Escape. The PR records the Lua loader stub and git diff checks; no CI is configured. I could not create a separate isolated clone here because the SSH proxy config has unsafe ownership and direct HTTPS DNS resolution failed. The separate bos.breadway.dev keybind table remains outside this PR, as noted in the issue body.

Breadway commented

2026-09-30 10:58:09 +08:00

Author

Owner

Reviewed, looks ready to merge. Acceptance mapping:

The Hyprland exit action moves from SUPER+N to SUPER+Shift+Escape in iso/airootfs/etc/skel/.config/hypr/binds.json:25.
The README shortcut table reflects the new binding in README.md:338.
The Unreleased changelog records the change in CHANGELOG.md:26-27.

The in-session cheatsheet uses the same binds.json source, so it does not need a separate edit. The PR is mergeable and the diff is limited to those three files. Forgejo reports no CI statuses for the head branch; the issue records JSON parsing, Lua loader stub, and diff checks as verified. The separate site keybind table remains blocked under BOS-42.

I am leaving the merge to the user because BOS is a release-sensitive repository.

Reviewed, looks ready to merge. Acceptance mapping: 1. The Hyprland exit action moves from SUPER+N to SUPER+Shift+Escape in `iso/airootfs/etc/skel/.config/hypr/binds.json:25`. 2. The README shortcut table reflects the new binding in `README.md:338`. 3. The Unreleased changelog records the change in `CHANGELOG.md:26-27`. The in-session cheatsheet uses the same binds.json source, so it does not need a separate edit. The PR is mergeable and the diff is limited to those three files. Forgejo reports no CI statuses for the head branch; the issue records JSON parsing, Lua loader stub, and diff checks as verified. The separate site keybind table remains blocked under BOS-42. I am leaving the merge to the user because BOS is a release-sensitive repository.

Breadway merged commit 4bbbf0ddba into main

2026-09-30 10:58:32 +08:00

Breadway referenced this pull request from a commit

2026-09-30 10:58:32 +08:00

Merge pull request 'Super+N logs out instantly with no confirmation' (#60) from fix/bos-42-confirm-session-exit into main

Breadway commented

2026-09-30 10:58:52 +08:00

Author

Owner

Independent review passed. PR #60 merged as 4bbbf0ddba330450ea3f8e465f99fc771228d8f7; no CI is configured. Marking this PR card DONE. Parent BOS-42 remains TODO for the separate, unreviewable website shortcut table.

Independent review passed. PR #60 merged as `4bbbf0ddba330450ea3f8e465f99fc771228d8f7`; no CI is configured. Marking this PR card DONE. Parent BOS-42 remains TODO for the separate, unreviewable website shortcut table.

Breadway referenced this pull request

2026-09-30 18:52:15 +08:00

Super+N logs out instantly with no confirmation #42