Skip to content

feat(gaming): Coordinate display profiles and optional USB forwarding - #29

Merged
tyvsmith merged 2 commits into
codex/omarchy-migration-conflictsfrom
codex/hyprmoncfg-sessions
Oct 20, 2026
Merged

tyvsmith merged 2 commits into
codex/omarchy-migration-conflictsfrom
codex/hyprmoncfg-sessions

Conversation

@tyvsmith

Copy link
Copy Markdown
Owner

Summary

  • coordinate Desktop, Gaming and client-sized streaming profiles through hyprmoncfg, with explicit game and Sunshine lifecycle hooks
  • forward the Shield controller through optional USB/IP hooks while allowing the same Steam application to stream from clients without controllers
  • track the deployed Sunshine applications and capture settings directly; keep credentials and pairing state local

Why

Local gaming and streaming need different monitor settings, and ending a stream must preserve HDR when a game remains active. Explicit hooks coordinate those settings and release the imported controller on Quit App or Sunshine failure.

Area Review focus
home/dot_local/bin/ profile precedence and USB ownership (start here)
home/dot_config/ monitor profiles, ScopeBuddy hooks and deployed Sunshine setup
home/run_onchange_5* session service setup and initial USB privilege installation

Validation

  • exercise installed hyprmoncfg and ScopeBuddy against disposable fixtures
  • cover controller-free clients, USB ownership, profile transitions and cross-profile chezmoi rendering
  • test Shield gameplay, USB reconnect and Quit App cleanup
  • force Sunshine exit and inspect cleanup while preserving Gaming

Reviewer notes

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 151bf7ab-a924-45df-b39e-485f8301d1d0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@gitar-bot

gitar-bot Bot commented Oct 20, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🔴 High risk · The NOPASSWD rule permits arbitrary root usbip attach/detach arguments, so a user could alter USB/IP attachments beyond the hook’s ownership checks.

Coordinates Desktop, Gaming, and client-sized streaming monitor profiles through hyprmoncfg, with explicit game and Sunshine lifecycle hooks that manage optional USB/IP forwarding for the Shield controller. No issues found.

Verification

Rules No rules evaluated

Functional validation Not enabled

Set up

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source

@tyvsmith
tyvsmith marked this pull request as ready for review October 20, 2026 17:31
Copilot AI balanced review requested due to automatic review settings October 20, 2026 17:31

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@tyvsmith
tyvsmith merged commit 32b281f into codex/omarchy-migration-conflicts Oct 20, 2026
7 checks passed
tyvsmith added a commit that referenced this pull request Oct 20, 2026
* fix(omarchy): Preserve migration fixes when applying dotfiles

* fix(pacman): Prefer CachyOS repositories over Omarchy

* feat(gaming): Coordinate display profiles and optional USB forwarding (#29)

* feat(gaming): Coordinate display profiles and optional USB forwarding

* fix(hypr): Default Steam Big Picture to fullscreen
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.

2 participants