Merge pull request #446 from webstackdev/bugfix/ci-types-error

Fix CI type error

Author: webstackdev

.cache/pages.d.ts

@@ -0,0 +1,17 @@
+/**
+ * Type declaration for generated pages.json file
+ * This file is generated during the build process
+ */
+
+type PageData =
+  | string
+  | { articles: string[] }
+  | { 'case-studies': string[] }
+  | { downloads: string[] }
+  | { services: string[] }
+  | { 'social-shares': string[] }
+  | { stories: string[] }
+  | { tags: string[] }
+
+declare const pagesData: PageData[]
+export default pagesData

.cache/pages.json

@@ -0,0 +1,53 @@
+[
+  "about",
+  {
+    "articles": [
+      "getting-started-with-astro",
+      "typescript-best-practices",
+      "useful-vs-code-extensions",
+      "writing-library-code"
+    ]
+  },
+  {
+    "case-studies": [
+      "division-15",
+      "ecommerce-modernization",
+      "english-first",
+      "enterprise-api-platform",
+      "labcorp",
+      "us-logistics"
+    ]
+  },
+  "contact",
+  "cookies",
+  {
+    "downloads": [
+      "api-tool-consolidation-whitepaper",
+      "identity-security-for-dummies",
+      "lakehouse-analytics-guide",
+      "observability-benefits-guide",
+      "ransomware-recovery-kit"
+    ]
+  },
+  "offline",
+  "privacy",
+  {
+    "services": ["create-custom-font-sets", "overview"]
+  },
+  {
+    "social-shares": ["template"]
+  },
+  {
+    "tags": [
+      "apiDesign",
+      "cms",
+      "code",
+      "crm",
+      "graphql",
+      "online-learning",
+      "react",
+      "services",
+      "typescript"
+    ]
+  }
+]

.gitignore

@@ -32,8 +32,10 @@ npm-debug.log*
 # Optional eslint cache
 .eslintcache
 
-# Cache
-.cache
+# Cache - ignore all contents except type declarations and placeholder data
+.cache/*
+!.cache/*.d.ts
+!.cache/pages.json
 
 # Runtime data
 pids

.husky/pre-commit

@@ -18,6 +18,11 @@ fi
 # Run TypeScript check (fast feedback on type errors)
 npm run check
 
+# Run linters (fail fast on each)
+npm run lint:style
+npm run lint:code
+npm run lint:json
+
 # Run unit tests (fast feedback)
 npm run test:unit
 

@types/remark-abbr.d.ts

@@ -1,3 +1,4 @@
+/* eslint-disable jsdoc/no-multi-asterisks */
 /**
  * Type definitions for remark-abbr
  *

@types/vitest-axe.d.ts

@@ -1,5 +1,5 @@
 /// <reference types="vitest" />
-
+/* eslint-disable @typescript-eslint/no-empty-object-type, no-unused-vars, @typescript-eslint/no-unused-vars, @typescript-eslint/no-explicit-any */
 import type { AxeMatchers } from 'vitest-axe/matchers'
 
 declare module 'vitest' {

astro.config.ts

@@ -17,11 +17,11 @@ import {
 import { callToActionValidator } from './src/integrations/CtaValidator/call-to-action-validator'
 import { serializeSitemapItem, writePagesJson } from './src/lib/config/sitemap-serialize'
 
-// Type guard for required environment variables (only in CI)
-const IS_CI = process.env['CI'] === 'true'
+// Type guard for required environment variables (only in Vercel)
+const IS_VERCEL = process.env['VERCEL']
 const SENTRY_AUTH_TOKEN = process.env['SENTRY_AUTH_TOKEN']
-if (IS_CI && !SENTRY_AUTH_TOKEN) {
-  throw new Error('SENTRY_AUTH_TOKEN environment variable is required in CI but not set')
+if (IS_VERCEL && !SENTRY_AUTH_TOKEN) {
+  throw new Error('SENTRY_AUTH_TOKEN environment variable is required in Vercel but not set')
 }
 
 export default defineConfig({
@@ -38,11 +38,11 @@ export default defineConfig({
     callToActionValidator({
       debug: true // Enable debug logging to see validation details
     }),
-    // Only include Sentry integration in CI environments
-    ...(IS_CI && SENTRY_AUTH_TOKEN ? [sentry({
+    // Only include Sentry integration in Vercel environments
+    ...(IS_VERCEL ? [sentry({
       project: "webstack-builders-corporate-website",
       org: "webstack-builders",
-      authToken: SENTRY_AUTH_TOKEN,
+      authToken: SENTRY_AUTH_TOKEN!, // Non-null assertion safe due to check above
     })] : []),
     sitemap({
       lastmod: new Date(),