Skip to content

Run CI for #2913#3007

Closed
sejas wants to merge 12 commits intotrunkfrom
run-ci-for-2913
Closed

Run CI for #2913#3007
sejas wants to merge 12 commits intotrunkfrom
run-ci-for-2913

Conversation

@sejas
Copy link
Copy Markdown
Member

@sejas sejas commented Apr 8, 2026

Related issues

Proposed Changes

Mirror branch of #2913 to re-run CI checks.

Testing Instructions

  • Verify CI passes on this branch.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors? (Tests pass, no errors)

wesleyfantinel and others added 12 commits March 25, 2026 18:08
- Introduced `JsonAdapter` for emitting NDJSON events to stdout.
- Enhanced `runCommand` to support headless mode.
- Updated AI agent to handle `autoApprove` for non-interactive prompts.
- Added tests to validate NDJSON output behavior and error handling.
…andling, so that the type is conformant to zod
Make costUsd optional since only JsonAdapter provides it, guard
replaySessionHistory behind InteractiveAdapter instanceof check.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sejas sejas self-assigned this Apr 8, 2026
@wpmobilebot
Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 6a45b6b vs trunk

app-size

Metric trunk 6a45b6b Diff Change
App Size (Mac) 1252.02 MB 1252.01 MB 0.01 MB ⚪ 0.0%

site-editor

Metric trunk 6a45b6b Diff Change
load 1853 ms 1897 ms +44 ms ⚪ 0.0%

site-startup

Metric trunk 6a45b6b Diff Change
siteCreation 8121 ms 8185 ms +64 ms 🔴 0.8%
siteStartup 4020 ms 4830 ms +810 ms 🔴 20.1%

Results are median values from multiple test runs.

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

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