Skip to content

feat(PLT-3073): forward Engine events to localhost - #927

Draft
TheMostlyGreat wants to merge 23 commits into
mainfrom
codex/local-event-relay
Draft

TheMostlyGreat wants to merge 23 commits into
mainfrom
codex/local-event-relay

Conversation

@TheMostlyGreat

@TheMostlyGreat TheMostlyGreat commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Outcome

Adds arcade event listen --forward-to http://localhost:<port>/<path> so developers can exercise the production event envelope locally without exposing their machine.

What changes

  • reads the authenticated project event feed introduced by ArcadeAI/monorepo#3742
  • defaults org/project from active CLI context and supports server-side selectors
  • signs each forward with a one-session Standard Webhooks secret and stable event ID
  • refreshes CLI authorization before each poll so long-running sessions survive token rotation
  • retries ambiguous failures in order; permanent local receiver rejection stops with an actionable URL/signing-secret error
  • permits only explicit loopback hosts, re-resolves every attempt, disables environment proxies and redirects
  • includes a minimal verifying and deduplicating receiver

Verification

  • 42 focused event-listener and receiver tests pass
  • 3,775 full-suite tests pass; 1 skipped
  • focused Ruff and mypy: clean
  • source and wheel build: green
  • real Engine → feed → CLI → signed receiver proof: passed

Merge gate

Ready for review. Merge remains intentionally gated until ArcadeAI/monorepo#3742 is available in the selected Engine release.

Linear: https://linear.app/arcadedev/issue/PLT-3073

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@codecov

codecov Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.05525% with 18 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
libs/arcade-cli/arcade_cli/event.py 89.94% 18 Missing ⚠️
Files with missing lines Coverage Δ
libs/arcade-cli/arcade_cli/main.py 33.55% <100.00%> (+1.11%) ⬆️
libs/arcade-cli/arcade_cli/event.py 89.94% <89.94%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8fca449. Configure here.

Comment thread libs/arcade-cli/arcade_cli/event.py

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@evantahler evantahler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

…. no event system is on main yet

@TheMostlyGreat
TheMostlyGreat marked this pull request as draft September 6, 2026 05:39
@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has had no activity for 14 days. It will be closed in 14 days if no further activity occurs. If this is still relevant, please leave a comment or remove the stale label.

@github-actions github-actions Bot added the stale label Sep 21, 2026

This branch has not been deployed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants