Skip to content

feat: add knowledge-grounded search suggestions - #229

Merged
lgqfhwy merged 2 commits into
feat/aishopping-field-aware-fallbackfrom
codex/knowledge-enable-suggestion-v1
Aug 11, 2026
Merged

feat: add knowledge-grounded search suggestions#229
lgqfhwy merged 2 commits into
feat/aishopping-field-aware-fallbackfrom
codex/knowledge-enable-suggestion-v1

Conversation

@lgqfhwy

@lgqfhwy lgqfhwy commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • add enable_suggestion support to the AI shopping /api/chat_search flow
  • generate three knowledge-grounded product-search suggestions concurrently with the reply after the final product snapshot is fixed
  • return suggestion failures in a nested payload without affecting the main streamed reply
  • add standalone POST /api/search_suggestion, which accepts a query, performs knowledge retrieval, and returns one suggestion
  • extract shared knowledge evidence handling and add a shared, config-driven suggestion generator, tool schema, context builder, and validator
  • add a knowledge-only HA3 recall type for standalone deployments without AIChatConfig or Ha3ChatRecallConf

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code review by ChatGPT

Comment thread service/aishopping/suggestion_coordinator.go
Comment thread service/recall/ha3_chat_knowledge.go
Comment thread web/search_suggestion_controller.go

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code review by ChatGPT

Comment thread web/search_suggestion_controller.go
@lgqfhwy
lgqfhwy marked this pull request as ready for review August 11, 2026 08:59
@lgqfhwy
lgqfhwy merged commit fd559f6 into feat/aishopping-field-aware-fallback Aug 11, 2026
2 checks passed
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