@prmichaelsen/remember-mcp 3.13.0 → 3.14.1

package/src/services/space-config.service.spec.ts

@@ -1,102 +0,0 @@
-import { getSpaceConfig, setSpaceConfig, DEFAULT_SPACE_CONFIG } from './space-config.service.js';
-import * as firestoreInit from '../firestore/init';
-
-jest.mock('../firestore/init', () => ({
-  getDocument: jest.fn(),
-  setDocument: jest.fn(),
-}));
-
-const mockGetDocument = firestoreInit.getDocument as jest.MockedFunction<typeof firestoreInit.getDocument>;
-const mockSetDocument = firestoreInit.setDocument as jest.MockedFunction<typeof firestoreInit.setDocument>;
-
-describe('SpaceConfigService', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe('getSpaceConfig', () => {
-    it('returns defaults when no Firestore doc exists', async () => {
-      mockGetDocument.mockResolvedValue(null);
-
-      const config = await getSpaceConfig('public', 'space');
-
-      expect(config).toEqual(DEFAULT_SPACE_CONFIG);
-      expect(config.require_moderation).toBe(false);
-      expect(config.default_write_mode).toBe('owner_only');
-    });
-
-    it('merges stored config with defaults', async () => {
-      mockGetDocument.mockResolvedValue({ require_moderation: true });
-
-      const config = await getSpaceConfig('moderated-group', 'group');
-
-      expect(config.require_moderation).toBe(true);
-      expect(config.default_write_mode).toBe('owner_only');
-    });
-
-    it('uses correct Firestore path for spaces', async () => {
-      mockGetDocument.mockResolvedValue(null);
-
-      await getSpaceConfig('public', 'space');
-
-      expect(mockGetDocument).toHaveBeenCalledWith('spaces/public/config', 'settings');
-    });
-
-    it('uses correct Firestore path for groups', async () => {
-      mockGetDocument.mockResolvedValue(null);
-
-      await getSpaceConfig('team-alpha', 'group');
-
-      expect(mockGetDocument).toHaveBeenCalledWith('groups/team-alpha/config', 'settings');
-    });
-
-    it('returns defaults on Firestore error', async () => {
-      mockGetDocument.mockRejectedValue(new Error('Firestore unavailable'));
-
-      const config = await getSpaceConfig('public', 'space');
-
-      expect(config).toEqual(DEFAULT_SPACE_CONFIG);
-    });
-  });
-
-  describe('setSpaceConfig', () => {
-    it('writes to correct Firestore path for spaces', async () => {
-      mockSetDocument.mockResolvedValue(undefined);
-
-      await setSpaceConfig('public', 'space', { require_moderation: true });
-
-      expect(mockSetDocument).toHaveBeenCalledWith(
-        'spaces/public/config',
-        'settings',
-        { require_moderation: true },
-        { merge: true }
-      );
-    });
-
-    it('writes to correct Firestore path for groups', async () => {
-      mockSetDocument.mockResolvedValue(undefined);
-
-      await setSpaceConfig('team-alpha', 'group', { default_write_mode: 'group_editors' });
-
-      expect(mockSetDocument).toHaveBeenCalledWith(
-        'groups/team-alpha/config',
-        'settings',
-        { default_write_mode: 'group_editors' },
-        { merge: true }
-      );
-    });
-
-    it('supports partial config updates', async () => {
-      mockSetDocument.mockResolvedValue(undefined);
-
-      await setSpaceConfig('public', 'space', { require_moderation: true });
-
-      expect(mockSetDocument).toHaveBeenCalledWith(
-        'spaces/public/config',
-        'settings',
-        { require_moderation: true },
-        { merge: true }
-      );
-    });
-  });
-});

package/src/services/space-config.service.ts

@@ -1,79 +0,0 @@
-/**
- * Space/Group Configuration Service
- *
- * Per-space and per-group behavioral configuration stored in Firestore.
- * Controls moderation requirements, write modes, and other behavioral rules.
- */
-
-import { getDocument, setDocument } from '../firestore/init.js';
-import { logger } from '../utils/logger.js';
-import type { WriteMode } from '../types/auth.js';
-
-export interface SpaceConfig {
-  require_moderation: boolean;
-  default_write_mode: WriteMode;
-}
-
-export const DEFAULT_SPACE_CONFIG: SpaceConfig = {
-  require_moderation: false,
-  default_write_mode: 'owner_only',
-};
-
-/**
- * Get the Firestore path for a space or group config document.
- */
-function getConfigPath(id: string, type: 'space' | 'group'): { collectionPath: string; docId: string } {
-  const prefix = type === 'space' ? 'spaces' : 'groups';
-  return {
-    collectionPath: `${prefix}/${id}/config`,
-    docId: 'settings',
-  };
-}
-
-/**
- * Get configuration for a space or group.
- * Returns defaults merged with any stored config.
- */
-export async function getSpaceConfig(
-  id: string,
-  type: 'space' | 'group'
-): Promise<SpaceConfig> {
-  try {
-    const { collectionPath, docId } = getConfigPath(id, type);
-    const doc = await getDocument(collectionPath, docId);
-
-    if (!doc) {
-      return { ...DEFAULT_SPACE_CONFIG };
-    }
-
-    return { ...DEFAULT_SPACE_CONFIG, ...doc };
-  } catch (error) {
-    logger.error('Failed to get space config', {
-      service: 'SpaceConfigService',
-      id,
-      type,
-      error: error instanceof Error ? error.message : String(error),
-    });
-    return { ...DEFAULT_SPACE_CONFIG };
-  }
-}
-
-/**
- * Set configuration for a space or group.
- * Merges partial config with existing values.
- */
-export async function setSpaceConfig(
-  id: string,
-  type: 'space' | 'group',
-  config: Partial<SpaceConfig>
-): Promise<void> {
-  const { collectionPath, docId } = getConfigPath(id, type);
-  await setDocument(collectionPath, docId, config, { merge: true });
-
-  logger.info('Space config updated', {
-    service: 'SpaceConfigService',
-    id,
-    type,
-    updatedKeys: Object.keys(config),
-  });
-}