Settings sync on @profullstack/synconfig, release 0.96.0 - #499
Merged
Merged
Conversation
…e it became The allowlist check, collecting, validating, planning, applying and the digest that /save and /load are built on now come from @profullstack/synconfig, the package this module was extracted into so myna and the rest of the fleet sync the same way. The app's /api/settings routes are the package's handlers over a SnapshotStore on this database, with the GROUP BY user_id HAVING that Turso needs kept in the store. What stays moshcode's: which files sync (SYNCED_FILES, the never lists, now also exported as SYNC_POLICY), the marker, the transport with its retry rule, both verbs and every message, and the /settings/sync pages. Both suites pass unchanged, including the pinned digest on each side. Snapshots now also carry `app: "moshcode <version>"` beside `moshcode`, the field name the package uses; the server reads either. Release 0.96.0. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01S7yeJUHGxA4P5N74xnsRPQ
ThreatCrush Security Scan1 finding(s) in the 6 file(s) this pull request changes. MEDIUM: 1
91 pre-existing finding(s) elsewhere in the repository — **HIGH/CRITICAL**: 8 | **MEDIUM**: 73 | **LOW**: 10Not introduced by this pull request. The full set is in the Security tab.
…and 71 more. Full results in the Security tab. Snippets are redacted; ThreatCrush never prints matched credential material. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Anthony: "modularize that feature ... synconfig ... deploy it everywhere it makes sense across ~/src".
/saveand/loadkeep every message and rule, but the primitives (allowlist check, collect, validate, plan, apply, digest) are now@profullstack/synconfig@0.1.0, the package extracted from this module. The app's/api/settingsroutes are the package'shandlePut/handleGet/handleRevisionsover aSnapshotStoreon this database; the Turso-safeGROUP BY user_id HAVINGinsert is kept verbatim in the store.SYNC_POLICYexported besideSYNCED_FILESand the never lists.app: "moshcode <version>"as well asmoshcode; the server reads either (versionOf).Tests: CLI suite 2,569 pass / 0 fail; app suite green, pinned digests unchanged on both sides.
🤖 Generated with Claude Code
https://claude.ai/code/session_01S7yeJUHGxA4P5N74xnsRPQ