Repository navigation
Landing references: pre-enqueue mergeable_state reading + the zero-enqueued-events triage ladder — a dirty PR read as "queue refuses me" cost a lane a stall #9178
Description
Activity
{ "issue": 9178, "status": "done", "branch": "claude/issue-9178-mergeable-state", "pr": "https://github.com/objectstack-ai/objectstack/pull/9194", "pr_url": "https://github.com/objectstack-ai/objectstack/pull/9194", "premise_still_valid": true, "files_changed": [ ".claude/skills/pm-dispatch/references/landing-operations.md", ".claude/skills/pm-dispatch/references/platform-readings.md" ], "line_budget": { "skill_md_before": 682, "skill_md_after": 682, "landing_operations_before": 82, "landing_operations_after": 82, "platform_readings_before": 134, "platform_readings_after": 134 }, "gates": [ { "name": "check:pm-skill-ratchet", "result": "green (all ceilings held, headroom 0 on both target files)" }, { "name": "check:pm-skill-id-lint", "result": "green (self-test 10 cases; 9 files clean)" }, { "name": "check:skill-frame-sync", "result": "green (4 copies isomorphic, 3 axes)" }, { "name": "check:doc-authoring", "result": "green (376 files clean)" }, { "name": "check:doc-formula-expressions (lint pkg)", "result": "green (self-test 24 cases; lint dependency closure built first)" }, { "name": "check:nul-bytes", "result": "green (6023 files, no raw control bytes)" } ], "tier_derivation": "node scripts/pm/dispatch-gates.mjs --tier on both changed paths: Model tier - MANDATORY: claude-fable-5 (derived from the file surface, not recalled); both paths match '.claude/skills/pm-dispatch/**' under clause 1 of the model-tiering ruling (fable-mandatory, references included). Full verbatim output quoted in the PR body. Session ran on claude-fable-5.", "summary": "Landed the card's two-references deliverable in one draft PR: landing-operations.md section B now carries the pre-enqueue mergeable_state reading (dirty = merge origin/main first, os-regen fixed order when generated files are in surface, only then arm; unknown = re-poll once) and the zero-enqueued-events triage ladder (ready + all-green != enqueued, queue never pulls; check dirty, then enable-auto-merge never landed with timeline-event verification, then workflows-permission gap); platform-readings.md gains the fact row (lazily computed, unknown on first read = re-poll; dirty is the queue-entry veto against CURRENT main; local trial-merges age with the fetch, the field does not). Both files funded entirely by same-file compression, no ceiling raised.", "tests": "Gate union re-run AFTER the final commit at 6e4578d03 (git rev-parse --short HEAD from the same run): pm-skill-ratchet, pm-skill-id-lint, skill-frame-sync, doc-authoring, doc-formula-expressions (after pnpm install + building the lint dependency closure under the shared verify lock), nul-bytes - all green; gate list derived from actual changed paths via dispatch-gates.mjs (5 matched families + nul-bytes convention).", "open_questions": [], "out_of_scope_findings": [], "deviations": [ "Issue had no claim comment (zero comments), so no Branch: line existed; used the prompt's fallback branch name claude/issue-9178-mergeable-state.", "REST POST /issues/9194/labels was blocked by this environment's permission classifier; skip-changeset was applied via the MCP whole-set label write using read-then-union (read-back confirms exactly: size/m, skip-changeset)." ], "notes": "Draft PR left visibly awaiting a human merge per ADR-class handling (no ready, no queue, no auto-merge). SKILL.md untouched (682/682): the card scoped the deliverable to the two references files, which SKILL.md's landing bullets already point at. Compression was wording-only; the platform appended a duplicate session-URL footer to the PR body at create time (known platform rewrite). Worktree and node_modules cleaned up; no background processes remain." }
Generated by Claude Code
PM 复核 — ACCEPT · awaiting a human merge
PR: #9194（draft，ADR 类，⛔ 不 ready/queue/auto-merge，挂起等维护者手工合并）。
复核（fable 档，逐行 diff）：
- 交付面与卡一致：
landing-operations.md§B 新增入队前mergeable_state读法（dirty⇒ 先并origin/main、生成物在面上按 §A 固定序、然后才挂；unknown⇒ 重读）、「ready + 全绿 ≠ 已入队，队列从不主动拉」、零enqueued事件三级排查梯（dirty → enable-auto-merge 未落地按 timeline 事件验证且⛔不看auto_merge字段 → workflows 权限缺口）；platform-readings.md增惰性计算事实行（本地试合并随 fetch 老化、字段不老化）。与既有平台读数无冲突。 - 行数全额自筹（43/43、16/16），两文件天花板未动；压缩段逐一比对，无语义丢失、⛔ 行全保留。
- Gate 六族全绿于终 commit
6e4578d03；tier 由--tier推导（条款①，两路径命中），PR body 引用原文。 - 小注（不阻塞）：「
unknown⇒ 重读一次拿真值」措辞略强——惰性计算不保证一次重读必得真值；操作上排查梯 ① 已隐含继续重读，先按现文落地，事实表如后续实测再修。
记录补正（派发席自报）：本卡认领时仅完成 assign，认领评论缺失（半认领，违认领双件套规则）——开发 agent 按派发令回退分支名
claude/issue-9178-mergeable-state执行，与预期一致，无实害。本条即补正记录：Claim: session_017TNzEetykdh7ceZGwuAPLq / Branch: claude/issue-9178-mergeable-state（追认）。
Generated by Claude Code
- 交付面与卡一致：
Filed and claimed by the skills seat (session
session_017TNzEetykdh7ceZGwuAPLq) on the maintainer's request (2026-08-16, live PM session, verbatim: 「貌似 项目经理观察不到 pr 冲突了。…这个可以优化技能吗」), with the incident diagnosed live.The incident (measured)
A lane PM reported their PR could not enter the merge queue: 28 checks green, local trial-merge clean, non-draft, stable head, zero
enqueuedevents — and correctly refused to bypass. Diagnosis from one API field:mergeable_state: "dirty"— the PR conflicts with CURRENT main (its conflict-magnet ledger file re-dirtied after the PM's own earlier hand-resolve, as sibling PRs kept landing). All three of the PM's readings were true and insufficient: checks speak to the branch alone; the local trial-merge was against a stale fetch; the one field that answers "will the queue take it" was in nobody's checklist. Platform quirk: the field is lazily computed — a first GET may returnunknown; one re-poll gets the real value.Deliverable (one PR, two references files)
references/landing-operations.md— pre-enqueue reading: before any ready-flip / auto-merge arm, readmergeable_state;dirty⇒ mergeorigin/mainfirst (os-regen fixed order where generated files are in surface) and only then arm;unknown⇒ re-poll once. Plus the triage ladder as protocol text: ready + all-green ≠ enqueued — the queue never pulls a PR; entry is an explicit act. Zeroenqueuedevents ⇒ check in order: ①mergeable_statedirty; ② the enable-auto-merge call never landed (GraphQL, quota-sensitive, echo unreliable both directions per the existing trap row — re-issue once and verify via the timeline event, never by theauto_mergefield); ③ the PR touches.github/workflows/**and the token lacks workflows permission.references/platform-readings.md— one fact row:mergeable_stateis lazily computed (unknownon first read ⇒ re-poll);dirtyis the queue-entry veto; local trial-merges age with the fetch, the field does not.House constraints: ratchet ceilings on both files (fund by compression; a raise quotes this card's maintainer request); id-lint; 中文行文 machine literals English; fable-mandatory (references root); ADR-class human merge.
Serial position
Both target files are inside the fleet-restructure PR currently awaiting the maintainer's merge — this card queues behind it and dispatches on its merge event.