Skip to content

fix(automations): use pnpm exec instead of npx for Playwright install#92

Merged
zacharias-ona merged 1 commit intomainfrom
fix/automations-npx-permissions
Apr 16, 2026
Merged

fix(automations): use pnpm exec instead of npx for Playwright install#92
zacharias-ona merged 1 commit intomainfrom
fix/automations-npx-permissions

Conversation

@Siddhant-K-code
Copy link
Copy Markdown
Member

npx hits a /usr/bin/env: bad interpreter: Operation not permitted error in the Ona automation runner. Switches to pnpm exec which works correctly.

npx hits a permissions error in the Ona automation runner.

Co-authored-by: Ona <no-reply@ona.com>
@Siddhant-K-code Siddhant-K-code added the ona-user PR created via interactive Ona session — no issue reference required label Apr 16, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
memo Ready Ready Preview, Comment Apr 16, 2026 2:11pm

Request Review

@zacharias-ona zacharias-ona merged commit 0ae0f8b into main Apr 16, 2026
6 checks passed
@zacharias-ona zacharias-ona deleted the fix/automations-npx-permissions branch April 16, 2026 14:12
@zacharias-ona
Copy link
Copy Markdown
Collaborator

✅ UI verification skipped — no UI files changed in this PR.

@zacharias-ona
Copy link
Copy Markdown
Collaborator

✅ Post-merge verification passed.

E2E suite: 27/27 tests passed (auth, editor drag-and-drop, editor links, slash commands, floating toolbar, page CRUD, sidebar drag-and-drop, workspace switcher).

Ad-hoc smoke tests:

  • Landing page — loaded, title present
  • /sign-in — rendered with email input
  • /api/health — responded OK
  • Authenticated flow — login, workspace load, page navigation, editor render
  • No console errors (unauthenticated or authenticated)

Skipped: /dashboard (not yet built)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ona-user PR created via interactive Ona session — no issue reference required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants