Skip to content

feat: add core placement constraint evaluator#18

Merged
intel352 merged 1 commit into
mainfrom
refactor/placement-constraints-core
May 25, 2026
Merged

feat: add core placement constraint evaluator#18
intel352 merged 1 commit into
mainfrom
refactor/placement-constraints-core

Conversation

@intel352
Copy link
Copy Markdown
Contributor

Summary

  • Add compute-core placement input DTOs for worker capabilities and task network policy.
  • Move placement constraint satisfaction semantics into compute-core without importing workflow-compute's richer worker/task models.

Verification

  • RED: GOWORK=off go test ./protocol -run TestPlacementConstraintsSatisfiedByEvaluatesWorkerAndTaskInputs -count=1 failed before implementation with undefined core API symbols.
  • GOWORK=off go test ./protocol -run TestPlacementConstraintsSatisfiedByEvaluatesWorkerAndTaskInputs -count=1
  • git diff --check
  • GOWORK=off go test ./... -count=1

Adversarial Review

  • Avoided pulling workflow-compute-local Capabilities or NetworkPolicy into core; the core API takes minimal placement DTOs.
  • Kept existing error messages and zero-constraint semantics so downstream behavior can delegate without user-visible drift.
  • No fallback implementation is introduced downstream; workflow-compute can reduce to an adapter shim.

@intel352 intel352 merged commit 48be3c0 into main May 25, 2026
4 checks passed
@intel352 intel352 deleted the refactor/placement-constraints-core branch May 25, 2026 07:31
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