Merge branch 'fix/siwe-cache-bypass' of github.com:elizaOS/eliza into fix/siwe-cache-bypass

Author: standujar

.github/workflows/cloud-gateway-discord.yml

@@ -446,6 +446,7 @@ jobs:
     if: |
       github.event_name != 'pull_request' &&
       (needs.detect-changes.outputs.app == 'true' || (github.event_name == 'workflow_dispatch' && github.event.inputs.action == 'deploy'))
+    environment: ${{ needs.detect-changes.outputs.environment }}
     permissions:
       contents: read
       packages: write
@@ -462,8 +463,8 @@ jobs:
         uses: docker/login-action@v4
         with:
           registry: ${{ env.REGISTRY }}
-          username: ${{ github.actor }}
-          password: ${{ secrets.GITHUB_TOKEN }}
+          username: ${{ secrets.GHCR_USERNAME }}
+          password: ${{ secrets.GHCR_TOKEN }}
 
       - name: Extract metadata
         id: meta

apps/app-lifeops/src/lifeops/service-mixin-browser.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 import crypto from "node:crypto";
 import {
   BROWSER_BRIDGE_KINDS,

apps/app-lifeops/src/lifeops/service-mixin-calendar.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 import type {
   CreateLifeOpsCalendarEventAttendee,
   CreateLifeOpsCalendarEventRequest,

apps/app-lifeops/src/lifeops/service-mixin-definitions.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 import type {
   CompleteLifeOpsOccurrenceRequest,
   CreateLifeOpsDefinitionRequest,

apps/app-lifeops/src/lifeops/service-mixin-discord.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 
 import { logger } from "@elizaos/core";
 import type {

apps/app-lifeops/src/lifeops/service-mixin-dossier.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 import crypto from "node:crypto";
 import { ModelType } from "@elizaos/core";
 import type { LifeOpsDossier } from "@elizaos/shared";

apps/app-lifeops/src/lifeops/service-mixin-drive.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 import type {
   LifeOpsConnectorMode,
   LifeOpsConnectorSide,

apps/app-lifeops/src/lifeops/service-mixin-email-unsubscribe.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 
 import crypto from "node:crypto";
 import type { LifeOpsConnectorGrant } from "@elizaos/shared";

apps/app-lifeops/src/lifeops/service-mixin-gmail.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 
 import { ModelType } from "@elizaos/core";
 import type {

apps/app-lifeops/src/lifeops/service-mixin-goals.ts

@@ -1,4 +1,10 @@
-// @ts-nocheck — mixin: type safety is enforced on the composed class
+// @ts-nocheck — Mixin pattern: each `withFoo()` returns a class that calls
+// methods belonging to sibling mixins (e.g. `this.recordScreenTimeEvent`).
+// Type checking each mixin in isolation surfaces 700+ phantom errors because
+// the local TBase constraint can't see sibling mixin methods. Real type
+// safety is enforced at the composed-service level (LifeOpsService class).
+// Refactoring requires either declaration-merging every cross-mixin method
+// or moving to a single composed interface — tracked as separate work.
 import type {
   CreateLifeOpsGoalRequest,
   LifeOpsChannelPolicy,