Skip to content

feat(ai-review): run writeback on worker with live progress (PR6)#23667

Merged
joaoviana merged 1 commit into
mainfrom
05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_
May 29, 2026
Merged

feat(ai-review): run writeback on worker with live progress (PR6)#23667
joaoviana merged 1 commit into
mainfrom
05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_

Conversation

@joaoviana
Copy link
Copy Markdown
Contributor

@joaoviana joaoviana commented May 29, 2026

Part 6/9. Moves the (multi-minute) writeback off the API request onto the Graphile worker, with live progress.

  • New aiAgentReviewWriteback EE scheduler task; POST .../writeback now enqueues + sets queued and returns immediately.
  • Worker runs the engine with onProgress streaming phase messages onto the item; lightweight GET .../review-items/{fingerprint} for cheap polling.
  • Migration adds pr_writeback_status / pr_writeback_message.

Regression analysis: changes the writeback trigger from sync → async. Only affects the new, flag-gated review-writeback path; the chat proposeWriteback tool is untouched.

🤖 Generated with Claude Code

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 29, 2026

🧪 Test Selection

✅ Tests that will run

Test Description
Preview Environment Deploys a preview environment for testing
Frontend E2E Tests Runs Cypress app tests
Backend API Tests Runs Vitest API tests
CLI Tests Runs CLI integration and dbt version tests

⏭️ Tests skipped (no relevant file changes detected)

Test How to trigger manually
Timezone Tests Add test-timezone to PR description

Tip: Add test-all to your PR description to run all tests.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 29, 2026

Your preview environment pr-23667 has been deployed.

Preview environment endpoints are available at:

@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_ branch from 598e162 to df961e4 Compare May 29, 2026 15:47
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_review-queue_ui_create_pr_dismiss_status_pr_link branch from 83696ce to cb8be66 Compare May 29, 2026 15:47
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_ branch from df961e4 to 6303b0d Compare May 29, 2026 16:04
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_review-queue_ui_create_pr_dismiss_status_pr_link branch from cb8be66 to 3768d10 Compare May 29, 2026 16:04
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_review-queue_ui_create_pr_dismiss_status_pr_link branch from 3768d10 to 74c35a6 Compare May 29, 2026 16:40
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_ branch from 6303b0d to 256997b Compare May 29, 2026 16:40
@github-actions
Copy link
Copy Markdown

Preview Environment

🌐 URL: https://lightdash-preview-pr-23667.lightdash.okteto.dev

📋 Logs: View in GCP Console

🔧 SSH: ./scripts/okteto-ssh.sh 23667

@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_ branch from 256997b to e6dce05 Compare May 29, 2026 17:11
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_review-queue_ui_create_pr_dismiss_status_pr_link branch from 15ce797 to 763c9fe Compare May 29, 2026 17:16
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_ branch from e6dce05 to 610015f Compare May 29, 2026 17:16
Base automatically changed from 05-29-feat_ai-review_review-queue_ui_create_pr_dismiss_status_pr_link to main May 29, 2026 17:16
@joaoviana joaoviana force-pushed the 05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_ branch from 610015f to ff905f4 Compare May 29, 2026 17:17
@joaoviana joaoviana marked this pull request as ready for review May 29, 2026 17:17
@joaoviana joaoviana merged commit c991df5 into main May 29, 2026
3 checks passed
@joaoviana joaoviana deleted the 05-29-feat_ai-review_run_writeback_on_worker_with_live_progress_pr6_ branch May 29, 2026 17:17
lightdash-bot pushed a commit that referenced this pull request May 29, 2026
# [0.3055.0](0.3054.0...0.3055.0) (2026-05-29)

### Bug Fixes

* **ai-review:** gate Create-PR button on server-computed eligibility (GitHub + flags) ([#23668](#23668)) ([2ecae83](2ecae83))
* **ai-review:** writeback hardening — progress-write race + resilience ([#23669](#23669)) ([ae39718](ae39718))

### Features

* **ai-review:** review-queue UI — Create PR, dismiss, status + PR link ([#23666](#23666)) ([2866822](2866822))
* **ai-review:** run writeback on worker with live progress (PR6) ([#23667](#23667)) ([c991df5](c991df5))
@lightdash-bot
Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 0.3055.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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