chore: merge main to feature#1888
Merged
Merged
Conversation
Bumps [dawidd6/action-download-artifact](https://github.com/dawidd6/action-download-artifact) from 16 to 19. - [Release notes](https://github.com/dawidd6/action-download-artifact/releases) - [Commits](dawidd6/action-download-artifact@v16...v19) --- updated-dependencies: - dependency-name: dawidd6/action-download-artifact dependency-version: '19' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* Fix X SDK link in OpenAI model usage guide Updated the link to the X SDK documentation for consistency. Signed-off-by: xiaohai <xiaohp1993@gmail.com> * Fix link to X SDK documentation in guide Signed-off-by: xiaohai <xiaohp1993@gmail.com> --------- Signed-off-by: xiaohai <xiaohp1993@gmail.com> Co-authored-by: Mickey <951203214@qq.com>
… to false (#1851) * fix(ThoughtChain): remove duplicate className in clsx call The `className` prop was passed twice to `clsx()` when computing the root element's merged class names — a copy-paste error. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(ThoughtChain): default contentOpen to false instead of undefined `expandedKeys?.includes(key)` returns `undefined` when `expandedKeys` is undefined (e.g. ThoughtChain used without controlled expandedKeys). This `undefined` was passed to CSSMotion's `visible` prop which expects a boolean. Added `?? false` to ensure a proper boolean default. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * test(ThoughtChain): use class attribute string to detect duplicate className tokens Per DOM spec, `Element.classList` (DOMTokenList) maintains an ordered set that de-duplicates tokens. So the previous test using `root.classList` could not actually detect duplicate class tokens in the source class attribute — it would pass even when the component incorrectly emitted duplicate classes. Fixed by reading the raw `class` attribute via `getAttribute('class')` and counting token occurrences via regex, so the test correctly fails if the duplicate is reintroduced. Thanks to @coderabbitai for catching this. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: Mickey <951203214@qq.com>
chore: Main merge feature
* chore: changelog of 2.6.0 * chore: changelog of 2.6.0
* feat(theme): add zeroRuntime support * fix useInternalToken type signature --------- Co-authored-by: Mickey <951203214@qq.com>
…nload-artifact-19
…widd6/action-download-artifact-19 chore(deps): bump dawidd6/action-download-artifact from 16 to 19
docs(dumi): add SearchBar
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request bumps the version to 2.6.0 across the monorepo and introduces several improvements and bug fixes. Key changes include the addition of a zeroRuntime theme option to skip runtime component style injection, a new expandable SearchBar component for the documentation site, and fixes for duplicate className and default expansion states in the ThoughtChain component. Additionally, the removeMessage API return type was updated to boolean in the documentation and skills, and the DeepSeek R1 integration example was significantly improved. I have no feedback to provide as there were no review comments to assess.
Contributor
|
This comment is automatically generated by the x-markdown performance CI. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
Contributor
size-limit report 📦
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feature #1888 +/- ##
========================================
Coverage 97.46% 97.47%
========================================
Files 158 158
Lines 5485 5505 +20
Branches 1596 1610 +14
========================================
+ Hits 5346 5366 +20
Misses 137 137
Partials 2 2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Bundle ReportChanges will increase total bundle size by 1.48MB (72.29%) ⬆️
Affected Assets, Files, and Routes:view changes for bundle: antdx-array-pushAssets Changed:
view changes for bundle: x-markdown-array-pushAssets Changed:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
中文版模板 / Chinese template
🤔 This is a ...
🔗 Related Issues
💡 Background and Solution
📝 Change Log