@olane/o-node 0.7.12-alpha.6 → 0.7.12-alpha.61

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (74) hide show
  1. package/dist/src/connection/interfaces/o-node-connection-manager.config.d.ts +1 -0
  2. package/dist/src/connection/interfaces/o-node-connection-manager.config.d.ts.map +1 -1
  3. package/dist/src/connection/interfaces/o-node-connection.config.d.ts +1 -0
  4. package/dist/src/connection/interfaces/o-node-connection.config.d.ts.map +1 -1
  5. package/dist/src/connection/o-node-connection.d.ts +7 -2
  6. package/dist/src/connection/o-node-connection.d.ts.map +1 -1
  7. package/dist/src/connection/o-node-connection.js +55 -32
  8. package/dist/src/connection/o-node-connection.manager.d.ts +19 -5
  9. package/dist/src/connection/o-node-connection.manager.d.ts.map +1 -1
  10. package/dist/src/connection/o-node-connection.manager.js +81 -65
  11. package/dist/src/connection/o-stream.request.d.ts +11 -0
  12. package/dist/src/connection/o-stream.request.d.ts.map +1 -0
  13. package/dist/src/connection/o-stream.request.js +7 -0
  14. package/dist/src/connection/stream-handler.config.d.ts +46 -0
  15. package/dist/src/connection/stream-handler.config.d.ts.map +1 -0
  16. package/dist/src/connection/stream-handler.config.js +1 -0
  17. package/dist/src/connection/stream-handler.d.ts +97 -0
  18. package/dist/src/connection/stream-handler.d.ts.map +1 -0
  19. package/dist/src/connection/stream-handler.js +303 -0
  20. package/dist/src/index.d.ts +2 -1
  21. package/dist/src/index.d.ts.map +1 -1
  22. package/dist/src/index.js +2 -1
  23. package/dist/src/interfaces/i-heartbeatable-node.d.ts +49 -0
  24. package/dist/src/interfaces/i-heartbeatable-node.d.ts.map +1 -0
  25. package/dist/src/interfaces/i-heartbeatable-node.js +1 -0
  26. package/dist/src/interfaces/i-reconnectable-node.d.ts +5 -0
  27. package/dist/src/interfaces/i-reconnectable-node.d.ts.map +1 -1
  28. package/dist/src/interfaces/o-node.config.d.ts +16 -8
  29. package/dist/src/interfaces/o-node.config.d.ts.map +1 -1
  30. package/dist/src/managers/o-connection-heartbeat.manager.d.ts +5 -9
  31. package/dist/src/managers/o-connection-heartbeat.manager.d.ts.map +1 -1
  32. package/dist/src/managers/o-connection-heartbeat.manager.js +51 -47
  33. package/dist/src/managers/o-reconnection.manager.d.ts +12 -0
  34. package/dist/src/managers/o-reconnection.manager.d.ts.map +1 -1
  35. package/dist/src/managers/o-reconnection.manager.js +138 -22
  36. package/dist/src/o-node.d.ts +22 -11
  37. package/dist/src/o-node.d.ts.map +1 -1
  38. package/dist/src/o-node.js +202 -72
  39. package/dist/src/o-node.notification-manager.d.ts.map +1 -1
  40. package/dist/src/o-node.notification-manager.js +17 -12
  41. package/dist/src/o-node.tool.d.ts +1 -0
  42. package/dist/src/o-node.tool.d.ts.map +1 -1
  43. package/dist/src/o-node.tool.js +18 -34
  44. package/dist/src/router/o-node.router.d.ts +1 -0
  45. package/dist/src/router/o-node.router.d.ts.map +1 -1
  46. package/dist/src/router/o-node.router.js +62 -9
  47. package/dist/src/router/o-node.routing-policy.d.ts.map +1 -1
  48. package/dist/src/router/o-node.routing-policy.js +7 -2
  49. package/dist/src/router/resolvers/o-node.resolver.d.ts.map +1 -1
  50. package/dist/src/router/resolvers/o-node.resolver.js +5 -1
  51. package/dist/src/router/resolvers/o-node.search-resolver.d.ts.map +1 -1
  52. package/dist/src/router/resolvers/o-node.search-resolver.js +34 -9
  53. package/dist/src/utils/index.d.ts +3 -0
  54. package/dist/src/utils/index.d.ts.map +1 -0
  55. package/dist/src/utils/index.js +2 -0
  56. package/dist/src/utils/stream.utils.d.ts +6 -0
  57. package/dist/src/utils/stream.utils.d.ts.map +1 -0
  58. package/dist/src/utils/stream.utils.js +31 -0
  59. package/dist/test/helpers/test-node.tool.d.ts +15 -0
  60. package/dist/test/helpers/test-node.tool.d.ts.map +1 -0
  61. package/dist/test/helpers/test-node.tool.js +27 -0
  62. package/package.json +6 -6
  63. package/dist/src/router/resolvers/o-node.child-resolver.d.ts +0 -11
  64. package/dist/src/router/resolvers/o-node.child-resolver.d.ts.map +0 -1
  65. package/dist/src/router/resolvers/o-node.child-resolver.js +0 -58
  66. package/dist/src/utils/leader-request-wrapper.d.ts +0 -45
  67. package/dist/src/utils/leader-request-wrapper.d.ts.map +0 -1
  68. package/dist/src/utils/leader-request-wrapper.js +0 -89
  69. package/dist/test/o-node.spec.d.ts +0 -2
  70. package/dist/test/o-node.spec.d.ts.map +0 -1
  71. package/dist/test/o-node.spec.js +0 -20
  72. package/dist/test/search-resolver.spec.d.ts +0 -2
  73. package/dist/test/search-resolver.spec.d.ts.map +0 -1
  74. package/dist/test/search-resolver.spec.js +0 -693
