feat(modules): add windows-rdp-keepalive module#700
feat(modules): add windows-rdp-keepalive module#700Quantum-Forge-Code wants to merge 1 commit intocoder:mainfrom
Conversation
Adds a new module that monitors active RDP connections and bumps workspace activity to prevent automatic shutdown during active RDP sessions. Features: - Configurable check interval (default 30s) - Detection via qwinsta and WMI fallback - Logging for debugging - Can be enabled/disabled Closes coder#200
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 993fc65195
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| assert { | ||
| condition = length(coder_script.rdp-keepalive) == 0 |
There was a problem hiding this comment.
Rename hyphenated resource reference
In Terraform expressions, attribute traversal uses identifiers, so coder_script.rdp-keepalive is parsed as coder_script.rdp - keepalive rather than a single resource name. That makes this assertion fail to evaluate (and any future references to the resource will error) because subtraction on a resource object is invalid and keepalive is undefined. Use an underscore in the resource label (e.g., rdp_keepalive) and update references accordingly to avoid plan/test failures.
Useful? React with 👍 / 👎.
Description
This PR adds a new
windows-rdp-keepalivemodule that monitors active RDP connections and extends workspace session timeouts while RDP is connected.Problem
The existing activity bump system only tracks SSH, VSCode, JetBrains, and reconnecting PTY connections. Windows users connecting via RDP for GUI applications may have their workspaces time out during active work.
Solution
A PowerShell script that:
Features
qwinstawith WMI fallback for RDP detection%TEMP%\rdp-keepalive.logUsage
Checklist
Closes #200
Bounty submission 💎 $150
Generated with Claude using OpenClaw