@nauticana/sail 0.5.0 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nauticana/sail",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Metadata-driven Angular component library for CRUD admin frontends backed by the keel Go backend. Tables, forms, navigation, OTP/2FA/social auth, billing.",
   "main": "src/index.ts",
   "files": [

package/src/component/table/table_search.ts

@@ -50,7 +50,7 @@ export class TableSearch extends BaseForm implements OnInit {
                 }
             });
         }
-        this.editableRecord = this.emptyRecord();
+        this.editableRecord = this.emptySearchRecord();
         this.searchColumns = this.getDisplayedColumns();
     }
 
@@ -63,7 +63,33 @@ export class TableSearch extends BaseForm implements OnInit {
     }
 
     onClear() {
-        this.editableRecord = this.emptyRecord();
+        this.editableRecord = this.emptySearchRecord();
+    }
+
+    /**
+     * Search-form variant of `emptyRecord()`. Boolean columns are forced to
+     * null so an unmodified Search click does not coerce filters from the
+     * table's HasDefault value (e.g. an `is_primary` column with a `FALSE`
+     * default would otherwise have the form submit `?IsPrimary=false` and
+     * silently exclude every primary-flag row).
+     *
+     * The user opts into a boolean filter by clicking the checkbox: first
+     * click sets it to `true`, a second click sets it to `false`, and the
+     * Clear button resets back to `null` (no filter). `buildSearchTerms`
+     * already skips `null` / `undefined` values, so the URL only carries
+     * filters the user explicitly toggled.
+     */
+    private emptySearchRecord(): any {
+        const rec = this.emptyRecord();
+        const tableDef = this.cacheService.getTableDefinition(this.tableName);
+        if (tableDef?.Columns) {
+            for (const col of tableDef.Columns) {
+                if (col.DataType === 'boolean') {
+                    rec[col.PascalName] = null;
+                }
+            }
+        }
+        return rec;
     }
 
     onAddRecord() {

package/src/service/api-response.interceptor.ts

@@ -1,12 +1,32 @@
 import { HttpErrorResponse, HttpInterceptorFn, HttpResponse } from '@angular/common/http';
 import { catchError, map, throwError } from 'rxjs';
 
+// Detect a final response event by structural shape rather than
+// `event instanceof HttpResponse`. Bundling can produce two distinct
+// HttpResponse class identities — the one captured inside sail's compiled
+// module and the one the host app's HttpClient emits — and instanceof
+// then silently returns false. The structural check (presence of `body`
+// + a callable `clone` + a successful `status`) is robust to that and
+// still correctly skips intermediate events like HttpHeaderResponse and
+// HttpProgressEvent.
+function isFinalResponse(e: unknown): e is HttpResponse<unknown> {
+  if (!e || typeof e !== 'object') return false;
+  const r = e as { type?: number; body?: unknown; clone?: unknown; status?: number };
+  // HttpEventType.Response === 4. Filtering by type is the most reliable
+  // gate (HttpResponse and HttpHeaderResponse have different type values
+  // and only HttpResponse carries `body`).
+  return r.type === 4
+    && 'body' in r
+    && typeof r.clone === 'function'
+    && typeof r.status === 'number';
+}
+
 export const apiResponseInterceptor: HttpInterceptorFn = (req, next) => {
   return next(req).pipe(
     map(event => {
-      if (event instanceof HttpResponse) {
+      if (isFinalResponse(event)) {
         const body = event.body as Record<string, unknown> | null;
-        if (body && 'data' in body) {
+        if (body && typeof body === 'object' && 'data' in body) {
           return event.clone({ body: body['data'] });
         }
       }