Menu ▾ ▴

#460 chore(linux): fuller AppStream metadata for software centers

closed
nobody
None
5 days ago
5 days ago
Anonymous
No

Originally created by: kristofferR

GNOME Software and KDE Discover read Chatpane's AppStream metainfo, but it had no search keywords, categories or branding, and its release notes stopped at 1.13.0.

This adds to app.chatpane.Chatpane.metainfo.xml:

  • categories and search keywords (Messenger, Facebook, Chat, Message)
  • light/dark branding colors
  • a paragraph saying a Facebook account is required, calls aren't available on Linux yet (see [#345]), and Chatpane is not affiliated with or endorsed by Meta
  • release notes for 1.14.0 through 3.1.0, summarized from the curated GitHub releases

The deb/rpm desktop template gets the same Keywords=, and AGENTS.md asks each release to add a metainfo entry.

appstreamcli validate --pedantic passes; its only note (uppercase letters in the ID) predates this change.

Summary by CodeRabbit

  • Documentation
  • Updated Linux app-store information to clarify that a Facebook account and internet connection are required, voice and video calls are unavailable on Linux, and Chatpane is not affiliated with Meta.
  • Added app categories, searchable keywords, and light and dark brand colors.
  • Added release notes for versions 1.14.0 through 3.1.0.
  • Chores
  • Added a release checklist step to include a summary in the app-store metadata before tagging a release.

Related

Tickets: #345

