mirror of
https://github.com/trustgraph-ai/trustgraph.git
synced 2026-07-20 02:31:02 +02:00
Wrap client socket close in Effect
This commit is contained in:
parent
461b04d064
commit
d93b0adda8
2 changed files with 34 additions and 5 deletions
|
|
@ -12,8 +12,8 @@ Verified source roots:
|
||||||
- Effect v4 subtree: `/home/elpresidank/YeeBois/projects/beep-effect2/.repos/effect-v4`
|
- Effect v4 subtree: `/home/elpresidank/YeeBois/projects/beep-effect2/.repos/effect-v4`
|
||||||
- Installed Effect beta used by this workspace: `ts/node_modules/effect`
|
- Installed Effect beta used by this workspace: `ts/node_modules/effect`
|
||||||
|
|
||||||
Current signal counts from `ts/packages` after the 2026-06-02 client RPC
|
Current signal counts from `ts/packages` after the 2026-06-02 client socket
|
||||||
acquisition cause tap slice:
|
close Effect boundary slice:
|
||||||
|
|
||||||
| Signal | Count |
|
| Signal | Count |
|
||||||
| --- | ---: |
|
| --- | ---: |
|
||||||
|
|
@ -142,6 +142,10 @@ Notes:
|
||||||
only to update connection state on runtime/client acquisition failure.
|
only to update connection state on runtime/client acquisition failure.
|
||||||
`effect-rpc-client.ts` now uses `Effect.tapCause` and `Cause.pretty` before
|
`effect-rpc-client.ts` now uses `Effect.tapCause` and `Cause.pretty` before
|
||||||
the public Promise boundary.
|
the public Promise boundary.
|
||||||
|
- The client socket close Effect boundary slice removed the Promise `.catch`
|
||||||
|
from `BaseApi.close()`. The void public facade now runs `rpc.close()` through
|
||||||
|
`Effect.tryPromise` and logs the tagged socket close error through
|
||||||
|
`Effect.catch`.
|
||||||
- `Record<string, any>` and `throwLibrarianServiceError` are now clean in
|
- `Record<string, any>` and `throwLibrarianServiceError` are now clean in
|
||||||
`ts/packages`.
|
`ts/packages`.
|
||||||
|
|
||||||
|
|
@ -1010,6 +1014,26 @@ Notes:
|
||||||
- `cd ts && bun run test`
|
- `cd ts && bun run test`
|
||||||
- `git diff --check`
|
- `git diff --check`
|
||||||
|
|
||||||
|
### 2026-06-02: Client Socket Close Effect Boundary Slice
|
||||||
|
|
||||||
|
- Status: migrated and root-verified.
|
||||||
|
- Completed:
|
||||||
|
- `ts/packages/client/src/socket/trustgraph-socket.ts` now wraps
|
||||||
|
`rpc.close()` with `Effect.tryPromise` inside the public `close(): void`
|
||||||
|
facade.
|
||||||
|
- Close failures are mapped to the existing tagged `TrustGraphSocketError`
|
||||||
|
shape and logged through `Effect.catch` instead of a Promise `.catch`.
|
||||||
|
- The remaining client socket Promise `.catch` matches are streaming callback
|
||||||
|
compatibility bridges that route failures to legacy `onError` callbacks.
|
||||||
|
- Verification:
|
||||||
|
- `cd ts && bun run check:tsgo`
|
||||||
|
- `bun run --cwd ts/packages/client build`
|
||||||
|
- `bun run --cwd ts/packages/client test`
|
||||||
|
- `cd ts && bun run check`
|
||||||
|
- `cd ts && bun run build`
|
||||||
|
- `cd ts && bun run test`
|
||||||
|
- `git diff --check`
|
||||||
|
|
||||||
## Subagent Findings To Preserve
|
## Subagent Findings To Preserve
|
||||||
|
|
||||||
- MCP/workbench:
|
- MCP/workbench:
|
||||||
|
|
|
||||||
|
|
@ -504,9 +504,14 @@ export function makeBaseApi(
|
||||||
* Closes the WebSocket connection and cleans up
|
* Closes the WebSocket connection and cleans up
|
||||||
*/
|
*/
|
||||||
close() {
|
close() {
|
||||||
rpc.close().catch((err) => {
|
Effect.runFork(
|
||||||
logClientError("[socket close error]", err);
|
Effect.tryPromise({
|
||||||
});
|
try: () => rpc.close(),
|
||||||
|
catch: (error) => socketError("socket-close", toErrorMessage(error, "Socket close failed")),
|
||||||
|
}).pipe(
|
||||||
|
Effect.catch((error) => Effect.sync(() => logClientError("[socket close error]", error))),
|
||||||
|
),
|
||||||
|
);
|
||||||
},
|
},
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue