7 commits

Author	SHA1	Message	Date
Arena Agent	12f59109d7	revert(nigig-site): ce9e12b was wrong — the HTTP wire field is body, not text ... Commits dfffe9e + 13ae501 landed nigig-lite/crates/common (package `nigig-common`) and a vendored xitca-web into this repository, which makes the /sync wire format readable for the first time. Reading it shows ce9e12b was incorrect. There are TWO MessageContent types, deliberately: - nigig_common::MessageContent::Text { body, formatted_body, mentions } — the HTTP/JSON REST DTO. Its own doc-comment: "Serialises externally tagged, e.g. {"Text":{"body":"hi",…}}". This is what SendMessageRequest.content and TimelineEvent::Message.content are. - nimanyatta_protocol::MessageContent::Text { text } — the WebSocket chat protocol, serialized with postcard. nigig-common's manifest says that format "must match nimanyatta/crates/nimanyatta-protocol (the client library)". ce9e12b changed the E2E test's outgoing posts from {"Text":{"body":…}} to {"Text":{"text":…}}, reasoning from the WebSocket type. The original `body` was correct for these HTTP endpoints; my change would have made both requests fail deserialization. Reverted, and the now-unused nimanyatta-protocol dev-dependency is dropped from nigig-site. What ce9e12b got right is kept: the parser had keyed on "type":"message", but TimelineEvent carries no serde tag attribute, so it is externally tagged — {"Message":{…,"sender":…,"content":…}} with sender and content at the variant level. event_text still accepts that envelope (plus the legacy lowercase shape and `text` as last-resort field names). The fixture's cross-check test was rebuilt against nigig_common's real shape instead of the WebSocket type. It is NOT verified by execution: nigig-site's test target pulls in makepad-widgets, which is SIGKILLed on this host. The parser was verified instead by extracting event_text/sync_timelines verbatim into a standalone harness over four cases — nigig_common Message with `body` -> Some(("alice","slab done")); Membership -> None; Redaction -> None; legacy lowercase -> Some(("bob","old")). All pass. The harness is scratch, not shipped. EXECUTION_PLAN.md SITE-12 rewritten: the sibling repos are no longer missing, the /sync DTOs are named with their file paths, both MessageContent types are distinguished, and the narrower remaining blockers are stated — nimanyatta still cannot be a workspace member (xitca-web/web/Cargo.toml inherits rust-version and lints from a workspace root, and nested here it resolves to ours, which defines neither), and a full `cargo check --features b_server` has not completed on this host, so no build claim is made. Verified: cargo test -p nimanyatta-protocol = 14; cargo test -p nigig-site-core = 228 / 1 ignored; clippy -D warnings clean; cargo fmt --check clean; cargo metadata exit 0.	2026-09-26 17:39:20 +00:00
Arena Agent	ce9e12b4b9	fix(nigig-site): send and parse the message shape the server actually uses ... Follow-up to ac40ac9, which fixed only the parse side and overstated the consequence. Three corrections and two further defects. CORRECTION to my own previous claim: ac40ac9's message and the SITE-12 plan row both said that with only `content.Text.body` supported, `live_round_trip` "would still pass". That is false. sync_e2e.rs asserts `timelines.iter().any(|(_, lines)| lines.iter().any(|l| l.contains(&marker)))`, so the run would have FAILED with "app parser must extract the marker" — a misleading diagnosis, but a failure, not a silent green. The fixture bug was real; the consequence I described was not. Defect 1 — the test also SENT the wrong shape. Both message posts in live_round_trip sent {"Text":{"body":...,"formatted_body":null,"mentions":[]}}. nimanyatta/src/routes/rooms.rs binds `req.content` to a `MessageContent`, whose Text variant has a single field named `text`, so those requests would fail at deserialization before anything reached the timeline. Both now send {"Text":{"text":...}}. Defect 2 — the parser keyed on the wrong envelope. `event_text` required `event["type"] == "message"`, but the server constructs `TimelineEvent::Message { event_id, room_id, sender, content, timestamp, edited, reply_to }` (verified at the construction site in routes/rooms.rs), and `sender`/`content` live at the variant level, not under a `type` tag. What is still unknown, and is NOT guessed here: TimelineEvent's serde tagging. It is defined in `nigig_common`, which is absent from this repository, so neither externally- nor internally-tagged can be confirmed. Rather than pick one, event_text now accepts both envelopes plus the legacy lowercase shape, and `body` is kept as a last-resort field name. Membership and Redaction events are still skipped. Verification, stated precisely: - The fixture's own test target cannot be compiled here: nigig-site's test target pulls in makepad-widgets, which is SIGKILLed on this 2 vCPU / 1.9 GiB host. The shipped cross-check test remains unverified until CI runs it. - event_text and sync_timelines were instead extracted verbatim from the fixture into a standalone harness and exercised over five cases: externally tagged -> Some(("alice","slab done")); internally tagged -> Some(("bob","poured")); legacy lowercase with `body` -> Some(("carol","old")); Membership -> None; Redaction -> None. All pass. That harness is scratch, not part of the tree. Unchanged and still passing: cargo test -p nimanyatta-protocol = 14; cargo test -p nigig-site-core = 228 / 1 ignored; clippy -D warnings clean; cargo fmt --check clean; cargo metadata exit 0; 6/6 workflow python gates.	2026-09-26 16:12:03 +00:00
Arena Agent	bbce8fb015	harden SITE-02 repository and native vault contracts	2026-09-13 06:51:39 +00:00
Arena Agent	5d2d890f70	feat(nigig-site): enforce SITE-01 fail-closed containment ... Contain production capabilities, remove the production sync surface, and keep legacy media/export/transport implementations test-only. Require authenticated existing-key storage with preservation-first recovery and sticky write disablement, backed by deterministic fault and concurrency tests plus dependency and workflow contracts.	2026-09-12 15:46:30 +00:00
Arena Agent	f49d8b16ac	ci(nigig-site): establish truthful SITE-00 gates	2026-09-12 09:08:22 +00:00
Arena Agent	672fdd54b7	style(nigig-site): establish SITE-00 formatting baseline	2026-09-12 09:04:00 +00:00
andodeki	ac8f8aa002	chore: sync full working tree to gitdab ... Whole-tree sync: cad-core/cad-ui split sources, nigig-build construction_frame migration, pdf port progress, mpesa/pay/uikit/doc updates, workspace members/profiles/lock, CI workflows and reviews. See individual file history for details.	2026-09-12 07:15:24 +03:00