nucleus-core-ts 0.9.791 → 0.9.792

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.791",
+	"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",