Skip to content

Add typed parsers - #454

Draft
GlassOfWhiskey wants to merge 1 commit into
mainfrom
typed-parsers-2
Draft

Add typed parsers#454
GlassOfWhiskey wants to merge 1 commit into
mainfrom
typed-parsers-2

Conversation

@GlassOfWhiskey

Copy link
Copy Markdown
Collaborator

Regenerate the v1.0, v1.1, and v1.2 CWL parsers using the latest Schema SALAD version, which adds type annotations to classes, loaders, and fields (FieldType, EnumFieldType, generic Loader[FieldType], etc.).

Adjust all downstream machinery to resolve the resulting type errors:

  • Update type aliases in cwl_utils.parser (PrimitiveType, InputParameterType, OutputParameterType, CommandInputParameterType, CommandOutputParameterType, EnvironmentDef, CWLVersion, CWLTypeSchema, ParameterType, ParameterTypeOrArraySchema)
  • Harden cwl_utils.types with is_sequence() and the SrcSink namedtuple
  • Fix expression refactorors, inputs_schema_gen (anonymous array schemas, Sequence doc values, SchemaDefRequirement cast), docker/cite extraction, and cwl_utils.utils
  • Adjust tests according to the new typing utilities

@GlassOfWhiskey
GlassOfWhiskey force-pushed the typed-parsers-2 branch 8 times, most recently from 17f7622 to c30b47e Compare August 17, 2026 13:31
Regenerate the v1.0, v1.1, and v1.2 CWL parsers using the latest
Schema SALAD version, which adds type annotations to classes, loaders,
and fields (`FieldType`, `EnumFieldType`, generic `Loader[FieldType]`,
etc.).

Adjust all downstream machinery to resolve the resulting type errors:

- Update type aliases in `cwl_utils.parser` (`PrimitiveType`,
  `InputParameterType`, `OutputParameterType`,
  `CommandInputParameterType`, `CommandOutputParameterType`,
  `EnvironmentDef`, `CWLVersion`, `CWLTypeSchema`, `ParameterType`,
  `ParameterTypeOrArraySchema`)
- Harden `cwl_utils.types` with `is_sequence()` and the `SrcSink`
  namedtuple
- Fix expression refactorors, `inputs_schema_gen` (anonymous array
  schemas, `Sequence` doc values, `SchemaDefRequirement` cast),
  docker/cite extraction, and `cwl_utils.utils`
- Adjust tests according to the new typing utilities
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