chore: remove four uncalled dev-tier flags

src/services/tier.ts

@@ -96,11 +96,6 @@ export function hasEmbeddings(): boolean {
   return getTier() !== "free";
 }
 
-/** Whether session close compliance validation is active (dev only) */
-export function hasCompliance(): boolean {
-  return getTier() === "dev";
-}
-
 /**
  * Whether scar variant A/B testing is active (dev only).
  *
@@ -139,21 +134,6 @@ export function hasMetrics(): boolean {
   return getTier() !== "free";
 }
 
-/** Whether advanced agent detection (5-agent matrix) is active (dev only) */
-export function hasAdvancedAgentDetection(): boolean {
-  return getTier() === "dev";
-}
-
-/** Whether multi-project support is active (dev only) */
-export function hasMultiProject(): boolean {
-  return getTier() === "dev";
-}
-
-/** Whether LLM-cooperative enforcement fields are generated (dev only) */
-export function hasEnforcementFields(): boolean {
-  return getTier() === "dev";
-}
-
 /**
  * Get the table prefix for the current tier
  */

tests/helpers/supabase-mocks.ts

@@ -17,15 +17,11 @@ export const SUPABASE_TIER_MOCKS = {
   hasTranscripts: vi.fn(() => false), // Avoid transcript capture complexity in most tests
   hasCacheManagement: vi.fn(() => true),
   hasVariants: vi.fn(() => true),
-  hasCompliance: vi.fn(() => false),
   hasEmbeddings: vi.fn(() => true),
   hasMetrics: vi.fn(() => true),
   hasProInsights: vi.fn(() => true),
   getTier: vi.fn(() => "pro" as const),
   resetTier: vi.fn(),
-  hasAdvancedAgentDetection: vi.fn(() => false),
-  hasMultiProject: vi.fn(() => false),
-  hasEnforcementFields: vi.fn(() => false),
   getTablePrefix: vi.fn(() => "orchestra_"),
   getTableName: vi.fn((base: string) => `orchestra_${base}`),
 };

tests/unit/tools/recall-surfacing.test.ts

@@ -27,13 +27,9 @@ vi.mock("../../../src/services/tier.js", () => ({
   hasVariants: () => false,
   hasMetrics: () => false,
   hasCacheManagement: () => true,
-  hasCompliance: () => false,
   hasTranscripts: () => false,
   hasBatchOperations: () => false,
   hasEmbeddings: () => true,
-  hasAdvancedAgentDetection: () => false,
-  hasMultiProject: () => false,
-  hasEnforcementFields: () => false,
   hasProInsights: () => false,
   getTablePrefix: () => "gitmem_",
   getTableName: (base: string) => `gitmem_${base}`,

tests/unit/tools/recall.test.ts

@@ -21,13 +21,9 @@ vi.mock("../../../src/services/tier.js", () => ({
   hasMetrics: () => false,
   hasProInsights: () => false,
   hasCacheManagement: () => true,
-  hasCompliance: () => false,
   hasTranscripts: () => false,
   hasBatchOperations: () => false,
   hasEmbeddings: () => true,
-  hasAdvancedAgentDetection: () => false,
-  hasMultiProject: () => false,
-  hasEnforcementFields: () => false,
   getTablePrefix: () => "gitmem_",
   getTableName: (base: string) => `gitmem_${base}`,
 }));

tests/unit/tools/session-close-bridge.test.ts

@@ -40,14 +40,10 @@ vi.mock("../../../src/services/tier.js", () => ({
   hasTranscripts: () => false,
   hasCacheManagement: () => true,
   hasVariants: () => true,
-  hasCompliance: () => false,
   hasEmbeddings: () => true,
   hasMetrics: () => true,
   getTier: () => "pro",
   resetTier: () => {},
-  hasAdvancedAgentDetection: () => false,
-  hasMultiProject: () => false,
-  hasEnforcementFields: () => false,
   hasProInsights: () => false,
   getTablePrefix: () => "orchestra_",
   getTableName: (base: string) => `orchestra_${base}`,

tests/unit/tools/session-close-supabase.test.ts

@@ -47,14 +47,10 @@ vi.mock("../../../src/services/tier.js", () => ({
   hasTranscripts: () => false,
   hasCacheManagement: () => true,
   hasVariants: () => true,
-  hasCompliance: () => false,
   hasEmbeddings: () => true,
   hasMetrics: () => true,
   getTier: () => "pro",
   resetTier: () => {},
-  hasAdvancedAgentDetection: () => false,
-  hasMultiProject: () => false,
-  hasEnforcementFields: () => false,
   hasProInsights: () => false,
   getTablePrefix: () => "orchestra_",
   getTableName: (base: string) => `orchestra_${base}`,