Review "Polifix" version 1 - GNOME Shell Extensions

Review of "Polifix" version 1

Details Page Preview

Polifix

Polish any text from anywhere. Press a shortcut, pick a transformation, and the result streams in and lands on your clipboard. Talks to your own AI provider with your own API key - Anthropic, Gemini, or anything that speaks the OpenAI contract. All your API Keys are stored locally on your machine.

Extension Homepage: https://github.com/JiruGutema/polifix

No comments.

FAQ

You can upload the next version by the same form you've posted this version.
You can change the extension icon by clicking on the icon in your extension page.
In case you want to inactivate an approved version, click on inactivate link in your extension page.
You can find developer guides in gjs.guide.
If you need any help with your extension, you can ask us on GNOME Matrix Channel.

Files

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

Shexli (experimental) warning 4 manual_review 1

Shexli found 5 issues that may need reviewer attention.

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

extension.js:123
```
St.Clipboard.get_default()
```
panel.js:456
```
St.Clipboard.get_default()
```

EGO-P-006 warning

unnecessary build and translation artifacts should not be shipped

Package contains files that often should not be shipped for review.

Don't include unnecessary files

.gitignore
```
.gitignore
```
Makefile
```
Makefile
```
.git/index
```
.git/index
```
.git/description
```
.git/description
```
.git/HEAD
```
.git/HEAD
```
.git/config
```
.git/config
```
.git/packed-refs
```
.git/packed-refs
```
.git/hooks/pre-push.sample
```
.git/hooks/pre-push.sample
```
.git/hooks/pre-merge-commit.sample
```
.git/hooks/pre-merge-commit.sample
```
.git/hooks/pre-applypatch.sample
```
.git/hooks/pre-applypatch.sample
```

EGO-P-006 warning

unnecessary build and translation artifacts should not be shipped

Compiled GSettings schemas should not be shipped for 45+ packages.

Don't include unnecessary files

schemas/gschemas.compiled
```
schemas/gschemas.compiled
```

EGO-P-007 warning

JavaScript files should be reachable from extension.js or prefs.js

Some JavaScript files are not reachable from `extension.js` or `prefs.js` imports.

Don't include unnecessary files

tests/live-check.js
tests/provider.test.js
tests/secrets.test.js
tests/transforms.test.js
tools/print-key.js
tools/save-key.js

EGO-L-001 warning

extension must not create GObject instances or modify shell before enable()

Resource creation or signal/source setup was found outside `enable()`.

Only use initialization for static resources

secrets.js:11

new Secret.Schema(
    'io.github.jirugutema.polifix.ApiKey',
    Secret.SchemaFlags.NONE,
    {provider: Secret.SchemaAttributeType.STRING}
)

All Versions

Version	Status
1	Unreviewed