instar 1.3.598 → 1.3.600
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.
- package/dist/commands/server.d.ts.map +1 -1
- package/dist/commands/server.js +79 -3
- package/dist/commands/server.js.map +1 -1
- package/dist/core/MachinePoolRegistry.d.ts +4 -0
- package/dist/core/MachinePoolRegistry.d.ts.map +1 -1
- package/dist/core/MachinePoolRegistry.js +1 -0
- package/dist/core/MachinePoolRegistry.js.map +1 -1
- package/dist/core/PeerPresencePuller.d.ts +4 -1
- package/dist/core/PeerPresencePuller.d.ts.map +1 -1
- package/dist/core/PeerPresencePuller.js +3 -1
- package/dist/core/PeerPresencePuller.js.map +1 -1
- package/dist/core/PlacementExecutor.d.ts +8 -0
- package/dist/core/PlacementExecutor.d.ts.map +1 -1
- package/dist/core/PlacementExecutor.js +29 -0
- package/dist/core/PlacementExecutor.js.map +1 -1
- package/dist/core/RebalancePlanner.d.ts.map +1 -1
- package/dist/core/RebalancePlanner.js +6 -0
- package/dist/core/RebalancePlanner.js.map +1 -1
- package/dist/core/SessionRouter.d.ts +8 -0
- package/dist/core/SessionRouter.d.ts.map +1 -1
- package/dist/core/SessionRouter.js +4 -0
- package/dist/core/SessionRouter.js.map +1 -1
- package/dist/core/devGatedFeatures.d.ts.map +1 -1
- package/dist/core/devGatedFeatures.js +6 -0
- package/dist/core/devGatedFeatures.js.map +1 -1
- package/dist/core/machineServesChannel.d.ts +42 -0
- package/dist/core/machineServesChannel.d.ts.map +1 -0
- package/dist/core/machineServesChannel.js +48 -0
- package/dist/core/machineServesChannel.js.map +1 -0
- package/dist/core/multiMachineCapstoneMatrix.d.ts +49 -0
- package/dist/core/multiMachineCapstoneMatrix.d.ts.map +1 -0
- package/dist/core/multiMachineCapstoneMatrix.js +131 -0
- package/dist/core/multiMachineCapstoneMatrix.js.map +1 -0
- package/dist/core/types.d.ts +7 -0
- package/dist/core/types.d.ts.map +1 -1
- package/dist/core/types.js.map +1 -1
- package/dist/messaging/slack/SlackAdapter.d.ts +6 -0
- package/dist/messaging/slack/SlackAdapter.d.ts.map +1 -1
- package/dist/messaging/slack/SlackAdapter.js +12 -0
- package/dist/messaging/slack/SlackAdapter.js.map +1 -1
- package/dist/server/AgentServer.d.ts +35 -0
- package/dist/server/AgentServer.d.ts.map +1 -1
- package/dist/server/AgentServer.js +1 -0
- package/dist/server/AgentServer.js.map +1 -1
- package/dist/server/CapabilityIndex.d.ts.map +1 -1
- package/dist/server/CapabilityIndex.js +1 -0
- package/dist/server/CapabilityIndex.js.map +1 -1
- package/dist/server/routes.d.ts +3 -0
- package/dist/server/routes.d.ts.map +1 -1
- package/dist/server/routes.js +195 -0
- package/dist/server/routes.js.map +1 -1
- package/package.json +1 -1
- package/src/data/builtin-manifest.json +47 -47
- package/upgrades/1.3.599.md +21 -0
- package/upgrades/1.3.600.md +44 -0
- package/upgrades/side-effects/live-test-capstone-runner-route.md +89 -0
- package/upgrades/side-effects/placement-platform-workspace-aware.md +43 -0
package/package.json
CHANGED
|
@@ -1,8 +1,8 @@
|
|
|
1
1
|
{
|
|
2
2
|
"$schema": "./builtin-manifest.schema.json",
|
|
3
3
|
"schemaVersion": 1,
|
|
4
|
-
"generatedAt": "2026-06-
|
|
5
|
-
"instarVersion": "1.3.
|
|
4
|
+
"generatedAt": "2026-06-16T13:58:56.562Z",
|
|
5
|
+
"instarVersion": "1.3.600",
|
|
6
6
|
"entryCount": 201,
|
|
7
7
|
"entries": {
|
|
8
8
|
"hook:session-start": {
|
|
@@ -418,7 +418,7 @@
|
|
|
418
418
|
"type": "route-group",
|
|
419
419
|
"domain": "monitoring",
|
|
420
420
|
"sourcePath": "src/server/routes.ts",
|
|
421
|
-
"contentHash": "
|
|
421
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
422
422
|
"since": "2025-01-01"
|
|
423
423
|
},
|
|
424
424
|
"route-group:agents": {
|
|
@@ -426,7 +426,7 @@
|
|
|
426
426
|
"type": "route-group",
|
|
427
427
|
"domain": "sessions",
|
|
428
428
|
"sourcePath": "src/server/routes.ts",
|
|
429
|
-
"contentHash": "
|
|
429
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
430
430
|
"since": "2025-01-01"
|
|
431
431
|
},
|
|
432
432
|
"route-group:backups": {
|
|
@@ -434,7 +434,7 @@
|
|
|
434
434
|
"type": "route-group",
|
|
435
435
|
"domain": "operations",
|
|
436
436
|
"sourcePath": "src/server/routes.ts",
|
|
437
|
-
"contentHash": "
|
|
437
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
438
438
|
"since": "2025-01-01"
|
|
439
439
|
},
|
|
440
440
|
"route-group:git": {
|
|
@@ -442,7 +442,7 @@
|
|
|
442
442
|
"type": "route-group",
|
|
443
443
|
"domain": "coordination",
|
|
444
444
|
"sourcePath": "src/server/routes.ts",
|
|
445
|
-
"contentHash": "
|
|
445
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
446
446
|
"since": "2025-01-01"
|
|
447
447
|
},
|
|
448
448
|
"route-group:memory": {
|
|
@@ -450,7 +450,7 @@
|
|
|
450
450
|
"type": "route-group",
|
|
451
451
|
"domain": "memory",
|
|
452
452
|
"sourcePath": "src/server/routes.ts",
|
|
453
|
-
"contentHash": "
|
|
453
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
454
454
|
"since": "2025-01-01"
|
|
455
455
|
},
|
|
456
456
|
"route-group:semantic": {
|
|
@@ -458,7 +458,7 @@
|
|
|
458
458
|
"type": "route-group",
|
|
459
459
|
"domain": "memory",
|
|
460
460
|
"sourcePath": "src/server/routes.ts",
|
|
461
|
-
"contentHash": "
|
|
461
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
462
462
|
"since": "2025-01-01"
|
|
463
463
|
},
|
|
464
464
|
"route-group:status": {
|
|
@@ -466,7 +466,7 @@
|
|
|
466
466
|
"type": "route-group",
|
|
467
467
|
"domain": "monitoring",
|
|
468
468
|
"sourcePath": "src/server/routes.ts",
|
|
469
|
-
"contentHash": "
|
|
469
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
470
470
|
"since": "2025-01-01"
|
|
471
471
|
},
|
|
472
472
|
"route-group:capabilities": {
|
|
@@ -474,7 +474,7 @@
|
|
|
474
474
|
"type": "route-group",
|
|
475
475
|
"domain": "mapping",
|
|
476
476
|
"sourcePath": "src/server/routes.ts",
|
|
477
|
-
"contentHash": "
|
|
477
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
478
478
|
"since": "2025-01-01"
|
|
479
479
|
},
|
|
480
480
|
"route-group:project-map": {
|
|
@@ -482,7 +482,7 @@
|
|
|
482
482
|
"type": "route-group",
|
|
483
483
|
"domain": "mapping",
|
|
484
484
|
"sourcePath": "src/server/routes.ts",
|
|
485
|
-
"contentHash": "
|
|
485
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
486
486
|
"since": "2025-01-01"
|
|
487
487
|
},
|
|
488
488
|
"route-group:coherence": {
|
|
@@ -490,7 +490,7 @@
|
|
|
490
490
|
"type": "route-group",
|
|
491
491
|
"domain": "coherence",
|
|
492
492
|
"sourcePath": "src/server/routes.ts",
|
|
493
|
-
"contentHash": "
|
|
493
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
494
494
|
"since": "2025-01-01"
|
|
495
495
|
},
|
|
496
496
|
"route-group:topic-bindings": {
|
|
@@ -498,7 +498,7 @@
|
|
|
498
498
|
"type": "route-group",
|
|
499
499
|
"domain": "sessions",
|
|
500
500
|
"sourcePath": "src/server/routes.ts",
|
|
501
|
-
"contentHash": "
|
|
501
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
502
502
|
"since": "2025-01-01"
|
|
503
503
|
},
|
|
504
504
|
"route-group:context": {
|
|
@@ -506,7 +506,7 @@
|
|
|
506
506
|
"type": "route-group",
|
|
507
507
|
"domain": "context",
|
|
508
508
|
"sourcePath": "src/server/routes.ts",
|
|
509
|
-
"contentHash": "
|
|
509
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
510
510
|
"since": "2025-01-01"
|
|
511
511
|
},
|
|
512
512
|
"route-group:scope-coherence": {
|
|
@@ -514,7 +514,7 @@
|
|
|
514
514
|
"type": "route-group",
|
|
515
515
|
"domain": "coherence",
|
|
516
516
|
"sourcePath": "src/server/routes.ts",
|
|
517
|
-
"contentHash": "
|
|
517
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
518
518
|
"since": "2025-01-01"
|
|
519
519
|
},
|
|
520
520
|
"route-group:canonical-state": {
|
|
@@ -522,7 +522,7 @@
|
|
|
522
522
|
"type": "route-group",
|
|
523
523
|
"domain": "state",
|
|
524
524
|
"sourcePath": "src/server/routes.ts",
|
|
525
|
-
"contentHash": "
|
|
525
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
526
526
|
"since": "2025-01-01"
|
|
527
527
|
},
|
|
528
528
|
"route-group:ci": {
|
|
@@ -530,7 +530,7 @@
|
|
|
530
530
|
"type": "route-group",
|
|
531
531
|
"domain": "monitoring",
|
|
532
532
|
"sourcePath": "src/server/routes.ts",
|
|
533
|
-
"contentHash": "
|
|
533
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
534
534
|
"since": "2025-01-01"
|
|
535
535
|
},
|
|
536
536
|
"route-group:sessions": {
|
|
@@ -538,7 +538,7 @@
|
|
|
538
538
|
"type": "route-group",
|
|
539
539
|
"domain": "sessions",
|
|
540
540
|
"sourcePath": "src/server/routes.ts",
|
|
541
|
-
"contentHash": "
|
|
541
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
542
542
|
"since": "2025-01-01"
|
|
543
543
|
},
|
|
544
544
|
"route-group:jobs": {
|
|
@@ -546,7 +546,7 @@
|
|
|
546
546
|
"type": "route-group",
|
|
547
547
|
"domain": "scheduling",
|
|
548
548
|
"sourcePath": "src/server/routes.ts",
|
|
549
|
-
"contentHash": "
|
|
549
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
550
550
|
"since": "2025-01-01"
|
|
551
551
|
},
|
|
552
552
|
"route-group:skip-ledger": {
|
|
@@ -554,7 +554,7 @@
|
|
|
554
554
|
"type": "route-group",
|
|
555
555
|
"domain": "scheduling",
|
|
556
556
|
"sourcePath": "src/server/routes.ts",
|
|
557
|
-
"contentHash": "
|
|
557
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
558
558
|
"since": "2025-01-01"
|
|
559
559
|
},
|
|
560
560
|
"route-group:telegram": {
|
|
@@ -562,7 +562,7 @@
|
|
|
562
562
|
"type": "route-group",
|
|
563
563
|
"domain": "communication",
|
|
564
564
|
"sourcePath": "src/server/routes.ts",
|
|
565
|
-
"contentHash": "
|
|
565
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
566
566
|
"since": "2025-01-01"
|
|
567
567
|
},
|
|
568
568
|
"route-group:attention": {
|
|
@@ -570,7 +570,7 @@
|
|
|
570
570
|
"type": "route-group",
|
|
571
571
|
"domain": "communication",
|
|
572
572
|
"sourcePath": "src/server/routes.ts",
|
|
573
|
-
"contentHash": "
|
|
573
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
574
574
|
"since": "2025-01-01"
|
|
575
575
|
},
|
|
576
576
|
"route-group:relationships": {
|
|
@@ -578,7 +578,7 @@
|
|
|
578
578
|
"type": "route-group",
|
|
579
579
|
"domain": "relationships",
|
|
580
580
|
"sourcePath": "src/server/routes.ts",
|
|
581
|
-
"contentHash": "
|
|
581
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
582
582
|
"since": "2025-01-01"
|
|
583
583
|
},
|
|
584
584
|
"route-group:feedback": {
|
|
@@ -586,7 +586,7 @@
|
|
|
586
586
|
"type": "route-group",
|
|
587
587
|
"domain": "feedback",
|
|
588
588
|
"sourcePath": "src/server/routes.ts",
|
|
589
|
-
"contentHash": "
|
|
589
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
590
590
|
"since": "2025-01-01"
|
|
591
591
|
},
|
|
592
592
|
"route-group:updates": {
|
|
@@ -594,7 +594,7 @@
|
|
|
594
594
|
"type": "route-group",
|
|
595
595
|
"domain": "updates",
|
|
596
596
|
"sourcePath": "src/server/routes.ts",
|
|
597
|
-
"contentHash": "
|
|
597
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
598
598
|
"since": "2025-01-01"
|
|
599
599
|
},
|
|
600
600
|
"route-group:dispatches": {
|
|
@@ -602,7 +602,7 @@
|
|
|
602
602
|
"type": "route-group",
|
|
603
603
|
"domain": "dispatches",
|
|
604
604
|
"sourcePath": "src/server/routes.ts",
|
|
605
|
-
"contentHash": "
|
|
605
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
606
606
|
"since": "2025-01-01"
|
|
607
607
|
},
|
|
608
608
|
"route-group:quota": {
|
|
@@ -610,7 +610,7 @@
|
|
|
610
610
|
"type": "route-group",
|
|
611
611
|
"domain": "monitoring",
|
|
612
612
|
"sourcePath": "src/server/routes.ts",
|
|
613
|
-
"contentHash": "
|
|
613
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
614
614
|
"since": "2025-01-01"
|
|
615
615
|
},
|
|
616
616
|
"route-group:publishing": {
|
|
@@ -618,7 +618,7 @@
|
|
|
618
618
|
"type": "route-group",
|
|
619
619
|
"domain": "publishing",
|
|
620
620
|
"sourcePath": "src/server/routes.ts",
|
|
621
|
-
"contentHash": "
|
|
621
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
622
622
|
"since": "2025-01-01"
|
|
623
623
|
},
|
|
624
624
|
"route-group:private-views": {
|
|
@@ -626,7 +626,7 @@
|
|
|
626
626
|
"type": "route-group",
|
|
627
627
|
"domain": "publishing",
|
|
628
628
|
"sourcePath": "src/server/routes.ts",
|
|
629
|
-
"contentHash": "
|
|
629
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
630
630
|
"since": "2025-01-01"
|
|
631
631
|
},
|
|
632
632
|
"route-group:tunnel": {
|
|
@@ -634,7 +634,7 @@
|
|
|
634
634
|
"type": "route-group",
|
|
635
635
|
"domain": "networking",
|
|
636
636
|
"sourcePath": "src/server/routes.ts",
|
|
637
|
-
"contentHash": "
|
|
637
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
638
638
|
"since": "2025-01-01"
|
|
639
639
|
},
|
|
640
640
|
"route-group:events": {
|
|
@@ -642,7 +642,7 @@
|
|
|
642
642
|
"type": "route-group",
|
|
643
643
|
"domain": "networking",
|
|
644
644
|
"sourcePath": "src/server/routes.ts",
|
|
645
|
-
"contentHash": "
|
|
645
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
646
646
|
"since": "2025-01-01"
|
|
647
647
|
},
|
|
648
648
|
"route-group:evolution": {
|
|
@@ -650,7 +650,7 @@
|
|
|
650
650
|
"type": "route-group",
|
|
651
651
|
"domain": "evolution",
|
|
652
652
|
"sourcePath": "src/server/routes.ts",
|
|
653
|
-
"contentHash": "
|
|
653
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
654
654
|
"since": "2025-01-01"
|
|
655
655
|
},
|
|
656
656
|
"route-group:watchdog": {
|
|
@@ -658,7 +658,7 @@
|
|
|
658
658
|
"type": "route-group",
|
|
659
659
|
"domain": "monitoring",
|
|
660
660
|
"sourcePath": "src/server/routes.ts",
|
|
661
|
-
"contentHash": "
|
|
661
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
662
662
|
"since": "2025-01-01"
|
|
663
663
|
},
|
|
664
664
|
"route-group:topic-memory": {
|
|
@@ -666,7 +666,7 @@
|
|
|
666
666
|
"type": "route-group",
|
|
667
667
|
"domain": "memory",
|
|
668
668
|
"sourcePath": "src/server/routes.ts",
|
|
669
|
-
"contentHash": "
|
|
669
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
670
670
|
"since": "2025-01-01"
|
|
671
671
|
},
|
|
672
672
|
"route-group:state-sync": {
|
|
@@ -674,7 +674,7 @@
|
|
|
674
674
|
"type": "route-group",
|
|
675
675
|
"domain": "coordination",
|
|
676
676
|
"sourcePath": "src/server/routes.ts",
|
|
677
|
-
"contentHash": "
|
|
677
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
678
678
|
"since": "2025-01-01"
|
|
679
679
|
},
|
|
680
680
|
"route-group:intent": {
|
|
@@ -682,7 +682,7 @@
|
|
|
682
682
|
"type": "route-group",
|
|
683
683
|
"domain": "intent",
|
|
684
684
|
"sourcePath": "src/server/routes.ts",
|
|
685
|
-
"contentHash": "
|
|
685
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
686
686
|
"since": "2025-01-01"
|
|
687
687
|
},
|
|
688
688
|
"route-group:triage": {
|
|
@@ -690,7 +690,7 @@
|
|
|
690
690
|
"type": "route-group",
|
|
691
691
|
"domain": "safety",
|
|
692
692
|
"sourcePath": "src/server/routes.ts",
|
|
693
|
-
"contentHash": "
|
|
693
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
694
694
|
"since": "2025-01-01"
|
|
695
695
|
},
|
|
696
696
|
"route-group:operations": {
|
|
@@ -698,7 +698,7 @@
|
|
|
698
698
|
"type": "route-group",
|
|
699
699
|
"domain": "safety",
|
|
700
700
|
"sourcePath": "src/server/routes.ts",
|
|
701
|
-
"contentHash": "
|
|
701
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
702
702
|
"since": "2025-01-01"
|
|
703
703
|
},
|
|
704
704
|
"route-group:sentinel": {
|
|
@@ -706,7 +706,7 @@
|
|
|
706
706
|
"type": "route-group",
|
|
707
707
|
"domain": "safety",
|
|
708
708
|
"sourcePath": "src/server/routes.ts",
|
|
709
|
-
"contentHash": "
|
|
709
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
710
710
|
"since": "2025-01-01"
|
|
711
711
|
},
|
|
712
712
|
"route-group:trust": {
|
|
@@ -714,7 +714,7 @@
|
|
|
714
714
|
"type": "route-group",
|
|
715
715
|
"domain": "safety",
|
|
716
716
|
"sourcePath": "src/server/routes.ts",
|
|
717
|
-
"contentHash": "
|
|
717
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
718
718
|
"since": "2025-01-01"
|
|
719
719
|
},
|
|
720
720
|
"route-group:monitoring": {
|
|
@@ -722,7 +722,7 @@
|
|
|
722
722
|
"type": "route-group",
|
|
723
723
|
"domain": "monitoring",
|
|
724
724
|
"sourcePath": "src/server/routes.ts",
|
|
725
|
-
"contentHash": "
|
|
725
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
726
726
|
"since": "2025-01-01"
|
|
727
727
|
},
|
|
728
728
|
"route-group:commitments": {
|
|
@@ -730,7 +730,7 @@
|
|
|
730
730
|
"type": "route-group",
|
|
731
731
|
"domain": "commitments",
|
|
732
732
|
"sourcePath": "src/server/routes.ts",
|
|
733
|
-
"contentHash": "
|
|
733
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
734
734
|
"since": "2025-01-01"
|
|
735
735
|
},
|
|
736
736
|
"route-group:episodes": {
|
|
@@ -738,7 +738,7 @@
|
|
|
738
738
|
"type": "route-group",
|
|
739
739
|
"domain": "memory",
|
|
740
740
|
"sourcePath": "src/server/routes.ts",
|
|
741
|
-
"contentHash": "
|
|
741
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
742
742
|
"since": "2025-01-01"
|
|
743
743
|
},
|
|
744
744
|
"route-group:messages": {
|
|
@@ -746,7 +746,7 @@
|
|
|
746
746
|
"type": "route-group",
|
|
747
747
|
"domain": "coordination",
|
|
748
748
|
"sourcePath": "src/server/routes.ts",
|
|
749
|
-
"contentHash": "
|
|
749
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
750
750
|
"since": "2025-01-01"
|
|
751
751
|
},
|
|
752
752
|
"route-group:system-reviews": {
|
|
@@ -754,7 +754,7 @@
|
|
|
754
754
|
"type": "route-group",
|
|
755
755
|
"domain": "monitoring",
|
|
756
756
|
"sourcePath": "src/server/routes.ts",
|
|
757
|
-
"contentHash": "
|
|
757
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
758
758
|
"since": "2025-01-01"
|
|
759
759
|
},
|
|
760
760
|
"route-group:machine-mesh": {
|
|
@@ -770,7 +770,7 @@
|
|
|
770
770
|
"type": "route-group",
|
|
771
771
|
"domain": "security",
|
|
772
772
|
"sourcePath": "src/server/routes.ts",
|
|
773
|
-
"contentHash": "
|
|
773
|
+
"contentHash": "5ece1de6ab8dc830be47b810a9784294c93a9f8cfa65ecf46fb61df83716b171",
|
|
774
774
|
"since": "2025-01-01"
|
|
775
775
|
},
|
|
776
776
|
"cli:init": {
|
|
@@ -1522,7 +1522,7 @@
|
|
|
1522
1522
|
"type": "subsystem",
|
|
1523
1523
|
"domain": "server",
|
|
1524
1524
|
"sourcePath": "src/server/AgentServer.ts",
|
|
1525
|
-
"contentHash": "
|
|
1525
|
+
"contentHash": "db68c7f638c8d48f69e69ff9432bdcc87e79d38136ef607d168eefe17c037142",
|
|
1526
1526
|
"since": "2025-01-01"
|
|
1527
1527
|
},
|
|
1528
1528
|
"subsystem:session-manager": {
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
# Upgrade Guide — vNEXT
|
|
2
|
+
|
|
3
|
+
<!-- assembled-by: assemble-next-md -->
|
|
4
|
+
<!-- bump: patch -->
|
|
5
|
+
|
|
6
|
+
## What Changed
|
|
7
|
+
|
|
8
|
+
Closes the bug the gold-standard live-test caught: multi-machine placement assigned a Slack channel to a machine connected to a DIFFERENT Slack workspace, so the channel "owned" by that machine black-holed (the user got a "working…" standby, never a reply). Placement now knows which channels each machine's adapters can actually reach (`MachineCapacity.servesChannels`, adapter-derived on the heartbeat) and refuses to own a Slack channel on a machine not connected to its workspace — three-valued (`yes`/`no`/`unknown`), `yes` ranked over `unknown` (fail-open on a missing/old signal), all-`no` → `queued` + an honest "no machine serves this" attention surface (never a black-hole pick). Mirrors CMT-1570 (which made placement quota-aware). Ships behind the existing multi-machine path; single-machine = no-op; inert until the Slack inbound threads the channel scope (telegram is shared, so it's deliberately not threaded).
|
|
9
|
+
|
|
10
|
+
## What to Tell Your User
|
|
11
|
+
|
|
12
|
+
Nothing user-facing yet — it ships behind the multi-machine path and is the fix for the cross-machine bug the live-test caught (a Slack channel served on the wrong machine). The payoff lands when the multi-machine flags are flipped on.
|
|
13
|
+
|
|
14
|
+
## Summary of New Capabilities
|
|
15
|
+
|
|
16
|
+
Internal: placement avoids a machine that can't reach a channel's platform/workspace (mirrors the quota-awareness fix). No new routes or flags; rides the existing multi-machine heartbeat.
|
|
17
|
+
|
|
18
|
+
## Evidence
|
|
19
|
+
|
|
20
|
+
- `tests/unit/machineServesChannel.test.ts` (8), `tests/unit/PlacementExecutor.test.ts` (+8 serve-filter), `tests/unit/SessionRouter.test.ts` (+3 incl. the behavioral proof that a Slack channel places on the machine serving its workspace, not the cheaper non-serving one — the bug fixed — and the failover wiring-integrity test). 71 relevant tests green; tsc clean.
|
|
21
|
+
- Spec converged 6 iterations (5 cross-model codex/gpt-5.5 + lessons-aware + decision-completeness), ~33 findings; second-pass implementation review concurs.
|
|
@@ -0,0 +1,44 @@
|
|
|
1
|
+
# Upgrade Guide — vNEXT
|
|
2
|
+
|
|
3
|
+
<!-- assembled-by: assemble-next-md -->
|
|
4
|
+
<!-- bump: patch -->
|
|
5
|
+
|
|
6
|
+
## What Changed
|
|
7
|
+
|
|
8
|
+
The (already-built, dark) multi-machine transfer CAPSTONE harness is now RUNNABLE behind
|
|
9
|
+
a dev-gated route. `POST /live-test/multi-machine-capstone` moves the seat FIRST (local
|
|
10
|
+
`/pool/transfer`), demands the honest `seatMoved` signal (records a `200 capstone:'aborted'`
|
|
11
|
+
on a non-move — never a misleading PASS), then drives the §7.5 risk-category matrix through
|
|
12
|
+
the REAL user surfaces and records a signed PASS/FAIL artifact (PASS only where the reply
|
|
13
|
+
came FROM the target machine). `GET /live-test/artifacts` lists the recorded artifacts.
|
|
14
|
+
A surface whose demo credential is absent is reported in `blockedSurfaces` and fails loudly
|
|
15
|
+
via the driver — never a fabricated reply. Ships DARK (`monitoring.liveTestRunner.enabled`
|
|
16
|
+
omitted → `resolveDevAgentGate`: live on dev, 503 on the fleet). Driven by the converged +
|
|
17
|
+
approved `live-user-channel-proof-standard` (§6 / §7.5).
|
|
18
|
+
|
|
19
|
+
## What to Tell Your User
|
|
20
|
+
|
|
21
|
+
Nothing user-facing yet — this is an experimental, developer-only self-test that ships
|
|
22
|
+
turned off on the fleet. It exists so the agent can prove a multi-machine conversation
|
|
23
|
+
move works end-to-end through the real channels (Telegram and Slack) before you ever have
|
|
24
|
+
to test it — the first application of the Live-User-Channel Proof standard. The payoff is
|
|
25
|
+
invisible reliability: features get caught failing in a live self-test instead of by you
|
|
26
|
+
on first use.
|
|
27
|
+
|
|
28
|
+
## Summary of New Capabilities
|
|
29
|
+
|
|
30
|
+
Internal / developer-only: a gated endpoint that runs the gold-standard live-test harness
|
|
31
|
+
over the cross-machine conversation move (it moves the seat first, asserts the reply comes
|
|
32
|
+
from the target machine, and records a signed pass/fail report), plus a read-only endpoint
|
|
33
|
+
to list those reports. No fleet-facing surface; off by default.
|
|
34
|
+
|
|
35
|
+
## Evidence
|
|
36
|
+
|
|
37
|
+
- `tests/unit/multiMachineCapstoneMatrix.test.ts` (8) — the §7.5 matrix builder.
|
|
38
|
+
- `tests/unit/LiveTestRunner-capstone.test.ts` (3) — seat-move-first + reply-from-target.
|
|
39
|
+
- `tests/integration/live-test-capstone-route.test.ts` (7) — route 200/400/aborted/auth +
|
|
40
|
+
seatMoved:true→artifact / seatMoved:false→no artifact.
|
|
41
|
+
- `tests/e2e/live-test-capstone-alive.test.ts` (4) — feature-alive on the REAL AgentServer
|
|
42
|
+
init path: wired→200, dark→503, Bearer-required, shared-store wiring-integrity.
|
|
43
|
+
- `tsc --noEmit` clean. Ratchets green: no-silent-fallbacks, CapabilityIndex,
|
|
44
|
+
feature-delivery-completeness, docs-coverage (all class floors met).
|
|
@@ -0,0 +1,89 @@
|
|
|
1
|
+
# Side-Effects Review — Live-Test Capstone Runner Route
|
|
2
|
+
|
|
3
|
+
**Change:** Make the (already-built, dark) multi-machine transfer CAPSTONE harness RUNNABLE.
|
|
4
|
+
Adds `POST /live-test/multi-machine-capstone` + `GET /live-test/artifacts`, the
|
|
5
|
+
`liveTestRunnerCtx` wiring (a `LiveTestRunnerWiring` factory on `AgentServer`), the
|
|
6
|
+
`multiMachineCapstoneMatrix` builder, and registers `liveTestRunner` in the dev-gated
|
|
7
|
+
feature registry. DEV-GATED + DARK (`monitoring.liveTestRunner.enabled` omitted from
|
|
8
|
+
ConfigDefaults → `resolveDevAgentGate`: live on a dev agent, dark on the fleet → routes
|
|
9
|
+
503). Driven by the converged + approved `docs/specs/live-user-channel-proof-standard.md`
|
|
10
|
+
(§6 / §7.5).
|
|
11
|
+
|
|
12
|
+
**Files:** `src/commands/server.ts` (ctx factory), `src/server/AgentServer.ts`
|
|
13
|
+
(`LiveTestRunnerWiring` type + ctor wiring), `src/server/routes.ts` (the two routes),
|
|
14
|
+
`src/core/multiMachineCapstoneMatrix.ts` (the §7.5 matrix builder),
|
|
15
|
+
`src/core/devGatedFeatures.ts` (registration). Tests: unit (matrix + runner-capstone),
|
|
16
|
+
integration (route 200/503/400/aborted/auth), e2e (feature-alive on the real AgentServer
|
|
17
|
+
init path).
|
|
18
|
+
|
|
19
|
+
## 1. Over-block — legitimate inputs wrongly rejected?
|
|
20
|
+
The route rejects only missing `targetMachine`/`telegramTopicId` (400). A surface whose
|
|
21
|
+
demo credential is absent is **not** rejected — it is recorded in `blockedSurfaces` and
|
|
22
|
+
its scenarios surface as a loud driver-error FAIL via `RealChannelDriver`'s "no real
|
|
23
|
+
sender configured" throw. No legitimate run is silently dropped: a non-moving seat is a
|
|
24
|
+
recorded `200 capstone:'aborted'` (the honesty contract), never a 400/500.
|
|
25
|
+
|
|
26
|
+
## 2. Under-block — failure modes still missed?
|
|
27
|
+
- A demo SENDER that is mis-configured (wrong token) fails at send time → the scenario
|
|
28
|
+
records a driver-error FAIL (honest), not a fabricated PASS. Acceptable.
|
|
29
|
+
- The Slack production path uses `SlackApiClient(demoSlackUserToken)` (a Bearer xoxp
|
|
30
|
+
token). Reality: the available SageMind-Live-Test creds are an `xoxc` web-client token
|
|
31
|
+
+ `d` cookie, not xoxp — so with today's creds the Slack surface reports
|
|
32
|
+
`credential-unavailable` (blocked, HONEST) rather than running. The Telegram surface is
|
|
33
|
+
the capstone's reply-from-target channel. **Tracked follow-up (CMT-1568):** an
|
|
34
|
+
xoxc+cookie Slack drive variant to actually run the Slack channel-parity scenario.
|
|
35
|
+
`<!-- tracked: CMT-1568 -->`
|
|
36
|
+
- The responder-machine resolver keys every surface on `telegramTopicId` (there is no
|
|
37
|
+
live Slack-channel→topic resolver) — documented inline as a GUESS; only affects the
|
|
38
|
+
optional Slack scenario. `<!-- tracked: CMT-1568 -->`
|
|
39
|
+
|
|
40
|
+
## 3. Level-of-abstraction fit
|
|
41
|
+
Correct layer: the route composes existing, separately-tested primitives
|
|
42
|
+
(`RealChannelDriver`, `TelegramLiveSender`/`SlackLiveSender`, `LiveTestHarness`,
|
|
43
|
+
`LiveTestRunner`, `PlacementResponderReader`, `DemoChannelRegistry`) and the shared
|
|
44
|
+
`LiveTestArtifactStore` (the SAME store `LiveTestGate` reads, so a run's artifact is
|
|
45
|
+
exactly the artifact the gate later verifies). No business logic is re-implemented in the
|
|
46
|
+
route — it is composition + request validation + the seat-move-first orchestration the
|
|
47
|
+
standard mandates.
|
|
48
|
+
|
|
49
|
+
## 4. Signal vs authority compliance
|
|
50
|
+
The route holds **no blocking authority over agent behavior**. It runs a test harness and
|
|
51
|
+
records a signed artifact. The only authority-adjacent component (`LiveTestGate`, which
|
|
52
|
+
computes a completion veto) is unchanged by this PR and remains dry-run. The route is a
|
|
53
|
+
pure producer of an observable artifact — Signal, not Authority. (`docs/signal-vs-authority.md`.)
|
|
54
|
+
|
|
55
|
+
## 5. Interactions
|
|
56
|
+
- Shares the `LiveTestArtifactStore` instance with `LiveTestGate` by construction (server.ts
|
|
57
|
+
threads the SAME store into both) — verified by the e2e wiring-integrity test (d).
|
|
58
|
+
- The seat-move calls the LOCAL `/pool/transfer` (production) — same route the operator
|
|
59
|
+
uses; it is read through the honest `seatMoved` signal (the #1188 fix), never assumed.
|
|
60
|
+
- `transferForRequest` / `driverForRequest` are TEST-ONLY injection seams (undefined in
|
|
61
|
+
production); they let the e2e exercise the real AgentServer route with no live pool/network.
|
|
62
|
+
- No double-fire / no shadowing: the routes are new prefixes; CapabilityIndex +
|
|
63
|
+
feature-delivery-completeness ratchets pass.
|
|
64
|
+
|
|
65
|
+
## 6. External surfaces
|
|
66
|
+
When RUN on a dev agent it posts a real demo message into a real (throwaway) channel and
|
|
67
|
+
moves a real seat — but it is DEV-GATED + DARK, requires explicit demo creds, and every
|
|
68
|
+
scenario is `safe`-volatility. On the fleet the routes 503 (no external effect). The run
|
|
69
|
+
is operator-gated in practice (demo-sender provisioning + a live destination machine).
|
|
70
|
+
|
|
71
|
+
## 7. Multi-machine posture (Cross-Machine Coherence)
|
|
72
|
+
**Machine-local BY DESIGN.** The artifact store + the route are per-machine: the capstone
|
|
73
|
+
is RUN from one machine (the test driver) and asserts the reply was served FROM the target
|
|
74
|
+
machine via `PlacementResponderReader` (GET /pool/placement). The cross-machine fact being
|
|
75
|
+
PROVEN is the responder identity; the runner itself does not replicate state. Recorded
|
|
76
|
+
artifacts are local observability (like the live-test gate's). No generated URLs; no
|
|
77
|
+
user-facing notice (dark infra) → no one-voice concern.
|
|
78
|
+
|
|
79
|
+
## 8. Rollback cost
|
|
80
|
+
Trivial. Dark by default (503 on fleet). Back-out = omit/disable
|
|
81
|
+
`monitoring.liveTestRunner.enabled` (already the fleet default) or revert the PR — no data
|
|
82
|
+
migration, no agent-state repair. The shared store is additive (read-only to the gate).
|
|
83
|
+
|
|
84
|
+
## Second-pass review
|
|
85
|
+
**Not required.** The change adds a test-harness RUN route with no block/allow authority
|
|
86
|
+
over agent behavior, no session-lifecycle mutation, and no gate/sentinel/watchdog decision
|
|
87
|
+
logic. It interacts with `LiveTestGate` only by sharing an artifact store (read path
|
|
88
|
+
unchanged). The seat-move uses the existing honest `/pool/transfer` contract. Risk is
|
|
89
|
+
bounded by the dev-gate + dark default + safe-only scenarios.
|
|
@@ -0,0 +1,43 @@
|
|
|
1
|
+
# Side-Effects Review — platform/workspace-aware placement (CMT-1568)
|
|
2
|
+
|
|
3
|
+
**Slug:** placement-platform-workspace-aware
|
|
4
|
+
**Spec:** docs/specs/placement-platform-workspace-aware.md (converged 6 iterations + approved)
|
|
5
|
+
**Files:** src/core/machineServesChannel.ts (+test), src/core/PlacementExecutor.ts (+test), src/core/types.ts, src/core/PeerPresencePuller.ts, src/core/MachinePoolRegistry.ts, src/messaging/slack/SlackAdapter.ts, src/commands/server.ts, src/core/SessionRouter.ts (+test)
|
|
6
|
+
**Posture:** ships behind the EXISTING multi-machine path. Single-machine = no-op (lone machine serves its own channels / fail-open). The filter is a strict no-op until a caller threads `req.channel` (only the Slack inbound does) — so the change is inert everywhere else.
|
|
7
|
+
|
|
8
|
+
## What it is
|
|
9
|
+
Closes the live-test-caught bug: placement assigned a Slack channel to a machine connected to a DIFFERENT workspace → black-hole. Adds `MachineCapacity.servesChannels` (adapter-DERIVED reachability), a three-valued (`yes`/`no`/`unknown`) `machineServesChannel` filter in `PlacementExecutor.decide()`, and threads the channel scope from the Slack inbound through the router into placement.
|
|
10
|
+
|
|
11
|
+
## Phase 1 — Principle check (signal vs authority)
|
|
12
|
+
The `yes`/`unknown` narrowing is a pure SIGNAL fed into the existing `PlacementExecutor` authority (same posture as CMT-1570's quota signal) — it never blocks a message. The one deterministic refusal (all-`no` → `queued`/`no-machine-serves-channel`) is a STRUCTURAL VALIDATOR per docs/signal-vs-authority.md: it rests on the enumerable objective fact "zero connected machines reach this workspace," not a judgment — and it reuses the EXISTING `queued`/null unsatisfiable outcome (no new blocking authority, no new return variant). Compliant.
|
|
13
|
+
|
|
14
|
+
## Phase 4 — Side-effects answers
|
|
15
|
+
1. **Over-block** — could a machine that CAN serve be dropped? Only on a PRESENT-and-fresh `no`. Absent/legacy signal = `unknown` = eligible (fail-open). `yes` is ranked ABOVE `unknown` so a missing signal never strands a known-reachable machine. Worst over-block: a brief stale `no` between a workspace reconnect and the next heartbeat → that machine is skipped for one window (recovered next beat). Bounded.
|
|
16
|
+
2. **Under-block** — workspace-membership ≠ channel-permission (private-channel/bot-membership/Slack-Connect). Accepted as a first-stage filter (the spec's reachability-not-authority scope); the failed-post-feedback follow-up unifies it with the false-`yes` case. The wrong-workspace STRUCTURAL black-hole (the actual bug) IS closed.
|
|
17
|
+
3. **Level-of-abstraction fit** — correct: mirrors CMT-1570 exactly (a capability signal on the capacity heartbeat + a parallel `decide()` filter). No new layer.
|
|
18
|
+
4. **Signal vs authority** — see Phase 1. Compliant.
|
|
19
|
+
5. **Interactions** — the filter runs AFTER the quota gate, BEFORE the ordering loop; it narrows `candidates` (and `eligible`, so a hard-pin to a `no` machine → the existing `hard-pin-unavailable`/`hard-pin-unsatisfiable` path). It does not shadow the quota gate (orthogonal signal) or the capability (`capOk`) gate. The all-`no` `queued` outcome flows into the EXISTING SessionRouter `decision.outcome === 'queued'` handler (raiseAttention + queueMessage) — no new consumer code; the dedup/wording refinement of that attention is a tracked follow-up (the generic attention already surfaces it, never a black-hole).
|
|
20
|
+
6. **External surfaces** — the producer reports `servesChannels` on the capacity heartbeat (other machines read it — same surface as quotaState). The Slack inbound now threads the channel scope. **KEY DECISION: telegram is NOT threaded** — telegram is a SHARED chat (every machine reports the same chatId → the filter is a no-op for telegram), so only the Slack inbound sets `req.channel`. This sidesteps the telegram-chatId-secret-resolution complication AND the "present-servesChannels-without-a-telegram-block reads as `no`" trap (telegram channels never consult the filter). The producer still reports telegram when its chatId is a concrete value, for completeness.
|
|
21
|
+
7. **Multi-machine posture** — REPLICATED: `servesChannels` rides the existing capacity-heartbeat path (PeerPresencePuller allow-list + recordHeartbeat + both PeerCapacity types; MachinePoolRegistry obs type + forward), exactly like quotaState. Single-machine = no-op. This is the whole point — it makes "follow the user across machines" correct for per-machine-scoped Slack workspaces.
|
|
22
|
+
8. **Rollback cost** — low: revert the commit. The filter is fail-open + inert without `req.channel`; `servesChannels` is an optional heartbeat field (older peers omit it → `unknown`). No migration, no durable-state change.
|
|
23
|
+
|
|
24
|
+
## Spec-implementation reconciliations (documented build-time refinements of the approved spec)
|
|
25
|
+
- **all-`no` returns `queued` (not "local-ingress machine"):** the real `decide()` contract ALREADY returns `queued`/null for unsatisfiable cases (`no-capable-machine`, `hard-pin-unavailable`); the pure `decide()` has no "ingress machine" concept. So all-`no` reuses `queued`/`no-machine-serves-channel` — contract-consistent, surfaced via the existing queued→attention path. Spec updated to match.
|
|
26
|
+
- **telegram not threaded** (finding G6 above) — documented in the spec's ingress-model section (telegram is shared → filter no-op) and here.
|
|
27
|
+
|
|
28
|
+
## No-deferrals
|
|
29
|
+
The security-hardening (corroborated/leased `yes`), the channel-ACL routing, and the Attention-dedup-wording refinement are tracked follow-ups (`<!-- tracked: CMT-1568 -->` in the spec) — NOT deferrals of THIS fix, which closes the wrong-workspace black-hole completely and is fully tested (71 unit tests across machineServesChannel + PlacementExecutor + SessionRouter, incl. the behavioral proof + the failover wiring-integrity test).
|
|
30
|
+
|
|
31
|
+
## Phase 5 — Second-pass review (routing decision point)
|
|
32
|
+
An independent implementation reviewer traced every `decide()` branch + the threading. **Concur — implementation correct on the routing hot path, no ship-blocking bugs.** Verified: yes-over-unknown ranking; all-`no` early-return guarded (`candidates.length>0` — never a misleading channel verdict when quota already emptied candidates); `eligible` narrowing makes pin-to-`no`→hard-pin-unsatisfiable while pin-to-`unknown` stays honored; producer is adapter-derived (slack gated on `isConnected`) and the telegram `typeof` guard correctly rejects a `{secret:true}` object marker; undefined workspaceId → `unknown`/fail-open (not `no`); `_connected` lifecycle correct across reconnects; the telegram trap is avoided (verified BOTH telegram `route()` callers + the mesh redrive thread NO `channel` → telegram always `unknown`/no-op).
|
|
33
|
+
|
|
34
|
+
Two non-blocking forward-looking notes (both tracked, neither a live bug):
|
|
35
|
+
- **RebalancePlanner.decide() doesn't thread `channel`** — a dormant black-hole door (rebalance could move a Slack session onto a non-serving machine). DORMANT: RebalancePlanner is not wired into server.ts. Added a tracked guard-comment at RebalancePlanner.ts:83 to plumb `channel` before rebalance is ever activated. <!-- tracked: CMT-1568 -->
|
|
36
|
+
- **machineServesChannel telegram→`no` foot-gun** — if a future caller threads a telegram channel, a Slack-only machine would be wrongly excluded. Safe NOW by producer discipline (telegram is never threaded — documented invariant). <!-- tracked: CMT-1568 -->
|
|
37
|
+
|
|
38
|
+
## Phase 6 — CI-ratchet fix (post-review, 2026-06-16)
|
|
39
|
+
Two structural-gate failures surfaced on CI after the advert field landed; both fixed without weakening the gates:
|
|
40
|
+
- **no-silent-fallbacks ratchet (475→474):** the `selfServesChannels` heartbeat producer originally wrapped its reads in a `try/catch { return undefined }`, which the ratchet counts as a silent fallback. Refactored to inline non-throwing reads — `config.messaging.find` + `typeof`/optional-chaining guards never throw on normal input, so an unresolvable field simply omits its block (→ placement treats the machine as `unknown`/fail-open, identical older-peer semantics). No catch, no `@silent-fallback-ok` exemption — the count returns to baseline 474 by ELIMINATING the fallback, not annotating it.
|
|
41
|
+
- **peer-presence-puller wiring-integrity ratchet:** the every-advert-field round-trip test (`SESSION_STATUS_ADVERT_FIELDS`) failed because its fixture didn't set the new `servesChannels` field. Fixture updated; the narrowing return already passed the field through. The ratchet did its job — a forgotten field failed loudly.
|
|
42
|
+
|
|
43
|
+
No behavior change to the placement hot path; the producer's reachability output is identical. 91 unit tests green (machineServesChannel + PlacementExecutor + SessionRouter + peer-presence-puller + no-silent-fallbacks).
|