@magicx-eng/ai-autocomplete-angular 0.5.7 → 0.6.0

package/fesm2020/magicx-eng-ai-autocomplete-angular.mjs

@@ -1,9 +1,10 @@
+import { buildAttributionUrl, getFooterHint, AIAutocomplete, getCursorOffset, extractPlainText, renderEditableContent, plainTextLength, setCursorOffset, ModeController, buildSubmitResult } from '@magicx-eng/ai-autocomplete-vanilla';
+export { buildSubmitResult, withSkippedParams } from '@magicx-eng/ai-autocomplete-vanilla';
 import * as i1 from '@angular/common';
 import { CommonModule, isPlatformBrowser } from '@angular/common';
 import * as i0 from '@angular/core';
 import { Component, ChangeDetectionStrategy, ViewEncapsulation, Input, EventEmitter, Output, ViewChild, inject, ChangeDetectorRef, ElementRef, Renderer2, NgZone, Directive, PLATFORM_ID, forwardRef, NgModule } from '@angular/core';
 import { NG_VALUE_ACCESSOR } from '@angular/forms';
-import { buildAttributionUrl, getFooterHint, AIAutocomplete, getCursorOffset, extractPlainText, renderEditableContent, plainTextLength, setCursorOffset, ModeController, buildQuery } from '@magicx-eng/ai-autocomplete-vanilla';
 import { BehaviorSubject, combineLatest, map as map$1, distinctUntilChanged, shareReplay, Subject, takeUntil } from 'rxjs';
 import { map } from 'rxjs/operators';
 
@@ -579,16 +580,21 @@ class AIAutocompleteController {
         // handed. The core notifies listeners synchronously, and some actions
         // trigger reentrant `store.set` calls (e.g. typing a non-matching char
         // during re-edit makes the core synchronously exit re-edit and kick off a
-        // fetch). Reentrancy makes the outer set's listener loop continue with a
-        // now-stale snapshot, so the listener can fire *out of order* — delivering
-        // an older state (still in re-edit) after a newer one (re-edit exited).
-        // Reading `getState()` at emission time collapses every notification onto
-        // the true latest state, so a stale `editingParam` can't resurrect and
-        // retrigger the editor's re-edit caret-park (which would yank the caret
-        // back to where re-editing started). distinctUntilChanged downstream drops
-        // the resulting duplicate emissions. React avoids this because it batches
-        // renders and its caret effects are dependency-gated; our subscription is
-        // synchronous, so we coalesce here instead.
+        // fetch).
+        //
+        // The core's store used to deliver such a nested set inline and then
+        // resume the outer listener loop, so a listener could be handed an older
+        // state (still in re-edit) *after* a newer one (re-edit exited) — which
+        // resurrected a stale `editingParam` and retriggered the editor's re-edit
+        // caret-park, yanking the caret back to where re-editing started. The
+        // store now queues notifications and drains them in order, so that
+        // out-of-order delivery can no longer happen at the source.
+        //
+        // Reading `getState()` at emission time is kept as belt-and-braces: it
+        // costs nothing (distinctUntilChanged downstream drops the duplicate
+        // emissions) and keeps this bridge correct for any future consumer that
+        // emits mid-cascade. React needs neither, since it batches renders and
+        // gates its caret effects on dependencies.
         this.unsubscribeCore = this.core.subscribe(() => {
             if (!this.isDestroyed)
                 this.rawState$.next(this.core.getState());
@@ -602,6 +608,7 @@ class AIAutocompleteController {
         // works fine.
         this.text$ = deriveField(this.rawState$, (s) => s.text);
         this.completedParams$ = deriveField(this.rawState$, (s) => s.completedParams, arrEq);
+        this.skippedParams$ = deriveField(this.rawState$, (s) => s.skippedParams, arrEq);
         this.suggestionPills$ = deriveField(this.rawState$, (s) => s.actionableSuggestions, arrEq);
         this.segments$ = deriveField(this.rawState$, (s) => s.segments, arrEq);
         this.newParamId$ = deriveField(this.rawState$, (s) => s.newParamId);
@@ -809,6 +816,7 @@ function computeState(s) {
     return {
         text: s.text,
         completedParams: s.completedParams,
+        skippedParams: s.skippedParams,
         suggestionPills: s.actionableSuggestions,
         segments: s.segments,
         newParamId: s.newParamId,
@@ -1388,13 +1396,9 @@ class AIAutocompleteComponent {
         if (!editor)
             return;
         const text = extractPlainText(editor);
-        const params = this.completedParams ?? this.controller.getState().completedParams;
-        const built = buildQuery(text, params);
-        this.submitted.emit({
-            query: text.trim(),
-            raw_query: built.rawQuery,
-            completed_params: built.completedParams,
-        });
+        const state = this.controller.getState();
+        const params = this.completedParams ?? state.completedParams;
+        this.submitted.emit(buildSubmitResult(text, params, state.skippedParams));
         this.controller.reset();
     }
     handleSubmitFromUserAction() {
@@ -1811,7 +1815,7 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "15.2.10", ngImpo
  * planned export list. Only symbols re-exported here are considered part of
  * the package's public contract (TDB §19.2).
  */
-// === Tier 1: Full component ===
+// === Submit-result builder (folds skipped params into completed_params) ===
 
 /**
  * Generated bundle index. Do not edit.