@memberjunction/server 6.1.4 → 6.2.0-edge.1

package/src/resolvers/TaskGraphFrameResolver.ts

@@ -26,86 +26,86 @@ export const TASK_GRAPH_FRAMES_TOPIC = 'TASK_GRAPH_FRAMES';
 @ObjectType()
 export class TaskGraphFrameNotification {
   @Field(() => String)
-  kind!: string;
+  kind!: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => ID)
-  parentTaskId!: string;
+  parentTaskId!: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => ID, { nullable: true })
-  taskId?: string;
+  taskId?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String, { nullable: true })
-  taskName?: string;
+  taskName?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String, { nullable: true })
-  status?: string;
+  status?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String, { nullable: true })
-  errorMessage?: string;
+  errorMessage?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => ID, { nullable: true })
-  assignedUserId?: string;
+  assignedUserId?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => Number, { nullable: true })
-  completedCount?: number;
+  completedCount?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => Number, { nullable: true })
-  totalCount?: number;
+  totalCount?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   // ── GateDecision ──────────────────────────────────────────────────────────
   @Field(() => ID, { nullable: true })
-  edgeId?: string;
+  edgeId?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => ID, { nullable: true })
-  dependsOnTaskId?: string;
+  dependsOnTaskId?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   /** 'satisfied' | 'notTaken' | 'held' */
   @Field(() => String, { nullable: true })
-  verdict?: string;
+  verdict?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String, { nullable: true })
-  conditionText?: string;
+  conditionText?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String, { nullable: true })
-  reason?: string;
+  reason?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   // ── ClaimChanged ──────────────────────────────────────────────────────────
   /** 'claimed' | 'heartbeat-lost' | 'reclaimed' */
   @Field(() => String, { nullable: true })
-  claimEvent?: string;
+  claimEvent?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String, { nullable: true })
-  claimedBy?: string;
+  claimedBy?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => String, { nullable: true })
-  claimExpiresAt?: string;
+  claimExpiresAt?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   // ── PassCompleted ─────────────────────────────────────────────────────────
   @Field(() => Number, { nullable: true })
-  passNumber?: number;
+  passNumber?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => Number, { nullable: true })
-  eligibleCount?: number;
+  eligibleCount?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => Number, { nullable: true })
-  heldCount?: number;
+  heldCount?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => Number, { nullable: true })
-  claimedCount?: number;
+  claimedCount?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   /** The dispatcher instance's own load across every graph — not this graph's in-flight count. */
   @Field(() => Number, { nullable: true })
-  instanceInFlightCount?: number;
+  instanceInFlightCount?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   // ── NodeProgress ──────────────────────────────────────────────────────────
   @Field(() => String, { nullable: true })
-  progressMessage?: string;
+  progressMessage?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => Number, { nullable: true })
-  progressPercent?: number;
+  progressPercent?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
   @Field(() => Date)
-  date!: Date;
+  date!: Date;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 /**
@@ -137,7 +137,7 @@ export interface TaskGraphFrameFilterContext {
  * another user's workflow — including its per-step error messages. **Fails closed** — a missing
  * identity on either side never matches.
  */
