Cleanup legacy nested harnesses before #184 #187

Merged

lost-rob0t merged 15 commits from cleanup/186-remove-legacy-harnesses into main

2026-08-25 05:06:17 +00:00

lost-rob0t commented

2026-08-25 04:44:03 +00:00

(Migrated from github.com)

First structural slice of #186. This restores the runtime-only repository boundary required before #184 implementation.

What changed

moved the renderer-independent prolog_agent_ui_v1 protocol into prolog/;
moved its runtime event facade into prolog/;
relocated deterministic fixture/server support and golden NDJSON under test/;
rewired PLUnit/static-load coverage to the retained core/test paths;
removed the entire nested agentProlog/ product harness;
removed the Bun/OpenTUI renderer, client, renderer tests, package files, and dedicated UI workflow;
reconciled current protocol, async, authority, repository-agent instructions, and AgentProlog roadmap docs to standalone/downstream product ownership.

Verified migration invariants

prolog/prolog_agent_ui_v1.pl is an exact blob relocation: 0 changed lines;
test/fixtures/prolog_agent_ui_v1_session.ndjson is an exact blob relocation: 0 changed lines;
no protocol/wire-version change;
no authority/effect/provider/agent semantic ownership moves downstream;
no new Harness implementation is introduced here;
the deleted nested agentProlog/ path is absent from this branch;
#184 is the approved Harness architecture; old PR #125/#124 are not implementation bases.

Exact-head verification

Head 7387cbaa4d19b66aa89b55688618c945902e85ea is green on:

deterministic CI, including static load, aggregate PlUnit, benchmark/conformance, deep recursion, CLI/trace smoke, fresh-process graph/artifact restart, and whitespace checks;
Nix flake;
clean SWI pack install/load;
Tree-sitter FFI;
paid OpenRouter;
the CI REAL OpenRouter integration job.

Ready for review/merge. #186 remains open until this lands on main and the post-merge repository boundary is rechecked.

Refs #186 #141 #184

First structural slice of #186. This restores the runtime-only repository boundary required before #184 implementation. ## What changed - moved the renderer-independent `prolog_agent_ui_v1` protocol into `prolog/`; - moved its runtime event facade into `prolog/`; - relocated deterministic fixture/server support and golden NDJSON under `test/`; - rewired PLUnit/static-load coverage to the retained core/test paths; - removed the entire nested `agentProlog/` product harness; - removed the Bun/OpenTUI renderer, client, renderer tests, package files, and dedicated UI workflow; - reconciled current protocol, async, authority, repository-agent instructions, and AgentProlog roadmap docs to standalone/downstream product ownership. ## Verified migration invariants - `prolog/prolog_agent_ui_v1.pl` is an exact blob relocation: **0 changed lines**; - `test/fixtures/prolog_agent_ui_v1_session.ndjson` is an exact blob relocation: **0 changed lines**; - no protocol/wire-version change; - no authority/effect/provider/agent semantic ownership moves downstream; - no new Harness implementation is introduced here; - the deleted nested `agentProlog/` path is absent from this branch; - #184 is the approved Harness architecture; old PR #125/#124 are not implementation bases. ## Exact-head verification Head `7387cbaa4d19b66aa89b55688618c945902e85ea` is green on: - deterministic CI, including static load, aggregate PlUnit, benchmark/conformance, deep recursion, CLI/trace smoke, fresh-process graph/artifact restart, and whitespace checks; - Nix flake; - clean SWI pack install/load; - Tree-sitter FFI; - paid OpenRouter; - the CI REAL OpenRouter integration job. Ready for review/merge. #186 remains open until this lands on `main` and the post-merge repository boundary is rechecked. Refs #186 #141 #184