[APMSVLS-501] refactor(bottlecap): preparatory work for "test-mode" binary by lucaspimentel · Pull Request #1201 · DataDog/datadog-lambda-extension

Add InvocationProcessorHandle::noop() for test-mode binary

lucaspimentel

committed

fix(bottlecap): replace unwrap() with expect() in noop() tests to satisfy clippy

lucaspimentel

committed

refactor(bottlecap): promote start_trace_agent to library crate

lucaspimentel

committed

refactor(bottlecap): add TraceAgent::with_flushing_service for test-mode /flush hook

lucaspimentel

committed

fix: return 500 from /flush handler on panic

lucaspimentel

committed

refactor(startup): remove redundant re-export and allow attr

lucaspimentel

committed

refactor(startup): remove unnecessary clippy::type_complexity allow

lucaspimentel

committed

refactor(traces): re-export TraceAgentPipeline from traces module

lucaspimentel

committed

refactor(noop): match real service channel capacity

lucaspimentel

committed

docs(trace_agent): explain /flush panic isolation

lucaspimentel

committed

docs(startup): warn about leaked tasks if agent not spawned

lucaspimentel

committed

refactor(startup): consolidate tokio::sync imports

lucaspimentel

committed

fix(trace_agent): add timeout and panic isolation to POST /flush

lucaspimentel

committed

test(noop): cover PlatformRuntimeDone and PlatformReport variants

lucaspimentel

committed

style: apply rustfmt

lucaspimentel

committed

fix(trace_agent): abort timed-out flush task to bound execution

lucaspimentel

committed

refactor(startup): replace TraceAgentPipeline tuple with named struct

lucaspimentel

committed

refactor(startup): move out of traces to top-level bottlecap::startup

lucaspimentel

committed

fix(trace_agent): return 504 directly after aborting flush task

lucaspimentel

committed

test(noop): call on_platform_runtime_done/on_platform_report via handle

lucaspimentel

committed

docs(startup,trace_agent,noop): describe behavior, not the consumer

lucaspimentel

committed

refactor(trace_agent): replace flushing_service Option with RouterExtension trait

lucaspimentel

committed

test(trace_agent): exercise RouterExtension seam through make_router

lucaspimentel

committed

docs(noop): fix inaccurate consequence described in exhaustive-match comment

lucaspimentel

committed

docs(startup): count all four background tasks in build_trace_agent comment

lucaspimentel

committed

fix(trace_agent): RouterExtension::extend returns Result to surface errors

lucaspimentel

committed

test(noop): guard all five request-response variants with an explicit timeout

lucaspimentel

committed

style(trace_agent): apply rustfmt to RouterExtension::extend signatures

lucaspimentel

committed

docs(trace_agent,startup): clarify RouterExtension err propagation

lucaspimentel

committed

feat(bottlecap): gate InvocationProcessorHandle::noop() behind test-mode feature

lucaspimentel

committed

docs(processor_service): document tokio runtime requirement on noop()

lucaspimentel

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[APMSVLS-501] refactor(bottlecap): preparatory work for "test-mode" binary#1201

[APMSVLS-501] refactor(bottlecap): preparatory work for "test-mode" binary#1201
lucaspimentel wants to merge 31 commits intomainfrom
lpimentel/bottlecap-test-mode

Commits on Apr 29, 2026

Add InvocationProcessorHandle::noop() for test-mode binary

fix(bottlecap): replace unwrap() with expect() in noop() tests to satisfy clippy

refactor(bottlecap): promote start_trace_agent to library crate

refactor(bottlecap): add TraceAgent::with_flushing_service for test-mode /flush hook

fix: return 500 from /flush handler on panic

refactor(startup): remove redundant re-export and allow attr

refactor(startup): remove unnecessary clippy::type_complexity allow

refactor(traces): re-export TraceAgentPipeline from traces module

refactor(noop): match real service channel capacity

docs(trace_agent): explain /flush panic isolation

docs(startup): warn about leaked tasks if agent not spawned

refactor(startup): consolidate tokio::sync imports

fix(trace_agent): add timeout and panic isolation to POST /flush

test(noop): cover PlatformRuntimeDone and PlatformReport variants

style: apply rustfmt

fix(trace_agent): abort timed-out flush task to bound execution

refactor(startup): replace TraceAgentPipeline tuple with named struct

refactor(startup): move out of traces to top-level bottlecap::startup

fix(trace_agent): return 504 directly after aborting flush task

test(noop): call on_platform_runtime_done/on_platform_report via handle

docs(startup,trace_agent,noop): describe behavior, not the consumer

refactor(trace_agent): replace flushing_service Option with RouterExtension trait

test(trace_agent): exercise RouterExtension seam through make_router

docs(noop): fix inaccurate consequence described in exhaustive-match comment

docs(startup): count all four background tasks in build_trace_agent comment

fix(trace_agent): RouterExtension::extend returns Result to surface errors

test(noop): guard all five request-response variants with an explicit timeout

style(trace_agent): apply rustfmt to RouterExtension::extend signatures

docs(trace_agent,startup): clarify RouterExtension err propagation

feat(bottlecap): gate InvocationProcessorHandle::noop() behind test-mode feature

docs(processor_service): document tokio runtime requirement on noop()