[hahn-kev-bot]

---

AI summary

Bumps SIL.Harmony, SIL.Harmony.Core and SIL.Harmony.Linq2db from 0.2.1-rc.397 to 0.2.1-rc.445, and registers Harmony's new SIL.Harmony ActivitySource in LexBoxApi and FwHeadless so its spans are exported.

Harmony changes pulled in:

• Trace sync planning and recovery harmony#155: traces sync planning. GetCommitsMissingFromRemote (used by the server's /changes endpoint) gets a PlanClients span with a harmony.sync.recovery event whenever a client falls back to resending its whole history.
• [claude] Enforce required members when deserializing commits harmony#152: commits missing clientId or hybridDateTime are rejected on deserialization again instead of getting default values.
• Make sync an injectable SyncService harmony#149: the static SyncHelper became an injectable SyncService, and JsonSyncable is now internal. Lexbox didn't use either.
• Keep applying the other sync direction when one fails harmony#148: pull and push are applied independently, so a pull that fails no longer blocks the push. SyncWith can now throw after the push has landed.
• Compact the SQLite file after regenerating snapshots harmony#143: the SQLite file is compacted after the initial sync and after regenerating snapshots.
• [claude] Test that deleting an entity cascades through referrers harmony#151: test only.

Test plan

• LexBoxApi, FwHeadless and FwLiteShared build against rc.445
• LcmCrdt.Tests sync and snapshot tests pass (DownloadProject, Migration, SnapshotAtCommitService, LegacyTranslationPatchReplay, DataModelSnapshot): 21 passed
• CI
• After deploy, SIL.Harmony spans show up in Honeycomb for /api/crdt/*/changes

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Repository UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: d2ca992f-b8c5-4859-b60d-0befe983f3d2

Commits

Reviewing files that changed from the base of the PR and between 56a6f2b and 238c53d.

Files selected for processing: 4

• backend/Directory.Packages.props
• backend/FwHeadless/Program.cs
• backend/LexBoxApi/Otel/OtelKernel.cs
• backend/LexBoxApi/openapi/public.yaml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The PR updates the centrally managed SIL.Harmony package versions. It registers Harmony’s activity source in FwHeadless and LexBoxApi. The public ServerCommit schema now requires clientId.

Priority: Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Possibly related PRs

• sillsdev/languageforge-lexbox#2788: Updates the same Harmony package entries and edits the public OpenAPI contract.

Suggested reviewers: hahn-kev, myieye

Merge Risk: Minimal · up to 238c5

No actionable merge-blocking risk remains from the reviewed changes.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title names the two main parts of the changeset: the Harmony version bump from 0.2.1-rc.397 to rc.445 in the package files, and the registration of Harmony's ActivitySource in LexBoxApi and FwHead…
Description check		The description states the purpose of the change: the bump of SIL.Harmony, SIL.Harmony.Core and SIL.Harmony.Linq2db, and the registration of the SIL.Harmony ActivitySource in LexBoxApi and FwHeadless.…
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title names the two main parts of the changeset: the Harmony version bump from 0.2.1-rc.397 to rc.445 in the package files, and the registration of Harmony's ActivitySource in LexBoxApi and FwHeadless so its traces are exported. It is short, specific, and matches the diff summary and PR objectives.

Full details: Description check

Explanation

The description states the purpose of the change: the bump of SIL.Harmony, SIL.Harmony.Core and SIL.Harmony.Linq2db, and the registration of the SIL.Harmony ActivitySource in LexBoxApi and FwHeadless. It also lists the upstream Harmony changes, the test plan, and the unchecked CI and Honeycomb verification items. It is closely related to the changeset.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 2 files. (2 skipped: 2 unsupported.)

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the traces bright,
And sees new Harmony sources in sight.
A commit now carries clientId,
The package versions hop in stride.
Then off I go beneath the moon.}

_{Comment @coderabbitai help to get the list of available commands.}