Skip to content

Use long text columns for conversation payloads#623

Open
gracjankubicki wants to merge 1 commit into
laravel:0.xfrom
gracjankubicki:fix/conversation-storage-longtext
Open

Use long text columns for conversation payloads#623
gracjankubicki wants to merge 1 commit into
laravel:0.xfrom
gracjankubicki:fix/conversation-storage-longtext

Conversation

@gracjankubicki
Copy link
Copy Markdown
Contributor

@gracjankubicki gracjankubicki commented May 19, 2026

Summary

This updates the default database conversation storage schema to use longText columns for conversation message payloads.

The following agent_conversation_messages columns now use longText:

  • content
  • attachments
  • tool_calls
  • tool_results
  • usage
  • meta

Motivation

Agent conversation messages may contain large payloads, including long assistant responses, attachment metadata, tool calls, tool results, usage information, and provider metadata.

On MySQL-compatible databases, regular text columns can be too small for these payloads. Using longText gives the default conversation storage schema more room for larger AI conversation data without changing the public API.

Upgrade note

New installations will receive longText payload columns from the default conversation migration.

Applications that have already published and run the previous conversation migration should update their published migration or add an application-level migration to alter the existing agent_conversation_messages payload columns from text to longText.

Tests

  • composer test -- tests/Feature/Storage/DatabaseConversationStoreTest.php
  • composer test:lint
  • vendor/bin/phpstan --memory-limit=-1
  • composer test

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

hey migration provided by the sdk should be forward looking. we probably don't need this migration. we can add it in release notes though.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sounds good, removed the follow-up migration and left the upgrade path as a release note in the PR description.

@gracjankubicki gracjankubicki force-pushed the fix/conversation-storage-longtext branch from abe2f49 to a859c37 Compare May 19, 2026 20:00
@pushpak1300 pushpak1300 requested a review from taylorotwell May 23, 2026 09:53
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