From f1bfee1be3391db40b7be0961ce8154dc5cede54 Mon Sep 17 00:00:00 2001 From: Michael Pursifull Date: Thu, 5 Feb 2026 09:42:39 -0600 Subject: [PATCH] fix: remove standalone and web_bundle from workflow schema The maintainer (Brian Madison) removed these fields from all workflow files in two commits: - f699a368: removed `standalone: true` from all workflow.yaml files - 731bee26: removed `web_bundle: false` from all workflow.yaml files These fields are no longer part of the workflow.yaml format. Update the schema to match upstream and remove related test fixtures. 24/24 tests passing. 9/9 workflow files valid. --- .../invalid/missing-standalone.workflow.yaml | 8 -------- .../invalid/missing-web-bundle.workflow.yaml | 8 -------- .../invalid/wrong-type-standalone.workflow.yaml | 11 ----------- .../invalid/wrong-type-web-bundle.workflow.yaml | 11 ----------- .../valid/excalidraw-style.workflow.yaml | 2 -- .../workflow-schema/valid/full-config.workflow.yaml | 2 -- .../valid/implementation-style.workflow.yaml | 2 -- .../workflow-schema/valid/minimal.workflow.yaml | 2 -- .../valid/template-false.workflow.yaml | 2 -- .../workflow-schema/valid/template-path.workflow.yaml | 2 -- .../valid/with-execution-hints.workflow.yaml | 2 -- .../valid/with-input-patterns.workflow.yaml | 2 -- .../valid/with-required-tools.workflow.yaml | 2 -- .../workflow-schema/valid/with-tags.workflow.yaml | 2 -- .../valid/with-variables.workflow.yaml | 2 -- tools/schema/workflow.js | 4 +--- 16 files changed, 1 insertion(+), 63 deletions(-) delete mode 100644 test/fixtures/workflow-schema/invalid/missing-standalone.workflow.yaml delete mode 100644 test/fixtures/workflow-schema/invalid/missing-web-bundle.workflow.yaml delete mode 100644 test/fixtures/workflow-schema/invalid/wrong-type-standalone.workflow.yaml delete mode 100644 test/fixtures/workflow-schema/invalid/wrong-type-web-bundle.workflow.yaml diff --git a/test/fixtures/workflow-schema/invalid/missing-standalone.workflow.yaml b/test/fixtures/workflow-schema/invalid/missing-standalone.workflow.yaml deleted file mode 100644 index ba518c42c..000000000 --- a/test/fixtures/workflow-schema/invalid/missing-standalone.workflow.yaml +++ /dev/null @@ -1,8 +0,0 @@ -# Expected: FAIL -# Error code: invalid_type -# Error path: standalone -# Tests: AC4 — missing standalone produces error -name: missing-standalone-test -description: "A workflow missing the standalone field" -author: "Test" -web_bundle: false diff --git a/test/fixtures/workflow-schema/invalid/missing-web-bundle.workflow.yaml b/test/fixtures/workflow-schema/invalid/missing-web-bundle.workflow.yaml deleted file mode 100644 index f43bb886f..000000000 --- a/test/fixtures/workflow-schema/invalid/missing-web-bundle.workflow.yaml +++ /dev/null @@ -1,8 +0,0 @@ -# Expected: FAIL -# Error code: invalid_type -# Error path: web_bundle -# Tests: AC5 — missing web_bundle produces error -name: missing-web-bundle-test -description: "A workflow missing the web_bundle field" -author: "Test" -standalone: true diff --git a/test/fixtures/workflow-schema/invalid/wrong-type-standalone.workflow.yaml b/test/fixtures/workflow-schema/invalid/wrong-type-standalone.workflow.yaml deleted file mode 100644 index bdba5c392..000000000 --- a/test/fixtures/workflow-schema/invalid/wrong-type-standalone.workflow.yaml +++ /dev/null @@ -1,11 +0,0 @@ -# Expected: FAIL -# Error code: invalid_type -# Error path: standalone -# Error expected: boolean -# Error received: string -# Tests: AC8 — standalone with non-boolean value produces wrong_type error -name: wrong-type-standalone-test -description: "A workflow with wrong type for standalone" -author: "Test" -standalone: "yes" -web_bundle: false diff --git a/test/fixtures/workflow-schema/invalid/wrong-type-web-bundle.workflow.yaml b/test/fixtures/workflow-schema/invalid/wrong-type-web-bundle.workflow.yaml deleted file mode 100644 index 01fa183df..000000000 --- a/test/fixtures/workflow-schema/invalid/wrong-type-web-bundle.workflow.yaml +++ /dev/null @@ -1,11 +0,0 @@ -# Expected: FAIL -# Error code: invalid_type -# Error path: web_bundle -# Error expected: boolean -# Error received: string -# Tests: AC9 — web_bundle with non-boolean value produces wrong_type error -name: wrong-type-web-bundle-test -description: "A workflow with wrong type for web_bundle" -author: "Test" -standalone: true -web_bundle: "true" diff --git a/test/fixtures/workflow-schema/valid/excalidraw-style.workflow.yaml b/test/fixtures/workflow-schema/valid/excalidraw-style.workflow.yaml index cb5e557ac..738c3ecf7 100644 --- a/test/fixtures/workflow-schema/valid/excalidraw-style.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/excalidraw-style.workflow.yaml @@ -19,5 +19,3 @@ library: "{shared_path}/excalidraw-library.json" default_output_file: "{output_folder}/excalidraw-diagrams/diagram-{timestamp}.excalidraw" -standalone: true -web_bundle: false diff --git a/test/fixtures/workflow-schema/valid/full-config.workflow.yaml b/test/fixtures/workflow-schema/valid/full-config.workflow.yaml index ad8f16a77..4c3582c24 100644 --- a/test/fixtures/workflow-schema/valid/full-config.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/full-config.workflow.yaml @@ -21,5 +21,3 @@ planning_artifacts: "{config_source}:planning_artifacts" implementation_artifacts: "{config_source}:implementation_artifacts" default_output_file: "{output_folder}/test-output.md" -standalone: true -web_bundle: false diff --git a/test/fixtures/workflow-schema/valid/implementation-style.workflow.yaml b/test/fixtures/workflow-schema/valid/implementation-style.workflow.yaml index 9bf6518cf..36157aa6a 100644 --- a/test/fixtures/workflow-schema/valid/implementation-style.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/implementation-style.workflow.yaml @@ -19,5 +19,3 @@ sprint_status: "{implementation_artifacts}/sprint-status.yaml" project_context: "**/project-context.md" implementation_artifacts: "{config_source}:implementation_artifacts" -standalone: true -web_bundle: false diff --git a/test/fixtures/workflow-schema/valid/minimal.workflow.yaml b/test/fixtures/workflow-schema/valid/minimal.workflow.yaml index 86d56f6c9..22448fa69 100644 --- a/test/fixtures/workflow-schema/valid/minimal.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/minimal.workflow.yaml @@ -3,5 +3,3 @@ name: minimal-test description: "A minimal workflow with only required fields" author: "Test" -standalone: true -web_bundle: false diff --git a/test/fixtures/workflow-schema/valid/template-false.workflow.yaml b/test/fixtures/workflow-schema/valid/template-false.workflow.yaml index dd6b5bc89..8e8dea46d 100644 --- a/test/fixtures/workflow-schema/valid/template-false.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/template-false.workflow.yaml @@ -3,6 +3,4 @@ name: template-false-test description: "A workflow with template set to false" author: "Test" -standalone: true -web_bundle: false template: false diff --git a/test/fixtures/workflow-schema/valid/template-path.workflow.yaml b/test/fixtures/workflow-schema/valid/template-path.workflow.yaml index 844df3b21..90840c249 100644 --- a/test/fixtures/workflow-schema/valid/template-path.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/template-path.workflow.yaml @@ -3,6 +3,4 @@ name: template-path-test description: "A workflow with template set to a path" author: "Test" -standalone: true -web_bundle: false template: "{installed_path}/template.md" diff --git a/test/fixtures/workflow-schema/valid/with-execution-hints.workflow.yaml b/test/fixtures/workflow-schema/valid/with-execution-hints.workflow.yaml index e23c37772..61d0d13ab 100644 --- a/test/fixtures/workflow-schema/valid/with-execution-hints.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/with-execution-hints.workflow.yaml @@ -3,8 +3,6 @@ name: execution-hints-test description: "A workflow with execution hints" author: "Test" -standalone: true -web_bundle: false execution_hints: interactive: false diff --git a/test/fixtures/workflow-schema/valid/with-input-patterns.workflow.yaml b/test/fixtures/workflow-schema/valid/with-input-patterns.workflow.yaml index a9da08c7c..e3f5b9a19 100644 --- a/test/fixtures/workflow-schema/valid/with-input-patterns.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/with-input-patterns.workflow.yaml @@ -3,8 +3,6 @@ name: input-patterns-test description: "A workflow with input_file_patterns using all load strategies" author: "Test" -standalone: true -web_bundle: false input_file_patterns: epics: diff --git a/test/fixtures/workflow-schema/valid/with-required-tools.workflow.yaml b/test/fixtures/workflow-schema/valid/with-required-tools.workflow.yaml index 36dda8afb..620db94db 100644 --- a/test/fixtures/workflow-schema/valid/with-required-tools.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/with-required-tools.workflow.yaml @@ -3,8 +3,6 @@ name: required-tools-test description: "A workflow with required tools" author: "Test" -standalone: true -web_bundle: false required_tools: - read_file diff --git a/test/fixtures/workflow-schema/valid/with-tags.workflow.yaml b/test/fixtures/workflow-schema/valid/with-tags.workflow.yaml index d53900a7f..26216e0a4 100644 --- a/test/fixtures/workflow-schema/valid/with-tags.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/with-tags.workflow.yaml @@ -3,8 +3,6 @@ name: tags-test description: "A workflow with tags" author: "Test" -standalone: true -web_bundle: false tags: - qa diff --git a/test/fixtures/workflow-schema/valid/with-variables.workflow.yaml b/test/fixtures/workflow-schema/valid/with-variables.workflow.yaml index 31583882f..530632689 100644 --- a/test/fixtures/workflow-schema/valid/with-variables.workflow.yaml +++ b/test/fixtures/workflow-schema/valid/with-variables.workflow.yaml @@ -3,8 +3,6 @@ name: variables-test description: "A workflow with variables section" author: "Test" -standalone: true -web_bundle: false variables: project_context: "**/project-context.md" diff --git a/tools/schema/workflow.js b/tools/schema/workflow.js index 80b963084..14ed83f84 100644 --- a/tools/schema/workflow.js +++ b/tools/schema/workflow.js @@ -29,12 +29,10 @@ const executionHintsSchema = z // Main workflow schema const workflowSchema = z .object({ - // Required fields (all 13 workflow.yaml files have these) + // Required fields name: z.string().min(1), description: z.string().min(1), author: z.string().min(1), - standalone: z.boolean(), - web_bundle: z.boolean(), // Structured optional fields template: z.union([z.string(), z.literal(false)]).optional(),