Skip to content

v0.6.77: mothership improvements, trigger.dev telemetry#4585

Merged
icecrasher321 merged 5 commits into
mainfrom
staging
May 14, 2026
Merged

v0.6.77: mothership improvements, trigger.dev telemetry#4585
icecrasher321 merged 5 commits into
mainfrom
staging

Conversation

@icecrasher321
Copy link
Copy Markdown
Collaborator

@icecrasher321 icecrasher321 commented May 13, 2026

Sg312 and others added 4 commits May 13, 2026 14:37
…4582)

* feat(mothership): pin tasks to keep them at the top of the sidebar

* fix(sidebar): address PR review feedback for pin tasks

* fix(posthog): register task_pinned and task_unpinned events

* fix(tasks): insert new optimistic tasks below pinned partition
* Add files to mship block

* Fixes

* Fix

* fix
…LP (#4583)

* feat(observability): export Trigger.dev telemetry to Grafana Cloud OTLP

Wire OTLP HTTP exporters for traces, logs, and metrics from the
Trigger.dev runtime to Grafana Cloud. Auth uses Basic with instance ID
and API token. Gated behind GRAFANA_OTLP_ENDPOINT, GRAFANA_INSTANCE_ID,
and GRAFANA_API_TOKEN — all three must be set together or all unset;
partial config throws at startup.

* improvement(observability): use OTLP HTTP/JSON for metrics for consistency with traces and logs

* feat(observability): tag Trigger.dev telemetry with deployment.environment.name

* improvement(observability): switch Grafana telemetry vars to OTLP-shaped trio
@vercel
Copy link
Copy Markdown

vercel Bot commented May 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped May 14, 2026 0:01am

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented May 13, 2026

PR Summary

Medium Risk
Moderate risk due to a DB migration (copilot_chats.pinned) affecting task ordering plus new file-attachment materialization and timeout handling that touch execution/tool boundaries.

Overview
Mothership tasks can now be pinned end-to-end: adds copilot_chats.pinned (migration), returns it in /api/mothership/chats, sorts pinned-first, allows PATCH updates, and emits PostHog task_pinned/task_unpinned events; the sidebar adds a Pin/Unpin context-menu action, shows a pin indicator, and introduces an optimistic useSetTaskPinned mutation that preserves pinned ordering.

Mothership block execution now supports file attachments: the block schema adds files, the executor materializes referenced/uploaded files into base64 fileAttachments (size-capped) and forwards them through /api/mothership/execute (contract updated), with tests covering payload shaping.

Copilot tool execution adds function_execute timeout support: tool schemas expose timeout (seconds), and the tool executor converts/validates it to milliseconds for Copilot-initiated calls with a 10s default and a cap at DEFAULT_EXECUTION_TIMEOUT_MS.

Observability: extracts OTLP header parsing into lib/monitoring/otlp, adds optional Trigger.dev telemetry exporters (traces/logs/metrics) to Grafana OTLP when GRAFANA_OTLP_* env vars are fully set, and adds the needed OTLP exporter dependencies.

Reviewed by Cursor Bugbot for commit 4de955d. Bugbot is set up for automated code reviews on this repo. Configure here.

@icecrasher321 icecrasher321 changed the title v0.6.77: mothership improvements v0.6.77: mothership improvements, trigger.dev telemetry May 13, 2026
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit ed39edb. Configure here.

Comment thread apps/sim/trigger.config.ts Outdated
Comment thread apps/sim/trigger.config.ts Outdated
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 13, 2026

Greptile Summary

Review completed and submitted via the greptile-review MCP tools.

Confidence Score: 4/5

Safe to merge with minor schema and naming feedback.

All three features are well-tested and self-contained. The only notable gap is the optional source field in the file attachment schema, which could allow a malformed payload to pass validation and reach the Go backend.

apps/sim/lib/api/contracts/mothership-tasks.ts and apps/sim/hooks/queries/tasks.ts

Reviews (1): Last reviewed commit: "feat(observability): export Trigger.dev ..." | Re-trigger Greptile

@icecrasher321 icecrasher321 merged commit ab156b5 into main May 14, 2026
31 checks passed
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.

4 participants