Skip to content

[WIP] JSON5 is not supported by STJ#2

Draft
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-d4b9bce8-b66f-448a-9ca1-5547c0a8067f
Draft

[WIP] JSON5 is not supported by STJ#2
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-d4b9bce8-b66f-448a-9ca1-5547c0a8067f

Conversation

Copilot AI commented Oct 3, 2025

Copy link
Copy Markdown

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original issue description:

Description

See https://json5.org

Reproduction Steps

Try to parse the kitchen sink example from json5.org:

{
  // comments
  unquoted: 'and you can quote me on that',
  singleQuotes: 'I can use "double quotes" here',
  lineBreaks: "Look, Mom! \
No \\n's!",
  hexadecimal: 0xdecaf,
  leadingDecimalPoint: .8675309, andTrailing: 8675309.,
  positiveSign: +1,
  trailingComma: 'in objects', andIn: ['arrays',],
  "backwardsCompatible": "with JSON",
}

Expected behavior

Parses fine.

Actual behavior

Fails

Regression?

No

Known Workarounds

No response

Configuration

No response

Other information

No response

Back this issue
Back this issue

Original prompt

This section details on the original issue you should resolve

<issue_title>JSON5 is not supported by STJ</issue_title>
<issue_description>### Description

See https://json5.org

Reproduction Steps

Try to parse the kitchen sink example from json5.org:

{
  // comments
  unquoted: 'and you can quote me on that',
  singleQuotes: 'I can use "double quotes" here',
  lineBreaks: "Look, Mom! \
No \\n's!",
  hexadecimal: 0xdecaf,
  leadingDecimalPoint: .8675309, andTrailing: 8675309.,
  positiveSign: +1,
  trailingComma: 'in objects', andIn: ['arrays',],
  "backwardsCompatible": "with JSON",
}

Expected behavior

Parses fine.

Actual behavior

Fails

Regression?

No

Known Workarounds

No response

Configuration

No response

Other information

No response

Back this issue
Back this issue

<agent_instructions>Create a separate implementation at the lowest possible level allowed by STJ extensibility. The new implementation must NOT reply on any APIs that are not public, since we might need to ship this as a separate nuget package.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #1

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@kzu

kzu commented Oct 3, 2025

Copy link
Copy Markdown
Member

where's the work @copilot ?

Copilot AI requested a review from kzu October 5, 2025 04:09
Copilot finished work on behalf of kzu October 5, 2025 04:09
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.

JSON5 is not supported by STJ

2 participants