Skip to content

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

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

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

Conversation

@youknowriad
Copy link
Copy Markdown
Contributor

@youknowriad youknowriad commented Apr 13, 2026

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)

Copy link
Copy Markdown
Contributor

@epeicher epeicher left a comment

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants