Add dedicated ERROR_RENAME / ERROR_MOVE keys + RENAME_ERROR / MOVE_ERROR codes (#308)

[CarlosNZ]

Closes #308.

Context

PR #303 made handleEdit surface an event-specific rejection message: a generic reject (onUpdate returning false) on an add/delete shows ERROR_ADD / ERROR_DELETE instead of the generic ERROR_UPDATE. The stated invariant (comment in handleEdit) is that the message is event-specific so it matches the onError code the node routes it to.

rename and move became first-class onUpdate events in #297 but were left out: they had no dedicated localisation key (a rejected rename/move showed "Update unsuccessful") and reused the generic UPDATE_ERROR code. add/delete already have both a dedicated message and code, so rename/move were the odd ones out.

What this does

Gives rename/move full parity — dedicated messages and dedicated codes — restoring the message↔code coherence the handleEdit comment promises.

Area	Change
src/types.ts	Add RENAME_ERROR / MOVE_ERROR to the public JsonEditorErrorCode union (additive)
src/localisation.ts	Add ERROR_RENAME: 'Rename unsuccessful' / ERROR_MOVE: 'Move unsuccessful'
src/JsonEditor.tsx	Replace the errorKey ternary in handleEdit with a module-scope ERROR_MESSAGE_KEY lookup typed Record<UpdateFunctionProps['event'], keyof LocalisedStrings> — exhaustive over the event union so a future event can't silently fall through
src/hooks/useCommon.ts	Rename emits RENAME_ERROR
src/hooks/useDragNDrop.tsx	Move emits MOVE_ERROR
README.md	Localisation table + onError code list
test/JsonEditor.test.tsx	Runnable rename-reject test (asserts both the message and the RENAME_ERROR code); move-reject as test.todo (DnD simulation deferred — #270)

Naming decisions

• Error codes follow the existing <OP>_ERROR suffix convention: RENAME_ERROR / MOVE_ERROR (consistent with ADD_ERROR / DELETE_ERROR).
• Localisation keys follow the existing ERROR_<OP> prefix convention: ERROR_RENAME / ERROR_MOVE (consistent with ERROR_ADD / ERROR_DELETE).

The two new codes are additive members of the public union — no internal exhaustive switch consumes it, so nothing breaks. A minor changeset is included.

Verification

• pnpm compile ✅ — union extension typechecks, keyof LocalisedStrings picks up the new keys, no dead exports
• pnpm lint ✅ clean
• pnpm test ✅ 295 passed + 3 todo
• Regression-first check: temporarily reverted the rename code mapping → the new test failed with the right mode (UPDATE_ERROR instead of RENAME_ERROR), then restored and re-confirmed green

🤖 Generated with Claude Code

[github-actions]

Bundle size impact

json-edit-react

Format	Base raw	PR raw	Δ raw	Base gzip	PR gzip	Δ gzip
esm	54.70 KB	54.78 KB	🔺 +80 B (+0.14%)	18.95 KB	19.00 KB	🔺 +49 B (+0.25%)
cjs	56.18 KB	56.26 KB	🔺 +80 B (+0.14%)	18.97 KB	19.03 KB	🔺 +53 B (+0.27%)

_{Measured from build/index.{cjs,esm}.js. Gzip at level 9.}