ops: post-merge !433/!434/!435 leftover verify #654

Closed
opened 2026-08-26 01:58:31 +00:00 by PlasticDigits · 12 comments
PlasticDigits commented 2026-08-26 01:58:31 +00:00 (Migrated from gitlab.com)

Parent

Merged on main (no CI wait, no automerge):

  • !1153 — #651 /tiers phone-width cards (4295334f)
  • !1152 — #652 Protocol inline Δ% + UTC daily volume (640bb28f)
  • !1151 — #653 one-chrome-layer / flat StatBox (c90c8934)

Conflict resolution for !1151 onto main (already containing !1153 + !1152) is 7245ac1c: keep #652 inline Δ% / daily chart on Protocol tiles; keep #653 stat-flat + hint + chrome guard.

Do not reopen #651 / #652 / #653 unless a merged invariant is wrong.

Leftovers

Operator / Coolify

  • #652 Apply indexer migration 20260826120000_protocol_volume_change_and_daily.sql on Coolify Postgres, then redeploy indexer. Already live 2026-08-26 (daily route 200; no further indexer migrate).
  • Confirm live GET /api/v1/overview includes volume_change_{24h,7d,30d}_pct (string or null). 24h/7d populated; 30d null until the prior 30d window fills.
  • Confirm live GET /api/v1/protocol/volume/daily?days=7 and days=30 return Protocol-catalog points (not Llama). days=1 / 90 / junk → 400.
  • Coolify frontend rebuild from main (c90c8934+). Live bundle index-CZcnodo-.js (was index-aylIou1i.js). /protocol has 7d/30d UTC chart + inline volume Δ%. /tiers 375px: 9 Hold phrases, no .w-28. 15 .stat-flat on Protocol. Real-phone QA still below.

Manual / device

  • #651 iPhone Brave and Safari, disconnected https://dex.cl8y.com/tiers: intact Hold {n} CL8Y + fee phrases; no empty Register column.
  • #651 connected Register ≥44px, correct register-tier-{id}; Android Chrome ~360×800 disconnected + connected.
  • #651 light + dark on a real phone. LocalTerra Register tier 1 is layout-only (msgs unchanged).
  • #652 /protocol at 375px and 1280px, light + dark: one liquidity cell ($ + % 24h + % 30d); volume/fees USD + inline Δ%; no card-in-card in Global stats / Protocol fees.
  • #653 /charts, /trader/:addr, /protocol, / at 375 + 1280, light + dark — no empty-ring tiles. Swap Pay/Receive still card-glass. Trade chart still a single shell-panel-strong. Theme toggle: no leftover brown/gold fills.

Do not

  • Wait on GitLab CI quota for the merges above (already on main; pipelines failed ci_quota_exceeded).
  • Add Llama from/to or N+1 GET /defillama/daily for the Protocol chart.
  • Flatten Swap IO cards or Trade sibling panels.

Verify (already run on source tips)

make verify-issue-651   # 9/9 including Playwright fee-tiers 14/14, 5 workers
make verify-issue-652   # 8/8 indexer lib+integration + Protocol RTL + e2e-smoke protocol-page (5 workers) + related 550/586
make verify-issue-653   # 14/14 StatBox/Charts/Trader/Protocol + check_chrome_nesting + tokens

Post-conflict tip (7245ac1c): StatBox + ProtocolPage + Charts + Trader RTL 82/82; python3 scripts/check_chrome_nesting.py; python3 scripts/check_design_tokens.py.

## Parent Merged on `main` (no CI wait, no automerge): - [!1153](https://gitlab.com/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/435) — #651 `/tiers` phone-width cards (`4295334f`) - [!1152](https://gitlab.com/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/434) — #652 Protocol inline Δ% + UTC daily volume (`640bb28f`) - [!1151](https://gitlab.com/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/433) — #653 one-chrome-layer / flat StatBox (`c90c8934`) Conflict resolution for !1151 onto `main` (already containing !1153 + !1152) is `7245ac1c`: keep #652 inline Δ% / daily chart on Protocol tiles; keep #653 `stat-flat` + `hint` + chrome guard. Do **not** reopen #651 / #652 / #653 unless a merged invariant is wrong. ## Leftovers ### Operator / Coolify - [x] #652 Apply indexer migration `20260826120000_protocol_volume_change_and_daily.sql` on Coolify Postgres, then redeploy indexer. **Already live 2026-08-26** (daily route 200; no further indexer migrate). - [x] Confirm live `GET /api/v1/overview` includes `volume_change_{24h,7d,30d}_pct` (string or `null`). 24h/7d populated; 30d `null` until the prior 30d window fills. - [x] Confirm live `GET /api/v1/protocol/volume/daily?days=7` and `days=30` return Protocol-catalog points (not Llama). `days=1` / `90` / junk → **400**. - [x] Coolify **frontend rebuild** from `main` (`c90c8934+`). Live bundle `index-CZcnodo-.js` (was `index-aylIou1i.js`). `/protocol` has 7d/30d UTC chart + inline volume Δ%. `/tiers` 375px: 9 Hold phrases, no `.w-28`. 15 `.stat-flat` on Protocol. Real-phone QA still below. ### Manual / device - [ ] #651 iPhone Brave **and** Safari, disconnected `https://dex.cl8y.com/tiers`: intact **Hold {n} CL8Y** + fee phrases; no empty Register column. - [ ] #651 connected Register ≥44px, correct `register-tier-{id}`; Android Chrome ~360×800 disconnected + connected. - [ ] #651 light + dark on a real phone. LocalTerra Register tier 1 is layout-only (msgs unchanged). - [ ] #652 `/protocol` at 375px and 1280px, light + dark: one liquidity cell (`$` + `% 24h` + `% 30d`); volume/fees USD + inline Δ%; no card-in-card in Global stats / Protocol fees. - [ ] #653 `/charts`, `/trader/:addr`, `/protocol`, `/` at 375 + 1280, light + dark — no empty-ring tiles. Swap Pay/Receive still `card-glass`. Trade chart still a single `shell-panel-strong`. Theme toggle: no leftover brown/gold fills. ### Do not - Wait on GitLab CI quota for the merges above (already on `main`; pipelines failed `ci_quota_exceeded`). - Add Llama `from`/`to` or N+1 `GET /defillama/daily` for the Protocol chart. - Flatten Swap IO cards or Trade sibling panels. ## Verify (already run on source tips) ```bash make verify-issue-651 # 9/9 including Playwright fee-tiers 14/14, 5 workers make verify-issue-652 # 8/8 indexer lib+integration + Protocol RTL + e2e-smoke protocol-page (5 workers) + related 550/586 make verify-issue-653 # 14/14 StatBox/Charts/Trader/Protocol + check_chrome_nesting + tokens ``` Post-conflict tip (`7245ac1c`): StatBox + ProtocolPage + Charts + Trader RTL 82/82; `python3 scripts/check_chrome_nesting.py`; `python3 scripts/check_design_tokens.py`.
PlasticDigits commented 2026-08-26 01:58:32 +00:00 (Migrated from gitlab.com)

marked as related to #651

marked as related to #651
PlasticDigits commented 2026-08-26 01:58:33 +00:00 (Migrated from gitlab.com)

marked as related to #652

marked as related to #652
PlasticDigits commented 2026-08-26 01:58:33 +00:00 (Migrated from gitlab.com)

marked as related to #653

marked as related to #653
PlasticDigits commented 2026-08-26 01:58:42 +00:00 (Migrated from gitlab.com)

mentioned in issue #651

mentioned in issue #651
PlasticDigits commented 2026-08-26 01:58:44 +00:00 (Migrated from gitlab.com)

mentioned in issue #652

mentioned in issue #652
PlasticDigits commented 2026-08-26 01:58:45 +00:00 (Migrated from gitlab.com)

mentioned in issue #653

mentioned in issue #653
PlasticDigits commented 2026-08-26 02:56:46 +00:00 (Migrated from gitlab.com)

marked the checklist item Confirm live GET /api/v1/protocol/volume/daily?days=7 and days=30 return Protocol-catalog points (not Llama). days=1 / 90 / junk → 400. as completed

marked the checklist item **Confirm live GET /api/v1/protocol/volume/daily?days\=7 and days\=30 return Protocol\-catalog points \(not Llama\)\. days\=1 / 90 / junk → 400\.** as completed
PlasticDigits commented 2026-08-26 02:56:49 +00:00 (Migrated from gitlab.com)

Agent leftover pass 2026-08-26 (main c90c8934)

Indexer APIs for #652 are already live. Coolify frontend is still the 2026-08-25 bundle.

Done (no more Coolify indexer work)

Probe Result
GET https://indexer.dex.cl8y.com/health 200
GET /api/v1/overview volume_change_24h_pct ~155.10, volume_change_7d_pct ~315.46, volume_change_30d_pct null (30d prior window empty — expected)
GET /api/v1/protocol/volume/daily?days=7 200 methodology=protocol_catalog, 7 UTC-day points
GET …/daily?days=30 200 Protocol catalog (not Llama)
days=1 / 90 / junk / missing 400 Invalid days, expected 7 or 30

No new Coolify env vars. Migration 20260826120000_protocol_volume_change_and_daily.sql is already applied on production (route would 404 otherwise).

Still required — Coolify frontend rebuild (no env change)

Production https://dex.cl8y.com still serves assets/index-aylIou1i.js (same as the 2026-08-25 #638 note). DOM check on /protocol:

  • protocol-volume-daily-chart absent
  • .stat-flat count 0; .card-glass count 22
  • Liquidity Δ% still sibling tiles (24h liquidity / 30d liquidity), not one cell
  • Volume tiles have no inline flow Δ%

/tiers at 375px still has 9 reserved .w-28 Register columns; no .tiers-how-it-works-mobile.

Coolify: rebuild the frontend image from main (c90c8934 or later). Keep existing bake:

VITE_NETWORK=mainnet
VITE_INDEXER_URL=https://indexer.dex.cl8y.com
VITE_COMMUNITY_TAX_CODE_ID=11630
VITE_COMMUNITY_MIGRATE_CODE_IDS=6036,10184,8266,8654

Do not set VITE_SHOW_TEST_TOKENS / VITE_FAUCET_ADDRESS. Do not append 3. After rebuild, bundle hash must change; then:

curl -sS https://dex.cl8y.com/ | grep -oE 'assets/index-[^"]+'
# must not be index-aylIou1i.js

Then device QA on the leftover list (iPhone Brave+Safari /tiers, /protocol 375+1280 light/dark, /charts /trader empty-ring tiles).

Local verify re-run on this tip

make verify-issue-651   # 9/9, Playwright fee-tiers 14/14
make verify-issue-652   # 8/8 including protocol-page e2e-smoke (5 workers) + related 550/586
make verify-issue-653   # 14/14
## Agent leftover pass 2026-08-26 (`main` `c90c8934`) Indexer APIs for #652 are **already live**. Coolify **frontend** is still the 2026-08-25 bundle. ### Done (no more Coolify indexer work) | Probe | Result | |-------|--------| | `GET https://indexer.dex.cl8y.com/health` | **200** | | `GET /api/v1/overview` | `volume_change_24h_pct` ~155.10, `volume_change_7d_pct` ~315.46, `volume_change_30d_pct` **null** (30d prior window empty — expected) | | `GET /api/v1/protocol/volume/daily?days=7` | **200** `methodology=protocol_catalog`, 7 UTC-day points | | `GET …/daily?days=30` | **200** Protocol catalog (not Llama) | | `days=1` / `90` / `junk` / missing | **400** `Invalid days, expected 7 or 30` | No new Coolify **env vars**. Migration `20260826120000_protocol_volume_change_and_daily.sql` is already applied on production (route would 404 otherwise). ### Still required — Coolify frontend rebuild (no env change) Production `https://dex.cl8y.com` still serves `assets/index-aylIou1i.js` (same as the 2026-08-25 #638 note). DOM check on `/protocol`: - `protocol-volume-daily-chart` **absent** - `.stat-flat` count **0**; `.card-glass` count **22** - Liquidity Δ% still sibling tiles (`24h liquidity` / `30d liquidity`), not one cell - Volume tiles have no inline flow Δ% `/tiers` at 375px still has **9** reserved `.w-28` Register columns; no `.tiers-how-it-works-mobile`. **Coolify:** rebuild the **frontend** image from `main` (`c90c8934` or later). Keep existing bake: ``` VITE_NETWORK=mainnet VITE_INDEXER_URL=https://indexer.dex.cl8y.com VITE_COMMUNITY_TAX_CODE_ID=11630 VITE_COMMUNITY_MIGRATE_CODE_IDS=6036,10184,8266,8654 ``` Do **not** set `VITE_SHOW_TEST_TOKENS` / `VITE_FAUCET_ADDRESS`. Do **not** append `3`. After rebuild, bundle hash must change; then: ```bash curl -sS https://dex.cl8y.com/ | grep -oE 'assets/index-[^"]+' # must not be index-aylIou1i.js ``` Then device QA on the leftover list (iPhone Brave+Safari `/tiers`, `/protocol` 375+1280 light/dark, `/charts` `/trader` empty-ring tiles). ### Local verify re-run on this tip ``` make verify-issue-651 # 9/9, Playwright fee-tiers 14/14 make verify-issue-652 # 8/8 including protocol-page e2e-smoke (5 workers) + related 550/586 make verify-issue-653 # 14/14 ```
PlasticDigits commented 2026-08-26 02:56:52 +00:00 (Migrated from gitlab.com)

mentioned in issue #638

mentioned in issue #638
PlasticDigits commented 2026-08-26 03:20:29 +00:00 (Migrated from gitlab.com)

changed the description

changed the description
PlasticDigits commented 2026-08-26 03:20:30 +00:00 (Migrated from gitlab.com)

Coolify frontend rebuild verified (2026-08-26)

Production left index-aylIou1i.js. Now https://dex.cl8y.com serves assets/index-CZcnodo-.js + index-Btz7QHG2.css.

Playwright against live (networkidle):

/protocol 1280

  • protocol-volume-daily-chart present; toggles protocol-volume-daily-7d / 30d
  • Liquidity one cell: $61K + +19.23%24h / —30d
  • Volume inline Δ%: Last 24h $3.7K +155.1%; 7d +315.5%; 30d em-dash
  • Fees inline Δ%: 24h +222.0%
  • .stat-flat count 15; no nested card-glass in Global stats / fees grids

/tiers 375 disconnected

  • tier-hold-1…9 — Hold 1 CL8Y … Hold 7.5K CL8Y
  • .w-28 count 0 (no empty Register column)
  • tiers-how-it-works-mobile present
  • Register buttons absent while disconnected (expected)

/charts 1280 — 8 .stat-flat overview tiles

Real iPhone Brave/Safari + Android connected Register + light/dark theme still need a device. No env change in this rebuild.

## Coolify frontend rebuild verified (2026-08-26) Production left `index-aylIou1i.js`. Now `https://dex.cl8y.com` serves **`assets/index-CZcnodo-.js`** + `index-Btz7QHG2.css`. Playwright against live (networkidle): **`/protocol` 1280** - `protocol-volume-daily-chart` present; toggles `protocol-volume-daily-7d` / `30d` - Liquidity one cell: `$61K` + `+19.23%24h` / `—30d` - Volume inline Δ%: Last 24h `$3.7K` **+155.1%**; 7d **+315.5%**; 30d em-dash - Fees inline Δ%: 24h **+222.0%** - `.stat-flat` count **15**; no nested `card-glass` in Global stats / fees grids **`/tiers` 375 disconnected** - `tier-hold-1`…`9` — **Hold 1 CL8Y** … **Hold 7.5K CL8Y** - `.w-28` count **0** (no empty Register column) - `tiers-how-it-works-mobile` present - Register buttons absent while disconnected (expected) **`/charts` 1280** — 8 `.stat-flat` overview tiles Real iPhone Brave/Safari + Android connected Register + light/dark theme still need a device. No env change in this rebuild.
PlasticDigits commented 2026-08-27 05:52:34 +00:00 (Migrated from gitlab.com)

No user reports related to changes, closing

No user reports related to changes, closing
PlasticDigits (Migrated from gitlab.com) closed this issue 2026-08-27 05:52:34 +00:00
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#654
No description provided.