@ethlete/query 6.0.0-next.44 → 6.0.0-next.46

package/generators/migrate-to-query-v3/cleanup-migration.js

@@ -10,7 +10,7 @@ import { createSourceFile, ensureImportFromQuery, ensureNamedImports, getLineNum
  */
 export const migrateDevtoolsUsage = (tree, scope, report) => {
     const updatedFiles = [];
-    const filesNeedingComponentsPackage = [];
+    const filesNeedingDevtoolsPackage = [];
     scope.visit(tree, (filePath) => {
         if (!filePath.endsWith('.ts')) {
             return;
@@ -27,7 +27,7 @@ export const migrateDevtoolsUsage = (tree, scope, report) => {
         tree.write(filePath, nextContent);
         updatedFiles.push(filePath);
         if (importsComponent) {
-            filesNeedingComponentsPackage.push(filePath);
+            filesNeedingDevtoolsPackage.push(filePath);
         }
     });
     if (updatedFiles.length === 0) {
@@ -35,16 +35,17 @@ export const migrateDevtoolsUsage = (tree, scope, report) => {
     }
     console.log('\n✅ Migrated query devtools usage to v3 in:');
     updatedFiles.forEach((filePath) => console.log(`   - ${filePath}`));
-    if (filesNeedingComponentsPackage.length > 0) {
+    if (filesNeedingDevtoolsPackage.length > 0) {
         // The component moved packages, which is the one part of this rewrite that can fail outside the
-        // file being edited: an app that only ever depended on `@ethlete/query` now needs `@ethlete/components`.
+        // file being edited: an app that only ever depended on `@ethlete/query` now needs
+        // `@ethlete/query-devtools`.
         report.addManualReview({
-            title: 'Add @ethlete/components for the query devtools',
-            summary: '`QueryDevtoolsComponent` now lives in `@ethlete/components`. The imports were rewritten, but the package may not be a dependency of these projects yet.',
-            action: 'Run `yarn add @ethlete/components` where needed, then `yarn install` and re-lint the affected libs so the `@nx/dependency-checks` rule sees the new dependency.',
-            locations: filesNeedingComponentsPackage.map((filePath) => ({ filePath })),
+            title: 'Add @ethlete/query-devtools for the query devtools',
+            summary: '`QueryDevtoolsComponent` now lives in `@ethlete/query-devtools`. The imports were rewritten, but the package may not be a dependency of these projects yet.',
+            action: 'Run `yarn add @ethlete/query-devtools` where needed, then `yarn install` and re-lint the affected libs so the `@nx/dependency-checks` rule sees the new dependency.',
+            locations: filesNeedingDevtoolsPackage.map((filePath) => ({ filePath })),
             source: 'cleanup-migration',
-            dedupeKey: 'devtools-components-dependency',
+            dedupeKey: 'devtools-package-dependency',
         });
     }
 };
@@ -100,7 +101,7 @@ const migrateDevtoolsInFile = (content) => {
         result = ensureNamedImports({
             content: result,
             importsNeeded: ['QueryDevtoolsComponent'],
-            moduleSpecifier: '@ethlete/components',
+            moduleSpecifier: '@ethlete/query-devtools',
         });
     }
     return { content: result, importsComponent };

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@ethlete/query",
-  "version": "6.0.0-next.44",
+  "version": "6.0.0-next.46",
   "license": "MIT",
   "sideEffects": false,
   "peerDependencies": {
     "@analogjs/vite-plugin-angular": "2.6.3",
-    "@angular/common": "22.0.7",
-    "@angular/core": "22.0.7",
-    "@angular/forms": "22.0.7",
-    "@angular/router": "22.0.7",
+    "@angular/common": "22.1.6",
+    "@angular/core": "22.1.6",
+    "@angular/forms": "22.1.6",
+    "@angular/router": "22.1.6",
     "@ethlete/core": "^5.0.0-beta.11",
     "@ethlete/types": "^2.0.0-beta.4",
     "@nx/devkit": "23.1.0",

package/types/ethlete-query-testing.d.ts

@@ -15,6 +15,11 @@ type QueryTestSetup = {
     createPut: ReturnType<typeof createPutQuery>;
     createPatch: ReturnType<typeof createPatchQuery>;
     createDelete: ReturnType<typeof createDeleteQuery>;
+    /**
+     * Puts back the `console.warn` / `console.error` that were installed before the first
+     * {@link setupQueryTest} call. Idempotent, and safe to call from any of the setups a file created.
+     */
+    restoreConsole: () => void;
 };
 type QueryTestSetupConfig = {
     baseUrl?: string;
@@ -235,6 +240,8 @@ type FakeQueryPersistenceStoreHandle = {
     calls: () => Record<'loadIndex' | 'read' | 'write' | 'remove' | 'clear', number>;
     /** Makes the next `count` writes reject - a full disk, by default. */
     failNextWrites: (count: number, error?: unknown) => void;
+    /** Makes the next `count` removals reject. */
+    failNextRemoves: (count: number, error?: unknown) => void;
     /** Makes the next `loadIndex` reject. */
     failNextLoadIndex: (error?: unknown) => void;
     /**
@@ -245,6 +252,14 @@ type FakeQueryPersistenceStoreHandle = {
     deferReads: () => void;
     /** Settles the reads held by {@link deferReads} and stops deferring. */
     flushReads: () => Promise<void>;
+    /**
+     * Holds every subsequent `loadIndex` pending until {@link flushLoadIndex} is called, for testing
+     * what happens to the store while a client's startup is still reading its index (a real disk takes
+     * milliseconds). The snapshot each held load returns is the one it would have read when it started.
+     */
+    deferLoadIndex: () => void;
+    /** Settles the index loads held by {@link deferLoadIndex} and stops deferring. */
+    flushLoadIndex: () => Promise<void>;
 };
 declare const createFakeQueryPersistenceStore: () => FakeQueryPersistenceStoreHandle;
 
@@ -266,20 +281,37 @@ type WebSocketTestDouble = {
         url: string;
         transports: string[] | undefined;
     } | null;
-    /** Every message the client sent, newest last - room joins and leaves included. */
+    /** The `withCredentials` the client asked the factory for, or `null` until it called it. */
+    withCredentials: () => boolean | null;
+    /** Every message the client emitted, newest last - room joins and leaves included, buffered ones too. */
     sent: () => {
         event: string;
         data: unknown;
     }[];
+    /** Every message that reached the server, newest last. An emit buffered while offline lands here on the next connect. */
+    delivered: () => {
+        event: string;
+        data: unknown;
+    }[];
     /** Whether the client asked the socket to connect, and whether it has since disconnected it. */
     state: () => {
         connectRequested: boolean;
         disconnected: boolean;
     };
-    /** Complete the handshake, so the client's `isConnected` turns true and queued rooms re-join. */
-    serverConnect: () => void;
+    /**
+     * Complete the handshake: flush every buffered emit, then turn the client's `isConnected` true. Pass
+     * `recovered: true` for a connection-state-recovered reconnect, in which the server kept the rooms.
+     */
+    serverConnect: (options?: {
+        recovered?: boolean;
+    }) => void;
     /** Drop the connection, so `isConnected` turns false. */
     serverDisconnect: () => void;
+    /**
+     * Let the ping expire: the socket still reports itself connected, but buffers every emit like socket.io
+     * does until it notices and closes. Follow it with {@link serverDisconnect}.
+     */
+    serverPingExpire: () => void;
     /** Deliver a message the way the server would. */
     serverSend: (message: SocketMessageView) => void;
     /** Deliver a raw frame, for the malformed-payload path. */
@@ -291,8 +323,8 @@ type WebSocketTestDouble = {
  *
  * Nothing here connects on its own: the client's own `connect()` only flips
  * {@link WebSocketTestDouble.state}, and it is {@link WebSocketTestDouble.serverConnect} that fires
- * the `connect` listener. That split is what makes "joined a room while disconnected, and the join
- * was re-sent on connect" testable.
+ * the `connect` listener. That split is what makes "joined a room while disconnected, and it was
+ * re-joined after a reconnect" testable.
  */
 declare const createWebSocketTestDouble: () => WebSocketTestDouble;