@try-works/dsh-memory 0.1.24 → 0.1.25

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (2) hide show
  1. package/package.json +1 -1
  2. package/src/tools.js +6 -1
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@try-works/dsh-memory",
3
- "version": "0.1.24",
3
+ "version": "0.1.25",
4
4
  "description": "Git-backed, workspace-scoped memory for the DeepSeek Harness (global layer + per-workspace layers)",
5
5
  "type": "module",
6
6
  "main": "src/index.js",
package/src/tools.js CHANGED
@@ -758,7 +758,12 @@ export function registerTools(ctx, store, extras = {}) {
758
758
  if (!syncNow) return { synced: false }
759
759
  const status = await syncNow()
760
760
  return {
761
- synced: Boolean(status && (status.synced || status.committed || status.initialized)),
761
+ // `synced` means SYNCED, not 'it did something'. It used to include committed and
762
+ // initialized, so a push that FAILED still reported synced:true whenever the local commit
763
+ // had succeeded - the result read {"synced":true,"conflict":true}, claiming success and
764
+ // failure at once. A local commit is progress, not a sync, and saying otherwise is the same
765
+ // defect as every other one this layer has produced: a green answer over a silent failure.
766
+ synced: Boolean(status && status.synced),
762
767
  ...(status && status.conflict ? { conflict: true } : {}),
763
768
  ...(status && status.error ? { error: String(status.error) } : {}),
764
769
  }