Quest/journal sync fixes, v14 compat, and review cleanup

[camrun91]

Review branch off staging for a second set of eyes before merging. All changes verified against the archivistai/archivist-api quest/journal contract and Foundry v13.351 / v14.359. npm run lint passes (0 errors); no automated test suite exists in this repo, so a manual smoke test of the setup wizard import and the quest sheet is the one gap.

Highlights for reviewers

API contract correctness (archivist-api.js)

• _normalizeQuestPayload rewritten as a strict whitelist. The API's QuestCreate/QuestUpdate declare extra="forbid"; the old mapper emitted read-only fields (quest_giver_id, progress_log_entries, first/last_session) that would 422 the entire write. Also drops empty-text objectives (min_length=1).
• keepalive limited to bodies < 60 KiB (Chromium rejects larger keepalive requests — was silently failing large journal writes).
• listQuests normalized back to camelCase for consistency with get/create/update.

Quest sheet (page-sheet-v2.js)

• Objective add/remove now PATCH the full list to Archivist (previously edited local flags only, never synced).
• Status/category labels localized via game.i18n with English fallback.
• Removed inert static TABS block and a dead journal length check.

Sync dialogs + realtime

• Fixed shift+click range-select double-toggling the clicked row.
• Escaped journal names in the delete-confirmation dialog (was raw HTML).
• Folder fallback for journal/quest imports on worlds set up before 2.0.
• Replaced per-item full re-renders with direct DOM progress updates (sync-dialog + world-setup import loops) — ~2 renders instead of ~N on large imports.
• Skip the redundant quest-name PATCH on quest page-content edits.

v14 compatibility

• module.json: minimum 13.351 / verified 14.359 (was reversed — as-was, the module could not install on v13).
• cleanHTML fallback chain and inert stripHtml.

Cleanup

• Removed committed utils.js.bak and dead ask-chat-sidebar-tab.js (both were shipping in the release zip).

Testing notes

• npm run lint: 0 errors.
• Node --check on all modified scripts: passes.
• Not yet run in a live Foundry world — worth a manual pass on the world-setup import (progress bar behavior changed) and quest objective editing (new API write path).

🤖 Generated with Claude Code