fixed ci fmt and updated readmes

Author: bengtlofgren

.cspell/custom-dictionary.txt

@@ -1,10 +1,15 @@
 # Custom Dictionary Words
 Aptos
+borsh
+Borsh
 CCTP
 Devnet
 hashv
 idls
 keccak
+keypair
+keypairs
+KEYPAIR
 Linea
 Mezo
 permissionless

.prettierignore

@@ -1,3 +1,11 @@
 evm/cache/
 evm/lib/
 evm/out/
+evm/broadcast/
+svm/anchor/target/
+svm/node_modules/
+svm/modules/*/target/
+svm/pinocchio/target/
+svm/pinocchio/programs/*/target/
+svm/pinocchio/tests/*/target/
+svm/target/

README.md

@@ -188,7 +188,6 @@ This will generate:
 After building, you can view the API documentation using:
 
 1. **Swagger Editor** (Online)
-
    - Visit https://editor.swagger.io/
    - Copy and paste the contents of `tsp-output/schema/openapi.yaml`
 

cspell.json

@@ -1,6 +1,6 @@
 {
   "version": "0.2",
-  "ignorePaths": [],
+  "ignorePaths": ["node_modules/", "svm/node_modules/", "evm/lib/"],
   "dictionaryDefinitions": [
     {
       "name": "custom-dictionary",

design/02_On_Chain_Quotes.md

@@ -7,7 +7,7 @@ Some would-be Executor integrators may need on-chain quotes as they do not neces
 ## Runtime Support
 
 - [x] [EVM](./evm/)
-- [ ] [SVM](./svm/)
+- [x] [SVM](./svm/)
 
 # **Background**
 
@@ -91,7 +91,6 @@ interface IExecutorQuoter {
 ```
 
 2. **ExecutorQuoterRouter** replaces **Executor** as the entry-point for integrators. It MUST be immutable and non-administered / fully permissionless. This provides three critical functionalities.
-
    1. `updateQuoterContract(bytes calldata gov)` allows a Quoter to set their `ExecutorQuoter` contract via signed governance (detailed below). This MUST
       1. Verify the chain ID matches the Executor’s `ourChain`.
       2. Verify the contract address is an EVM address.
@@ -135,11 +134,33 @@ interface IExecutorQuoterRouter {
 
 ### SVM
 
-The SVM implementation should follow the requirements above relevant to the SVM Executor implementation.
+On SVM, two programs are introduced.
+
+1. **ExecutorQuoter** is a Pinocchio-based program that implements the quoting logic. It exposes two CPI-callable instructions:
+   - `RequestQuote` (discriminator `[2, 0, 0, 0, 0, 0, 0, 0]`): Returns `(payee_address, required_payment)` via return data.
+   - `RequestExecutionQuote` (discriminator `[3, 0, 0, 0, 0, 0, 0, 0]`): Returns `(required_payment, payee_address, quote_body)` as 72 bytes via return data.
+
+   The quoter reads pricing data from on-chain PDAs (`ChainInfo`, `QuoteBody`) maintained by an authorized updater.
+
+2. **ExecutorQuoterRouter** is the entry-point for integrators. It provides three instructions:
+   1. `UpdateQuoterContract` registers or updates a quoter's implementation mapping. This MUST:
+      - Verify the chain ID matches `OUR_CHAIN`.
+      - Verify the sender matches `universal_sender_address` in the governance message.
+      - Verify the governance has not expired.
+      - Verify the secp256k1 signature recovers to the quoter address (20-byte EVM address).
+      - Create or update a `QuoterRegistration` PDA seeded by `["quoter_registration", quoter_address]`.
+
+   2. `QuoteExecution` gets a quote from a registered quoter via CPI. The quoter's return data is forwarded to the caller.
+
+   3. `RequestExecution` requests execution through the router. This MUST:
+      - CPI to the quoter's `RequestExecutionQuote` to get payment/payee/quote body.
+      - Verify payment amount is sufficient.
+      - Construct an `EQ02` signed quote on-chain.
+      - CPI to Executor's `request_for_execution`.
+
+   **Account Handling**: The router uses a fixed account layout for quoter CPIs. The quoter accounts (`config`, `chain_info`, `quote_body`) are passed by the client and forwarded to the quoter program. This allows different quoter implementations to use different account structures while maintaining a standardized router interface.
 
-<aside>
-⚠️ TODO: The primary additional consideration is how to handle the accounts used for fetching the quote from an ExecutorQuoter in a standardized way.
-</aside>
+   **Quoter Identity**: Quoters are identified by their 20-byte EVM address (derived from secp256k1 public key). The `QuoterRegistration` PDA maps this address to a Solana program ID that implements the quoting logic.
 
 ### Other
 

svm/pinocchio/README.md

@@ -11,9 +11,9 @@ These programs use the [Pinocchio](https://github.com/febo/pinocchio) framework,
 
 ## Devnet Deployments
 
-| Program | Address |
-|---------|---------|
-| executor-quoter | `957QU51h6VLbnbAmAPgtXz1kbFE1QhchDmNfgugW9xCc` |
+| Program                | Address                                        |
+| ---------------------- | ---------------------------------------------- |
+| executor-quoter        | `957QU51h6VLbnbAmAPgtXz1kbFE1QhchDmNfgugW9xCc` |
 | executor-quoter-router | `5pkyS8pnbbMforEqAR91gkgPeBs5XKhWpiuuuLdw6hbk` |
 
 ## Directory Structure

svm/pinocchio/programs/executor-quoter-router/README.md

@@ -9,15 +9,18 @@ The router manages quoter registrations and routes CPI calls to the appropriate
 ## Instructions
 
 **UpdateQuoterContract (discriminator: 0)**
+
 - Registers or updates a quoter's implementation mapping
 - Accounts: `[payer, sender, config, quoter_registration, system_program]`
 
 **QuoteExecution (discriminator: 1)**
+
 - Gets a quote from a registered quoter via CPI
 - Accounts: `[quoter_registration, quoter_program, config, chain_info, quote_body]`
 - CPI to quoter's `RequestQuote` instruction (discriminator: `[2, 0, 0, 0, 0, 0, 0, 0]`)
 
 **RequestExecution (discriminator: 2)**
+
 - Executes cross-chain request through the router
 - Accounts: `[payer, config, quoter_registration, quoter_program, executor_program, payee, refund_addr, system_program, quoter_config, chain_info, quote_body, event_cpi]`
 - CPI to quoter's `RequestExecutionQuote` instruction (discriminator: `[3, 0, 0, 0, 0, 0, 0, 0]`)
@@ -27,11 +30,13 @@ The router manages quoter registrations and routes CPI calls to the appropriate
 Quoter implementations must support the following CPI interface:
 
 ### RequestQuote
+
 - Discriminator: 8 bytes (`[2, 0, 0, 0, 0, 0, 0, 0]`)
 - Accounts: `[config, chain_info, quote_body]`
 - Returns: `u64` (big endian) payment amount via `set_return_data`
 
 ### RequestExecutionQuote
+
 - Discriminator: 8 bytes (`[3, 0, 0, 0, 0, 0, 0, 0]`)
 - Accounts: `[config, chain_info, quote_body, event_cpi]`
 - Returns: 72 bytes via `set_return_data`:

svm/pinocchio/programs/executor-quoter/README.md

@@ -9,11 +9,13 @@ The quoter interface is defined by CPI calls from the `executor-quoter-router` p
 ### Required Instructions
 
 **RequestQuote (discriminator: 2)**
+
 - Returns a quote for cross-chain execution
 - Accounts: `[config, chain_info, quote_body]`
 - Returns: `u64` payment amount (8 bytes, big-endian)
 
 **RequestExecutionQuote (discriminator: 3)**
+
 - Returns full execution details including payment, payee, and quote body
 - Accounts: `[config, chain_info, quote_body, event_cpi]`
 - Returns: 72 bytes (`u64` payment + 32-byte payee address + 32-byte quote body)
@@ -23,6 +25,7 @@ Both instructions support up to 8-byte discriminators for Anchor compatibility (
 ### Instruction Data Layout
 
 Both instructions share the same input format (after discriminator):
+
 - `dst_chain`: u16 (LE)
 - `dst_addr`: [u8; 32]
 - `refund_addr`: [u8; 32]