Skip to content

fix(runtime): isolate unique repairs from #8177 - #8298

Merged
diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.49from
backryun:agent/8177-focused-runtime-fixes
Jul 24, 2026
Merged

diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.49from
backryun:agent/8177-focused-runtime-fixes

Conversation

@backryun

Copy link
Copy Markdown
Contributor

Summary

  • Follow up on the maintainer feedback in fix(release): repair build and quality gate drift #8177 with a focused branch against the current release/v3.8.49 tip.
  • Sanitize Muse Spark GraphQL fetch exceptions before they reach either executor logs or client-facing error handling.
  • Resolve the sql.js WASM asset from the package's public entrypoint and remove the unexported sql.js/package.json lookup.
  • Remove the now-obsolete Muse Spark exception from the error-helper gate.

Scope

This intentionally excludes the overlapping base-red reconciliation work from #8177:

Only the two runtime fixes that remain unique on the current release tip are included.

Root cause

Muse Spark sanitized its HTTP response after a GraphQL fetch failure, but logged the intermediate error first. That left absolute source paths and stack-tail text visible in executor logs.

The sql.js fallback resolved sql.js/package.json, but sql.js does not export that private subpath. The already-present public sql.js entrypoint resolves beside sql-wasm.wasm, so the metadata lookup is unnecessary and causes repeated Turbopack diagnostics.

Validation

  • node --import tsx/esm --test tests/unit/muse-spark-web-continuation.test.ts tests/unit/sqljs-build-warning-8135.test.ts — 8/8 passed.
  • npm run typecheck:core — passed.
  • node scripts/check/check-error-helper.mjs — passed with zero frozen exceptions.
  • npm run build — passed; the sql.js/package.json missing-subpath warning is absent.
  • git diff --check — passed.

@backryun
backryun force-pushed the agent/8177-focused-runtime-fixes branch from ec55545 to 0a6f3ab Compare July 23, 2026 14:31
@backryun
backryun marked this pull request as ready for review July 23, 2026 14:31
@backryun
backryun requested a review from diegosouzapw as a code owner July 23, 2026 14:31
@backryun
backryun force-pushed the agent/8177-focused-runtime-fixes branch from 0a6f3ab to 278dbf1 Compare July 23, 2026 18:58
@diegosouzapw
diegosouzapw merged commit b84f86a into cheaper-inference:release/v3.8.49 Jul 24, 2026
5 checks passed
@diegosouzapw

Copy link
Copy Markdown
Collaborator

Merged into release/v3.8.49 — thanks @backryun! Validated via local merge-train (32-core .113) + a per-file discriminator confirming zero regressions vs the pure release tip; CI reds on this PR were pre-existing base-red drift (tracked separately). 🙏

@backryun
backryun deleted the agent/8177-focused-runtime-fixes branch July 27, 2026 14:41
@diegosouzapw diegosouzapw mentioned this pull request Jul 28, 2026
HouMinXi pushed a commit to HouMinXi/OmniRoute that referenced this pull request Aug 2, 2026
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
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