chore(precompiles): cleanup

[0xrusowsky]

this PR:

• drops encode_to_slot / decode_from_slot in favor of typed patterns leveraging StorageCtx
• remove dead dispatch code and unused helpers
• exposes precompile constants from tempo-contracts so alloy consumers don't need tempo-precompiles deps
• clean up stale TODOs and comments

NOTE: no behavioral change

[github-actions]

📊 Tempo Precompiles Coverage

📦 Download full HTML report

[github-actions]

⚠️ Changelog not found.

A changelog entry is required before merging. We've generated a suggested changelog based on your changes:

Preview

---
tempo-contracts: minor
tempo-precompiles: major
tempo-transaction-pool: minor
---

Removed `encode_to_slot`/`decode_from_slot` helpers from `AuthorizedKey` and `PolicyData`, and removed the `validator_tokens` public method from `TipFeeManager` in favor of the existing `get_validator_token`. Exported shared constants (`DECIMALS`, `MIN_TICK`, `MAX_TICK`, `PRICE_SCALE`) from `tempo-contracts` and refactored the transaction pool's AMM liquidity cache to use a read-only storage context instead of raw slot decoding.

Add changelog to commit this to your branch.

[0xKitsune]

nit: The reason that Maker transfer policy auth is not checked is to ensure that swaps can complete, rather than for gas efficiency. Consider updating the comment to something like the following:

    /// NOTE: Maker transfer policy is not enforced here to not block swaps on the pair. 
    /// Note that TIP403 checks on order placement and withdraws are enforced.
    /// [`cancel_stale_order`](Self::cancel_stale_order) can be used to remove orders. 

[0xKitsune]

nit: same thing here, it is important to not fail on 403 policies during swaps to ensure swap execution rather than this being a gas optimization.

[0xKitsune]

I think this should be fine, but lets make sure re-execution passes. Previously self.validator_tokens returned a Vec.

[0xrusowsky]

i dropped it cause it returned an Address, so it's identical:

    /// Reads the stored fee token preference for a validator, defaulting to [`DEFAULT_FEE_TOKEN`].
    pub fn validator_tokens(&self, call: IFeeManager::validatorTokensCall) -> Result<Address> {
        let token = self.validator_tokens[call.validator].read()?;

        if token.is_zero() {
            Ok(DEFAULT_FEE_TOKEN)
        } else {
            Ok(token)
        }
    }

    /// Returns the validator's preferred fee token, falling back to [`DEFAULT_FEE_TOKEN`].
    pub fn get_validator_token(&self, beneficiary: Address) -> Result<Address> {
        let token = self.validator_tokens[beneficiary].read()?;

        if token.is_zero() {
            Ok(DEFAULT_FEE_TOKEN)
        } else {
            Ok(token)
        }
    }