@bongos/core 1.19.917 → 1.19.919
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/.bongos-core.json +61 -31
- package/clients/bongos-client/README.md +1 -1
- package/clients/bongos-client/bongos-client.global.js +6 -0
- package/clients/bongos-client/index.cjs +6 -0
- package/clients/bongos-client/index.d.ts +8 -0
- package/clients/bongos-client/index.mjs +6 -0
- package/docs/adr/0314-module-protection-is-per-path-not-per-module.md +10 -5
- package/docs/adr/0318-listing-a-file-for-task-affinity-is-not-owning-it.md +101 -0
- package/docs/adr/0319-a-removal-verdict-is-a-row-beside-the-question.md +69 -0
- package/docs/adr/README.md +2 -0
- package/docs/api/openapi.json +134 -3
- package/docs/api-reference.md +8 -1
- package/docs/copy-inventory.md +15 -11
- package/docs/copy-registry.json +48 -12
- package/docs/module-api-changelog.md +4 -0
- package/modules/hall-ui/public/goal-inbox.js +102 -0
- package/modules/hall-ui/public/style.css +12 -0
- package/modules/lifecycle/cascade.js +9 -1
- package/modules/lifecycle/migrations/lifecycle_009_removal_verdicts.sql +111 -0
- package/modules/lifecycle/module.json +2 -1
- package/modules/lifecycle/removal-proposals.js +259 -0
- package/modules/lifecycle/routes/removal-proposals.js +140 -0
- package/package-lock.json +2 -2
- package/package.json +1 -1
- package/scripts/hall-preview/refresh-fixtures.js +6 -0
- package/src/bongos/module-scope-map.js +14 -4
- package/src/module-api.js +1 -1
- package/tests/blockers_hall.mjs +16 -2
- package/tests/module-scope-map.mjs +100 -7
- package/tests/removal_verdict.mjs +311 -0
package/.bongos-core.json
CHANGED
|
@@ -2,22 +2,22 @@
|
|
|
2
2
|
"artifact": "bongos-core",
|
|
3
3
|
"manifest_schema": 1,
|
|
4
4
|
"generator": "scripts/gds/package-core.js",
|
|
5
|
-
"core_version": "1.19.
|
|
6
|
-
"core_contract": "1.19.
|
|
7
|
-
"source_commit": "
|
|
5
|
+
"core_version": "1.19.919",
|
|
6
|
+
"core_contract": "1.19.919",
|
|
7
|
+
"source_commit": "bc2cae9667f75f9a2df301fc776a5e0b79451325",
|
|
8
8
|
"source_ref": "HEAD",
|
|
9
|
-
"built_at": "2026-09-
|
|
9
|
+
"built_at": "2026-09-20T19:06:48.075Z",
|
|
10
10
|
"redaction": {
|
|
11
11
|
"model": "docs-redacted+functional-verbatim",
|
|
12
|
-
"docs_redacted":
|
|
12
|
+
"docs_redacted": 508,
|
|
13
13
|
"agent_docs_stubbed": 26,
|
|
14
|
-
"functional_verbatim":
|
|
14
|
+
"functional_verbatim": 2302,
|
|
15
15
|
"rules": 3,
|
|
16
16
|
"gate_literals": 3,
|
|
17
17
|
"gate": "passed"
|
|
18
18
|
},
|
|
19
|
-
"file_count":
|
|
20
|
-
"tree_sha256": "
|
|
19
|
+
"file_count": 2836,
|
|
20
|
+
"tree_sha256": "fcb4d5d17d74106fc9ba7d49143956885bb9c5dadc48c2e1a3aa39ebcc76a851",
|
|
21
21
|
"files": [
|
|
22
22
|
{
|
|
23
23
|
"path": ".claude/skills/ask-for-help/SKILL.md",
|
|
@@ -347,12 +347,12 @@
|
|
|
347
347
|
{
|
|
348
348
|
"path": "clients/bongos-client/README.md",
|
|
349
349
|
"mode": "0000644",
|
|
350
|
-
"sha256": "
|
|
350
|
+
"sha256": "56d3e08ff4e4893d4d6b30c9292f2c1b0730deb2bff99ea64d7d5df7971995c1"
|
|
351
351
|
},
|
|
352
352
|
{
|
|
353
353
|
"path": "clients/bongos-client/bongos-client.global.js",
|
|
354
354
|
"mode": "0000644",
|
|
355
|
-
"sha256": "
|
|
355
|
+
"sha256": "7cfab2d88dc9d36fc57334db3b43c0c31b3656e2bff7ebff11d0c18b12a61850"
|
|
356
356
|
},
|
|
357
357
|
{
|
|
358
358
|
"path": "clients/bongos-client/examples/hello-world.mjs",
|
|
@@ -362,17 +362,17 @@
|
|
|
362
362
|
{
|
|
363
363
|
"path": "clients/bongos-client/index.cjs",
|
|
364
364
|
"mode": "0000644",
|
|
365
|
-
"sha256": "
|
|
365
|
+
"sha256": "8a5ed1946540c3adb30ff986f1845bdc65b5a6e441698723f8c31eb8e1374ef2"
|
|
366
366
|
},
|
|
367
367
|
{
|
|
368
368
|
"path": "clients/bongos-client/index.d.ts",
|
|
369
369
|
"mode": "0000644",
|
|
370
|
-
"sha256": "
|
|
370
|
+
"sha256": "e84a082d51b6f1e3890886a087763ec155fa31d17ac697a291e557df5bce2345"
|
|
371
371
|
},
|
|
372
372
|
{
|
|
373
373
|
"path": "clients/bongos-client/index.mjs",
|
|
374
374
|
"mode": "0000644",
|
|
375
|
-
"sha256": "
|
|
375
|
+
"sha256": "7676427e266c9528793e47c3c9431c329b67f0b29edff522724f430b4c29a37f"
|
|
376
376
|
},
|
|
377
377
|
{
|
|
378
378
|
"path": "clients/bongos-client/package.json",
|
|
@@ -2062,7 +2062,7 @@
|
|
|
2062
2062
|
{
|
|
2063
2063
|
"path": "docs/adr/0314-module-protection-is-per-path-not-per-module.md",
|
|
2064
2064
|
"mode": "0000644",
|
|
2065
|
-
"sha256": "
|
|
2065
|
+
"sha256": "d408c3f7bee29bbdf5f537ec7ed4559b88fae5d8074a8d58e556dc9e65bff46d"
|
|
2066
2066
|
},
|
|
2067
2067
|
{
|
|
2068
2068
|
"path": "docs/adr/0315-a-plane-with-no-proxy-is-refused-not-given-a-no-op-ingress.md",
|
|
@@ -2079,20 +2079,30 @@
|
|
|
2079
2079
|
"mode": "0000644",
|
|
2080
2080
|
"sha256": "4dd5bbb9c14e4ba19568646ceea2d29b66c46b99555d6c236a25b7b40097756c"
|
|
2081
2081
|
},
|
|
2082
|
+
{
|
|
2083
|
+
"path": "docs/adr/0318-listing-a-file-for-task-affinity-is-not-owning-it.md",
|
|
2084
|
+
"mode": "0000644",
|
|
2085
|
+
"sha256": "d5364169236db8c8d1a7968051fba352bb368aca53b19e554f172e0c3dc90d5d"
|
|
2086
|
+
},
|
|
2087
|
+
{
|
|
2088
|
+
"path": "docs/adr/0319-a-removal-verdict-is-a-row-beside-the-question.md",
|
|
2089
|
+
"mode": "0000644",
|
|
2090
|
+
"sha256": "2fbb3e040b55162284242f545af76b1254fafbecfe042b8c792dc0dca475c4f8"
|
|
2091
|
+
},
|
|
2082
2092
|
{
|
|
2083
2093
|
"path": "docs/adr/README.md",
|
|
2084
2094
|
"mode": "0000644",
|
|
2085
|
-
"sha256": "
|
|
2095
|
+
"sha256": "087b5151c182476c1f541e239776099319ad7fa95e6ebc2dc5ccf2b320c84f0c"
|
|
2086
2096
|
},
|
|
2087
2097
|
{
|
|
2088
2098
|
"path": "docs/api-reference.md",
|
|
2089
2099
|
"mode": "0000644",
|
|
2090
|
-
"sha256": "
|
|
2100
|
+
"sha256": "8c709c0a86391ef42df7c594087d8ebe77bb26c19b8f92f95a21c19da30de87e"
|
|
2091
2101
|
},
|
|
2092
2102
|
{
|
|
2093
2103
|
"path": "docs/api/openapi.json",
|
|
2094
2104
|
"mode": "0000644",
|
|
2095
|
-
"sha256": "
|
|
2105
|
+
"sha256": "6e2d8fc4d48d267ee7b16a81f48651c6d19dfa55716d7736c92e5cb734cc6b4f"
|
|
2096
2106
|
},
|
|
2097
2107
|
{
|
|
2098
2108
|
"path": "docs/architecture.md",
|
|
@@ -2117,12 +2127,12 @@
|
|
|
2117
2127
|
{
|
|
2118
2128
|
"path": "docs/copy-inventory.md",
|
|
2119
2129
|
"mode": "0000644",
|
|
2120
|
-
"sha256": "
|
|
2130
|
+
"sha256": "4f848ba8e139ff864151d71e4f743f799b23cf1d6a59e3638aae13ed45bce4d6"
|
|
2121
2131
|
},
|
|
2122
2132
|
{
|
|
2123
2133
|
"path": "docs/copy-registry.json",
|
|
2124
2134
|
"mode": "0000644",
|
|
2125
|
-
"sha256": "
|
|
2135
|
+
"sha256": "dc177418430b6f87b508867d35d8ef7e821d113d0ede598462a690402d699f46"
|
|
2126
2136
|
},
|
|
2127
2137
|
{
|
|
2128
2138
|
"path": "docs/design/apex-pass-2-direction.md",
|
|
@@ -2652,7 +2662,7 @@
|
|
|
2652
2662
|
{
|
|
2653
2663
|
"path": "docs/module-api-changelog.md",
|
|
2654
2664
|
"mode": "0000644",
|
|
2655
|
-
"sha256": "
|
|
2665
|
+
"sha256": "e15b7bc78c0b64fc27725df643ec05f20e047322623ac182f3614fea341521bb"
|
|
2656
2666
|
},
|
|
2657
2667
|
{
|
|
2658
2668
|
"path": "docs/modules-contract.md",
|
|
@@ -4837,7 +4847,7 @@
|
|
|
4837
4847
|
{
|
|
4838
4848
|
"path": "modules/hall-ui/public/goal-inbox.js",
|
|
4839
4849
|
"mode": "0000644",
|
|
4840
|
-
"sha256": "
|
|
4850
|
+
"sha256": "8e8d581eb99bbff050e088b727266b9f4c0ab957d1c98fd407e2c4d592a5c8b3"
|
|
4841
4851
|
},
|
|
4842
4852
|
{
|
|
4843
4853
|
"path": "modules/hall-ui/public/goals-lib.js",
|
|
@@ -5257,7 +5267,7 @@
|
|
|
5257
5267
|
{
|
|
5258
5268
|
"path": "modules/hall-ui/public/style.css",
|
|
5259
5269
|
"mode": "0000644",
|
|
5260
|
-
"sha256": "
|
|
5270
|
+
"sha256": "a7824a889c87fe40725350fd9543929e69e1370fb17023af0447f8254887d72e"
|
|
5261
5271
|
},
|
|
5262
5272
|
{
|
|
5263
5273
|
"path": "modules/hall-ui/public/task-detail.js",
|
|
@@ -5502,7 +5512,7 @@
|
|
|
5502
5512
|
{
|
|
5503
5513
|
"path": "modules/lifecycle/cascade.js",
|
|
5504
5514
|
"mode": "0000644",
|
|
5505
|
-
"sha256": "
|
|
5515
|
+
"sha256": "fd8aeec221eede847aa3b0047c01721ac7e2076e95675a71a117eff3bf3bbc55"
|
|
5506
5516
|
},
|
|
5507
5517
|
{
|
|
5508
5518
|
"path": "modules/lifecycle/check-rollup.js",
|
|
@@ -5739,6 +5749,11 @@
|
|
|
5739
5749
|
"mode": "0000644",
|
|
5740
5750
|
"sha256": "4e4099764712b6ac8ad322cb846eb1e4b1118e936d03628e01a6a04adb4d4ebf"
|
|
5741
5751
|
},
|
|
5752
|
+
{
|
|
5753
|
+
"path": "modules/lifecycle/migrations/lifecycle_009_removal_verdicts.sql",
|
|
5754
|
+
"mode": "0000644",
|
|
5755
|
+
"sha256": "e98efbb1537f0d99adb6b6b66c1fdccfd4bbac16b97362d479af903eb9de492c"
|
|
5756
|
+
},
|
|
5742
5757
|
{
|
|
5743
5758
|
"path": "modules/lifecycle/mingle.js",
|
|
5744
5759
|
"mode": "0000644",
|
|
@@ -5747,7 +5762,7 @@
|
|
|
5747
5762
|
{
|
|
5748
5763
|
"path": "modules/lifecycle/module.json",
|
|
5749
5764
|
"mode": "0000644",
|
|
5750
|
-
"sha256": "
|
|
5765
|
+
"sha256": "01f96d398408491bc4f1d9357c23c259f16e52184569849a0caf775f5b38836b"
|
|
5751
5766
|
},
|
|
5752
5767
|
{
|
|
5753
5768
|
"path": "modules/lifecycle/pg-constraint-fail.js",
|
|
@@ -5774,6 +5789,11 @@
|
|
|
5774
5789
|
"mode": "0000644",
|
|
5775
5790
|
"sha256": "ef63843d5d00230a90063c454a5ad7a9dc170ce26cedc99377807ae1ced3f7c0"
|
|
5776
5791
|
},
|
|
5792
|
+
{
|
|
5793
|
+
"path": "modules/lifecycle/removal-proposals.js",
|
|
5794
|
+
"mode": "0000644",
|
|
5795
|
+
"sha256": "8270ce55255998a4f0bbf574a54df0ba2597013c727b4afb1ce2abad9c9d4d2a"
|
|
5796
|
+
},
|
|
5777
5797
|
{
|
|
5778
5798
|
"path": "modules/lifecycle/rot.js",
|
|
5779
5799
|
"mode": "0000644",
|
|
@@ -5859,6 +5879,11 @@
|
|
|
5859
5879
|
"mode": "0000644",
|
|
5860
5880
|
"sha256": "a8e851ca630a24b197e7ecb8f8cd8ac3d169801bacbac72097528500f95d6cdb"
|
|
5861
5881
|
},
|
|
5882
|
+
{
|
|
5883
|
+
"path": "modules/lifecycle/routes/removal-proposals.js",
|
|
5884
|
+
"mode": "0000644",
|
|
5885
|
+
"sha256": "329f130d3556dc05981ce6b4b6ce4d55984c6ca42fd0f421fd70cba63eb277ac"
|
|
5886
|
+
},
|
|
5862
5887
|
{
|
|
5863
5888
|
"path": "modules/lifecycle/routes/rot.js",
|
|
5864
5889
|
"mode": "0000644",
|
|
@@ -7847,12 +7872,12 @@
|
|
|
7847
7872
|
{
|
|
7848
7873
|
"path": "package-lock.json",
|
|
7849
7874
|
"mode": "0000644",
|
|
7850
|
-
"sha256": "
|
|
7875
|
+
"sha256": "c74ec04aa7a18c9c94e42afecaac34a114331a3769e1df4af00e192555155f66"
|
|
7851
7876
|
},
|
|
7852
7877
|
{
|
|
7853
7878
|
"path": "package.json",
|
|
7854
7879
|
"mode": "0000644",
|
|
7855
|
-
"sha256": "
|
|
7880
|
+
"sha256": "fc839d817a279254882dd33532bbbb2831174de15f28b3c72a3830021e8ded5f"
|
|
7856
7881
|
},
|
|
7857
7882
|
{
|
|
7858
7883
|
"path": "public-docs/index.html",
|
|
@@ -9447,7 +9472,7 @@
|
|
|
9447
9472
|
{
|
|
9448
9473
|
"path": "scripts/hall-preview/refresh-fixtures.js",
|
|
9449
9474
|
"mode": "0000644",
|
|
9450
|
-
"sha256": "
|
|
9475
|
+
"sha256": "3974f78c40edde5fd07b2e14dcb0c8ceb45e7b0bd510cafde3a0670b41421586"
|
|
9451
9476
|
},
|
|
9452
9477
|
{
|
|
9453
9478
|
"path": "scripts/hall-preview/server.js",
|
|
@@ -9662,7 +9687,7 @@
|
|
|
9662
9687
|
{
|
|
9663
9688
|
"path": "src/bongos/module-scope-map.js",
|
|
9664
9689
|
"mode": "0000644",
|
|
9665
|
-
"sha256": "
|
|
9690
|
+
"sha256": "40f2196830c746ed440ba91b938bece8ba48fdf225285c6a6d5dccd109d47d3d"
|
|
9666
9691
|
},
|
|
9667
9692
|
{
|
|
9668
9693
|
"path": "src/bongos/module-submissions.js",
|
|
@@ -9852,7 +9877,7 @@
|
|
|
9852
9877
|
{
|
|
9853
9878
|
"path": "src/module-api.js",
|
|
9854
9879
|
"mode": "0000644",
|
|
9855
|
-
"sha256": "
|
|
9880
|
+
"sha256": "e5fb0cc646c7821da2c99ceb43ee9a9214399fb81b1ee26e1c9bc84ff5bff380"
|
|
9856
9881
|
},
|
|
9857
9882
|
{
|
|
9858
9883
|
"path": "src/module-loader/catalog.js",
|
|
@@ -10372,7 +10397,7 @@
|
|
|
10372
10397
|
{
|
|
10373
10398
|
"path": "tests/blockers_hall.mjs",
|
|
10374
10399
|
"mode": "0000644",
|
|
10375
|
-
"sha256": "
|
|
10400
|
+
"sha256": "b071e16e40b5acfeab02eb756c56ddba608d07068072b7f767737a4dcd588b68"
|
|
10376
10401
|
},
|
|
10377
10402
|
{
|
|
10378
10403
|
"path": "tests/blockers_kit_pager.mjs",
|
|
@@ -12357,7 +12382,7 @@
|
|
|
12357
12382
|
{
|
|
12358
12383
|
"path": "tests/module-scope-map.mjs",
|
|
12359
12384
|
"mode": "0000644",
|
|
12360
|
-
"sha256": "
|
|
12385
|
+
"sha256": "0a6878a91c21b5e8c31f0c06518177969f907cb0a980733c2ec1a89f8621bcb8"
|
|
12361
12386
|
},
|
|
12362
12387
|
{
|
|
12363
12388
|
"path": "tests/module_api.mjs",
|
|
@@ -13124,6 +13149,11 @@
|
|
|
13124
13149
|
"mode": "0000644",
|
|
13125
13150
|
"sha256": "b447287dca1ffe65a926715e0a5f07c0a05e3503066febf85b7f6d317b805c8f"
|
|
13126
13151
|
},
|
|
13152
|
+
{
|
|
13153
|
+
"path": "tests/removal_verdict.mjs",
|
|
13154
|
+
"mode": "0000644",
|
|
13155
|
+
"sha256": "16017d19192929aea21956a6cf5e52fa1767e2c12a7e37c5453c17b9e83b8368"
|
|
13156
|
+
},
|
|
13127
13157
|
{
|
|
13128
13158
|
"path": "tests/rename_history_restraint.mjs",
|
|
13129
13159
|
"mode": "0000644",
|
|
@@ -5,7 +5,7 @@ A **generated**, zero-dependency typed client for the Bongos API — produced fr
|
|
|
5
5
|
by hand; it regenerates when the spec changes, so it can never drift from the routes.
|
|
6
6
|
|
|
7
7
|
- API version: **v1** (served at `/api/bongos/v1`)
|
|
8
|
-
-
|
|
8
|
+
- 403 operations across 61 resource groups
|
|
9
9
|
|
|
10
10
|
## Use it from your project
|
|
11
11
|
|
|
@@ -802,6 +802,12 @@ function createClient(opts = {}) {
|
|
|
802
802
|
// GET /public/versions — rank: public — GET /public/versions
|
|
803
803
|
getPublicVersions: (args) => request("GET", "/public/versions", { hasBody: false }, args),
|
|
804
804
|
},
|
|
805
|
+
"removalProposals": {
|
|
806
|
+
// GET /removal-proposals — rank: metic+archon — GET /removal-proposals
|
|
807
|
+
getRemovalProposals: (args) => request("GET", "/removal-proposals", { hasBody: false }, args),
|
|
808
|
+
// POST /removal-proposals/{id}/decide — rank: metic+archon — POST /removal-proposals/:id/decide
|
|
809
|
+
postRemovalProposalsIdDecide: (args) => request("POST", "/removal-proposals/{id}/decide", { hasBody: true }, args),
|
|
810
|
+
},
|
|
805
811
|
"scouting": {
|
|
806
812
|
// GET /scouting — rank: metic+archon — GET /scouting
|
|
807
813
|
getScouting: (args) => request("GET", "/scouting", { hasBody: false }, args),
|
|
@@ -801,6 +801,12 @@ function createClient(opts = {}) {
|
|
|
801
801
|
// GET /public/versions — rank: public — GET /public/versions
|
|
802
802
|
getPublicVersions: (args) => request("GET", "/public/versions", { hasBody: false }, args),
|
|
803
803
|
},
|
|
804
|
+
"removalProposals": {
|
|
805
|
+
// GET /removal-proposals — rank: metic+archon — GET /removal-proposals
|
|
806
|
+
getRemovalProposals: (args) => request("GET", "/removal-proposals", { hasBody: false }, args),
|
|
807
|
+
// POST /removal-proposals/{id}/decide — rank: metic+archon — POST /removal-proposals/:id/decide
|
|
808
|
+
postRemovalProposalsIdDecide: (args) => request("POST", "/removal-proposals/{id}/decide", { hasBody: true }, args),
|
|
809
|
+
},
|
|
804
810
|
"scouting": {
|
|
805
811
|
// GET /scouting — rank: metic+archon — GET /scouting
|
|
806
812
|
getScouting: (args) => request("GET", "/scouting", { hasBody: false }, args),
|
|
@@ -128,6 +128,7 @@ export interface GetPublicRepoInfoResponse { blob_base: unknown; tree_base: unkn
|
|
|
128
128
|
export interface GetPublicTasksIdResponse { task: unknown }
|
|
129
129
|
export interface GetPublicUptimeResponse { services: unknown }
|
|
130
130
|
export interface GetPublicVersionsResponse { versions: unknown; page: PageInfo }
|
|
131
|
+
export interface GetRemovalProposalsResponse { pending: unknown; count: unknown }
|
|
131
132
|
export interface GetScoutingHandleResponse { sliver: unknown }
|
|
132
133
|
export interface GetScoutingResponse { builders: unknown }
|
|
133
134
|
export interface GetSecurityDocsResponse { docs: unknown }
|
|
@@ -372,6 +373,7 @@ export interface PostProvisioningInstancesIdGithubAppResponse { state: unknown;
|
|
|
372
373
|
export interface PostProvisioningInstancesIdRestartResponse { ok: boolean; queued: unknown; action: unknown; status: unknown; message: unknown }
|
|
373
374
|
export interface PostProvisioningInstancesIdTeardownResponse { ok: boolean; queued: unknown; action: unknown; status: unknown; superseded: unknown; message: unknown }
|
|
374
375
|
export interface PostProvisioningInstancesRequest { slug: string; target_ref?: string; hosting_shape?: string; domain?: string; no_address?: boolean; tier?: string; onboard_mode?: string; type?: string; description?: string; detail?: Record<string, unknown>; modules?: string[] }
|
|
376
|
+
export interface PostRemovalProposalsIdDecideRequest { verdict: string; note?: string }
|
|
375
377
|
export interface PostSearchMarkRequest { chunk_id?: StringifiableId; useful?: unknown }
|
|
376
378
|
export interface PostSearchRequest { expand?: unknown; q?: string; source_kind?: string; limit?: unknown }
|
|
377
379
|
export interface PostSearchResponse { expand?: unknown; chunk?: unknown; query?: unknown; count?: unknown; total_full_tokens?: unknown; results?: unknown; corpus?: unknown }
|
|
@@ -1175,6 +1177,12 @@ export interface BongosClient {
|
|
|
1175
1177
|
/** GET /public/versions — rank: public */
|
|
1176
1178
|
getPublicVersions(args?: RequestArgs): Promise<GetPublicVersionsResponse>;
|
|
1177
1179
|
};
|
|
1180
|
+
"removalProposals": {
|
|
1181
|
+
/** GET /removal-proposals — rank: metic+archon */
|
|
1182
|
+
getRemovalProposals(args?: RequestArgs): Promise<GetRemovalProposalsResponse>;
|
|
1183
|
+
/** POST /removal-proposals/{id}/decide — rank: metic+archon */
|
|
1184
|
+
postRemovalProposalsIdDecide(args: RequestArgs & { body: PostRemovalProposalsIdDecideRequest }): Promise<ApiResponse>;
|
|
1185
|
+
};
|
|
1178
1186
|
"scouting": {
|
|
1179
1187
|
/** GET /scouting — rank: metic+archon */
|
|
1180
1188
|
getScouting(args?: RequestArgs): Promise<GetScoutingResponse>;
|
|
@@ -798,6 +798,12 @@ export function createClient(opts = {}) {
|
|
|
798
798
|
// GET /public/versions — rank: public — GET /public/versions
|
|
799
799
|
getPublicVersions: (args) => request("GET", "/public/versions", { hasBody: false }, args),
|
|
800
800
|
},
|
|
801
|
+
"removalProposals": {
|
|
802
|
+
// GET /removal-proposals — rank: metic+archon — GET /removal-proposals
|
|
803
|
+
getRemovalProposals: (args) => request("GET", "/removal-proposals", { hasBody: false }, args),
|
|
804
|
+
// POST /removal-proposals/{id}/decide — rank: metic+archon — POST /removal-proposals/:id/decide
|
|
805
|
+
postRemovalProposalsIdDecide: (args) => request("POST", "/removal-proposals/{id}/decide", { hasBody: true }, args),
|
|
806
|
+
},
|
|
801
807
|
"scouting": {
|
|
802
808
|
// GET /scouting — rank: metic+archon — GET /scouting
|
|
803
809
|
getScouting: (args) => request("GET", "/scouting", { hasBody: false }, args),
|
|
@@ -111,11 +111,16 @@ new derivation mechanism. Filed separately:
|
|
|
111
111
|
structural path test — directories under `scripts/` are territory and still
|
|
112
112
|
protect), narrowing the derived set from twelve to the nine predicted below.
|
|
113
113
|
Nothing was unprotected to achieve it and the path wall did not move.
|
|
114
|
-
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
118
|
-
|
|
114
|
+
- ~~**`builder-settings`, deliberately out of scope.**~~ **Resolved** by
|
|
115
|
+
[ADR 0318](0318-listing-a-file-for-task-affinity-is-not-owning-it.md)
|
|
116
|
+
([task 1004110](https://cloudbongos.com/builders#/task/1004110)), and by neither
|
|
117
|
+
of the two routes predicted here. Its overlap was `src/bongos/routes/me.js`
|
|
118
|
+
caught by the blanket `src/bongos/routes/` prefix — but that glob was a task
|
|
119
|
+
AFFINITY listing, not an ownership claim, on a file `MODULE_GLOBS` itself
|
|
120
|
+
called a core facade. Ownership moved to `kernel`, where the aggregator
|
|
121
|
+
belongs; the blanket glob was left alone (all eighteen route files are
|
|
122
|
+
protected by it and nothing else) and no strangler move was needed (the
|
|
123
|
+
aggregator resolves ten module ports). The derived set narrows nine to eight.
|
|
119
124
|
- **Nine protected paths are owned by no module at all** — `gen-repo-map.js`,
|
|
120
125
|
`gen-file-map.js`, `gen-session-index.js`, `install-git-hooks.js`, `cli-lib.js`,
|
|
121
126
|
`run-unit-tests.js`, `git-merge-regen.js`, `scripts/gds/hooks/` and
|
|
@@ -0,0 +1,101 @@
|
|
|
1
|
+
# ADR 0318 — Listing a file for task affinity is not claiming to own it
|
|
2
|
+
|
|
3
|
+
- **Status:** accepted — **implemented** by [task 1004110](https://cloudbongos.com/builders#/task/1004110) (2026-09-20)
|
|
4
|
+
- **Date:** 2026-09-20
|
|
5
|
+
- **Task:** [task 1004110](https://cloudbongos.com/builders#/task/1004110), goal 1000095 — BONGOS-V2
|
|
6
|
+
- **Extends** [ADR 0314](0314-module-protection-is-per-path-not-per-module.md), which left this case open by name.
|
|
7
|
+
- **Supersedes nothing.**
|
|
8
|
+
|
|
9
|
+
## The question
|
|
10
|
+
|
|
11
|
+
After [ADR 0314](0314-module-protection-is-per-path-not-per-module.md), nine
|
|
12
|
+
modules derive `protected` — making a goal scoped to them Archon-only to create.
|
|
13
|
+
`builder-settings` was one of them, and it earned that on exactly one glob:
|
|
14
|
+
`src/bongos/routes/me.js`, caught by the blanket `src/bongos/routes/` prefix in
|
|
15
|
+
`PROTECTED_GLOBS`.
|
|
16
|
+
|
|
17
|
+
ADR 0314 named this and left it, because clearing it looked like it needed one of
|
|
18
|
+
two expensive things: finishing the strangler move of `me.js` into
|
|
19
|
+
`modules/builder-settings/`, or narrowing a blanket protected glob. Both were
|
|
20
|
+
scoped out. The task that picked this up was written the same way — those two
|
|
21
|
+
options, and *"the choice is the deliverable"*.
|
|
22
|
+
|
|
23
|
+
Both are wrong, and measuring them is what shows a third thing was actually broken.
|
|
24
|
+
|
|
25
|
+
## What the measurements said
|
|
26
|
+
|
|
27
|
+
**Narrowing the blanket prefix removes the only wall that exists.** All eighteen
|
|
28
|
+
files under `src/bongos/routes/` are protected by that prefix *and nothing else* —
|
|
29
|
+
no route file has a second protecting glob. Narrowing it unwalls `auth.js`,
|
|
30
|
+
`builders.js`, `backup.js` and `core-update.js` in the same stroke. This is now
|
|
31
|
+
pinned as a test rather than left as a caution, so the next reader does not have
|
|
32
|
+
to re-derive it.
|
|
33
|
+
|
|
34
|
+
**Moving `me.js` into `modules/builder-settings/` inverts the port architecture.**
|
|
35
|
+
`GET /me` is a deliberate aggregator: it resolves **ten** module ports —
|
|
36
|
+
`lifecycle`, `reward`, `onboarding`, `government`, `specialities`, `grade`,
|
|
37
|
+
`box.hasEverConnected`, `art.sharedKeyConfigured`, `art.getBuilderGeminiKeyRow`,
|
|
38
|
+
`discord.isLinked` — and `builder-settings` is one of the ten. Hosting it inside
|
|
39
|
+
one module would mean that module resolving nine foreign ports from the inside,
|
|
40
|
+
which is the `core -> module` edge [ADR 0083](0083-modular-architecture-strangler-build-plan.md)
|
|
41
|
+
rule 1 exists to forbid. A core facade is precisely what prevents that.
|
|
42
|
+
|
|
43
|
+
Three places already said so, in the code, before this task was written: the file
|
|
44
|
+
header (*"This file kept only the read aggregation"*), `module.json` (*"the
|
|
45
|
+
`GET /me` aggregator stays a thin core facade"*), and `MODULE_GLOBS` itself.
|
|
46
|
+
|
|
47
|
+
## The decision
|
|
48
|
+
|
|
49
|
+
**`MODULE_GLOBS` listed `src/bongos/routes/me.js` under `builder-settings` for
|
|
50
|
+
task affinity — and said so in the same comment — while `isProtected()` read it
|
|
51
|
+
as an ownership claim.** Affinity is a convenience for authoring; ownership is a
|
|
52
|
+
statement about territory. One listing, read as the other, made a whole module
|
|
53
|
+
Archon-only on the strength of a file its own comment called core.
|
|
54
|
+
|
|
55
|
+
So: **ownership follows what the file is.** `src/bongos/routes/me.js` moves to
|
|
56
|
+
`kernel`, which already owns `src/bongos/routes/security.js` and is the home of
|
|
57
|
+
the core route surface. `builder-settings` keeps its own directory and its
|
|
58
|
+
pref-mirroring CLI, and stops being protected.
|
|
59
|
+
|
|
60
|
+
This is [ADR 0314](0314-module-protection-is-per-path-not-per-module.md) one step
|
|
61
|
+
on. That decision said *owning the command is not owning the authority*, for a CLI
|
|
62
|
+
entrypoint that merely lives near a module. The same error wearing different
|
|
63
|
+
clothes: **being listed next to a module is not owning it either.** 0314 solved
|
|
64
|
+
its case structurally, with a path test (`isCommandPath`). This one is solved by
|
|
65
|
+
correcting the data, because there is exactly one such listing — see below.
|
|
66
|
+
|
|
67
|
+
## Why not a general affinity tier
|
|
68
|
+
|
|
69
|
+
The obvious generalisation is to mark affinity entries in `MODULE_GLOBS` so
|
|
70
|
+
`isProtected()` skips them, mirroring `isCommandPath`. Rejected, because the
|
|
71
|
+
audit does not support it. Of the nine protected modules, three others
|
|
72
|
+
(`kernel`, `devsecops`, `methodology`) are protected only by paths outside their
|
|
73
|
+
own directory — but in every one of those the foreign path genuinely *is* the
|
|
74
|
+
domain: `kernel` **is** `src/bongos/auth.js`, `devsecops` **is** `.github/`,
|
|
75
|
+
`methodology` **is** `CLAUDE.md` and `docs/`.
|
|
76
|
+
|
|
77
|
+
`builder-settings` was the only module protected by a path its own source
|
|
78
|
+
described as belonging elsewhere. A new tier in `MODULE_GLOBS` would be a new
|
|
79
|
+
concept, and a new place for a hole to hide, built for a population of one.
|
|
80
|
+
|
|
81
|
+
## Consequences
|
|
82
|
+
|
|
83
|
+
- `protectedModules()` goes **9 -> 8**. `builder-settings` leaves the set.
|
|
84
|
+
- **The wall that matters does not move.** This decides who may CREATE a goal
|
|
85
|
+
whose territory includes `/me`. It does not decide who may WRITE it: a
|
|
86
|
+
sub-Metic builder touching `src/bongos/routes/me.js` is still refused by
|
|
87
|
+
`checkPermissionPaths`, which the tests assert directly rather than assume.
|
|
88
|
+
- A `builder-settings`-scoped goal can no longer author a task touching `/me`;
|
|
89
|
+
that needs `kernel` scope. Correct, rather than a loss — the aggregator spans
|
|
90
|
+
ten modules and belongs to none of them.
|
|
91
|
+
- `src/bongos/routes/` now has **two** of eighteen files owned, up from two with
|
|
92
|
+
one of them wrong. The other sixteen remain authorable by no goal;
|
|
93
|
+
[task 1004116](https://cloudbongos.com/builders#/task/1004116) holds that
|
|
94
|
+
decision, and this change removes the builder-settings overlap that made its
|
|
95
|
+
option (a) awkward.
|
|
96
|
+
|
|
97
|
+
## Not built here
|
|
98
|
+
|
|
99
|
+
- **The other sixteen route files.** Still unowned, still
|
|
100
|
+
[task 1004116](https://cloudbongos.com/builders#/task/1004116). The blanket-prefix
|
|
101
|
+
tripwire added here is the measurement that task needs, landed early.
|
|
@@ -0,0 +1,69 @@
|
|
|
1
|
+
# ADR 0319 — A removal verdict is a row beside the question, because both answers close it the same way
|
|
2
|
+
|
|
3
|
+
**Status:** accepted · **Date:** 2026-09-20 · **Task:** [task 1004095](https://cloudbongos.com/builders#/task/1004095) (goal 1000095 — BONGOS-V2)
|
|
4
|
+
|
|
5
|
+
**Related:** [ADR 0313](0313-a-removal-is-proposed-by-the-ship-and-confirmed-by-a-person.md) (the decision of record — the system proposes, a person confirms) · [ADR 0242](0242-a-cascade-is-a-declaration-table-on-the-event-that-already-exists.md) (the runner a rule is appended to, and may not fan out) · [ADR 0299](0299-the-nod-queue-stays-scoped-to-routed-filings.md) (why the nod queue is not widened) · [ADR 0312](0312-provenance-is-a-second-ideas-owned-edge-table.md) (the descent a confirmed removal dims)
|
|
6
|
+
|
|
7
|
+
## Context
|
|
8
|
+
|
|
9
|
+
[ADR 0313](0313-a-removal-is-proposed-by-the-ship-and-confirmed-by-a-person.md) settled that a ship which deletes an earlier shipped task's files files a **question**, and that a person answers it: Metic+, `task.promote`, on the existing awaiting-nod surface, with an unanswered proposal rotting on the normal clock. Task 1004090 built the question. Nothing could answer it.
|
|
10
|
+
|
|
11
|
+
Answering turns out to have one property that decides the whole design: **both answers close the proposal the same way.** The proposal is a question, not work — confirming leaves nothing to build, and so does dismissing. Either way the only sensible end state for that task row is `abandoned`, the ledger's one spelling of "finished with the project, not delivered" (`modules/lifecycle/task-status.js`).
|
|
12
|
+
|
|
13
|
+
So a status alone cannot be the record. "Yes, that work is gone" and "no, that was a refactor" would be the same row, and brightness — the live share of a descent, the next task in this chain — would have no way to tell the star it must dim from the one it must not. ADR 0313 already ruled out the obvious alternative: a `removed` task status, because `shipped` is terminal by design and a removal is "a fact about work's fate, recorded beside it, not a new terminal state that ship rollup would have to learn."
|
|
14
|
+
|
|
15
|
+
## Decision
|
|
16
|
+
|
|
17
|
+
### The verdict is its own row, and there is exactly one of them per proposal
|
|
18
|
+
|
|
19
|
+
`lifecycle_removal_verdicts` (migration `lifecycle_009`) holds `proposal_task_id` (**primary key**), `ship_task_id`, `verdict ∈ {confirmed, dismissed}`, `decided_by`, `decided_at`, `note`. It is the "beside it" ADR 0313 asked for. The verdict write and the status change commit in **one transaction**: a verdict with no closed task would be re-answerable, and a closed task with no verdict is precisely the erasure this exists to prevent.
|
|
20
|
+
|
|
21
|
+
The primary key on `proposal_task_id` is a decision, not a default. A second answer is **refused** (`ALREADY_DECIDED`, 409), never upserted and never treated as an idempotent success, because overwriting destroys the only record that a person answered differently. The proposal row is locked `FOR UPDATE` before any of the row-scoped checks, so two people pressing Confirm and Dismiss at the same moment cannot both read "undecided"; the second loses on the refusal rather than winning on write order. Amending a verdict is deliberately not built: if it is ever needed it is a second row and an append-only history, not an UPDATE.
|
|
22
|
+
|
|
23
|
+
`ALREADY_DECIDED` is checked **before** `PROPOSAL_CLOSED`, because an answered proposal is also abandoned — checking status first would tell someone "this is not an open question" when the useful answer is "it was dismissed, by builder 7, yesterday".
|
|
24
|
+
|
|
25
|
+
### The verdict is what this ships; the target list is not
|
|
26
|
+
|
|
27
|
+
The proposal names the affected tasks and paths as **prose in its body**, because a cascade files one task per rule per ship with no field for a target and ADR 0242 forbids widening the runner to fan out (owner decision, 2026-09-20, recorded in `cascade.js:removalProposalBody`). This ADR does not change that and does not parse it. A body is written for a person to read; a reader that silently depends on its line shape breaks the day somebody improves the wording.
|
|
28
|
+
|
|
29
|
+
What brightness gets from here is structured: the verdict, and `ship_task_id` — denormalised out of the proposal's `source_ref`, which encodes it as `cascade-removal-proposed-on-ship-<id>`, a string nothing can join on (the same complaint `ideas_007` makes about the same field). Brightness designs its own storage for targets when it needs one, which is where the owner put that decision.
|
|
30
|
+
|
|
31
|
+
A ref that cannot be parsed still yields an answerable proposal, with `ship_task_id` NULL. The verdict is the point; refusing to record one because a convenience column could not be filled would be the tail wagging the dog.
|
|
32
|
+
|
|
33
|
+
### The proposals get their own slice on the attention card, not a row in the nod queue
|
|
34
|
+
|
|
35
|
+
Spec decision 15 says "the existing awaiting-nod surface". The **surface** is the hall's one Needs-your-attention card; the **queue** is `listRoutedAwaitingNod`, which filters `origin = 'route'` on `ideas_task_lineage` and which [ADR 0299](0299-the-nod-queue-stays-scoped-to-routed-filings.md) decided — on live numbers — stays scoped to routed filings and is honestly named for that population. A removal proposal has no idea lineage at all, so widening that predicate to reach it would both break ADR 0299 and make an ideas-owned table answer a lifecycle question.
|
|
36
|
+
|
|
37
|
+
So: same card, same floor, separate slice (`#attention-removals`, straight after the nod), fed by its own routes.
|
|
38
|
+
|
|
39
|
+
### The permission is reused; the verb is not
|
|
40
|
+
|
|
41
|
+
`GET /removal-proposals` and `POST /removal-proposals/:id/decide` both gate on the **existing** `task.promote` atom — Metic+, the same floor as the nod, so a caller who can see the queue can act on it by construction rather than by coincidence. No new permission and no new page, as spec decision 15 requires.
|
|
42
|
+
|
|
43
|
+
A new *route* is nonetheless needed, and that is not a contradiction. The nod could reuse `POST /tasks/:id/promote` because promoting is exactly what a nod does; there is no existing route that means "that work is genuinely gone". `/tasks/:id/abandon` would close the question without recording which answer was given — the one thing this feature exists to produce.
|
|
44
|
+
|
|
45
|
+
The path is top-level (`/removal-proposals`, the `/override-requests` precedent) rather than `/tasks/removal-proposals`, because the manifest mounts the `tasks` route file first and its `/tasks/:id` would swallow a literal single segment declared in a later file — the trap `modules/ideas/routes/inbox.js` documents for `/inbox/awaiting-nod`, avoided here by not sharing the prefix.
|
|
46
|
+
|
|
47
|
+
### Rot is inherited, not rebuilt
|
|
48
|
+
|
|
49
|
+
An unanswered proposal sits at `backlog`, which is in `rot.TASK_ROTTABLE_STATUSES`, so it enters the existing rot feed on the project's clock with no code written for it — spec decision 14, and why that decision had no competing option. Answering it writes `abandoned`, which is terminal and not rottable, so it leaves the feed by the same mechanism. Both halves are pinned against the rot module's own exported vocabulary rather than asserted in a comment.
|
|
50
|
+
|
|
51
|
+
## Consequences
|
|
52
|
+
|
|
53
|
+
**A confirmed removal still changes nothing else.** No status on the removed work moves, no credit is clawed back, `shipped` stays terminal. The verdict is a fact recorded beside the work, and brightness is the only thing that will read it.
|
|
54
|
+
|
|
55
|
+
**A cascade task created before this shipped is answerable.** The predicate is `source = 'cascade'` plus the removal rule's ref prefix, both read from `cascade.js` rather than re-typed, so any proposal the rule has ever filed qualifies. In practice the set is empty: the rule could not fire in production until task 1004090's final commit.
|
|
56
|
+
|
|
57
|
+
**The rot feed shows these only to the owner.** A cascade-filed task has `created_by = NULL`, and `rottingFor` addresses a null-creator row solely to a caller holding `builder.roster.read` — the core_227 convention, deliberate rather than accidental, since there is no person a system-filed row could honestly be addressed to. The attention slice is not scoped that way: it shows every open proposal to every Metic+.
|
|
58
|
+
|
|
59
|
+
**A verdict cannot be corrected through the API.** Stated plainly because it will be noticed: the recovery for a mis-answered proposal is currently a row deletion by someone with SQL, and the right fix — should it ever be asked for — is an append-only amendment, not an UPDATE on this table.
|
|
60
|
+
|
|
61
|
+
## Alternatives rejected
|
|
62
|
+
|
|
63
|
+
- **The status alone as the record.** Both answers close the same way, so it records neither.
|
|
64
|
+
- **A `removed` task status, or reusing `merged_into_task_id`.** ADR 0313 rejected the first; the second is CHECK-constrained to `abandoned` rows and describes work cut *before* it landed.
|
|
65
|
+
- **Parsing the affected tasks out of the proposal body at confirm time.** Couples a machine read to prose written for a person, and the owner already placed that storage decision with brightness.
|
|
66
|
+
- **Widening the nod queue to carry removal proposals.** Breaks ADR 0299 and makes an ideas-owned lineage table answer a lifecycle question.
|
|
67
|
+
- **Two routes, `/confirm` and `/dismiss`.** The same act with opposite content; splitting it invites the halves to drift in what they close, record and refuse.
|
|
68
|
+
- **Upserting a second verdict, or treating it as an idempotent 200.** Destroys the record that a person answered differently.
|
|
69
|
+
- **A `LIKE 'prefix%'` match on `source_ref`.** A rule key may contain an underscore, and an underscore is a LIKE wildcard — a silent widening of the read that decides what may be answered.
|
package/docs/adr/README.md
CHANGED
|
@@ -409,3 +409,5 @@ This keeps the decision history honest and traceable.
|
|
|
409
409
|
| 0316 | [**The ignite floor is a scheduling dial, not the trusted tier** ([task 1004084](https://cloudbongos.com/builders#/task/1004084), goal 1000095 — BONGOS-V2). BV2 decision 3 makes the ignite floor — who may turn a quick idea straight into claimable work — a per-instance setting, and decision 16 adds the descent decay rate beside it, the two to share one home. Both the spec and the seeded task described the mechanism the same way: the floor IS the `METIC_PLUS` constant in `modules/ideas/routing.js`, so this is "a setting, not a refactor". Tracing the code before building refuted that: `METIC_PLUS` is spelled once but READ for four semantically different gates — the two `decideLanding` bug branches (land at `ready` or at `backlog`), `mayProposeCriterion` (who may file a `kind='criterion-proposal'`, a planning-scope write), and the protected-scope wall in `authorizeRoutedLanding` (who may route work into the permission / ship-grade / deploy / migration core). Only the first two are the ignite floor, so making the CONSTANT configurable would mean an instance setting its floor to `xenos` — exactly the "project of one wants every thought claimable" case the owner asked for — ALSO gaining the right to file criterion-proposals and to breach the protected core. The codebase already said so in three places nobody had read into the spec: the constant's own header ("It grants nothing — the server-side rank floor and the goal's scope wall are separate, load-bearing gates"), the protected-scope branch ("the two are layers, not alternatives"), and `tests/filing_abuse_matrix.mjs`, which models the widening as an ATTACK and pins the literal. Decision: a separate dial. `modules/ideas/config.js` + `config/ideas.neutral.json` carry both knobs on the `modules/government/config.js` precedence (env -> instance root `config/ideas.json` -> core root neutral; the ADR 0268 dual-root split matters, since one root reads the host pack out of `node_modules` where `npm ci` erases it). `igniteFloor` defaults to `metic`, resolving to exactly the `METIC_PLUS` set, and is threaded into `decideLanding` as a ready-made `trustedRanks` **Set** by the callers — `routing.js` is TOLD the floor and never reads config, staying pure and DB-free — while `METIC_PLUS` stays a FIXED constant for the other two gates, neither of which takes a floor argument (pinned structurally). `descentDecayRate` defaults to 0.5 and ships with no production caller, because R11 is its intended reader and depends on the unbuilt R08 — the same deliberate slice boundary `modules/government/config.js` documents for itself, and what decision 16 asked for. Everything fails to VANILLA rather than dead: a malformed pack, an unknown rank or an out-of-band rate falls back, and an unrecognised floor yields the DEFAULT set, never an empty one (failing closed would stop every ignition on a typo). Rejected: configuring `METIC_PLUS` as written (a security widening in a config file's clothes); splitting it into four named constants (the other three want the same fixed tier, and naming them invites configuring them); having `routing.js` read the config itself (makes the landing matrix impure and untestable at a different floor). Spec decision 3's "spelled in one place" describes the intent, not the mechanism; the spec was not edited and the task description was amended in flight.](0316-the-ignite-floor-is-not-the-trusted-tier.md) | ideas / permissions |
|
|
410
410
|
| 0315 | [**A plane with no proxy is refused, not given a no-op ingress** ([task 1004017](https://cloudbongos.com/builders#/task/1004017), goal 1000106 — BONGOS-V2). The piece [ADR 0306](0306-instance-supervision-is-a-seam-with-systemd-as-the-default.md) explicitly deferred: the self-host image ships no proxy, so provisioning step 6's `<slug>.caddy` snippet write and `systemctl reload caddy` cannot work there and a child instance gets no front door. **Two failure shapes, only one visible**: a DOMAINED child already half-refused, because `caddy adapt` fails with no binary and [ADR 0303](0303-provisioning-verifies-the-base-caddyfile-and-never-edits-it.md)'s own catch turns that into a gap — but the message told the operator to add a directive to `/etc/caddy/Caddyfile`, a file the container does not have and that ADR 0303 §1 forbids provisioning from creating; a DOMAINLESS child skipped that preflight entirely (it is gated on `inst.domain`), reached step 6 and died on the `sudo` write, an uncaught throw that bypasses the `{ok:false}` pending-retry path and hard-errors the instance on attempt one — a crash recorded where a refusal was the honest answer. Decision: `ingressPlaneGap(exec)` probes the plane with `caddy version` — one cheap unprivileged read, the same posture as `checkCaddyWiring` — and both standup legs refuse with `Ingress preflight failed: …`, deliberately NOT domain-gated, because the domainless child is the one 0303 could not see. **Only a missing binary proves there is no proxy**: under remote exec `boxExec` is `ssh <host> bash -s`, so an unreachable co-hosting box throws here too, and diagnosing a network blip as 'this plane has no proxy, provision elsewhere' would be a confident wrong answer that burns the intent's retry budget — exit 127 is read as proof, anything else answers 'could not confirm a reverse proxy on this plane' with the real cause leading. The three refusals are ordered **capacity → ingress → wiring**, cheapest and most fundamental first and all before any mutation, which moves 0303's preflight behind the ADR 0285 capacity measurement; both still refuse before anything changes, so only the precedence between two refusals moves. **No ingress seam and no no-op backend**, despite the obvious symmetry with 0306: a `none` implementation whose `writeSite`/`reload` are no-ops lets the standup SUCCEED while leaving the child unreachable — neither half of the task's done-when, and exactly the silently-broken instance it exists to prevent. Does not decide: what a child's front door IS on a container plane (a published port, a bundled proxy, an explicit localhost-only shape) — a future ADR that adds one replaces this refusal rather than extending it. Teardown is left alone on purpose: it already reloads `allowFail`, and refusing there would strand a row instead of cleaning it.](0315-a-plane-with-no-proxy-is-refused-not-given-a-no-op-ingress.md) | provisioning / ingress |
|
|
411
411
|
| 0317 | [**A plane that cannot give an instance its own account is refused, not given a shared uid** ([task 1004018](https://cloudbongos.com/builders#/task/1004018), goal 1000106 — BONGOS-V2). The last of the three container gaps [ADR 0306](0306-instance-supervision-is-a-seam-with-systemd-as-the-default.md) deferred, and the sibling of [ADR 0315](0315-a-plane-with-no-proxy-is-refused-not-given-a-no-op-ingress.md). [ADR 0281](0281-an-instance-identity-is-its-own-unix-account-and-pg-role.md) makes an instance's identity its own unix account, and the unit's `User=`/`Group=` is what stops one co-tenant reading a sibling's `/proc/<pid>/environ` — where every instance's `EnvironmentFile` carries its GitHub App private key and OAuth client secret (audit B3). `useradd` needs uid 0; the self-host image is `node:22-bookworm-slim` running `USER node` with **no sudo**, so there the account cannot be created at all. **Two failure shapes, neither honest**: today step 1c's `boxExec(ensureInstanceUserCmd(...))` is not `allowFail`, so it throws AFTER every preflight passed, AFTER the status flip to `provisioning`, and — for a standalone — after a repo pull and an `npm ci`, recording a crash with the box half-changed; and tomorrow it goes quiet, because a compose self-host only stops short of step 1c by ADR 0315's coincidence, and 0315 says in its own consequences that a future front door **replaces** that refusal. Decision: `accountPlaneGap(exec, { privileged })` in `provision-repo.js` probes with `${sudo}sh -c 'test "$(id -u)" = 0'` and `provisionInstance` refuses with `Account preflight failed: …` before any mutation. **The probe runs the SAME escalation the mutation will** (`PROVISION_SUDO=1` renders the same `sudo ` prefix `ensureInstanceUserCmd` uses), so a pass is evidence rather than inference. **`sh -c`, not a bare `test`, is load-bearing**: under sudo the outer unprivileged shell would expand `$(id -u)` first and answer about the wrong user, passing on exactly the plane this refuses — and for the same reason there is no pipeline, whose exit status is its last stage's. **Only a shell that ran and answered proves it**: `boxExec` is `ssh <host> bash -s` under remote exec, so exit 1 (`test` said no) and 127 (no sudo binary) are read as proof while ssh's own 255 answers "could not confirm" with the real cause leading — ADR 0315's rule applied to a different question. Ordered **third**, after capacity and ingress and before 0303's wiring check: the first three ask about the PLANE, 0303 asks about this instance's DOMAIN, and 0315's order is extended rather than rearranged. **Rejected: accepting it** — documenting a self-host container as single-tenant with a shared uid is a silent downgrade of the vector 0281 closed, observable nowhere on the running box, and "single-tenant" is not what a container hosting several child projects is; **an identity seam with a container implementation**, because nothing exists to put behind it and a seam whose second backend does nothing is 0315's no-op wearing a seam's clothes; **probing `useradd --help` / `command -v useradd`**, which succeed unprivileged on the very image this refuses; **`${sudo}id -u` compared against `0`**, which decides on captured stdout where every sibling preflight and every test fake treats exit status as the contract; and **refusing inside step 1c**, which keeps the refusal after the status flip. Not covered, deliberately: root-but-no-`useradd` (an Alpine-style plane fails loudly at step 1c as today), and teardown (0315's reason — a refusal there strands a row instead of cleaning it). No `PROVISION_ALLOW_SHARED_UID`: a runner either reaches root on this plane or it does not.](0317-a-plane-that-cannot-give-an-instance-its-own-account-is-refused.md) | provisioning / identity |
|
|
412
|
+
| 0319 | [**A removal verdict is a row beside the question, because both answers close it the same way** ([task 1004095](https://cloudbongos.com/builders#/task/1004095), goal 1000095 — BONGOS-V2). [ADR 0313](0313-a-removal-is-proposed-by-the-ship-and-confirmed-by-a-person.md) settled that a ship deleting an earlier shipped task’s files files a QUESTION and a person answers it; task 1004090 built the question and nothing could answer it. Building the answer surfaced the property that decides the design: a proposal is a question, not work, so CONFIRMING and DISMISSING both leave nothing to build and both end the task at `abandoned` — meaning a status alone records neither, and brightness could not tell the star it must dim from the one it must not. ADR 0313 had already rejected the obvious alternative (a `removed` status; `shipped` is terminal and much of the ledger leans on it — a removal is a fact recorded BESIDE work, not a new terminal state). Decision: `lifecycle_removal_verdicts` (migration `lifecycle_009`), PRIMARY KEY on `proposal_task_id` so one proposal holds one answer ever — a second decide is REFUSED (409), never upserted and never an idempotent 200, because overwriting destroys the record that a person answered differently; the row is locked `FOR UPDATE` first, so simultaneous Confirm and Dismiss cannot both read “undecided”, and `ALREADY_DECIDED` is checked BEFORE `PROPOSAL_CLOSED` because an answered proposal is also abandoned and the useful message is the one naming the earlier verdict. The verdict and the status change commit in ONE transaction. What this does NOT ship is the target list: the affected tasks and paths stay PROSE in the proposal body (the owner’s 2026-09-20 decision, since [ADR 0242](0242-a-cascade-is-a-declaration-table-on-the-event-that-already-exists.md) forbids widening the runner to fan out), and nothing here parses it — brightness designs that storage. What brightness does get is structured: the verdict plus `ship_task_id`, denormalised out of the `source_ref` that encodes it as a string nothing can join on. Surface: spec decision 15’s “existing awaiting-nod surface” is the CARD, not the QUEUE — [ADR 0299](0299-the-nod-queue-stays-scoped-to-routed-filings.md) keeps that queue scoped to routed filings and a removal proposal has no idea lineage at all, so it gets its own slice on the same card at the same floor. The PERMISSION is reused (`task.promote`, Metic+, read and write both) and the VERB is not: no existing route means “that work is genuinely gone”, and `/tasks/:id/abandon` would close the question without recording which answer was given. Top-level `/removal-proposals` rather than `/tasks/removal-proposals`, because the manifest mounts `tasks` first and its `/tasks/:id` swallows a later file’s literal segment. Rot is INHERITED: `backlog` is rottable and `abandoned` is not, so an unanswered proposal rots and an answered one leaves the feed with no code written for either — pinned against the rot module’s own exported vocabulary. Rejected: the status as the record; parsing the body at confirm time; widening the nod queue; separate `/confirm` and `/dismiss` routes; upserting a second verdict; a `LIKE ‘prefix%’` match (a rule key may contain an underscore, which is a LIKE wildcard).](0319-a-removal-verdict-is-a-row-beside-the-question.md) | ideas / lifecycle |
|
|
413
|
+
| 0318 | [**Listing a file for task affinity is not claiming to own it** ([task 1004110](https://cloudbongos.com/builders#/task/1004110), goal 1000095 — BONGOS-V2). The case [ADR 0314](0314-module-protection-is-per-path-not-per-module.md) named and left open, closed by neither of the two routes it predicted. `builder-settings` derived `protected` — Archon-only to scope a goal to — on exactly one glob, `src/bongos/routes/me.js`, caught by the blanket `src/bongos/routes/` prefix. **That glob was never an ownership claim:** `MODULE_GLOBS` listed it for task AFFINITY and said in the same comment that the `GET /me` aggregator stays a thin core facade, and `isProtected()` cannot tell the two apart. **Both predicted routes were measured and rejected.** Narrowing the blanket prefix: all eighteen files under `src/bongos/routes/` are protected by that prefix AND NOTHING ELSE, so narrowing unwalls `auth.js`, `builders.js`, `backup.js` and `core-update.js` together — now pinned as a tripwire test rather than left as a caution. Moving `me.js` into `modules/builder-settings/`: the aggregator resolves TEN module ports (`lifecycle`, `reward`, `onboarding`, `government`, `specialities`, `grade`, `box`, `art` ×2, `discord`) and `builder-settings` is one of them, so hosting it inside one module means resolving nine foreign ports from the inside — the `core -> module` edge [ADR 0083](0083-modular-architecture-strangler-build-plan.md) rule 1 forbids. Decision: **ownership follows what the file is** — `me.js` moves to `kernel`, which already owns `src/bongos/routes/security.js`. This is 0314 one step on: owning the command is not owning the authority, and **being listed next to a module is not owning it either**. **Rejected: a general affinity tier** in `MODULE_GLOBS` mirroring `isCommandPath` — the audit does not support it. Three other modules (`kernel`, `devsecops`, `methodology`) are also protected only by paths outside their own directory, but in each the foreign path genuinely IS the domain; `builder-settings` was the only module protected by a path its own source called core, so a new tier would be a new place for a hole to hide, built for a population of one. `protectedModules()` goes 9 -> 8. **The wall that matters does not move:** a sub-Metic builder touching `/me` is still refused by `checkPermissionPaths`, asserted directly rather than assumed. The sixteen still-unowned route files remain [task 1004116](https://cloudbongos.com/builders#/task/1004116), whose option (a) this de-risks.](0318-listing-a-file-for-task-affinity-is-not-owning-it.md) | permissions / module scope |
|