Skip to content

Tier D: library-edge programs for utf8.codes/offset and math.fmod/ult/string.reverse (#61, #62) - #68

Merged
InauguralPhysicist merged 4 commits into
mainfrom
cloud-61
Oct 28, 2026
Merged

InauguralPhysicist merged 4 commits into
mainfrom
cloud-61

Conversation

@InauguralPhysicist

Copy link
Copy Markdown
Member

Closes #61
Closes #62

  • corpus/D/utf8_codes_offset.lua covers negative and out-of-range i, n = 0, continuation-byte starts, invalid sequences under pcall, and lax. It MATCHes reference Lua 5.4.9.
  • corpus/D/math_fmod_ult_reverse.lua covers math.fmod (negative operands, integer and float zero divisors, mixed), math.ult across the sign boundary, and string.reverse of empty and multibyte strings. It diverges only within the existing EigenScript gap classes: non-finite values, the float subtype, 64-bit integers and NUL bytes. Each diverging line is itemized in LEDGER.md. The program was not bent to avoid them.
  • Tier D after the additions: SUMMARY S=0/0 D=146/159 fail=0 diverge=13 hang=0 crash=0 unsupported=0 examined=159.

LEDGER.md is merged with round 15's refresh (#66): the summary paragraph and the Counts table now show 28 EigenScript-gap rows (S 15, D 13) and 148 PASS/MATCH (S 2, D 146).

Built by Codex Cloud task task_e_6ae23c7f94e8832d8594714ee3b25674. Its diff conflicted with #66 in LEDGER.md and was applied by hand.

🤖 Generated with Claude Code

https://claude.ai/code/session_01KC99CmwatKssQYggkBCgwF

InauguralPhysicist and others added 2 commits October 28, 2026 14:15
…/string.reverse (#61, #62)

utf8_codes_offset.lua MATCHes the reference; math_fmod_ult_reverse.lua
diverges only within the existing EigenScript gap classes (non-finite values,
the float subtype, 64-bit integers, NUL bytes), each line itemized in
LEDGER.md. Tier D after the additions: D=146/159, diverge=13. LEDGER's
summary and Counts table are merged with round 15's refresh (#66): 28
EigenScript-gap rows, 148 PASS/MATCH.

Built by Codex Cloud task task_e_6ae23c7f94e8832d8594714ee3b25674, applied
by hand (its LEDGER.md diff conflicted with #66).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KC99CmwatKssQYggkBCgwF
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-28T19:16:46.447125Z cc6397d PR opened
🔒 Security Review ✅ Completed 2026-10-28T19:17:09.416670Z cc6397d PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cc6397d18e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread corpus/D.manifest
Comment on lines +159 to +160
utf8_codes_offset.lua unspecified-audit: ok
math_fmod_ult_reverse.lua unspecified-audit: ok

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Update the parser test's expected corpus population

Adding these two programs increases Tier D from 157 to 159 files, but tests/parse_all.sh:22 still requires exactly 157. With the expected EigenScript executable available, the test now exits with HARNESS: population D=159 S=23, want 157 and 23 before parsing any files. Update the population guard alongside these additions so the parser regression suite remains runnable.

Useful? React with 👍 / 👎.

@InauguralPhysicist
InauguralPhysicist merged commit ac3cbee into main Oct 28, 2026
1 check passed
@InauguralPhysicist
InauguralPhysicist deleted the cloud-61 branch October 28, 2026 19:48
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.

Tier D: cover math.fmod, math.ult and string.reverse edges Tier D: cover utf8.codes / utf8.offset edge cases

1 participant