Skip to content

Bump deps#5813

Draft
compulim wants to merge 33 commits intomicrosoft:mainfrom
compulim:feat-bump
Draft

Bump deps#5813
compulim wants to merge 33 commits intomicrosoft:mainfrom
compulim:feat-bump

Conversation

@compulim
Copy link
Copy Markdown
Contributor

@compulim compulim commented Apr 22, 2026

Changelog Entry

Changed

Description

Bump dependencies.

TypeScript 5 -> 6:

  • strict: true is now default
    • We fixed some strictness issues, temporarily setting strict: false, will revisit in future
  • types: [] is now default (replacing types: ['*'] which import /node_modules/@types/* automatically)
  • ignoreDeprecations: '6.0' is required for tsup as it use baseUrl
  • TS4094: exporting anonymous class may not have private/protected members
    • The root cause is tsc failed to build .d.ts which contains ES-private members (member prefixed with #)
    • Instead of exporting as-is, we are exporting as needed (i.e. exporting an abstract class which exclude private members)
  • valibot.lazy() requires fully typed out GenericSchema<Input, Output>
    • We are now fully typing out OrgSchemaXXX, and all non-@ properties are always output as array for simplicity and alignment with graph
  • StrictStyleOptions is now first-class citizen, StyleOptions is a derived from it
    • This is because Required<{ one?: number | undefined }> results in { one: number }
      • We could do a remap like { [P in keyof T]-?: T[P] | undefined }, however, it's better to clean up the typing (strict first, relax later)

We are also removing some deprecated style options for simplicity:

  • hideScrollToEndButton
  • newMessagesButtonFontSize
  • suggestedActionsActiveBackground
  • suggestedActionsBackground
  • suggestedActionsDisabledBackground
  • suggestedActionsDisabledBorderColor
  • suggestedActionsDisabledBorderStyle
  • suggestedActionsDisabledBorderWidth
  • suggestedActionsDisabledTextColor
  • suggestedActionsFocusBackground
  • suggestedActionsHoverBackground

Specific Changes

  • Fixed some bump scripts failure
  • npm run bump
  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

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.

1 participant