Repository navigation
Upgrade @objectstack/* to 18.3.0; the contract page follows the v3 record page - #198
Merged
Merged
Conversation
Moves the four @objectstack pins and the eight pnpm overrides from 18.2.0 to 18.3.0; every @objectstack package in the lockfile resolves to 18.3.0. 18.3 retired `page.slots.tabs` (ADR-0085, objectstack#27053), so the contract page's authored seven-tab strip no longer parses. Per the maintainer's ruling (hotclm decision box batch 11, letter A, 2026-11-03) the page follows the platform's v3 record page: the slotted page keeps its header, highlights and path, and the body is composed from the object — the key-term field groups, then six full-width child tables placed by body-table entries in `clm_contract.fieldGroups`, with the columns the page used to carry moved to each child FK's `relatedListColumns`. Renewals stay a side card; `parent_contract` is kept off the page as before. DESIGN.md §05 and the README describe the new shape. 18.3 also makes a master-detail field's `inlineTitle` translatable, so the i18n gate now counts the six child grid titles; both bundles carry them. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QsR2jUw8KJgAwuwk6sqYLD
Contributor
席位核对 — ACCEPT · head
|
| check | reading |
|---|---|
| shape | draft → main (base bc47708, which includes PR #194), one commit; mergeable once CI finished |
| CI on the head | both Validate check-runs success on e1080b1 = PR head.sha |
| scope | 19 files: the @objectstack/* 18.2.0 → 18.3.0 pins and overrides plus the lockfile. contract_detail.page.ts drops slots.tabs, because 18.3 refuses it (ADR-0085, objectstack#27053). Body-table fieldGroups and related-list ordering are added on clm_contract and the six children, with translations. In DESIGN.md, only the §05 page paragraph changes. §05 is outside the governed §01–04, and the layout follows the maintainer's batch-11 ruling A. The header action list keeps every action, record_finance_review (PR #194) included |
| lint | exit 0. The 17 new security-superuser-field-mask-inert warnings are 18.3 reporting a fact that already held on 18.2: clm_admin's modifyAllRecords makes 17 field locks inert. The upgrade does not cause it; the decision is carded as hotclm#196. The 18th warning is the existing #64 |
| measured | a fresh pnpm demo:zh run in the browser, signed in as the real personas, against the full regression report (/mnt/project-files/hotclm-objectstack-18-3-upgrade/report.md). Covered: landings; both intake paths; the approval ladder and rejection; the full NDA lifecycle; renewal ×2; backfill; Setup positions in zh-CN; the inbox. All 6 daily flows were run twice under an uncommitted cron override, and every run completed. The post-decision page now reads 「你的审批决定已记录」 (objectstack#26589 shipped) |
| residuals | 4 detail-page regressions are carded upstream (objectstack#29942–#29945), as batch 11 ruled. New app cards: hotclm#195, #196, #197 |
Landing: under the maintainer's merge authorization (project thread: 「你可以自己审核的就自己合并」).
Generated by Claude Code
zhuangjianguo
marked this pull request as ready for review
November 3, 2026 16:44
This was referenced Nov 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Steedos · project thread
维护者速读
page.slots.tabs(ADR-0085),旧页面无法通过校验;走平台编排是决策箱第 11 批裁定 A。pnpm demo:zh上 11 个真实角色跑完发起、审批阶梯、驳回、NDA 全生命周期、续签、补录、Setup、6 个定时流程两轮。demoStaff(18.3) — drop the README's manual "Set Manager" step #197 新卡。What changed
@objectstack/*pins and 8pnpm-workspace.yamloverrides 18.2.0 → 18.3.0@objectstackpackage in the lockfile resolves to 18.3.0src/pages/contract_detail.page.tsslots.tabsand the emptydiscussionoverride removed; header, highlights and path keptpage.slots.tabs(ADR-0085, objectstack#27053) —validaterefuses itclm_contractfieldGroupsgains six body-table entries (versions, reviews, deviations, obligations, payments, signatures) after the key terms;relatedListsorder;parent_contractrelatedList: false(it was never on the page)contractFK'srelatedListColumns;renewed_fromgets the renewals columnsinlineTitlefor the six master-detail FKs (18.3 made it translatable, objectstack#26142, so the i18n gate counts it); labels for the six table groups; the orphanedarchivesection andcontract_renewalspage keys removedRuling: batch #11 item 1 · letter A · maintainer 「同意」 2026-11-03T16:16:59Z — decision doc https://claude.ai/artifact/Bq678GdTudDmAa5dzKGT1W
Gates
17 of the 18 warnings are new in 18.3 (
security-superuser-field-mask-inert:clm_admin'smodifyAllRecordsonclm_contractmakes 17 field edit locks inert) → filed #196. The 18th is #64's existing policy.Browser evidence (fresh
pnpm demo:zh, real personas, 1440×900 zh-CN)✓ Server is ready, no degraded capabilities / missing tables / failed plugins.in_review,legal_owner= counsel1. Blank party → NDA-2026-0022 and the newclm_partyrow.finance_only→approved; toast 「已通过“18.3 升级验证 — 订单”」. After deciding, the manager sees 「你的审批决定已记录」 (no longer 「未找到记录」).rejected.disabledReason, so 送审 / 发起签署 / 生效 can say what is missing #195); v2 clean version, review approved (decided_atstamped), 送审 → executive approves → records 发起签署 → counsel1 wet-ink signature (completed_atstamped) → records 生效 →active.executed_upload): NDA-2026-0024active,is_backfilled = true— no 403.OS_AUTOMATION_SCHEDULED_WORK_ENABLED=true: two runs each, allcompleted.bc47708(Legal and finance can edit a counterparty they do not own #193 / Finance records its own review through a Record Finance Review action #194): renders, header includesrecord_finance_review.Full report and screenshots: project folder
hotclm-objectstack-18-3-upgrade/report.md.Filed from this pass
Platform (the four regressions the ruling sends upstream): objectstack#29942 (related-list title untranslatable — renewals card reads 「合同」), #29943 (body-table totals sum version_no / seq), #29944 (empty body table reads 「暂无相关记录 偏离 新建」), #29945 (body tables clip columns in the details column).
hotclm: #195 (author
disabledReason), #196 (clm_admin inert field locks), #197 (demoStaff.managerreplaces the README's manual step).Generated by Claude Code