|
| 1 | +--- |
| 2 | +ts: 2026-05-21T06:27:01Z |
| 3 | +kind: result |
| 4 | +role: boatman |
| 5 | +repo: endojs/endo |
| 6 | +project: endo |
| 7 | +refs: |
| 8 | + - entries/2026/05/21/062106Z-dispatch-liaison-f8fc9c.md |
| 9 | +--- |
| 10 | + |
| 11 | +Ferried `endojs/endo-but-for-bots#68` (Compartment availability and OOM |
| 12 | +limits docs) upstream as a new non-draft PR on `endojs/endo`. |
| 13 | + |
| 14 | +- Upstream PR: <https://github.com/endojs/endo/pull/3275> |
| 15 | +- Upstream PR number: 3275 (non-draft, OPEN). |
| 16 | +- Upstream branch: `kriskowal-compartment-oom-docs-2742` (first-push via |
| 17 | + `git push origin HEAD:refs/heads/kriskowal-compartment-oom-docs-2742`). |
| 18 | +- Upstream head SHA: `4d8ed6be74c861472a85df3f38af7081f1d6689c`. |
| 19 | +- Base: `master` at `bf951df346cfcf605a6709e6a5479f2fdd526113`. |
| 20 | +- Closing keyword: `Closes: #2742` in the PR body; `Fixes #2742` in the |
| 21 | + commit message. |
| 22 | + |
| 23 | +Source-side: cherry-picked 4 of the 5 commits on |
| 24 | +`endojs/endo-but-for-bots@design/issue-2742-compartment-limits-doc` |
| 25 | +(source head `838d18ff9bae5bd6d16032efb8f0c575e079ef68`) with |
| 26 | +`--no-commit` into a single squashed commit, skipping the off-topic |
| 27 | +`cb8d6286` AGENTS.md em-dash rule (which is unrelated to issue #2742; |
| 28 | +a future ferry can carry it). |
| 29 | + |
| 30 | +Picked: `8a1fc626`, `59957425`, `f299f8e4`, `838d18ff`. |
| 31 | +Skipped: `cb8d6286` (AGENTS.md). |
| 32 | + |
| 33 | +Tree-identity verification (with AGENTS.md exclusion): |
| 34 | + |
| 35 | +- `git diff origin/master..HEAD --name-only` lists exactly |
| 36 | + `docs/lockdown.md` and `packages/ses/README.md`. |
| 37 | +- `git show 838d18ff:docs/lockdown.md` is byte-identical to |
| 38 | + `git show HEAD:docs/lockdown.md`. |
| 39 | +- `git show 838d18ff:packages/ses/README.md` is byte-identical to |
| 40 | + `git show HEAD:packages/ses/README.md`. |
| 41 | +- `git show HEAD:AGENTS.md` is byte-identical to |
| 42 | + `git show origin/master:AGENTS.md` (commit 3 correctly excluded). |
| 43 | + |
| 44 | +Attribution and trailer discipline: |
| 45 | + |
| 46 | +- `git log origin/master..HEAD --pretty=fuller` shows one commit, both |
| 47 | + author and committer `Kris Kowal <kriskowal@kriskowal.com>`. |
| 48 | +- `git interpret-trailers --parse` on the commit message is empty: no |
| 49 | + `Co-Authored-By:`, no `Generated with [Claude Code]` line, no |
| 50 | + Kriscendo Bot residue. Particularly important here because the carried |
| 51 | + content from `f299f8e4` was originally authored by Kriscendo Bot; the |
| 52 | + squash rewrites attribution to Kris Kowal and drops the bot trailer. |
| 53 | +- The `(#68)` source-branch suffix on commit 5's subject was subsumed |
| 54 | + by the squash. The "address review" framing from commit 2 was |
| 55 | + dropped. The new subject is verbatim from commit 1: |
| 56 | + `docs(ses): document Compartment availability and OOM limits (#2742)`. |
| 57 | + |
| 58 | +PR body uses the endo PR template section headings verbatim |
| 59 | +(Description, Security, Scaling, Documentation, Testing, Compatibility, |
| 60 | +Upgrade Considerations). Behavior over diff. No checklists. No file |
| 61 | +callouts beyond the load-bearing `docs/lockdown.md` (Limitations |
| 62 | +section) and `packages/ses/README.md` (Compartment cross-link). No |
| 63 | +methodology leak. No reference to the source PR, the |
| 64 | +`endo-but-for-bots` fork, or any garden artifact. |
| 65 | + |
| 66 | +Identity discipline: no direct comments posted on the new upstream PR. |
| 67 | +The only cross-link comment is on the source-side PR |
| 68 | +`endojs/endo-but-for-bots#68` |
| 69 | +(<https://github.com/endojs/endo-but-for-bots/pull/68#issuecomment-4505381907>), |
| 70 | +naming the upstream URL, the head SHA, the 4-of-5 squash with the |
| 71 | +explicit `cb8d6286` AGENTS.md exclusion, and the `Fixes #2742` addition. |
| 72 | + |
| 73 | +Self-improvement: tree-identity verification when the squash is |
| 74 | +relative to a different base than the source's base (here, source was |
| 75 | +based on a months-older `master`) cannot be a single |
| 76 | +`git diff source HEAD -- . ':!excluded'` because that pulls in every |
| 77 | +unrelated change between the two bases. The reliable form is a |
| 78 | +per-file `git show source:path` vs `git show HEAD:path` byte |
| 79 | +comparison on exactly the files the squash touches, plus a check that |
| 80 | +the excluded file's HEAD content equals the base's. Worth lifting into |
| 81 | +a `pr-handoff` skill the next time a boatman ferries with exclusion. |
0 commit comments