Address gobinding inconsistency

[huangzhen1997]

Summary

• Updated TLB opcodes for MessageValidated (#cbc4af76 → #1fa60374) and MessageValidationFailed (#0f756150 → #bcf0ab0f) to match on-chain changes
• Changed Context field encoding from maybe ^ to . (RemainingBitsAndRefs pattern) in FeeQuoter message types
• Renamed Metadata field to Context in ccipsendexecutor structs
• Fixed TokenAmount.Amount type from *big.Int to *tlb.Coins to match on-chain encoding
• Updated PendingOwner in ownable2step and FeeToken in CCIPSend to use maybe addr for optional addresses
• Updated offramp error codes: renamed ErrorSenderIsNotRouter → ErrorInsufficientFee, added ErrorTooManyMessagesInReport
• Added input validation and error handling to PackGasPrice/UnpackGasPrice functions (nil checks, negative value checks, bit overflow checks)
• Fixed WithdrawFeeTokens.FeeTokens encoding from inline to ref cell (^)
• Removed obsolete WithdrawJettons struct and added new OnRamp event topics
• Restructured UpdateTokenTransferFeeConfig to match on-chain dictionary format
• Removed the duplicate types and reused the correct ones from the feequoter package

[huangzhen1997]

@vicentevieytes why are we using uint64 on-chain for TokenPriceStalenessThreshold ?

[Copilot]

Pull request overview

This PR addresses inconsistencies between the Go bindings and on-chain TON contract schemas by updating TLB (Type-Length-Value) encodings, opcodes, and error codes to match the current on-chain implementation.

Changes:

• Updated TLB opcodes and field encodings (Context, FeeToken, PendingOwner, TokenAmount) to match on-chain schema changes
• Added comprehensive input validation and error handling to PackGasPrice/UnpackGasPrice functions with nil checks, negative value checks, and bit overflow checks
• Restructured types to eliminate duplication by reusing feequoter package types in ccipsendexecutor, and updated UpdateTokenTransferFeeConfig to match on-chain dictionary format

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
pkg/ccip/codec/commitcodec.go	Updated to handle new error returns from PackGasPrice/UnpackGasPrice functions
pkg/ccip/chainaccessor/ton_accessor.go	Added error handling for PackGasPrice calls
pkg/ccip/chainaccessor/config.go	Added overflow check for uint64 to uint32 conversion of StalenessThreshold
pkg/ccip/bindings/router/router.go	Changed TokenAmount.Amount from *big.Int to tlb.Coins and FeeToken to optional address
pkg/ccip/bindings/ownable2step/storage.go	Updated PendingOwner encoding to use "maybe addr" for optional addresses
pkg/ccip/bindings/onramp/onramp_test.go	Added test cases to verify CRC32 values for event topics
pkg/ccip/bindings/onramp/onramp.go	Updated event topics, renamed structs, removed obsolete WithdrawJettons, and fixed WithdrawFeeTokens encoding
pkg/ccip/bindings/offramp/offramp.go	Updated error codes by renaming ErrorSenderIsNotRouter to ErrorInsufficientFee and adding ErrorTooManyMessagesInReport
pkg/ccip/bindings/offramp/exitcode_string.go	Regenerated string representations for updated error codes
pkg/ccip/bindings/feequoter/reader.go	Updated StalenessThreshold type from uint32 to uint64 and added documentation for GetDestChainConfig
pkg/ccip/bindings/feequoter/gaspriceutils_test.go	Added comprehensive error case tests for PackGasPrice/UnpackGasPrice and renamed helper function
pkg/ccip/bindings/feequoter/gaspriceutils.go	Added input validation with error returns for nil, negative, and bit overflow checks
pkg/ccip/bindings/feequoter/fee_quoter.go	Updated Context field encoding, restructured UpdateTokenTransferFeeConfig, and added documentation
pkg/ccip/bindings/ccipsendexecutor/ccipsendexecutor.go	Removed duplicate types and reused feequoter package types

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The struct name 'TokenAmount' is confusing because it appears in multiple packages (router, ccipsendexecutor) with the same name but potentially different purposes. Consider adding a package-specific prefix (e.g., ExecutorTokenAmount) or documenting why this duplicates the router.TokenAmount type instead of reusing it.

Suggested change

	// TokenAmount structure (reused from router package concept)
	// TokenAmount represents an amount of a given token for the CCIP Send Executor.
	// It intentionally mirrors the schema and naming of router.TokenAmount instead of
	// reusing that type directly, to keep this bindings package self-contained and to
	// match the on-chain / TLB schema for executor messages.

[huangzhen1997]

Hey @patricios-space, can you help review these off-chain bindings updates

[krebernisak]

Thanks for the comment - imo this is super confusing and the naming is not best we can do.

[krebernisak]

Looks good! Needs CI fix.

[krebernisak]

This was stale/unused, right?