Rewrite SequencingHooks as an interface and don't stop sequencing on large transactions

[amsanghi]

Fixes: NIT-3857

[github-actions]

❌ 4 Tests Failed:

Tests completed	Failed	Passed	Skipped
2142	4	2138	0

View the top 3 failed tests by shortest run time

TestSequencerBlockFilterAccept

Stack Traces | 0.210s run time

=== RUN   TestSequencerBlockFilterAccept
=== PAUSE TestSequencerBlockFilterAccept
=== CONT  TestSequencerBlockFilterAccept
    seq_filter_test.go:72: goroutine 55 [running]:
        runtime/debug.Stack()
        	/opt/hostedtoolcache/go/1.25.3/x64/src/runtime/debug/stack.go:26 +0x5e
        github.com/offchainlabs/nitro/util/testhelpers.RequireImpl({0x4083890, 0xc000483dc0}, {0x40415a0, 0xc000510ea0}, {0x0, 0x0, 0x0})
        	/home/runner/work/nitro/nitro/util/testhelpers/testhelpers.go:29 +0x55
        github.com/offchainlabs/nitro/system_tests.Require(0xc000483dc0, {0x40415a0, 0xc000510ea0}, {0x0, 0x0, 0x0})
        	/home/runner/work/nitro/nitro/system_tests/common_test.go:1723 +0x5d
        github.com/offchainlabs/nitro/system_tests.TestSequencerBlockFilterAccept(0xc000483dc0)
        	/home/runner/work/nitro/nitro/system_tests/seq_filter_test.go:72 +0xd9
        testing.tRunner(0xc000483dc0, 0x3ccb8a0)
        	/opt/hostedtoolcache/go/1.25.3/x64/src/testing/testing.go:1934 +0xea
        created by testing.(*T).Run in goroutine 1
        	/opt/hostedtoolcache/go/1.25.3/x64/src/testing/testing.go:1997 +0x465
        
    seq_filter_test.go:72: �[31;1m [] error fetching next transaction to sequence, userTxsProcessed: 0, err: FullSequencingHooks: GetNextTx detected out of order request to sequence tx. hookTxErrors: 0, nextTxIdToBeSequenced: 1 �[0;0m
--- FAIL: TestSequencerBlockFilterAccept (0.21s)

TestSequencerBlockFilterAccept

Stack Traces | 1.110s run time

... [CONTENT TRUNCATED: Keeping last 20 lines]
WARN [10-17|15:23:12.378] Sequencer ReadFromTxQueueTimeout is higher than MaxBlockSpeed ReadFromTxQueueTimeout=1s MaxBlockSpeed=10ms
=== PAUSE TestSequencerBlockFilterAccept
=== CONT  TestSequencerBlockFilterAccept
    seq_filter_test.go:72: goroutine 1253871 [running]:
        runtime/debug.Stack()
        	/opt/hostedtoolcache/go/1.25.3/x64/src/runtime/debug/stack.go:26 +0x5e
        github.com/offchainlabs/nitro/util/testhelpers.RequireImpl({0x4083890, 0xc06a1fc8c0}, {0x40415a0, 0xc062ac4a00}, {0x0, 0x0, 0x0})
        	/home/runner/work/nitro/nitro/util/testhelpers/testhelpers.go:29 +0x55
        github.com/offchainlabs/nitro/system_tests.Require(0xc06a1fc8c0, {0x40415a0, 0xc062ac4a00}, {0x0, 0x0, 0x0})
        	/home/runner/work/nitro/nitro/system_tests/common_test.go:1723 +0x5d
        github.com/offchainlabs/nitro/system_tests.TestSequencerBlockFilterAccept(0xc06a1fc8c0)
        	/home/runner/work/nitro/nitro/system_tests/seq_filter_test.go:72 +0xd9
        testing.tRunner(0xc06a1fc8c0, 0x3ccb8a0)
        	/opt/hostedtoolcache/go/1.25.3/x64/src/testing/testing.go:1934 +0xea
        created by testing.(*T).Run in goroutine 1
        	/opt/hostedtoolcache/go/1.25.3/x64/src/testing/testing.go:1997 +0x465
        
    seq_filter_test.go:72: �[31;1m [] error fetching next transaction to sequence, userTxsProcessed: 0, err: FullSequencingHooks: GetNextTx detected out of order request to sequence tx. hookTxErrors: 0, nextTxIdToBeSequenced: 1 �[0;0m
ERROR[10-17|15:26:49.788] Dangling trie nodes after full cleanup
--- FAIL: TestSequencerBlockFilterAccept (1.11s)

TestVersion40

Stack Traces | 5.310s run time

... [CONTENT TRUNCATED: Keeping last 20 lines]
=== PAUSE TestVersion40
=== CONT  TestVersion40
    precompile_inclusion_test.go:90: goroutine 454828 [running]:
        runtime/debug.Stack()
        	/opt/hostedtoolcache/go/1.25.3/x64/src/runtime/debug/stack.go:26 +0x5e
        github.com/offchainlabs/nitro/util/testhelpers.RequireImpl({0x4083890, 0xc0b1c9afc0}, {0x4041a20, 0xc0355cff20}, {0x0, 0x0, 0x0})
        	/home/runner/work/nitro/nitro/util/testhelpers/testhelpers.go:29 +0x55
        github.com/offchainlabs/nitro/system_tests.Require(0xc0b1c9afc0, {0x4041a20, 0xc0355cff20}, {0x0, 0x0, 0x0})
        	/home/runner/work/nitro/nitro/system_tests/common_test.go:1723 +0x5d
        github.com/offchainlabs/nitro/system_tests.testPrecompiles(0xc0b1c9afc0, 0x28, {0xc08a28bdf8, 0x5, 0x39?})
        	/home/runner/work/nitro/nitro/system_tests/precompile_inclusion_test.go:90 +0x371
        github.com/offchainlabs/nitro/system_tests.TestVersion40(0xc0b1c9afc0?)
        	/home/runner/work/nitro/nitro/system_tests/precompile_inclusion_test.go:71 +0x64b
        testing.tRunner(0xc0b1c9afc0, 0x3ccbb00)
        	/opt/hostedtoolcache/go/1.25.3/x64/src/testing/testing.go:1934 +0xea
        created by testing.(*T).Run in goroutine 1
        	/opt/hostedtoolcache/go/1.25.3/x64/src/testing/testing.go:1997 +0x465
        
    precompile_inclusion_test.go:90: �[31;1m [] execution aborted (timeout = 5s) �[0;0m
--- FAIL: TestVersion40 (5.31s)

📣 Thoughts on this report? Let Codecov know! | Powered by Codecov