Update dependency @effect/tsgo to v0.53.0

package.json

@@ -41,7 +41,7 @@
   "devDependencies": {
     "@commy/testing": "workspace:*",
     "@biomejs/biome": "2.6.3",
-    "@effect/tsgo": "0.52.0",
+    "@effect/tsgo": "0.53.0",
     "bun-types": "1.4.4",
     "turbo": "2.11.10",
     "typescript": "7.0.2"

packages/core/ports.test.ts

@@ -17,14 +17,20 @@ import {
   UnknownIdentity,
 } from './ports.ts'
 
-test('UnknownIdentity names both the name and the substrate in its message', () => {
-  const err = new UnknownIdentity({ name: 'polecat-7', substrate: 'discord' })
+test('UnknownIdentity names both the bot name and the substrate in its message', () => {
+  const err = new UnknownIdentity({ botName: 'polecat-7', substrate: 'discord' })
   expect(err).toBeInstanceOf(Error)
   expect(err.message).toContain('polecat-7')
   expect(err.message).toContain('discord')
   expect(err._tag).toBe('UnknownIdentity')
 })
 
+test('UnknownIdentity keeps its error type as its name', () => {
+  const err = new UnknownIdentity({ botName: 'polecat-7', substrate: 'discord' })
+  expect(err.name).not.toBe('polecat-7')
+  expect(String(err)).not.toStartWith('polecat-7')
+})
+
 const runEither = <A, E>(effect: Effect.Effect<A, E>): Either.Either<A, E> =>
   Effect.runSync(Effect.either(effect))
 

packages/core/ports.ts

@@ -889,11 +889,11 @@ export interface AgentComms {
  * from a generic `IdentityError`.
  */
 export class UnknownIdentity extends Data.TaggedError('UnknownIdentity')<{
-  readonly name: string
+  readonly botName: string
   readonly substrate: string
 }> {
   override get message(): string {
-    return `acquire(${this.name}) failed — ${this.substrate} substrate has no entry for that name and cannot mint one`
+    return `acquire(${this.botName}) failed — ${this.substrate} substrate has no entry for that name and cannot mint one`
   }
 }
 

packages/memory/adapter.ts

@@ -621,7 +621,7 @@ const memorySeat = (realm: MemoryRealm, config: MemorySeatConfig): Effect.Effect
                     ),
               onNone: (): Effect.Effect<AcquiredIdentity, UnknownIdentity | IdentityError> =>
                 allowlist !== undefined && !allowlist.has(name)
-                  ? Effect.fail(new UnknownIdentity({ name, substrate: 'memory' }))
+                  ? Effect.fail(new UnknownIdentity({ botName: name, substrate: 'memory' }))
                   : isCaseVariantOfExisting(name) || isNameOfHuman(name)
                     ? Effect.fail(
                         new IdentityError({

packages/zulip/adapter.ts

@@ -258,11 +258,11 @@ const identityError = (operation: IdentityError['operation'], cause: unknown): I
  * an operator provisioning gap, not something to paper over by self-minting.
  */
 class AttachIdentityNotFound extends Data.TaggedError('AttachIdentityNotFound')<{
-  readonly name: BotName
+  readonly botName: BotName
 }> {
   override get message(): string {
     return (
-      `attach mode: no provisioned bot named '${this.name}' on the realm. ` +
+      `attach mode: no provisioned bot named '${this.botName}' on the realm. ` +
       `Provision the persona once and supply its stable api key before attaching.`
     )
   }
@@ -1182,7 +1182,7 @@ export const zulipAdapter = (
         Effect.flatMap(
           Option.match({
             onNone: (): Effect.Effect<BoundBot, AttachIdentityNotFound | ParseResult.ParseError> =>
-              Effect.fail(new AttachIdentityNotFound({ name })),
+              Effect.fail(new AttachIdentityNotFound({ botName: name })),
             onSome: (
               existing,
             ): Effect.Effect<BoundBot, AttachIdentityNotFound | ParseResult.ParseError> =>