Skip to content

On demand summarizer for on demand summaries#25394

Merged
WillieHabi merged 56 commits intomicrosoft:mainfrom
WillieHabi:on-demand-summarizer
Oct 13, 2025
Merged

On demand summarizer for on demand summaries#25394
WillieHabi merged 56 commits intomicrosoft:mainfrom
WillieHabi:on-demand-summarizer

Conversation

@WillieHabi
Copy link
Contributor

@WillieHabi WillieHabi commented Sep 4, 2025

Description

This change introduces new public API loadSummarizerContainerAndMakeSummary to load a summarizer client container to produce an on-demand summary before disposing itself.

We also add new ISummaryConfigurationWithSummaryOnRequest summary config that we check during summarizer initialzation to turn of client election and internal heurstics.

@github-actions github-actions bot added base: main PRs targeted against main branch area: loader Loader related issues area: runtime Runtime related issues public api change Changes to a public API labels Sep 4, 2025
@github-actions github-actions bot added the area: tests Tests to add, test infrastructure improvements, etc label Sep 4, 2025
@WillieHabi WillieHabi requested a review from jatgarg September 5, 2025 20:56
@WillieHabi WillieHabi requested a review from jatgarg September 23, 2025 18:02
@WillieHabi WillieHabi marked this pull request as ready for review September 24, 2025 21:36
@WillieHabi WillieHabi requested a review from a team as a code owner September 24, 2025 21:36
Copilot AI review requested due to automatic review settings September 24, 2025 21:36
WillieHabi and others added 2 commits October 9, 2025 13:55
…succeeds while normal summary is inflight, on-demand summary succeeds after normal summary completes
@WillieHabi WillieHabi requested a review from jatgarg October 9, 2025 22:41
@github-actions
Copy link
Contributor

🔗 Found some broken links! 💔

Run a link check locally to find them. See
https://github.com/microsoft/FluidFramework/wiki/Checking-for-broken-links-in-the-documentation for more information.

linkcheck output


> fluid-framework-docs-site@0.0.0 ci:check-links /home/runner/work/FluidFramework/FluidFramework/docs
> start-server-and-test "npm run serve -- --no-open" 3000 check-links

1: starting server using command "npm run serve -- --no-open"
and when url "[ 'http://127.0.0.1:3000' ]" is responding with HTTP status code 200
running tests using command "npm run check-links"


> fluid-framework-docs-site@0.0.0 serve
> docusaurus serve --no-open

[SUCCESS] Serving "build" directory at: http://localhost:3000/

> fluid-framework-docs-site@0.0.0 check-links
> linkcheck http://localhost:3000 --skip-file skipped-urls.txt

 ELIFECYCLE  Command failed with exit code 1.

@WillieHabi WillieHabi requested a review from jatgarg October 10, 2025 19:31
@WillieHabi
Copy link
Contributor Author

@jatgarg @anthony-murphy gentle nudge on this.

@anthony-murphy
Copy link
Contributor

i won't have a chance to re-review here. looks like all my feedback was included, so don't consider me blocking

@WillieHabi WillieHabi merged commit 4fc1a72 into microsoft:main Oct 13, 2025
37 checks passed
anthony-murphy-agent pushed a commit to anthony-murphy-agent/FluidFramework that referenced this pull request Jan 14, 2026
## Description 

This change introduces new public API
`loadSummarizerContainerAndMakeSummary` to load a summarizer client
container to produce an on-demand summary before disposing itself.

We also add new `ISummaryConfigurationWithSummaryOnRequest` summary
config that we check during summarizer initialzation to turn of client
election and internal heurstics.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: framework Framework is a tag for issues involving the developer framework. Eg Aqueduct area: loader Loader related issues area: runtime Runtime related issues area: tests Tests to add, test infrastructure improvements, etc base: main PRs targeted against main branch public api change Changes to a public API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants