Update sentry files to use environment methods, fix docs

Author: webstackdev

sentry.server.config.js

@@ -1,6 +1,6 @@
 import * as Sentry from "@sentry/astro"
 import { SENTRY_DSN } from "astro:env/client"
-
+import { isDev, isProd } from '@lib/config'
 /**
  * Server-side Sentry initialization
  *
@@ -11,11 +11,8 @@ import { SENTRY_DSN } from "astro:env/client"
  * The server SDK only needs SENTRY_DSN to report errors.
  */
 
-const isProd = import.meta.env.PROD
-const isDev = import.meta.env.DEV
-
 // Initialize Sentry in production if DSN is available
-if (isProd) {
+if (isProd()) {
   Sentry.init({
     dsn: SENTRY_DSN,
 
@@ -51,7 +48,7 @@ if (isProd) {
      */
     beforeSend(event, _hint) {
       // Don't send in development
-      if (isDev) {
+      if (isDev()) {
         return null
       }
       return event

src/components/scripts/sentry/client.ts

@@ -11,6 +11,7 @@ import {
   linkedErrorsIntegration,
 } from '@sentry/browser'
 import { SENTRY_DSN } from 'astro:env/client'
+import { isDev } from '@components/scripts/utils'
 import { $consent } from '@components/scripts/store/consent'
 
 /**
@@ -37,7 +38,6 @@ export class SentryBootstrap {
 
     const client = new BrowserClient({
       dsn: SENTRY_DSN,
-
       integrations: [
         // Core integrations
         breadcrumbsIntegration({
@@ -105,7 +105,7 @@ export class SentryBootstrap {
        */
       beforeSend(event, _hint) {
         // Don't send errors in development
-        if (import.meta.env.DEV) {
+        if (isDev()) {
           return null
         }
 

src/lib/config/environmentServer.ts

@@ -1,13 +1,7 @@
-/** Cannot use path aliases in this file */
-import { BuildError } from '../errors/BuildError'
-
 /**
  * This method is only intended to be called from astro.config.ts
  */
 export const getSentryAuthToken = () => {
-  if (!process.env['SENTRY_AUTH_TOKEN']) {
-    throw new BuildError('Running in CI environment, but SENTRY_AUTH_TOKEN is not set.')
-  }
   return process.env['SENTRY_AUTH_TOKEN']
 }
 

src/lib/config/environmentalVariableValidation.ts

@@ -20,11 +20,11 @@ import type { AstroUserConfig } from 'astro'
  * Secret client variables: not supported because there is no safe way to send this data to the client.
  *
  * ✅ CORRECT: Public client variables are available from BOTH modules
- * import { PUBLIC_SENTRY_DSN } from 'astro:env/client'  // Works in client code
- * import { PUBLIC_SUPABASE_KEY } from 'astro:env/server' // Works in server code (API routes)
+ * import { CLIENT_PUBLIC_DSN } from 'astro:env/client'  // Works in client code
+ * import { CLIENT_PUBLIC_KEY } from 'astro:env/server' // Works in server code (API routes)
  *
  * ❌ WRONG: Server variables are ONLY available from astro:env/server
- * import { RESEND_API_KEY } from 'astro:env/client' // TypeScript error!
+ * import { CLIENT_PUBLIC_KEY } from 'astro:env/client' // TypeScript error!
  */
 
 export const environmentalVariablesConfig: AstroUserConfig['env'] = {
@@ -110,11 +110,14 @@ export const environmentalVariablesConfig: AstroUserConfig['env'] = {
     }),
     /**
      * Site uses Sentry for monitoring site errors and user path tracing
+     *
+     * - Only used in code called by astro.config.ts, uses process.env
+     * - Used to upload source maps during build
      */
     SENTRY_AUTH_TOKEN: envField.string({
-      access: 'public',
+      access: 'secret',
       context: 'server',
-      optional: false, // Only required for uploading source maps during build
+      optional: false,
     }),
     SENTRY_DSN: envField.string({
       access: 'public',
@@ -124,12 +127,12 @@ export const environmentalVariablesConfig: AstroUserConfig['env'] = {
     /**
      * Site uses Suprabase for managing GDPR consent records
      */
-    PUBLIC_SUPABASE_URL: envField.string({
+    SUPABASE_URL: envField.string({
       access: 'public',
       context: 'client',
       optional: false,
     }),
-    PUBLIC_SUPABASE_KEY: envField.string({
+    SUPABASE_KEY: envField.string({
       access: 'public',
       context: 'server',
       optional: false,