chore(timeboost): drop dead code, dedupe tx helpers, simplify no-bid path

- Remove 7 setXxxLogger setters (never called) and switch `let log` → `const log`.
- Drop dead exports: DOCS_DEFAULT_TIMING, currentRoundFor, roundStartTimestamp,
  waitForNextRound, bigintToRpcHex, partialObservationForExpressLane,
  TimeboostRuntime, ConditionalOptions.
- Remove 3 `void X` suppressions and their unused imports.
- Merge sendNativeEth + sendCall in timeboostDemoRunner into one signAndSend.
- Export submitNormalTx + completeObservation; reuse them in the no-bid round
  path instead of hand-rolling nonce/gas/sign/receipt/timeboosted extraction.
- Drop unused walletClient field from DeployContractsInput and its caller.

Net -195 lines across 12 timeboost files; tsc + prettier clean. No behaviour change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: Jason-W123

src/playbooks/timeboost/auctionMonitor.ts

@@ -12,20 +12,16 @@
  * for the HTML report.
  */
 
-import { type Address, type Hash, type PublicClient, decodeEventLog, parseAbi } from 'viem';
+import { type Address, type Hash, type PublicClient, decodeEventLog } from 'viem';
 import { expressLaneAuctionArtifact } from './abis.js';
 import type { AuctionEvent } from './types.js';
 
