Skip to content

pnpm-workspace.yaml's SCIM pin note under-counts stable's models — it enumerates six and omits scimProjectionGrant, in the exact file the pin-mover reads #11943

Description

@os-sam

Filed unassigned by the domain:services seat while re-deriving #11632's leg seams. Not fixed here — PM writes no code.

The defect

pnpm-workspace.yaml's @better-auth/scim hold note (the block above the '@better-auth/scim@<1.7.0-rc.1': '1.7.0-rc.1' pin) describes what stable ships:

replaced by code-defined connections plus six new
models (scimUser, scimGroup, scimGroupMember, scimSubject,
scimConnectionBinding, scimIdentityTombstone — all six present). So
moving this pin is still the ADR-0071 feature migration tracked by #3653

That enumeration is short by one. scimProjectionGrant is missing from it.

Measured, not inferred

The authoritative reading is an executed probe, not a source read: #11697's parity pass constructed a real @better-auth/scim@1.7.1 plugin from the published tarball (npm pack, shasum 715077319aeba4a35f114f7dfc2a8e0523aa9f6f) and had it print its own declared model list (#11697 comment 5399511853, §3):

declared models: scimConnectionBinding, scimIdentityTombstone, scimSubject, scimUser,
scimProjectionGrant, scimGroup, scimGroupMember

Seven, including scimProjectionGrant. Two other records agree with the seven and against the six: #11632's scope item 1 names all seven, and the repaired parity gate's model diff (#11632 comment 5395087489) lists scimProjectionGrant in its gained column. So the workspace comment is the lone outlier, and the instrument that disagrees with it is the one that asked the package itself.

Why this is worth a card rather than a shrug

⚠️It is wrong in the file whose whole job is to brief the person moving the pin. The note's own last clause points at the migration ("moving this pin is still the ADR-0071 feature migration tracked by #3653"), so an implementer who opens the pin to move it reads this enumeration as the target surface — and provisions six platform objects instead of seven. The miscount is not visible from the pin line itself, and nothing cross-checks a YAML comment against the package.

⭐ The same note is otherwise unusually careful (it carries the security floor, the peer-range reasoning, and the better-call skew correction from #10326), which is exactly why a reader would trust the enumeration without re-deriving it.

Scope of the fix

Comment-only. Add scimProjectionGrant and correct "six" → "seven". ⛔ Do not move the pin in the same stroke — that is the #3653 / #11632 leg-2 migration and is decision-gated; this card is only the note going stale against a measurement that already exists in the repo's own records.

⚠️ Check the surrounding sentence while editing: the scimProvider half of the note ("no scimProvider model … 0 occurrences in dist/index.mjs") is correct and independently re-confirmed by the same probe — leave it alone.

Refs: #11632 (epic, scope item 1) · #11697 (the executed probe) · #3653 (the pin move itself) · ADR-0071

Metadata

Metadata

Assignees

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions