Skip to content

Trace data migrations#26809

Merged
thdxr merged 1 commit into
devfrom
opencode/data-migration-span
May 11, 2026
Merged

Trace data migrations#26809
thdxr merged 1 commit into
devfrom
opencode/data-migration-span

Conversation

@thdxr
Copy link
Copy Markdown
Member

@thdxr thdxr commented May 11, 2026

Summary

  • Wrap each data migration run in an Effect span named DataMigration.
  • Add the migration name as a span attribute for observability.

Testing

  • bun typecheck from packages/opencode

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@thdxr thdxr enabled auto-merge (squash) May 11, 2026 03:22
@thdxr thdxr merged commit 7235c9c into dev May 11, 2026
12 checks passed
@thdxr thdxr deleted the opencode/data-migration-span branch May 11, 2026 03:23
joshue031 pushed a commit to joshue031/opencode that referenced this pull request May 11, 2026
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.

1 participant