[rancescolon]

Capture a region of your screen on Linux to perform OCR or scan QR codes/barcodes, automatically copying the extracted text directly to your clipboard.

Two no-view commands: Scan & copy text (OCR) and Scan QR code (ZBar decoding)
One view command: Select Recognition Languages to pick and save preferred Tesseract languages (stored in LocalStorage)
Missing language detection: If Tesseract data is missing, automatically copies the required system package name (e.g. tesseract-data-spa) to clipboard for quick installation
Refactored execution flow with retry validation to ensure seamless region capture via Spectacle and clipboard delivery

Source repository: https://github.com/rancescolon/extensions

Notes for review

System utilities required: spectacle (region capture), tesseract (OCR), and zbarimg (QR scanning).
No remote network calls or analytics.
Files written: temporary image files in the system temp folder (cleaned up automatically in finally blocks).
Explicit error handling for missing CLI utilities and missing language data files.

Checks run locally

npm run check-dependencies passes
npm ci and npm run build succeed from a clean checkout
npm run lint, npm run format, and npm run typecheck pass
Tested manually in a local Vicinae instance on Linux

[clankus-aurelius]

Thanks for contributing an extension to Vicinae! 👋

Before publication, this pull request receives two reviews:

1. An automated review for extension guidelines, safety, error handling, and likely correctness issues.
2. A final review from a Vicinae maintainer.

✅ Ready for human review. The automated reviewer approved the latest commit and a maintainer has been notified.

No blocking findings remain on the latest commit.

The automated reviewer examines only the current commit. New commits invalidate its previous decision and start another review.

[clankus-aurelius]

The extension has several error paths that silently fail or report success incorrectly, and its setup documentation overstates or misrepresents system-specific requirements.

---

Automated extension review passed. A maintainer review is still required.

This is an AI-generated first pass and may be mistaken. If a finding is unclear or incorrect, reply in the relevant thread and mention @aurelleb.

[clankus-aurelius]

@aurelleb automated review passed for bf50248; this extension is ready for your review.

[clankus-aurelius]

Four issues remain: two README requirements are still misleading, and the new numeric exit-code checks misclassify common cancellation/no-result outcomes.

---

Automated extension review passed. A maintainer review is still required.

This is an AI-generated first pass and may be mistaken. If a finding is unclear or incorrect, reply in the relevant thread and mention @aurelleb.

[clankus-aurelius]

Three issues remain: the README still overstates distro-specific package accuracy, while the revised error handling silently hides Spectacle failures and still does not reliably recognize ZBar’s no-symbol exit status.

---

Automated extension review passed. A maintainer review is still required.

This is an AI-generated first pass and may be mistaken. If a finding is unclear or incorrect, reply in the relevant thread and mention @aurelleb.

[clankus-aurelius]

🟠 Warning — Package helper still implies distro-specific accuracy

Rule: MANIFEST-001

The README calls the generated value the required package for the user's distro, but the implementation performs no distribution detection and always constructs tesseract-data-<language>.

Suggested resolution: Describe it as an example package name that users must verify for their distribution, or implement distribution-specific mappings.

Suggested change

	3. The tool processes the image (via Tesseract or ZBar), copies the decoded result to your clipboard, and confirms via a HUD toast.
	4. *Missing Language Pack Helper*: If a Tesseract language pack is missing, it copies an example package name (e.g., `tesseract-data-spa`) to your clipboard; verify the correct package name for your distribution.

[clankus-aurelius]

🟠 Warning — Spectacle failures are silently treated as cancellation

Rule: UX-001

Every Spectacle error other than ENOENT now returns false when no output exists, so permission, display-server, and other operational failures silently exit as though the user cancelled.

Suggested resolution: Treat only Spectacle's cancellation exit status as cancellation; rethrow other failures so performScanAndCopy shows actionable failure feedback.

[clankus-aurelius]

🟠 Warning — ZBar no-symbol exit status remains unrecognized

Rule: CORRECTNESS-001

The revised condition still ignores ZBar's numeric no-symbol exit status 4. If stderr lacks the selected phrases, an ordinary image with no symbol is reported as a ZBar failure; matching every WARNING can also suppress unrelated failures.

Suggested resolution: Recognize exit status 4 explicitly and remove the broad WARNING match.

Suggested change

	}
	// zbarimg returns non-zero when no symbols are found
	const isNoSymbols =
	// zbarimg returns exit status 4 when no symbols are found
	const isNoSymbols =
	errOutput.includes("no symbols found") || err.status === 4 || err.code === 4 || err.code === "4";