Skip to content

fix: preserve struct definition schemas - #28

Merged
tiye merged 1 commit into
mainfrom
codex/fix-struct-schemas-028
Aug 22, 2026
Merged

fix: preserve struct definition schemas#28
tiye merged 1 commit into
mainfrom
codex/fix-struct-schemas-028

Conversation

@tiye

@tiye tiye commented Aug 22, 2026

Copy link
Copy Markdown
Member

Follow-up to #27 addressing the review comment about defstruct metadata. Restores :: 'StructDef for Site, ChatMessage, ChatSession, ChatState, MessageBoxState, and Store in the canonical snapshot.\n\nValidation: calcit docs agents --full; calcit calcit.cirru --check-only. The explicit metadata is applied after formatting because the current formatter rewrites these markers to Enum; this PR preserves the review-correct metadata.

Copilot AI lite review requested due to automatic review settings August 22, 2026 18:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Restores StructDef metadata for six struct definitions in the canonical Calcit snapshot.

Changes:

  • Corrects schemas for Site and application state/message structs.
  • Preserves metadata after formatting.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@tiye
tiye merged commit eca18e1 into main Aug 22, 2026
2 checks passed
@tiye
tiye deleted the codex/fix-struct-schemas-028 branch August 22, 2026 18:42
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