Skip to content

fix(site-components): preserve latest doc search results - #896

Open
patrick-andstar wants to merge 1 commit into
Tencent:mainfrom
patrick-andstar:codex/doc-search-request-race
Open

patrick-andstar wants to merge 1 commit into
Tencent:mainfrom
patrick-andstar:codex/doc-search-request-race

Conversation

@patrick-andstar

Copy link
Copy Markdown
Contributor

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

No related issue. Duplicate search keywords: doc-search race, td-doc-search.

💡 需求背景和解决方案

An older documentation search can settle after a newer search has completed and clear the current results. The request controller is now captured for each search and results are applied only while that controller is still current.

Added a regression test covering a late failure from an older request after newer results are displayed.

Validation:

  • pnpm --filter @tdesign/site-components exec jest --coverage --runInBand
  • pnpm exec eslint packages/site-components/src/components/td-doc-search/state.js packages/site-components/test/td-doc-search-state.test.js
  • pnpm exec prettier --check packages/site-components/src/components/td-doc-search/state.js packages/site-components/test/td-doc-search-state.test.js
  • pnpm --filter @tdesign/site-components build

📝 更新日志

  • fix(site-components): preserve the latest documentation search results.

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

@pkg-pr-new

pkg-pr-new Bot commented Jul 28, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@tdesign/site-components@896
npm i https://pkg.pr.new/@tdesign/theme-generator@896

commit: f4b7fe8

@github-actions

Copy link
Copy Markdown

TDesign Component Site Preview CI Open

Sites (9)
Component Preview
tdesign-site 完成
tdesign-vue 完成
tdesign-react 完成
tdesign-vue-next 完成
@tdesign-vue-next/chat 完成
tdesign-mobile-vue 完成
tdesign-mobile-react 完成
tdesign-miniprogram 完成
tdesign-uniapp 完成

This branch has not been deployed

No deployments
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