Skip to content

[codex] Keep known hosts local during sync#863

Merged
binaricat merged 1 commit into
mainfrom
codex/known-hosts-local-only-sync
Apr 29, 2026
Merged

[codex] Keep known hosts local during sync#863
binaricat merged 1 commit into
mainfrom
codex/known-hosts-local-only-sync

Conversation

@binaricat
Copy link
Copy Markdown
Owner

Summary

  • Stop including known hosts in cloud sync payloads.
  • Ignore legacy cloud known-host data when applying or merging sync snapshots.
  • Keep known hosts in local backup and restore flows.
  • Add regression tests for cloud payloads, local backups, merge behavior, and shrink protection.

Why

Known hosts are device-local trust records. Syncing them across machines can overwrite local state with records from another device, so cloud sync should leave them alone while still preserving them in local backups.

Related to #859.

Validation

  • node --test --import tsx application/syncPayload.test.ts domain/syncMerge.test.ts domain/syncGuards.test.ts
  • npm run lint
  • npm test
  • npm run build

@binaricat binaricat marked this pull request as ready for review April 29, 2026 03:01
@binaricat binaricat merged commit 4dd2465 into main Apr 29, 2026
@binaricat binaricat deleted the codex/known-hosts-local-only-sync branch April 29, 2026 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant