Skip to content

feat(proxy): add opportunistic traffic class core#657

Merged
Soju06 merged 21 commits into
Soju06:split/567-sticky-thresholds-independentfrom
Komzpa:split/567-opportunistic-core
May 17, 2026
Merged

feat(proxy): add opportunistic traffic class core#657
Soju06 merged 21 commits into
Soju06:split/567-sticky-thresholds-independentfrom
Komzpa:split/567-opportunistic-core

Conversation

@Komzpa
Copy link
Copy Markdown
Collaborator

@Komzpa Komzpa commented May 15, 2026

Draft split from #567.

Purpose

  • Opportunistic traffic class routing core.
  • API-key traffic-class surface.
  • Regression tests for opportunistic routing, admission, malformed settings JSON, non-burn denial mapping, and split stackability.

Validation

  • Head: 3a283a279b571114bf4a9b6d8aced368f878d41d.
  • Exact-head GitHub CI is green.
  • Fresh @codex review completed for 3a283a279b571114bf4a9b6d8aced368f878d41d.
  • Local checks passed: ruff format/check, ty, focused pytest for opportunistic/admission/traffic-class routing slices, SQLite migration upgrade/check after adding the small PR-local Alembic merge revision.
  • Earlier frontend lint/typecheck/vitest for API-key traffic-class dialogs/schemas passed before the latest backend-only migration-head repair.

Relationship

@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

@Komzpa Komzpa force-pushed the split/567-opportunistic-core branch from 1cc28aa to 9087121 Compare May 15, 2026 13:07
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 90871212c2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/modules/settings/service.py Outdated
@Komzpa Komzpa force-pushed the split/567-opportunistic-core branch from 9087121 to ebb8933 Compare May 15, 2026 13:22
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

@Komzpa Komzpa force-pushed the split/567-opportunistic-core branch from ebb8933 to dfabc9b Compare May 15, 2026 13:25
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ebb893368e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/modules/accounts/mappers.py Outdated
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: dfabc9b036

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/modules/proxy/service.py Outdated
Comment thread app/modules/proxy/api.py
@Komzpa Komzpa force-pushed the split/567-opportunistic-core branch from dfabc9b to 4c1ea71 Compare May 15, 2026 13:48
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4c1ea71bad

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/modules/proxy/load_balancer.py Outdated
@Komzpa Komzpa force-pushed the split/567-opportunistic-core branch from 4c1ea71 to 0cea94d Compare May 15, 2026 14:05
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0cea94d94f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/modules/accounts/mappers.py Outdated
…7-opportunistic-core

# Conflicts:
#	app/modules/proxy/load_balancer.py
#	app/modules/proxy/service.py
@Komzpa Komzpa changed the base branch from main to split/567-sticky-thresholds-independent May 15, 2026 14:49
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 725bacd9e3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 15, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3a283a279b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/modules/proxy/load_balancer.py
@Komzpa Komzpa marked this pull request as ready for review May 15, 2026 17:03
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 16, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4aaac7d762

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/modules/accounts/mappers.py Outdated
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 16, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1b5b59ae48

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/core/balancer/logic.py Outdated
Comment thread app/modules/proxy/service.py Outdated
@Komzpa
Copy link
Copy Markdown
Collaborator Author

Komzpa commented May 16, 2026

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Komzpa Komzpa added 🤖 codex: ok [@codex review] says no issues found. and removed 🤖 codex: ok [@codex review] says no issues found. labels May 16, 2026
@Soju06 Soju06 merged commit 0f1998c into Soju06:split/567-sticky-thresholds-independent May 17, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖 codex: ok [@codex review] says no issues found.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants