Skip to content

Tool-picker: implement tool recommendation#6

Merged
AdityaKhatri merged 11 commits intodevelopfrom
feat/add-tool-mutation
Feb 20, 2026
Merged

Tool-picker: implement tool recommendation#6
AdityaKhatri merged 11 commits intodevelopfrom
feat/add-tool-mutation

Conversation

@sudip-khanal
Copy link
Copy Markdown
Contributor

Changes

  • Implement scoring-based recommendation logic comparing user and tool answers
  • Select best matching tool per submission
  • Add user submission mutation to capture user answers
  • Add test cases for submission and recommendation flow
  • Add type hints for related managers to fix static typing issues

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • n+1 queries
  • flake8 issues
  • print
  • typos
  • unwanted comments

This PR contains valid:

  • tests
  • permission checks (tests here too)

@sudip-khanal sudip-khanal changed the title feat(tool-picker): implement scoring-based tool recommendation Tool-picker: implement tool recommendation Feb 3, 2026
Comment thread apps/tool_picker/models.py
Comment thread apps/tool_picker/serializers.py Outdated
Comment thread apps/tool_picker/serializers.py Outdated
Comment thread apps/tool_picker/serializers.py Outdated
Comment thread apps/tool_picker/serializers.py Outdated
Comment thread apps/tool_picker/utils.py Outdated
Comment thread apps/tool_picker/utils.py Outdated
Comment thread apps/tool_picker/utils.py Outdated
@Rup-Narayan-Rajbanshi
Copy link
Copy Markdown

Rup-Narayan-Rajbanshi commented Feb 3, 2026

@sudip-khanal lets always resolve the conflicts after sending PR and before assigning PR assignee..

@sudip-khanal sudip-khanal force-pushed the feat/add-tool-mutation branch 2 times, most recently from 1f56bdf to db51d16 Compare February 4, 2026 06:04
Comment thread main/graphql/schema.py Outdated
Comment thread apps/tool_picker/tests/mutation_test.py Outdated
Comment thread apps/tool_picker/utils.py Outdated
Comment thread apps/tool_picker/utils.py Outdated
Comment thread apps/tool_picker/serializers.py Outdated
Comment thread apps/tool_picker/serializers.py Outdated
Comment thread apps/tool_picker/utils.py Outdated
@sudip-khanal sudip-khanal force-pushed the feat/add-tool-mutation branch 3 times, most recently from bb01244 to 9a22767 Compare February 18, 2026 05:33
- Implement scoring-based recommendation logic comparing user and tool answers
- Select best matching tool per submission
- Add user submission mutation to capture user answers
- Add test cases for submission and recommendation flow
- Add type hints for related managers to fix static typing issues
… catalog questions

- Validate missing and extra catalog questions
- Show question titles in error messages
…calculation logic

- Add a filter in the recommendation calculation logic to consider only the selected options from checkbox-type questions.
@sudip-khanal sudip-khanal force-pushed the feat/add-tool-mutation branch from 9a22767 to 1e77bd4 Compare February 18, 2026 08:59
Comment thread apps/tool_picker/graphql/types.py Outdated
Comment thread apps/tool_picker/utils.py
Comment thread apps/tool_picker/serializers.py Outdated
@AdityaKhatri AdityaKhatri force-pushed the feat/add-tool-mutation branch from 2d78d79 to edb9786 Compare February 20, 2026 04:29
@AdityaKhatri AdityaKhatri merged commit ef1b4ae into develop Feb 20, 2026
2 checks passed
@AdityaKhatri AdityaKhatri deleted the feat/add-tool-mutation branch February 20, 2026 04:31
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.

4 participants