Skip to content

feat(gui): add managed GUI app install lifecycle#48

Merged
ian-pascoe merged 12 commits intomainfrom
feat/gui-app-integration
Feb 24, 2026
Merged

feat(gui): add managed GUI app install lifecycle#48
ian-pascoe merged 12 commits intomainfrom
feat/gui-app-integration

Conversation

@ian-pascoe
Copy link
Contributor

Summary

  • add first-class gui_apps manifest schema and validation in crosspack-core for GUI app metadata, file associations, and protocol handlers
  • implement installer GUI asset projection/exposure/state tracking with collision-safe ownership keys and deterministic stale cleanup on upgrade/uninstall
  • wire CLI install preflight + outcome reporting for GUI assets, and update architecture/install/manifest docs to describe the lifecycle and .gui sidecar state

Verification

  • cargo fmt --all
  • cargo clippy -p crosspack-core -p crosspack-installer -p crosspack-cli --all-targets -- -D warnings
  • cargo test --workspace

Introduce manifest GUI app metadata and wire installer/CLI preflight, exposure, and stale cleanup so GUI assets are handled deterministically with collision safety across install, upgrade, and uninstall.
Split GUI launcher rendering into per-target cfg functions so Windows and Linux builds no longer report unreachable code under -D warnings while preserving platform-specific launcher output.
@ian-pascoe
Copy link
Contributor Author

@codex thoroughly review for bugs and security issues

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a29c6e106a

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ian-pascoe
Copy link
Contributor Author

@codex thoroughly review for bugs and security issues

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6384e2b059

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ian-pascoe ian-pascoe merged commit 542b8d9 into main Feb 24, 2026
4 checks passed
@ian-pascoe ian-pascoe deleted the feat/gui-app-integration branch February 24, 2026 17:55
@crosspack-bot crosspack-bot bot mentioned this pull request Feb 24, 2026
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.

1 participant