Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 17, 2026

The Criteria dialog exposed "Ask AI" as a selectable criteria option, creating confusion since a dedicated Ask AI button already provides this functionality with a different UX.

Changes

  • Added filtering logic in teachertool/src/state/helpers.ts to exclude ai_question criteria from the getCatalogCriteria helper
  • Preserved ai_question criteria configuration in common-docs/teachertool/catalog-shared.json (needed for Ask AI overlay's default questions)
  • Preserved Ask AI instances in general-code-quality.json example checklist

Impact
Users now access Ask AI exclusively through the dedicated button. The Ask AI criteria configuration remains available for the AskAIOverlay component to provide default question prompts, while being filtered out from the "Add Criteria" dialog display. The AskAIButton, AskAIOverlay, and addAiQuestionCriteriaToChecklist components remain unchanged and functional.

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove Ask AI option from Criteria dialog window</issue_title>
<issue_description>Now that we have a separate Ask AI button, let's remove the Ask AI option from the Criteria dialog window.
This creates confusion since there are 2 different ways to do the same thing, and the experiences are different.

Image

</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: abchatra <6107272+abchatra@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove Ask AI option from Criteria dialog window Remove Ask AI criteria from catalog dialog Jan 17, 2026
Copilot AI requested a review from abchatra January 17, 2026 06:27
@jwunderl
Copy link
Member

@copilot the configuration there is still used for e.g. default questions. Probably best to remove from the teachertool portion itself, we already have the opposite logic to fill ask ai so just filter them out from the add criteria section

…ing from config

Co-authored-by: jwunderl <5615930+jwunderl@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 20, 2026

@copilot the configuration there is still used for e.g. default questions. Probably best to remove from the teachertool portion itself, we already have the opposite logic to fill ask ai so just filter...

Updated in commit 30c1218. Restored the catalog configuration and added filtering logic in CatalogOverlay.tsx to exclude ai_question criteria from the display while keeping it available for the Ask AI overlay's default questions.

Co-authored-by: jwunderl <5615930+jwunderl@users.noreply.github.com>
Copilot AI requested a review from jwunderl January 20, 2026 19:29
@jwunderl jwunderl marked this pull request as ready for review January 21, 2026 00:22
@jwunderl jwunderl enabled auto-merge (squash) January 21, 2026 00:23
@jwunderl jwunderl merged commit 17e29df into master Jan 21, 2026
20 checks passed
@jwunderl jwunderl deleted the copilot/remove-ask-ai-option branch January 21, 2026 00:32
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.

Remove Ask AI option from Criteria dialog window

3 participants