feat(local-server-stress-tests): add support for dynamic generator weights by anthony-murphy-agent · Pull Request #26324 · microsoft/FluidFramework

anthony-murphy-agent · 2026-01-29T18:40:36Z

Summary

Add dynamicWeightGenerator.ts with createWeightedAsyncGeneratorWithDynamicWeights
Enable generator weights to be functions that receive state and return a weight
This allows weights to vary based on the current test state

The standard createWeightedAsyncGenerator from stochastic-test-utils only supports static numeric weights. This enhancement enables dynamic weighting based on runtime conditions like container attach state.

Test plan

local-server-stress-tests pass
No breaking changes to existing behavior (weights can still be static numbers)

related to AB#8839

🤖 Generated with Claude Code

…ights Enable generator weights to be functions that receive state and return a weight, instead of only supporting static numeric weights. This allows weights to vary based on the current test state. Co-Authored-By: anthony-murphy <anthony.murphy@microsoft.com> Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

anthony-murphy · 2026-01-29T18:44:31Z

/azp run Build - api-markdown-documenter, Build - benchmark-tool, Build - build-common, Build - build-tools, Build - client packages, Build - common-utils, Build - eslint-config-fluid, Build - eslint-plugin-fluid, Build - protocol-definitions, Build site

anthony-murphy · 2026-01-29T18:44:37Z

/azp run Build - test-tools, repo-policy-check, server-gitrest, server-gitssh, server-historian, server-routerlicious

azure-pipelines · 2026-01-29T18:44:49Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2026-01-29T18:44:49Z

Azure Pipelines successfully started running 1 pipeline(s).

Copilot

Pull request overview

This PR adds infrastructure for dynamic weight functions in weighted random generators for the local-server-stress-tests package. Currently, the stochastic-test-utils library only supports static numeric weights, but this enhancement enables weights that can be computed dynamically based on runtime test state.

Changes:

Introduces dynamicWeightGenerator.ts with support for both static and dynamic (function-based) weights
Updates baseModel.ts to use the new dynamic weight generator instead of the static one from stochastic-test-utils
Maintains full backward compatibility - existing code with static weights continues to work

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
`packages/test/local-server-stress-tests/src/dynamicWeightGenerator.ts`	New module implementing `createWeightedAsyncGeneratorWithDynamicWeights`, a variant of the stochastic-test-utils function that evaluates weight functions dynamically at runtime
`packages/test/local-server-stress-tests/src/baseModel.ts`	Updated to use the new dynamic weight generator, with type changes from `AsyncWeights` to `DynamicAsyncWeights`

packages/test/local-server-stress-tests/src/dynamicWeightGenerator.ts

anthony-murphy · 2026-02-03T17:21:32Z

/azp run Build - api-markdown-documenter, Build - benchmark-tool, Build - build-common, Build - build-tools, Build - client packages, Build - common-utils, Build - eslint-config-fluid, Build - eslint-plugin-fluid, Build - protocol-definitions, Build - test-tools

anthony-murphy · 2026-02-03T17:21:36Z

/azp run repo-policy-check, server-gitrest, server-gitssh, server-historian, server-routerlicious

azure-pipelines · 2026-02-03T17:21:52Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2026-02-03T17:21:56Z

Azure Pipelines successfully started running 1 pipeline(s).

packages/test/local-server-stress-tests/src/dynamicWeightGenerator.ts

…ation Simplify the dynamic weight generator by evaluating shouldAccept during the cumulative sums calculation phase rather than using rejection sampling after selection. This means cumulativeSums only contains valid entries, eliminating the need for the do-while loop. Co-Authored-By: anthony-murphy <anthony.murphy@microsoft.com> Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

anthony-murphy · 2026-02-04T22:09:14Z

/azp run Build - client packages, Build - api-markdown-documenter, Build - benchmark-tool, Build - build-common, Build - build-tools, Build - common-utils, Build - eslint-config-fluid, Build - eslint-plugin-fluid, Build - protocol-definitions, Build - test-tools

anthony-murphy · 2026-02-04T22:09:18Z

/azp run Build site, repo-policy-check, server-gitrest, server-gitssh, server-historian, server-routerlicious

azure-pipelines · 2026-02-04T22:09:31Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2026-02-04T22:09:33Z

Azure Pipelines successfully started running 1 pipeline(s).

packages/test/local-server-stress-tests/src/dynamicWeightGenerator.ts

anthony-murphy · 2026-02-04T22:11:13Z

/azp run Build site, repo-policy-check, server-gitrest, server-gitssh, server-historian, server-routerlicious

anthony-murphy · 2026-02-04T22:11:21Z

