test(defi/clob): replace JS tests with LiteSVM Rust, align with repo conventions

Bring the CLOB example in line with the repo's Anchor 1.0 + LiteSVM Rust
test convention established by tokens/escrow and defi/asset-leasing.
Previously CLOB shipped a TypeScript suite backed by Codama-generated
clients and @solana/kit, which was a one-off in the current examples
set.

Why:
- Every other defi/*/anchor and the tokens/escrow example now uses
  LiteSVM-driven Rust tests that `include_bytes!` the built .so,
  share a common test-stack (litesvm + solana-kite + solana-signer),
  and run under a plain `cargo test`. Contributors moving between
  examples had to relearn the CLOB harness (Codama client generation,
  surfpool-vs-legacy validator selection, tsx + node:test runner).
- The JS suite also required `anchor test --validator legacy` because
  Anchor 1.0's default surfpool validator does not expose the
  websocket RPC methods Kit uses for confirmation. Switching to
  LiteSVM Rust sidesteps that entirely — no validator at all.

Changes:
- Anchor.toml: drop anchor_version, package_manager, [hooks], [test]
  blocks. Set `[scripts] test = "cargo test"` matching asset-leasing.
  Keep solana_version = 3.1.8 pinned so BPF toolchain stays in lock-step.
- programs/clob/Cargo.toml: add [dev-dependencies] for litesvm 0.11,
  solana-signer 3.0, solana-keypair 3.0.1, solana-kite 0.3. Versions
  match the rest of the repo to avoid drift.
- programs/clob/tests/test_clob.rs: 13 LiteSVM tests covering
  initialize_market (happy path + zero-tick + oversized-fee rejection),
  create_user_account, place_order (bid locks quote, ask locks base,
  zero-price / unaligned-tick / below-min rejections), cancel_order
  (owner refund credited to unsettled, non-owner rejected), and
  settle_funds (drains unsettled balance from vault to user ATA for
  both an ask cancel and a bid cancel + full refund round-trip).
- programs/clob/src/lib.rs and instructions/*.rs: rename the verbose
  `*AccountConstraints` struct suffix to the plain `InitializeMarket`,
  `PlaceOrder`, etc. naming that every other Anchor example in the
  repo uses. Rename handler functions from bare `initialize_market` to
  `handle_initialize_market` to match escrow and asset-leasing.
- README.md: replace the TS/Codama test instructions with the Rust
  LiteSVM flow; drop the surfpool caveat (no longer relevant).
- Remove tests/clob.test.ts, package.json, tsconfig.json: no JS/TS
  scaffolding needed now that testing is pure Rust.
- .gitignore: drop the now-unused `dist` entry (the Codama client
  output directory).

Scope note carried into the tests: the program does not run a
matching engine, it only keeps the book and escrow the funds. The
test file's header comment calls this out so the reader does not
look for cross-order settlement tests that cannot exist yet.

Test result: `anchor build && cargo test` → 13 passed, 0 failed.

Author: Edward (OpenClaw)

defi/clob/anchor/.gitignore

@@ -5,4 +5,3 @@ target
 node_modules
 test-ledger
 .yarn
-dist

defi/clob/anchor/Anchor.toml

@@ -1,8 +1,7 @@
 [toolchain]
-anchor_version = "1.0.0"
+# Pin Solana to the version used across the repo's Anchor 1.0 examples so the
+# bundled test validator and BPF toolchain stay in lock-step.
 solana_version = "3.1.8"
-# pnpm matches the program-examples root package manager.
-package_manager = "pnpm"
 
 [features]
 resolution = true
@@ -12,23 +11,10 @@ skip-lint = false
 clob = "C69UJ8irfmHq5ysyLek7FKApHR86FBeupiz4JnoyPzzx"
 
 [provider]
-cluster = "localnet"
+cluster = "Localnet"
 wallet = "~/.config/solana/id.json"
 
 [scripts]
-# Generate the Codama TS client from the built IDL, then run node:test tests
-# via tsx. Run with: `anchor test --validator legacy`
-#
-# The `legacy` flag selects `solana-test-validator`, which Kit can talk to.
-# Anchor 1.0's default "surfpool" validator does not accept the websocket RPC
-# methods Kit uses for confirmation, so tests time out waiting for their
-# first transaction. Revisit when surfpool adds full Kit support.
-test = "npx create-codama-clients && npx tsx --test --test-reporter=spec tests/*.ts"
-
-[hooks]
-
-[test]
-# Give the local validator enough time to become reachable before tests start.
-startup_wait = 10000
-shutdown_wait = 2000
-upgradeable = false
+# LiteSVM Rust tests live under `programs/clob/tests/` and include the built
+# `.so` via `include_bytes!`, so a fresh `anchor build` must run first.
+test = "cargo test"

defi/clob/anchor/README.md

@@ -34,13 +34,11 @@ anchor build
 ## Test
 
 ```shell
-anchor test --validator legacy
+anchor test
 ```
 
-The `--validator legacy` flag is required: Anchor 1.0's default "surfpool" validator does not yet accept the websocket RPC methods Solana Kit uses for transaction confirmation, so tests hang waiting for their first transaction. `solana-test-validator` works.
-
-The test script (defined in `Anchor.toml`) first runs `npx create-codama-clients` to generate a TypeScript client from the built IDL into `dist/clob-client/`, then executes the `node:test` suite with `tsx`.
+Tests are pure Rust, running against [LiteSVM](https://github.com/LiteSVM/litesvm). They live in `programs/clob/tests/test_clob.rs` and include the built `.so` via `include_bytes!`, so a fresh `anchor build` must run first. `anchor test` does this automatically; alternatively run `anchor build && cargo test`.
 
 ## Credit
 
-Ported and modernised from [anchor-decentralized-exchange-clob](https://github.com/mikemaccana/anchor-decentralized-exchange-clob). Migrated from Anchor 0.32.1 to Anchor 1.0.0 and conformed to the [Solana Anchor coding skill](https://github.com/mikemaccana/solana-anchor-claude-skill) (Kit + Kite + Codama, `node:test`, no `@coral-xyz/anchor`, no magic numbers, `Box`-ed interface accounts to keep BPF stack size within budget).
+Ported and modernised from [anchor-decentralized-exchange-clob](https://github.com/mikemaccana/anchor-decentralized-exchange-clob). Migrated from Anchor 0.32.1 to Anchor 1.0.0 and conformed to the repo's LiteSVM-Rust-tests convention (no magic numbers, `Box`-ed interface accounts to keep BPF stack size within budget).

defi/clob/anchor/package.json

@@ -23,5 +23,14 @@ custom-panic = []
 anchor-lang = "1.0.0"
 anchor-spl = "1.0.0"
 
+[dev-dependencies]
+# Match the test stack used by tokens/escrow, defi/asset-leasing, and the
+# other LiteSVM-based Anchor examples so contributors can move between them
+# without version drift.
+litesvm = "0.11.0"
+solana-signer = "3.0.0"
+solana-keypair = "3.0.1"
+solana-kite = "0.3.0"
+
 [lints.rust]
 unexpected_cfgs = { level = "warn", check-cfg = ['cfg(target_os, values("solana"))'] }

defi/clob/anchor/programs/clob/Cargo.toml

@@ -6,7 +6,7 @@ use crate::state::{
     OrderStatus, UserAccount, ORDER_BOOK_SEED, ORDER_SEED, USER_ACCOUNT_SEED,
 };
 
-pub fn cancel_order(context: Context<CancelOrderAccountConstraints>) -> Result<()> {
+pub fn handle_cancel_order(context: Context<CancelOrder>) -> Result<()> {
     let order = &mut context.accounts.order;
 
     require!(
@@ -58,7 +58,7 @@ pub fn cancel_order(context: Context<CancelOrderAccountConstraints>) -> Result<(
 }
 
 #[derive(Accounts)]
-pub struct CancelOrderAccountConstraints<'info> {
+pub struct CancelOrder<'info> {
     pub market: Account<'info, Market>,
 
     #[account(

defi/clob/anchor/programs/clob/src/instructions/cancel_order.rs

@@ -2,7 +2,7 @@ use anchor_lang::prelude::*;
 
 use crate::state::{Market, UserAccount, USER_ACCOUNT_SEED};
 
-pub fn create_user_account(context: Context<CreateUserAccountAccountConstraints>) -> Result<()> {
+pub fn handle_create_user_account(context: Context<CreateUserAccount>) -> Result<()> {
     let user_account = &mut context.accounts.user_account;
     user_account.market = context.accounts.market.key();
     user_account.owner = context.accounts.owner.key();
@@ -15,7 +15,7 @@ pub fn create_user_account(context: Context<CreateUserAccountAccountConstraints>
 }
 
 #[derive(Accounts)]
-pub struct CreateUserAccountAccountConstraints<'info> {
+pub struct CreateUserAccount<'info> {
     #[account(
         init,
         payer = owner,

defi/clob/anchor/programs/clob/src/instructions/create_user_account.rs

@@ -8,8 +8,8 @@ use crate::state::{Market, OrderBook, MARKET_SEED, ORDER_BOOK_SEED};
 // would be nonsensical, so we cap here.
 const MAX_FEE_BASIS_POINTS: u16 = 10_000;
 
-pub fn initialize_market(
-    context: Context<InitializeMarketAccountConstraints>,
+pub fn handle_initialize_market(
+    context: Context<InitializeMarket>,
     fee_basis_points: u16,
     tick_size: u64,
     min_order_size: u64,
@@ -46,7 +46,7 @@ pub fn initialize_market(
 }
 
 #[derive(Accounts)]
-pub struct InitializeMarketAccountConstraints<'info> {
+pub struct InitializeMarket<'info> {
     #[account(
         init,
         payer = authority,

defi/clob/anchor/programs/clob/src/instructions/initialize_market.rs

@@ -13,8 +13,8 @@ use crate::state::{
 // PlaceOrder check reads clearly and the limit is documented in one place.
 const MAX_OPEN_ORDERS_PER_USER: usize = 20;
 
-pub fn place_order(
-    context: Context<PlaceOrderAccountConstraints>,
+pub fn handle_place_order(
+    context: Context<PlaceOrder>,
     side: OrderSide,
     price: u64,
     quantity: u64,
@@ -105,7 +105,7 @@ pub fn place_order(
 
 #[derive(Accounts)]
 #[instruction(side: OrderSide, price: u64, quantity: u64)]
-pub struct PlaceOrderAccountConstraints<'info> {
+pub struct PlaceOrder<'info> {
     #[account(mut)]
     pub market: Account<'info, Market>,
 

defi/clob/anchor/programs/clob/src/instructions/place_order.rs

@@ -5,7 +5,7 @@ use anchor_spl::token_interface::{
 
 use crate::state::{Market, UserAccount, MARKET_SEED, USER_ACCOUNT_SEED};
 
-pub fn settle_funds(context: Context<SettleFundsAccountConstraints>) -> Result<()> {
+pub fn handle_settle_funds(context: Context<SettleFunds>) -> Result<()> {
     let user_account = &mut context.accounts.user_account;
     let market = &context.accounts.market;
 
@@ -63,7 +63,7 @@ pub fn settle_funds(context: Context<SettleFundsAccountConstraints>) -> Result<(
 }
 
 #[derive(Accounts)]
-pub struct SettleFundsAccountConstraints<'info> {
+pub struct SettleFunds<'info> {
     #[account(mut)]
     pub market: Account<'info, Market>,
 
@@ -74,7 +74,7 @@ pub struct SettleFundsAccountConstraints<'info> {
     )]
     pub user_account: Account<'info, UserAccount>,
 
-    // Boxed for the same reason as in PlaceOrderAccountConstraints —
+    // Boxed for the same reason as in PlaceOrder —
     // InterfaceAccount is too large to keep on the BPF stack in bulk.
     #[account(mut)]
     pub base_vault: Box<InterfaceAccount<'info, TokenAccount>>,