Add enableMemory parameter to debug_traceTransaction and debug_traceBlockByNumber

[vivek-0509]

PR description

Adds the enableMemory param (default false) to TransactionTraceParams. When both enableMemory and disableMemory are provided, enableMemory takes precedence.

Changes

• Added enableMemory JSON property to TransactionTraceParams following the same nullable Boolean pattern as the existing disableMemory param
• Updated traceOptions() precedence: enableMemory -> disableMemory -> non-opcode tracer fallback
• Added 4 unit tests covering enableMemory true/false and precedence over disableMemory

Fixed Issue(s)

Partial fix for #10115 (item 8)

Thanks for sending a pull request! Have you done the following?

• Checked out our contribution guidelines?
• Considered documentation and added the doc-change-required label to this PR if updates are required.
• Considered the changelog and included an update if required.
• For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

• spotless: ./gradlew spotlessApply
• unit tests: ./gradlew build
• acceptance tests: ./gradlew acceptanceTest
• integration tests: ./gradlew integrationTest
• reference tests: ./gradlew ethereum:referenceTests:referenceTests
• hive tests: Engine or other RPCs modified?

[vivek-0509]

@macfarla do take a look, build passes locally

[vivek-0509]

@fab-10 @macfarla Please take a look when you get a chance

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new enableMemory boolean parameter to transaction/block debug tracing params, with defined precedence over the existing disableMemory flag.

Changes:

• Introduces enableMemory (nullable Boolean) to TransactionTraceParams and updates traceOptions() precedence (enableMemory → disableMemory → non-opcode tracer fallback).
• Adds unit tests for enableMemory true/false, precedence over disableMemory, and behavior with non-opcode tracers.
• Updates CHANGELOG.md to announce the new parameter and its defaults.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
ethereum/api/src/main/java/org/hyperledger/besu/ethereum/api/jsonrpc/internal/parameters/TransactionTraceParams.java	Adds enableMemory param and updates memory-tracing precedence logic.
ethereum/api/src/test/java/org/hyperledger/besu/ethereum/api/jsonrpc/internal/parameters/TransactionTraceParamsTest.java	Adds unit tests for enableMemory behavior and precedence.
CHANGELOG.md	Documents the new enableMemory parameter for debug trace RPCs.

[Copilot]

The changelog states memory is 'only included ... when explicitly enabled', but TransactionTraceParams#traceOptions() still enables memory by default for non-opcode tracers (the existing fallback behavior). Consider clarifying this entry (e.g., specify the default applies to the opcode tracer path, and note non-opcode tracers may still capture memory unless explicitly disabled), or adjust the implementation if the intent is to make memory truly opt-in for all tracers.

[Copilot]

The precedence rule is only asserted for the enableMemory=true case. Add a complementary test where enableMemory=false conflicts with disableMemory=false (or disableMemory=true) to ensure enableMemory consistently wins in both directions (i.e., enableMemory=false forces memory off regardless of disableMemory).

Suggested change

	@Test
	@Test
	public void enableMemoryFalseShouldTakePrecedenceOverDisableMemory() throws Exception {
	final TransactionTraceParams params =
	MAPPER.readValue(
	"{\"enableMemory\": false, \"disableMemory\": false}", TransactionTraceParams.class);
	final OpCodeTracerConfig config = params.traceOptions().opCodeTracerConfig();
	assertThat(config.traceMemory())
	.describedAs("enableMemory=false should take precedence over disableMemory=false")
	.isFalse();
	}
	@Test