openclaw - 💡(How to fix) Fix compaction.model config not passed through safeguard compaction path [1 comments, 2 participants]

// reply‑DeXK9BLT.js  (around line 38904)
async function compactSession(session, customInstructions) {
  // New: pull the full config from the session (or from agents.defaults)
  const compactionConfig = session.getConfig?.() ?? agents.defaults?.compaction ?? {};

  // Pass both the custom instructions *and* the config forward
  await session.compact(customInstructions, compactionConfig);
}

// legacy context engine (around line 5252)
Session.prototype.compact = async function (customInstructions, config = {}) {
  // keep existing param shape for callers that don’t pass config
  const params = {
    customInstructions,
    sessionId: this.id,
    config,               // <-- new field
    // any other existing fields …
  };

  // Preserve original behaviour for backward‑compatibility
  const runtimeContext = { config };   // will be spread in the next call
  const result = await compactEmbeddedPiSessionDirect({
    ...runtimeContext,
    sessionId: params.sessionId,
    // …other args unchanged
  });
  return result;
};

// compactEmbeddedPiSessionDirect (around line 93302)
async function compactEmbeddedPiSessionDirect({ sessionId, config = {}, ...rest }) {
  // Existing logic …
  const compactionModelOverride =
        config?.compaction?.model ??
        agents.defaults?.compaction?.model;   // global fallback

  const modelToUse = compactionModelOverride ?? sessionPrimaryModel(sessionId);
  // use