@superblocksteam/shared 0.9618.2 → 0.9619.0

package/src/signing/JSON/index.test.ts

@@ -1,7 +1,7 @@
 import { describe, test, expect } from 'vitest';
 
-import { stableJSONStringify, testStringOut, testStringSrc } from '.';
 import { stableJSONStringify_fast } from './fast.js';
+import { type stableJSONStringify, testStringOut, testStringSrc } from './index.js';
 import { stableJSONStringify_portable } from './portable.js';
 
 // an object with keys that are intentionally out of order

package/src/signing/constants.ts

@@ -1,6 +1,6 @@
-import { Api as ApiPb } from '@superblocksteam/types';
+import type { Api as ApiPb } from '@superblocksteam/types';
 
-import { AnyPageDSL, ApplicationConfiguration, ApplicationSettings } from '../types/index.js';
+import type { AnyPageDSL, ApplicationConfiguration, ApplicationSettings } from '../types/index.js';
 
 /** A signature, as produced by the agent. */
 export interface Signature {

package/src/signing/hashing.ts

@@ -1,4 +1,4 @@
-import { BufferSource } from 'node:stream/web';
+import type { BufferSource } from 'node:stream/web';
 
 import { Base64 } from 'js-base64';
 

package/src/signing/index.test.ts

@@ -2,14 +2,14 @@ import { describe, it, expect } from 'vitest';
 
 import { UtilsV1 as UtilsV1Pb } from '@superblocksteam/types';
 
-import { AnyPageDSL, ApplicationConfiguration, ApplicationSettings } from '../types/index.js';
+import type { AnyPageDSL, ApplicationConfiguration, ApplicationSettings } from '../types/index.js';
 import {
-  ApplicationSignatureTreeV1,
-  ApplicationSignatureTreeV2,
+  type ApplicationSignatureTreeV1,
+  type ApplicationSignatureTreeV2,
   createApplicationSignatureTree,
   getAlgorithmName,
   verifyApplicationHashTree
-} from './index';
+} from './index.js';
 
 const pageDsl = {
   version: 8,

package/src/signing/index.ts

@@ -1,11 +1,11 @@
-import { isEqual } from 'lodash';
+import { isEqual } from 'lodash-es';
 
 import { UtilsV1 as UtilsV1Pb } from '@superblocksteam/types';
 
-import { ApplicationConfiguration, ApplicationSettings, IApplicationV2Dto } from '../types/application/index.js';
-import { AnyPageDSL } from '../types/index.js';
+import type { ApplicationConfiguration, ApplicationSettings, IApplicationV2Dto } from '../types/application/index.js';
+import type { AnyPageDSL } from '../types/index.js';
 import unreachable from '../utils/unreachable.js';
-import {
+import type {
   ApplicationSettingsHashes,
   ApplicationSignatureTree,
   ApplicationSignatureTreeUpdate,
@@ -14,9 +14,8 @@ import {
 } from './constants.js';
 import { sha256Base64 } from './hashing.js';
 import { stableJSONStringify } from './JSON/index.js';
-import { getEnumProtoName } from './proto.js';
 
-export * from './constants.js';
+export type * from './constants.js';
 
 /**
  * Returns the SHA-256 hash of the given value, encoded as a base64 string. The hash is stable, meaning that the same
@@ -317,11 +316,15 @@ export async function updateApplicationHashTree({
   }
 }
 
-export function getAlgorithmName(algorithm: UtilsV1Pb.Signature_Algorithm): string {
-  const protoName = getEnumProtoName(UtilsV1Pb.Signature_Algorithm, algorithm);
-  if (protoName) {
-    return protoName;
-  }
+// The enum names come from the generated Signature message rather than from
+// @bufbuild/protobuf. Under Node, @superblocksteam/types loads the CommonJS build of
+// @bufbuild/protobuf and this package loads its ES module build, and enum metadata
+// registered by one build cannot be read by the other.
+function getAlgorithmProtoName(algorithm: UtilsV1Pb.Signature_Algorithm): string | undefined {
+  const field = UtilsV1Pb.Signature.fields.findJsonName('algorithm');
+  return field?.kind === 'enum' ? field.T.findNumber(algorithm)?.name : undefined;
+}
 
-  return getEnumProtoName(UtilsV1Pb.Signature_Algorithm, UtilsV1Pb.Signature_Algorithm.UNSPECIFIED) ?? '';
+export function getAlgorithmName(algorithm: UtilsV1Pb.Signature_Algorithm): string {
+  return getAlgorithmProtoName(algorithm) ?? getAlgorithmProtoName(UtilsV1Pb.Signature_Algorithm.UNSPECIFIED) ?? '';
 }

package/src/socket/protocol.ts

@@ -1,14 +1,14 @@
 // This file contains the definition of the protocol used for communication between the SB server and clients
 
-import {
+import type {
   DirectoryEntry,
   GetManyRequestPayload,
   GetManyResponsePayload,
   PutManyRequestPayload,
   PutManyResponsePayload
 } from '../dbfs/types.js';
-import { ApiToSign, ApiToVerify, AppToSign, AppToVerify, ApplicationSignatureTreeSigned, Signature } from '../signing/constants.js';
-import {
+import type { ApiToSign, ApiToVerify, AppToSign, AppToVerify, ApplicationSignatureTreeSigned, Signature } from '../signing/constants.js';
+import type {
   ApplicationConfiguration,
   ApplicationPageClonePayload,
   CreateApplicationPageResponseBody,
@@ -32,8 +32,8 @@ import {
   LockInfo,
   DegradedMode,
   CheckpointSkipReason
-} from '../types';
-import {
+} from '../types/index.js';
+import type {
   AiChatMessageDto,
   AiChatMessagePayload,
   AiAbandonTurnEvent,
@@ -50,7 +50,7 @@ import {
   NpmInstallBlockedAuditReport,
   NpmInstallOutcomeReport
 } from '../types/index.js';
-import { MethodSchema } from './types.js';
+import type { MethodSchema } from './types.js';
 
 export const serverWsPath = 'v1/rpc-ws';
 

package/src/socket/socket.ts

@@ -1,4 +1,4 @@
-import WebSocket from 'isomorphic-ws';
+import type WebSocket from 'isomorphic-ws';
 
 import {
   SocketErrorException,

package/src/socket/socketWithClientAuth.ts

@@ -1,9 +1,9 @@
-import { Tracer } from '@opentelemetry/api';
-import WebSocket from 'isomorphic-ws';
+import type { Tracer } from '@opentelemetry/api';
+import type WebSocket from 'isomorphic-ws';
 
 import { createISocketClient } from './socket.js';
 import { TracedSocket } from './tracedSocket.js';
-import { GenericMiddleware, ISocketClient, MethodHandlers, RequestContextBase, SocketTimeouts } from './types.js';
+import type { GenericMiddleware, ISocketClient, MethodHandlers, RequestContextBase, SocketTimeouts } from './types.js';
 import { connectWebSocket } from './util.js';
 
 // a subclass of ISocket that sends an auth token on the first request

package/src/socket/tracedSocket.ts

@@ -1,17 +1,17 @@
 import {
-  Attributes,
+  type Attributes,
   ROOT_CONTEXT,
-  Span,
+  type Span,
   SpanKind,
   SpanStatusCode,
-  TimeInput,
+  type TimeInput,
   context as otelCtx,
   propagation,
-  Tracer,
+  type Tracer,
   trace
 } from '@opentelemetry/api';
 import { SEMATTRS_MESSAGING_DESTINATION_KIND, SEMATTRS_MESSAGING_SYSTEM } from '@opentelemetry/semantic-conventions';
-import WebSocket from 'isomorphic-ws';
+import type WebSocket from 'isomorphic-ws';
 
 import {
   OBS_SOCKET_STATUS_CODE,
@@ -23,8 +23,8 @@ import {
   OBS_TAG_HTTP_STATUS_CODE
 } from '../observability/index.js';
 import { sanitizeError } from '../tracing/errorSanitizer.js';
-import { ISocket, SocketError, SocketMessage } from './socket.js';
-import { GenericMiddleware, ISocketClient, MethodHandler, MethodHandlers, RequestContextBase, SocketTimeouts } from './types.js';
+import { ISocket, type SocketError, type SocketMessage } from './socket.js';
+import type { GenericMiddleware, ISocketClient, MethodHandler, MethodHandlers, RequestContextBase, SocketTimeouts } from './types.js';
 
 function isPromise(obj: unknown): obj is Promise<unknown> {
   return obj !== null && typeof obj === 'object' && typeof (obj as Promise<unknown>).then === 'function';

package/src/socket/util.connections.test.ts

@@ -0,0 +1,61 @@
+import { createServer } from 'node:http';
+import type { Duplex } from 'node:stream';
+
+import WebSocket from 'isomorphic-ws';
+import { expect, it, onTestFinished, vi } from 'vitest';
+
+import { connectWebSocket } from './util.js';
+
+it('releases a timed-out handshake so a later connection can use the server slot', async () => {
+  const server = createServer();
+  server.maxConnections = 1;
+  const sockets = new Set<Duplex>();
+  server.on('connection', (socket) => {
+    sockets.add(socket);
+    socket.on('end', () => socket.destroy());
+  });
+  const wss = new WebSocket.Server({ noServer: true });
+  onTestFinished(async () => {
+    vi.useRealTimers();
+    sockets.forEach((socket) => socket.destroy());
+    await Promise.all([
+      new Promise<void>((resolve) => wss.close(() => resolve())),
+      new Promise<void>((resolve) => server.close(() => resolve()))
+    ]);
+  });
+  let markBlocked!: (socket: Duplex) => void;
+  const blocked = new Promise<Duplex>((resolve) => {
+    markBlocked = resolve;
+  });
+  let warming = true;
+  server.on('upgrade', (request, socket, head) => {
+    if (warming) {
+      socket.resume();
+      markBlocked(socket);
+      return;
+    }
+    wss.handleUpgrade(request, socket, head, () => {});
+  });
+
+  await new Promise<void>((resolve) => server.listen(0, '127.0.0.1', resolve));
+  const address = server.address();
+  if (!address || typeof address === 'string') throw new Error('Server has no TCP address');
+  const url = `ws://127.0.0.1:${address.port}`;
+  vi.useFakeTimers({ toFake: ['setTimeout', 'clearTimeout'] });
+  const first = connectWebSocket(url, { timeout: 500 }).catch((error: unknown) => error);
+  const stalled = await Promise.race([
+    blocked,
+    first.then((error) => {
+      throw error;
+    })
+  ]);
+  await vi.advanceTimersByTimeAsync(500);
+  vi.useRealTimers();
+  expect(await first).toMatchObject({ message: '[internal] WebSocket connection timeout' });
+  warming = false;
+  await vi.waitFor(() => expect(stalled.destroyed).toBe(true));
+
+  const replacement = await connectWebSocket(url, { timeout: 1_000 });
+  expect(replacement.readyState).toBe(WebSocket.OPEN);
+  replacement.close();
+});

package/src/socket/util.ts

@@ -66,6 +66,7 @@ export function connectWebSocket(wsUrl: string, options: ConnectWebSocketOptions
 
     const timeoutId = setTimeout(() => {
       reject(new Error('[internal] WebSocket connection timeout'));
+      ws.close();
     }, timeout);
 
     ws.addEventListener('open', () => {

package/src/tracing/methodTracing.ts

@@ -1,4 +1,4 @@
-import { Context, context, Span, trace, Tracer } from '@opentelemetry/api';
+import { type Context, context, type Span, trace, type Tracer } from '@opentelemetry/api';
 
 import { sanitizeError } from './errorSanitizer.js';
 

package/src/types/agent/agent.ts

@@ -1,5 +1,5 @@
-import { BillingPlan } from '../billing/index.js';
-import { Agent } from '../organization/index.js';
+import type { BillingPlan } from '../billing/index.js';
+import type { Agent } from '../organization/index.js';
 
 export type AgentRegisterResponseDto = {
   agent: Agent;

package/src/types/agent/index.ts

@@ -9,6 +9,6 @@ export const AGENT_INTERNAL_HOST_URL_HEADER = 'x-superblocks-agent-internal-host
 export const CORRELATION_ID_HEADER = 'x-superblocks-correlation-id';
 export const DATA_DOMAIN_HEADER = 'x-superblocks-data-domain';
 
-export * from './agent.js';
-export * from './logging.js';
-export * from './request.js';
+export type * from './agent.js';
+export type * from './logging.js';
+export type * from './request.js';

package/src/types/agent/logging.ts

@@ -1,6 +1,6 @@
-import { BillingPlan } from '../billing/index.js';
-import { EventEntityType } from '../event/index.js';
-import { RequestSource, UserType } from '../user/index.js';
+import type { BillingPlan } from '../billing/index.js';
+import type { EventEntityType } from '../event/index.js';
+import type { RequestSource, UserType } from '../user/index.js';
 
 export type LogFields = {
   // Required (global)

package/src/types/agent/request.ts

@@ -1,3 +1,3 @@
-import { Metrics } from '../common/index.js';
+import type { Metrics } from '../common/index.js';
 
 export type PostHealthcheckRequestBody = Metrics;

package/src/types/ai/index.ts

@@ -1,4 +1,4 @@
-import { DeploymentTypeEnum, RegionEnum } from '../common/index.js';
+import type { DeploymentTypeEnum, RegionEnum } from '../common/index.js';
 
 export type AiChatMessagePayload = Record<string, unknown>;
 

package/src/types/ai/quota-paywall.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from 'vitest';
 
-import { AI_QUOTA_PAYWALL_REASON_CONFIG, getAiQuotaPaywallReasonFromMessage } from './quota-paywall';
+import { AI_QUOTA_PAYWALL_REASON_CONFIG, getAiQuotaPaywallReasonFromMessage } from './quota-paywall.js';
 
 describe('getAiQuotaPaywallReasonFromMessage', () => {
   it('returns no_seat_assigned for various seat-missing phrasings', () => {

package/src/types/api/action.ts

@@ -1,4 +1,4 @@
-import {
+import type {
   AdlsPluginV1 as Adls,
   AthenaPluginV1 as Athena,
   CosmosDbPluginV1 as CosmosDb,
@@ -12,11 +12,11 @@ import {
   SmtpPluginV1 as Smtp
 } from '@superblocksteam/types';
 
-import { GoogleSheetsActionType, GoogleSheetsDestinationType, GoogleSheetsFormatType } from '../../plugins/index.js';
-import { Property } from '../common/property.js';
-import { IntegrationAuthType } from '../datasource/index.js';
-import { PluginMetadata, RestApiBodyDataType } from '../plugin/index.js';
-import { ApiTriggerType, ExecutionParam, ScheduleConfig } from './index.js';
+import type { GoogleSheetsActionType, GoogleSheetsDestinationType, GoogleSheetsFormatType } from '../../plugins/index.js';
+import type { Property } from '../common/property.js';
+import type { IntegrationAuthType } from '../datasource/index.js';
+import type { PluginMetadata, RestApiBodyDataType } from '../plugin/index.js';
+import type { ApiTriggerType, ExecutionParam, ScheduleConfig } from './index.js';
 
 export type ActionConfigParamIn = 'HEADER' | 'QUERY' | 'PATH';
 export type ActionConfigParamValue = { in: ActionConfigParamIn; value: string };

package/src/types/api/execution.ts

@@ -1,8 +1,8 @@
-import _ from 'lodash';
+import { cloneDeep, isArray, isEmpty, mergeWith } from 'lodash-es';
 
-import { Code } from '@superblocksteam/types';
+import type { Code } from '@superblocksteam/types';
 
-import { Global, PlaceholdersInfo } from '.';
+import type { Global, PlaceholdersInfo } from './index.js';
 
 // TODO(pbardea): Rename "view mode" everywhere to something more descriptive.
 export const PUBLISHED_VIEW_MODE = true;
@@ -30,21 +30,21 @@ export class ExecutionContext {
   };
 
   constructor(context?: ExecutionContext) {
-    this.globals = context && context.globals ? _.cloneDeep(context.globals) : {};
-    this.outputs = context && context.outputs ? _.cloneDeep(context.outputs) : {};
-    this.preparedStatementContext = context && context.preparedStatementContext ? _.cloneDeep(context.preparedStatementContext) : [];
+    this.globals = context && context.globals ? cloneDeep(context.globals) : {};
+    this.outputs = context && context.outputs ? cloneDeep(context.outputs) : {};
+    this.preparedStatementContext = context && context.preparedStatementContext ? cloneDeep(context.preparedStatementContext) : [];
   }
 
   // Do not merge the objects in arrays. Just replace the entire array.
   private static customMerger(objValue, srcValue) {
-    if (_.isArray(objValue)) {
+    if (isArray(objValue)) {
       return srcValue;
     }
   }
 
   addGlobalVariable(name: string, value: unknown): void {
     const obj = { [name]: value };
-    this.globals = _.mergeWith(obj, this.globals, ExecutionContext.customMerger);
+    this.globals = mergeWith(obj, this.globals, ExecutionContext.customMerger);
   }
 
   addGlobalsOverride(global: Global): void {
@@ -55,7 +55,7 @@ export class ExecutionContext {
 
   addGlobalVariableOverride(name: string, value: unknown): void {
     const obj = { [name]: value };
-    this.globals = _.mergeWith(this.globals, obj, ExecutionContext.customMerger);
+    this.globals = mergeWith(this.globals, obj, ExecutionContext.customMerger);
   }
 
   addOutput(name: string, output: ExecutionOutput): void {
@@ -74,14 +74,14 @@ export class ExecutionContext {
   }
 
   globalBindingKeys(): string[] {
-    if (_.isEmpty(this.globals)) {
+    if (isEmpty(this.globals)) {
       return [];
     }
     return Object.keys(this.globals);
   }
 
   outputBindingKeys(): string[] {
-    if (_.isEmpty(this.outputs)) {
+    if (isEmpty(this.outputs)) {
       return [];
     }
     return Object.keys(this.outputs);

package/src/types/api/global.ts

@@ -6,7 +6,7 @@
  * Global variables which can be reference during runtime.
  */
 
-import { merge } from 'lodash';
+import { merge } from 'lodash-es';
 
 export class Global {
   private payloads: Record<string, any> = {};

package/src/types/api/index.ts

@@ -1,15 +1,15 @@
-import { Api as ApiPb, Trigger_Job, Trigger_Workflow } from '@superblocksteam/types';
-import { Metadata, UserType as ProtoUserType } from '@superblocksteam/types';
-
-import { Param } from '../common/param.js';
-import { DatasourceEnvironments } from '../datasource/index.js';
-import { IntegrationDto } from '../integration/index.js';
-import { EntityProfileSettings } from '../profile/index.js';
-import { IResourceMetadata } from '../resource/index.js';
-import { Action, ActionId } from './action.js';
-import { ExecutionParam } from './execution.js';
-import { Global } from './global.js';
-import { ScheduleState } from './schedule.js';
+import type { Api as ApiPb, Trigger_Job, Trigger_Workflow } from '@superblocksteam/types';
+import type { Metadata, UserType as ProtoUserType } from '@superblocksteam/types';
+
+import type { Param } from '../common/param.js';
+import type { DatasourceEnvironments } from '../datasource/index.js';
+import type { IntegrationDto } from '../integration/index.js';
+import type { EntityProfileSettings } from '../profile/index.js';
+import type { IResourceMetadata } from '../resource/index.js';
+import type { Action, ActionId } from './action.js';
+import type { ExecutionParam } from './execution.js';
+import type { Global } from './global.js';
+import type { ScheduleState } from './schedule.js';
 
 // ApiNotificationConfiguration configures how the UI should be informed after
 // an API runs.
@@ -167,7 +167,7 @@ export type ApiSettings = EntityProfileSettings;
 
 export { ViewMode as OrchestratorViewMode } from '@superblocksteam/types';
 export * from './action.js';
-export * from './resolvedActionProp.js';
+export type * from './resolvedActionProp.js';
 export * from './schedule.js';
 export * from './execution.js';
 export * from './global.js';

package/src/types/application/index.test.ts

@@ -10,7 +10,7 @@ import {
   isSdkApiCommitTemplate,
   isSdkApiTemplate,
   LEGACY_COMMIT_TEMPLATE_NAME
-} from './index';
+} from './index.js';
 
 describe('SDK API application templates', () => {
   it('classifies app-fullstack as the 3.0 SDK API template', () => {
@@ -90,9 +90,7 @@ describe('SDK_API_TEMPLATES stays in sync with the demo-apps template config', (
   // would not fail safe -- it would confidently classify those apps as 2.0. The
   // template config is the source of truth, so pin the two together here rather
   // than relying on the "must stay in sync" comment alone.
-  // This package compiles to CommonJS, so import.meta is unavailable; vitest runs
-  // with the package directory as cwd.
-  const configPath = resolve(process.cwd(), '../demo-apps/config.json');
+  const configPath = resolve(import.meta.dirname, '../../../../demo-apps/config.json');
   const templates = (JSON.parse(readFileSync(configPath, 'utf8')) as { templates: Record<string, { sdkApi?: boolean }> }).templates;
 
   it('has at least one template to check', () => {

package/src/types/application/index.ts

@@ -1,4 +1,5 @@
-import {
+import type { ApplicationSignatureTreeSigned, Signature } from '../../signing/index.js';
+import type {
   ApplicationUserDto,
   BaseBlock,
   BranchDto,
@@ -13,12 +14,11 @@ import {
   Plugin,
   RepositoryDto,
   SupersetIntegrationDto
-} from '..';
-import { ApplicationSignatureTreeSigned, Signature } from '../../signing/index.js';
-import { ResourceMetadata } from '../metadata/index.js';
-import { Dimension, EntityWithBindings6, IPageV2Dto, MultiStepDef, Padding, PageDSL, PageSummary } from '../page/index.js';
-import { EntityProfileSettings } from '../profile/index.js';
-import { IResourceMetadata } from '../resource/index.js';
+} from '../index.js';
+import type { ResourceMetadata } from '../metadata/index.js';
+import type { Dimension, EntityWithBindings6, IPageV2Dto, MultiStepDef, Padding, PageDSL, PageSummary } from '../page/index.js';
+import type { EntityProfileSettings } from '../profile/index.js';
+import type { IResourceMetadata } from '../resource/index.js';
 
 export interface Application {
   id: string;

package/src/types/audit/apiRunDetails.ts

@@ -2,8 +2,8 @@
  * @deprecated Legacy audit run details. Use OCSFAuditEvent instead.
  */
 
-import { ApiLocationContext } from '../api/index.js';
-import { AuditLogDetails } from './audit.js';
+import type { ApiLocationContext } from '../api/index.js';
+import type { AuditLogDetails } from './audit.js';
 
 /** @deprecated Use OCSFAuditEvent instead. */
 interface ApiRunDetails extends AuditLogDetails {

package/src/types/audit/audit.ts

@@ -4,10 +4,10 @@
  * system fully replaces the legacy audit_log table.
  */
 
-import { UserType } from '../../index.js';
-import { ApiLocationContext } from '../api/index.js';
-import { ApiResourceTiming } from '../common/timing.js';
-import { ApiRunStatus } from './apiRunDetails.js';
+import type { UserType } from '../../index.js';
+import type { ApiLocationContext } from '../api/index.js';
+import type { ApiResourceTiming } from '../common/timing.js';
+import type { ApiRunStatus } from './apiRunDetails.js';
 
 /** @deprecated Use OCSF resource types instead. */
 export enum AuditLogEntityType {

package/src/types/audit/ocsf.ts

@@ -154,7 +154,7 @@ export const AUDIT_EVENT_TYPE_CATALOG: AuditEventTypeEntry[] = [
     category_uid: OCSFCategoryUid.APPLICATION_ACTIVITY,
     category_name: 'APPLICATION_ACTIVITY',
     resource_type: 'AppDatabase',
-    operations: ['database.migrate', 'database.provision']
+    operations: ['database.import', 'database.migrate', 'database.provision']
   },
   {
     class_uid: OCSFClassUid.APPLICATION_LIFECYCLE,
@@ -357,6 +357,7 @@ export const AUDIT_EVENT_TYPE_CATALOG: AuditEventTypeEntry[] = [
       'scim.user.update',
       'user.activate',
       'user.add',
+      'user.email.update',
       'user.invite',
       'user.role.update',
       'user.roles.bulk_update'

package/src/types/auth/index.ts

@@ -1,2 +1,2 @@
 export * from './apiKey.js';
-export * from './oauth2.js';
+export type * from './oauth2.js';

package/src/types/billing/index.ts

@@ -1,5 +1,5 @@
 export * from './freeCreditsTier.js';
 export * from './pricingModel.js';
 export * from './billing.js';
-export * from './spendAlert.js';
+export type * from './spendAlert.js';
 export * from './usageSource.js';

package/src/types/commit/index.ts

@@ -1,4 +1,4 @@
-import { BuildStatus } from '@superblocksteam/sabs-types';
+import type { BuildStatus } from '@superblocksteam/sabs-types';
 
 export interface CommitDto {
   commitMessage: string;

package/src/types/common/cli.ts

@@ -2,8 +2,10 @@ import slugify from 'slugify';
 
 export function slugifyName(originalName: string): string {
   // this replace is very brittle but it's what we do in the CLI and fixes the average case
-  return slugify(originalName, {
-    replacement: '_',
-    lower: true
-  }).replace("'", "\\'");
+  return slugify
+    .default(originalName, {
+      replacement: '_',
+      lower: true
+    })
+    .replace("'", "\\'");
 }

package/src/types/common/index.ts

@@ -1,8 +1,8 @@
 export * from './file.js';
-export * from './param.js';
-export * from './property.js';
+export type * from './param.js';
+export type * from './property.js';
 export * from './response.js';
-export * from './types.js';
+export type * from './types.js';
 export * from './observability.js';
 export * from './env.js';
 export * from './region.js';

package/src/types/common/observability.ts

@@ -1,4 +1,4 @@
-import { AgentStatus, AgentTags } from '../organization/agent.js';
+import type { AgentStatus, AgentTags } from '../organization/agent.js';
 
 enum AgentMessageType {
   INTEGRATION_ERROR = 'integrationError', // User errors found in plugin execution code