Skip to content

Show real-time progress feedback during push/pull from Studio Code - #3071

Merged
youknowriad merged 5 commits into
trunkfrom
push-feedback-studio-code
Apr 13, 2026
Merged

Show real-time progress feedback during push/pull from Studio Code#3071
youknowriad merged 5 commits into
trunkfrom
push-feedback-studio-code

Conversation

@youknowriad

@youknowriad youknowriad commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Related issues

How AI was used in this PR

Claude identified the root causes and implemented the fix. I reviewed and tested the changes.

Proposed Changes

  • Forward progress messages to the agent UI: captureCommandOutput now forwards progress messages to the previous callback instead of silently collecting them, so the agent UI receives live updates during push/pull operations.
  • Route progress through the callback system: Push and pull commands were using logger.spinner.text (terminal-only) for progress updates. Switched to logger.reportProgress() which routes through the callback system and falls back to the spinner in terminal mode.
  • Update progress in place: setLoaderMessage in the agent UI now updates a single Text node in place (via setText) instead of appending a new child for every progress message, matching how the terminal spinner works.
  • Distinguish phases from progress updates: Added an update flag to the progress callback. reportProgress passes update: true to update the current line in place, while reportStart/reportSuccess/emitProgress create new lines — so phase changes (e.g., "Uploading…" → "Backing up remote site…") still appear as separate lines.

Testing Instructions

  • Open Studio Code and push a local site to WordPress.com (e.g., "Push my site to example.com")
  • Verify progress messages appear in real-time with percentage updates
  • Verify phase changes (uploading → initiating import → backing up → applying changes) appear as separate lines
  • Verify percentage updates within a phase update in place (not flooding new lines)
  • Pull a site and verify backup/download progress is also shown
  • Run studio push / studio pull from the CLI terminal and verify the spinner still works as before

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
youknowriad and others added 5 commits April 13, 2026 10:32
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
reportProgress passes an update flag through the callback chain so the
UI updates the existing line in place. reportStart/reportSuccess/emitProgress
create new lines, matching the terminal spinner behavior.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@youknowriad youknowriad self-assigned this Apr 13, 2026
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 09cf1c1 vs trunk

app-size

Metric trunk 09cf1c1 Diff Change
App Size (Mac) 1286.16 MB 1286.17 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 09cf1c1 Diff Change
load 1857 ms 1859 ms +2 ms ⚪ 0.0%

site-startup

Metric trunk 09cf1c1 Diff Change
siteCreation 8114 ms 9107 ms +993 ms 🔴 12.2%
siteStartup 4168 ms 4175 ms +7 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@epeicher epeicher 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.

Thanks @youknowriad for adding that! It greatly improves the UX for the push/pull processes. I have tested it, and it works as expected. I have also tested the existing studio push/pull commands and found no issues. LGTM!

CleanShot.2026-04-13.at.13.30.03.mp4

For reference, the error Error: result (122,311 characters) exceeds maximum allowed tokens. Output has been saved to is not related to this task.

@youknowriad
youknowriad merged commit 10698ca into trunk Apr 13, 2026
11 checks passed
@youknowriad
youknowriad deleted the push-feedback-studio-code branch April 13, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants