@fluidframework/container-runtime 2.63.0-359962 → 2.63.0

package/src/summary/index.ts

@@ -40,6 +40,7 @@ export {
 	type ISummaryConfigurationDisableHeuristics,
 	type ISummaryConfigurationDisableSummarizer,
 	type ISummaryConfigurationHeuristics,
+	type ISummaryConfigurationWithSummaryOnRequest,
 	type ISummaryBaseConfiguration,
 } from "./summarizerTypes.js";
 export {
@@ -98,6 +99,7 @@ export {
 	summarizerRequestUrl,
 	DefaultSummaryConfiguration,
 	isSummariesDisabled,
+	isSummaryOnRequest,
 } from "./summaryHelpers.js";
 export {
 	type IdCompressorMode,

package/src/summary/summarizerTypes.ts

@@ -759,10 +759,19 @@ export interface ISummaryConfigurationDisableHeuristics extends ISummaryBaseConf
 	state: "disableHeuristics";
 }
 
+/**
+ * Configuration used internally to indicate on-demand summaries only (no election/heuristics).
+ * @legacy @beta
+ */
+export interface ISummaryConfigurationWithSummaryOnRequest extends ISummaryBaseConfiguration {
+	state: "summaryOnRequest";
+}
+
 /**
  * @legacy @beta
  */
 export type ISummaryConfiguration =
 	| ISummaryConfigurationDisableSummarizer
 	| ISummaryConfigurationDisableHeuristics
-	| ISummaryConfigurationHeuristics;
+	| ISummaryConfigurationHeuristics
+	| ISummaryConfigurationWithSummaryOnRequest;

package/src/summary/summaryDelayLoadedModule/runningSummarizer.ts

@@ -227,14 +227,12 @@ export class RunningSummarizer
 			},
 		});
 
-		if (configuration.state !== "disableHeuristics") {
-			assert(
-				this.configuration.state === "enabled",
-				0x2ea /* "Configuration state should be enabled" */,
-			);
+		// Only create heuristics when in 'enabled' mode.
+		// This bypasses heuristics for 'disableHeuristics' and 'summaryOnRequest' configurations.
+		if (configuration.state === "enabled") {
 			this.heuristicRunner = new SummarizeHeuristicRunner(
 				heuristicData,
-				this.configuration,
+				configuration,
 				(reason) => this.trySummarize(reason),
 				this.mc.logger,
 			);

package/src/summary/summaryHelpers.ts

@@ -25,6 +25,10 @@ export function isSummariesDisabled(
 	return config.state === "disabled";
 }
 
+export function isSummaryOnRequest(config: { state?: string }): boolean {
+	return config.state === "summaryOnRequest";
+}
+
 /**
  * @legacy @beta
  */