fixes

Author: Sandeepan-Ghosh-0312

packages/coded-action-apps/src/index.ts

@@ -1,3 +1,10 @@
-export { CodedActionAppsService as CodedActionApps, CodedActionAppsService} from './coded-action-apps-service';
+// Side-effect import: bootstraps the CAA package's TelemetryClient
+// before any service is constructed. Kept explicit (rather than relying
+// on `coded-action-apps-service.ts` transitively importing `track` from
+// `./telemetry`) so initialization survives any future refactor of the
+// service file's imports.
+import './telemetry';
+
+export { CodedActionAppsService as CodedActionApps, CodedActionAppsService } from './coded-action-apps-service';
 export * from './coded-action-apps.models';
 export * from './types';

packages/telemetry/tests/track.test.ts

@@ -34,9 +34,17 @@ function multiply(a: number, b: number): number {
     return a * b;
 }
 
-function namedFn(): void {}
+// Empty body — fixture only exists so the wrapper can read its `name` and
+// fall back to it (`'namedFn'`) for telemetry attribution.
+function namedFn(): void {
+    /* fixture: intentionally empty */
+}
 
-const anonFn = function () {};
+// Empty body — fixture only exists so the wrapper can detect the missing
+// `name` and fall back to `'unknown_function'`.
+const anonFn = function () {
+    /* fixture: intentionally empty */
+};
 Object.defineProperty(anonFn, 'name', { value: '' });
 
 describe('createTrack — method decorator', () => {
@@ -84,7 +92,9 @@ describe('createTrack — method decorator', () => {
 
         class Service {
             @track('Tasks.Create', { attributes: { Source: 'unit-test' } })
-            public create(): void {}
+            public create(): void {
+                /* fixture: only the decorator side effect matters */
+            }
         }
 
         new Service().create();
@@ -121,7 +131,9 @@ describe('createTrack — method decorator', () => {
             // public `TrackOptions.condition` signature; the callback
             // sees the runtime args as expected.
             @track('Tasks.Create', { condition: condition as unknown as (...args: unknown[]) => boolean })
-            public create(_value: number): void {}
+            public create(_value: number): void {
+                /* fixture: only the decorator side effect matters */
+            }
         }
 
         const svc = new Service();
@@ -139,7 +151,9 @@ describe('createTrack — method decorator', () => {
 
         class Service {
             @track('Tasks.Create')
-            public create(): void {}
+            public create(): void {
+                /* fixture: only the decorator side effect matters */
+            }
         }
 
         const svc = new Service();
@@ -176,7 +190,9 @@ describe('createTrack — method decorator', () => {
 
         class Service {
             @track('Tasks.Create')
-            public create(): void {}
+            public create(): void {
+                /* fixture: only the decorator side effect matters */
+            }
         }
 
         new Service().create();

src/core/telemetry/constants.ts

@@ -1,30 +1,18 @@
 /**
- * SDK Telemetry constants
+ * SDK Telemetry constants.
  *
- * The Application Insights connection string is no longer carried here —
- * it is injected into `@uipath/telemetry` itself at publish time. Only the
- * SDK's identity (version, service name, role name, …) lives here.
+ * Only the SDK's identity (version, service name, role name, …) lives
+ * here. The Application Insights connection string is injected into
+ * `@uipath/telemetry` itself at publish time, and the generic attribute
+ * keys (`Version`, `Service`, `CloudOrganizationName`, …) are owned by
+ * `@uipath/telemetry` and re-exported from the shim in `index.ts`.
  */
 
-// SDK Version placeholder — patched by the SDK publish workflow.
-export const SDK_VERSION = "$SDK_VERSION";
+/** SDK version placeholder — patched by the SDK publish workflow. */
+export const SDK_VERSION = '$SDK_VERSION';
 
-export const VERSION = "Version";
-export const SERVICE = "Service";
-export const CLOUD_ORGANIZATION_NAME = "CloudOrganizationName";
-export const CLOUD_TENANT_NAME = "CloudTenantName";
-export const CLOUD_URL = "CloudUrl";
-export const CLOUD_CLIENT_ID = "CloudClientId";
-export const CLOUD_REDIRECT_URI = "CloudRedirectUri";
-export const APP_NAME = "ApplicationName";
-export const CLOUD_ROLE_NAME = "uipath-ts-sdk";
+export const CLOUD_ROLE_NAME = 'uipath-ts-sdk';
+export const SDK_SERVICE_NAME = 'UiPath.TypeScript.Sdk';
+export const SDK_LOGGER_NAME = 'uipath-ts-sdk-telemetry';
 
-// Service and logger names
-export const SDK_SERVICE_NAME = "UiPath.TypeScript.Sdk";
-export const SDK_LOGGER_NAME = "uipath-ts-sdk-telemetry";
-
-// Event names
-export const SDK_RUN_EVENT = "Sdk.Run";
-
-// Default value for unknown/empty attributes
-export const UNKNOWN = "";
+export const SDK_RUN_EVENT = 'Sdk.Run';

src/core/telemetry/index.ts

@@ -44,6 +44,21 @@ export type {
     TelemetryContext as TelemetryConfig,
     TrackOptions,
 } from '@uipath/telemetry';
+
+// Generic attribute-key constants live in `@uipath/telemetry` (single
+// source of truth). Re-exported here so the public
+// `@uipath/uipath-typescript/core` API surface stays unchanged.
+export {
+    APP_NAME,
+    CLOUD_CLIENT_ID,
+    CLOUD_ORGANIZATION_NAME,
+    CLOUD_REDIRECT_URI,
+    CLOUD_TENANT_NAME,
+    CLOUD_URL,
+    SERVICE,
+    UNKNOWN,
+    VERSION,
+} from '@uipath/telemetry';
 export * from './constants';
 
 /**