Skip to content

fix(auto): log empty auto-family pools once per process - #10820

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
RaviTharuma:fix/10346-empty-pool-once
Aug 20, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
RaviTharuma:fix/10346-empty-pool-once

Conversation

@RaviTharuma

Copy link
Copy Markdown
Contributor

Summary

  • auto/<family> matched no connected models is a stable empty-pool state.
  • Warn once per process per label, not every minute.
  • Real AUTO failures (auth, 5xx, all candidates failed) are unchanged.

Related Issues

Validation

  • Change type: other (backend logging)
  • Focused tests: tests/unit/auto-empty-pool-warn-once.test.ts (2 pass)
  • npm run lint
  • Reconciled with release/v3.8.50
  • Production-code changes include a new or updated automated test in this PR

Tests Added Or Updated

  • tests/unit/auto-empty-pool-warn-once.test.ts

Coverage Notes

  • warnEmptyAutoPoolOnce in open-sse/services/autoCombo/virtualFactory.ts is covered by the unit file.

Reviewer Notes

  • Replaces the 60s debounce that still produced a metronome (~once/minute), matching the original live cadence.

@RaviTharuma
RaviTharuma force-pushed the fix/10346-empty-pool-once branch 2 times, most recently from 448091f to e823501 Compare August 20, 2026 13:05
@RaviTharuma

Copy link
Copy Markdown
Contributor Author

Rebased onto current release/v3.8.50 via #10828 (f5be5dd). Unique commit replayed on the drain HEAD. Merge is MERGEABLE. Does not merge.

@RaviTharuma

Copy link
Copy Markdown
Contributor Author

Rebased onto current release/v3.8.50 via #10828 (0a09b6a / 4821f9f). Unique commit replayed. HEAD 9108c54. Does not merge.

Steady empty auto/<family> is not a 1-minute warn metronome. Warn once
per label until process restart (tests reset the set).

Fixes #10346
@RaviTharuma
RaviTharuma force-pushed the fix/10346-empty-pool-once branch from 9108c54 to 1d769a8 Compare August 20, 2026 13:57
@RaviTharuma

Copy link
Copy Markdown
Contributor Author

Removed the shared #10828 drain commits from this branch so the diff is feature-only again (it had been ~100 files of i18n/catalog/CI dump stacked under the real change). Rebased onto current release/v3.8.50 (4821f9ffd). Does not merge.

@diegosouzapw
diegosouzapw merged commit 2f73158 into diegosouzapw:release/v3.8.50 Aug 20, 2026
5 checks passed
@RaviTharuma
RaviTharuma deleted the fix/10346-empty-pool-once branch September 23, 2026 19:38
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…#10820)

Merged — validated together with a batch of related RaviTharuma PRs in one combined worktree (typecheck:core clean, complexity/file-size/changelog gates green, focused tests passing). Thanks for the contribution!
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.

fix(backend): deduplicate AUTO empty-pool warnings per family

2 participants