Review of "ClipFlow Pro" version 1.3.5 43.44 (22)

Details Page Preview

A modern, powerful clipboard manager for GNOME Shell with intelligent organization, beautiful UI, and comprehensive history management.

Extension Homepage
https://github.com/nickotmazgin/clipflow-pro

No comments.

Diff Against

Files

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

All Versions

Version Status
1.3.7 45.47 (27) Rejected
1.3.7 (26) Rejected
1.3.6 45.47 (25) Rejected
1.3.6 (24) Rejected
1.3.5 45.47 (23) Rejected
1.3.5 43.44 (22) Rejected
1.3.4 45.47 (21) Rejected
1.3.4 43.44 (20) Rejected
1.3.3 (19) Active
1.3.3 43.44 (18) Rejected
1.3.1 43.44 (17) Rejected
1.3.1 (16) Inactive
1.3.0 (15) Inactive
1.3.0 43.44 (14) Rejected
1.3.0 (13) Rejected
1.3.0 (12) Rejected
1.2.17 (11) Inactive
1.2.16 (10) Inactive
1.2.15 (9) Rejected
1.2.14 (8) Rejected
1.2.13 (7) Inactive
1.2.11 (6) Rejected
1.2.10 (5) Rejected
1.2.9 (4) Rejected
1.2.8 (3) Rejected
1.2.6 (2) Rejected
1.2.3 (1) Rejected

Previous Reviews on this Version

Ghost posted a review
Reviewer Notes (1.3.5) Removed “stylesheet” from metadata.json (both zips; packaging enforces removal). No Gtk in the Shell process; Gtk only in prefs.js (separate prefs process). File reads in the Shell process are async (history/import use Gio.File.load_contents_async). Logging gated behind enable-debug-logs; non‑critical info logs under a debug guard; recurring warnings throttled. Removed all spawn‑based clipboard fallbacks (no xclip/xsel/exec); Shell APIs only. UI simplifies and de‑duplicates actions: Single “Actions” submenu at the bottom. Sort and Filter moved into Actions; filter also applied to Enhanced mode. Added toggles in Actions: “Capture PRIMARY Selection” and “Pause Monitoring”. Import/Export improvements: Export JSON includes id/text/timestampUnix/pinned/starred/sensitive. Import is async and restores pinned/starred and original timestamp when present. Folder is configurable in prefs (io-folder); runtime fallback: Desktop → Downloads → Home. Packaging is EGO‑compliant: Two zips under the same UUID for 43–44 and 45–47 (correct shell-version arrays). Flat structure; only XML schemas included; no compiled schemas. Cleanups: Removed duplicate root icon; use icons/ only. Compiled schema excluded from builds and zips (and removed from repo index). 45+ package aligned to the same structure and rules as 43–44. Versions for this submission: 43–44 zip: version: 30, version-name: 1.3.5 43.44 45–47 zip: version: 30, version-name: 1.3.5 45.47
JustPerfection rejected
Hey Nick, 1. There are still many unnecessary checks with `=== 'function'`. Why `print` needs to be checked whether it is a function (line 16 `extension.js`)?! 2. `console` is available in 43 and 44. No need to check. 3. While I asked for removing unnecessary try-catch wrappers before, still there are many of them here. 4. As mentioned before, the code cannot pass the review with unnecessary lines like this. But feel free to ask for other reviewers to review the code, simply by asking in this review page.