fix: as many typos as possible (#16029)

* fix: as many typos as possible

* fix: even more typos

Author: smartcontracts

cannon/Makefile

@@ -24,7 +24,7 @@ cannon64-impl:
 	env GO111MODULE=on GOOS=$(TARGETOS) GOARCH=$(TARGETARCH) go build -v $(LDFLAGS) -o ./bin/cannon64-impl .
 
 # Note: This target is used by ./scripts/build-legacy-cannons.sh
-# It should build the individual versions of cannons and copy them into place in hte multicannon/embeds directory
+# It should build the individual versions of cannons and copy them into place in the multicannon/embeds directory
 # Ideally, preserve backwards compatibility with this behaviour but if it needs to change, build-legacy-cannons.sh will
 # need to be updated to account for different behaviours in different versions.
 # Each embed is suffixed with the latest `StateVersion` number corresponding to the target VM and architecture.

cannon/mipsevm/tests/fuzz_evm_multithreaded_test.go

@@ -46,12 +46,12 @@ func FuzzStateSyscallCloneMT(f *testing.F) {
 		expected.PrestateActiveThread().Registers[7] = 0
 		// Set expectations for new, cloned thread
 		expected.ActiveThreadId = nextThreadId
-		epxectedNewThread := expected.ExpectNewThread()
-		epxectedNewThread.PC = state.GetCpu().NextPC
-		epxectedNewThread.NextPC = state.GetCpu().NextPC + 4
-		epxectedNewThread.Registers[register.RegSyscallNum] = 0
-		epxectedNewThread.Registers[register.RegSyscallErrno] = 0
-		epxectedNewThread.Registers[register.RegSP] = stackPtr
+		expectedNewThread := expected.ExpectNewThread()
+		expectedNewThread.PC = state.GetCpu().NextPC
+		expectedNewThread.NextPC = state.GetCpu().NextPC + 4
+		expectedNewThread.Registers[register.RegSyscallNum] = 0
+		expectedNewThread.Registers[register.RegSyscallErrno] = 0
+		expectedNewThread.Registers[register.RegSP] = stackPtr
 		expected.NextThreadId = nextThreadId + 1
 		expected.StepsSinceLastContextSwitch = 0
 		if state.TraverseRight {

cannon/mipsevm/testutil/arch.go

@@ -57,7 +57,7 @@ func RandomizeWordAndSetUint32(mem *memory.Memory, addr Word, val uint32, random
 	exec.StoreSubWord(mem, addr, 4, Word(val), new(exec.NoopMemoryTracker))
 }
 
-// ToSignedInteger converts the unsigend Word to a SignedInteger.
+// ToSignedInteger converts the unsigned Word to a SignedInteger.
 // Useful for avoiding Go compiler warnings for literals that don't fit in a signed type
 func ToSignedInteger(x Word) arch.SignedInteger {
 	return arch.SignedInteger(x)

devnet-sdk/book/src/dsl/style_guide.md

@@ -69,7 +69,7 @@ Required inputs to methods are specified as normal parameters, so type checking
 
 Optional inputs to methods are specified by a config struct and accept a vararg of functions that can update that struct.
 This is roughly inline with the typical opts pattern in Golang but with significantly reduced boilerplate code since
-so many methods wil define their own config. With* methods are only provided for the most common optional args and
+so many methods will define their own config. With* methods are only provided for the most common optional args and
 tests will normally supply a custom function that sets all the optional values they need at once.
 
 ## Logging

devnet-sdk/system/periphery/go-ethereum/fees_test.go

@@ -49,7 +49,7 @@ func TestMultiplyBigInt(t *testing.T) {
 	}
 
 	for _, testCase := range testCases {
-		t.Run(fmt.Sprintf("should return %d for %d multplied by %f", testCase.expected.Int64(), testCase.value.Int64(), testCase.multiplier), func(t *testing.T) {
+		t.Run(fmt.Sprintf("should return %d for %d multiplied by %f", testCase.expected.Int64(), testCase.value.Int64(), testCase.multiplier), func(t *testing.T) {
 			result := multiplyBigInt(testCase.value, testCase.multiplier)
 			require.Equal(t, testCase.expected, result)
 		})
@@ -114,7 +114,7 @@ func TestEstimateEIP1559Fees(t *testing.T) {
 		})
 
 		t.Run("with custom tip multiplier", func(t *testing.T) {
-			t.Run("should set the GasTipCap to the client's suggested tip cap multplied by the tip multiplier", func(t *testing.T) {
+			t.Run("should set the GasTipCap to the client's suggested tip cap multiplied by the tip multiplier", func(t *testing.T) {
 				tipCapValue := big.NewInt(5)
 				tipMultiplier := 10.0
 				// The expected tip is a product of the tip cap and the tip multiplier
@@ -210,7 +210,7 @@ func TestEstimateEIP1559Fees(t *testing.T) {
 		})
 
 		t.Run("with custom base multiplier", func(t *testing.T) {
-			t.Run("should set the GasFeeCap to the block base fee multplied by the base multiplier", func(t *testing.T) {
+			t.Run("should set the GasFeeCap to the block base fee multiplied by the base multiplier", func(t *testing.T) {
 				baseMultiplier := 1.2
 				baseFeeValue := big.NewInt(9)
 				blockValue := types.NewBlock(&types.Header{

docs/handbook/pr-guidelines.md

@@ -35,7 +35,7 @@ This is organized by current state of PR, so it can be easily referenced frequen
 - **Guide PR reviewers:** Let them know about areas of concern, under-tested areas, or vague requirements that should be ironed out.
 
 ### Triggering CI on PRs from external forks
-If the PR is from an external fork, our CI suite will not automatically run on the PR. A reviewer with sufficient permissions (e.g. the automatically assigened reviewer) needs to comment on the PR wih
+If the PR is from an external fork, our CI suite will not automatically run on the PR. A reviewer with sufficient permissions (e.g. the automatically assigened reviewer) needs to comment on the PR with
 
 > /ci authorize COMMITHASH
 

kurtosis-devnet/README.md

@@ -98,7 +98,7 @@ interop-devnet: (devnet "interop.yaml")
 ## devnet output
 
 One important aspect of the devnet workflow is that the output should be
-*consumable*. Going forward we want to integrate them into larger worfklows
+*consumable*. Going forward we want to integrate them into larger workflows
 (serving as targets for tests for example, or any other form of automation).
 
 To address this, the deployment tool outputs a document with (hopefully!) useful
@@ -156,7 +156,7 @@ In particular, cleaning up a devnet can be achieved using
 
 ### Autofix mode
 
-Autofix mode helps recover from failed devnet deployments by automatically 
+Autofix mode helps recover from failed devnet deployments by automatically
 cleaning up the environment. It has two modes:
 
 1. **Normal Mode** (`AUTOFIX=true`)
@@ -189,7 +189,7 @@ older kurtosis engine. This typically happens if the kurtosis
 command-line managed by mise gets updated while some enclaves are
 already running.
 
-To help recover, you can either run with `AUTOFIX=nuke` or kill the 
+To help recover, you can either run with `AUTOFIX=nuke` or kill the
 old engine with:
 
 ```shell

op-acceptance-tests/tests/interop/interop_tx_test.go

@@ -91,7 +91,7 @@ func messagePassingScenario(sourceChainIdx, destChainIdx uint64, sourceWalletGet
 			ChainId:     chainA.ID(),
 		}
 
-		SatisfyExecMsgContraint(t, logger, sys, identifier.BlockNumber.Uint64(), identifier.Timestamp.Uint64())
+		SatisfyExecMsgConstraint(t, logger, sys, identifier.BlockNumber.Uint64(), identifier.Timestamp.Uint64())
 
 		// Execute message
 		logger.Info("Execute message", "address", sha256PrecompileAddr, "message", dummyMessage)
@@ -138,11 +138,11 @@ func TestMessagePassing(t *testing.T) {
 	sourceChainIdx := uint64(0)
 	destChainIdx := uint64(1)
 	sourceWalletGetter, sourcefundsValidator := validators.AcquireL2WalletWithFunds(sourceChainIdx, sdktypes.NewBalance(big.NewInt(1.0*constants.ETH)))
-	destWalletGetter, destfundsValiator := validators.AcquireL2WalletWithFunds(destChainIdx, sdktypes.NewBalance(big.NewInt(1.0*constants.ETH)))
+	destWalletGetter, destfundsValidator := validators.AcquireL2WalletWithFunds(destChainIdx, sdktypes.NewBalance(big.NewInt(1.0*constants.ETH)))
 
 	systest.InteropSystemTest(t,
 		messagePassingScenario(sourceChainIdx, destChainIdx, sourceWalletGetter, destWalletGetter),
 		sourcefundsValidator,
-		destfundsValiator,
+		destfundsValidator,
 	)
 }

op-acceptance-tests/tests/interop/interop_txplan_test.go

@@ -24,9 +24,9 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-// SatisfyExecMsgContraint waits supervisor to index initiating message,
+// SatisfyExecMsgConstraint waits supervisor to index initiating message,
 // and destination chain has advanced enough to satisfy messaging time invariant.
-func SatisfyExecMsgContraint(t systest.T, logger log.Logger, sys system.InteropSystem, initBlockNum uint64, initTimestamp uint64) {
+func SatisfyExecMsgConstraint(t systest.T, logger log.Logger, sys system.InteropSystem, initBlockNum uint64, initTimestamp uint64) {
 	ctx := t.Context()
 
 	supervisor, err := sys.Supervisor(ctx)
@@ -78,7 +78,7 @@ func initAndExecMsg(
 		blockA, err := txA.PlannedTx.IncludedBlock.Eval(ctx)
 		require.NoError(t, err)
 		// wait for supervisor and destination chain be in sync
-		SatisfyExecMsgContraint(t, logger, sys, blockA.Number, blockA.Time)
+		SatisfyExecMsgConstraint(t, logger, sys, blockA.Number, blockA.Time)
 
 		// Intent to validate message on chain B
 		txB := txintent.NewIntent[*txintent.ExecTrigger, *txintent.InteropOutput](opts[1])
@@ -125,7 +125,7 @@ func initAndExecMultipleMsg(
 		blockA, err := txA.PlannedTx.IncludedBlock.Eval(ctx)
 		require.NoError(t, err)
 		// wait for supervisor and destination chain be in sync
-		SatisfyExecMsgContraint(t, logger, sys, blockA.Number, blockA.Time)
+		SatisfyExecMsgConstraint(t, logger, sys, blockA.Number, blockA.Time)
 
 		// Intent to validate messages on chain B
 		txB := txintent.NewIntent[*txintent.MultiTrigger, *txintent.InteropOutput](opts[1])
@@ -169,7 +169,7 @@ func execSameMsgTwice(
 		blockA, err := txA.PlannedTx.IncludedBlock.Eval(ctx)
 		require.NoError(t, err)
 		// wait for supervisor and destination chain be in sync
-		SatisfyExecMsgContraint(t, logger, sys, blockA.Number, blockA.Time)
+		SatisfyExecMsgConstraint(t, logger, sys, blockA.Number, blockA.Time)
 
 		// Intent to validate same message two times on chain B
 		txB := txintent.NewIntent[*txintent.MultiTrigger, *txintent.InteropOutput](opts[1])
@@ -200,7 +200,7 @@ func execMsgDifferentTopicCount(
 		eventLoggerAddress, err := DeployEventLogger(ctx, wallets[0], logger)
 		require.NoError(t, err)
 
-		// Intent to initiate message with differet topic counts on chain A
+		// Intent to initiate message with different topic counts on chain A
 		initCalls := make([]txintent.Call, 5)
 		for topicCnt := range 5 {
 			index := topicCnt
@@ -223,7 +223,7 @@ func execMsgDifferentTopicCount(
 		blockA, err := txA.PlannedTx.IncludedBlock.Eval(ctx)
 		require.NoError(t, err)
 		// wait for supervisor and destination chain be in sync
-		SatisfyExecMsgContraint(t, logger, sys, blockA.Number, blockA.Time)
+		SatisfyExecMsgConstraint(t, logger, sys, blockA.Number, blockA.Time)
 
 		// Intent to validate message on chain B
 		txB := txintent.NewIntent[*txintent.MultiTrigger, *txintent.InteropOutput](opts[1])
@@ -242,9 +242,9 @@ func execMsgDifferentTopicCount(
 	}
 }
 
-// execMsgOpagueData tests below scenario:
+// execMsgOpaqueData tests below scenario:
 // Execute message that links with initiating message with: 0, 10KB of opaque event data in it
-func execMsgOpagueData(
+func execMsgOpaqueData(
 	l2ChainNums int,
 	walletGetters []validators.WalletGetter,
 ) systest.InteropSystemTestFunc {
@@ -275,7 +275,7 @@ func execMsgOpagueData(
 		blockA, err := txA.PlannedTx.IncludedBlock.Eval(ctx)
 		require.NoError(t, err)
 		// wait for supervisor and destination chain be in sync
-		SatisfyExecMsgContraint(t, logger, sys, blockA.Number, blockA.Time)
+		SatisfyExecMsgConstraint(t, logger, sys, blockA.Number, blockA.Time)
 
 		// Intent to validate messages on chain B
 		txB := txintent.NewIntent[*txintent.MultiTrigger, *txintent.InteropOutput](opts[1])
@@ -325,7 +325,7 @@ func execMsgDifferEventIndexInSingleTx(
 		blockA, err := txA.PlannedTx.IncludedBlock.Eval(ctx)
 		require.NoError(t, err)
 		// wait for supervisor and destination chain be in sync
-		SatisfyExecMsgContraint(t, logger, sys, blockA.Number, blockA.Time)
+		SatisfyExecMsgConstraint(t, logger, sys, blockA.Number, blockA.Time)
 
 		// Intent to validate messages on chain B
 		txB := txintent.NewIntent[*txintent.MultiTrigger, *txintent.InteropOutput](opts[1])
@@ -445,7 +445,7 @@ func executeMessageInvalidAttributes(
 		blockA, err := txA.PlannedTx.IncludedBlock.Eval(ctx)
 		require.NoError(t, err)
 		// wait for supervisor and destination chain be in sync
-		SatisfyExecMsgContraint(t, logger, sys, blockA.Number, blockA.Time)
+		SatisfyExecMsgConstraint(t, logger, sys, blockA.Number, blockA.Time)
 
 		// construct txplan opts for testing failed validating messages
 		optsForFail := txplan.Combine(
@@ -497,7 +497,7 @@ func randomDirectedGraph(
 
 		// pubSubPairCnt is the count of (publisher, subscriber) pairs which
 		// - publisher initiates messages
-		// - subsciber validates messages
+		// - subscriber validates messages
 		pubSubPairCnt := 10
 		// txCnt is the count of transactions that each publisher emits
 		txCnt := 3
@@ -649,11 +649,11 @@ func TestInteropExecMsgDifferentTopicCount(t *testing.T) {
 	)
 }
 
-func TestInteropExecMsgOpagueData(t *testing.T) {
+func TestInteropExecMsgOpaqueData(t *testing.T) {
 	l2ChainNums := 2
 	walletGetters, totalValidators := SetupDefaultInteropSystemTest(l2ChainNums)
 	systest.InteropSystemTest(t,
-		execMsgOpagueData(l2ChainNums, walletGetters),
+		execMsgOpaqueData(l2ChainNums, walletGetters),
 		totalValidators...,
 	)
 }

op-acceptance-tests/tests/interop/message/interop_msg_test.go

@@ -21,7 +21,7 @@ func TestInitExecMsg(gt *testing.T) {
 	eventLoggerAddress := alice.DeployEventLogger()
 	// Trigger random init message at chain A
 	initIntent, _ := alice.SendInitMessage(interop.RandomInitTrigger(rng, eventLoggerAddress, rng.Intn(5), rng.Intn(30)))
-	// Make sure supervisor indexs block which includes init message
+	// Make sure supervisor indexes block which includes init message
 	sys.Supervisor.AdvancedUnsafeHead(alice.ChainID(), 2)
 	// Single event in tx so index is 0
 	bob.SendExecMessage(initIntent, 0)

op-acceptance-tests/tests/interop/sync/redundant_interop/interop_sync_test.go

@@ -38,7 +38,7 @@ func TestUnsafeChainKnownToL2CL(gt *testing.T) {
 	sys.L2CLA.DisconnectPeer(sys.L2CLA2)
 	sys.L2CLA2.DisconnectPeer(sys.L2CLA)
 
-	// For making verifer not sync at all, both unsafe haead and safe head
+	// For making verifier not sync at all, both unsafe head and safe head
 	// The sequencer will advance unsafe head and safe head, as well as synced with supervisor.
 	logger.Info("stop verifier")
 	sys.L2CLA2.Stop()

op-acceptance-tests/tests/isthmus/preinterop/interop_readiness_test.go

@@ -61,10 +61,10 @@ func init() {
 }
 
 func TestInteropReadiness(t *testing.T) {
-	systest.SystemTest(t, interopReadinessTestSenario())
+	systest.SystemTest(t, interopReadinessTestScenario())
 }
 
-func interopReadinessTestSenario() systest.SystemTestFunc {
+func interopReadinessTestScenario() systest.SystemTestFunc {
 	return func(t systest.T, sys system.System) {
 		logger := testlog.Logger(t, log.LevelInfo)
 		logger.Info("Started test")

op-batcher/batcher/channel_manager.go

@@ -231,7 +231,7 @@ func (s *channelManager) TxData(l1Head eth.BlockID, isPectra bool) (txData, erro
 		return emptyTxData, err
 	}
 	// If the channel has already started being submitted,
-	// return now and ensure no requeueing happens
+	// return now and ensure no requeuing happens
 	if !channel.NoneSubmitted() {
 		return s.nextTxData(channel)
 	}

op-batcher/batcher/driver.go

@@ -400,7 +400,7 @@ func (l *BatchSubmitter) sendToThrottlingLoop(pendingBytesUpdated chan int64) {
 	}
 }
 
-// trySignal trys to send an emptry struct  on the provided channel.
+// trySignal tries to send an empty struct on the provided channel.
 // It is not blocking, no signal will be sent if the channel is full.
 func trySignal(c chan struct{}) {
 	select {
@@ -715,7 +715,7 @@ func (l *BatchSubmitter) waitNodeSync() error {
 }
 
 // publishStateToL1 queues up all pending TxData to be published to the L1, returning when there is no more data to
-// queue for publishing or if there was an error queing the data.
+// queue for publishing or if there was an error queuing the data.
 func (l *BatchSubmitter) publishStateToL1(ctx context.Context, queue *txmgr.Queue[txRef], receiptsCh chan txmgr.TxReceipt[txRef], daGroup *errgroup.Group) {
 	for {
 		select {

op-batcher/batcher/driver_test.go

@@ -251,7 +251,7 @@ func TestBatchSubmitter_ThrottlingEndpoints(t *testing.T) {
 			// Start throttling loop in a goroutine
 			go bs.throttlingLoop(&wg1, pendingBytesUpdated)
 
-			// Simulate block loading by sending perodically on pendingBytesUpdated
+			// Simulate block loading by sending periodically on pendingBytesUpdated
 			wg2 := sync.WaitGroup{}
 			blockLoadingCtx, cancelBlockLoading := context.WithCancel(context.Background())
 			defer cancelBlockLoading()

op-batcher/batcher/sync_actions.go

@@ -66,7 +66,7 @@ func computeSyncActions[T channelStatuser](
 
 	safeL2 := newSyncStatus.SafeL2
 	if preferLocalSafeL2 {
-		// This is preffered when running interop, but not yet enabled by default.
+		// This is preferred when running interop, but not yet enabled by default.
 		safeL2 = newSyncStatus.LocalSafeL2
 	}
 

op-chain-ops/script/forking/db.go

@@ -24,7 +24,7 @@ type ForkDB struct {
 
 // Reader for read-only access to a known state. All cold reads go through this.
 // So the state-DB creates one initially, and then holds on to it.
-// The diff will be overlayed on the reader still. To get rid of the diff, it has to be explicitly cleared.
+// The diff will be overlaid on the reader still. To get rid of the diff, it has to be explicitly cleared.
 // Warning: diffs applied to the original state that the reader wraps will be visible.
 // Geth StateDB is meant to be reinitialized after committing state.
 func (f *ForkDB) Reader(root common.Hash) (state.Reader, error) {

op-chain-ops/script/script.go

@@ -452,7 +452,7 @@ func (h *Host) SetNonce(addr common.Address, nonce uint64) {
 	h.state.SetNonce(addr, nonce, tracing.NonceChangeUnspecified)
 }
 
-// GetNonce returs an account's nonce from state.
+// GetNonce returns an account's nonce from state.
 func (h *Host) GetNonce(addr common.Address) uint64 {
 	return h.state.GetNonce(addr)
 }

op-challenger/README.md

@@ -214,7 +214,7 @@ The same CLI options as `op-challenger` itself are supported to configure the tr
 option allows specifying which prestates to use. The format is `traceType/name/prestateHash` where traceType is the
 trace type to use with the prestate (e.g cannon or asterisc-kona), name is an arbitrary name for the prestate to use
 when reporting metrics and prestateHash is the hex encoded absolute prestate commitment to use. If name is omitted the
-trace type name is used.If the prestateHash is omitted, the absolute prestate hash used for new games on-chain.
+trace type name is used. If the prestateHash is omitted, the absolute prestate hash used for new games on-chain is used.
 
 For example to run both the production cannon prestate and a custom
 prestate, use `--run cannon,cannon/next-prestate/0x03c1f0d45248190f80430a4c31e24f8108f05f80ff8b16ecb82d20df6b1b43f3`.

op-challenger/game/fault/solver/game_solver_test.go

@@ -257,7 +257,7 @@ func TestMultipleRounds(t *testing.T) {
 			actor: incorrectAttackLastClaim,
 		},
 		{
-			name:  "LinearDefendInorrect",
+			name:  "LinearDefendIncorrect",
 			actor: incorrectDefendLastClaim,
 		},
 		{

op-challenger/game/fault/trace/vm/op_program_server_executor_test.go

@@ -182,7 +182,7 @@ func TestOpProgramFillHostCommand(t *testing.T) {
 		require.Equal(t, common.Bytes2Hex(val), pairs["--l2.agreed-prestate"])
 	})
 
-	t.Run("WithouDepsetConfig", func(t *testing.T) {
+	t.Run("WithoutDepsetConfig", func(t *testing.T) {
 		pairs := oracleCommand(t, log.LvlInfo, func(c *Config, _ *utils.LocalGameInputs) {
 			c.DepsetConfigPath = ""
 		})