Review of "Clipman Clipboard Monitor" version 1

Details Page Preview

Native Wayland clipboard change detection for the Clipman clipboard manager. Listens to Meta.Selection owner-changed signals and forwards clipboard content to the Clipman daemon via D-Bus. Provides paste simulation and window positioning services. Requires the Clipman daemon (clipman.service) to be running.

Extension Homepage
https://github.com/MohammedEl-sayedAhmed/clipman

No comments.

FAQ

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Version Status
4 Active
3 Active
2 Active
1 Rejected

Previous Reviews on this Version

JustPerfection waiting for author
Is this code generated by AI? If so, we have a rule for that: [EGO Review Guidelines: AI](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#extensions-must-not-be-ai-generated)
MohammedEl-sayedAhmed posted a review
No, this is not AI-generated. I wrote this extension as part of Clipman, my clipboard manager. I can explain every part of it: - owner-changed signal on Meta.Selection detects clipboard changes without polling - D-Bus interface exposes SimulatePaste (virtual keyboard via Clutter) and MoveWindowToCursor for the Python daemon to call - _getClipboardText tries MIME types in priority order for XWayland compatibility - _sendToDaemon forwards clipboard content over session D-Bus to the Python backend Happy to clarify anything specific.
JustPerfection posted a review
Where is the git repo? and why aren't you using `St.Clipboard.get_default().get_text()` instead of `_getClipboardText()`?
MohammedEl-sayedAhmed posted a review
Repo: https://github.com/MohammedEl-sayedAhmed/clipman I use get_content() with a MIME type fallback chain instead of get_text() for XWayland compatibility. get_text() only reads text/plain;charset=utf-8, but many XWayland apps (VSCode, Electron) only expose UTF8_STRING or STRING. The fallback chain in _getClipboardText() tries text/plain;charset=utf-8 → UTF8_STRING → text/plain → STRING to ensure clipboard content is captured from all apps.
JustPerfection rejected
1. D-Bus interface name should start with `org.gnome.Shell.Extensions`. 2. Remove unnecessary try-catch wrapper (line 173 `extension.js`). 3. Please add GitHub or GitLab url to the `metadata.json`: [EGO Review Guidelines: url](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#metadata-json-must-be-well-formed) 4. Add a more descriptive description and also mention the dependencies in it. You can also add `donations` to the `metadata.json`, so people can donate to you if they want: [Extension Anatomy: donations](https://gjs.guide/extensions/overview/anatomy.html#donations) If you need any help with your extension you can ask us on: - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions