feat(pair): read-only OrderStatus query (#505) #1042

Merged
PlasticDigits merged 1 commit from feat/505-order-status-query into main 2026-08-05 02:05:57 +00:00
PlasticDigits commented 2026-08-05 02:03:21 +00:00 (Migrated from gitlab.com)

Summary

  • Adds pair QueryMsg::OrderStatus { order_id } returning typed Active / ParkedRefund / Unknown from existing ORDERS and EXPIRED_LIMIT_CLAIMS only (GitLab #505).
  • Public schema in dex_common::pair; no new storage, tombstones, owner index, or execute-path dual-writes. Bumps pair cw2 to 1.9.0.
  • Documents invariant L21, integrator notes, and agent playbook skills/AGENTS_ORDER_STATUS_QUERY.md.

Caller note: use OrderStatus for contract consumers. Unknown means no live custody row — not proof of fill. Transport/decode failures must stay errors.

Closes #505

Test plan

  • Unit: cargo test -p cl8y-dex-pair order_status
  • Integration: cargo test -p cl8y-dex-tests order_status (T1–T12 matrix + pause + legacy query unchanged)
  • Migration: pair_migration_preserves_* asserts post-migrate OrderStatus Active (T13)
  • Clippy -D warnings on dex-common + pair; cargo fmt --check
  • Review: no ORDER_TOMB / OWNER_ORDERS / execute inventory writes
  • Reviewer: confirm no execute-path gas/state growth in diff
## Summary - Adds pair `QueryMsg::OrderStatus { order_id }` returning typed `Active` / `ParkedRefund` / `Unknown` from existing `ORDERS` and `EXPIRED_LIMIT_CLAIMS` only (GitLab #505). - Public schema in `dex_common::pair`; no new storage, tombstones, owner index, or execute-path dual-writes. Bumps pair cw2 to `1.9.0`. - Documents invariant **L21**, integrator notes, and agent playbook `skills/AGENTS_ORDER_STATUS_QUERY.md`. **Caller note:** use `OrderStatus` for contract consumers. `Unknown` means no live custody row — **not** proof of fill. Transport/decode failures must stay errors. Closes #505 ## Test plan - [x] Unit: `cargo test -p cl8y-dex-pair order_status` - [x] Integration: `cargo test -p cl8y-dex-tests order_status` (T1–T12 matrix + pause + legacy query unchanged) - [x] Migration: `pair_migration_preserves_*` asserts post-migrate `OrderStatus` Active (T13) - [x] Clippy `-D warnings` on `dex-common` + pair; `cargo fmt --check` - [x] Review: no `ORDER_TOMB` / `OWNER_ORDERS` / execute inventory writes - [ ] Reviewer: confirm no execute-path gas/state growth in diff
project_80162261_bot_e646d24e9b17aa4b22040bfe08d3fd1c commented 2026-08-05 02:03:24 +00:00 (Migrated from gitlab.com)

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the [Cursor dashboard](https://www.cursor.com/dashboard/bugbot) to get automatic reviews on future PRs.
PlasticDigits commented 2026-08-05 02:03:43 +00:00 (Migrated from gitlab.com)

mentioned in issue #505

mentioned in issue #505
PlasticDigits (Migrated from gitlab.com) merged commit 42f1b455d4 into main 2026-08-05 02:05:57 +00:00
PlasticDigits commented 2026-08-05 02:06:37 +00:00 (Migrated from gitlab.com)

mentioned in commit 42f1b455d4

mentioned in commit 42f1b455d4556d6ffca8990920e498c87d8ea179
Sign in to join this conversation.
No reviewers
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!1042
No description provided.