Skip to content

style(proxy-registry): bring frozen files back to prettier format with rebaseline - #14218

Merged
diegosouzapw merged 3 commits into
cheaper-inference:release/v3.8.51from
maxmad64bis:fix/n91-prettier-frozen
Sep 22, 2026
Merged

diegosouzapw merged 3 commits into
cheaper-inference:release/v3.8.51from
maxmad64bis:fix/n91-prettier-frozen

Conversation

@maxmad64bis

@maxmad64bis maxmad64bis commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

⚠️ base-red inherited: #13866

Summary

Three proxy files drifted from prettier format at the base, so the first commit touching them absorbs unrelated formatting noise (or trips the frozen size ceiling). This brings them back to prettier format with no behavior change, and ratchets one inherited size ceiling to its measured size.

Related Issues

No linked issue — repo hygiene follow-up, no upstream issue covers these frozen-file drifts.

Validation

  • Change type: maintenance
  • Focused tests and category gates from the golden path
  • npm run lint — ESLint 0 on the touched files with the repo suppressions file (3 pre-existing frozen no-unused-vars outside the hunks); the full eslint . run is red on the base, unrelated to this diff.
  • Reconciled with the current active release base; focused checks rerun afterward — rev-list --count HEAD..upstream/release/v3.8.51 = 0, gates re-run after the last amend.
  • Production-code changes include a new or updated automated test in this PR — format-only change, no behavior to cover; existing neighbor test tests/unit/proxy-registry-manager.test.ts re-run green (35 pass, 0 fail).

Tests Added Or Updated

  • No new tests (format-only change, no production behavior changed); re-ran the existing neighbor test tests/unit/proxy-registry-manager.test.ts (35 pass, 0 fail).

Coverage Notes

  • Format-only diff on 3 files (9 hunks, 1+4+4); 4 already-clean files untouched; behavior identical before/after. No touched file lost coverage (no logic changed).

Reviewer Notes

  • Rebaseline: _rebaseline_2026_09_20_prettier_frozen — open-sse/utils/proxyFetch.ts ceiling 1276->1275 (inherited shrink, measured green on the base); src/sse/handlers/chat.ts 2547= and ProxyRegistryManager.tsx 1477= unchanged. Local gates: prettier --check green on all 7 files, check-file-size OK (153 frozen + 43 test), targeted ESLint 0 with the repo suppressions file, typecheck:core 0.

  • CI: red jobs are inherited from the red base (🔴 Release branch not green: release/v3.8.51 #13866), not from this diff — the third-party fix: accept ChatGPT cookie headers for clean-room sessions #14219 at the same base fails the same gates (api-typecheck 2 ✖ rerankProviderNodes.ts + antigravity.ts TS2677, plus quality/docs/eslint/unit fast-path reds); every file cited by the failing gates is outside this diff (git diff over them is empty). Non-blocking for this PR.

  • CI reds are inherited from the red base (🔴 Release branch not green: release/v3.8.51 #13866), not from this diff: the third-party PR fix(sse): treat antigravity empty completions with a normal stop as valid 200s (#14160) #14243 on the same base release/v3.8.51 fails the same 9 jobs (API Route Typecheck, Docs Gates, Fast Quality Gates, Merge integrity, ESLint, Unit fast-path 1-4/4); every file cited by the failing gates is outside this diff. Non-blocking for this PR.

@maxmad64bis
maxmad64bis force-pushed the fix/n91-prettier-frozen branch from 99b469b to 260f2a7 Compare September 19, 2026 22:52
@maxmad64bis
maxmad64bis marked this pull request as ready for review September 19, 2026 22:59
@maxmad64bis
maxmad64bis force-pushed the fix/n91-prettier-frozen branch 3 times, most recently from 2a6f474 to a97b003 Compare September 19, 2026 23:12
@maxmad64bis maxmad64bis changed the title style(proxy): prettier frozen files with rebaseline style(proxy-registry): bring frozen files back to prettier format with rebaseline Sep 19, 2026
…h rebaseline

The next commit touching these files absorbed unrelated formatting noise.
Format the three drifted files back to prettier style (1+4+4 hunks); behavior
unchanged (verified: prettier check green, tests 35/35).
Ratchet the inherited open-sse/utils/proxyFetch.ts ceiling 1276->1275 to its
measured size (shrink predates this change, verified green on the base).
@maxmad64bis
maxmad64bis force-pushed the fix/n91-prettier-frozen branch from a97b003 to e71e67a Compare September 19, 2026 23:13
# Conflicts:
#	config/quality/file-size-baseline.json
# Conflicts:
#	config/quality/file-size-baseline.json
@diegosouzapw
diegosouzapw merged commit 9e44a5d into cheaper-inference:release/v3.8.51 Sep 22, 2026
9 of 16 checks passed
diegosouzapw added a commit to maxmad64bis/OmniRouteFork that referenced this pull request Sep 22, 2026
…-attrib

Conflicts:
- open-sse/executors/opencode.ts: cheaper-inference#14464 replaced the free-tier refusal exit
  with handleLoopFreeTierRefusal(); kept the release shape and moved the
  attribution flush to just before that call.
- config/quality/file-size-baseline.json: kept both sides' _rebaseline notes;
  re-measured on the reconciled tree — opencode.ts 1386, proxyFetch.ts 1287
  (the tip shrank it to 1268 in the cheaper-inference#14218 Prettier pass).
@maxmad64bis
maxmad64bis deleted the fix/n91-prettier-frozen branch September 24, 2026 21:14
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