Skip to content

Launch Windows desktop without a console window - #95

Merged
trissim merged 1 commit into
mainfrom
codex/brand-assets-gui-launcher
Jul 31, 2026
Merged

Launch Windows desktop without a console window#95
trissim merged 1 commit into
mainfrom
codex/brand-assets-gui-launcher

Conversation

@trissim

@trissim trissim commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • Declare openhcs-gui as a genuine GUI entry point instead of a console script.
  • Add the GUI entry-point name to the shared installer contract and validate it against package metadata.
  • Make the Windows desktop shortcut launch the installed GUI-subsystem executable directly.
  • Keep the stable PowerShell adapter and openhcs.exe mcp route unchanged for MCP clients.
  • Extend Windows CI to verify the shortcut target, empty argument list, and PE GUI subsystem.

Why

The desktop shortcut eventually launched a console-subsystem executable, so a terminal window could remain open for the lifetime of the GUI. Additional hidden-window flags would not correct the executable classification. Packaging metadata is the authoritative boundary for whether Windows builds a console or GUI launcher.

User impact

Launching OpenHCS from its Windows desktop shortcut opens the existing startup-progress window and application without retaining a terminal window. MCP registration and updater restart behavior remain intact.

Validation

  • 53 focused installer and desktop-updater tests passed.
  • A wheel build placed openhcs-gui only in [gui_scripts] and retained OpenHCS/MCP commands in [console_scripts].
  • TOML and GitHub Actions workflow parsing passed.
  • git diff --check and Ruff passed.
  • Windows CI performs the native PowerShell and PE-subsystem acceptance checks.

@trissim
trissim marked this pull request as ready for review July 31, 2026 01:18
Copilot AI review requested due to automatic review settings July 31, 2026 01:18
@trissim
trissim merged commit b8edc1a into main Jul 31, 2026
66 of 67 checks passed

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.

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