Skip to content

test(contracts): add domain APIs schema coverage - #227

Merged
h4yfans merged 1 commit into
mainfrom
debt/phase-4-u6-contracts-domain-apis
Apr 16, 2026
Merged

test(contracts): add domain APIs schema coverage#227
h4yfans merged 1 commit into
mainfrom
debt/phase-4-u6-contracts-domain-apis

Conversation

@h4yfans

@h4yfans h4yfans commented Apr 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Phase 4 U6 — adds Zod schema validation tests for 9 previously-untested contracts files:

  • calendar-api, graph-api, linking-api, search-api, properties-api
  • bookmark-types, reminder-types, property-types, ai-inline-channels

Tests use safeParse to exercise valid + invalid inputs per schema.

Test plan

  • 9 test files created, all under 800-line pre-commit limit
  • CI: pnpm --dir apps/desktop test:coverage green

Ref: .claude/plans/tech-debt-remediation.md Phase 4.4

Phase 4 U6 — adds Zod schema validation tests for 9 previously-untested
contract files in the domain APIs cluster:
- calendar-api, graph-api, linking-api, search-api
- properties-api, bookmark-types, reminder-types, property-types
- ai-inline-channels

Each test covers valid + invalid inputs via safeParse.

Ref: .claude/plans/tech-debt-remediation.md Phase 4.4 (contracts domain cluster)
@h4yfans
h4yfans merged commit 2e8b81a into main Apr 16, 2026
4 checks passed
@h4yfans
h4yfans deleted the debt/phase-4-u6-contracts-domain-apis branch April 16, 2026 14:05
h4yfans added a commit that referenced this pull request May 6, 2026
Phase 4 U6 — adds Zod schema validation tests for 9 previously-untested
contract files in the domain APIs cluster:
- calendar-api, graph-api, linking-api, search-api
- properties-api, bookmark-types, reminder-types, property-types
- ai-inline-channels

Each test covers valid + invalid inputs via safeParse.

Ref: .claude/plans/tech-debt-remediation.md Phase 4.4 (contracts domain cluster)
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