Skip to content

Tag asserts for release - #27640

Merged
Craig Macomber (Microsoft) (CraigMacomber) merged 1 commit into
microsoft:mainfrom
CraigMacomber:tagAsserts7126435
Jul 6, 2026
Merged

Tag asserts for release#27640
Craig Macomber (Microsoft) (CraigMacomber) merged 1 commit into
microsoft:mainfrom
CraigMacomber:tagAsserts7126435

Conversation

@CraigMacomber

Copy link
Copy Markdown
Contributor

Description

Run pnpm policy-check:asserts for release.

Reviewer Guidance

The review process is outlined on this wiki page.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Hi! Thank you for opening this PR. Want me to review it?

Based on the diff (29 lines, 6 files), I've queued these reviewers:

  • Correctness — logic errors, race conditions, lifecycle issues
  • Security — vulnerabilities, secret exposure, injection
  • API Compatibility — breaking changes, release tags, type design
  • Performance — algorithmic regressions, memory leaks
  • Testing — coverage gaps, hollow tests

How this works

  • Adjust the reviewer set by ticking/unticking boxes above. Reviewer toggles alone don't trigger anything.

  • Tick Start review below to dispatch the review fleet.

  • After review finishes, tick Start review again to request another run — it auto-resets after each dispatch.

  • This comment updates as new commits land; your reviewer selections are preserved.

  • Start review

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Bundle size comparison

Base commit: 32bbb06cf7f7a59075624cc48c89c81b7687d890
Head commit: a25b634124158bc79bce2fa9d59a2d3d77546f8b

Notable changes

No bundles changed by ≥ 500 bytes parsed.

Per-bundle deltas

@fluid-example/bundle-size-tests

  • azureClient.js: parsed 619098 → 619154 (+56), gzip 164824 → 164872 (+48)
  • odspClient.js: parsed 591824 → 591880 (+56), gzip 158916 → 158959 (+43)
  • aqueduct.js: parsed 525587 → 525622 (+35), gzip 140709 → 140737 (+28)
  • fluidFramework.js: parsed 393001 → 392766 (-235), gzip 111618 → 111539 (-79)
  • sharedTree.js: parsed 382388 → 382146 (-242), gzip 109007 → 108920 (-87)
  • containerRuntime.js: parsed 303937 → 303951 (+14), gzip 83213 → 83222 (+9)
  • sharedString.js: parsed 175984 → 175991 (+7), gzip 49445 → 49453 (+8)
  • experimentalSharedTree.js: parsed 160798 → 160798 (0), gzip 45804 → 45804 (0)
  • matrix.js: parsed 159845 → 159852 (+7), gzip 45411 → 45418 (+7)
  • loader.js: parsed 145256 → 145270 (+14), gzip 39062 → 39076 (+14)
  • odspDriver.js: parsed 104329 → 104350 (+21), gzip 32649 → 32658 (+9)
  • directory.js: parsed 66616 → 66623 (+7), gzip 18532 → 18540 (+8)
  • 748.js: parsed 58793 → 58793 (0), gzip 17826 → 17826 (0)
  • map.js: parsed 46709 → 46716 (+7), gzip 14310 → 14318 (+8)
  • odspPrefetchSnapshot.js: parsed 45642 → 45656 (+14), gzip 15276 → 15285 (+9)
  • 594.js: parsed 44493 → 44493 (0), gzip 13744 → 13744 (0)
  • summarizerDelayLoadedModule.js: parsed 30749 → 30749 (0), gzip 7753 → 7753 (0)
  • socketModule.js: parsed 26477 → 26484 (+7), gzip 7885 → 7894 (+9)
  • createNewModule.js: parsed 12480 → 12480 (0), gzip 4786 → 4786 (0)
  • summaryModule.js: parsed 3797 → 3797 (0), gzip 1860 → 1860 (0)
  • connectionState.js: parsed 724 → 724 (0), gzip 429 → 429 (0)
  • sharedTreeAttributes.js: parsed 666 → 673 (+7), gzip 432 → 441 (+9)
  • debugAssert.js: parsed 429 → 429 (0), gzip 299 → 299 (0)
  • FluidFramework-HashFallback.js: parsed 422 → 422 (0), gzip 316 → 316 (0)

@CraigMacomber
Craig Macomber (Microsoft) (CraigMacomber) merged commit 477345b into microsoft:main Jul 6, 2026
31 checks passed
@CraigMacomber
Craig Macomber (Microsoft) (CraigMacomber) deleted the tagAsserts7126435 branch July 6, 2026 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants