Skip to content

AI assistant: fix for sometimes not seeing the attached file #2434

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jurgenwerk
Copy link
Contributor

@jurgenwerk jurgenwerk commented Apr 15, 2025

This bug was reported several times but was really hard to reproduce 😵‍💫. It turns out that in code mode, when attaching files, we were always adding the "'You are unable to edit any cards..." message and in some cases AI took that very seriously even though there was an attached gts file and the user wanted to edit templates, for example.

image

The fix is to not include the "no cards to edit" system prompt message when there is an attached file present.

@jurgenwerk jurgenwerk force-pushed the cs-8406-ai-assistant-in-code-mode-sometimes-seems-to-not-see-the branch from 0e7a491 to 5228fd6 Compare April 15, 2025 08:36
@jurgenwerk jurgenwerk force-pushed the cs-8406-ai-assistant-in-code-mode-sometimes-seems-to-not-see-the branch from 5228fd6 to abc93e1 Compare April 15, 2025 08:37
@jurgenwerk jurgenwerk marked this pull request as ready for review April 15, 2025 08:42
@jurgenwerk jurgenwerk requested review from lukemelia, IanCal and a team April 15, 2025 08:45
Copy link

github-actions bot commented Apr 15, 2025

Host Test Results

  1 files  ±0    1 suites  ±0   29m 49s ⏱️ -3s
910 tests ±0  904 ✅ ±0  6 💤 ±0  0 ❌ ±0 
917 runs  ±0  911 ✅ ±0  6 💤 ±0  0 ❌ ±0 

Results for commit 76e1a93. ± Comparison against base commit 08efb10.

♻️ This comment has been updated with latest results.

@jurgenwerk jurgenwerk changed the title AI assistant: fix for sometimes not seeing to not seeing the attached file AI assistant: fix for sometimes not seeing the attached file Apr 15, 2025
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.

3 participants