Skip to content

Feat/flatpak#265

Open
claytonlin1110 wants to merge 4 commits into
vouchdev:testfrom
claytonlin1110:feat/flatpak
Open

Feat/flatpak#265
claytonlin1110 wants to merge 4 commits into
vouchdev:testfrom
claytonlin1110:feat/flatpak

Conversation

@claytonlin1110

Copy link
Copy Markdown
Contributor

What changed

Adds desktop/flatpak/ with a com.vouchdev.vouch Flatpak manifest on org.freedesktop.Platform//23.08, a vouch-review-ui launcher, .desktop entry, AppStream metainfo, hicolor icons, Flathub submission template, build/validate scripts, tests/test_flatpak*.py, a flatpak CI workflow, and docs/desktop-flatpak.md. Also adds a [desktop] optional extra (alias of [web]) and a make flatpak target.

Closes #211

Why

Fedora/Arch users without snap need a sandboxed Linux install path for the review console (#211). Flatpak covers that audience with --filesystem=home for .vouch/ trees under $HOME and --share=network for the localhost review-ui server. Flathub submission assets are included so the app can be published as com.vouchdev.vouch once approved upstream.

What might break

Nothing for existing .vouch/ directories or kb.* behaviour. This is additive packaging only — no on-disk layout, protocol, or runtime changes for pipx/CLI users.

VEP

Not required — no object model, kb.* method, on-disk layout, bundle format, or audit-log shape changes.

Tests

  • make check passes locally (lint + mypy + pytest)
  • New / changed behaviour has a test (tests/test_flatpak.py, tests/test_flatpak_validate_extended.py)
  • CHANGELOG.md updated under ## [Unreleased]

@coderabbitai

coderabbitai Bot commented Jun 23, 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: Pro Plus

Run ID: 6605c1d8-0287-4cf3-8b58-4fb95e3f8f73

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@claytonlin1110 claytonlin1110 changed the base branch from main to test June 23, 2026 01:16
@claytonlin1110

Copy link
Copy Markdown
Contributor Author

@plind-junior Please review

@github-actions github-actions Bot added docs documentation, specs, examples, and repo guidance ci github actions and automation packaging packaging, build metadata, and make targets tests tests and fixtures size: XL 1000 or more changed non-doc lines labels Jun 29, 2026
@claytonlin1110

Copy link
Copy Markdown
Contributor Author

@plind-junior Please review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci github actions and automation docs documentation, specs, examples, and repo guidance packaging packaging, build metadata, and make targets size: XL 1000 or more changed non-doc lines tests tests and fixtures

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(desktop): flatpak package

1 participant