Skip to content

Align provider catalog with workflow-compute#1

Merged
intel352 merged 1 commit into
mainfrom
codex/compute-provider-alignment
May 20, 2026
Merged

Align provider catalog with workflow-compute#1
intel352 merged 1 commit into
mainfrom
codex/compute-provider-alignment

Conversation

@intel352
Copy link
Copy Markdown
Contributor

Summary

  • align compute.provider_catalog with workflow-compute's typed protocol.ProviderContract
  • expose the provider catalog module in runtime schemas and plugin.json
  • add a PR alignment check that checks out GoCodeAlone/workflow-compute and runs catalog tests with a local module replace

Drift Found

The plugin had started carrying grouped provider details (executor_providers, dependency_providers, verification_providers, reward_providers, network_providers) while workflow-compute now defines provider catalog entries as ProviderContract records. The module was also absent from plugin runtime registration and plugin.json.

Verification

  • GOWORK=off GOPRIVATE=github.com/GoCodeAlone/* GONOSUMDB=github.com/GoCodeAlone/* go test ./...
  • GOPRIVATE=github.com/GoCodeAlone/* GONOSUMDB=github.com/GoCodeAlone/* scripts/check-workflow-compute-alignment.sh /Users/jon/workspace/workflow-compute-salvage-audit
  • wfctl validate workflow.yaml
  • GOWORK=off wfctl build --config workflow.yaml --no-push --tag local
  • git diff --check

TDD Proof

With fix missing:

$ GOWORK=off go test ./internal -run 'Test(ModuleTypes|PluginManifestModuleTypesMatchRuntime|ProviderCatalog)' -count=1
FAIL: module.config.Contracts undefined; protocol.ProviderContract undefined on old workflow-compute pin; module types omitted compute.provider_catalog

With fix restored:

$ GOWORK=off GOPRIVATE=github.com/GoCodeAlone/* GONOSUMDB=github.com/GoCodeAlone/* go test ./internal -run 'Test(ModuleTypes|PluginManifestModuleTypesMatchRuntime|ProviderCatalog)' -count=1
ok  github.com/GoCodeAlone/workflow-plugin-compute/internal  0.671s

Copilot AI review requested due to automatic review settings May 20, 2026 23:38
@intel352 intel352 force-pushed the codex/compute-provider-alignment branch from 01b1b14 to c9c1330 Compare May 20, 2026 23:41
Copy link
Copy Markdown

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

Aligns this plugin’s provider catalog shape and runtime registration with workflow-compute’s typed protocol.ProviderContract, and adds CI coverage to detect catalog drift against workflow-compute main.

Changes:

  • Adds new module type compute.provider_catalog backed by protocol.ProviderContract + validation and tests.
  • Exposes the new module in runtime schemas and plugin.json capabilities.
  • Adds a CI “alignment” job step and a helper script that checks out GoCodeAlone/workflow-compute and runs catalog-focused tests with a local go.mod replace.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
SPEC.md Documents the new provider-catalog alignment constraint and CI drift gate.
scripts/check-workflow-compute-alignment.sh Adds a helper script to run alignment tests against a local workflow-compute checkout.
README.md Documents that provider catalog uses core ProviderContract records.
plugin.json Registers compute.provider_catalog in manifest capabilities and updates description text.
internal/plugin.go Registers/constructs the new module type and exposes its schema.
internal/module.go Implements compute.provider_catalog module + adds provider_catalog_ref to pool config/schema.
internal/module_test.go Adds tests for provider catalog typing/validation and manifest moduleTypes alignment.
go.mod Bumps workflow-compute dependency version (and indirect pgx bump).
go.sum Records new dependency sums.
.github/workflows/build.yml Adds checkout of workflow-compute and runs the alignment script in CI.

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

Comment thread internal/module.go
ConfigFields: []sdk.ConfigField{
{Name: "provider_ref", Type: "string", Description: "Name of the compute.provider module.", Required: true},
{Name: "provider_ref", Type: "string", Description: "Name of the compute.provider control-plane module.", Required: true},
{Name: "provider_catalog_ref", Type: "string", Description: "Optional name of the compute.provider_catalog module that constrains provider policy."},
Comment thread plugin.json
Comment on lines 2 to 5
"name": "workflow-plugin-compute",
"version": "0.1.0",
"description": "Workflow adapter for workflow-compute dispatch, wait, map, provider, and pool integration",
"description": "Workflow adapter for workflow-compute dispatch, wait, map, provider, pool, and provider catalog integration",
"author": "GoCodeAlone",
@intel352 intel352 merged commit e591f7a into main May 20, 2026
4 checks passed
@intel352 intel352 deleted the codex/compute-provider-alignment branch May 20, 2026 23:48
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