ref: Fix up some tsconfig schenanigans

.gitignore

@@ -38,4 +38,5 @@ sentry.properties
 junit.xml
 
 # Build artifacts
-*.tgz
+*.tgz
+tsconfig.tsbuildinfo

packages/astro/src/index.ts

@@ -2,8 +2,8 @@ import type { SpotlightInitOptions } from '@spotlightjs/spotlight/vite-plugin';
 import type { AstroConfig, AstroIntegration } from 'astro';
 import { buildServerSnippet } from './snippets';
 
-import path from 'path';
-import url from 'url';
+import path from 'node:path';
+import url from 'node:url';
 
 import spotlight, { buildClientInit } from '@spotlightjs/spotlight/vite-plugin';
 

packages/overlay/src/App.tsx

@@ -44,9 +44,9 @@ export default function App({
       }
     }
 
-    const result: Record<string, (event: { data: string | Buffer }) => void> = Object.create(null);
+    const result: Record<string, (event: { data: string | Uint8Array }) => void> = Object.create(null);
     for (const [contentType, integrations] of contentTypeToIntegrations.entries()) {
-      const listener = (event: { data: string | Buffer }): void => {
+      const listener = (event: { data: string | Uint8Array }): void => {
         log(`Received new ${contentType} event`);
         for (const integration of integrations) {
           const newIntegrationData = integration.processEvent

packages/overlay/src/index.tsx

@@ -2,22 +2,22 @@ import fontStyles from '@fontsource/raleway/index.css?inline';
 import { CONTEXT_LINES_ENDPOINT } from '@spotlightjs/sidecar/constants';
 import { MemoryRouter } from 'react-router-dom';
 import colors from 'tailwindcss/colors';
-import App from './App.tsx';
-import { DEFAULT_ANCHOR, DEFAULT_EXPERIMENTS, DEFAULT_SIDECAR_URL } from './constants.ts';
+import App from './App';
+import { DEFAULT_ANCHOR, DEFAULT_EXPERIMENTS, DEFAULT_SIDECAR_URL } from './constants';
 import globalStyles from './index.css?inline';
-import { initIntegrations, type SpotlightContext } from './integrations/integration.ts';
-import { default as sentry } from './integrations/sentry/index.ts';
-import { off, on, trigger } from './lib/eventTarget.ts';
-import { activateLogger, log } from './lib/logger.ts';
-import { SpotlightContextProvider } from './lib/useSpotlightContext.tsx';
-import { React, ReactDOM } from './react-instance.tsx'; // Import specific exports
-import type { SpotlightOverlayOptions, WindowWithSpotlight } from './types.ts';
-
-export { default as console } from './integrations/console/index.ts';
-export { default as hydrationError } from './integrations/hydration-error/index.ts';
-export { default as sentry } from './integrations/sentry/index.ts';
-export { default as viteInspect } from './integrations/vite-inspect/index.ts';
-export type { SpotlightOverlayOptions, WindowWithSpotlight } from './types.ts';
+import { initIntegrations, type SpotlightContext } from './integrations/integration';
+import { default as sentry } from './integrations/sentry/index';
+import { off, on, trigger } from './lib/eventTarget';
+import { activateLogger, log } from './lib/logger';
+import { SpotlightContextProvider } from './lib/useSpotlightContext';
+import { React, ReactDOM } from './react-instance'; // Import specific exports
+import type { SpotlightOverlayOptions, WindowWithSpotlight } from './types';
+
+export { default as console } from './integrations/console/index';
+export { default as hydrationError } from './integrations/hydration-error/index';
+export { default as sentry } from './integrations/sentry/index';
+export { default as viteInspect } from './integrations/vite-inspect/index';
+export type { SpotlightOverlayOptions, WindowWithSpotlight } from './types';
 export {
   CONTEXT_LINES_ENDPOINT,
   DEFAULT_ANCHOR,

packages/overlay/src/integrations/integration.ts

@@ -1,5 +1,5 @@
-import { type ComponentType } from 'react';
-import { type ExperimentsConfig, type NotificationCount } from '~/types';
+import type { ComponentType } from 'react';
+import type { ExperimentsConfig, NotificationCount } from '../types';
 
 export type SpotlightContext = {
   open: (path: string | undefined) => void;
@@ -114,15 +114,15 @@ export type RawEventContext = {
    *
    * Return the processed object or undefined if the event should be ignored.
    */
-  data: string | Buffer;
+  data: string | Uint8Array;
 };
 
 type TeardownFunction = () => void | Promise<() => void>;
 
 // export type IntegrationParameter = Array<Integration<unknown>>;
 
 export async function initIntegrations(
-  integrations: Integration[] = [],
+  integrations: Integration[],
   context: SpotlightContext,
 ): Promise<[Integration[], TeardownFunction[]]> {
   if (!integrations) {

packages/overlay/test/integrations/sentry/data/sentryDataCache.test.ts → packages/overlay/src/integrations/sentry/data/sentryDataCache.spec.ts

@@ -1,8 +1,8 @@
 import { describe, expect, test } from 'vitest';
-import sentryDataCache from '~/integrations/sentry/data/sentryDataCache';
-import { processEnvelope } from '~/integrations/sentry/index';
+import { processEnvelope } from '../index';
+import sentryDataCache from './sentryDataCache';
 
-import fs from 'fs';
+import fs from 'node:fs';
 
 describe('SentryDataCache', () => {
   // We need to refactor this to make it actually testable

packages/overlay/test/integrations/sentry/index.test.ts → packages/overlay/src/integrations/sentry/index.spec.ts

@@ -1,9 +1,9 @@
 import { describe, expect, test } from 'vitest';
-import { processEnvelope } from '~/integrations/sentry/index';
+import { processEnvelope } from './index';
 
-import { Event } from '@sentry/types';
-import fs from 'fs';
-import sentryDataCache from '~/integrations/sentry/data/sentryDataCache';
+import type { Event } from '@sentry/types';
+import fs from 'node:fs';
+import sentryDataCache from './data/sentryDataCache';
 
 describe('Sentry Integration', () => {
   test('Process Envelope Empty', () => {

packages/overlay/src/integrations/sentry/index.ts

@@ -117,7 +117,7 @@ function getLineEnd(data: Uint8Array): number {
   return end;
 }
 
-function parseJSONFromBuffer(data: Uint8Array) {
+function parseJSONFromBuffer(data: Uint8Array): object {
   return JSON.parse(new TextDecoder().decode(data));
 }
 
@@ -144,19 +144,21 @@ export function processEnvelope(rawEvent: RawEventContext) {
   while (buffer.length) {
     const itemHeader = parseJSONFromBuffer(readLine()) as EnvelopeItem[0];
     const payloadLength = itemHeader.length;
-    let itemPayload = readLine(payloadLength != null ? payloadLength : undefined);
+    const itemPayloadRaw = readLine(payloadLength != null ? payloadLength : undefined);
 
+    let itemPayload: EnvelopeItem[1];
     try {
-      itemPayload = parseJSONFromBuffer(itemPayload);
+      itemPayload = parseJSONFromBuffer(itemPayloadRaw);
+      // data sanitization
+      if (itemHeader.type) {
+        // @ts-expect-error ts(2339) -- We should really stop adding type to payloads
+        itemPayload.type = itemHeader.type;
+      }
     } catch (err) {
+      itemPayload = itemPayloadRaw;
       log(err);
     }
 
-    // data sanitization
-    if (itemHeader.type && typeof itemPayload === 'object') {
-      // @ts-expect-error -- Does not like assigning to `type` on random object
-      itemPayload.type = itemHeader.type;
-    }
     items.push([itemHeader, itemPayload] as EnvelopeItem);
   }
 

packages/overlay/test/integrations/sentry/utils/traces.test.ts → packages/overlay/src/integrations/sentry/utils/traces.spec.ts

@@ -1,7 +1,7 @@
 import { describe, expect, test } from 'vitest';
-import { Span } from '~/integrations/sentry/types';
-import { groupSpans } from '~/integrations/sentry/utils/traces';
-import { generate_uuidv4 } from '~/lib/uuid';
+import { generate_uuidv4 } from '../../../lib/uuid';
+import type { Span } from '../types';
+import { groupSpans } from './traces';
 
 function mockSpan({ duration, ...span }: Partial<Span> & { duration?: number } = {}): Span {
   const defaultTimestamp = new Date().getTime();

packages/overlay/tsconfig.json

@@ -4,8 +4,10 @@
     "baseUrl": ".",
     "paths": {
       "~/*": ["src/*"]
-    }
+    },
+    "outDir": "dist",
+    "rootDir": "src"
   },
-  "include": ["src", "test"],
+  "include": ["src"],
   "references": [{ "path": "./tsconfig.node.json" }]
 }

packages/overlay/vitest.config.ts

@@ -12,7 +12,7 @@ export default defineConfig({
       reporter: ['json'],
     },
     globals: true,
-    include: ['./test/**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
+    include: ['./src/**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     watchExclude: ['.*\\/node_modules\\/.*', '.*\\/dist\\/.*'],
   },
   resolve: {

packages/sidecar/tsconfig.json

@@ -9,11 +9,9 @@
     "esModuleInterop": false,
     "target": "esnext",
     "module": "nodenext",
+    /* paths */
     "outDir": "dist",
-    "noEmit": false,
-    "allowImportingTsExtensions": false,
-    "declaration": true,
-    "emitDeclarationOnly": true
+    "rootDir": "src"
   },
   "include": ["src/*"]
 }

packages/spotlight/tsconfig.json

@@ -3,9 +3,7 @@
   "compilerOptions": {
     "baseUrl": ".",
     "skipLibCheck": true,
-    "emitDeclarationOnly": true,
-    "declaration": true,
-    "noEmit": false,
+    /* paths */
     "outDir": "dist",
     "rootDir": "src"
   },

packages/tsconfig/tsconfig.json

@@ -5,13 +5,16 @@
     "lib": ["ES2020", "DOM", "DOM.Iterable"],
     "module": "ESNext",
     "skipLibCheck": true,
+    "allowImportingTsExtensions": false,
+    /* cannot enable composite mode yet as vite clears output directory, requiring a tsc rebuild all the time */
+    "composite": false,
+    "declaration": true,
+    "emitDeclarationOnly": true,
 
     /* Bundler mode */
     "moduleResolution": "bundler",
-    "allowImportingTsExtensions": true,
     "resolveJsonModule": true,
     "isolatedModules": true,
-    "noEmit": true,
     "jsx": "react-jsx",
 
     /* Linting */