fix: runTrigger with transfer and use oneOf for transfer_log and evm_log

[chrisli30]

No description provided.

[Copilot]

Pull Request Overview

This PR enforces mutually exclusive evm_log and transfer_log fields via a oneof in EventTrigger.Output, updates all code paths and tests to use the new structure, and enhances the event‐trigger search strategy with chain‐specific ranges and improved logging.

• Introduce oneof output_type in protobuf/avs.proto and regenerate Go bindings.
• Refactor operator, VM, engine, and RPC handler to set and read OutputType instead of raw EvmLog/TransferLog fields.
• Improve runEventTriggerImmediately with 3-month/6-month search ranges, chunked queries, richer logs, and a 60 s timeout; update docs and tests.

Reviewed Changes

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

Show a summary per file

File	Description
protobuf/avs.proto	Add oneof output_type to EventTrigger.Output for exclusive fields
protobuf/avs.pb.go	Regenerate bindings to support the new OutputType interface
operator/worker_loop.go	Set OutputType when notifying the aggregator
docs/EventTrigger-Chain-Specific-Search.md	Add a new doc outlining 3- and 6-month chain-specific ranges
core/testutil/utils.go	Update test util to populate OutputType in dummy trigger data
core/taskengine/vm_test.go	Update tests to construct oneof data and initialize VM without separate transfer param
core/taskengine/vm.go	Remove old EvmLog/TransferLog fields; use Event oneof
core/taskengine/token_metadata_test.go	Adjust test for whitelist-only metadata behavior
core/taskengine/token_metadata.go	Change comment to clarify nil return when no RPC client
core/taskengine/run_node_immediately.go	Switch to chain-specific search ranges, richer logging, and 60 s timeout
core/taskengine/engine.go	Refactor enrichment to set OutputType for transfer vs. raw events
core/taskengine/blockchain_constants.go	Add GetBlockSearchRanges and GetChainSearchRanges helpers
core/taskengine/blockchain_constants_test.go	New tests for block range calculations and slice helper

Comments suppressed due to low confidence (3)

core/taskengine/vm_test.go:538

• The tests no longer cover the raw EvmLog branch of the new oneof OutputType. Add a test case where only an EvmLog is provided to verify the VM correctly populates and retrieves the EvmLog branch.

func TestEvaluateEvent(t *testing.T) {

core/taskengine/token_metadata_test.go:276

• [nitpick] The test name TestGetTokenMetadataNotFound no longer matches its behavior of returning nil metadata without error. Consider renaming it to something like TestGetTokenMetadataWhitelistOnlyReturnsNil for clarity.

func TestGetTokenMetadataNotFound(t *testing.T) {

protobuf/avs.proto:164

• [nitpick] Grammar nit: These shape of data is is incorrect. Consider changing to The shape of the data is or This shape of data is.

// These shape of data is https://docs.ethers.org/v6/api/providers/#Log