Skip to content

Check the validity of the JSONL file after it is created#292

Merged
dsyme merged 8 commits intomainfrom
def2
Sep 3, 2025
Merged

Check the validity of the JSONL file after it is created#292
dsyme merged 8 commits intomainfrom
def2

Conversation

@dsyme
Copy link
Copy Markdown
Collaborator

@dsyme dsyme commented Sep 3, 2025

Annoyingly the model wrote invalid content to the JSONL file, horking everything, an invalid control character, likely emoji related

{"type": "create-pull-request", "branch": "claude-test-pr", "title": "A Pull Request from Claude", "body": "World\n\n## A Haiku About GitHub Actions and AI\n\nCode flows like rivers,  \nAI agents merge and build�  \nAutomation dreams."}

@pelikhan
Copy link
Copy Markdown
Collaborator

pelikhan commented Sep 3, 2025

@dsyme there is a port of jsonrepair for go that handles a lot of little idiosyncrasies of llms

https://github.com/kaptinlin/jsonrepair

@pelikhan
Copy link
Copy Markdown
Collaborator

pelikhan commented Sep 3, 2025

There is a Boolean expression tree builder/renderer in expressions.go that we can use to create and/or...

Eventually, I would like to support multiline expressions with comments so that we can "read" those and understand them in the compiled code. Also leads to better diffs

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