Review of "Polifix" version 1

Details Page Preview

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.

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

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) 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
2 Unreviewed
1 Rejected

Previous Reviews on this Version

jirugutema auto- rejected
Auto-rejected because of new version 2 was uploaded