Onboarding: palace layout parity, egress receipt docs, path bugs - #87
Merged
Conversation
huximaxi
marked this pull request as ready for review
September 9, 2026 06:45
- loci-cli now accepts the rooms/ layout the templates kit builds (validate() gains a RoomsDir variant; loci status no longer says 'no palace found' against a kit-built palace). Five new unit tests, cargo build and cargo test both clean. - Document the egress receipt: loci audit and wal verify get a real section in loci-cli/README.md, and CHANGELOG's Unreleased is filled in (the receipt shipped on main in July with no public docs). - Rust prerequisite added before cargo install; the handover path (soul/handovers/ vs handover/) and template pointers (_templates/ vs templates/) reconciled across SETUP-GUIDE, FIRST-SESSION, AGENT-SETUP and the CLAUDE-master template; the sandbox-only find /sessions example replaced with something that works on a real machine. - A Palace: line in the global identity block and the LOCI_PALACE env var give tools one place to find the palace; SETUP-GUIDE and AGENT-SETUP each get one paragraph naming what the other door builds, since they currently produce slightly different palaces. Verified: cargo test -p loci-cli palace:: (5/5 pass), cargo build clean, diff bleed-checked for internal names and local paths.
The three new layout tests (this branch) used real private-palace room names (great-hall, engine-room, observatory) as test fixtures. Public repo, so swapped for generic placeholders consistent with the fourth test's own convention (work-room, writing-room). No behavior change; cargo build and cargo test -p loci-cli both clean, 5/5 pass.
FreshEyes caught it during the gauntlet Cipher pass: the sentence listed guides for crystals, the garden, retrieval and scheduled tasks as if that named everything beyond the core five, but the folder holds a dozen more files. Read as inexact rather than wrong.
huximaxi
force-pushed
the
fix/onboarding-gaps
branch
from
September 9, 2026 06:53
14cb84a to
88c43cd
Compare
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
1 participant
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.
The templates kit builds
rooms/<room>/CLAUDE.md;loci-clionly recognised_palace/or rooms-at-root, soloci statusagainst a kit-built palace returned no palace found. Found and reproduced during a from-zero onboarding rehearsal.loci-cligains a third layout (RoomsDir); five unit tests cover all three shapes plus the negative cases.cargo buildandcargo testboth clean.loci-wal,loci audit,loci wal verify) shipped tomainin July with no public documentation anywhere. Documented now, including the honest bound (the hash chain alone is not keyed tamper-evidence; that needs a verified bundle).findexample, a missing Rust prerequisite beforecargo install.Palace:line in the identity block plus the existingLOCI_PALACEvariable give tooling one place to find the palace; the two setup doors (manual, agent-driven) now each say what the other builds.Diff bleed-checked for internal names and local paths.