Skip to content

fix(proxy): reconcile continuity stack integration - #1442

Closed
Komzpa wants to merge 196 commits into
Soju06:mainfrom
Komzpa:fix/live-compact-urgent-20260722
Closed

fix(proxy): reconcile continuity stack integration#1442
Komzpa wants to merge 196 commits into
Soju06:mainfrom
Komzpa:fix/live-compact-urgent-20260722

Conversation

@Komzpa

@Komzpa Komzpa commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Status

This remains a draft integration carrier for compatibility testing across focused proxy, compact, retry, migration, and continuity changes. It is not a reviewable single-concern pull request and must not be merged in its current form.

Current head: edafbadcece570e481808b2b59671b1a5d0bede4, based on e9273d209798a787eb58c9f24f8d0d643e381259.

The carrier was rebuilt from current main plus the current focused heads for:

Current main already contains the earlier focused pieces that this carrier used to embed separately, including Soju06/codex-lb#1353, Soju06/codex-lb#1389, Soju06/codex-lb#1418, and Soju06/codex-lb#1441.

Integration Behavior

The carrier combines the focused behaviors above with compatibility fixes that reconcile their contracts:

  • retain durable response, side-effect, tool-call, and replay boundaries;
  • retry model-capacity, overload, and transport failures only when the request is output-free and replay-safe;
  • preserve previous-response, turn-state, session-header, hard-owner, and security-lineage continuity across HTTP bridge, direct stream, compact, and WebSocket paths;
  • require security-work authorization for marked durable lineages without downgrading detached markers;
  • preserve required compact tool tails, historical side-effect pairs, inline image elision context, and occurrence-aware tool identity; and
  • keep native WebSocket receive closes account-neutral while preserving deferred settlement ownership.

The branch currently changes 64 files (+6034/-271 lines) relative to current main. That breadth is why this remains an integration carrier rather than a merge candidate.

Validation Status

Local validation at current head edafbadcece570e481808b2b59671b1a5d0bede4:

  • uvx ruff check app tests .github/scripts openspec --exclude frontend passed.
  • Carrier-owned OpenSpec changes validated individually with --strict: allow-compact-inline-image-elision, harden-compact-tool-tail-anchors, forward-codex-alpha-search, retry-model-capacity-errors, retry-server-is-overloaded, persist-security-lineage-markers, and preserve-websocket-drop-account-health.
  • Focused aggregate pytest selection passed: 591 passed, 1032 deselected, with the current SQLAlchemy expression-index reflection warnings.
  • Follow-up WebSocket/security retry slice passed: 345 passed, 566 deselected.
  • The exact stale type-cast regression in test_relay_upstream_websocket_classified_stream_incomplete_drop_is_penalized passed after the aggregate compatibility commit.

Hosted CI for this head is running in workflow run 30338571892. Current-head Codex review has not yet been refreshed.

Maintainer Action

  • Do not merge this pull request in its current form.
  • Review, validate, and merge focused pull requests independently.
  • Treat this branch as compatibility evidence for the current focused-head order only.

Non-Claims

  • No component pull request is claimed merged, superseded, or ready because it appears in this carrier.
  • This carrier does not establish a maintainer-approved merge order.
  • Local aggregate validation does not substitute for green hosted CI, focused review state, or a fresh current-head Codex review.

Komzpa added 30 commits July 16, 2026 00:35
Signed-off-by: Darafei Praliaskouski <me@komzpa.net>
Signed-off-by: Darafei Praliaskouski <me@komzpa.net>
…y-retry

# Conflicts:
#	.github/scripts/check_all_contributors.py
#	.github/scripts/detect_changed_areas.py
#	.github/scripts/fetch_pr_labels.py
#	.github/scripts/github_api.py
#	tests/unit/test_check_all_contributors.py
#	tests/unit/test_github_ci_scripts.py
# Conflicts:
#	app/modules/proxy/_service/http_bridge/mixin.py
#	app/modules/proxy/load_balancer.py
@Komzpa
Komzpa force-pushed the fix/live-compact-urgent-20260722 branch from edafbad to 66040c4 Compare July 28, 2026 07:51
@Komzpa
Komzpa force-pushed the fix/live-compact-urgent-20260722 branch from 66040c4 to b9efeba Compare July 28, 2026 07:55
Komzpa and others added 19 commits July 28, 2026 12:01
* i18n(zh-CN): complete missing translations and unify terminology

- Translate 101 untranslated zh-CN entries across accounts, apiKeys,
  apis, automations, dashboard, firewall, formatters, modelSources,
  quotaPlanner, stickySessions, upstreamProxy, and common namespaces
- Fix aria text rendering mixed-language strings (e.g. sorted by
  ascending) by translating dashboard.accountList.ascending/descending
- Unify accountBurnProjection with the existing settings.appearance
  translation (账户消耗预测)
- Translate automations trigger filter label and runs column header
  (触发方式) to match the surrounding 状态/类型 filter labels
- Keep domain terms in English where zh-CN already does so
  (Model, Source, credits, quota, pace, etc.)
- Add OpenSpec change complete-zh-cn-dashboard-i18n covering zh-CN
  locale completeness and terminology consistency requirements

* docs(openspec): document zh-CN locale decisions

* i18n(zh-CN): translate remaining upstreamProxy UI copy

Translate ordinary UI strings left in English in the upstreamProxy
namespace (actions, dialogs, toasts, validation) while keeping the
Endpoint/Pool technical terms per the locale decisions in the OpenSpec
change context.

* i18n(zh-CN): translate stale English strings missed by en-identical scan

These 18 values were English copies of older en.json text, so a scan
for values identical to current en.json skipped them. Cross-checking
against ko.json (which translates all of them) shows they are ordinary
UI copy, not intentional English terms. Verified post-merge that no
other ko-translated keys remain in English.

* docs: add crowscc as a contributor

* docs(openspec): drop brittle locale key count from tasks

The exact count goes stale whenever main adds keys; the requirement is
parity across en/zh-CN/ko, not a fixed number.

* fix(ui): address zh-CN localization review findings

---------

Co-authored-by: crowscc <17800561+crowscc@users.noreply.github.com>
Co-authored-by: Darafei Praliaskouski <me@komzpa.net>
)

Restores chronological order in the all-contributors list. Two entries (codemoo, diogenesc) had been inserted near the top of .all-contributorsrc inside their own feature PR commits instead of being appended; all 105 entries are now sorted by first merged PR date (maintainer pinned first), README grid regenerated with all-contributors-cli. No entries added or removed.
@Soju06

Soju06 commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Triage audit: this carrier is by design not mergeable, and its compatibility evidence is now stale — the head (last commit 07-22) predates the current state of the focused PRs it embeds (e.g. #1383 was rebased on 07-28), and all six embedded PRs (#1183, #1457, #1235, #1383, #1384, #1402) remain open as the actual review surfaces. Suggest closing this PR to keep the triage queue focused on mergeable work; the branch remains available, and a fresh carrier can be rebuilt from current main + current focused heads if cross-PR integration testing is needed again. @Komzpa if you prefer to keep it, please refresh it against the current heads and keep the body's head reference accurate.

@Soju06 Soju06 closed this Jul 30, 2026
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. db migration PR changes Alembic database migrations; maintainer must coordinate merge order

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants