design: edge rate limit, Paul's two rulings recorded (#358)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 file changed,
+16
−17
docs/superpowers/specs/2026-10-24-edge-rate-limit-design.md+16 −17
CI passed — run details
CI log
=== gate ===
.amadan/test-gate.sh
=== launchctx (0 s)
ok amadan.net/carlos/platform.git/launchctx 0.013s
? amadan.net/carlos/platform.git/launchctx/controlname [no test files]
? amadan.net/carlos/platform.git/launchctx/wireid [no test files]
=== vet (1 s)
=== gofmt (0 s)
gofmt -l .: clean
=== rest (154 s)
? github.com/carlosframework/platform/cmd/changelog [no test files]
ok github.com/carlosframework/platform/cmd/console 4.742s
ok github.com/carlosframework/platform/cmd/moveprobe 12.100s
ok github.com/carlosframework/platform/cmd/zddprobe 0.741s
ok github.com/carlosframework/platform/internal/activator 57.531s
ok github.com/carlosframework/platform/internal/agentsock 0.053s
ok github.com/carlosframework/platform/internal/alerting 0.011s
ok github.com/carlosframework/platform/internal/analytics 0.036s
ok github.com/carlosframework/platform/internal/appconfig 3.231s
ok github.com/carlosframework/platform/internal/apperr 0.177s
ok github.com/carlosframework/platform/internal/applog 0.271s
ok github.com/carlosframework/platform/internal/awsinv 0.007s
ok github.com/carlosframework/platform/internal/boxchoice 0.006s
ok github.com/carlosframework/platform/internal/boxid 1.019s
? github.com/carlosframework/platform/internal/boxid/boxidtest [no test files]
? github.com/carlosframework/platform/internal/boxid/internal/signtest [no test files]
ok github.com/carlosframework/platform/internal/boxvolumes 0.010s
ok github.com/carlosframework/platform/internal/caps 0.014s
ok github.com/carlosframework/platform/internal/certwarm 0.002s
ok github.com/carlosframework/platform/internal/cgroup 0.844s
ok github.com/carlosframework/platform/internal/changelog 0.580s
ok github.com/carlosframework/platform/internal/claimactivation 1.435s
ok github.com/carlosframework/platform/internal/clamd 0.062s
? github.com/carlosframework/platform/internal/cliflag [no test files]
? github.com/carlosframework/platform/internal/controlname [no test files]
? github.com/carlosframework/platform/internal/controlsock [no test files]
ok github.com/carlosframework/platform/internal/dbdoor 7.967s
ok github.com/carlosframework/platform/internal/dbdoor/dbdoortest 0.012s
ok github.com/carlosframework/platform/internal/dbquery 0.012s
ok github.com/carlosframework/platform/internal/dbreads 2.179s
? github.com/carlosframework/platform/internal/dbreads/dbreadstest [no test files]
ok github.com/carlosframework/platform/internal/dbverify 0.006s
ok github.com/carlosframework/platform/internal/dnsrec 0.003s
ok github.com/carlosframework/platform/internal/docsite 0.031s
ok github.com/carlosframework/platform/internal/domains 3.069s
ok github.com/carlosframework/platform/internal/doorkey 0.008s
ok github.com/carlosframework/platform/internal/economics 0.004s
ok github.com/carlosframework/platform/internal/edge 45.966s
? github.com/carlosframework/platform/internal/edgeproof [no test files]
ok github.com/carlosframework/platform/internal/edgesfile 0.009s
ok github.com/carlosframework/platform/internal/fleetapi 0.222s
ok github.com/carlosframework/platform/internal/gate 1.828s
ok github.com/carlosframework/platform/internal/genselect 0.029s
ok github.com/carlosframework/platform/internal/geo 0.018s
ok github.com/carlosframework/platform/internal/geoip 0.039s
ok github.com/carlosframework/platform/internal/historyreadback 0.230s
ok github.com/carlosframework/platform/internal/host 150.893s
ok github.com/carlosframework/platform/internal/hostactors 5.725s
ok github.com/carlosframework/platform/internal/hostactors/awsmembership 0.006s
ok github.com/carlosframework/platform/internal/hostbox 3.361s
ok github.com/carlosframework/platform/internal/hostclaim 0.825s
ok github.com/carlosframework/platform/internal/hostenv 0.012s
ok github.com/carlosframework/platform/internal/hostevidence 0.011s
ok github.com/carlosframework/platform/internal/hostexec 6.122s
ok github.com/carlosframework/platform/internal/hostfence 0.055s
ok github.com/carlosframework/platform/internal/hostfiles 10.677s
ok github.com/carlosframework/platform/internal/hostfloor 0.028s
ok github.com/carlosframework/platform/internal/hostguard 1.093s
ok github.com/carlosframework/platform/internal/hostops 2.383s
? github.com/carlosframework/platform/internal/hostops/fencerelease [no test files]
ok github.com/carlosframework/platform/internal/hostops/internal/fencerelease 0.009s
ok github.com/carlosframework/platform/internal/hostquarantine 0.002s
ok github.com/carlosframework/platform/internal/hostrelocate 0.677s
ok github.com/carlosframework/platform/internal/hostresolve 0.238s
ok github.com/carlosframework/platform/internal/hostwrite 0.006s
ok github.com/carlosframework/platform/internal/infraview 0.006s
ok github.com/carlosframework/platform/internal/instancectl 5.256s
ok github.com/carlosframework/platform/internal/instanceexec 2.800s
ok github.com/carlosframework/platform/internal/instancefs 0.006s
ok github.com/carlosframework/platform/internal/instancekey 0.004s
ok github.com/carlosframework/platform/internal/instanceupdates 0.021s
ok github.com/carlosframework/platform/internal/layout 2.145s
ok github.com/carlosframework/platform/internal/lbhealth 0.003s
ok github.com/carlosframework/platform/internal/lease 10.298s
ok github.com/carlosframework/platform/internal/lease/leasetest 0.675s
ok github.com/carlosframework/platform/internal/ledger 0.014s
ok github.com/carlosframework/platform/internal/mail 0.239s
ok github.com/carlosframework/platform/internal/maintenance 0.088s
ok github.com/carlosframework/platform/internal/maintenanceexec 5.610s
ok github.com/carlosframework/platform/internal/maintenanceguard 0.067s
ok github.com/carlosframework/platform/internal/metrics 0.004s
ok github.com/carlosframework/platform/internal/moves 22.443s
ok github.com/carlosframework/platform/internal/moves/archive 1.049s
ok github.com/carlosframework/platform/internal/objstore 0.311s
ok github.com/carlosframework/platform/internal/ops 0.253s
ok github.com/carlosframework/platform/internal/park 0.224s
ok github.com/carlosframework/platform/internal/plans 0.006s
ok github.com/carlosframework/platform/internal/privatefile 0.004s
ok github.com/carlosframework/platform/internal/provision 17.795s
ok github.com/carlosframework/platform/internal/publicrelease 0.011s
? github.com/carlosframework/platform/internal/regionname [no test files]
ok github.com/carlosframework/platform/internal/registry 5.930s
ok github.com/carlosframework/platform/internal/relay 7.298s
ok github.com/carlosframework/platform/internal/release 28.504s
ok github.com/carlosframework/platform/internal/rollout 3.872s
ok github.com/carlosframework/platform/internal/rolloutwords 0.003s
ok github.com/carlosframework/platform/internal/rootaws 0.007s
ok github.com/carlosframework/platform/internal/rootparticipation 0.060s
ok github.com/carlosframework/platform/internal/rootstate 0.029s
ok github.com/carlosframework/platform/internal/runs 0.587s
ok github.com/carlosframework/platform/internal/s3compat 0.011s
ok github.com/carlosframework/platform/internal/safefile 0.004s
ok github.com/carlosframework/platform/internal/schedule 2.775s
ok github.com/carlosframework/platform/internal/seeds 0.011s
ok github.com/carlosframework/platform/internal/sim/engines 2.984s
ok github.com/carlosframework/platform/internal/sim/hostlife 16.541s
ok github.com/carlosframework/platform/internal/sim/litestreamfacts 3.750s
ok github.com/carlosframework/platform/internal/status 0.105s
ok github.com/carlosframework/platform/internal/steering 0.009s
ok github.com/carlosframework/platform/internal/stopevidence 0.188s
ok github.com/carlosframework/platform/internal/svcrole 0.002s
ok github.com/carlosframework/platform/internal/telemetry 0.004s
ok github.com/carlosframework/platform/internal/termfence 0.012s
ok github.com/carlosframework/platform/internal/tunnelclient 0.009s
ok github.com/carlosframework/platform/internal/unixdial 0.003s
ok github.com/carlosframework/platform/internal/updatewords 0.005s
ok github.com/carlosframework/platform/internal/usage 0.007s
ok github.com/carlosframework/platform/internal/version 0.004s
ok github.com/carlosframework/platform/internal/vet 1.734s
=== internal_console@0 (89 s)
ok github.com/carlosframework/platform/internal/console 87.142s
=== internal_console@1 (46 s)
ok github.com/carlosframework/platform/internal/console 43.811s
=== internal_console@2 (43 s)
ok github.com/carlosframework/platform/internal/console 40.888s
=== internal_console@3 (43 s)
ok github.com/carlosframework/platform/internal/console 40.483s
=== internal_console@4 (45 s)
ok github.com/carlosframework/platform/internal/console 42.714s
=== internal_console@5 (42 s)
ok github.com/carlosframework/platform/internal/console 40.231s
=== internal_console@6 (39 s)
ok github.com/carlosframework/platform/internal/console 36.215s
=== internal_console@7 (48 s)
ok github.com/carlosframework/platform/internal/console 45.823s
=== root@0 (113 s)
ok github.com/carlosframework/platform 111.136s
=== root@1 (87 s)
ok github.com/carlosframework/platform 84.896s
=== root@2 (95 s)
ok github.com/carlosframework/platform 93.181s
=== cross ===
--- shard 0 of 5 ---
cross-compiling darwin/arm64
--- shard 1 of 5 ---
cross-compiling darwin/amd64
--- shard 2 of 5 ---
cross-compiling linux/arm64
linux/arm64 carlos binary: 50659490 bytes (ceiling 52000000)
--- shard 3 of 5 ---
cross-compiling windows/amd64
--- shard 4 of 5 ---
cross-compiling windows/arm64
=== minio-contract ===
minio contract: arm64, minio RELEASE.2025-09-07T16-13-09Z, mc RELEASE.2025-08-13T08-35-41Z
minio: cached, digest verified
mc: cached, digest verified
minio healthy on 127.0.0.1:20999
Added `local` successfully.
Bucket created successfully `local/contract-bucket`.
Bucket created successfully `local/denied-bucket`.
Created policy `contract-only` successfully.
Added user `scopeduser` successfully.
Attached Policies: [contract-only]
To User: scopeduser
buckets and scoped user ready
=== RUN TestDeliveredContractMinio
=== RUN TestDeliveredContractMinio/put
=== RUN TestDeliveredContractMinio/get_round-trips_the_body
=== RUN TestDeliveredContractMinio/list_shows_the_key
=== RUN TestDeliveredContractMinio/head_and_list_agree_on_the_version
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_its_own_object
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_its_own_object/not_by_PutObjectTagging
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_its_own_object/and_not_by_x-amz-tagging_on_upload
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/not_by_CreateMultipartUpload
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/not_by_CopyObject_with_a_REPLACE_tagging_directive
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/but_multipart_itself_is_allowed
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/and_copy_itself_is_allowed
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/a_POST-policy_forged_tag_with_a_matching_version_is_still_not_a_verdict
=== RUN TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/and_an_ordinary_tagged-with-something-else_upload_is_allowed
=== RUN TestDeliveredContractMinio/delete_removes_it
=== RUN TestDeliveredContractMinio/cross-bucket_denial
--- PASS: TestDeliveredContractMinio (0.03s)
--- PASS: TestDeliveredContractMinio/put (0.00s)
--- PASS: TestDeliveredContractMinio/get_round-trips_the_body (0.00s)
--- PASS: TestDeliveredContractMinio/list_shows_the_key (0.00s)
--- PASS: TestDeliveredContractMinio/head_and_list_agree_on_the_version (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_its_own_object (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_its_own_object/not_by_PutObjectTagging (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_its_own_object/and_not_by_x-amz-tagging_on_upload (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route (0.02s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/not_by_CreateMultipartUpload (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/not_by_CopyObject_with_a_REPLACE_tagging_directive (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/but_multipart_itself_is_allowed (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/and_copy_itself_is_allowed (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/a_POST-policy_forged_tag_with_a_matching_version_is_still_not_a_verdict (0.00s)
--- PASS: TestDeliveredContractMinio/the_app_cannot_tag_by_any_other_route/and_an_ordinary_tagged-with-something-else_upload_is_allowed (0.00s)
--- PASS: TestDeliveredContractMinio/delete_removes_it (0.00s)
--- PASS: TestDeliveredContractMinio/cross-bucket_denial (0.00s)
PASS
ok github.com/carlosframework/platform/internal/objstore 0.036s
=== RUN TestT4StorageProbeMinio
2026/10/24 20:15:31 moves: T4: STORE: list: operation error S3: ListObjectsV2, https response error StatusCode: 403, RequestID: 18E1906EB379FEE7, HostID: 7c420053e0fbcd7f41653fdd92ffa2802c903d05303b1aa15adc2e8bb65d183f, api error SignatureDoesNotMatch: The request signature we calculated does not match the signature you provided. Check your key and signing method.
--- PASS: TestT4StorageProbeMinio (0.01s)
PASS
ok github.com/carlosframework/platform/internal/host 0.027s
=== RUN TestS3StoreMinioContract
=== RUN TestS3StoreMinioContract/conditional_get_and_etag_listing
=== RUN TestS3StoreMinioContract/bump_through_every_mutation_path
--- PASS: TestS3StoreMinioContract (0.05s)
--- PASS: TestS3StoreMinioContract/conditional_get_and_etag_listing (0.01s)
--- PASS: TestS3StoreMinioContract/bump_through_every_mutation_path (0.03s)
PASS
ok github.com/carlosframework/platform/internal/release 0.055s
Bucket created successfully `local/dbf-deploy`.
Bucket created successfully `local/dbf-data-ie`.
Bucket created successfully `local/dbf-data-uk`.
local/dbf-deploy versioning is enabled
Created policy `dbf-console` successfully.
Created policy `dbf-ukbox` successfully.
Added user `dbfconsole` successfully.
Added user `dbfukbox` successfully.
Attached Policies: [dbf-console]
To User: dbfconsole
Attached Policies: [dbf-ukbox]
To User: dbfukbox
=== RUN TestDBForwardMinio
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000000 txid.db=0000000000000001
2026/10/24 20:15:34 INFO ltx file uploaded system=store db=source.db replica=file level=0 minTXID=0000000000000001 maxTXID=0000000000000001 size=720
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000001 txid.db=0000000000000001
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000001 txid.db=0000000000000002
2026/10/24 20:15:34 INFO ltx file uploaded system=store db=source.db replica=file level=0 minTXID=0000000000000002 maxTXID=0000000000000002 size=370
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000002 txid.db=0000000000000002
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000002 txid.db=0000000000000003
2026/10/24 20:15:34 INFO ltx file uploaded system=store db=source.db replica=file level=0 minTXID=0000000000000003 maxTXID=0000000000000003 size=454
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000003 txid.db=0000000000000003
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000000 txid.db=0000000000000001
2026/10/24 20:15:34 INFO ltx file uploaded system=store db=source.db replica=file level=0 minTXID=0000000000000001 maxTXID=0000000000000001 size=720
2026/10/24 20:15:34 INFO replica sync system=store db=source.db replica=file txid.replica=0000000000000001 txid.db=0000000000000001
2026/10/24 20:15:34 console: CARLOS_SECRETS_PUBKEY is unset, so no region outside the home can be chosen automatically
2026/10/24 20:15:34 console: could not notify member@example.com of a CLI token issue: keymail: lookup member@example.com: keymail: GET /api/federation/lookup?addr=member%40example.com&from=console%40127.0.0.1%3A35385: could not determine target mailbox
agent.go:135: dbreads: dbr_job35ko3kfsdb2t7ge5ppplasq: planned 3 files (1544 bytes) and restored to 3 in 15ms
agent.go:135: dbreads: dbr_job35ko3kfsdb2t7ge5ppplasq: output 5 bytes in 1 confirmed chunks in 2ms
2026/10/24 20:15:35 console: db read dbr_job35ko3kfsdb2t7ge5ppplasq: relayed 1 chunks (102 bytes) in 1.007s, ended done
agent.go:135: dbreads: dbr_uwtwlr5ieb5qlqgedmwxmbai7y: planned 1 files (720 bytes) and restored to 1 in 14ms
agent.go:135: dbreads: dbr_uwtwlr5ieb5qlqgedmwxmbai7y: output 5 bytes in 1 confirmed chunks in 2ms
2026/10/24 20:15:36 console: db read dbr_uwtwlr5ieb5qlqgedmwxmbai7y: relayed 1 chunks (102 bytes) in 1.007s, ended done
agent.go:135: dbreads: dbr_yvyynlhyctf3b2lef6zwleluwe: planned 3 files (1544 bytes) and held through 3 in 17ms
agent.go:135: dbreads: dbr_yvyynlhyctf3b2lef6zwleluwe: output 1544 bytes in 1 confirmed chunks in 1ms
2026/10/24 20:15:37 console: db read dbr_yvyynlhyctf3b2lef6zwleluwe: relayed 1 chunks (1641 bytes) in 1.006s, ended done
=== RUN TestDBForwardMinio/by_disconnect
=== NAME TestDBForwardMinio
agent.go:135: dbreads: dbr_5scrp3xdwbbvhakqejlehxihfe: planned 3 files (1544 bytes) and restored to 3 in 16ms
2026/10/24 20:15:37 console: db read dbr_5scrp3xdwbbvhakqejlehxihfe: relayed 0 chunks (0 bytes) in 20ms, ended client gone
agent.go:135: dbreads: dbr_5scrp3xdwbbvhakqejlehxihfe: output 5 bytes in 1 confirmed chunks in 105ms
=== RUN TestDBForwardMinio/by_timeout
=== NAME TestDBForwardMinio
agent.go:135: dbreads: dbr_p7iowhkioc3755cu46kr4girqi: planned 3 files (1544 bytes) and restored to 3 in 17ms
2026/10/24 20:15:58 console: db read dbr_p7iowhkioc3755cu46kr4girqi: relayed 0 chunks (0 bytes) in 19.967s, ended no-result
agent.go:135: dbreads: dbr_p7iowhkioc3755cu46kr4girqi: output 5 bytes in 1 confirmed chunks in 19.935s
--- PASS: TestDBForwardMinio (23.44s)
--- PASS: TestDBForwardMinio/by_disconnect (0.23s)
--- PASS: TestDBForwardMinio/by_timeout (20.08s)
PASS
ok github.com/carlosframework/platform/internal/console 23.459s
minio contract: PASS
diff --git a/docs/superpowers/specs/2026-10-24-edge-rate-limit-design.md b/docs/superpowers/specs/2026-10-24-edge-rate-limit-design.md| index c09abcdb4..95c3e260f 100644 |
| --- a/docs/superpowers/specs/2026-10-24-edge-rate-limit-design.md |
| +++ b/docs/superpowers/specs/2026-10-24-edge-rate-limit-design.md |
| @@ -1,6 +1,6 @@ |
| # Edge rate limit: fair share under pressure, set per instance |
| -**Date:** 2026-10-24 · **Status:** draft, awaiting Paul's read · **Requested |
| +**Date:** 2026-10-24 · **Status:** draft, rulings in §9, awaiting Paul's read of the whole · **Requested |
| by:** Paul, from amadan carlos/platform#358 ("Can we take |
| carlos/platform#358 and make sure it's configurable per instance?"). |
| @@ -78,7 +78,7 @@ counted per host somewhere readable. |
| - State shared between boxes. Each instance has one home box and every |
| request for it ends on that box's edge (§5.1), so local state is complete. |
| - Queueing requests instead of refusing them. |
| -- An app-wide default that instances inherit (§9, question for Paul). |
| +- An app-wide default that instances inherit (§9, ruling 2). |
| - Keying by credential, cookie or session. |
| **Anti-goals: it fails if any of these happen, however well it works |
| @@ -276,7 +276,8 @@ spec says so in the docs, rather than keying every client as the hub. |
| `plans.Limits` (`internal/plans/catalog.go:26-40`) gains optional ceilings |
| for `client_share`, `rate` and `burst`. Missing means no ceiling. They are |
| -**not** pack-additive. |
| +**not** pack-additive, and a `rate` or `burst` ceiling below the platform |
| +default is refused when the catalog loads (§9, ruling 1). |
| - **At the console door:** a value above the account's effective ceiling is |
| refused with a message naming the ceiling. The message is copy and goes |
| @@ -358,20 +359,18 @@ those stay on the instance record, as #358 asked. |
| that is not woken by a refused request, and a converge that changes a |
| setting and is enforced after the reload. |
| -## 9. Open for Paul |
| +## 9. Paul's rulings on the builder check (2026-10-24) |
| Astra's builder check (`CODEX-FINDINGS-EDGE-RATE-LIMIT-builder-check.md`) |
| raised four questions and two corrections. The corrections and questions 1, |
| -2 and 4 are settled above as my calls. Two remain for Paul: |
| - |
| -1. **Can a plan push the backstop below the platform default?** (builder |
| - check 3.) If a Free plan's ceiling were lower than the default `rate` or |
| - `burst`, a quiet instance on that plan would refuse a busy shared |
| - address, which breaks decision 4. Recommendation: no. A catalog ceiling |
| - may never be below the platform default (the catalog loader refuses |
| - one), so a plan can only limit how far an owner raises the numbers. |
| -2. **An app-wide default that instances inherit?** Tito runs one instance |
| - per customer. With per-instance settings only, a change of policy is one |
| - call per instance. Recommendation: not in v1; the API makes the loop |
| - cheap, and the layer can be added later as app default → instance |
| - override without changing anything here. |
| +2 and 4 are settled above as my calls. Paul ruled on the other two ("Yes" to |
| +both recommendations): |
| + |
| +1. **A plan can never push the backstop below the platform default.** A |
| + catalog ceiling for `rate` or `burst` below the platform default is |
| + refused when the catalog loads, so a plan only limits how far an owner |
| + raises the numbers. This keeps decision 4 true on every plan. |
| +2. **No app-wide default in v1.** Settings are per instance only. An app |
| + default that instances inherit can be added later as a layer between the |
| + platform default and the instance's own values, without changing anything |
| + in this design. |