Skip to content

test(e2e): print the item size-panel payload numbers to the ci log - #1313

Open
JOY (JOY) wants to merge 2 commits into
devfrom
test/1285-print-payload-numbers
Open

JOY (JOY) wants to merge 2 commits into
devfrom
test/1285-print-payload-numbers

Conversation

@JOY

@JOY JOY (JOY) commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Refs #1285

Follow-up to PR #1296: the Playwright line reporter does not print test annotations, so the RSC payload numbers the item-size-panels spec records never reached the CI log. The spec now also writes them with console.warn, so the item-model E2E job log carries them and they can be posted on #1285. Test-only change.

🤖 Generated with Claude Code


📌 TL;DR

This PR adds a console.warn statement to the item-size-panels E2E test to explicitly log RSC byte sizes and size-route JSON metrics to the CI console, ensuring these performance data points are visible in build logs despite the line reporter not printing test annotations.

🎯 Type of Change

  • 🚀 New feature
  • 🐛 Bugfix
  • 🧹 Refactor
  • ⚡ Performance
  • 📚 Documentation
  • ⚙️ CI / Configuration

🔍 Changes Walkthrough

File Summary of Changes
apps/web/e2e/item-model/item-size-panels.spec.ts Added a console.warn call to output specific RSC byte counts and size-route JSON metrics to the CI log. This ensures performance data is visible in standard CI output, as the existing test reporter does not display test annotations.

📊 Architectural Flow

(Omitted: This change is a simple logging addition within a test file and does not alter component interactions, APIs, or workflow state.)

JOY (JOY) and others added 2 commits October 29, 2026 02:26
Refs #1285

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Refs #1285

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
rate Ready Ready Preview Oct 28, 2026 7:27pm UTC

Request Review

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

⚠️ Code review skipped — your organization has no extra usage available to pay for this review.

If your organization's extra usage balance is empty, an organization admin can add extra usage credits at claude.ai/admin-settings/usage. If its monthly spend limit was reached, an admin can raise it on the same page. If neither applies, contact Anthropic support.

Once extra usage is available, someone with write access to this repository can comment @claude review on this pull request to trigger a review.

@coderabbitai

coderabbitai Bot commented Oct 28, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 6 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 998fa103-2ddf-4265-850c-616f97f12974
Commits

Reviewing files that changed from the base of the PR and between 4eca23d and 50dade6.

Files selected for processing: 1
  • apps/web/e2e/item-model/item-size-panels.spec.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

dos[bot]
dos Bot approved these changes Oct 28, 2026

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

⏱️ Code Review completed (1 eligible files · 824 chars · 1 model call(s) · 1 PR unit(s))

Whole diff with bounded context at base 4eca23d / head 50dade6.

✅ CLEAN_PASS: Complete diff reviewed

This branch was successfully deployed

1 active deployment
Preview — 50dade60 Deployed Oct 28, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Frontend / Next.js / UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant