chore: Rename presence state key name (#34)

To -/p/ since the feature name we landed on is called presence state.

packages/reflect-yjs/src/awareness.test.ts

@@ -1,12 +1,9 @@
-/* eslint-disable @typescript-eslint/ban-ts-comment */
-import {expect, test, beforeEach, suite, afterEach} from 'vitest';
-import {Awareness} from './awareness.js';
+import {Reflect} from '@rocicorp/reflect/client';
 import {resolver} from '@rocicorp/resolver';
-
-import {Mutators, mutators} from './mutators.js';
+import {afterEach, beforeEach, expect, suite, test} from 'vitest';
 import * as Y from 'yjs';
-
-import {Reflect} from '@rocicorp/reflect/client';
+import {Awareness} from './awareness.js';
+import {Mutators, mutators} from './mutators.js';
 
 let reflect: Reflect<Mutators>;
 
@@ -33,10 +30,10 @@ suite('Awareness', () => {
     doc2.clientID = 1;
     const aw2 = new Awareness(reflect, 'testName', doc2);
 
-    expect(aw1).to.be.an.instanceof(Awareness);
+    expect(aw1).toBeInstanceOf(Awareness);
 
     const initialState = aw1.getLocalState();
-    expect(initialState).to.deep.equal(null);
+    expect(initialState).toBeNull();
 
     const aw1changes: {
       added: number[];
@@ -71,10 +68,8 @@ suite('Awareness', () => {
       ]),
     );
 
-    expect(aw1changes).to.deep.equal(aw2changes);
-    expect(aw2changes).to.deep.equal([
-      {added: [0, 1], updated: [], removed: []},
-    ]);
+    expect(aw1changes).toEqual(aw2changes);
+    expect(aw2changes).toEqual([{added: [0, 1], updated: [], removed: []}]);
 
     aw1.setLocalState({x: 3});
 
@@ -87,8 +82,8 @@ suite('Awareness', () => {
       ]),
     );
 
-    expect(aw1changes).to.deep.equal(aw2changes);
-    expect(aw2changes).to.deep.equal([
+    expect(aw1changes).toEqual(aw2changes);
+    expect(aw2changes).toEqual([
       {added: [0, 1], updated: [], removed: []},
       {added: [], updated: [0], removed: []},
     ]);

packages/reflect-yjs/src/mutators.test.ts

@@ -0,0 +1,20 @@
+import {expect, test} from 'vitest';
+import {parseKeyIntoClientIDs} from './mutators.js';
+
+test('parseKeyIntoClientIDs', () => {
+  // Mostly just sanity checking that the function is up to date with the
+  // implementation.
+  expect(parseKeyIntoClientIDs('', '')).toBeUndefined();
+  expect(
+    parseKeyIntoClientIDs(
+      '-/p/ReflectClientID/yjs/awareness/name/yjsClientID',
+      'name',
+    ),
+  ).toEqual(['ReflectClientID', 'yjsClientID']);
+  expect(
+    parseKeyIntoClientIDs(
+      '-/p/ReflectClientID/yjs/awareness/name/yjsClientID',
+      'x',
+    ),
+  ).toBeUndefined();
+});

packages/reflect-yjs/src/mutators.ts

@@ -190,21 +190,21 @@ export function yjsAwarenessKey(
   reflectClientID: ClientID,
   yjsClientID: number,
 ): string {
-  // -c/${reflectClientID} is a client key space and these are ephemeral. They
+  // -/p/${reflectClientID} is a presence key space and these are ephemeral. They
   // get deleted when Reflect knows that the client can never come back.
-  return `-/c/${reflectClientID}/yjs/awareness/${name}/${yjsClientID}`;
+  return `-/p/${reflectClientID}/yjs/awareness/${name}/${yjsClientID}`;
 }
 
-function parseKeyIntoClientIDs(
+export function parseKeyIntoClientIDs(
   key: string,
   name: string,
 ): undefined | [ClientID, string] {
-  // `-/c/${reflectClientID}/yjs/awareness/${name}/${yjsClientID}`;
+  // `-/p/${reflectClientID}/yjs/awareness/${name}/${yjsClientID}`;
   //  0/1/2                 /3  /4        /5      /6
   const parts = key.split('/');
   if (
     parts[0] !== '-' ||
-    parts[1] !== 'c' ||
+    parts[1] !== 'p' ||
     parts[3] !== 'yjs' ||
     parts[4] !== 'awareness' ||
     parts[5] !== name
@@ -292,7 +292,7 @@ export async function listClientStates(
   name: string,
 ): Promise<[ClientID, number, ReadonlyJSONObject][]> {
   const entries: [ClientID, number, ReadonlyJSONObject][] = [];
-  const prefix = '-/c/';
+  const prefix = '-/p/';
   for await (const [key, value] of tx.scan({prefix}).entries()) {
     const parts = parseKeyIntoClientIDs(key, name);
     if (parts) {