Skip to content

Consolidate deploy-worker into deploy-service template - #9

Merged
mashkovd merged 1 commit into
mainfrom
claude/refactor-component-registration-LtfPs
Feb 19, 2026
Merged

mashkovd merged 1 commit into
mainfrom
claude/refactor-component-registration-LtfPs

Conversation

@mashkovd

Copy link
Copy Markdown
Member

Summary

Merged the separate deploy-worker template into the deploy-service template, allowing a single template to handle both web services and background workers based on user input.

Key Changes

  • Removed the dedicated deploy-worker template and its skeleton files
  • Updated deploy-service template to support both service types:
    • Made "Network Configuration" section optional instead of required
    • Added conditional logic to set service_type based on whether a host is provided (base-service for web services, worker-service for workers)
    • Updated description to clarify that the host field is optional for background workers
    • Added "worker" tag to the template
  • Simplified user experience by consolidating two similar workflows into one flexible template

Implementation Details

  • The service_type workflow input now uses a conditional expression: ${{ 'base-service' if parameters.host else 'worker-service' }}
  • Port and host fields remain optional, allowing users to deploy workers without ingress configuration
  • All environment variable and secret handling remains consistent between service types

https://claude.ai/code/session_015MSdbW3Yq2d7zYBjZyQQPt

- Unified deploy-service and deploy-worker templates into a single template
- Ingress host/port are now optional: providing a host deploys as a web
  service (base-service), leaving it empty deploys as a background worker
- service_type is inferred automatically from the host parameter
- Remove the separate deploy-worker template directory

https://claude.ai/code/session_015MSdbW3Yq2d7zYBjZyQQPt
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