Skip to content

fix yaml temporal imports#46

Merged
murrayju merged 1 commit intomainfrom
codex/fix-yaml-import-temporal
Apr 24, 2026
Merged

fix yaml temporal imports#46
murrayju merged 1 commit intomainfrom
codex/fix-yaml-import-temporal

Conversation

@murrayju
Copy link
Copy Markdown
Member

Summary

Allows YAML and Markdown imports to accept the {start, end?} object form for temporal fields. Previously this form was only accepted in JSON, which meant YAML and Markdown files produced by me memory export could not be re-imported without manual conversion.

Changes

  • packages/cli/parsers/validation.ts: Removed the format restriction in parseTemporalInput so all import formats accept the object shape (which is the normalized API/export shape).
  • packages/cli/parsers/import.test.ts: New tests covering YAML and Markdown imports of the object-form temporal field, including start-only.
  • docs/formats.md: Updated the temporal input table and examples to reflect that all formats accept the object form, and removed the YAML-specific restriction note.

Motivation

Export → import round-tripping should be lossless. YAML and Markdown exports emit temporal as {start, end?}, so the import path needs to accept that shape.

@murrayju murrayju requested a review from jgpruitt as a code owner April 24, 2026 17:32
@murrayju murrayju merged commit 7894d44 into main Apr 24, 2026
3 checks passed
@murrayju murrayju deleted the codex/fix-yaml-import-temporal branch April 24, 2026 18:45
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.

2 participants