fix(ui): remove duplicate instanceId getter breaking main CI

steipete · steipete · commit 3d3b74f4fea9 · 2026-07-19T03:13:41.000-07:00
PRs #111225 and #110831 each added an identical `get instanceId()` to GatewayApi; merged together they declared the identifier twice, failing typecheck (TS2300) and lint across all CI (prod/test-types, lint, boundaries, QA smoke, compact shards). Keep the typed getter, drop the duplicate.
diff --git a/ui/src/api/gateway.ts b/ui/src/api/gateway.ts
@@ -379,10 +379,6 @@ export class GatewayBrowserClient {
     return this.client.connected;
   }
 
-  get instanceId() {
-    return this.opts.instanceId;
-  }
-
   get recoveryScope() {
     return this.recoveryScopeTracker.scope;
   }

Original file line number	Diff line number	Diff line change
`@@ -379,10 +379,6 @@ export class GatewayBrowserClient {`
`379`	`379`	`return this.client.connected;`
`380`	`380`	`}`
`381`	`381`
`382`		`- get instanceId() {`
`383`		`- return this.opts.instanceId;`
`384`		`- }`
`385`		`-`
`386`	`382`	`get recoveryScope() {`
`387`	`383`	`return this.recoveryScopeTracker.scope;`
`388`	`384`	`}`