nucleus-core-ts 0.9.790 → 0.9.792

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

@@ -2,7 +2,7 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useGSAP } from '@gsap/react';
 import gsap from 'gsap';
-import { useEffect, useEffectEvent, useRef } from 'react';
+import { useEffect, useEffectEvent, useRef, useState } from 'react';
 import { cn } from '../../../utils/cn';
 import { useIntegrationsStore } from '../store';
 import { integrationsPageTheme } from '../theme';
@@ -26,6 +26,19 @@ export function IntegrationsPage({ title = 'Integrations', subtitle = 'Connect a
     const store = useIntegrationsStore();
     const containerRef = useRef(null);
     const contentRef = useRef(null);
+    /**
+   * Bumped after a connection is written, to pull the rail again.
+   *
+   * `ConnectionList` and `ConnectionSettingsTab` were both built to be told
+   * about a save and neither was ever told: the shell mounted them with only
+   * `actions`. Creating the first connection therefore selected a record the
+   * rail had never fetched, so the rail kept saying "0 connections" while the
+   * 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 = ()=>{
+        setSourcesReloadToken((token)=>token + 1);
+    };
     // One measured entrance. Anything longer makes an operator wait to read counts
     // they came here to check.
     useGSAP(()=>{
@@ -114,7 +127,8 @@ export function IntegrationsPage({ title = 'Integrations', subtitle = 'Connect a
                         "aria-label": "Connections",
                         className: theme.layout.aside,
                         children: /*#__PURE__*/ _jsx(ConnectionList, {
-                            actions: actions
+                            actions: actions,
+                            reloadToken: sourcesReloadToken
                         })
                     }),
                     /*#__PURE__*/ _jsx("section", {
@@ -138,7 +152,8 @@ export function IntegrationsPage({ title = 'Integrations', subtitle = 'Connect a
                                     role: "tabpanel",
                                     children: [
                                         tab === 'connection' ? /*#__PURE__*/ _jsx(ConnectionSettingsTab, {
-                                            actions: actions
+                                            actions: actions,
+                                            onSaved: handleSourceSaved
                                         }) : null,
                                         tab === 'endpoints' ? /*#__PURE__*/ _jsx(EndpointsTab, {
                                             actions: actions
@@ -163,7 +178,8 @@ export function IntegrationsPage({ title = 'Integrations', subtitle = 'Connect a
                                     children: "Select a connection on the left, or set up a new one here."
                                 }),
                                 /*#__PURE__*/ _jsx(ConnectionSettingsTab, {
-                                    actions: actions
+                                    actions: actions,
+                                    onSaved: handleSourceSaved
                                 })
                             ]
                         })

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

@@ -2,6 +2,7 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useId } from 'react';
 import { integrationsPageTheme as theme } from '../theme';
+import { endpointLabel } from './formSupport';
 import { Field } from './Primitives';
 export function ParamBindingRow({ binding, onChange, onRemove, siblings, entities, disabled }) {
     const rowId = useId();
@@ -107,7 +108,7 @@ export function ParamBindingRow({ binding, onChange, onRemove, siblings, entitie
                         value: binding.endpointId,
                         children: siblings.map((sibling)=>/*#__PURE__*/ _jsx("option", {
                                 value: sibling.id,
-                                children: sibling.tag
+                                children: endpointLabel(sibling)
                             }, sibling.id))
                     })
                 }),

package/dist/fe/components/IntegrationsPage/components/formSupport.d.ts

@@ -1,6 +1,17 @@
 /**
- * Two things every form in this panel needs and neither of them deserves a file.
+ * The small things every form in this panel needs and none of them deserves a file.
  */
+import type { IntegrationEndpoint } from '../types';
+/**
+ * How an endpoint should read in a list someone has to choose from.
+ *
+ * The parameter-binding picker showed `tag`, which groups the endpoints of one
+ * API — so a source whose endpoints are all tagged `corporate-portal` offered
+ * three identical options and no way to tell which one supplied the value.
+ * The name is what distinguishes them; the path is what distinguishes them when
+ * nobody typed a name, since it is the one field that cannot be blank.
+ */
+export declare function endpointLabel(endpoint: IntegrationEndpoint): string;
 /**
  * The ids a control should point `aria-describedby` at.
  *

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

@@ -1,6 +1,21 @@
 /**
- * Two things every form in this panel needs and neither of them deserves a file.
+ * The small things every form in this panel needs and none of them deserves a file.
  */ /**
+ * How an endpoint should read in a list someone has to choose from.
+ *
+ * The parameter-binding picker showed `tag`, which groups the endpoints of one
+ * API — so a source whose endpoints are all tagged `corporate-portal` offered
+ * three identical options and no way to tell which one supplied the value.
+ * The name is what distinguishes them; the path is what distinguishes them when
+ * nobody typed a name, since it is the one field that cannot be blank.
+ */ export function endpointLabel(endpoint) {
+    const name = endpoint.name?.trim();
+    if (name) return name;
+    const path = endpoint.path?.trim();
+    if (path) return `${endpoint.method} ${path}`;
+    return endpoint.tag;
+}
+/**
  * The ids a control should point `aria-describedby` at.
  *
  * {@link Field} renders its hint as `<id>-hint` and its error as `<id>-error`,

package/dist/fe/components/IntegrationsPage/components/formSupport.test.d.ts

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

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

@@ -0,0 +1,59 @@
+import { describe, expect, test } from 'bun:test';
+import { describedBy, endpointLabel, errorText } from './formSupport';
+const endpoint = (over)=>({
+        id: 'e1',
+        sourceId: 's1',
+        tag: 'corporate-portal',
+        method: 'GET',
+        path: '/api/v1/ext/corporate-portal/company',
+        ...over
+    });
+/**
+ * Seen while wiring a chained import against a real API: every endpoint of one
+ * service shares a tag, and the binding picker labelled its options with that
+ * tag — three entries reading "corporate-portal" and no way to tell which one
+ * feeds the parameter.
+ */ describe('endpointLabel', ()=>{
+    test('the name is what distinguishes one endpoint from its siblings', ()=>{
+        expect(endpointLabel(endpoint({
+            name: 'Şirketler'
+        }))).toBe('Şirketler');
+    });
+    test('without a name, the method and path do — never the shared tag', ()=>{
+        const label = endpointLabel(endpoint({
+            name: null
+        }));
+        expect(label).toBe('GET /api/v1/ext/corporate-portal/company');
+        expect(label).not.toBe('corporate-portal');
+    });
+    test('a name of only spaces is not a name', ()=>{
+        expect(endpointLabel(endpoint({
+            name: '   '
+        }))).toBe('GET /api/v1/ext/corporate-portal/company');
+    });
+    test('the tag is the last resort, not the first', ()=>{
+        expect(endpointLabel(endpoint({
+            name: null,
+            path: ''
+        }))).toBe('corporate-portal');
+    });
+});
+describe('describedBy', ()=>{
+    test('names only the ids that were rendered', ()=>{
+        expect(describedBy('f', true, true)).toBe('f-hint f-error');
+        expect(describedBy('f', true, false)).toBe('f-hint');
+        expect(describedBy('f', false, true)).toBe('f-error');
+        expect(describedBy('f', false, false)).toBeUndefined();
+    });
+});
+describe('errorText', ()=>{
+    test("the server's own wording is kept, because only it knows what was wrong", ()=>{
+        expect(errorText(new Error('Token request failed'), 'fallback')).toBe('Token request failed');
+        expect(errorText('plain reason', 'fallback')).toBe('plain reason');
+    });
+    test('an empty or unrecognised reason falls back rather than saying nothing', ()=>{
+        expect(errorText(new Error(''), 'fallback')).toBe('fallback');
+        expect(errorText(undefined, 'fallback')).toBe('fallback');
+        expect(errorText({}, 'fallback')).toBe('fallback');
+    });
+});

package/package.json

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