@@ -1,693 +0,0 @@
1
- import { expect } from 'chai';
2
- import { oSearchResolver } from '../src/router/resolvers/o-node.search-resolver.js';
3
- import { oAddress, RestrictedAddresses, NodeState, TransportType, } from '@olane/o-core';
4
- import { oNodeTransport } from '../src/router/o-node.transport.js';
5
- import { oNodeAddress } from '../src/router/o-node.address.js';
6
- import { oProtocolMethods } from '@olane/o-protocol';
7
- describe('oSearchResolver', () => {
8
- let resolver;
9
- let mockNode;
10
- const testAddress = new oNodeAddress('o://test-node');
11
- // Helper to create a proper oRouterRequest
12
- const createRouterRequest = () => ({
13
- method: oProtocolMethods.ROUTE,
14
- params: {
15
- _connectionId: 'test-connection',
16
- _requestMethod: 'test',
17
- address: 'o://test',
18
- payload: {},
19
- },
20
- jsonrpc: '2.0',
21
- id: '1',
22
- state: {},
23
- connectionId: 'test-connection',
24
- toJSON: () => ({}),
25
- setState: () => { },
26
- logger: {},
27
- });
28
- // Helper to create a proper oResponse
29
- const createResponse = (result) => ({
30
- jsonrpc: '2.0',
31
- id: '1',
32
- result,
33
- });
34
- beforeEach(() => {
35
- resolver = new oSearchResolver(testAddress);
36
- // Create mock node with necessary properties
37
- mockNode = {
38
- address: testAddress,
39
- state: NodeState.RUNNING,
40
- leader: new oNodeAddress(RestrictedAddresses.LEADER, [
41
- new oNodeTransport('/ip4/127.0.0.1/tcp/4001'),
42
- ]),
43
- hierarchyManager: {
44
- getChild: (address) => null,
45
- },
46
- use: async (address, request) => {
47
- // Mock registry search response
48
- return createResponse({
49
- data: [],
50
- });
51
- },
52
- };
53
- });
54
- describe('Basic resolver configuration', () => {
55
- it('should have custom transport for /search', () => {
56
- const transports = resolver.customTransports;
57
- expect(transports).to.have.lengthOf(1);
58
- expect(transports[0].value).to.equal('/search');
59
- });
60
- it('should use registry as default search target', () => {
61
- const registryAddress = resolver.getRegistryAddress();
62
- expect(registryAddress.value).to.equal(RestrictedAddresses.REGISTRY);
63
- });
64
- it('should use "search" as default method', () => {
65
- const method = resolver.getSearchMethod();
66
- expect(method).to.equal('search');
67
- });
68
- });
69
- describe('resolve() - Basic flow', () => {
70
- it('should skip search if address already has transports', async () => {
71
- const addressWithTransports = new oNodeAddress('o://test', [
72
- new oNodeTransport('/ip4/127.0.0.1/tcp/5001'),
73
- ]);
74
- const result = await resolver.resolve({
75
- address: addressWithTransports,
76
- targetAddress: addressWithTransports,
77
- node: mockNode,
78
- request: createRouterRequest(),
79
- });
80
- expect(result.nextHopAddress.value).to.equal('o://test');
81
- expect(result.nextHopAddress.transports).to.have.lengthOf(1);
82
- });
83
- it('should return original address when no search results found', async () => {
84
- const address = new oNodeAddress('o://unknown-service');
85
- mockNode.use = async () => createResponse({ data: [] });
86
- const result = await resolver.resolve({
87
- address,
88
- targetAddress: address,
89
- node: mockNode,
90
- request: createRouterRequest(),
91
- });
92
- expect(result.nextHopAddress.value).to.equal('o://unknown-service');
93
- });
94
- it('should resolve address using registry search results', async () => {
95
- const address = new oNodeAddress('o://embeddings-text');
96
- mockNode.use = async () => createResponse({
97
- data: [
98
- {
99
- address: 'o://leader/services/embeddings',
100
- staticAddress: 'o://embeddings-text',
101
- transports: [
102
- {
103
- value: '/ip4/127.0.0.1/tcp/6001',
104
- type: TransportType.LIBP2P,
105
- },
106
- ],
107
- },
108
- ],
109
- });
110
- const result = await resolver.resolve({
111
- address,
112
- targetAddress: address,
113
- node: mockNode,
114
- request: createRouterRequest(),
115
- });
116
- expect(result.targetAddress.value).to.equal('o://leader/services/embeddings');
117
- expect(result.targetAddress.transports).to.have.lengthOf(1);
118
- });
119
- it('should preserve extra path parameters in resolved address', async () => {
120
- const address = new oNodeAddress('o://embeddings-text/method/params');
121
- mockNode.use = async () => createResponse({
122
- data: [
123
- {
124
- address: 'o://leader/services/embeddings',
125
- staticAddress: 'o://embeddings-text',
126
- transports: [
127
- {
128
- value: '/ip4/127.0.0.1/tcp/6001',
129
- type: TransportType.LIBP2P,
130
- },
131
- ],
132
- },
133
- ],
134
- });
135
- const result = await resolver.resolve({
136
- address,
137
- targetAddress: address,
138
- node: mockNode,
139
- request: createRouterRequest(),
140
- });
141
- expect(result.targetAddress.value).to.equal('o://leader/services/embeddings/method/params');
142
- });
143
- it('should handle hierarchical address resolution', async () => {
144
- const address = new oNodeAddress('o://leader/services/embeddings-text');
145
- mockNode.use = async () => createResponse({
146
- data: [
147
- {
148
- address: 'o://leader/services/embeddings-text',
149
- transports: [
150
- {
151
- value: '/ip4/127.0.0.1/tcp/6001',
152
- type: TransportType.LIBP2P,
153
- },
154
- ],
155
- },
156
- ],
157
- });
158
- const result = await resolver.resolve({
159
- address,
160
- targetAddress: address,
161
- node: mockNode,
162
- request: createRouterRequest(),
163
- });
164
- expect(result.targetAddress.transports).to.have.lengthOf(1);
165
- });
166
- });
167
- describe('buildSearchParams()', () => {
168
- it('should include both staticAddress and address', () => {
169
- const address = new oNodeAddress('o://leader/services/embeddings-text');
170
- const params = resolver.buildSearchParams(address);
171
- expect(params.staticAddress).to.equal('o://leader');
172
- expect(params.address).to.equal('o://leader/services/embeddings-text');
173
- });
174
- it('should use root address for staticAddress', () => {
175
- const address = new oNodeAddress('o://leader/deep/nested/path');
176
- const params = resolver.buildSearchParams(address);
177
- expect(params.staticAddress).to.equal('o://leader');
178
- expect(params.address).to.equal('o://leader/deep/nested/path');
179
- });
180
- it('should handle static addresses', () => {
181
- const address = new oNodeAddress('o://embeddings-text');
182
- const params = resolver.buildSearchParams(address);
183
- expect(params.staticAddress).to.equal('o://embeddings-text');
184
- expect(params.address).to.equal('o://embeddings-text');
185
- });
186
- });
187
- describe('filterSearchResults()', () => {
188
- it('should filter out registry address to prevent loops', () => {
189
- const results = [
190
- {
191
- staticAddress: RestrictedAddresses.REGISTRY,
192
- address: 'o://registry',
193
- },
194
- { staticAddress: 'o://service', address: 'o://service' },
195
- ];
196
- const filtered = resolver.filterSearchResults(results, mockNode);
197
- expect(filtered).to.have.lengthOf(1);
198
- expect(filtered[0].staticAddress).to.equal('o://service');
199
- });
200
- it('should filter out self address to prevent loops', () => {
201
- const results = [
202
- { address: 'o://test-node', staticAddress: 'o://test' },
203
- { address: 'o://other-node', staticAddress: 'o://other' },
204
- ];
205
- const filtered = resolver.filterSearchResults(results, mockNode);
206
- expect(filtered).to.have.lengthOf(1);
207
- expect(filtered[0].address).to.equal('o://other-node');
208
- });
209
- it('should allow all valid results', () => {
210
- const results = [
211
- { address: 'o://service-1', staticAddress: 'o://s1' },
212
- { address: 'o://service-2', staticAddress: 'o://s2' },
213
- { address: 'o://service-3', staticAddress: 'o://s3' },
214
- ];
215
- const filtered = resolver.filterSearchResults(results, mockNode);
216
- expect(filtered).to.have.lengthOf(3);
217
- });
218
- it('should handle empty results array', () => {
219
- const filtered = resolver.filterSearchResults([], mockNode);
220
- expect(filtered).to.be.an('array').that.is.empty;
221
- });
222
- });
223
- describe('selectResult()', () => {
224
- it('should select first result from multiple options', () => {
225
- const results = [
226
- { address: 'o://service-1', peerId: 'peer-1' },
227
- { address: 'o://service-2', peerId: 'peer-2' },
228
- { address: 'o://service-3', peerId: 'peer-3' },
229
- ];
230
- const selected = resolver.selectResult(results);
231
- expect(selected).to.equal(results[0]);
232
- expect(selected.peerId).to.equal('peer-1');
233
- });
234
- it('should return null for empty results', () => {
235
- const selected = resolver.selectResult([]);
236
- expect(selected).to.be.null;
237
- });
238
- it('should return single result', () => {
239
- const results = [{ address: 'o://only-service', peerId: 'only-peer' }];
240
- const selected = resolver.selectResult(results);
241
- expect(selected).to.equal(results[0]);
242
- });
243
- });
244
- describe('mapTransports()', () => {
245
- it('should map transport objects to oNodeTransport instances', () => {
246
- const result = {
247
- transports: [
248
- { value: '/ip4/127.0.0.1/tcp/4001', type: TransportType.LIBP2P },
249
- { value: '/ip4/192.168.1.1/tcp/4002', type: TransportType.LIBP2P },
250
- ],
251
- };
252
- const transports = resolver.mapTransports(result);
253
- expect(transports).to.have.lengthOf(2);
254
- expect(transports[0]).to.be.instanceOf(oNodeTransport);
255
- expect(transports[0].value).to.equal('/ip4/127.0.0.1/tcp/4001');
256
- expect(transports[1].value).to.equal('/ip4/192.168.1.1/tcp/4002');
257
- });
258
- it('should handle empty transports array', () => {
259
- const result = { transports: [] };
260
- const transports = resolver.mapTransports(result);
261
- expect(transports).to.be.an('array').that.is.empty;
262
- });
263
- it('should handle single transport', () => {
264
- const result = {
265
- transports: [{ value: '/memory', type: TransportType.CUSTOM }],
266
- };
267
- const transports = resolver.mapTransports(result);
268
- expect(transports).to.have.lengthOf(1);
269
- expect(transports[0].value).to.equal('/memory');
270
- });
271
- });
272
- describe('resolveNextHopTransports()', () => {
273
- it('should use leader transports when next hop is leader', () => {
274
- const nextHop = new oNodeAddress(RestrictedAddresses.LEADER);
275
- const targetTransports = [
276
- new oNodeTransport('/ip4/192.168.1.1/tcp/9999'),
277
- ];
278
- const resolved = resolver.resolveNextHopTransports(nextHop, targetTransports, mockNode);
279
- expect(resolved).to.have.lengthOf(1);
280
- expect(resolved[0].value).to.equal('/ip4/127.0.0.1/tcp/4001'); // Leader transport from mock
281
- });
282
- it('should use child transports when next hop is known child', () => {
283
- const childAddress = new oNodeAddress('o://known-child', [
284
- new oNodeTransport('/ip4/10.0.0.1/tcp/7001'),
285
- ]);
286
- mockNode.hierarchyManager = {
287
- getChild: (address) => {
288
- if (address.value === 'o://known-child') {
289
- return childAddress;
290
- }
291
- return null;
292
- },
293
- };
294
- const nextHop = new oNodeAddress('o://known-child');
295
- const targetTransports = [
296
- new oNodeTransport('/ip4/192.168.1.1/tcp/9999'),
297
- ];
298
- const resolved = resolver.resolveNextHopTransports(nextHop, targetTransports, mockNode);
299
- expect(resolved).to.have.lengthOf(1);
300
- expect(resolved[0].value).to.equal('/ip4/10.0.0.1/tcp/7001');
301
- });
302
- it('should use target transports for unknown next hop', () => {
303
- const nextHop = new oNodeAddress('o://unknown-node');
304
- const targetTransports = [
305
- new oNodeTransport('/ip4/192.168.1.1/tcp/8001'),
306
- ];
307
- const resolved = resolver.resolveNextHopTransports(nextHop, targetTransports, mockNode);
308
- expect(resolved).to.have.lengthOf(1);
309
- expect(resolved[0].value).to.equal('/ip4/192.168.1.1/tcp/8001');
310
- });
311
- it('should return empty array when leader has no transports', () => {
312
- // Create a new mock node with leader that has no transports
313
- const mockNodeNoLeaderTransports = {
314
- ...mockNode,
315
- leader: new oNodeAddress(RestrictedAddresses.LEADER, []),
316
- };
317
- const nextHop = new oNodeAddress(RestrictedAddresses.LEADER);
318
- const targetTransports = [
319
- new oNodeTransport('/ip4/192.168.1.1/tcp/9999'),
320
- ];
321
- const resolved = resolver.resolveNextHopTransports(nextHop, targetTransports, mockNodeNoLeaderTransports);
322
- expect(resolved).to.be.an('array').that.is.empty;
323
- });
324
- });
325
- describe('determineNextHop()', () => {
326
- it('should use oAddress.next() for hierarchy routing', async () => {
327
- const resolvedTarget = new oNodeAddress('o://leader/services/embeddings');
328
- const searchResult = {
329
- transports: [
330
- { value: '/ip4/127.0.0.1/tcp/6001', type: TransportType.LIBP2P },
331
- ],
332
- };
333
- mockNode.address = new oNodeAddress(RestrictedAddresses.LEADER);
334
- const nextHop = resolver.determineNextHop(mockNode, resolvedTarget, searchResult);
335
- // From leader to o://leader/services/embeddings, next hop should be o://leader/services
336
- expect(nextHop.value).to.equal('o://leader/services');
337
- expect(nextHop.transports).to.not.be.empty;
338
- });
339
- it('should set transports on next hop address', async () => {
340
- const resolvedTarget = new oNodeAddress('o://leader/services');
341
- const searchResult = {
342
- transports: [
343
- { value: '/ip4/127.0.0.1/tcp/6001', type: TransportType.LIBP2P },
344
- ],
345
- };
346
- mockNode.address = new oNodeAddress(RestrictedAddresses.LEADER);
347
- const nextHop = resolver.determineNextHop(mockNode, resolvedTarget, searchResult);
348
- expect(nextHop.transports).to.not.be.undefined;
349
- expect(nextHop.transports).to.not.be.empty;
350
- });
351
- it('should handle routing to immediate child', async () => {
352
- const resolvedTarget = new oNodeAddress('o://leader/services');
353
- const searchResult = {
354
- transports: [
355
- { value: '/ip4/127.0.0.1/tcp/6001', type: TransportType.LIBP2P },
356
- ],
357
- };
358
- mockNode.address = new oNodeAddress(RestrictedAddresses.LEADER);
359
- const nextHop = resolver.determineNextHop(mockNode, resolvedTarget, searchResult);
360
- expect(nextHop.value).to.equal('o://leader/services');
361
- });
362
- });
363
- describe('Custom resolver subclassing', () => {
364
- it('should allow overriding getRegistryAddress()', () => {
365
- class CustomRegistryResolver extends oSearchResolver {
366
- getRegistryAddress() {
367
- return new oAddress('o://custom-registry');
368
- }
369
- }
370
- const customResolver = new CustomRegistryResolver(testAddress);
371
- const registryAddress = customResolver.getRegistryAddress();
372
- expect(registryAddress.value).to.equal('o://custom-registry');
373
- });
374
- it('should allow overriding getSearchMethod()', () => {
375
- class CustomMethodResolver extends oSearchResolver {
376
- getSearchMethod() {
377
- return 'custom_search';
378
- }
379
- }
380
- const customResolver = new CustomMethodResolver(testAddress);
381
- const method = customResolver.getSearchMethod();
382
- expect(method).to.equal('custom_search');
383
- });
384
- it('should allow custom result selection (e.g., round-robin)', () => {
385
- class RoundRobinResolver extends oSearchResolver {
386
- constructor() {
387
- super(...arguments);
388
- this.currentIndex = 0;
389
- }
390
- selectResult(results) {
391
- if (results.length === 0)
392
- return null;
393
- const result = results[this.currentIndex % results.length];
394
- this.currentIndex++;
395
- return result;
396
- }
397
- }
398
- const customResolver = new RoundRobinResolver(testAddress);
399
- const results = [
400
- { address: 'o://service-1' },
401
- { address: 'o://service-2' },
402
- { address: 'o://service-3' },
403
- ];
404
- // First call
405
- let selected = customResolver.selectResult(results);
406
- expect(selected.address).to.equal('o://service-1');
407
- // Second call
408
- selected = customResolver.selectResult(results);
409
- expect(selected.address).to.equal('o://service-2');
410
- // Third call
411
- selected = customResolver.selectResult(results);
412
- expect(selected.address).to.equal('o://service-3');
413
- // Fourth call - wraps around
414
- selected = customResolver.selectResult(results);
415
- expect(selected.address).to.equal('o://service-1');
416
- });
417
- it('should allow custom filtering logic', () => {
418
- class StatusFilterResolver extends oSearchResolver {
419
- filterSearchResults(results, node) {
420
- // First apply parent filtering
421
- const filtered = super.filterSearchResults(results, node);
422
- // Then apply custom filtering
423
- return filtered.filter((result) => result.status === 'active');
424
- }
425
- }
426
- const customResolver = new StatusFilterResolver(testAddress);
427
- const results = [
428
- { address: 'o://service-1', status: 'active', staticAddress: 'o://s1' },
429
- {
430
- address: 'o://service-2',
431
- status: 'inactive',
432
- staticAddress: 'o://s2',
433
- },
434
- { address: 'o://service-3', status: 'active', staticAddress: 'o://s3' },
435
- ];
436
- const filtered = customResolver.filterSearchResults(results, mockNode);
437
- expect(filtered).to.have.lengthOf(2);
438
- expect(filtered.every((r) => r.status === 'active')).to.be.true;
439
- });
440
- it('should allow direct routing by overriding determineNextHop()', () => {
441
- class DirectRoutingResolver extends oSearchResolver {
442
- determineNextHop(node, resolvedTargetAddress, searchResult) {
443
- // Always route directly to target, bypassing hierarchy
444
- const targetTransports = this.mapTransports(searchResult);
445
- resolvedTargetAddress.setTransports(targetTransports);
446
- return resolvedTargetAddress;
447
- }
448
- }
449
- const customResolver = new DirectRoutingResolver(testAddress);
450
- const resolvedTarget = new oNodeAddress('o://leader/services/embeddings');
451
- const searchResult = {
452
- transports: [
453
- { value: '/ip4/127.0.0.1/tcp/6001', type: TransportType.LIBP2P },
454
- ],
455
- };
456
- mockNode.address = new oNodeAddress(RestrictedAddresses.LEADER);
457
- const nextHop = customResolver.determineNextHop(mockNode, resolvedTarget, searchResult);
458
- // Direct routing: next hop is the target itself
459
- expect(nextHop.value).to.equal('o://leader/services/embeddings');
460
- expect(nextHop.transports).to.have.lengthOf(1);
461
- });
462
- });
463
- describe('Integration with registry calls', () => {
464
- it('should call node.use() with correct registry address', async () => {
465
- const address = new oNodeAddress('o://test-service');
466
- let capturedAddress;
467
- let capturedRequest;
468
- mockNode.use = async (addr, req) => {
469
- capturedAddress = addr;
470
- capturedRequest = req;
471
- return createResponse({ data: [] });
472
- };
473
- await resolver.resolve({
474
- address,
475
- targetAddress: address,
476
- node: mockNode,
477
- request: createRouterRequest(),
478
- });
479
- expect(capturedAddress?.value).to.equal(RestrictedAddresses.REGISTRY);
480
- expect(capturedRequest.method).to.equal('search');
481
- });
482
- it('should pass correct search parameters to registry', async () => {
483
- const address = new oNodeAddress('o://leader/services/embeddings');
484
- let capturedParams;
485
- mockNode.use = async (addr, req) => {
486
- capturedParams = req.params;
487
- return createResponse({ data: [] });
488
- };
489
- await resolver.resolve({
490
- address,
491
- targetAddress: address,
492
- node: mockNode,
493
- request: createRouterRequest(),
494
- });
495
- expect(capturedParams.staticAddress).to.equal('o://leader');
496
- expect(capturedParams.address).to.equal('o://leader/services/embeddings');
497
- });
498
- it('should handle registry errors gracefully', async () => {
499
- const address = new oNodeAddress('o://test-service');
500
- mockNode.use = async () => {
501
- throw new Error('Registry unavailable');
502
- };
503
- // Should throw error since registry is unavailable
504
- try {
505
- await resolver.resolve({
506
- address,
507
- targetAddress: address,
508
- node: mockNode,
509
- request: createRouterRequest(),
510
- });
511
- expect.fail('Should have thrown error');
512
- }
513
- catch (error) {
514
- expect(error.message).to.equal('Registry unavailable');
515
- }
516
- });
517
- });
518
- describe('Address duplication bug fix', () => {
519
- it('should NOT duplicate path segments when registry returns full hierarchical address', async () => {
520
- // This test verifies the fix for the bug where o://services/embeddings-text
521
- // was being resolved to o://leader/services/embeddings-text/services/embeddings-text
522
- const address = new oNodeAddress('o://services/embeddings-text');
523
- mockNode.use = async () => createResponse({
524
- data: [
525
- {
526
- address: 'o://leader/services/embeddings-text',
527
- staticAddress: 'o://embeddings-text',
528
- transports: [
529
- {
530
- value: '/ip4/127.0.0.1/tcp/6001',
531
- type: TransportType.LIBP2P,
532
- },
533
- ],
534
- },
535
- ],
536
- });
537
- const result = await resolver.resolve({
538
- address,
539
- targetAddress: address,
540
- node: mockNode,
541
- request: createRouterRequest(),
542
- });
543
- // Should be o://leader/services/embeddings-text, NOT o://leader/services/embeddings-text/embeddings-text
544
- expect(result.targetAddress.value).to.equal('o://leader/services/embeddings-text');
545
- });
546
- it('should NOT duplicate when calling via static address', async () => {
547
- const address = new oNodeAddress('o://embeddings-text');
548
- mockNode.use = async () => createResponse({
549
- data: [
550
- {
551
- address: 'o://leader/services/embeddings-text',
552
- staticAddress: 'o://embeddings-text',
553
- transports: [
554
- {
555
- value: '/ip4/127.0.0.1/tcp/6001',
556
- type: TransportType.LIBP2P,
557
- },
558
- ],
559
- },
560
- ],
561
- });
562
- const result = await resolver.resolve({
563
- address,
564
- targetAddress: address,
565
- node: mockNode,
566
- request: createRouterRequest(),
567
- });
568
- expect(result.targetAddress.value).to.equal('o://leader/services/embeddings-text');
569
- });
570
- it('should still append legitimate extra params beyond the service name', async () => {
571
- // If someone calls o://embeddings-text/custom/path, we should preserve /custom/path
572
- const address = new oNodeAddress('o://embeddings-text/custom/path');
573
- mockNode.use = async () => createResponse({
574
- data: [
575
- {
576
- address: 'o://leader/services/embeddings-text',
577
- staticAddress: 'o://embeddings-text',
578
- transports: [
579
- {
580
- value: '/ip4/127.0.0.1/tcp/6001',
581
- type: TransportType.LIBP2P,
582
- },
583
- ],
584
- },
585
- ],
586
- });
587
- const result = await resolver.resolve({
588
- address,
589
- targetAddress: address,
590
- node: mockNode,
591
- request: createRouterRequest(),
592
- });
593
- // Should append the extra /custom/path
594
- expect(result.targetAddress.value).to.equal('o://leader/services/embeddings-text/custom/path');
595
- });
596
- });
597
- describe('Edge cases', () => {
598
- it('should handle address with long nested paths', async () => {
599
- const address = new oNodeAddress('o://leader/a/b/c/d/e/f');
600
- mockNode.use = async () => createResponse({
601
- data: [
602
- {
603
- address: 'o://leader/a/b/c/d/e/f',
604
- transports: [{ value: '/memory', type: TransportType.CUSTOM }],
605
- },
606
- ],
607
- });
608
- const result = await resolver.resolve({
609
- address,
610
- targetAddress: address,
611
- node: mockNode,
612
- request: createRouterRequest(),
613
- });
614
- expect(result.targetAddress.value).to.equal('o://leader/a/b/c/d/e/f');
615
- });
616
- it('should handle static addresses correctly', async () => {
617
- const address = new oNodeAddress('o://embeddings-text');
618
- mockNode.use = async () => createResponse({
619
- data: [
620
- {
621
- address: 'o://leader/services/embeddings',
622
- staticAddress: 'o://embeddings-text',
623
- transports: [
624
- {
625
- value: '/ip4/127.0.0.1/tcp/6001',
626
- type: TransportType.LIBP2P,
627
- },
628
- ],
629
- },
630
- ],
631
- });
632
- const result = await resolver.resolve({
633
- address,
634
- targetAddress: address,
635
- node: mockNode,
636
- request: createRouterRequest(),
637
- });
638
- expect(result.targetAddress.value).to.equal('o://leader/services/embeddings');
639
- });
640
- it('should handle when leader is undefined', async () => {
641
- const address = new oNodeAddress('o://test');
642
- // Create a new mock node with undefined leader
643
- const mockNodeNoLeader = {
644
- ...mockNode,
645
- leader: undefined,
646
- };
647
- mockNodeNoLeader.use = async () => createResponse({
648
- data: [
649
- {
650
- address: 'o://test',
651
- transports: [{ value: '/memory', type: TransportType.CUSTOM }],
652
- },
653
- ],
654
- });
655
- const result = await resolver.resolve({
656
- address,
657
- targetAddress: address,
658
- node: mockNodeNoLeader,
659
- request: createRouterRequest(),
660
- });
661
- // Should still work, just won't have leader transports
662
- expect(result.targetAddress.value).to.equal('o://test');
663
- });
664
- it('should handle multiple transports in search result', async () => {
665
- const address = new oNodeAddress('o://multi-transport');
666
- mockNode.use = async () => createResponse({
667
- data: [
668
- {
669
- address: 'o://multi-transport',
670
- transports: [
671
- {
672
- value: '/ip4/127.0.0.1/tcp/4001',
673
- type: TransportType.LIBP2P,
674
- },
675
- {
676
- value: '/ip4/192.168.1.1/tcp/4002',
677
- type: TransportType.LIBP2P,
678
- },
679
- { value: '/memory', type: TransportType.CUSTOM },
680
- ],
681
- },
682
- ],
683
- });
684
- const result = await resolver.resolve({
685
- address,
686
- targetAddress: address,
687
- node: mockNode,
688
- request: createRouterRequest(),
689
- });
690
- expect(result.targetAddress.transports).to.have.lengthOf(3);
691
- });
692
- });
693
- });