instar 1.3.688 → 1.3.690
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 +106 -0
- package/dist/commands/server.js.map +1 -1
- package/dist/config/ConfigDefaults.d.ts.map +1 -1
- package/dist/config/ConfigDefaults.js +11 -0
- package/dist/config/ConfigDefaults.js.map +1 -1
- package/dist/core/SessionManager.d.ts.map +1 -1
- package/dist/core/SessionManager.js +8 -0
- package/dist/core/SessionManager.js.map +1 -1
- package/dist/core/devGatedFeatures.d.ts.map +1 -1
- package/dist/core/devGatedFeatures.js +5 -0
- package/dist/core/devGatedFeatures.js.map +1 -1
- package/dist/core/leaseGatedSpawn.d.ts +232 -0
- package/dist/core/leaseGatedSpawn.d.ts.map +1 -0
- package/dist/core/leaseGatedSpawn.js +216 -0
- package/dist/core/leaseGatedSpawn.js.map +1 -0
- package/dist/core/nobodyPollingRecovery.d.ts +138 -0
- package/dist/core/nobodyPollingRecovery.d.ts.map +1 -0
- package/dist/core/nobodyPollingRecovery.js +147 -0
- package/dist/core/nobodyPollingRecovery.js.map +1 -0
- 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.map +1 -1
- package/dist/server/routes.js +95 -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.689.md +30 -0
- package/upgrades/1.3.690.md +28 -0
- package/upgrades/side-effects/mesh-self-heal-g2-core.md +32 -0
- package/upgrades/side-effects/mesh-self-heal-g2-observe.md +34 -0
- package/upgrades/side-effects/mesh-self-heal-g3.md +45 -0
- package/upgrades/side-effects/mesh-self-heal-spec.md +37 -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-28T09:16:28.110Z",
|
|
5
|
+
"instarVersion": "1.3.690",
|
|
6
6
|
"entryCount": 202,
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
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": "f53b56a0023769d5746f77707be0cee1718f0574871c0c42d916c044b516c8af",
|
|
774
774
|
"since": "2025-01-01"
|
|
775
775
|
},
|
|
776
776
|
"cli:init": {
|
|
@@ -1538,7 +1538,7 @@
|
|
|
1538
1538
|
"type": "subsystem",
|
|
1539
1539
|
"domain": "sessions",
|
|
1540
1540
|
"sourcePath": "src/core/SessionManager.ts",
|
|
1541
|
-
"contentHash": "
|
|
1541
|
+
"contentHash": "3e13762a59be73368a3e1f870e0c0585a957edee6a8db3b26b3614b3001b0c3e",
|
|
1542
1542
|
"since": "2025-01-01"
|
|
1543
1543
|
},
|
|
1544
1544
|
"subsystem:auto-updater": {
|
|
@@ -0,0 +1,30 @@
|
|
|
1
|
+
# Upgrade Guide — vNEXT
|
|
2
|
+
|
|
3
|
+
<!-- assembled-by: assemble-next-md -->
|
|
4
|
+
<!-- bump: patch -->
|
|
5
|
+
|
|
6
|
+
## What Changed
|
|
7
|
+
|
|
8
|
+
Mesh Self-Heal **G3** — the first build increment of `MESH-SELF-HEAL-SPEC` (lease↔job binding, machine-independence). It makes duplicate sessions structurally impossible across a multi-machine setup:
|
|
9
|
+
|
|
10
|
+
- **Lease-gated spawn** (`src/core/leaseGatedSpawn.ts`): when an inbound Telegram message has no session yet, a machine spawns a session for that topic ONLY if it genuinely holds the fenced awake-lease; otherwise it forwards to the holder ("spawn iff holder, else forward"). It keys on the one trustworthy authority (`holdsLease()`), never an unreliable placement view, and FAILS toward spawn when no forward seam exists — it never strands a message to avoid a duplicate.
|
|
11
|
+
- **Single-writer binding lifecycle**: a topic→session binding is cleared the instant its session is killed ("a binding exists IFF a live session exists"), so a dead session can't silently resurrect and double-serve. The cleanup deliberately SKIPS context-exhaustion / recovery kills (which are immediately followed by a same-topic respawn), mirroring the resume-UUID-save guard, so promoting it can never break recovery.
|
|
12
|
+
- **Evaluable soak evidence** (`GET /mesh-selfheal/g3`): a read-only ledger records the counterfactual ("how many duplicate sessions I would have prevented if enabled" / "stale bindings I would have cleared") plus a deterministic promotion recommendation — so this dark feature can actually earn graduation instead of rotting silently.
|
|
13
|
+
|
|
14
|
+
Ships **dark + dry-run** behind `multiMachine.sessionPool.ownershipCheckedSpawn` (dev-gated): a strict no-op on a single machine or with the flag off. Next increments: G2 (nobody-serving alarm), G1 (badge↔job binding).
|
|
15
|
+
|
|
16
|
+
## What to Tell Your User
|
|
17
|
+
|
|
18
|
+
Nothing changes yet — this ships **off by default** (⚗️ experimental, dark + dry-run) while it soaks on a real two-machine pair. When it's active it prevents the "two of me replying to one message" duplicate-session problem on multi-machine setups, with no action needed from you. Single-machine agents are unaffected.
|
|
19
|
+
|
|
20
|
+
## Summary of New Capabilities
|
|
21
|
+
|
|
22
|
+
- `GET /mesh-selfheal/g3` — read-only soak-evidence + promotion recommendation for the lease-gated-spawn gate (agent observability; not a user-invokable capability).
|
|
23
|
+
- Config: `multiMachine.sessionPool.ownershipCheckedSpawn` `{ enabled, dryRun }` — opt-in lease-gated spawn + single-writer binding lifecycle (dark + dry-run by default).
|
|
24
|
+
|
|
25
|
+
## Evidence
|
|
26
|
+
|
|
27
|
+
- New unit suite `tests/unit/leaseGatedSpawn.test.ts` (30 tests) covers both sides of every decision boundary — spawn-iff-holder, forward, dry-run counterfactual, spawn-when-no-seam, binding clear/skip/respawn-imminent.
|
|
28
|
+
- New integration suite `tests/integration/mesh-selfheal-g3-route.test.ts` (3 tests) drives `GET /mesh-selfheal/g3` over the real HTTP path and confirms both counters + the promotion recommendation.
|
|
29
|
+
- `tests/unit/lint-dev-agent-dark-gate.test.ts` updated for the new gate's dark-gate exclusion entry; full dark-gate lint green (24 tests).
|
|
30
|
+
- Typecheck clean; 57 targeted tests green. Independent second-pass review concurred (audited the recovery-bounce fix).
|
|
@@ -0,0 +1,28 @@
|
|
|
1
|
+
# Upgrade Guide — vNEXT
|
|
2
|
+
|
|
3
|
+
<!-- assembled-by: assemble-next-md -->
|
|
4
|
+
<!-- bump: patch -->
|
|
5
|
+
|
|
6
|
+
## What Changed
|
|
7
|
+
|
|
8
|
+
Mesh Self-Heal **G2 — core decision logic** (increment 1 of the nobody-serving alarm, `MESH-SELF-HEAL-SPEC` §3.2). Adds `src/core/nobodyPollingRecovery.ts`: the PURE, deterministic decision core for the silent-loss backstop — when NO machine is polling Telegram (the zombie-lease-holder state behind the message-drop incidents), exactly ONE fit machine must take over, never zero (drops), never two (the 409 poll-war).
|
|
9
|
+
|
|
10
|
+
- `electPollClaimant` — deterministic single-claimant election (F4-preferred-awake if fit, else lowest-machineId fit). Machine-agnostic: every machine elects the SAME claimant, which is the structural defense against split-brain double-claim.
|
|
11
|
+
- `decideNobodyPollingClaim` — reduces over the EXISTING B5 detector (`pollerCount.ts`): `ok`→no-op, `dual`→veto (claiming into 2 pollers IS the 409 war), `indeterminate`→fail-closed, unconfirmed silence→await, peer-confirmed global outage→hold, confirmed silence→elect + claim/stand-down/escalate.
|
|
12
|
+
- `decidePostCasSelfReverify` — "CAS-win is necessary but not sufficient": re-check live local poll-freshness before serving; on self-unfit, relinquish + self-exclude.
|
|
13
|
+
- `NobodyPollingLedger` — evaluable soak evidence (episodes, claims, stand-downs, self-exclusions, vetoes), mirroring G3's close-the-loop ledger.
|
|
14
|
+
|
|
15
|
+
It also wires the core to a **read-only observe surface**, `GET /mesh-selfheal/g2`: it computes the live nobody-polling verdict over the pool, debounces a silence across reads, runs the decision (who would claim), records the soak counterfactual, and returns it. The surface is OBSERVE-only — it reports the decision but does NOT enact it (no lease acquire, no polling change). The enforce-mode actuation (actually taking over polling via the existing poll-follows-lease lever) is the next increment.
|
|
16
|
+
|
|
17
|
+
## What to Tell Your User
|
|
18
|
+
|
|
19
|
+
Nothing changes in behavior yet — this is the detector + a read-only status surface for the upcoming "nobody-serving alarm," not yet acting anywhere. When the enforce step lands (next increment), it will detect the "nobody is polling Telegram" state across your machines and have exactly one fit machine take over automatically, so messages stop dropping when a machine goes quiet. No action needed.
|
|
20
|
+
|
|
21
|
+
## Summary of New Capabilities
|
|
22
|
+
|
|
23
|
+
- `GET /mesh-selfheal/g2` — read-only observe surface for the nobody-polling detector: live verdict + the single-claimant decision + soak evidence (agent observability; not a user-invokable capability). No config flag or write surface yet; observe-only (no actuation).
|
|
24
|
+
|
|
25
|
+
## Evidence
|
|
26
|
+
|
|
27
|
+
- `tests/unit/nobodyPollingRecovery.test.ts` — 18 unit tests covering both sides of every decision boundary (election: preferred-fit / preferred-unfit / no-fit / determinism; verdict reduction: ok/dual/indeterminate/await/hold/claim/stand-down/escalate; self-reverify: fresh→serve / stale→relinquish; ledger accounting).
|
|
28
|
+
- `tests/integration/mesh-selfheal-g2-route.test.ts` — 4 integration tests over the real HTTP pipeline (feature-alive 200; confirmed-silence→claim; transient-silence→await-confirm; two-pollers→veto). All green; typecheck clean.
|
|
@@ -0,0 +1,32 @@
|
|
|
1
|
+
# Side-Effects Review — Mesh Self-Heal G2 (core decision logic)
|
|
2
|
+
|
|
3
|
+
**Change:** The PURE decision core of G2 (nobody-polling detector + single-claimant recovery) from `MESH-SELF-HEAL-SPEC.md` §3.2 — `src/core/nobodyPollingRecovery.ts` + 18 unit tests. Three pure functions (`electPollClaimant`, `decideNobodyPollingClaim`, `decidePostCasSelfReverify`) + a soak ledger. This is increment 1 of G2: the decision logic only. It is NOT YET WIRED to any tick, route, or actuation — it has ZERO runtime effect until a later increment consumes it. (Wiring + the `/mesh-selfheal/g2` observe route + the enforce-mode actuation via the existing poll-follows-lease lever + CAS are the next increment — see MESH-SELF-HEAL-G2-BUILD.md.)
|
|
4
|
+
|
|
5
|
+
**Decision point?** The MODULE encodes a decision (who claims poll-ownership), but nothing CALLS it yet, so in this increment there is no live decision-point. Signal-vs-authority still assessed (Q4).
|
|
6
|
+
|
|
7
|
+
## 1. Over-block (legitimate inputs wrongly rejected?)
|
|
8
|
+
N/A as shipped — unwired, no inputs flow through it. By design the decision FAILS toward NOT-claiming on every ambiguity (`dual`→veto, `indeterminate`→fail-closed, unconfirmed silence→await, global-outage→hold, no-fit→escalate). The only `claim` outcome requires confirmed real silence + this machine being the deterministically-elected single claimant. A wrongful claim is the harm (a 2nd poller / 409 war), so the gate is biased hard against claiming.
|
|
9
|
+
|
|
10
|
+
## 2. Under-block (failure modes still missed?)
|
|
11
|
+
As pure logic, none in-scope. The KNOWN not-yet-built pieces (tracked, not orphan-deferred): the actuation (CAS acquire + poll-lever start/stop) and the post-claim live-verify of `lifeline-poll-active.json`. Until those land, this module cannot cause OR prevent anything — it is inert decision logic with full test coverage.
|
|
12
|
+
|
|
13
|
+
## 3. Level-of-abstraction fit
|
|
14
|
+
Correct layer. It REDUCES over the existing B5 detector (`pollerCount.ts` `evaluatePollerCount`) rather than re-folding poll-counts (spec finding Int2-A — reuse, don't reinvent). It is a pure decision module mirroring `leaseGatedSpawn.ts` (G3) — no I/O, fully unit-testable, consumed by a thin wiring layer later.
|
|
15
|
+
|
|
16
|
+
## 4. Signal vs authority compliance
|
|
17
|
+
COMPLIANT. Pure functions (no I/O, no blocking authority). The election is deterministic + machine-agnostic (every machine computing it over the same inputs elects the SAME claimant — the structural defense against split-brain double-claim). The actual authority (acquiring the fenced epoch-CAS) lives in the existing lease coordinator, which the future wiring will call; this module only DECIDES, it does not ENACT. `decidePostCasSelfReverify` encodes "CAS-win is necessary but not sufficient" (Adv2-F1) so the eventual enactment re-checks live local freshness before serving.
|
|
18
|
+
|
|
19
|
+
## 5. Interactions
|
|
20
|
+
None in this increment (unwired). The design's intended interactions (documented for the wiring increment): consumes B5's verdict; the claim drives the existing poll-follows-lease lever; self-exclusion advertises into the existing heartbeat. No shadowing — it adds a decision layer ON TOP of B5, not parallel to it.
|
|
21
|
+
|
|
22
|
+
## 6. External surfaces
|
|
23
|
+
None yet. No route, no config flag, no audit write in this increment — the module is pure. The exported `sharedG2NobodyPollingLedger` singleton mirrors `sharedG3SoakLedger`; it is written/read only once the wiring + `/mesh-selfheal/g2` route land.
|
|
24
|
+
|
|
25
|
+
## 7. Multi-machine posture (Cross-Machine Coherence)
|
|
26
|
+
This IS multi-machine coherence logic. Posture: the decision is computed LOCALLY on each machine from the replicated/advertised pool capacities + B5 verdict, and is deterministic so all machines agree on the single claimant without a coordination round-trip (the election is the coherence mechanism). The watermarks it will eventually read (`pollSucceededMonoMs` etc.) are machine-local, never replicated (spec §3.1 Sca-F1) — but that plumbing is the wiring increment, not this one.
|
|
27
|
+
|
|
28
|
+
## 8. Rollback cost
|
|
29
|
+
Trivial — revert the commit. The module is unreferenced by any runtime path, so removing it cannot affect a running agent. No config, no migration, no state.
|
|
30
|
+
|
|
31
|
+
## Second-pass review
|
|
32
|
+
Not triggered for this increment: the module is unwired pure logic with no live decision-point, block/allow authority, or session-lifecycle touch. The Phase-5 second-pass IS required for the WIRING increment (it will consume the lease/poll-ownership authority — "lease"/"sentinel"/session-lifecycle territory) and is flagged in MESH-SELF-HEAL-G2-BUILD.md.
|
|
@@ -0,0 +1,34 @@
|
|
|
1
|
+
# Side-Effects Review — Mesh Self-Heal G2 (observe route)
|
|
2
|
+
|
|
3
|
+
**Change:** Wires the G2 pure core (this PR's first commit) to a read-only observe surface: `GET /mesh-selfheal/g2` (`src/server/routes.ts`) + 4 integration tests. The route computes the B5 verdict over the live pool (`machinePoolRegistry.getCapacities()`), debounces a `silence` across reads (`G2_NOBODY_POLLING_CONFIRM_OBSERVATIONS = 3`), runs `decideNobodyPollingClaim`, records the counterfactual to `sharedG2NobodyPollingLedger`, and returns `{ verdict, silenceStreak, silenceConfirmed, selfMachineId, decision, ledger }`.
|
|
4
|
+
|
|
5
|
+
**Decision point?** The route REPORTS a decision (who would claim poll-ownership) but does NOT ENACT it — no fenced-CAS acquire, no poll-lever write. It is observe/read-only. The enacting authority (the enforce increment) is explicitly NOT in this change.
|
|
6
|
+
|
|
7
|
+
## 1. Over-block
|
|
8
|
+
N/A — read-only, blocks nothing. The decision it reports is biased hard against claiming (dual→veto, indeterminate→fail-closed, unconfirmed-silence→await, no-fit→escalate); only a confirmed real silence with this machine as the deterministic single claimant yields `claim`.
|
|
9
|
+
|
|
10
|
+
## 2. Under-block
|
|
11
|
+
The observe surface does not act, so it can neither over- nor under-block. KNOWN not-yet-built (tracked, not orphan-deferred): the enforce actuation (CAS acquire + `writePollIntent` poll-lever + post-claim live-verify of `lifeline-poll-active.json`) and peer-evidence-of-global-outage plumbing (`globalOutageEvidence` is hard-coded false here → a confirmed silence proceeds to elect, which is the observe-correct behavior; the HOLD-on-global path is the enforce increment).
|
|
12
|
+
|
|
13
|
+
## 3. Level-of-abstraction fit
|
|
14
|
+
Correct. The route reuses the EXACT capacity-gathering + `poolPollerVerdict` the adjacent `GET /pool/poller-count` already uses (no new pool plumbing), and the pure decision lives in `nobodyPollingRecovery.ts`. The route is a thin reporter over both.
|
|
15
|
+
|
|
16
|
+
## 4. Signal vs authority compliance
|
|
17
|
+
COMPLIANT. Read-only observability with no blocking/enacting authority. `selfMachineId` is read from `coordinator.identity?.machineId`; the election is deterministic + machine-agnostic. The route records soak evidence (signal) and returns the decision (signal) — it does not acquire the lease or change polling.
|
|
18
|
+
|
|
19
|
+
## 5. Interactions
|
|
20
|
+
- Reuses `poolPollerVerdict` (B5) — no reimplementation, no double-fold.
|
|
21
|
+
- The `_g2SilenceStreak` debounce counter is module-scoped per `createRoutes` call (one server process = one counter) — correct (a single server's consecutive reads). It is reset on any non-silence verdict.
|
|
22
|
+
- Records to the process-wide `sharedG2NobodyPollingLedger` — the same singleton the enforce increment's evaluator will use; read-driven recording means the ledger advances as the route is polled (documented; the periodic evaluator is the enforce increment).
|
|
23
|
+
|
|
24
|
+
## 6. External surfaces
|
|
25
|
+
New route `GET /mesh-selfheal/g2` — Bearer-gated like all routes; classified under the existing `mesh-selfheal` INTERNAL_PREFIXES entry (added by G3, already on main) so it's an agent-read, not a user capability. No config flag, no write surface, no audit file in this increment.
|
|
26
|
+
|
|
27
|
+
## 7. Multi-machine posture (Cross-Machine Coherence)
|
|
28
|
+
This IS the cross-machine coherence read. The decision is computed locally from the pool's advertised capacities + the deterministic election, so every machine's `/mesh-selfheal/g2` agrees on the single claimant without a coordination round-trip. `selfMachineId` is machine-local. Read-only — nothing replicates or strands on transfer.
|
|
29
|
+
|
|
30
|
+
## 8. Rollback cost
|
|
31
|
+
Trivial — revert the commit. The route is read-only and unreferenced by any actuation path; removing it cannot affect a running agent's behavior. No migration, no state.
|
|
32
|
+
|
|
33
|
+
## Second-pass review
|
|
34
|
+
Not triggered: the route holds NO block/allow authority, touches NO session lifecycle, and performs NO actuation — it is read-only observability over a pure decision. The Phase-5 second-pass IS required for the ENFORCE increment (CAS + poll-lever actuation = lease/poll-ownership authority) and remains flagged in MESH-SELF-HEAL-G2-BUILD.md.
|
|
@@ -0,0 +1,45 @@
|
|
|
1
|
+
# Side-Effects Review — Mesh Self-Heal G3 (lease-gated spawn + single-writer binding lifecycle)
|
|
2
|
+
|
|
3
|
+
**Change:** G3 of MESH-SELF-HEAL-SPEC. A machine spawns a session for an inbound topic ONLY if it genuinely holds the fenced awake-lease, else it forwards to the holder ("spawn iff holder, else forward"); and a topic→session binding is cleared the instant its session is killed ("a binding exists IFF a live session exists"). Both fed by a shared soak-evidence ledger surfaced read-only at `GET /mesh-selfheal/g3`. Ships **dark + dryRun** behind `multiMachine.sessionPool.ownershipCheckedSpawn`.
|
|
4
|
+
|
|
5
|
+
**Decision point?** YES — this gates session SPAWN (a decision that constrains agent behavior). Signal-vs-authority applies (Q4 below).
|
|
6
|
+
|
|
7
|
+
## 1. Over-block (legitimate inputs wrongly rejected?)
|
|
8
|
+
The spawn gate's only non-spawn outcome is `forward` (a non-holder forwards to the holder) — that is the CORRECT behavior, not a block, and it only fires when a real forward seam exists. When the forward seam is unavailable the gate SPAWNS anyway (`spawn-forward-unavailable`) — it never strands a message to avoid a duplicate. Single-machine and flag-off resolve to `spawn` (byte-for-byte legacy). The binding-cleanup only clears a binding whose session is being killed (a dead pointer) — it cannot remove a binding for a live session. **No legitimate spawn is rejected.**
|
|
9
|
+
|
|
10
|
+
## 2. Under-block (failure modes still missed?)
|
|
11
|
+
With the flag OFF (default), nothing is enforced — both bugs (duplicate spawn, stale-binding resurrection) can still occur; this is intentional (dark rollout). In dryRun, the gate records the counterfactual but still spawns, so it OBSERVES rather than prevents. The HTTP-path (`/internal/telegram-forward`) gate is record-only (no forward seam there) so even when enabled it does not forward — it relies on the cold-spawn gate for the primary prevention. G2 (nobody-polling) is the bounded backstop for the fail-closed forward path; G2 is a separate increment (tracked, not deferred-orphan).
|
|
12
|
+
|
|
13
|
+
## 3. Level-of-abstraction fit
|
|
14
|
+
The gate keys on the ONE trustworthy authority — `MultiMachineCoordinator.holdsLease()` (the fenced lease) — not a placement view (which the spec foundation note B shows is unreliable from a non-router machine). Binding cleanup hooks the ONE convergence point all kill paths fire (`beforeSessionKill`), so it covers kill/reaper/recovery/transfer-closeout uniformly rather than being bolted onto one spawn path. Correct layer.
|
|
15
|
+
|
|
16
|
+
## 4. Signal vs authority compliance
|
|
17
|
+
`leaseGatedSpawn.ts` is a PURE decision function (no I/O) — a signal producer. Its only authority is "spawn locally vs forward," and it FAILS CLOSED to forward only when a real seam exists, else spawns (never strands). The binding-cleanup is gated dark+dryRun and clears only dead pointers. The `/mesh-selfheal/g3` route is READ-ONLY observability. No brittle check holds destructive blocking authority. **Compliant.**
|
|
18
|
+
|
|
19
|
+
## 5. Interactions
|
|
20
|
+
- `beforeSessionKill` binding-cleanup is registered AFTER the resume-UUID-save listener, so the UUID (kept in TopicResumeMap, keyed by topicId — independent store) is saved before the binding is cleared; clearing never loses resume. Verified by test.
|
|
21
|
+
- **Recovery-respawn paths (FIXED — see Second-Pass §RESOLVED).** A context-exhaustion / recovery kill is immediately followed by a SAME-TOPIC respawn (`respawnSession` / `respawnSessionFresh`) that resolves its target via `getSessionForTopic`; clearing the binding first would null that lookup and silently abort recovery. The binding-cleanup now skips such kills (`respawnImminent`, fed by `contextExhaustionKills`) — identical skip-on-respawn semantics to the resume-UUID-save sibling. So "kill → fire the same listener → binding cleared" applies to TERMINAL kills (reaper / operator / transfer-closeout), NOT the kill-then-respawn-same-topic recovery paths.
|
|
22
|
+
- The cold-spawn gate and the HTTP-path record share ONE `sharedG3SoakLedger` singleton (server.ts records, routes.ts reads) — no double-counting (distinct callsites).
|
|
23
|
+
- Transfer-closeout already closes the old session by KILLING it → fires the same `beforeSessionKill` cleanup. No race with post-transfer closeout; it IS the closeout's binding-clear.
|
|
24
|
+
- No shadowing of existing reaper/recovery (they kill → fire the same listener → binding cleared, which is correct).
|
|
25
|
+
|
|
26
|
+
## 6. External surfaces
|
|
27
|
+
New route `GET /mesh-selfheal/g3` (classified INTERNAL_PREFIXES — agent-reads, not a user capability). New audit file `logs/mesh-selfheal.jsonl` (one line per binding transition). No user-visible behavior change while dark. Depends on the lease coordinator being present for non-single-machine gating (null → single-machine no-op).
|
|
28
|
+
|
|
29
|
+
## 7. Multi-machine posture (Cross-Machine Coherence)
|
|
30
|
+
**Machine-local BY DESIGN.** The topic→session binding names a LOCAL tmux session — it cannot be meaningfully replicated (spec §3.3). Its lifecycle binds to the local session. The gate's whole PURPOSE is cross-machine coherence (prevent two machines serving one topic) and it keys on the cross-machine fenced lease. The soak ledger + audit are per-machine observability. One-voice/transfer/URL concerns: the forward path reuses the existing SessionRouter (one owner serves); transfer-closeout clears the binding via the same hook (no strand — durable inbound queue carries in-flight to the new owner).
|
|
31
|
+
|
|
32
|
+
## 8. Rollback cost
|
|
33
|
+
Trivial. Ships dark (`ownershipCheckedSpawn.enabled:false, dryRun:true`) — a strict no-op until deliberately enabled. To back out: leave the flag off (no behavior), or revert the commit. No data migration, no agent-state repair. The audit file is append-only observability (safe to ignore/delete).
|
|
34
|
+
|
|
35
|
+
## Second-Pass Review
|
|
36
|
+
|
|
37
|
+
**Concern raised: when the binding-cleanup is ENABLED + live (not dryRun), it breaks context-exhaustion recovery — the artifact's §5 "recovery is unaffected" claim is wrong for that path.**
|
|
38
|
+
|
|
39
|
+
What I independently verified against the code:
|
|
40
|
+
- **Q4 (signal-vs-authority) and the dark+dryRun no-op are sound.** `decideLeaseGatedSpawn` returns `spawn` for `!flagEnabled` (line 77-79) and for `singleMachine` (line 80-82) before ever consulting the lease — a strict byte-for-byte no-op, confirmed. The spawn gate fails toward spawn (`spawn-forward-unavailable` when no seam) and never strands; the HTTP path (`forwardAvailable:false`, routes.ts:16230) is genuinely record-only and unconditionally spawns. No brittle check holds destructive authority.
|
|
41
|
+
- **Q5 (resume never lost) is true for the happy path.** The two `beforeSessionKill` listeners are registered UUID-save-first (server.ts:8738) then binding-cleanup (server.ts:8840); EventEmitter fires in registration order, and `TopicResumeMap` persists keyed by `topicId` in a separate file-backed store, so `unregisterTopic` (a topic→tmux map delete) cannot touch the resume UUID. Confirmed.
|
|
42
|
+
- **The defect (Q1/Q2 omission).** The UUID-save listener deliberately SKIPS context-exhaustion kills (`!contextExhaustionKills.has(...)`, server.ts:8742) to avoid a death loop, but the binding-cleanup listener had NO such guard. SessionRecovery kills via `killForRecovery` → `terminateSession` → synchronous `emit('beforeSessionKill')` → `unregisterTopic(topicId)` BEFORE it calls the respawn callback. Both Telegram recovery respawns then resolve their target via `telegram.getSessionForTopic(topicId)` and bail on null (`respawnSession` server.ts:9309-9310; `respawnSessionFresh` server.ts:9437-9441). With the binding cleared, that lookup returns null → recovery silently aborts and the context-exhausted session is killed but never respawned. Invisible today (dark+dryRun) but a real regression the moment the flag is promoted — exactly the state the soak is evidence FOR.
|
|
43
|
+
|
|
44
|
+
### RESOLVED (in this commit)
|
|
45
|
+
Took the structural fix, not a "fix-before-promotion" prose note (a deferral the standards forbid): the pure `decideBindingCleanupOnKill` now takes a `respawnImminent` input and returns a distinct `skip-respawn-kill` action (`clearNow:false`) that takes precedence over both `clear` and `dry-run-would-clear` — so a recovery kill neither clears the binding nor records a misleading would-clear counterfactual. The `beforeSessionKill` wiring feeds `respawnImminent: contextExhaustionKills.has(session.tmuxSession)`, giving the binding-cleanup listener IDENTICAL skip-on-respawn semantics to its resume-UUID-save sibling above it. Any gap in `contextExhaustionKills` coverage for fresh-respawn (wedge/AUP) is a PRE-EXISTING property of the UUID-save guard, not introduced by G3 — the two siblings are now consistent. Both sides of the new boundary are unit-tested (`decide` level + `applyBindingCleanupOnKill` wiring: binding SURVIVES, recovery's `getSessionForTopic` still resolves, no audit, no counterfactual). 33 G3 tests green, typecheck clean. The reviewer's §5 narrowing is applied above.
|