Skip to content

CicsTask.within: primitive-table and cross-width numeric values of the LINKed DTO (X39, Part of #5060) - #5381

Merged
squid-protocol merged 1 commit into
mainfrom
churn/5060-20261017T2207Z
Oct 17, 2026
Merged

squid-protocol merged 1 commit into
mainfrom
churn/5060-20261017T2207Z

Conversation

@squid-protocol

Copy link
Copy Markdown
Owner

Part of #5060 (first checkbox: model-port scalar fields; the base was shipped in #5332, this closes its remaining holes).

Class: XCTL of storage a LINK caller owns (X39), model-port path: a value COMMAREA is traced by equality, so every way a DTO holds a value must be compared the same way.

Change (template-only, CICS_TASK_JAVA): within now (1) walks primitive arrays (int[], char[] tables) for a value part, (2) compares numbers by numeric value whatever width or scale (sameValue: int vs long, 1.5 vs 1.50). Equal-valued copies stay over-refused (refuse, never guess).

Tests: test_model_link_xctl_4956.py adds table int/char element, widened/narrowed long/int, BigDecimal scale, boolean (refused) and a distinct char / distinct number (not refused). Ran with the 4816/4957 XCTL tests and tests/cics_crucible/test_cics_runtimes.py: 222 passed, 3 skipped. pr_gates --fast: 6/6.

det_port check: every port's generated CicsTask.java changes by 25 template lines (runtime text only; no port uses a LINKed DTO XCTL path differently). Re-prove list not run (120 ports).
Not run: golden bless (no parsing-logic change).

Sibling gaps: copy vs same storage needs a real z/OS run (stays REFUSED, not churn). No burned-estate program known to need these forms; not grepped.

Cross-repo: none.

Lessons for agents

  • det_port.py check reports every port CHANGED for any CICS_TASK_JAVA edit; it is template text, not behaviour.

🤖 Generated with Claude Code

…values count as the LINKed DTO's values (X39, Part of #5060)

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

Copy link
Copy Markdown
Owner Author

No description provided.

@squid-protocol
squid-protocol marked this pull request as ready for review October 17, 2026 22:22
@squid-protocol

Copy link
Copy Markdown
Owner Author

No description provided.

@squid-protocol
squid-protocol enabled auto-merge (squash) October 17, 2026 22:24
@github-actions

Copy link
Copy Markdown
Contributor

🐦‍⬛ Muninn Security Scan

✅ No security issues found.

🐦‍⬛ Powered by Muninn · Skald Lab

@squid-protocol
squid-protocol merged commit 75bd4e8 into main Oct 17, 2026
81 checks passed
@squid-protocol
squid-protocol deleted the churn/5060-20261017T2207Z branch October 17, 2026 23:09
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