Skip to content

det: a RETURN / XCTL COMMAREA passes LENGTH bytes (Part of #4817, #4695) - #5085

Merged
squid-protocol merged 2 commits into
mainfrom
churn/4817-commarea-length-bytes
Oct 14, 2026
Merged

squid-protocol merged 2 commits into
mainfrom
churn/4817-commarea-length-bytes

Conversation

@squid-protocol

Copy link
Copy Markdown
Owner

Part of #4817, #4695.

A RETURN TRANSID / RETURN IMMEDIATE / XCTL COMMAREA passes LENGTH bytes (the next task's EIBCALEN). The det port keeps the record's whole span beside its DTO, so CicsTask now cuts those bytes at LENGTH (areaOf), as the DTO's reading already was. A LINK's area is passed by address and is not cut.

#4695 decision: the facade's handleTransaction is Object-typed, so a RETURN with LENGTH past the DTO answers its byte[] by design; documented, and returned(Dto.class) refuses it by name with a message pointing to Object. No emission change (det/cics.py / det/program.py are #5081's).

Register X51 (ASSUMED). Test: tests/cobol_mainframe/test_commarea_length_bytes_4817.py (fails without the fix).

Class member Status
RETURN TRANSID / RETURN IMMEDIATE / XCTL bytes done earlier (#4871, #4806); LENGTH cut here
LINK bytes done (#4818); not cut by design
Facade byte answer (#4695) documented
#5060 model-port String/number XCTL sibling gap (needs a static mark in the generator)
#5060 copy vs same storage stays refused (needs z/OS)

Gates: pr_gates --fast 6/6, --ratchets 7/7, nothing skipped. No golden or ledger movement.

Cross-repo: none.

🤖 Generated with Claude Code

squid-protocol and others added 2 commits October 13, 2026 22:30
…of a LENGTH past the DTO is by design (Part of #4817, #4695)

CicsTask keeps min(LENGTH, bytes) of the bytes passed beside the DTO for RETURN TRANSID, RETURN IMMEDIATE and XCTL (a LINK's
area is by address, not cut). #4695: returnedBytes / returned(Dto) documented and the refusal names the Object answer.
Register X51.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

🐦‍⬛ Muninn Security Scan

✅ No security issues found.

🐦‍⬛ Powered by Muninn · Skald Lab

@squid-protocol

Copy link
Copy Markdown
Owner Author

No description provided.

@squid-protocol
squid-protocol marked this pull request as ready for review October 14, 2026 02:50
@squid-protocol
squid-protocol enabled auto-merge (squash) October 14, 2026 02:50
@squid-protocol

Copy link
Copy Markdown
Owner Author

No description provided.

@squid-protocol
squid-protocol merged commit 9577603 into main Oct 14, 2026
68 checks passed
@squid-protocol
squid-protocol deleted the churn/4817-commarea-length-bytes branch October 14, 2026 03:37
squid-protocol added a commit that referenced this pull request Oct 14, 2026
…EA-LENGTH (#4711)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
squid-protocol added a commit that referenced this pull request Oct 14, 2026
…alibration per runtime rule (Part of #4711) (#5086)

* WIP cobolrt rule catalogue (#4711): catalogue, checker, markers, comment-insensitive runtime compare

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* WIP cobolrt rule catalogue (#4711): two-way test, rendered page, #5073's new refusal classified

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* cobolrt rule catalogue: the generated-docs publish list pin names the new page (#4711)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* cobolrt rule catalogue: #5085's layoutBytes under RT-CICS-XCTL-COMMAREA-LENGTH (#4711)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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