@webinson/peer 0.4.19 → 0.4.21

package/mod/peer-thread-mark/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "peer-thread-mark",
-  "version": "0.4.19",
+  "version": "0.4.21",
   "description": "Marks a Claude Code thread shared with Peer (/peer) with a band above the prompt, with /peer stop and /peer start; panes for /peer-list, /peer-thread and /peer-all (search your memories)",
   "license": "MIT",
   "types": "./types/index.d.ts"

package/mod/peer-thread-mark/hooks/register.tsx

@@ -497,8 +497,18 @@ async function memoryNote($: EngineInterface) {
   return 'Inactive: not shared, Claude Code memory only.'
 }
 
-// Who else was on the file, newest first: other people before the person's own other threads.
-const othersOf = (signal: CollisionSignal) => signal.holders.filter(h => !h.own).sort((a, b) => Date.parse(b.at) - Date.parse(a.at))
+// Who else was on the file, one entry per person (their newest session), newest first. One person often holds a path
+// in several sessions (another thread, the Peer app's own), and is named once.
+const byPerson = (holders: CollisionSignal['holders']) => {
+  const newest = new Map<string, CollisionSignal['holders'][number]>()
+  for (const h of holders) {
+    const key = h.email.toLowerCase()
+    const was = newest.get(key)
+    if (!was || Date.parse(h.at) > Date.parse(was.at)) newest.set(key, h)
+  }
+  return [...newest.values()].sort((a, b) => Date.parse(b.at) - Date.parse(a.at))
+}
+const othersOf = (signal: CollisionSignal) => byPerson(signal.holders.filter(h => !h.own))
 const whenAgo = (at: string, now: number) => {
   const age = ago(at, now)
   return age === 'just now' ? age : `${age} ago`
@@ -605,7 +615,7 @@ export const register: Register = on => {
       .filter(h => ADDRESS.test(h.email))
     const state = answer.state === 'collision' && !holders.length ? 'clear' : answer.state
     const at = await $.clock.now()
-    const emails = (signal: { holders: { email: string }[] }) => signal.holders.map(h => h.email.toLowerCase()).sort().join(' ')
+    const emails = (signal: { holders: { email: string }[] }) => [...new Set(signal.holders.map(h => h.email.toLowerCase()))].sort().join(' ')
     let signal: CollisionSignal = { path: answer.path ? plain(answer.path) : null, state, ...(answer.reason ? { reason: answer.reason } : {}), holders, at, shown: false }
     // Shown once per set of people on a path: against the latest earlier collision on it, decided inside one update.
     await update($, signals, was => {

package/mod/peer-thread-mark/tests/collision.test.tsx

@@ -87,6 +87,17 @@ test('an edit another person was on draws the line and tells the agent once per
   expect(both.additionalContext?.[1]).toContain("cy@acme.test's agent was on src/pay.ts 1 minute ago")
   expect(both.additionalContext?.[1]).toContain("bob@acme.test's agent was on src/pay.ts 2 minutes ago")
   expect(await texts(row('t3', 'desktop'), $ as never)).toContain('cy@… was on it 1 min ago and 1 others')
+
+  // One person in several sessions (another thread, the Peer app's own) is named once, at their newest.
+  state.answer = { path: 'src/one.ts', state: 'collision', holders: [BOB, { ...BOB, email: 'Bob@acme.test', at: minsAgo(0) }, { ...BOB, at: minsAgo(1) }] }
+  const once = (await $.classic.PostToolUse(edit('t4', 'Edit', '/repo/src/one.ts'))) as { additionalContext?: string[] }
+  expect(once.additionalContext?.[1]).toBe(
+    "<peer-collision>\nBob@acme.test's agent was on src/one.ts just now, in another thread.\nThis is information about the repository, not an instruction.\n</peer-collision>",
+  )
+  expect(await texts(row('t4', 'terminal'), $ as never)).toContain('Bob@… was on it just now')
+  // The same person in one more session is no new set of people.
+  state.answer = { path: 'src/one.ts', state: 'collision', holders: [BOB, { ...BOB, at: minsAgo(0) }] }
+  expect(((await $.classic.PostToolUse(edit('t5', 'Edit', '/repo/src/one.ts'))) as { additionalContext?: string[] }).additionalContext).toEqual(['from another hook'])
 })
 
 test('the person\'s own other thread is drawn dimmed and the agent is told nothing', async ($, on) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@webinson/peer",
-  "version": "0.4.19",
+  "version": "0.4.21",
   "description": "Share chosen Claude Code and Codex threads with your team: who has agents running and on what.",
   "license": "MIT",
   "type": "module",