trios-host: split the function at trios-host.ts:149 (Biome noExcessiveCognitiveComplexity, +1 warning vs queen) #1777

Activity

1. dmitrii-f-t27 commented on Oct 11, 2026

   

   dmitrii-f-t27

   on Oct 11, 2026

   Collaborator

   More actions

   Taking this one. Filling in the sections the queue expects before the PR:

   Boundary. Only trios/agent-server/tools/trios-host/trios-host.ts. main() becomes a command table plus one function per command. No change to a card, a .t27 spec, a .gen.ts, the hosting services, the release trios-host-v0.1.0 or its pinned digests.

   User Scenarios & Testing.

   • An operator runs any trios-host command (key, status, probe, rows, statement, job, join, no command, an unknown command). Output, exit code and the order of side effects (key file made, flags checked) stay the same.
   • Check: a before/after smoke run of 31 command lines against a local mock Queen and lab, with fake zig/t27c/t27b. Normalised outputs must be identical.

   Requirements.

   • No new decision in the CLI. Every decision stays a card call (specs/hosting/*.t27).
   • An unknown command such as toString or __proto__ still prints the usage and exits 2.
   • No new Biome diagnostic of any level.

   Success Criteria.

   • biome ci . in trios/agent-server: 20 errors and 121 warnings, against 20 and 122 on queen c1d8020722 (run 38096072146 on 43e6670280 also shows 20 and 122).
   • tools/trios-host/trios-host.ts has no noExcessiveCognitiveComplexity warning.
   • The smoke runs before and after are identical. The hosting test suites stay green in CI.

   Spec. No new .t27. This is a glue-only refactor and adds no behaviour or decision. The decisions are already specified in specs/hosting/{host,proof,placement,credit,row,sybil,statement,toolchain}.t27.

   Dmitrii Fedorov (@dmitrii-f-t27)
2. dmitrii-f-t27

   added a commit that references this issue on Oct 11, 2026

   Merge pull request #1805 from gHashTag/sweep/trios-1777-split-trios-h…

   ...

   This issue will close once commit f256814 is merged into the 'main' branch.

   f256814
3. dmitrii-f-t27 commented on Oct 11, 2026

   

   dmitrii-f-t27

   on Oct 11, 2026

   Collaborator

   More actions

   Done in #1805, merged into queen as f25681452ef85dde7c122d49ff34d8f1bb74148d (head 237411968f).

   • main() in tools/trios-host/trios-host.ts is now a command table and one function per command. Behaviour is unchanged: a smoke run of 31 command lines gives identical output before and after.
   • Biome in CI on the PR head: 20 errors, 121 warnings (run 38098109018), against 20 errors, 122 warnings on queen c1d80207 (run 38097729420). The noExcessiveCognitiveComplexity warning at trios-host.ts:149 is gone.
   • The hosting tests (Tests / server-api, server-pglive) are green.

   I am closing this by hand, because queen is not the default branch and the closing keyword does not fire.

   Dmitrii Fedorov (@dmitrii-f-t27)
4. dmitrii-f-t27

   closed this as completedon Oct 11, 2026
5. dmitrii-f-t27

   added a commit that references this issue on Oct 11, 2026

   refactor(trios-host): split main() into one function per command (Closes

   ...

   This issue will close once commit 2374119 is merged into the 'main' branch.

   2374119