Skip to content

fix(cloudflare): reduce staged CDN probe work - #3168

Draft
james-elicx wants to merge 12 commits into
codex/cdn-v2-http-transportfrom
codex/cdn-v2-performance
Draft

fix(cloudflare): reduce staged CDN probe work#3168
james-elicx wants to merge 12 commits into
codex/cdn-v2-http-transportfrom
codex/cdn-v2-performance

Conversation

@james-elicx

@james-elicx james-elicx commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

  • classify each concrete route pathname once and defer alternate HTML/RSC/data representation admission to the final warm render
  • use the existing uncapped --warm-cdn-concurrency value for both probing and final warming, keeping that shared pool saturated by unlocking siblings per pattern while preserving late route-resolution safety
  • scope middleware-sensitive probe planning to runtime-equivalent matcher semantics, including decoded paths and i18n locale provenance
  • route bypass and cacheability-probe renders through a cache-disabled response entrypoint so the gateway never executes renderer code locally
  • preserve tag invalidation by routing uncached-render purges through the cache-bearing entrypoint

Validation

  • vp check
  • vp run vinext#build
  • vp run @cloudflare/vinext#build
  • 651 focused unit, deploy-flow, generated-config, and emitted-build tests
  • five independent focused reviews clean after requested coverage
  • Big Bonk: LGTM
  • exact-head CI: 73/73 checks passed on fdeb0c74631d92800c9994f6f40efb0d8596a787

Stacked on #3155.

@james-elicx james-elicx changed the title codex/cdn v2 performance fix(cloudflare): reduce staged CDN probe work Sep 3, 2026
@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

Comment thread packages/vinext/src/build/prerender-paths.ts Outdated
@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Posted 1 actionable inline finding.

github run

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

Comment thread packages/vinext/src/build/prerender-paths.ts Outdated
@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Posted 1 actionable inline finding.

github run

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

@james-elicx
james-elicx marked this pull request as ready for review September 3, 2026 11:27
@james-elicx
james-elicx marked this pull request as draft September 3, 2026 11:27
@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

@james-elicx Bonk workflow was cancelled.

View workflow run · To retry, trigger Bonk again.

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues. You have an 8 minute time limit; return the most important actionable findings before then.

Comment thread packages/cloudflare/src/cache/cdn-adapter.worker.ts
@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Posted 1 actionable inline finding.

github run

@james-elicx

james-elicx commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

/bigbonk review for issues. Please review the current exact head c72b4b9 with an 8 minute time limit.

Comment thread packages/cloudflare/src/cacheability-probe.ts
@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Posted 1 actionable inline finding.

github run

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues

Please limit this review to 8 minutes.

@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

UnknownError: The operation timed out.

github run

@james-elicx

Copy link
Copy Markdown
Member Author

/bigbonk review for issues in the latest performance-layer diff only

Please limit this review to 8 minutes.

@ask-bonk

ask-bonk Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

LGTM!

github run

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.

1 participant