Skip to content

Conversation

@tarcisiozf
Copy link

TICKET: DEVSVCS-2981

Refactoring and modularization:

  • Replaces the internal workflowArtifact struct with the new shared artifact.Artifact type throughout the deploy and register code, and updates test code accordingly.
  • Refactors the workflow compilation logic to use the new build package for resolving build parameters and compiling workflows, removing duplicated code and simplifying the Compile method.
  • Refactors artifact preparation to use the new artifact.Builder and artifact.Inputs, replacing custom logic for reading and encoding binaries and configs.

New features:

  • Adds a new workflow id subcommand that builds the workflow and prints its computed ID, using the shared artifact and build logic. (cmd/workflow/id/id.go)

General improvements:

  • Fix potential memory leak by closing brotli.NewWriter
  • Improve error messages when Anvil fails

@tarcisiozf tarcisiozf requested a review from a team as a code owner December 16, 2025 14:33
@tarcisiozf tarcisiozf marked this pull request as draft December 16, 2025 14:47
@tarcisiozf tarcisiozf force-pushed the DEVSVCS-2981/workflow-id branch from 49f16bf to 0deba2b Compare December 18, 2025 12:03
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