fix(#59): wire read()'s onSkip into every caller — incl. a v0.34.0 env-loss regression in `mcpm update` - #199

Merged

m1ngshum merged 18 commits into

mainfrom

fix/onskip-callers-59

Sep 4, 2026

Commits on Sep 4, 2026

fix(update): refuse to overwrite a client entry read() could not validate (#59)
m1ngshum
committed
1c154f3
test: remove duplicate `read` keys in six adapter test helpers
m1ngshum
committed
1cc847d
fix(sync): a malformed entry is no longer reported as a missing server (#59)
m1ngshum
committed
26166e4
test(#59): pin the sync matrix cell and doctor's cross-client agreement
m1ngshum
committed
a3a22b1
fix(#59): wire read()'s onSkip into diff, export, import and up --strict
m1ngshum
committed
479dbbc
fix(#59): three false statements the fixes themselves introduced, found by dogfooding
m1ngshum
committed
f74e62f
docs: CHANGELOG for the #59 fixes
m1ngshum
committed
cd1b58d
fix(#59): address 19 adversarial-review findings
m1ngshum
committed
d551267
fix(#59): third-review findings — a hole in the headline fix, and a silent CI failure
m1ngshum
committed
0167c15
fix(update): a note about a client that WAS updated no longer reads as 'could not update' it
m1ngshum
committed
77fb695
fix(#59): round-4 findings — and untrack a local settings file
m1ngshum
committed
0c79ea9
test(#59): the neighbour-suppression assertion could never match
m1ngshum
committed
dbca73e
fix(#59): round-5 — update --json emitted the malformed-entry notice nowhere
m1ngshum
committed
e3355dd
refactor(update): key the neighbour map with JSON.stringify([client, name])
m1ngshum
committed
8b7e9dc
fix(update): suppress the neighbour notice by (client, name), not by name
m1ngshum
committed
856ab0f
refactor(drift): drop collectClientStates, orphaned by this PR
m1ngshum
committed
9bbd460
test(#59): pin the last three unpinned lines
m1ngshum
committed
0ee7ef1
refactor(list): one skip-notice builder, not two
m1ngshum
committed
33e3878

Navigation Menu

Uh oh!

fix(#59): wire read()'s onSkip into every caller — incl. a v0.34.0 env-loss regression in `mcpm update` - #199

fix(#59): wire read()'s onSkip into every caller — incl. a v0.34.0 env-loss regression in `mcpm update`#199
m1ngshum merged 18 commits into
mainfrom
fix/onskip-callers-59

Commits on Sep 4, 2026

fix(update): refuse to overwrite a client entry read() could not validate (#59)

test: remove duplicate `read` keys in six adapter test helpers

fix(sync): a malformed entry is no longer reported as a missing server (#59)

test(#59): pin the sync matrix cell and doctor's cross-client agreement

fix(#59): wire read()'s onSkip into diff, export, import and up --strict

fix(#59): three false statements the fixes themselves introduced, found by dogfooding

docs: CHANGELOG for the #59 fixes

fix(#59): address 19 adversarial-review findings

fix(#59): third-review findings — a hole in the headline fix, and a silent CI failure

fix(update): a note about a client that WAS updated no longer reads as 'could not update' it

fix(#59): round-4 findings — and untrack a local settings file

test(#59): the neighbour-suppression assertion could never match

fix(#59): round-5 — update --json emitted the malformed-entry notice nowhere

refactor(update): key the neighbour map with JSON.stringify([client, name])

fix(update): suppress the neighbour notice by (client, name), not by name

refactor(drift): drop collectClientStates, orphaned by this PR

test(#59): pin the last three unpinned lines

refactor(list): one skip-notice builder, not two