Skip to content

Optional fields should use Effect Option type #61

Open

Description

@dallenpyrah

Problem

File: src/parallel/types.ts

Optional fields use TypeScript optional syntax instead of Effect's Option type:

  • Line 3: mode?: "one-shot" | "agentic" - Could be Option<...>
  • Line 4: objective?: string - Should be Option<string>
  • Line 30: objective?: string - Same issue

Proposed Fix

For consistency with Effect TS patterns, consider using Option<T> for optional fields in domain types.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestlowLow priority

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions