From 6cb31a10e4b97bf0eac5b03cd1747b6fdeeaf351 Mon Sep 17 00:00:00 2001 From: Dai Ha Date: Tue, 22 Sep 2026 10:50:59 +0700 Subject: [PATCH] fleetd #618: fix the third stale spot the brief missed The method-level javadoc on FleetConfig.warnConflictingAutoCompactWindows (above the log.warn call) still claimed the autoCompactWindow vs CLAUDE_CODE_AUTO_COMPACT_WINDOW precedence was 'intentionally not asserted' and cited fleetd.yaml's now-corrected comment as evidence the question was open. Replace it with the measured answer from #618: the env var wins, so autoCompactWindow is inert on a profile that sets both. Kept the WARN-not-throw rationale paragraph above it untouched (#601) and kept the ClaudeCodeArguments cross-reference, which now points to an agreeing claim instead of a contradicting one. No behaviour change. --- .../main/java/dev/ltms/fleet/config/FleetConfig.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fleetd/src/main/java/dev/ltms/fleet/config/FleetConfig.java b/fleetd/src/main/java/dev/ltms/fleet/config/FleetConfig.java index 45b415a..62da9a8 100644 --- a/fleetd/src/main/java/dev/ltms/fleet/config/FleetConfig.java +++ b/fleetd/src/main/java/dev/ltms/fleet/config/FleetConfig.java @@ -2240,11 +2240,11 @@ public record FleetConfig( * information — which profiles, and now both values, so they can fix it without reading the * source — without ever taking the fleet down. * - *

Which of the two inputs Claude Code actually follows when they disagree is intentionally - * not asserted here. {@code ClaudeCodeArguments}'s javadoc used to state the - * environment variable always wins; nobody had measured that, and this host's own - * {@code fleetd.yaml} asserts the opposite in a comment. This method only detects and reports - * the disagreement — see {@link dev.ltms.fleet.launch.ClaudeCodeArguments}. + *

fleetd #618 measured which of the two inputs Claude Code actually follows when they + * disagree: the environment variable wins, so {@code autoCompactWindow} is inert on a profile + * that also sets the env var. This method only detects and reports the disagreement — it does + * not correct it — see {@link dev.ltms.fleet.launch.ClaudeCodeArguments} for the full measured + * precedence. * *

Equal values never warn: either input then produces the same session window, so there is * nothing to reconcile.