ops: post-merge leftover after PRs 1302-1304 #1305

Closed
opened 2026-09-21 06:14:44 +00:00 by PlasticDigits · 4 comments

Tracking leftover ops after merging PRs 1302–1304 onto origin/main (729b097f).

Merged

PR Linked Note
1302 #1290 / #1240 make verify-issue-1290 bundle (P1240 + hubPriceTicker + Playwright)
1303 #1277 / #1292 trader lifetime heal from swap_events (migrate + gated poller before D5)
1304 #1303 / #1277 sqlx 0.8 &mut **tx so apply_trader_lifetime_heal_tx compiles

Woodpecker ci/woodpecker/pr/woodpecker and ci/woodpecker/push/woodpecker succeeded. Local gitleaks clean.

Workstation verify (this merge)

  • make verify-issue-1290 4/4 including Playwright protocol-page (--workers=5)
  • make verify-issue-1277 6/6 after #1304 (docs, code, I10 rolling+heal, #577 decay, #548 catalog, api_traders)
  • #1303 as merged did not compile (Executor not implemented for &mut Transaction); fixed in #1304

Leftover (this issue)

  1. Coolify /protocol visual (#1290 AC): hub <dt> cUSTC / USD + cLUNC / USD vs oracle tabs USTC / LUNC / vFDUSD (selected tab not VFDUSD).
  2. Coolify indexer migrate: apply 20260921130000_traders_lifetime_heal_from_swaps after the #1277 widen + #1263 30d versions already listed on #1300. Confirm _sqlx_migrations then D5 refresh / heal no-op or log.
  3. ADR 0008 collision: do not merge origin/cac-design-issue-1300, origin/cac-design-issue-1302, or origin/cac-design-issue-1277 as-is. Other workers; 0005 is protocol-fee; two branches both want 0008. Re-slot if a design ADR is still needed.
  4. Optional I10: USD-only lifetime skew does not re-trip the poller gate (gate is trades+raw only).
  5. Merge recipe: .* @code/maintainers CODEOWNERS + block_on_official_review_requests makes fj pr merge return HTTP 405 until the team self-request is dismissed. Dismiss then fj pr merge. Do not force_merge (docs/runbooks/forgejo-pr-merge.md). #1302/#1303 were merged with force_merge before this was followed; both commits are ancestors of origin/main. #1304 used dismiss + normal merge.

Sister leftover for PRs 1287–1298 remains #1300.

Tracking leftover ops after merging PRs **1302–1304** onto `origin/main` (`729b097f`). ## Merged | PR | Linked | Note | |----|--------|------| | 1302 | #1290 / #1240 | `make verify-issue-1290` bundle (P1240 + hubPriceTicker + Playwright) | | 1303 | #1277 / #1292 | trader lifetime heal from `swap_events` (migrate + gated poller before D5) | | 1304 | #1303 / #1277 | sqlx 0.8 `&mut **tx` so `apply_trader_lifetime_heal_tx` compiles | Woodpecker `ci/woodpecker/pr/woodpecker` and `ci/woodpecker/push/woodpecker` succeeded. Local gitleaks clean. ## Workstation verify (this merge) - [x] `make verify-issue-1290` 4/4 including Playwright `protocol-page` (`--workers=5`) - [x] `make verify-issue-1277` 6/6 **after #1304** (docs, code, I10 rolling+heal, #577 decay, #548 catalog, `api_traders`) - [x] #1303 as merged did **not** compile (`Executor` not implemented for `&mut Transaction`); fixed in #1304 ## Leftover (this issue) 1. **Coolify `/protocol` visual** (#1290 AC): hub `<dt>` `cUSTC / USD` + `cLUNC / USD` vs oracle tabs `USTC` / `LUNC` / `vFDUSD` (selected tab not `VFDUSD`). 2. **Coolify indexer migrate**: apply `20260921130000_traders_lifetime_heal_from_swaps` after the #1277 widen + #1263 30d versions already listed on #1300. Confirm `_sqlx_migrations` then D5 refresh / heal no-op or log. 3. **ADR 0008 collision**: do **not** merge `origin/cac-design-issue-1300`, `origin/cac-design-issue-1302`, or `origin/cac-design-issue-1277` as-is. Other workers; `0005` is protocol-fee; two branches both want `0008`. Re-slot if a design ADR is still needed. 4. **Optional I10**: USD-only lifetime skew does not re-trip the poller gate (gate is trades+raw only). 5. **Merge recipe**: `.* @code/maintainers` CODEOWNERS + `block_on_official_review_requests` makes `fj pr merge` return HTTP 405 until the team self-request is dismissed. Dismiss then `fj pr merge`. Do not `force_merge` (`docs/runbooks/forgejo-pr-merge.md`). #1302/#1303 were merged with `force_merge` before this was followed; both commits **are** ancestors of `origin/main`. #1304 used dismiss + normal merge. Sister leftover for PRs 1287–1298 remains **#1300**.
Author
Owner

cl8y-agent-control: queued design_author job 7e99e9de-e820-4832-b906-baade8f9bb09 (not executed; no Hetzner VM).

cl8y-agent-control: queued `design_author` job `7e99e9de-e820-4832-b906-baade8f9bb09` (not executed; no Hetzner VM).
Author
Owner

Leftover ops (complete)

Coolify leftover for this ticket is done. Indexer and frontend are on origin/main 729b097f.

Indexer

  • GET /health → status=ok, git_sha=729b097f5fe1f52565740a4207a6703efc646b8a
  • Healthy boot implies sqlx 20260921120000 / 20260921120001 / 20260921130000 applied (dirty migrate would fail start)
  • GET /api/v1/protocol/top-pairs → HTTP 200, 5 economic rows (UST1/cUSTC, UST1/USTR, cLUNC/cUSTC, CL8Y-cb/cUSTC, CL8Y-cb/cLUNC)
  • ?limit=6 → HTTP 400 Invalid limit, expected 5

Frontend /protocol

Playwright against production passed:

  • Hub <dt>: cUSTC / USD and cLUNC / USD (mixed case; not flattened CUSTC)
  • Oracle tabs: USTC (selected), LUNC, vFDUSD (not VFDUSD)

Local make verify-issue-1290 still has one Playwright tick-overlap fail (YY-MM monthly x-axis). That is not this Coolify leftover.

Closing this issue. Chain leftover (pair wasm 1.17.0, AC1 gas, Lunc Dash device QA) stays on #1300 / #1234 / #1264 / #1279.

## Leftover ops (complete) Coolify leftover for this ticket is done. Indexer and frontend are on origin/main `729b097f`. ### Indexer - `GET /health` → `status=ok`, `git_sha=729b097f5fe1f52565740a4207a6703efc646b8a` - Healthy boot implies sqlx `20260921120000` / `20260921120001` / `20260921130000` applied (dirty migrate would fail start) - `GET /api/v1/protocol/top-pairs` → HTTP 200, 5 economic rows (UST1/cUSTC, UST1/USTR, cLUNC/cUSTC, CL8Y-cb/cUSTC, CL8Y-cb/cLUNC) - `?limit=6` → HTTP 400 `Invalid limit, expected 5` ### Frontend `/protocol` Playwright against production passed: - Hub `<dt>`: `cUSTC / USD` and `cLUNC / USD` (mixed case; not flattened `CUSTC`) - Oracle tabs: `USTC` (selected), `LUNC`, `vFDUSD` (not `VFDUSD`) Local `make verify-issue-1290` still has one Playwright tick-overlap fail (`YY-MM` monthly x-axis). That is not this Coolify leftover. Closing this issue. Chain leftover (pair wasm 1.17.0, AC1 gas, Lunc Dash device QA) stays on #1300 / #1234 / #1264 / #1279.
PlasticDigits 2026-09-21 06:46:53 +00:00
  • closed this issue
  • removed the
    ready
    label
Author
Owner

Reopened as a leftover precondition. The 06:46 close comment (top-pairs + /health SHA) is #1300 / ADR 0006, not leftover-complete for this card. Leftover-complete still records on #1305 after ADR 0010 operator evidence.

Reopened as a leftover precondition. The 06:46 close comment (`top-pairs` + `/health` SHA) is #1300 / ADR 0006, not leftover-complete for this card. Leftover-complete still records on #1305 after ADR 0010 operator evidence.
Author
Owner

PRs 1306 / 1309 / 1310 are on origin/main (0d6eaeee). Catch-all CODEOWNERS is gone (this issue leftover 5). New leftover for Coolify frontend + Lunc Dash device AC after !1310 closed #1308: #1311. This ticket still owns leftover 1 (/protocol visual) and leftover 2 (indexer migrate). Do not merge cac-design-issue-1300 / 1302 / 1306 as-is (leftover 3).

PRs **1306 / 1309 / 1310** are on `origin/main` (`0d6eaeee`). Catch-all CODEOWNERS is gone (this issue leftover 5). New leftover for Coolify frontend + Lunc Dash device AC after !1310 closed #1308: [#1311](https://git.cl8y.com/code/cl8y-dex-terraclassic/issues/1311). This ticket still owns leftover 1 (`/protocol` visual) and leftover 2 (indexer migrate). Do not merge `cac-design-issue-1300` / `1302` / `1306` as-is (leftover 3).
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
code/cl8y-dex-terraclassic#1305
No description provided.