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.
This commit is contained in:
parent
6cd0ed3530
commit
f1bfee1be3
|
|
@ -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
|
|
||||||
|
|
@ -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
|
|
||||||
|
|
@ -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
|
|
||||||
|
|
@ -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"
|
|
||||||
|
|
@ -19,5 +19,3 @@ library: "{shared_path}/excalidraw-library.json"
|
||||||
|
|
||||||
default_output_file: "{output_folder}/excalidraw-diagrams/diagram-{timestamp}.excalidraw"
|
default_output_file: "{output_folder}/excalidraw-diagrams/diagram-{timestamp}.excalidraw"
|
||||||
|
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
|
||||||
|
|
@ -21,5 +21,3 @@ planning_artifacts: "{config_source}:planning_artifacts"
|
||||||
implementation_artifacts: "{config_source}:implementation_artifacts"
|
implementation_artifacts: "{config_source}:implementation_artifacts"
|
||||||
default_output_file: "{output_folder}/test-output.md"
|
default_output_file: "{output_folder}/test-output.md"
|
||||||
|
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
|
||||||
|
|
@ -19,5 +19,3 @@ sprint_status: "{implementation_artifacts}/sprint-status.yaml"
|
||||||
project_context: "**/project-context.md"
|
project_context: "**/project-context.md"
|
||||||
implementation_artifacts: "{config_source}:implementation_artifacts"
|
implementation_artifacts: "{config_source}:implementation_artifacts"
|
||||||
|
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
|
||||||
|
|
@ -3,5 +3,3 @@
|
||||||
name: minimal-test
|
name: minimal-test
|
||||||
description: "A minimal workflow with only required fields"
|
description: "A minimal workflow with only required fields"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
|
||||||
|
|
@ -3,6 +3,4 @@
|
||||||
name: template-false-test
|
name: template-false-test
|
||||||
description: "A workflow with template set to false"
|
description: "A workflow with template set to false"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
template: false
|
template: false
|
||||||
|
|
|
||||||
|
|
@ -3,6 +3,4 @@
|
||||||
name: template-path-test
|
name: template-path-test
|
||||||
description: "A workflow with template set to a path"
|
description: "A workflow with template set to a path"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
template: "{installed_path}/template.md"
|
template: "{installed_path}/template.md"
|
||||||
|
|
|
||||||
|
|
@ -3,8 +3,6 @@
|
||||||
name: execution-hints-test
|
name: execution-hints-test
|
||||||
description: "A workflow with execution hints"
|
description: "A workflow with execution hints"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
||||||
execution_hints:
|
execution_hints:
|
||||||
interactive: false
|
interactive: false
|
||||||
|
|
|
||||||
|
|
@ -3,8 +3,6 @@
|
||||||
name: input-patterns-test
|
name: input-patterns-test
|
||||||
description: "A workflow with input_file_patterns using all load strategies"
|
description: "A workflow with input_file_patterns using all load strategies"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
||||||
input_file_patterns:
|
input_file_patterns:
|
||||||
epics:
|
epics:
|
||||||
|
|
|
||||||
|
|
@ -3,8 +3,6 @@
|
||||||
name: required-tools-test
|
name: required-tools-test
|
||||||
description: "A workflow with required tools"
|
description: "A workflow with required tools"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
||||||
required_tools:
|
required_tools:
|
||||||
- read_file
|
- read_file
|
||||||
|
|
|
||||||
|
|
@ -3,8 +3,6 @@
|
||||||
name: tags-test
|
name: tags-test
|
||||||
description: "A workflow with tags"
|
description: "A workflow with tags"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
||||||
tags:
|
tags:
|
||||||
- qa
|
- qa
|
||||||
|
|
|
||||||
|
|
@ -3,8 +3,6 @@
|
||||||
name: variables-test
|
name: variables-test
|
||||||
description: "A workflow with variables section"
|
description: "A workflow with variables section"
|
||||||
author: "Test"
|
author: "Test"
|
||||||
standalone: true
|
|
||||||
web_bundle: false
|
|
||||||
|
|
||||||
variables:
|
variables:
|
||||||
project_context: "**/project-context.md"
|
project_context: "**/project-context.md"
|
||||||
|
|
|
||||||
|
|
@ -29,12 +29,10 @@ const executionHintsSchema = z
|
||||||
// Main workflow schema
|
// Main workflow schema
|
||||||
const workflowSchema = z
|
const workflowSchema = z
|
||||||
.object({
|
.object({
|
||||||
// Required fields (all 13 workflow.yaml files have these)
|
// Required fields
|
||||||
name: z.string().min(1),
|
name: z.string().min(1),
|
||||||
description: z.string().min(1),
|
description: z.string().min(1),
|
||||||
author: z.string().min(1),
|
author: z.string().min(1),
|
||||||
standalone: z.boolean(),
|
|
||||||
web_bundle: z.boolean(),
|
|
||||||
|
|
||||||
// Structured optional fields
|
// Structured optional fields
|
||||||
template: z.union([z.string(), z.literal(false)]).optional(),
|
template: z.union([z.string(), z.literal(false)]).optional(),
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue