@openparachute/vault 0.4.3 → 0.4.4-rc.12

package/core/src/schema-defaults.ts

@@ -47,7 +47,17 @@ import { Database } from "bun:sqlite";
 // ---------------------------------------------------------------------------
 
 export interface SchemaField {
-  type?: "string" | "number" | "boolean" | "array" | "object";
+  /**
+   * Declared type for the field's metadata value. `"integer"` is distinct
+   * from `"number"` only at validation time — JSON has no separate integer
+   * type, so a JSON number with zero fractional part (`5`, `5.0`,
+   * `Number.isInteger(n) === true`) is accepted as integer and a non-zero
+   * fractional value (`5.5`) is rejected. This matches the indexed-fields
+   * `"integer"` storage type (TYPE_MAP) and removes the false-positive
+   * `type_mismatch` warning that previously fired on every integer-shaped
+   * field because the validator had no `"integer"` case. See vault#310.
+   */
+  type?: "string" | "number" | "integer" | "boolean" | "array" | "object";
   enum?: string[];
   description?: string;
 }
@@ -270,6 +280,14 @@ function valueMatchesType(value: unknown, type: SchemaField["type"]): boolean {
       return typeof value === "string";
     case "number":
       return typeof value === "number" && Number.isFinite(value);
+    case "integer":
+      // JSON has no separate integer type — `5.0` and `5` decode to the
+      // same JS Number. Accept any finite Number whose fractional part is
+      // zero; reject `5.5`, `NaN`, `Infinity`, and non-Number types.
+      // vault#310 (Gitcoin Brain drift detector emits JSON for diffs;
+      // without this case, every integer-typed field warned
+      // `type_mismatch` and buried the real warnings).
+      return typeof value === "number" && Number.isInteger(value);
     case "boolean":
       return typeof value === "boolean";
     case "array":

package/core/src/store.ts

@@ -202,6 +202,19 @@ export class BunSqliteStore implements Store {
     }
   }
 
+  async restoreNoteTimestamps(id: string, createdAt: string, updatedAt: string): Promise<void> {
+    // Import-only: direct UPDATE so the importer can restore a note's
+    // historical `created_at`/`updated_at` from the portable-md export
+    // bytes. `updateNote` either bumps `updated_at` to wall-clock-now or
+    // (with `skipUpdatedAt: true`) leaves it untouched — neither lets
+    // the importer write a specific historical timestamp. Skips hooks
+    // by design: this isn't a user-edit, it's a state restoration.
+    // See vault#308 PR 2.
+    this.db
+      .prepare("UPDATE notes SET created_at = ?, updated_at = ? WHERE id = ?")
+      .run(createdAt, updatedAt, id);
+  }
+
   async deleteNote(id: string): Promise<void> {
     // Read before delete so we can invalidate config caches on the way out.
     const existing = noteOps.getNote(this.db, id);

package/core/src/types.ts

@@ -143,6 +143,21 @@ export interface Store {
   getNoteByPath(path: string): Promise<Note | null>;
   getNotes(ids: string[]): Promise<Note[]>;
   updateNote(id: string, updates: { content?: string; append?: string; prepend?: string; path?: string; metadata?: Record<string, unknown>; created_at?: string; skipUpdatedAt?: boolean; if_updated_at?: string }): Promise<Note>;
+  /**
+   * Set a note's `created_at` and `updated_at` explicitly. Import-only:
+   * used by the portable-md round-trip path to restore timestamps from
+   * the export bytes. The regular `updateNote` either bumps `updated_at`
+   * to wall-clock-now or (with `skipUpdatedAt: true`) leaves it
+   * untouched — neither shape lets the importer write a specific
+   * historical timestamp. Bypasses hooks. See vault#308 PR 2.
+   */
+  restoreNoteTimestamps(id: string, createdAt: string, updatedAt: string): Promise<void>;
+  /**
+   * Sync wikilinks for every note in the vault. Cheap O(n) walk; used
+   * after bulk-imports to rebuild link rows from `[[brackets]]` in
+   * content. Returns counts for caller logging.
+   */
+  syncAllWikilinks(): Promise<{ synced: number; totalAdded: number; totalRemoved: number }>;
   deleteNote(id: string): Promise<void>;
   queryNotes(opts: QueryOpts): Promise<Note[]>;
   searchNotes(query: string, opts?: { tags?: string[]; limit?: number }): Promise<Note[]>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openparachute/vault",
-  "version": "0.4.3",
+  "version": "0.4.4-rc.12",
   "description": "Agent-native knowledge graph. Notes, tags, links over MCP.",
   "module": "src/cli.ts",
   "type": "module",