-let log = {
+const log = {
   info: (m: string) => console.log('ℹ', m),
   warn: (m: string) => console.log('⚠', m),
   event: (m: string) => console.log('•', m),
 };
 
-export function setAuctionMonitorLogger(l: typeof log): void {
-  log = l;
-}
-
 export interface AuctionMonitorHandle {
   events: AuctionEvent[];
   stop: () => void;
@@ -142,6 +138,3 @@ function jsonReplacer(_key: string, v: unknown): unknown {
   if (typeof v === 'bigint') return v.toString();
   return v;
 }
-
-// Suppress unused-import warning for parseAbi (kept available for future use).
-void parseAbi;

src/playbooks/timeboost/auctionRunner.ts

@@ -17,17 +17,13 @@ import { approveAndDeposit, submitBid, type SubmittedBid } from './bidder.js';
 import { snapshotRound, formatRoundLine, waitUntilRound, type RoundTiming } from './roundClock.js';
 import type { AuctionEvent } from './types.js';
 
-let log = {
+const log = {
   info: (m: string) => console.log('ℹ', m),
   warn: (m: string) => console.log('⚠', m),
   success: (m: string) => console.log('✔', m),
   section: (m: string) => console.log('\n▸', m, '\n'),
 };
 
-export function setAuctionRunnerLogger(l: typeof log): void {
-  log = l;
-}
-
 export interface RunOneAuctionInput {
   publicClient: PublicClient;
   auctionAddress: Address;

src/playbooks/timeboost/bidder.ts

@@ -24,24 +24,19 @@ import {
   encodeAbiParameters,
   encodeFunctionData,
   keccak256,
-  pad,
   toBytes,
   toHex,
 } from 'viem';
 import { sign as signRawHash, signatureToHex, privateKeyToAccount } from 'viem/accounts';
 import { ierc20Abi, expressLaneAuctionArtifact } from './abis.js';
 import { rawRpcCall, TimeboostRpcError } from './expressLaneRunner.js';
 
-let log = {
+const log = {
   info: (m: string) => console.log('ℹ', m),
   warn: (m: string) => console.log('⚠', m),
   success: (m: string) => console.log('✔', m),
 };
 
-export function setBidderLogger(l: typeof log): void {
-  log = l;
-}
-
 // ---------------------------------------------------------------------------
 // Types
 // ---------------------------------------------------------------------------
@@ -248,7 +243,3 @@ async function sendCall(
   if (receipt.status !== 'success') throw new Error(`Tx ${txHash} reverted`);
   return txHash;
 }
-
-// Suppress unused-import warning for `pad` — exported as part of viem's API
-// surface that we may need in tests.
-void pad;

src/playbooks/timeboost/deployContracts.ts

@@ -18,8 +18,6 @@ import {
   type Hex,
   type LocalAccount,
   type PublicClient,
-  type WalletClient,
-  encodeAbiParameters,
   encodeDeployData,
   encodeFunctionData,
 } from 'viem';
@@ -43,19 +41,11 @@ export const DEFAULT_DEMO_TIMING: RoundTimingConfig = {
   reserveSubmissionSeconds: 3,
 };
 
-/** Closer-to-production timing matching the docs example. */
-export const DOCS_DEFAULT_TIMING: RoundTimingConfig = {
-  roundDurationSeconds: 60,
-  auctionClosingSeconds: 15,
-  reserveSubmissionSeconds: 15,
-};
-
 export interface DeployContractsInput {
   /** Deployer (must hold child chain ETH for gas). */
   deployer: LocalAccount<string>;
-  /** Both clients for the child chain. */
+  /** Public client for the child chain (used to read nonce/gas + submit signed txs). */
   publicClient: PublicClient;
-  walletClient: WalletClient;
   /** Address that will resolve auctions and hold AUCTIONEER_ROLE. */
   auctioneer: Address;
   /** Address that receives auction proceeds when `flushBeneficiaryBalance()` is called. */
@@ -287,26 +277,3 @@ function roundOffsetForRoundDuration(roundDurationSeconds: number): number {
   const nowSec = Math.floor(Date.now() / 1000);
   return Math.floor(nowSec / roundDurationSeconds) * roundDurationSeconds;
 }
-
-// ---------------------------------------------------------------------------
-// Convenience: compute current round client-side from the deployed config.
-// Mirrors nitro/timeboost/roundtiminginfo.go.
-// ---------------------------------------------------------------------------
-
-export function currentRoundFor(
-  deployed: Pick<DeployedContracts, 'initialOffsetTimestamp' | 'timing'>,
-  nowSec?: number,
-): number {
-  const t = nowSec ?? Math.floor(Date.now() / 1000);
-  return Math.floor((t - deployed.initialOffsetTimestamp) / deployed.timing.roundDurationSeconds);
-}
-
-export function roundStartTimestamp(
-  deployed: Pick<DeployedContracts, 'initialOffsetTimestamp' | 'timing'>,
-  round: number,
-): number {
-  return deployed.initialOffsetTimestamp + round * deployed.timing.roundDurationSeconds;
-}
-
-/** Suppress unused-import warning (we may need encodeAbiParameters in tests later). */
-void encodeAbiParameters;

src/playbooks/timeboost/experimentRecorder.ts

@@ -14,7 +14,7 @@ import { type Address, type Hash, type Hex, type LocalAccount, type PublicClient
 import { submitExpressLaneTransaction, type ExpressLaneSubmitInput, type RunnerLogger } from './expressLaneRunner.js';
 import type { ExperimentRecord, TxObservation } from './types.js';
 
-let recorderLogger: RunnerLogger & {
+const recorderLogger: RunnerLogger & {
   section: (m: string) => void;
   raw: (m: string) => void;
   success: (m: string) => void;
@@ -27,10 +27,6 @@ let recorderLogger: RunnerLogger & {
   success: (m) => console.log('✔', m),
 };
 
-export function setRecorderLogger(l: typeof recorderLogger): void {
-  recorderLogger = l;
-}
-
 // ---------------------------------------------------------------------------
 // Public API
 // ---------------------------------------------------------------------------
@@ -118,19 +114,20 @@ export async function runExperimentPair(input: RunExperimentInput): Promise<Expe
 }
 
 // ---------------------------------------------------------------------------
-// Internals — normal tx submission
+// Normal tx submission (exported so the no-bid round demo can reuse it
+// instead of hand-rolling the same nonce/gas/sign/send dance)
 // ---------------------------------------------------------------------------
 
-interface SubmitNormalTxInput extends NormalTxInput {
+export interface SubmitNormalTxInput extends NormalTxInput {
   label: string;
 }
 
-interface SubmitNormalTxResult {
+export interface SubmitNormalTxResult {
   txHash: Hash;
   sentAtMs: number;
 }
 
-async function submitNormalTx(input: SubmitNormalTxInput): Promise<SubmitNormalTxResult> {
+export async function submitNormalTx(input: SubmitNormalTxInput): Promise<SubmitNormalTxResult> {
   const { senderAccount, childClient, chainId, to, valueEth = '0', label } = input;
 
   const nonce = await childClient.getTransactionCount({
@@ -160,10 +157,10 @@ async function submitNormalTx(input: SubmitNormalTxInput): Promise<SubmitNormalT
 }
 
 // ---------------------------------------------------------------------------
-// Internals — receipt polling + timeboosted extraction
+// Receipt polling + timeboosted extraction (exported for the no-bid round path)
 // ---------------------------------------------------------------------------
 
-interface CompleteObservationInput {
+export interface CompleteObservationInput {
   lane: TxObservation['lane'];
   childClient: PublicClient;
   txHash: Hash;
@@ -173,7 +170,7 @@ interface CompleteObservationInput {
   timeoutMs: number;
 }
 
-async function completeObservation(input: CompleteObservationInput): Promise<TxObservation> {
+export async function completeObservation(input: CompleteObservationInput): Promise<TxObservation> {
   const { childClient, txHash, sentAtMs, sender, lane, round, timeoutMs } = input;
 
   // Use a tighter poll than the default — we want a wall-clock-accurate

src/playbooks/timeboost/expressLaneRunner.ts

@@ -17,33 +17,26 @@ import {
   parseEther,
   toHex,
 } from 'viem';
-import { DONT_CARE_SEQUENCE, type JsonExpressLaneSubmission, type TxObservation } from './types.js';
+import { DONT_CARE_SEQUENCE, type JsonExpressLaneSubmission } from './types.js';
 import { signExpressLaneSubmission } from './expressLaneSigner.js';
 
 /**
  * Minimal logger surface so Phase 6 modules don't have to import the
  * playbook-wide logger.ts (which transitively pulls in `ora` →
  * `cli-spinners` and breaks on Node < 20.10 due to import attributes).
- *
- * Default is a `console.log`-backed no-op-ish sink. Production callers can
- * pass the real logger via `setRunnerLogger()`.
  */
 export interface RunnerLogger {
   event: (msg: string) => void;
   info: (msg: string) => void;
   warn: (msg: string) => void;
 }
 
-let activeLogger: RunnerLogger = {
+const activeLogger: RunnerLogger = {
   event: (m) => console.log('•', m),
   info: (m) => console.log('ℹ', m),
   warn: (m) => console.log('⚠', m),
 };
 
-export function setRunnerLogger(l: RunnerLogger): void {
-  activeLogger = l;
-}
-
 export interface ExpressLaneSubmitInput {
   /** Controller account (must be a LocalAccount<string>; needs to deterministically sign tx + envelope). */
   controllerAccount: LocalAccount<string>;
@@ -145,24 +138,6 @@ export async function submitExpressLaneTransaction(input: ExpressLaneSubmitInput
   return { txHash, sentAtMs, rlpTx, submission };
 }
 
-/**
- * Convenience: yield the partial TxObservation that the experiment recorder
- * will later complete with on-chain receipt fields.
- */
-export function partialObservationForExpressLane(
-  result: ExpressLaneSubmitResult,
-  controllerAddress: Address,
-  round: number,
-): Pick<TxObservation, 'lane' | 'sentAtMs' | 'txHash' | 'sender' | 'round'> {
-  return {
-    lane: 'express',
-    sentAtMs: result.sentAtMs,
-    txHash: result.txHash,
-    sender: controllerAddress,
-    round,
-  };
-}
-
 // ---------------------------------------------------------------------------
 // Minimal JSON-RPC client
 // ---------------------------------------------------------------------------

src/playbooks/timeboost/expressLaneSigner.ts

@@ -26,7 +26,6 @@ import {
   numberToBytes,
   pad,
   toBytes,
-  toHex,
 } from 'viem';
 
 const TIMEBOOST_DOMAIN_LABEL = 'TIMEBOOST_BID';
@@ -93,11 +92,3 @@ export async function signExpressLaneSubmission(account: Account, input: BuildSu
 
   return local.signMessage({ message: { raw: messageBytes } });
 }
-
-/**
- * Convenience: convert a bigint to its standard hex representation
- * compatible with the sequencer's `hexutil.Big` / `hexutil.Uint64` decoders.
- */
-export function bigintToRpcHex(value: bigint): Hex {
-  return toHex(value);
-}

src/playbooks/timeboost/roundClock.ts

@@ -73,13 +73,3 @@ export async function waitUntilRound(timing: RoundTiming, target: number, signal
     await new Promise((r) => setTimeout(r, sleepMs));
   }
 }
-
-/**
- * Wait until the next round transitions and return the new round number.
- * Convenience for the demo runner's "watch round transitions" step.
- */
-export async function waitForNextRound(timing: RoundTiming, signal?: AbortSignal): Promise<number> {
-  const start = snapshotRound(timing).current;
-  await waitUntilRound(timing, start + 1, signal);
-  return start + 1;
-}

src/playbooks/timeboost/serviceManager.ts

@@ -29,16 +29,12 @@ const REDIS_HOST_PORT = 6379;
 export const BID_VALIDATOR_HOST_PORT = 9372;
 const AUCTIONEER_HTTP_HOST_PORT = 9373; // not strictly required; helpful for poking
 
-let log = {
+const log = {
   info: (m: string) => console.log('ℹ', m),
   warn: (m: string) => console.log('⚠', m),
   success: (m: string) => console.log('✔', m),
 };
 
-export function setServiceManagerLogger(l: typeof log): void {
-  log = l;
-}
-
 // ---------------------------------------------------------------------------
 // Public types
 // ---------------------------------------------------------------------------