Skip to content

fix: Generate title for attachment-only messages#1674

Merged
charlesvien merged 2 commits intomainfrom
04-15-generate_title_for_attachment-only_messages
Apr 16, 2026
Merged

fix: Generate title for attachment-only messages#1674
charlesvien merged 2 commits intomainfrom
04-15-generate_title_for_attachment-only_messages

Conversation

@charlesvien
Copy link
Copy Markdown
Member

@charlesvien charlesvien commented Apr 15, 2026

Problem

Chats started with only a pasted file and no text get stuck with "Untitled" because the title generator never fires.

Changes

  1. Use extractPromptDisplayContent to check for both text and attachments in user prompts
  2. Fall back to attachment filenames when no visible text is present
  3. Ensures promptCount increments so title generation triggers

How did you test this?

Manually

@charlesvien charlesvien changed the title Generate title for attachment-only messages fix: Generate title for attachment-only messages Apr 15, 2026
@charlesvien charlesvien force-pushed the 04-15-generate_title_for_attachment-only_messages branch from e9a68ca to d6d1721 Compare April 15, 2026 21:01
@charlesvien charlesvien marked this pull request as ready for review April 15, 2026 21:03
@charlesvien charlesvien merged commit 11f451d into main Apr 16, 2026
16 checks passed
Copy link
Copy Markdown
Member Author

Merge activity

@charlesvien charlesvien deleted the 04-15-generate_title_for_attachment-only_messages branch April 16, 2026 12:48
vdekrijger pushed a commit that referenced this pull request Apr 16, 2026
## Problem

Chats started with only a pasted file and no text get stuck with "Untitled" because the title generator never fires.<!-- Who is this for and what problem does it solve? -->

<!-- Closes #ISSUE_ID -->

## Changes

1. Use extractPromptDisplayContent to check for both text and attachments in user prompts
2. Fall back to attachment filenames when no visible text is present
3. Ensures promptCount increments so title generation triggers

<!-- What did you change and why? -->

<!-- If there are frontend changes, include screenshots. -->

## How did you test this?

Manually

<!-- Describe what you tested -- manual steps, automated tests, or both. -->

<!-- If you're an agent, only list tests you actually ran. -->
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.

2 participants