Skip to content

Fix Sixel rendering in Windows local sessions - #41

Merged
Marc-André Moreau (mamoreau-devolutions) merged 3 commits into
masterfrom
copilot/xkcd-show-image-render
Sep 24, 2026
Merged

Marc-André Moreau (mamoreau-devolutions) merged 3 commits into
masterfrom
copilot/xkcd-show-image-render

Conversation

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor

Summary

  • Bundle Microsoft's pinned ConPTY runtime with the matching OpenConsole.exe hosts so local Windows sessions preserve Sixel DCS payloads rather than silently dropping them through the system console host.
  • Default DECSDM to scrolling/cursor-anchored mode and convert Sixel image geometry from physical pixels to DIPs on scaled displays.
  • Deploy the hosts for app publishes and Control NuGet consumers; update third-party notice and protocol/release docs.

Validation

  • Core advanced-protocol tests: 50 passed.
  • Renderer tests: 40 passed.
  • Connection contract tests, including the live ConPTY Sixel transport regression: 49 passed.
  • Visually confirmed Get-XKCD -Show renders the comic at the correct size and position.
  • Verified a win-x64 non-AOT publish and both RID-less and RID-specific NuGet-consumer host layouts.

NativeAOT publishing was not verified locally because the required C++ linker workload is unavailable; CI should cover it.

Bundle the Microsoft ConPTY runtime and hosts, correct DECSDM defaults and high-DPI image geometry, and cover the complete rendering path with regression tests.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Normalize Windows separators correctly so ConPTY host copies occupy distinct WiX directories and receive unique component GUIDs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Wait for the PowerShell prompt and real command result before exiting, and allow scheduling headroom while retaining the Azure socket-close deadline assertion.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

This branch was successfully deployed

1 active deployment
publish-dry-run — 7fc4a774 Deployed Sep 24, 2026 by mamoreau-devolutions via MSIX packages #257
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant