Skip to content

Void control orchestrator#28

Open
dpsoft wants to merge 2 commits intomainfrom
feat/void-control-orchestrator
Open

Void control orchestrator#28
dpsoft wants to merge 2 commits intomainfrom
feat/void-control-orchestrator

Conversation

@dpsoft
Copy link
Contributor

@dpsoft dpsoft commented Mar 22, 2026

This pull request introduces several enhancements centered around artifact management, error handling, and improved support for mock sandboxes. The main changes include new APIs and data structures for handling named artifacts, updates to error codes to reflect new artifact-related cases, and improved test coverage. Additionally, the mock sandbox has been extended to better simulate file operations.

Artifact Management Enhancements:

  • Introduced new data structures (ArtifactPublicationStatus, ArtifactManifestEntry, ArtifactPublication, StageState) and extended the RunState struct to track artifact publication status, manifest, and per-stage state. [1] [2]
  • Added new trait methods to PersistenceProvider for saving, loading, and listing named artifacts, and implemented these for DiskPersistenceProvider with validation and directory management. [1] [2]
  • Added a helper (persist_workflow_artifacts) to extract and persist workflow artifacts, including support for artifacts listed in a result.json file. [1] [2]

Error Handling Improvements:

  • Added new error codes to ApiErrorCode for artifact and structured output scenarios, and provided corresponding constructors in ApiError. [1] [2]

Mock Sandbox Improvements:

  • Extended MockSandbox to support file storage and simulate file existence and reading for commands like cat and test, and added support for running shell scripts. [1] [2] [3] [4]

Test Coverage:

  • Added tests for named artifact round-trip, missing artifact handling, artifact listing, and round-trip persistence of RunState with artifact publication details. [1] [2]

Runtime and Workflow Integration:

  • Updated runtime and workflow execution paths to support artifact persistence and to properly handle mock sandboxes by skipping guest/OCI image resolution when appropriate. [1] [2] [3] [4]

These changes collectively improve artifact handling, error reporting, and testability, while also making the system more robust and extensible for future artifact-related features.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dpsoft dpsoft force-pushed the feat/void-control-orchestrator branch from fc4628f to 385b2ac Compare March 22, 2026 14:48
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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