@drawbridge/drawbridge-utils 0.0.173 → 0.0.175

package/dist/providers.d.cts

@@ -74,10 +74,10 @@ const providerFields = ( slug ) => (
 
 // The vendors an admin can configure, in a stable order.
 //
-// DERIVED FROM WHO DECLARES A VENDOR, not from the catalog: `webhook` is a
-// connection with no vendor behind it — nothing to authenticate against and
-// nothing to type — so it must not appear on a screen whose whole purpose is
-// entering credentials.
+// FROM THE VENDOR DIRECTORY, not from the connection catalog. It used to be
+// derived from which manifests declared a vendor, which meant a vendor could not
+// exist without a connection — Stripe needs credentials and has no manifest. A
+// connection with no vendor behind it, like `webhook`, simply names none.
 const providerSlugs = () => Object.keys( vendors ).sort();
 
 // LIVE IS DERIVED, never stored. A stored flag is a second thing to keep in
@@ -236,7 +236,7 @@ const vendorEnabled = async ({ controller, vendor }) => (
 // lookups inside the window.
 const vendorSettings = async ({ controller, slug }) => {
 
-	const declared = Object.hasOwn( connections, slug ) ? Object.keys( connections[ slug ].provider?.vendors || {} ) : [];
+	const declared = Object.hasOwn( connections, slug ) ? ( connections[ slug ].vendors || [] ) : [];
 
 	let merged = {};
 
@@ -262,7 +262,7 @@ const vendorSettings = async ({ controller, slug }) => {
 // three, or the two would disagree about which vendors a connection depends on.
 const vendorsEnabled = async ({ controller, slug }) => {
 
-	const declared = Object.hasOwn( connections, slug ) ? Object.keys( connections[ slug ].provider?.vendors || {} ) : [];
+	const declared = Object.hasOwn( connections, slug ) ? ( connections[ slug ].vendors || [] ) : [];
 
 	for( const vendor of declared ){
 

package/dist/providers.d.ts

@@ -74,10 +74,10 @@ const providerFields = ( slug ) => (
 
 // The vendors an admin can configure, in a stable order.
 //
-// DERIVED FROM WHO DECLARES A VENDOR, not from the catalog: `webhook` is a
-// connection with no vendor behind it — nothing to authenticate against and
-// nothing to type — so it must not appear on a screen whose whole purpose is
-// entering credentials.
+// FROM THE VENDOR DIRECTORY, not from the connection catalog. It used to be
+// derived from which manifests declared a vendor, which meant a vendor could not
+// exist without a connection — Stripe needs credentials and has no manifest. A
+// connection with no vendor behind it, like `webhook`, simply names none.
 const providerSlugs = () => Object.keys( vendors ).sort();
 
 // LIVE IS DERIVED, never stored. A stored flag is a second thing to keep in
@@ -236,7 +236,7 @@ const vendorEnabled = async ({ controller, vendor }) => (
 // lookups inside the window.
 const vendorSettings = async ({ controller, slug }) => {
 
-	const declared = Object.hasOwn( connections, slug ) ? Object.keys( connections[ slug ].provider?.vendors || {} ) : [];
+	const declared = Object.hasOwn( connections, slug ) ? ( connections[ slug ].vendors || [] ) : [];
 
 	let merged = {};
 
@@ -262,7 +262,7 @@ const vendorSettings = async ({ controller, slug }) => {
 // three, or the two would disagree about which vendors a connection depends on.
 const vendorsEnabled = async ({ controller, slug }) => {
 
-	const declared = Object.hasOwn( connections, slug ) ? Object.keys( connections[ slug ].provider?.vendors || {} ) : [];
+	const declared = Object.hasOwn( connections, slug ) ? ( connections[ slug ].vendors || [] ) : [];
 
 	for( const vendor of declared ){