Skip to content

Move prerender reproduction URL to its own log category#4224

Merged
habdelra merged 1 commit intomainfrom
prerender-log-cleanup
Mar 21, 2026
Merged

Move prerender reproduction URL to its own log category#4224
habdelra merged 1 commit intomainfrom
prerender-log-cleanup

Conversation

@habdelra
Copy link
Contributor

@habdelra habdelra commented Mar 20, 2026

Summary

  • The prerenderer debug logs are too noisy right now — in practice, the only things we need to see are errors and the reproduction URL (so we can manually visit a bad render). This moves the "manually visit prerendered url" message into a dedicated prerenderer-reproduce log category so we can turn down the noise by running with DEBUG=prerenderer-reproduce instead of DEBUG=prerenderer, while still getting the one message that matters for debugging bad renders.

Separate the "manually visit prerendered url" debug message into a
dedicated `prerenderer-reproduce` log category so it can be enabled
independently of the noisy `prerenderer` logs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@habdelra habdelra requested a review from a team March 20, 2026 18:32
@habdelra habdelra changed the title Move prerender reproduce URL to its own log category Move prerender reproduction URL to its own log category Mar 20, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 186f0b67c1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@github-actions
Copy link

Host Test Results

    1 files  ±0      1 suites  ±0   2h 18m 11s ⏱️ - 4m 0s
2 030 tests ±0  2 015 ✅ ±0  15 💤 ±0  0 ❌ ±0 
2 045 runs  ±0  2 030 ✅ ±0  15 💤 ±0  0 ❌ ±0 

Results for commit 186f0b6. ± Comparison against base commit 505f18d.

@habdelra habdelra merged commit 9122f95 into main Mar 21, 2026
108 of 109 checks passed
habdelra added a commit that referenced this pull request Mar 23, 2026
Separate the "manually visit prerendered url" debug message into a
dedicated `prerenderer-reproduce` log category so it can be enabled
independently of the noisy `prerenderer` logs.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

2 participants