Repository navigation
Write loop hives as hive.json, agents.json, desks.json and tools.json - #18
Merged
Merged
Conversation
…ates The hive template now ships agents.json and desks.json alongside hive.json, and the scaffold writes all three into the loop folder. The roles, seats and desks blocks were moved out of hive.json so each concern lives in its own file. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
The scaffold no longer patches the hive.json template to inject tools; it now writes a dedicated tools.json built from scratch. This keeps the template untouched and makes the generated manifest independent of the template's contents. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
The scaffolded hive now writes its identity, roles and seats to agents.json, its desks to desks.json, and its tools and toolpacks to tools.json instead of one large hive.json, matching the layout tinyhivemind allows. Tests were updated to read the toolkit from the new tools file and to assert that the manifest is split. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
The loops spec now describes the hive directory as a four-way manifest split into hive.json, agents.json, desks.json and tools.json, and the running-a-loop section requires an rsi-ralph that reads the split manifest, noting that older binaries fail with a missing field error. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
The module documentation now names the manifest file that `loop new` writes (`tools.json`) alongside the `tools` array it describes, so readers can find the declaration without guessing which file holds it. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
Reworded the module doc comment to describe the tools manifest as kept in tools.json within the loops that loop new writes, making the sentence easier to read without changing any behaviour. Auto-committed-on: cuttlefish Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
Caution Review failedThe pull request is closed. Finishing Touches
Comment |
senamakel
marked this pull request as ready for review
October 25, 2026 10:03
Tiny Sweeper review
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
loop newnow writes the hive's manifest in four files instead of one largehive.json, using the split tinyhivemind now supports (tinyhumansai/tinyhivemind#134):hive/hive.json: identity, context, policies, coordinatorhive/agents.json: roles and seatshive/desks.json: the deskhive/tools.json: the toolkit's executable tools and domain toolpacks, generated from the registry as before. It is now written as a file of its own instead of being injected intohive.json.Merge after tinyhumansai/rsi-ralph#82. That PR pins the tinyhivemind that reads split manifests. Until it lands, released
rsi-ralphbinaries refuse these folders (missing field seatsinhive.json), so this PR is a draft.Public API / behaviour changes
supermath-mcp loop newwriteshive/agents.json,hive/desks.jsonandhive/tools.json.hive/hive.jsonno longer holds roles, seats, desks, tools or toolpacks.docs/specs/loops.mdnow says.Validation
cargo fmt --all -- --check: cleancargo clippy --all-targets -- -D warnings: cleancargo test: all pass. New test:the_hive_manifest_is_split_into_agents_desks_and_tools. The toolkit test now readstools.jsonand checks seat toolpacks againstagents.json..github/scripts/check-file-coverage.sh 90 coverage.json: pass (cli/scaffold/mod.rs96.8%)RUSTDOCFLAGS="-D warnings" cargo doc --no-deps --workspace: cleanloops/statements/gauss-sum.leanand ranrsi-ralph checkon it.ok: demo (dir subject, 1 scorers, decide proof by seeds, hive lead lead)invalid json in hive.json: missing field seats, which is why this has to wait for #82.ADR 0010 still says
hive.jsondeclares the tools; ADRs are immutable, so it is left as written.