Certify pm CLI 2026.9.23 and adopt the guarded pm-ops merge-driver launcher - #110
Conversation
…uncher - Pins @unbrained/pm-cli 2026.9.21 -> 2026.9.23, pm-ops 2026.9.18 -> 2026.9.23, pm-changelog 2026.9.18 -> 2026.9.23 exactly (package.json and package-lock.json). - scripts/prepare-merge-driver.ts is now the launcher template pm-ops ships, copied unchanged: it imports nothing from pm-ops, so `npm ci --omit=dev` in a clone skips with one notice instead of failing, while a stale or broken pm-ops still fails. A test keeps it byte-identical to the pinned template and proves it registers the drivers. pm items: pm-ts-starter-r5s2. Companion epic pm-cli-website-5s6z. release:check exits 0.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Reviewer's GuideThis PR upgrades the pm toolchain to 2026.9.23 and adopts its guarded merge-driver template, resolving pm-ops dynamically so omit-dev installs skip safely while stale packages and installer failures remain fatal. Tests model fresh consumer checkouts and validate template identity, driver registration, and failure propagation. Sequence diagram for guarded merge-driver installationsequenceDiagram
participant NPM
participant Launcher as prepare-merge-driver.ts
participant Resolver as Node module resolver
participant PMOps as pm-ops installer
participant Git as Git config
NPM->>Launcher: run prepare script
Launcher->>Resolver: resolve pm-ops/merge-driver/prepare
alt pm-ops absent
Resolver-->>Launcher: MODULE_NOT_FOUND
Launcher->>Resolver: probe pm-ops/package.json
Resolver-->>Launcher: package absent
Launcher-->>NPM: print notice and skip
else pm-ops available
Resolver-->>Launcher: installer path
Launcher->>PMOps: spawn installer
PMOps->>Git: register merge drivers
Git-->>PMOps: registration result
PMOps-->>Launcher: exit status
Launcher-->>NPM: propagate status
end
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
|
@greptileai review |
|
Review round complete before merge. All 2 inline bot threads were answered in-thread, and each was either fixed in this PR or tracked on the companion hub (launcher template: pm-cli-website-dqg7; release-workflow verifier: pm-cli-website-mxrp). Greptile's latest review covers the current head 749f23d with Confidence Score: 5/5. Sourcery skipped this head (its weekly diff allowance), so its earlier findings on this PR are the last it gave, and all were answered. CodeRabbit posted only its free-plan notice: this repository has fewer than 10 stars, so it doesn't review automatically, and its one-review-per-hour allowance is kept for PRs that need it. Every required check is green on 749f23d. |
Summary
Fleet wave of 2026-09-25 (companion epic
pm-cli-website-5s6z), applied by the fleet's deterministic wave script and verified by this repository's own gates.pm-cli-website-xy19):scripts/prepare-merge-driver.tsis now the template pm-ops 2026.9.23 ships, copied unchanged (it replaces the static-import launcher). It imports nothing from pm-ops, so a production install of a clone (npm ci --omit=dev) skips with one notice instead of dying onERR_MODULE_NOT_FOUND, while a stale or broken pm-ops still fails the install.prepare-merge-driver.test.tscompares the file byte-for-byte withnode_modules/pm-ops/templates/prepare-merge-driver.ts(pm-ops covers every branch of that template with real fixtures) and runs it to prove the drivers register.pm items
Verification
git config --get-regexp '^merge\.pm'afternpm cipm health ... --require-merge-drivers), executed from ci.ymlnpm run release:checkok - the flag changes the heading: '## 2026.1.2 - 2026-01-02' with it, '## 2026.1.2' witho)changelog:fullthenchangelog:checkSummary by Sourcery
Certify the 2026.9.23 pm tooling and adopt its guarded merge-driver launcher for reliable clone installation.
Bug Fixes:
Enhancements:
Documentation:
Tests:
Chores:
Summary by cubic
Certifies the pm CLI toolchain to 2026.9.23 and replaces the merge-driver
preparehook with the guarded launcher template pm-ops ships.@unbrained/pm-cli2026.9.21 → 2026.9.23,pm-ops2026.9.18 → 2026.9.23, andpm-changelog2026.9.18 → 2026.9.23 inpackage.jsonandpackage-lock.json.ERR_MODULE_NOT_FOUND; a stale or broken pm-ops still fails the install.Written for commit 749f23d. Summary will update on new commits.