uncaged/json-cas
Archived
10
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: clean error message for invalid schema in schema put command #61

Merged
xiaomo merged 1 commits from fix/54-schema-put-invalid-error into main 2026-05-31 09:15:38 +00:00
Conversation 1 Commits 1 Files Changed 2
+144 -2
xiaoju commented 2026-05-31 08:58:32 +00:00
Owner
Copy Link
Copy Source

Fixes #54

Fixes #54
xiaoju added 1 commit 2026-05-31 08:58:33 +00:00
fix: clean error message for invalid schema in schema put command 3c8b16d7b1 
Fixes #54

The `json-cas schema put` command now catches SchemaValidationError
and displays a clean error message instead of showing a raw stack trace.

Changes:
- Import SchemaValidationError from @uncaged/json-cas
- Wrap putSchema() call in try-catch in cmdSchemaPut
- Catch SchemaValidationError specifically and call die(e.message)
- Add 5 comprehensive tests for invalid schema error handling

Test cases:
1. Invalid JSON Schema type value
2. Unknown schema keys
3. Invalid nested schema
4. Non-object root schema
5. Valid schema regression test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
xiaomo approved these changes 2026-05-31 09:15:20 +00:00
xiaomo left a comment
Owner

LGTM ✅

LGTM ✅
xiaomo merged commit 10c5c8f98e into main 2026-05-31 09:15:38 +00:00
xiaomo deleted branch fix/54-schema-put-invalid-error 2026-05-31 09:15:38 +00:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No Reviewers
xiaomo
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aobing jiashuang (Jia Shuang) jiayi (Jiayi) luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
2 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/json-cas#61
Delete Branch "fix/54-schema-put-invalid-error"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?