/azp run Build - client packages, Build - api-markdown-documenter, Build - benchmark-tool, Build - build-common, Build - build-tools, Build - common-utils, Build - eslint-config-fluid, Build - eslint-plugin-fluid, Build - protocol-definitions, Build - test-tools

azure-pipelines · 2026-02-04T22:11:26Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2026-02-04T22:11:39Z

Azure Pipelines successfully started running 1 pipeline(s).

anthony-murphy-agent · 2026-02-04T22:12:37Z

Azure Pipelines commands (copy and post separately, limit of 10 at a time):

/azp run Build - client packages, Build - api-markdown-documenter, Build - benchmark-tool, Build - build-common, Build - build-tools, Build - common-utils, Build - eslint-config-fluid, Build - eslint-plugin-fluid, Build - protocol-definitions, Build - test-tools

/azp run Build site, repo-policy-check, server-gitrest, server-gitssh, server-historian, server-routerlicious

Resolve merge conflicts in baseModel.ts and dynamicWeightGenerator.ts after upstream changes from PR microsoft#26324 were merged to main. Preserved the detached operation phasing logic from this PR while adopting the updated acceptance condition filtering from upstream. Co-Authored-By: anthony-murphy <anthony.murphy@microsoft.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

…26325) ## Summary Introduces operation phasing during the detached state before attach: - **Creation phase** (first 7 ops): prioritizes `createDataStore` and `createChannel` with high weights - **DDS ops phase** (ops 8-20): prioritizes `DDSModelOp` operations with increased weight This ensures tests create sufficient data structure (datastores and channels) before exercising DDS operations, providing better test coverage. ## Dependencies - Depends on #26324 (dynamic generator weights) ## Test plan - [x] local-server-stress-tests pass - [x] Verified phasing behavior produces expected operation distribution related to [AB#8839](https://dev.azure.com/fluidframework/235294da-091d-4c29-84fc-cdfc3d90890b/_workitems/edit/8839) 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: anthony-murphy-agent <anthony-murphy-agent@users.noreply.github.com> Co-authored-by: anthony-murphy <anthony.murphy@microsoft.com> Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com> Co-authored-by: yann-achard-MS <97201204+yann-achard-MS@users.noreply.github.com>

This was referenced Jan 29, 2026

feat(local-server-stress-tests): add phasing for detached operations #26325

Merged

feat(local-server-stress-tests): increase ops before attach to 100 #26326

Closed

anthony-murphy requested a review from Copilot January 29, 2026 18:43

Copilot started reviewing on behalf of anthony-murphy January 29, 2026 18:43 View session

Copilot AI reviewed Jan 29, 2026

View reviewed changes

packages/test/local-server-stress-tests/src/dynamicWeightGenerator.ts Show resolved Hide resolved

Merge branch 'main' into fix/dynamic-generator-weights

cba4dc7

anthony-murphy reviewed Feb 4, 2026

View reviewed changes

packages/test/local-server-stress-tests/src/dynamicWeightGenerator.ts Show resolved Hide resolved

anthony-murphy reviewed Feb 4, 2026

View reviewed changes

packages/test/local-server-stress-tests/src/dynamicWeightGenerator.ts Outdated Show resolved Hide resolved

Apply suggestion from @anthony-murphy

cf5651d

dannimad approved these changes Feb 5, 2026

View reviewed changes

anthony-murphy merged commit 8b7ff27 into microsoft:main Feb 5, 2026
30 checks passed

Comments

Conversation

anthony-murphy-agent commented Jan 29, 2026 • edited by azure-boards bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

anthony-murphy commented Jan 29, 2026

Uh oh!

anthony-murphy commented Jan 29, 2026

Uh oh!

azure-pipelines bot commented Jan 29, 2026

Uh oh!

azure-pipelines bot commented Jan 29, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

anthony-murphy commented Feb 3, 2026

Uh oh!

anthony-murphy commented Feb 3, 2026

Uh oh!

azure-pipelines bot commented Feb 3, 2026

Uh oh!

azure-pipelines bot commented Feb 3, 2026

Uh oh!

Uh oh!

anthony-murphy commented Feb 4, 2026

Uh oh!

anthony-murphy commented Feb 4, 2026

Uh oh!

azure-pipelines bot commented Feb 4, 2026

Uh oh!

azure-pipelines bot commented Feb 4, 2026

Uh oh!

Uh oh!

anthony-murphy commented Feb 4, 2026

Uh oh!

anthony-murphy commented Feb 4, 2026

Uh oh!

azure-pipelines bot commented Feb 4, 2026

Uh oh!

azure-pipelines bot commented Feb 4, 2026

Uh oh!

anthony-murphy-agent commented Feb 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

anthony-murphy-agent commented Jan 29, 2026 •

edited by azure-boards bot

Loading