Skip to content

det: JSON GENERATE takes OCCURS DEPENDING ON tables (Part of #7524) - #7932

Merged
squid-shepherd[bot] merged 6 commits into
mainfrom
churn/7524-20261106T1624Z
Nov 6, 2026
Merged

squid-shepherd[bot] merged 6 commits into
mainfrom
churn/7524-20261106T1624Z

Conversation

@squid-protocol

Copy link
Copy Markdown
Owner

Summary

JSON GENERATE now translates a source group holding OCCURS ... DEPENDING ON tables: the table is a JSON array of as many elements as the ODO object holds at run time, [] for none (IBM Enterprise COBOL 6.3 LR, JSON GENERATE: SUPPRESS examples and Operation page). JsonGenerate.table(name, count, low, odo, elements...) is the runtime half; gen.py json_tree emits it. Register X76 updated (ASSUMED, oracle-unbacked: no L2+ claim). No burned program uses JSON; fixture tests only.

Class / Sibling gaps

Class: source shapes json_tree (gen.py) refuses by name for JSON GENERATE. Handled in this PR: OCCURS DEPENDING ON tables (group and elementary, count 0 / mid / max), each with a test. Same mechanism but left refused because IBM's pages do not state them: an ODO count outside the clause's bounds (runtime Hole, undefined by IBM, tested) and an ODO table inside another table (refused by name, tested).
Sibling gaps (none needed by a burned program, since no burned program uses JSON):

Remaining slices

The siblings above, in that order; merge origin/main before merge (overlap with #7905 / #7930 in gen.py, X76.md, tests).

Checklist: partial [c:bb8e1760] -- NAME OMITTED, elementary source, floats left; the item's named phrases are covered once #7905 and #7930 merge

Facts checked: IBM LR pages (JSON GENERATE, Operation of JSON GENERATE) state n elements for ODO value n and [] for zero; out-of-range and nesting are not stated there. Gates: det_port check 270/270 unchanged; audit_check clear; pr_gates --fast: only test_cobolrt_rules failed, fixed (plumbing entry) and re-passing; 146 JSON tests pass.

Lessons for agents

python is not on PATH in a worker shell: export the selfscan venv bin in every Bash call. A new public static runtime method needs a plumbing entry in docs/language_status/cobolrt_rules.json.

🤖 Generated with Claude Code

@squid-protocol squid-protocol added the shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md label Nov 6, 2026
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd
squid-shepherd Bot marked this pull request as ready for review November 6, 2026 16:48
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd
squid-shepherd Bot enabled auto-merge (squash) November 6, 2026 16:54
@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

🐦‍⬛ Muninn Security Scan

✅ No security issues found.

🐦‍⬛ Powered by Muninn · Skald Lab

@squid-shepherd
squid-shepherd Bot disabled auto-merge November 6, 2026 17:03
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

This PR conflicts with main, so no checks ran. Merge origin/main into it and regenerate generated files with their tools (never hand-merge them), then push.

@squid-shepherd squid-shepherd Bot added the shepherd:needs-fix Bot: a real failure on this head; see the digest comment. Guide: docs/ci.md label Nov 6, 2026
squid-protocol and others added 2 commits November 6, 2026 12:06
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…art of #7524)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@squid-shepherd squid-shepherd Bot removed the shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md label Nov 6, 2026
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

CI shepherd: the head of this PR (95ced4c) is not the one a maintainer labelled shepherd:merge on (new commits, or never approved), so the label was removed and nothing merges unreviewed code. Re-add the label when the new head is ready (shepherd:reapprove marks this PR until then). Until then it is a draft and runs the cheap audits only; re-adding the label marks it ready, which starts the full set on that head.

@squid-shepherd
squid-shepherd Bot marked this pull request as draft November 6, 2026 17:22
@squid-shepherd squid-shepherd Bot added shepherd:reapprove Bot: new commits dropped shepherd:merge; re-add it to merge this head. Guide: docs/ci.md and removed shepherd:needs-fix Bot: a real failure on this head; see the digest comment. Guide: docs/ci.md labels Nov 6, 2026
@squid-protocol squid-protocol added shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md and removed shepherd:reapprove Bot: new commits dropped shepherd:merge; re-add it to merge this head. Guide: docs/ci.md labels Nov 6, 2026
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd
squid-shepherd Bot marked this pull request as ready for review November 6, 2026 17:32
@squid-shepherd squid-shepherd Bot added the shepherd:retrying Bot: an infra/flake rerun is pending on this head. Guide: docs/ci.md label Nov 6, 2026
@squid-shepherd
squid-shepherd Bot enabled auto-merge (squash) November 6, 2026 17:34
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd squid-shepherd Bot removed the shepherd:retrying Bot: an infra/flake rerun is pending on this head. Guide: docs/ci.md label Nov 6, 2026
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-protocol squid-protocol added shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md and removed shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md labels Nov 6, 2026
@squid-shepherd

squid-shepherd Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd
squid-shepherd Bot merged commit 0279ad3 into main Nov 6, 2026
58 checks passed
@squid-shepherd
squid-shepherd Bot deleted the churn/7524-20261106T1624Z branch November 6, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant