Skip to content

Fix swapped schedule date/time input model filenames #547

Description

@jjoonleo

Problem

The schedule date/time input model files appear swapped: the date-named file exports the time input model, and the time-named file exports the date input model. This makes single-file imports misleading and fragile.

Evidence

  • lib/presentation/schedule_create/schedule_date_time/input_models/schedule_date_input_model.dart defines ScheduleTimeValidationError and ScheduleTimeInputModel.
  • lib/presentation/schedule_create/schedule_date_time/input_models/schedule_time_input_model.dart defines ScheduleDateValidationError and ScheduleDateInputModel.
  • Existing schedule date/time consumers and tests import both files together, so current targeted tests can still pass while the file contract is misleading.

Proposed direction

Rename/move the model definitions so each file exports the class matching its filename, then update imports if needed. Add direct input model tests that import each file independently.

Acceptance criteria

  • schedule_date_input_model.dart defines ScheduleDateInputModel and ScheduleDateValidationError.
  • schedule_time_input_model.dart defines ScheduleTimeInputModel and ScheduleTimeValidationError.
  • Existing schedule date/time cubit and state tests pass.
  • New or updated tests prove each input model can be imported and validated independently from its matching file.

Source: Codex codebase audit on 2026-06-28.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingcodex-readyCan be started independently by a Codex thread nowschedulerelated to schedule featuretest

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions