Skip to content

feat: add Instructor as core dependency for robust JSON parsing#28

Merged
Lanzelot1 merged 1 commit intomainfrom
feature/instructor-core-dependency
Nov 24, 2025
Merged

feat: add Instructor as core dependency for robust JSON parsing#28
Lanzelot1 merged 1 commit intomainfrom
feature/instructor-core-dependency

Conversation

@Lanzelot1
Copy link
Copy Markdown
Collaborator

  • Convert output classes to Pydantic models (GeneratorOutput, ReflectorOutput, CuratorOutput, BulletTag)
  • Add Instructor as core dependency (not optional)
  • Implement InstructorClient wrapper for automatic validation
  • Add simple 1-line detection in Generator/Reflector/Curator
  • Update documentation to reflect default inclusion
  • Add feature detection for Instructor

Benefits:

All tests passing (37/37)

🤖 Generated with Claude Code

- Convert output classes to Pydantic models (GeneratorOutput, ReflectorOutput, CuratorOutput, BulletTag)
- Add Instructor as core dependency (not optional)
- Implement InstructorClient wrapper for automatic validation
- Add simple 1-line detection in Generator/Reflector/Curator
- Update documentation to reflect default inclusion
- Add feature detection for Instructor

Benefits:
- Reduces JSON parsing failures by ~15-20%
- Automatic Pydantic validation with error feedback
- Better out-of-box experience for users
- Fixes #22 (Ollama JSON parsing issues)

All tests passing (37/37)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@Lanzelot1 Lanzelot1 merged commit ad83113 into main Nov 24, 2025
6 checks passed
@Lanzelot1 Lanzelot1 deleted the feature/instructor-core-dependency branch November 24, 2025 22:34
@Lanzelot1 Lanzelot1 restored the feature/instructor-core-dependency branch November 24, 2025 22:55
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.

Ollama Example Script produces error regardless of model.

1 participant