Move camelcase lint error suppressions to eslint config

Author: webstackdev

.clinerules/project-standards.md

@@ -10,7 +10,6 @@ Follow the coding standards defined in `.github/instructions/`:
 Key rules:
 
 - Use TypeScript exclusively
-- Testing with Container API and happy-dom
 - Never use manual HTML fixtures
 - No apologizing, be direct/concise
 - Wait for permission before implementing suggestions

.github/instructions/astro.instructions.md

@@ -12,4 +12,4 @@ applyTo: "docs/**/*.astro"
 - When testing Astro components, use the Astro Container API instead of manual HTML fixtures.
 - This ensures tests stay in sync with component changes automatically.
 - Reference: Astro Container API Documentation at docs.astro.build/en/reference/container-reference/
-- Note: The Container API does not work reliably in jsdom/vitest environments. Use happy-dom instead.
+- There is a model component and test demonstrating this pattern in the src/components/Test directory named webComponent.

.github/instructions/general.instructions.md

@@ -67,13 +67,9 @@ Always use the `navigateToPage()` method for client-side navigation - never ad-h
 - Reference the working example in src/components/Test/container.astro and its test file.
 - Use experimental_AstroContainer.create() to instantiate the container.
 - Use container.renderToString(Component) to get rendered HTML from actual Astro components.
-- For DOM unit testing with Container API: use `// @vitest-environment happy-dom` for better DOM compatibility than jsdom or node.
 - Configure Vitest with getViteConfig() from 'astro/config' to support Astro Container API.
 - Test files should follow a client.spec.ts naming pattern or similar.
 - Fixture files should follow a componentName.fixture.astro naming pattern (e.g., newsletter.fixture.astro).
-- Use `// @vitest-environment happy-dom` as the first line of test files that need DOM support with Container API. Never include Vitest directives inside JSDoc comments.
-- happy-dom provides proper document, window, and localStorage globals without manual mocking.
-- JavaScript loading warnings from happy-dom are silenced in vitest.setup.ts for clean test output.
 - A working example test using the Container API is available at /home/kevin/Repos/Webstack Builders/Corporate Website/astro.webstackbuilders.com/src/components/Test/container.spec.ts
 
 ## E2E Testing Standards

.github/instructions/testing.instructions.md

@@ -8,13 +8,8 @@ applyTo: "**/*.spec.ts"
 
 - **NEVER use manual HTML strings** - use Astro's Container API with actual .astro templates
 - **Test fixtures MUST import actual components**, not duplicate HTML
-- Reference: https://docs.astro.build/en/reference/container-reference/
-- Use `experimental_AstroContainer.create()` to instantiate container
-- Use `container.renderToString(Component)` to render
-- Use `// @vitest-environment happy-dom` for DOM support
-- Configure Vitest with `getViteConfig()` from 'astro/config'
-- Naming: `client.spec.ts` for tests, `componentName.fixture.astro` for fixtures
-- Working example: `src/components/Test/container.spec.ts`
+- Naming: `filename.spec.ts` for tests, `componentName.fixture.astro` for fixtures
+- Working example: `src/components/Test/__tests__/webComponent.spec.ts`
 
 ## E2E Testing
 

eslint.config.ts

@@ -429,10 +429,10 @@ export default [
 
   {
     files: [
-      'src/pages/api/gdpr/consent.ts',
-      'src/pages/api/gdpr/request-data.ts',
-      'src/pages/api/gdpr/verify.ts',
-      'src/pages/api/newsletter/_token.ts',
+      'src/lib/config/serviceWorker.ts',
+      'src/components/scripts/store/__tests__/socialEmbeds.spec.ts',
+      'src/pages/api/**/*',
+      'src/pages/manifest.json.ts',
     ],
     rules: {
       camelcase: 'off',

src/components/scripts/store/__tests__/socialEmbeds.spec.ts

@@ -107,10 +107,8 @@ describe('Embed Cache Management', () => {
 
   it('should expose cache state getters and setters', () => {
     const state = {
-      /* eslint-disable camelcase */
       entry_a: { data: { html: 'A' }, timestamp: 0, ttl: 1000 },
       entry_b: { data: { html: 'B' }, timestamp: 0, ttl: 1000 },
-      /* eslint-enable camelcase */
     }
 
     setEmbedCacheState(state)

src/lib/config/serviceWorker.ts

@@ -8,9 +8,7 @@ export const serviceWorkerConfig: PwaOptions = {
   registerType: 'autoUpdate',
   manifest: {
     name: 'Webstack Builders',
-    // eslint-disable-next-line camelcase
     short_name: 'WSB',
-    // eslint-disable-next-line camelcase
     theme_color: '#ffffff',
     icons: [
       {

src/pages/api/newsletter/index.ts

@@ -100,17 +100,14 @@ export async function subscribeToConvertKit(
       subscriber: {
         id: 999999,
         state: 'active',
-        /* eslint-disable camelcase */
         email_address: data.email,
         first_name: data.firstName || null,
         created_at: new Date().toISOString(),
-        /* eslint-enable camelcase */
         fields: {},
       },
     }
   }
 
-  /* eslint-disable camelcase */
   const subscriberData: ConvertKitSubscriber = {
     email_address: data.email,
     state: 'active',
@@ -119,7 +116,6 @@ export async function subscribeToConvertKit(
   if (data.firstName) {
     subscriberData.first_name = data.firstName.trim()
   }
-  /* eslint-enable camelcase */
 
   try {
     const response = await fetch('https://api.kit.com/v4/subscribers', {

src/pages/manifest.json.ts

@@ -25,7 +25,6 @@ const buildManifestPayload = () => {
     })
   }
 
-  /* eslint-disable camelcase */
   const manifest = {
     lang: 'en_US',
     dir: 'ltr',