Skip to content

better json schema errors - #178

Closed
pelikhan wants to merge 13 commits into
github:mainfrom
githubnext:main
Closed

better json schema errors#178
pelikhan wants to merge 13 commits into
github:mainfrom
githubnext:main

Conversation

@pelikhan

Copy link
Copy Markdown
Collaborator

Doing some magic to locate errors from json schema back into the source.

CopilotAIand others added 12 commits August 21, 2025 03:55
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Add internal/mapper package with MapErrorToSpans API
- RFC6901 JSON Pointer decoding with proper escaping
- goccy/go-yaml AST traversal with position metadata
- Error-kind specific mapping (type, required, additionalProperties)
- Fallback heuristics for missing properties and out-of-range indices
- Confidence scoring system (0.2-1.0 range)
- Comprehensive test suite with 100% coverage
- CLI test runner for interactive debugging
- Full documentation with algorithm details
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…e locations
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…n.go
- Extract all JSON schema validation related functions to dedicated file
- Move httpURLLoader, validateWorkflowSchema*, formatSchemaValidationError, extractErrorMeta, formatValidationMessage, generateValidationHint, and extractContextLines
- Remove unused imports from compiler.go (net/http, jsonschema, mapper)
- Maintain all existing functionality and method signatures
- All tests continue to pass with the refactored code structure
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…af62-a58980a2e9e1
Add draft boolean option to output.pull-request feature
…g integration
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…b73e-fe64be5b3b8a
Add comprehensive JSON Schema validation test suite with error mapping integration
@github-actionsgithub-actionsBot mentioned this pull request Jun 17, 2026
Sign up for freeto 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

@pelikhan