-export function taskGraphFrameFilter(data: {
+export function TaskGraphFrameFilter(data: {
   payload: TaskGraphFramePayload;
   args: TaskGraphFrameArgs;
   context: TaskGraphFrameFilterContext | undefined;
@@ -156,6 +156,15 @@ export function taskGraphFrameFilter(data: {
   return UUIDsEqual(payload.ownerUserId, connectionUserId);
 }
 
+/** @deprecated Use {@link TaskGraphFrameFilter}. */
+export function taskGraphFrameFilter(data: {
+  payload: TaskGraphFramePayload;
+  args: TaskGraphFrameArgs;
+  context: TaskGraphFrameFilterContext | undefined;
+}): boolean {
+  return TaskGraphFrameFilter(data);
+}
+
 /**
  * MJServer's `TaskGraphObserver` — turns dispatcher frames into a PubSub publish.
  *
@@ -189,9 +198,9 @@ export class TaskGraphFrameResolver {
   @Subscription(() => TaskGraphFrameNotification, {
     topics: TASK_GRAPH_FRAMES_TOPIC,
     filter: (data: ResolverFilterData<TaskGraphFramePayload, TaskGraphFrameArgs, TaskGraphFrameFilterContext>) =>
-      taskGraphFrameFilter(data),
+      TaskGraphFrameFilter(data),
   })
-  taskGraphFrames(
+  taskGraphFrames(  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
     @Root() payload: TaskGraphFramePayload,
     @Arg('parentTaskId', () => ID) parentTaskId: string
   ): TaskGraphFrameNotification {

package/src/resolvers/TelemetryResolver.ts

@@ -13,58 +13,58 @@ import {
 @ObjectType()
 export class MemorySnapshotGQL {
     @Field(() => Float)
-    heapUsed: number;
+    heapUsed: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    heapTotal: number;
+    heapTotal: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    timestamp: number;
+    timestamp: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class TelemetryEventGQL {
     @Field(() => String)
-    id: string;
+    id: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    category: string;
+    category: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    operation: string;
+    operation: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    fingerprint: string;
+    fingerprint: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    startTime: number;
+    startTime: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float, { nullable: true })
-    endTime?: number;
+    endTime?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float, { nullable: true })
-    elapsedMs?: number;
+    elapsedMs?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    userId?: string;
+    userId?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    params: string; // JSON stringified
+    params: string; // JSON stringified — case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => [String], { nullable: true })
-    tags?: string[];
+    tags?: string[];  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    stackTrace?: string;
+    stackTrace?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => MemorySnapshotGQL, { nullable: true })
-    memoryBefore?: MemorySnapshotGQL;
+    memoryBefore?: MemorySnapshotGQL;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => MemorySnapshotGQL, { nullable: true })
-    memoryAfter?: MemorySnapshotGQL;
+    memoryAfter?: MemorySnapshotGQL;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    parentEventId?: string;
+    parentEventId?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 // ============================================================================
@@ -74,52 +74,52 @@ export class TelemetryEventGQL {
 @ObjectType()
 export class CallerLocationGQL {
     @Field(() => String)
-    location: string;
+    location: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    count: number;
+    count: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class TelemetryPatternGQL {
     @Field(() => String)
-    fingerprint: string;
+    fingerprint: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    category: string;
+    category: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    operation: string;
+    operation: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    sampleParams: string; // JSON stringified
+    sampleParams: string; // JSON stringified — case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    count: number;
+    count: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    totalElapsedMs: number;
+    totalElapsedMs: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    avgElapsedMs: number;
+    avgElapsedMs: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    minElapsedMs: number;
+    minElapsedMs: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    maxElapsedMs: number;
+    maxElapsedMs: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => [CallerLocationGQL])
-    callerLocations: CallerLocationGQL[];
+    callerLocations: CallerLocationGQL[];  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    firstSeen: number;
+    firstSeen: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    lastSeen: number;
+    lastSeen: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    windowStartTime: number;
+    windowStartTime: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 // ============================================================================
@@ -129,37 +129,37 @@ export class TelemetryPatternGQL {
 @ObjectType()
 export class TelemetryInsightGQL {
     @Field(() => String)
-    id: string;
+    id: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    severity: string;
+    severity: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    analyzerName: string;
+    analyzerName: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    category: string;
+    category: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    title: string;
+    title: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    message: string;
+    message: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    suggestion: string;
+    suggestion: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => [String])
-    relatedEventIds: string[];
+    relatedEventIds: string[];  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    entityName?: string;
+    entityName?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    metadata?: string; // JSON stringified
+    metadata?: string; // JSON stringified — case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    timestamp: number;
+    timestamp: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 // ============================================================================
@@ -169,31 +169,31 @@ export class TelemetryInsightGQL {
 @ObjectType()
 export class CategoryStatsGQL {
     @Field(() => String)
-    category: string;
+    category: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    events: number;
+    events: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float)
-    avgMs: number;
+    avgMs: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class TelemetryStatsGQL {
     @Field(() => Int)
-    totalEvents: number;
+    totalEvents: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    totalPatterns: number;
+    totalPatterns: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    totalInsights: number;
+    totalInsights: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    activeEvents: number;
+    activeEvents: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => [CategoryStatsGQL])
-    byCategory: CategoryStatsGQL[];
+    byCategory: CategoryStatsGQL[];  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 // ============================================================================
@@ -203,64 +203,64 @@ export class TelemetryStatsGQL {
 @ObjectType()
 export class CategoryOverrideGQL {
     @Field(() => String)
-    category: string;
+    category: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Boolean)
-    enabled: boolean;
+    enabled: boolean;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    level?: string;
+    level?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class AutoTrimSettingsGQL {
     @Field(() => Boolean)
-    enabled: boolean;
+    enabled: boolean;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int, { nullable: true })
-    maxEvents?: number;
+    maxEvents?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int, { nullable: true })
-    maxAgeMs?: number;
+    maxAgeMs?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class DuplicateDetectionSettingsGQL {
     @Field(() => Boolean)
-    enabled: boolean;
+    enabled: boolean;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    windowMs: number;
+    windowMs: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class AnalyzerSettingsGQL {
     @Field(() => Boolean)
-    enabled: boolean;
+    enabled: boolean;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int)
-    dedupeWindowMs: number;
+    dedupeWindowMs: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @ObjectType()
 export class TelemetrySettingsGQL {
     @Field(() => Boolean)
-    enabled: boolean;
+    enabled: boolean;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String)
-    level: string;
+    level: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => [CategoryOverrideGQL])
-    categoryOverrides: CategoryOverrideGQL[];
+    categoryOverrides: CategoryOverrideGQL[];  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => AutoTrimSettingsGQL)
-    autoTrim: AutoTrimSettingsGQL;
+    autoTrim: AutoTrimSettingsGQL;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => DuplicateDetectionSettingsGQL)
-    duplicateDetection: DuplicateDetectionSettingsGQL;
+    duplicateDetection: DuplicateDetectionSettingsGQL;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => AnalyzerSettingsGQL)
-    analyzers: AnalyzerSettingsGQL;
+    analyzers: AnalyzerSettingsGQL;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 // ============================================================================
@@ -270,46 +270,46 @@ export class TelemetrySettingsGQL {
 @InputType()
 export class TelemetryEventFilterInput {
     @Field(() => String, { nullable: true })
-    category?: string;
+    category?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    operation?: string;
+    operation?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float, { nullable: true })
-    minElapsedMs?: number;
+    minElapsedMs?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Float, { nullable: true })
-    since?: number;
+    since?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int, { nullable: true })
-    limit?: number;
+    limit?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @InputType()
 export class TelemetryPatternFilterInput {
     @Field(() => String, { nullable: true })
-    category?: string;
+    category?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int, { nullable: true })
-    minCount?: number;
+    minCount?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    sortBy?: string;
+    sortBy?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 @InputType()
 export class TelemetryInsightFilterInput {
     @Field(() => String, { nullable: true })
-    severity?: string;
+    severity?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    category?: string;
+    category?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => String, { nullable: true })
-    entityName?: string;
+    entityName?: string;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 
     @Field(() => Int, { nullable: true })
-    limit?: number;
+    limit?: number;  // case-violation-ok-legacy-back-compat: the property name is the GraphQL schema field name — renaming it breaks every client query
 }
 
 // ============================================================================

package/src/resolvers/TransactionGroupResolver.ts

@@ -78,6 +78,19 @@ export class TransactionOutputType {
 
 
 
+/**
+ * Renders one refused transaction-group item for the server log, naming the position, entity and
+ * operation so an operator can tie the refusal back to the row the client sent.
+ *
+ * `CompleteMessage` rather than `Message` because a refusal can report itself through any of
+ * `Message`, `Error` or the `Errors` array — a validation failure populates only the last of the
+ * three, so reading `Message` alone logs an empty reason for exactly the case that matters most.
+ */
+function describeRefusedItem(index: number, item: TransactionItemInputType, entity: BaseEntity): string {
+    const reason = entity.LatestResult?.CompleteMessage?.trim();
+    return `  [${index}] ${item.OperationType} '${item.EntityName}': ${reason || 'refused without a reported reason'}`;
+}
+
 export class TransactionResolver extends ResolverBase {
     /**
      * Maps a TransactionGroup item's operation type onto the SAME API-key scope path the singular
@@ -123,8 +136,14 @@ export class TransactionResolver extends ResolverBase {
             const tg = await md.CreateTransactionGroup();
             const entityObjects: BaseEntity[] = [];
             const objectValues: any[] = [];
+            // #4309: Save()/Delete() report a LOGICAL refusal by returning false — they do not throw —
+            // and a refused row is never enrolled, because TransactionGroup.AddTransaction() is reached
+            // only from inside ProviderToUse.Save()/Delete(). Discarding that boolean let a group whose
+            // rows were all refused arrive at Submit() empty, take its legitimate "nothing to do"
+            // branch, and report success for writes that never happened.
+            const refusals: string[] = [];
 
-            for (const item of group.Items) {
+            for (const [index, item] of group.Items.entries()) {
                 // instantiate a new entity object for the item
                 const entity = await md.GetEntityObject(item.EntityName, context.userPayload.userRecord);
                 entityObjects.push(entity); // save for later for mapping variables if needed
@@ -147,17 +166,51 @@ export class TransactionResolver extends ResolverBase {
                         objectValues.push(itemValues);
                         entity.SetMany(itemValues, true);
                         entity.TransactionGroup = tg;
-                        await entity.Save();
+                        if (!await entity.Save()) {
+                            refusals.push(describeRefusedItem(index, item, entity));
+                        }
                         break;
                     case "Delete":
                         await entity.InnerLoad(pkey);
-                        objectValues.push(entity.GetDataObject());
+                        // AWAITED: GetDataObject() is async, and pushing the un-awaited Promise put
+                        // one into `objectValues` — which is `any[]`, so neither tsc nor a
+                        // floating-promise lint could see it. PrepareReturnValue's Delete branch
+                        // then stringified that Promise, and JSON.stringify(Promise) is "{}", so
+                        // EVERY Delete in a transaction group reported an empty payload, successful
+                        // ones included. Worse than losing the payload: GraphQLDataProvider's Delete
+                        // transaction callback validates the commit with
+                        // `pk.Value !== results[pk.FieldName]`, so "{}" made every key mismatch and
+                        // reported 'Transaction failed to commit' for a delete that DID commit.
+                        // Awaiting costs nothing here — DataObjectParams defaults
+                        // includeRelatedEntityData to false, which guards the method's only awaited
+                        // work, so with no params this is GetAll() plus a filter loop.
+                        objectValues.push(await entity.GetDataObject());
                         entity.TransactionGroup = tg;
-                        await entity.Delete();
+                        if (!await entity.Delete()) {
+                            refusals.push(describeRefusedItem(index, item, entity));
+                        }
                         break;
                 }
             }
 
+            // A refused row never enrolled, so submitting now would commit only the SURVIVORS and
+            // still report unqualified success. Nothing has been written at this point — enrolment
+            // is deferral, and the provider does not touch the database until Submit() — so
+            // returning here leaves the database exactly as we found it.
+            //
+            // The predicate is the RETURN VALUE, not whether the group ended up empty: a row that
+            // is not dirty also fails to enrol and correctly returns true, and an empty group
+            // legitimately means "nothing to do" for a caller that enrolled nothing. That is also
+            // why this belongs here rather than in TransactionGroupBase.Submit() — this is the only
+            // layer that still knows which row was refused and why.
+            if (refusals.length > 0) {
+                LogError(
+                    `TransactionResolver::ExecuteTransactionGroup --- ${refusals.length} of ${group.Items.length} ` +
+                    `item(s) were refused, so the group was not submitted:\n${refusals.join('\n')}`
+                );
+                return await this.PrepareReturnValue(false, entityObjects, objectValues, group);
+            }
+
             // now, we need to set the variables
             if (group.Variables && group.Variables.length > 0) {
                 for (const networkVar of group.Variables) {