Skip to content

Conversation

@c121914yu
Copy link
Collaborator

No description provided.

@c121914yu c121914yu merged commit 25207c5 into labring:main Sep 9, 2025
1 of 5 checks passed
@gru-agent
Copy link
Contributor

gru-agent bot commented Sep 9, 2025

TestGru Assignment

Summary

Link CommitId Status Reason
Detail 49b6c35 🚫 Skipped No files need to be tested {"packages/service/core/workflow/dispatch/dataset/search.ts":"File path does not match include patterns."}

History Assignment

Tip

You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

@c121914yu c121914yu deleted the perf-dataset-response branch September 9, 2025 06:06
content: `${item.q}\n${item.a}`.trim()
}))
}
: 'No results'
Copy link

Choose a reason for hiding this comment

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

Bug: Inconsistent toolResponses Types Cause Errors

The toolResponses field now has inconsistent types. It returns an object when results are found, a string 'No results' when searchRes.length is zero, and an empty array [] in other no-result scenarios. This type variation can cause runtime errors for code expecting a consistent object structure.

Fix in Cursor Fix in Web

c121914yu added a commit that referenced this pull request Sep 11, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* temp: list value

* backspace

* optimize code

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 11, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* fix debug variables

* fix

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 11, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* temp: list value

* backspace

* optimize code

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 11, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* fix debug variables

* fix

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 12, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* temp: list value

* backspace

* optimize code

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 12, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* fix debug variables

* fix

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 14, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* temp: list value

* backspace

* optimize code

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 14, 2025
* perf: search result (#5608)

* fix: table size (#5598)

* fix debug variables

* fix

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
c121914yu added a commit that referenced this pull request Sep 15, 2025
* fix: push again, user select option button and form input radio content overflow (#5601)

* fix: push again, user select option button and form input radio content overflow

* fix: use useCallback instead of useMemo, fix unnecessary delete

* fix: Move the variable inside the component

* fix: do not pass valueLabel to MySelect

* ui

* del collection api adapt

* refactor: inherit permission (#5529)

* refactor: permission update conflict check function

* refactor(permission): app collaborator update api

* refactor(permission): support app update collaborator

* feat: support fe permission conflict check

* refactor(permission): app permission

* refactor(permission): dataset permission

* refactor(permission): team permission

* chore: fe adjust

* fix: type error

* fix: audit pagiation

* fix: tc

* chore: initv4130

* fix: app/dataset auth logic

* chore: move code

* refactor(permission): remove selfPermission

* fix: mock

* fix: test

* fix: app & dataset auth

* fix: inherit

* test(inheritPermission): test syncChildrenPermission

* prompt editor add list plugin (#5620)

* perf: search result (#5608)

* fix: table size (#5598)

* temp: list value

* backspace

* optimize code

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>

* fix: fe & member list (#5619)

* chore: initv4130

* fix: MemberItemCard

* fix: MemberItemCard

* chore: fe adjust & init script

* perf: test code

* doc

* fix debug variables (#5617)

* perf: search result (#5608)

* fix: table size (#5598)

* fix debug variables

* fix

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>

* perf: member ui

* fix: inherit bug (#5624)

* refactor(permission): remove getClbsWithInfo, which is useless

* fix: app list privateApp

* fix: get infos

* perf(fe): remove delete icon when it is disable in MemberItemCard

* fix: dataset private dataset

* Apply suggestion from @Copilot

Co-authored-by: Copilot <[email protected]>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: Archer <[email protected]>
Co-authored-by: Copilot <[email protected]>

* perf: auto coupon

* chore: upgrade script & get infos avatar  (#5625)

* fix: get infos

* chore: initv4130

* feat: support WecomRobot publish, and fix AesKey can not save bug (#5526)

* feat: resolve conflicts

* fix: add param 'show_publish_wecom'

* feat: abstract out WecomCrypto type

* doc: wecom robot document

* fix: solve instability in AI output

* doc: update some pictures

* feat: remove functions from request.ts to chat.ts and toolCall.ts

* doc: wecom robot doc update

* fix

* delete unused code

* doc: update version and prompt

* feat: remove wecom crypto, delete wecom code in workflow

* feat: delete unused codes

---------

Co-authored-by: heheer <[email protected]>

* remove test

* rename init shell

* feat: collection page store

* reload sandbox

* pysandbox

* remove log

* chore: remove useless code (#5629)

* chore: remove useless code

* fix: checkConflict

* perf: support hidden type for RoleList

* fix: copy node

* update doc

* fix(permission): some bug (#5632)

* fix: app/dataset list

* fix: inherit bug

* perf: del app;i18n;save chat

* fix: test

* i18n

* fix: sumper overflow return OwnerRoleVal (#5633)

* remove invalid code

* fix: scroll

* fix: objectId

* update next

* update package

* object id

* mock redis

* feat: add redis append to resolve wecom stream response  (#5643)

* feat: resolve conflicts

* fix: add param 'show_publish_wecom'

* feat: abstract out WecomCrypto type

* doc: wecom robot document

* fix: solve instability in AI output

* doc: update some pictures

* feat: remove functions from request.ts to chat.ts and toolCall.ts

* doc: wecom robot doc update

* fix

* delete unused code

* doc: update version and prompt

* feat: remove wecom crypto, delete wecom code in workflow

* feat: delete unused codes

* feat: add redis append method

---------

Co-authored-by: heheer <[email protected]>

* cache per

* fix(test): init team sub when creating mocked user (#5646)

* fix: button is not vertically centered (#5647)

* doc

* fix: gridFs objectId (#5649)

---------

Co-authored-by: Zeng Qingwen <[email protected]>
Co-authored-by: Finley Ge <[email protected]>
Co-authored-by: heheer <[email protected]>
Co-authored-by: 伍闲犬 <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: heheer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant