Repository navigation
bump stitch-wasm to 0.2.2, add config tests, update all dependencies - #32
Merged
Merged
Conversation
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
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.
Adopts
@laboverwire/stitch-wasm0.2.2, which implements config-override fields that 0.2.1 declared but did not honor. OurStoreConfigalready exposed all of these (added during the 0.5.0 work, ahead of the wasm), so no type or doc changes were needed — the fields flip from "declared but inert" to functional. Adds integration coverage against the real 0.2.2 wasm.Changes
@laboverwire/stitch-wasm^0.2.1 → ^0.2.2(package.json+ lockfile).npm audit fix(brace-expansion,postcss,sync) — installing 0.2.2 regenerated the lockfile and surfaced newly-published advisories; cleared back to 0.tests/integration/wasm-config-fields.test.ts(3 tests).What the tests actually verify
localOnlyEntities— full CRUD (create/read/list/update/delete) on a local-only entity, readable without areplaceScope. Behaviorally verified.topLevelEntities— create/read/list on a globally-synced entity. Behaviorally verified.versionField/updatedAtField/userScopeField/syncTopicPrefix/responseTopicPrefix— these govern sync & offline-queue semantics that are not observable without a live MQTT broker. The test verifies only that 0.2.2 accepts the full extended config and core scoped CRUD is unaffected (config round-trips viastore.config, create/update/version-increment work). Their actual sync semantics remain covered by stitch-rs's own tests, not here.Verification
tsc,eslint,prettier --checkclean.npm audit(full and--omit=dev) = 0..d.tsdiff is docs-only; the work is a +11.5 KB binary implementation behind the existinganyconfig surface). No runtime API change in this package.Second commit: update all dev & example dependencies to latest
Bumps every outdated dependency across the main repo and all three examples to latest, including majors where the toolchain supports them.
TypeScript held at 6.0.3 (all trees)
TypeScript 7 was evaluated and deliberately not adopted:
typescript-eslinthard-throws on TS 7 ("does not support TS 7.0", tracking #10940) andvue-tscneeds the TS 6lib/tsc. TS 7's official model is running the@typescript/typescript6compat API side-by-side for exactly these tools — which works, but for a sub-second build it adds real fragility for no functional gain (lint & vue type-check still execute TS 6). Decision: stay on TS 6.0.3 (latest 6.x) uniformly; revisit when the toolchain ships TS 7 support.Verification
tsc,eslint,prettier --checkclean, 33 tests pass, build emits all entry points.npm audit= 0 across all four trees.