Review of "Text Extractor" version 8

Details Page Preview

Extract text from screen using OCR. Select a screen region, and the text is automatically copied to clipboard. Requires Tesseract OCR.

Extension Homepage
https://github.com/Aditya190803/TextExtractor

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) warning 1 manual_review 1

Shexli found 2 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:484
    St.Clipboard.get_default()

EGO-A-004 warning

extension files should not contain excessive ungated console logging

File contains 12 ungated console.log/warn/error calls (threshold: 5).

No excessive logging

  • extension.js:93
    console.error(`[TextExtractor] Failed to create directory: ${e.message}`)
  • extension.js:168
    console.error(`[TextExtractor] Screenshot error: ${e.message}`)
  • extension.js:271
    console.error(`[TextExtractor] Failed to delete screenshot: ${deleteErr.message}`)
  • extension.js:274
    console.error(`[TextExtractor] Failed to copy screenshot: ${copyErr.message}`)
  • extension.js:346
    console.error(`[TextExtractor] Error parsing response: ${e.message}`)
  • extension.js:400
    console.error(`[TextExtractor] DBus call error: ${e.message}`)
  • extension.js:419
    console.error(`[TextExtractor] Screenshot error: ${e.message}`)
  • extension.js:497
    console.error(`[TextExtractor] OCR error: ${ocrResult.error}`)
  • extension.js:501
    console.error(`[TextExtractor] Failed to parse OCR result: ${parseError.message}`)
  • extension.js:505
    console.error(`[TextExtractor] OCR produced no output. stderr: ${stderr}`)

All Versions

Version Status
10 Active
9 Rejected
8 Rejected
7 Active
6 Rejected
5 Inactive
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

Aditya Mer auto- rejected
Auto-rejected because of new version 9 was uploaded