Skip to content

feat(core): category manager level for dispositions raised by a non-CM (#47) - #56

Merged
marchtian merged 1 commit into
mainfrom
claude/project-thread-sdia45
Oct 24, 2026
Merged

marchtian merged 1 commit into
mainfrom
claude/project-thread-sdia45

Conversation

@marchtian

Copy link
Copy Markdown
Collaborator

Requested by 田騏 · project thread

Closes #47 (director ruling, v0.4 overview #34).

改了什么

  • srm_disposition_approval:照 srm_contract_approval,开头查发起人是否持 srm_category_manager 岗位。
    • 是:不变,发起即品类经理一级,直接到采购总监(可退回修改)。
    • 不是(如采购主管发起):先由品类经理岗位审批(可通过 / 驳回 / 退回修改,退回后从品类经理重新审批),再到采购总监。
    • 这条路径上采购总监是第 2 个审批节点,受 objectstack#24817 限制只能通过或驳回,所以单列一个 director_review_after_category 节点(maxRevisions: 0、无 revise 边);平台修复后合并两个采购总监节点。
    • BR-11:各级比对改用变量(categoryActor / directorActor / gmActor,初始化为空串),品类经理审批人也计入后面几级的撞人比对。
  • srm_supplier_disposition:新增只读字段 category_approver(品类经理审批人,加字段、不改已有名称),英文翻译同步。
  • 文档:03 §5 实现要点、§12 冻结 / 淘汰行;02 处置单字段;GL-08 6.2.1、6.3.1、6.3.2、第 7 章控制表、附录 A;ZC-02 第 2 章、QP-01 附录 C 各加一句(ADR-0005 体系文件与控制点同步)。

验证

  • test/disposition.test.ts 新增 3 条(issue 里写的 test/approval-matrix.test.ts 不存在,用例放在处置单测试里):
    • 采购主管发起冻结:先品类经理、再采购总监,category_approver / director_approver 写入;
    • 品类经理退回采购主管的处置单,采购主管改完重新提交,从品类经理重新审批;
    • 同时任品类经理和总经理的人先审了品类经理一级,到总经理一级再审不计入,换另一位总经理后通过。
    • 品类经理发起的用例补断言:直接到采购总监、category_approver 为空。
  • 本地:pnpm validate、pnpm lint(多出 2 条与其他审批节点同类的 info 建议)、pnpm typecheck、pnpm i18n:check、pnpm test(16 文件 158 条全过)、pnpm build 全部通过;governance/scripts/build.py --check 及 --packs bulk,hazmat,soe 通过。

Generated by Claude Code

srm_disposition_approval treated initiation as the category manager level,
so a disposition raised by a buyer lead (who also holds srm_supplier_admin)
went straight to the procurement director and skipped the CM level of the
03 §12 matrix. Like srm_contract_approval, the flow now looks up whether the
initiator holds srm_category_manager; if not, a category manager approves
first (recorded in the new read-only category_approver field), and the
later levels' BR-11 checks also compare against that approver.

The CM level is the first approval node on that path, so it can send the
disposition back; the director then sits on a second node that can only
approve or reject until objectstack#24817 ships. CM-initiated dispositions
are unchanged.

Docs: 03 §5 / §12, 02 disposition fields, GL-08 6.2.1 / 6.3.1 / 6.3.2 /
ch. 7 / appendix A, ZC-02 and QP-01 appendix C notes.

Closes #47

Claude-Session: https://claude.ai/code/session_01Ut39cZ4zdvuJmqenJzvnKh
Co-authored-by: Claude <noreply@anthropic.com>
@marchtian marchtian self-assigned this Oct 24, 2026
@marchtian
marchtian merged commit 64aaeed into main Oct 24, 2026
2 checks passed
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.

处置单审批:非品类经理发起时缺少品类经理一级

2 participants