@dxos/observability 0.8.4-main.84f28bd → 0.8.4-main.8baae0fced

package/src/{helpers/node-observability.ts → storage/node.ts}

@@ -1,17 +1,12 @@
+import yaml from 'js-yaml';
 //
 // Copyright 2022 DXOS.org
 //
-import yaml from 'js-yaml';
 import { existsSync, statSync } from 'node:fs';
 import { mkdir, readFile, writeFile } from 'node:fs/promises';
 import { join } from 'node:path';
 import { v4 as uuid, validate as validateUuid } from 'uuid';
 
-import type { Config } from '@dxos/client';
-import { log } from '@dxos/log';
-
-import { Observability, type Mode } from '../observability';
-
 /**
  * Print observability banner once per installation.
  */
@@ -21,7 +16,6 @@ export const showObservabilityBanner = async (configDir: string, bannercb: (inpu
     return;
   }
   bannercb(
-    // eslint-disable-next-line no-multi-str
     'Basic observability data will be sent to the DXOS team in order to improve the product. This includes \
     performance metrics, error logs, and usage data. No personally identifiable information, other than your \
     public key, is included with this data and no private data ever leaves your devices. To disable sending \
@@ -31,7 +25,43 @@ export const showObservabilityBanner = async (configDir: string, bannercb: (inpu
   await writeFile(path, '', 'utf-8');
 };
 
-export const getObservabilityState = async (configDir: string): Promise<PersistentObservabilityState> => {
+/**
+ * @param configDir - Filesystem path to the directory containing the `observability.yml` state file.
+ */
+export const isObservabilityDisabled = async (configDir: string): Promise<boolean> => {
+  const observabilityState = await getObservabilityState(configDir);
+  return observabilityState.disabled;
+};
+
+/**
+ * @param configDir - Filesystem path to the directory containing the `observability.yml` state file.
+ */
+// NOTE: read-modify-write is not atomic; concurrent callers could overwrite each other.
+// Acceptable here because writes are infrequent and triggered by explicit user action.
+export const storeObservabilityDisabled = async (configDir: string, value: boolean) => {
+  const observabilityState = await getObservabilityState(configDir);
+  observabilityState.disabled = value;
+  await writeFile(join(configDir, 'observability.yml'), yaml.dump(observabilityState), 'utf-8');
+};
+
+/**
+ * @param configDir - Filesystem path to the directory containing the `observability.yml` state file.
+ */
+export const getObservabilityGroup = async (configDir: string): Promise<string | undefined> => {
+  const observabilityState = await getObservabilityState(configDir);
+  return observabilityState.group;
+};
+
+/**
+ * @param configDir - Filesystem path to the directory containing the `observability.yml` state file.
+ */
+export const storeObservabilityGroup = async (configDir: string, value: string) => {
+  const observabilityState = await getObservabilityState(configDir);
+  observabilityState.group = value;
+  await writeFile(join(configDir, 'observability.yml'), yaml.dump(observabilityState), 'utf-8');
+};
+
+const getObservabilityState = async (configDir: string): Promise<PersistentObservabilityState> => {
   // check whether configDir exists and if it's a directory
 
   if (existsSync(configDir)) {
@@ -53,8 +83,8 @@ export const getObservabilityState = async (configDir: string): Promise<Persiste
 
 export type PersistentObservabilityState = {
   installationId: string;
+  disabled: boolean;
   group?: string;
-  mode: Mode;
 };
 
 // create initial state and write to file, using environment variables to override defaults.
@@ -62,8 +92,8 @@ const initializeState = async (idPath: string): Promise<PersistentObservabilityS
   // TODO(nf): read initial values from config or seed file
   const observabilityState = {
     installationId: uuid(),
+    disabled: process.env.DX_DISABLE_OBSERVABILITY ? true : false,
     group: process.env.DX_OBSERVABILITY_GROUP ?? undefined,
-    mode: (process.env.DX_DISABLE_OBSERVABILITY ? 'disabled' : process.env.DX_OBSERVABILITY_MODE ?? 'basic') as Mode,
   };
 
   await writeFile(
@@ -80,66 +110,7 @@ const validate = (contextString: string) => {
   if (Boolean(context.installationId) && validateUuid(context.installationId!)) {
     return {
       ...context,
-      mode: process.env.DX_DISABLE_OBSERVABILITY ? 'disabled' : context.mode ?? 'basic',
+      disabled: process.env.DX_DISABLE_OBSERVABILITY ? true : (context.disabled ?? false),
     };
   }
 };
-
-export type NodeObservabilityOptions = {
-  installationId: string;
-  group?: string;
-  namespace: string;
-  version: string;
-  config: Config;
-  mode?: Mode;
-  tracingEnable?: boolean;
-  replayEnable?: boolean;
-  // TODO(nf): options for providers?
-};
-
-export const initializeNodeObservability = async ({
-  namespace,
-  version,
-  config,
-  installationId,
-  group,
-  mode = 'basic',
-  tracingEnable = true,
-  replayEnable = true,
-}: NodeObservabilityOptions): Promise<Observability> => {
-  log('initializeCliObservability', { config });
-
-  // TODO(nf): make CLI build populate runtime.app.build config?
-  const release = `${namespace}@${version}`;
-  const environment = process.env.DX_ENVIRONMENT ?? 'unknown';
-
-  const observability = new Observability({
-    mode,
-    namespace,
-    release,
-    environment,
-    group,
-    errorLog: {
-      sentryInitOptions: {
-        environment,
-        release,
-        // TODO(wittjosiah): Configure this.
-        sampleRate: 1.0,
-      },
-    },
-  });
-
-  observability.setTag('installationId', installationId);
-
-  // TODO(nf): cache ipdata to avoid repeated requests
-  const IPDATA_API_KEY = config.get('runtime.app.env.DX_IPDATA_API_KEY');
-  try {
-    const res = await fetch(`https://api.ipdata.co/?api-key=${IPDATA_API_KEY}`);
-    const ipData = await res.json();
-    ipData && observability.setIPDataTelemetryTags(ipData);
-  } catch (err) {
-    observability?.captureException(err);
-  }
-
-  return observability;
-};

package/src/vite-import-meta.d.ts

@@ -0,0 +1,14 @@
+//
+// Copyright 2025 DXOS.org
+//
+
+/**
+ * Vite replaces `import.meta.env` at build time. Aligns with `vite/client` without taking a Vite dependency.
+ * @see https://vitejs.dev/guide/env-and-mode.html
+ */
+interface ImportMetaEnv {
+  readonly DEV: boolean;
+  readonly PROD: boolean;
+  readonly MODE: string;
+  readonly SSR: boolean;
+}