Skip to content

debug: probe fastify-static 9.1.1 test failures (do not merge)#8229

Closed
aitchiss wants to merge 2 commits intomainfrom
debug/fastify-static-9.1.1-probe
Closed

debug: probe fastify-static 9.1.1 test failures (do not merge)#8229
aitchiss wants to merge 2 commits intomainfrom
debug/fastify-static-9.1.1-probe

Conversation

@aitchiss
Copy link
Copy Markdown

Purpose

Throwaway branch off #8165 to investigate why integration tests fail with @fastify/static 9.0.0 → 9.1.1. Do not merge.

What this adds

Three console.error probes that bubble through devServer.output into vitest's failure log:

  • [EF-PROBE] <method> <url> matched=<functionNames> — at src/lib/edge-functions/proxy.ts after registry.matchURLPath. Tells us whether requests reached the EF proxy and what the registry returned.
  • [STATIC-PROBE] starting; root=<path> — at src/utils/static-server.ts boot.
  • [STATIC-PROBE] <method> <url> — on every static-server request.

How to read the output

[EF-PROBE] for /ordertest? matched= Diagnosis
missing request bypassed EF proxy entirely
present [] EF registry hadn't finished loading (race exposed by faster 9.1.1 boot)
present full chain EF matched but origin response leaked through
present partial chain registry partially loaded

Looking at the ubuntu Integration shard 3/4 logs for tests/integration/commands/dev/edge-functions.test.ts > should run edge functions in correct order.

Cleanup

Delete this branch once we have an answer; revert / never merge.

renovate Bot and others added 2 commits April 29, 2026 16:15
Temporary instrumentation to investigate why integration tests fail on PR #8165 (@fastify/static 9.0.0 -> 9.1.1). Logs whether requests reach the EF proxy, what registry.matchURLPath returns, and whether the static server handles them. Throwaway branch — do not merge.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

📊 Benchmark results

Comparing with 89ce39f

  • Dependency count: 1,061 (no change)
  • Package size: 357 MB ⬆️ 0.62% increase vs. 89ce39f
  • Number of ts-expect-error directives: 355 (no change)

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 106647df-0c25-440e-b54a-4a5e1c730c17

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch debug/fastify-static-9.1.1-probe

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

@aitchiss
Copy link
Copy Markdown
Author

Closing — original failures on #8165 were flaky CI, confirmed across three runs. No regression from @fastify/static 9.1.1. Throwaway branch.

@aitchiss aitchiss closed this Apr 30, 2026
@aitchiss aitchiss deleted the debug/fastify-static-9.1.1-probe branch April 30, 2026 10:07
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