Discussion

  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: coderabbitai[bot]

    Action performed

    Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.
     
  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: chatgpt-codex-connector[bot]

    Codex Review Summary: ✅ Completed This comment shows the latest Codex review activity on this pull request. | Review | Status | Commit | Review trigger | | --- | --- | --- | --- | | 📝 **Code Review** | ✅ **Completed** <relative-time datetime="2026-11-09T15:00:11.917739Z">2026-11-09T15:00:11.917739Z</relative-time> | `75d18be` | Manual request |
    ℹ️ About Codex in GitHub
    [Your team has set up Codex to review pull requests in this repo](https://chatgpt.com/codex/cloud/settings/general). Reviews are triggered when you - Open a pull request for review - Mark a draft as ready - Comment "@codex review" or "@codex security review". Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.
     
  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: chatgpt-codex-connector[bot]

    Codex Review: Didn't find any major issues. You're on a roll.

    Reviewed commit: 0fc2dfa20f

    ℹ️ About Codex in GitHub
    [Your team has set up Codex to review pull requests in this repo](https://chatgpt.com/codex/cloud/settings/general). 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".
     
  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: coderabbitai[bot]

    <picture>Explain Change →</picture>

    Walkthrough

    Linux AppStream metadata now describes account requirements, call availability, network access, and brand colors. It adds categories, search keywords, and release notes. The desktop entry adds search keywords, and the release checklist adds an instruction for updating AppStream release notes before tagging. **Changes: Linux application metadata** |Layer / File(s)|Summary| |:---|:---| |**Application description and discovery metadata**
    `src-tauri/linux/app.chatpane.Chatpane.metainfo.xml`, `src-tauri/linux/chatpane.desktop.hbs`|The AppStream metadata describes the Facebook account requirement, Linux call availability, and Chatpane’s independence from Meta. It adds categories, keywords, an always-required internet declaration, and light and dark brand colors. The desktop entry adds Messenger, Facebook, Chat, and Message keywords.| |**Release notes and release checklist**
    `src-tauri/linux/app.chatpane.Chatpane.metainfo.xml`, `AGENTS.md`|AppStream release notes are added for versions 3.1.0 through 1.14.0. The release checklist instructs contributors to add a short draft summary to the metainfo file before tagging.| **Estimated code review effort:** 2 (Simple) | ~10 minutes **Change:** Other **Possibly related issues** - Issue 345 describes the Linux WebRTC limitation. This PR documents that Linux calls are unavailable. **Possibly related PRs** - [[kristofferR/Chatpane#456](https://github.com/kristofferR/Chatpane/issues/456)](https://github.com/kristofferR/Chatpane/pull/456): It updates the Linux metadata integration and the AppStream and desktop-entry paths used by this PR. - [[kristofferR/Chatpane#447](https://github.com/kristofferR/Chatpane/issues/447)](https://github.com/kristofferR/Chatpane/pull/447): Its chat export changes are described in the AppStream release notes added by this PR.
    Merge Risk: <picture></picture> Minimal · up to 0fc2d

    The change updates Linux discovery metadata and release notes. No merge-blocking risk was identified.
    Pre-merge checks: <picture>Passed</picture> 3 · <picture>Failed</picture> 1

    <picture>Failed</picture> Failed checks: 1 warning | Check name | Status | Explanation | Resolution | | :--------------- | :----------------------------------------------------------------------------------------------: | :-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | :------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ | | Description check | ![Warning](https://storage.googleapis.com/coderabbit_public_assets/pre-merge-checks-warning.svg) | The description covers the change well. It lists the metainfo additions (categories, search keywords, light/dark branding colors, a Facebook account and Meta non-affiliation paragraph, and release not… | Add the template sections to the PR description. Use a '## Summary' heading for the problem and change. Add an 'AI models used:' line that names every model used to create or edit code, tests, or PR text, with its version, or write 'None'. … |
    <picture>Passed</picture> Passed checks: 3 passed

    | Check name | Status | Explanation | | :------------------------ | :--------------------------------------------------------------------------------------------: | :-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | Title check | ![Passed](https://storage.googleapis.com/coderabbit_public_assets/pre-merge-checks-passed.svg) | The title names a real part of the change: it adds fuller AppStream metadata (categories, keywords, branding, release notes) to the Linux metainfo file for software centers. It is a short, specific co… | | Linked Issues check | ![Passed](https://storage.googleapis.com/coderabbit_public_assets/pre-merge-checks-passed.svg) | Check skipped because no linked issues were found for this pull request. | | Out of Scope Changes check | ![Passed](https://storage.googleapis.com/coderabbit_public_assets/pre-merge-checks-passed.svg) | Check skipped because no linked issues were found for this pull request. |
    Full details: Title check

    **Explanation** The title names a real part of the change: it adds fuller AppStream metadata (categories, keywords, branding, release notes) to the Linux metainfo file for software centers. It is a short, specific conventional-commit style title. The scope is accurate, though it does not mention the release notes or the Facebook/Meta disclosure text.
    Full details: Description check

    **Explanation** The description covers the change well. It lists the metainfo additions (categories, search keywords, light/dark branding colors, a Facebook account and Meta non-affiliation paragraph, and release notes for 1.14.0 through 3.1.0), the matching desktop template Keywords= change, and an appstreamcli validation result. It does not follow the repository template. It has no '## Summary' heading, no 'AI models used' field, and no 'Reasoning levels' field. The author did not state whether AI helped write the change, and the template requires that answer from contributors other than the repository owner. **Resolution** Add the template sections to the PR description. Use a '## Summary' heading for the problem and change. Add an 'AI models used:' line that names every model used to create or edit code, tests, or PR text, with its version, or write 'None'. Add a 'Reasoning levels' line for each AI model, or write 'Reasoning levels: Unavailable (not exposed by tool)'. Do not guess these values.
    Finishing Touches

    Generate unit tests (beta)

    - [ ] Commit to this branch - [ ] Create a new PR
    • [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

    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

    - [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai) - [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai) - [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai) - [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)
     
  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: kristofferR

    @coderabbitai review

     
  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: kristofferR

    @codex review for consequential defects introduced by this PR; do not expand its original scope with unrelated refactors, features, or speculative hardening

     
  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: coderabbitai[bot]

    Action not completed

    Review rate limited. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.
     
  • Anonymous

    Anonymous - 5 days ago

    Ticket changed by: kristofferR

    • status: open --> closed
     
  • Anonymous

    Anonymous - 5 days ago

    Originally posted by: chatgpt-codex-connector[bot]

    Codex Review: Didn't find any major issues. Breezy!

    Reviewed commit: 75d18be29e

    ℹ️ About Codex in GitHub
    [Your team has set up Codex to review pull requests in this repo](https://chatgpt.com/codex/cloud/settings/general). 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".
     

Log in to post a comment.