Skip to content

Remove Temporal runtime and workflow surface; route notifications via OTel#52

Merged
jaromil merged 20 commits intomainfrom
remove-temporal-otel-notify
Feb 18, 2026
Merged

Remove Temporal runtime and workflow surface; route notifications via OTel#52
jaromil merged 20 commits intomainfrom
remove-temporal-otel-notify

Conversation

@jaromil
Copy link
Member

@jaromil jaromil commented Feb 18, 2026

  • Removed Temporal workflow APIs/UI/config surface and status fields; ensured removed routes return 404.
  • Replaced workflow-driven flow dispatch with in-process flow runtime and OTel-backed notifications.
  • Cleaned prompts/docs/AGENTS, updated tests/fixtures, and added a CI guard to prevent Temporal reintroduction.
  • Added frontend dependency orga.

Key changes

  • Backend: workflow routes/types removed, events SSE trimmed, status payload updated, flow event types cleaned.
  • Frontend: workflow client methods and Temporal UI removed; FlowView/event samples updated.
  • Config/docs: Temporal settings removed; docs, README, AGENTS updated; migration checklist added.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request removes the Temporal workflow runtime and replaces workflow-driven flow dispatch with an in-process flow runtime that routes notifications via OTel. The change affects backend workflow APIs, frontend Temporal UI integration, configuration settings, and documentation.

Changes:

  • Removed Temporal workflow APIs (/api/workflows, /api/sessions/:id/workflow/resume, /api/sessions/:id/workflow/history) and status fields
  • Replaced workflow-driven flow dispatch with in-process flow runtime using OTel-backed notifications
  • Added CI guard script to prevent Temporal reintroduction and frontend dependency orga

Reviewed changes

Copilot reviewed 114 out of 116 changed files in this pull request and generated no comments.

Show a summary per file
File Description
scripts/check-no-temporal.sh New CI guard to prevent Temporal patterns from being reintroduced
internal/terminal/session*.go Removed workflow methods and fields from Session type
internal/terminal/session_factory.go Added NotificationSink for direct event emission
internal/terminal/manager.go Removed Temporal client/worker integration
internal/api/workflow_routes_test.go New test verifying removed routes return 404
internal/api/rest_*.go Removed workflow endpoints and Temporal UI URL from status
internal/api/notifications_sse_handler.go Updated to read from OTel LogHub instead of notification bus
internal/flow/service.go Replaced Temporal client with Dispatcher interface
internal/flow/runtime/dispatcher*.go New in-process dispatcher for flow activities
internal/notify/*.go New notification sink abstraction for OTel integration
internal/config/settings.go Removed Temporal configuration settings
cmd/gestalt/temporal.go Removed Temporal dev server lifecycle management
cmd/gestalt/server_command.go Removed Temporal client/worker startup and shutdown phases
frontend/src/lib/apiClient.js Removed workflow API methods
frontend/src/components/Terminal.svelte Removed Temporal UI button and added connection failure callback
frontend/src/views/*.svelte Removed Temporal context loading and UI elements
frontend/package.json Added orga dependency for org-mode parsing
docs/**/*.md Updated documentation to reflect workflow removal
config/gestalt.toml Removed Temporal settings section
.github/workflows/tests.yml Added Temporal guard check to CI
Files not reviewed (1)
  • frontend/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jaromil jaromil merged commit efcc616 into main Feb 18, 2026
6 checks passed
@jaromil jaromil deleted the remove-temporal-otel-notify branch February 18, 2026 20:57
@github-actions
Copy link

🎉 This PR is included in version 1.14.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants