nucleus-core-ts 0.9.846 → 0.9.847

package/dist/fe/components/IntegrationsPage/components/ConnectionDelete.js

@@ -25,6 +25,7 @@ export function ConnectionDelete({ actions, source, onDeleted }) {
             }
             setAsking(false);
             setTyped('');
+            store.removeSource(source.id);
             store.selectSource(null);
             onDeleted?.();
         }).catch((error)=>{

package/dist/fe/components/IntegrationsPage/components/IntegrationsPage.js

@@ -43,7 +43,12 @@ export function IntegrationsPage({ title = 'Integrations', subtitle = 'Connect a
    * detail treated the missing record as still loading and sat on skeletons
    * until the page was reloaded by hand. The write itself had succeeded.
    */ const [sourcesReloadToken, setSourcesReloadToken] = useState(0);
-    const handleSourceSaved = ()=>{
+    const handleSourceSaved = (saved)=>{
+        // Into the store FIRST. Bumping the token only tells the rail to refetch,
+        // and choosing a connection unmounts the rail — so on a NEW connection the
+        // token reached nobody and the detail below could not find the record it
+        // had just been switched to.
+        store.upsertSource(saved);
         setSourcesReloadToken((token)=>token + 1);
     };
     /**

package/dist/fe/components/IntegrationsPage/store/index.js

@@ -1,12 +1,28 @@
 'use client';
 import { batch, createStore } from 'h-state';
 import { initial } from './state';
+import { isNewTo, upsertInto } from './upsertInto';
 export const { useStore: useIntegrationsStore } = createStore(initial, {
     setSources: (store)=>(items, total)=>batch(()=>{
                 store.sources = items;
                 store.sourcesTotal = total;
                 store.sourcesLoading = false;
             }),
+    upsertSource: (store)=>(source)=>batch(()=>{
+                // The count moves only for an addition — an edit that bumped the total
+                // would make the rail claim a connection nobody can find.
+                if (isNewTo(store.sources, source)) store.sourcesTotal = store.sourcesTotal + 1;
+                store.sources = upsertInto(store.sources, source);
+            }),
+    removeSource: (store)=>(id)=>batch(()=>{
+                const kept = store.sources.filter((candidate)=>candidate.id !== id);
+                // Only when it was actually on this page — the count must not fall for a
+                // record the list never held.
+                if (kept.length !== store.sources.length) {
+                    store.sourcesTotal = Math.max(0, store.sourcesTotal - 1);
+                    store.sources = kept;
+                }
+            }),
     setSourcesSearch: (store)=>(search)=>batch(()=>{
                 store.sourcesSearch = search;
                 // A new search is a new result set; staying on page 4 would show an empty

package/dist/fe/components/IntegrationsPage/store/state.d.ts

@@ -58,6 +58,18 @@ export type State = {
 export type Actions = {
     [key: string]: unknown;
     setSources: (items: IntegrationSource[], total: number) => void;
+    /**
+     * Put one connection into the list, whether or not it is already there.
+     *
+     * The rail is what fetches connections, and choosing one UNMOUNTS the rail —
+     * so a connection created from the form was selected into a list that had
+     * never heard of it, and every panel below sat on skeletons until the page
+     * was reloaded by hand. The save already returns the record; this is where it
+     * goes so the detail can find it.
+     */
+    upsertSource: (source: IntegrationSource) => void;
+    /** Drop one connection from the list, for when it has just been deleted. */
+    removeSource: (id: string) => void;
     setSourcesSearch: (search: string) => void;
     setSourcesLoading: (loading: boolean) => void;
     selectSource: (id: string | null) => void;

package/dist/fe/components/IntegrationsPage/store/upsertInto.d.ts

@@ -0,0 +1,20 @@
+/**
+ * Where a saved record goes in a list that may or may not already hold it.
+ *
+ * Kept as a function so the rule can be tested apart from the store: the panel
+ * unmounts the connection rail the moment a connection is chosen, so a newly
+ * created connection is selected into a list that never fetched it. Putting the
+ * returned record in by hand is what stops the detail below from waiting for a
+ * fetch that no mounted component will make.
+ *
+ * A new record goes to the FRONT — it is the one just made, and the operator is
+ * about to be switched to it; an existing one is replaced where it already sits,
+ * so an edit never reorders a list somebody is reading.
+ */
+export declare function upsertInto<T extends {
+    id: string;
+}>(items: T[], record: T): T[];
+/** Whether `record` would be an addition rather than a replacement. */
+export declare function isNewTo<T extends {
+    id: string;
+}>(items: T[], record: T): boolean;

package/dist/fe/components/IntegrationsPage/store/upsertInto.js

@@ -0,0 +1,21 @@
+/**
+ * Where a saved record goes in a list that may or may not already hold it.
+ *
+ * Kept as a function so the rule can be tested apart from the store: the panel
+ * unmounts the connection rail the moment a connection is chosen, so a newly
+ * created connection is selected into a list that never fetched it. Putting the
+ * returned record in by hand is what stops the detail below from waiting for a
+ * fetch that no mounted component will make.
+ *
+ * A new record goes to the FRONT — it is the one just made, and the operator is
+ * about to be switched to it; an existing one is replaced where it already sits,
+ * so an edit never reorders a list somebody is reading.
+ */ export function upsertInto(items, record) {
+    return items.some((candidate)=>candidate.id === record.id) ? items.map((candidate)=>candidate.id === record.id ? record : candidate) : [
+        record,
+        ...items
+    ];
+}
+/** Whether `record` would be an addition rather than a replacement. */ export function isNewTo(items, record) {
+    return !items.some((candidate)=>candidate.id === record.id);
+}

package/dist/fe/components/IntegrationsPage/store/upsertInto.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/fe/components/IntegrationsPage/store/upsertInto.test.js

@@ -0,0 +1,67 @@
+import { describe, expect, it } from 'bun:test';
+import { isNewTo, upsertInto } from './upsertInto';
+const a = {
+    id: 'a',
+    name: 'First'
+};
+const b = {
+    id: 'b',
+    name: 'Second'
+};
+describe('upsertInto', ()=>{
+    it('puts a new record at the front', ()=>{
+        expect(upsertInto([
+            a
+        ], b)).toEqual([
+            b,
+            a
+        ]);
+    });
+    it('adds to an empty list', ()=>{
+        expect(upsertInto([], a)).toEqual([
+            a
+        ]);
+    });
+    it('replaces an existing record where it already sits', ()=>{
+        const renamed = {
+            id: 'a',
+            name: 'Renamed'
+        };
+        expect(upsertInto([
+            b,
+            a
+        ], renamed)).toEqual([
+            b,
+            renamed
+        ]);
+    });
+    it('never duplicates — saving the same record twice leaves one', ()=>{
+        const once = upsertInto([], a);
+        expect(upsertInto(once, {
+            id: 'a',
+            name: 'Again'
+        })).toHaveLength(1);
+    });
+    it('leaves the original array untouched', ()=>{
+        const items = [
+            a
+        ];
+        upsertInto(items, b);
+        expect(items).toEqual([
+            a
+        ]);
+    });
+});
+describe('isNewTo', ()=>{
+    it('is true only for a record the list does not hold', ()=>{
+        expect(isNewTo([
+            a
+        ], b)).toBe(true);
+        expect(isNewTo([
+            a
+        ], {
+            id: 'a',
+            name: 'Renamed'
+        })).toBe(false);
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "nucleus-core-ts",
-	"version": "0.9.846",
+	"version": "0.9.847",
 	"description": "Production-ready, enterprise-grade TypeScript framework for building multi-tenant APIs",
 	"author": "Hidayet Can Özcan <hidayetcan@gmail.com>",
 	"license": "SEE LICENSE IN LICENSE",