Review of "Dynamic Music Pill" version 1.5.2 (53)

Details Page Preview

An elegant, pill-shaped music player for your desktop. Features a smooth audio visualizer, scrolling text, and seamless integration with Dash-to-Dock and the Top Panel. Comes with a Pop-Up menu as well.

Extension Homepage
https://github.com/Andbal23/dynamic-music-pill

No comments.

Diff Against

Files

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

Shexli (experimental) error 1 warning 1 manual_review 1

Shexli found 3 issues that may need reviewer attention.

EGO-X-004 warning

extensions should avoid synchronous file IO in shell code

Shell code should avoid synchronous file IO APIs like `GLib.file_get_contents()` and `Gio.File.load_contents()`.

File Operations

  • src/blurHelper.js:83
    GLib.file_get_contents(metaPath)

EGO-A-005 manual_review

extensions should not access the clipboard directly

Direct clipboard access via `St.Clipboard.get_default()` requires reviewer scrutiny.

Review Guidelines

  • src/uiExpandedPlayer.js:2087
    St.Clipboard.get_default()

EGO-M-004 error

metadata shell-version must only include plausible stable releases and at most one development release

Field `shell-version` contains invalid values, more than one development release, or implausible future releases.

metadata.json must be well-formed

  • metadata.json
    shell-version=['50', '51']

All Versions

Version Status
1.6 (56) Unreviewed
1.6 (55) Rejected
1.6 (54) Rejected
1.5.2 (53) Rejected
1.5.0 (52) Active
1.4.0 (51) Active
1.3.0 (50) Active
1.3.0 (49) Rejected
1.2.1 (48) Active
1.2.0 (47) Active
1.2.0 (46) Rejected
1.1.0 (45) Rejected
1.1.0 (44) Active
1.1.0 (43) Active
1.0.0 (42) Active
1.0.0 (41) Active
1.0.0 (40) Rejected
1.0.0 (39) Rejected
1.0.0 (38) Rejected
1.0.0 (37) Rejected
36 Rejected
35 Rejected
34 Rejected
33 Rejected
32 Rejected
31 Rejected
30 Rejected
29 Active
28 Rejected
27 Active
26 Active
25 Active
24 Rejected
23 Active
22 Rejected
21 Rejected
20 Active
19 Active
18 Rejected
17 Active
16 Active
15 Active
14 Rejected
13 Rejected
12 Rejected
11 Rejected
10 Rejected
9 Rejected
8 Rejected
7 Rejected
6 Rejected
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
Didn't review. Search for these issues and remove all the unnecessary ones: - https://gjs.guide/extensions/review-guidelines/best-practices.html#avoid-unnecessary-try-catch-wrappers - https://gjs.guide/extensions/review-guidelines/best-practices.html#avoid-unnecessary-checks Please note that since many AI generated extensions are being submitted here, we are very busy these days. We have very strict rules regarding AI. Please review your code before sending it here: - [EGO Review Guidelines: AI](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#extensions-must-not-be-ai-generated) - [Extensions Best Practices Guidelines](https://gjs.guide/extensions/review-guidelines/best-practices.html#submissions-require-maintainership)
JustPerfection posted a review
BTW, you should search for those issues in the entire package, not just related to this diff.
andbal posted a review
Okay, thanks for the feedback.