Commit Graph
3 Commits
Author SHA1 Message Date
Devin FoleyandPaperclip 6cd9e7eb3c fix: address Greptile P1 findings on browser lifecycle
- Validate URL and compute origin before launching browser so invalid
  URLs fail fast without leaking a Chromium process
- Wrap entire browser lifecycle (context, page, route setup) inside
  try-finally so browser.close() is always called
- Require apiBase in loadBoardToken to prevent undefined URL construction

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-04-03 09:06:49 -07:00
Devin FoleyandPaperclip 792f0f8977 fix: address Greptile review feedback on screenshot utility
- P1: Replace process.exit(1) with process.exitCode = 1 so the finally
  block runs and browser.close() is called on failure
- P2: Fix flag parsing to handle --wait 0 correctly using Number.isNaN
  instead of falsy || fallback
- P2: Scope Bearer token to Paperclip origin only via page.route()
  instead of extraHTTPHeaders on all requests

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-04-03 08:55:37 -07:00
Devin FoleyandPaperclip 5d0db75ba3 feat: add authenticated screenshot utility for Paperclip UI
Reads board token from ~/.paperclip/auth.json and injects it as a
Bearer header via Playwright, enabling agents to take screenshots
of authenticated pages without storing email/password credentials.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
2026-04-02 23:33:10 -07:00