memorix 1.6.2 → 1.6.3

package/src/store/obs-store.ts

@@ -59,7 +59,7 @@ export interface ObservationStore {
   /** Load one project's observations without scanning the shared flat store. */
   loadByProject(
     projectId: string,
-    options?: { status?: string; limit?: number; offset?: number; afterId?: number },
+    options?: { status?: string; limit?: number; offset?: number; afterId?: number; newestFirst?: boolean },
   ): Promise<Observation[]>;
 
   /** Count one project's observations without materializing the rows. */
@@ -231,7 +231,7 @@ export class DegradedBackend implements ObservationStore {
     return [];
   }
 
-  async loadByProject(_projectId: string, _options?: { status?: string; limit?: number; offset?: number; afterId?: number }): Promise<Observation[]> {
+  async loadByProject(_projectId: string, _options?: { status?: string; limit?: number; offset?: number; afterId?: number; newestFirst?: boolean }): Promise<Observation[]> {
     return [];
   }
 

package/src/store/sqlite-db.ts

@@ -16,6 +16,7 @@ import { createRequire } from 'node:module';
 import path from 'node:path';
 import fs from 'node:fs';
 import { createDatabase, loadSqlite } from './bun-sqlite-compat.js';
+import { assertNotHomeDataDir } from '../project/launch-root.js';
 
 // Dynamic require for SQLite (better-sqlite3, node:sqlite, or bun:sqlite)
 let BetterSqlite3: any;
@@ -998,6 +999,7 @@ const _dbCache = new Map<string, any>();
  */
 export function getDatabase(dataDir: string): any {
   const normalized = path.resolve(dataDir);
+  assertNotHomeDataDir(normalized);
   const existing = _dbCache.get(normalized);
   if (existing) return existing;
 

package/src/store/sqlite-store.ts

@@ -122,6 +122,8 @@ export class SqliteBackend implements ObservationStore {
   private stmtSelectByProjectStatus: any = null;
   private stmtSelectByProjectAfterId: any = null;
   private stmtSelectByProjectStatusAfterId: any = null;
+  private stmtSelectByProjectNewest: any = null;
+  private stmtSelectByProjectStatusNewest: any = null;
   private stmtCountByProject: any = null;
   private stmtCountByProjectStatus: any = null;
   private stmtSelectByTopicKey: any = null;
@@ -175,6 +177,14 @@ export class SqliteBackend implements ObservationStore {
       SELECT * FROM observations WHERE projectId = ? AND status = ? AND id > ?
       ORDER BY id ASC LIMIT ?
     `);
+    this.stmtSelectByProjectNewest = this.db.prepare(`
+      SELECT * FROM observations WHERE projectId = ?
+      ORDER BY createdAt DESC, id DESC LIMIT ? OFFSET ?
+    `);
+    this.stmtSelectByProjectStatusNewest = this.db.prepare(`
+      SELECT * FROM observations WHERE projectId = ? AND status = ?
+      ORDER BY createdAt DESC, id DESC LIMIT ? OFFSET ?
+    `);
     this.stmtCountByProject = this.db.prepare(
       `SELECT COUNT(*) AS count FROM observations WHERE projectId = ?`,
     );
@@ -288,15 +298,20 @@ export class SqliteBackend implements ObservationStore {
 
   async loadByProject(
     projectId: string,
-    options: { status?: string; limit?: number; offset?: number; afterId?: number } = {},
+    options: { status?: string; limit?: number; offset?: number; afterId?: number; newestFirst?: boolean } = {},
   ): Promise<Observation[]> {
     const limit = options.limit == null ? -1 : Math.max(1, Math.floor(options.limit));
     const offset = options.offset == null ? 0 : Math.max(0, Math.floor(options.offset));
     const afterId = options.afterId == null ? null : Math.max(0, Math.floor(options.afterId));
+    const newestFirst = options.newestFirst === true && afterId === null;
     const rows = afterId === null
-      ? (options.status
-        ? this.stmtSelectByProjectStatus.all(projectId, options.status, limit, offset)
-        : this.stmtSelectByProject.all(projectId, limit, offset))
+      ? (newestFirst
+        ? (options.status
+          ? this.stmtSelectByProjectStatusNewest.all(projectId, options.status, limit, offset)
+          : this.stmtSelectByProjectNewest.all(projectId, limit, offset))
+        : (options.status
+          ? this.stmtSelectByProjectStatus.all(projectId, options.status, limit, offset)
+          : this.stmtSelectByProject.all(projectId, limit, offset)))
       : (options.status
         ? this.stmtSelectByProjectStatusAfterId.all(projectId, options.status, afterId, limit)
         : this.stmtSelectByProjectAfterId.all(projectId, afterId, limit));