Skip to content

Keep mini-benchmark JSON artifacts Prettier-clean - #4

Merged
yifanxuaaa merged 4 commits into
Ephemeral-AI-Lab:mainfrom
runyuan-wang:fix/mini-benchmark-prettier
Aug 13, 2026
Merged

Keep mini-benchmark JSON artifacts Prettier-clean#4
yifanxuaaa merged 4 commits into
Ephemeral-AI-Lab:mainfrom
runyuan-wang:fix/mini-benchmark-prettier

Conversation

@runyuan-wang

Copy link
Copy Markdown
Contributor

Summary

  • format mini-benchmark JSON artifacts with the repository's resolved Prettier configuration before writing them
  • update the two currently drifted A5-one-byte-edit.json artifacts to the generator's canonical output

Why

tests/performance/mini-bench.mjs previously wrote JSON with JSON.stringify(..., null, 2). That output is valid JSON, but it does not always match this repository's Prettier formatting, so regenerating the artifacts can leave the style gate red.

This keeps formatting at the generation boundary rather than relying on a later manual cleanup. The two artifact changes are whitespace-only: their parsed keys, values, numbers, array order, and object order are unchanged.

Validation

  • node --check tests/performance/mini-bench.mjs
  • Prettier check passes for the generator and both changed artifacts
  • a real A-group generation produced 8/8 parseable artifacts, all Prettier-clean and byte-idempotent after a second formatting pass
  • repository style, typecheck, build, exports, API snapshot, and accepted pre-evidence suites passed in the isolated validation environment
  • the full test command still reports three Cloudflare-driver failures, but the same three failures reproduce on the clean base commit; they are not introduced by this patch

Telegram: @wangrunyuansecbot | Nickname: 一心
Powered by LingTai AI: https://github.com/Lingtai-AI/lingtai

yifanxuaaa added a commit that referenced this pull request Aug 13, 2026
@yifanxuaaa
yifanxuaaa merged commit 71b45a2 into Ephemeral-AI-Lab:main Aug 13, 2026
2 of 4 checks passed
yifanxuaaa added a commit that referenced this pull request Aug 13, 2026
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