Remove dead pointers and sibling-repo references left by the public move - #132
Merged
Merged
Conversation
Every path a tracked file names now exists, or the sentence says plainly that the artifact is unpublished history. The documentation stands alone as a public library's own documentation.
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.
Two kinds of residue from the cleanup PRs: paths that name files no longer in the tree, and documentation that routes a public reader to the owner's private repositories.
Stale pointers
benchmarks/authorities.jsongenerated_from: droppedbenchmarks/claims.json,docs/internal/UPSTREAM_SURFACE_MAP.md,docs/internal/COVERAGE_MATRIX.mdanddocs/internal/PARITY_VALIDATION_PROGRAM.md, which are gone; repointed the two that survived the move todocs/differentials/INSTRUCTION_OPTIMIZER_FIDELITY.mdanddocs/differentials/RESEARCH_LANDSCAPE.md.benchmarks/authorities.jsondataset_protocol.references: removed the sevendocs/internal/PARITY_VALIDATION_PROGRAM.md#...references; where that emptied the list the status moves fromprotocol_defined/pinnedtogap, because nothing in the repository defines the protocol any more (familiesoptimizer_few_shot_search,optimizer_instruction_search,optimizer_simba,optimizer_gepa,runtime_performance);tools_agents_reactandoptimizer_miprov2keep their statuses on their remaining references.benchmarks/authorities.jsonlocal_differential.artifacts: removed fourteen entries underbenchmarks/results/andbenchmarks/evidence/, none of which was ever tracked; six families whose only evidence was such an entry move tostatus: gap, artifacts: [], and three notes that still spoke of an admitted artifact now say the artifact is unpublished history.docs/differentials/COMBEE_FIDELITY.md,CONFIDENCE_CALIBRATION.md(two),MULTIMODAL_FIDELITY.md,PLAYBOOK_OPTIMIZER.mdandRLM_FIDELITY.md(two) cited result JSONs underbenchmarks/results/as retained evidence; each sentence now states the observation with its date and says the result JSON is unpublished history not retained here.examples/matched_instruction_optimizers_trec/README.md: the recompute command now matchesdocs/BENCHMARKS.mdexactly and reads the committed rows underexamples/matched_instruction_optimizers_trec/data/; the sealed campaign artifact is stated as unpublished history rather than given a deletedbenchmarks/evidence/archive/...path.benchmarks/config/support-ticket-lift-openrouter-free-v2.json,-v3.json,local-mlx-lifecycle-banking77-v2.jsonand-v2-deployment-continuation.jsonkeep theiroutput_artifactandpredecessor.artifactstrings, which are a write target and a preregistration record respectively, and each gains onenotekey saying thatbenchmarks/results/is untracked and a predecessor named there is unpublished history.Self-containment
README.md"Where this fits" is now four sentences about Imp alone: a library, itsdeepfates/ex_mcpfork dependency as the one MCP and ACP implementation it uses, no protocol endpoint on ordinary startup, and a host application owning product lifetimes.decisions.md: dropped the "(workshop)" tags, restated "WorkshopAGENTS.md" as "Owner ruling, 2026-09-11" (that file does not carry these rulings, so citing it would have been false) or asCONTRIBUTING.mdand.github/workflows/ci.ymlwhere those do; removed the sibling names from the ex_mcp pin row and the "Haven preset" mention; rewrote the "Nothing leaves these repositories" row as outside contributions welcome through GitHub with ex_mcp patches upstreamed only by the owner.decisions.mdticket row: it cited aCONTRIBUTING.mdsentence and a 57-file.tickets/directory that are both gone, so it is now recorded as resolved.examples/matched_instruction_optimizers_trec/: "workshop spend ceiling" is now "spend ceiling" andWORKSHOP_CEILINGisSPEND_CEILINGacrossREADME.md,contract.exs,run_paired.pyandpaired_coordinator_test.py. The renamedcontract.exslaunch-status value and the renamed coordinator output key have no readers.lib/imp/mcp/oauth.ex,lib/imp/acp/local.ex,mix.exs,test/mcp_oauth_test.exs,test/react_v2_test.exs: replaced "a Dwell resident, Haven", "a resident application", "Shared constellation reference", "Haven's shape" and "a resident's client" with host-application wording.Deliberately left
examples/deployment/data/banking77-mipro-stage1.jsonembeds 37 repository paths underexposure.scanned_files[].path, some now deleted. It is a leakage-audit receipt;examples/deployment/banking77_mipro.exsreads onlycondition_id,digestsand the three splits, so nothing treats those paths as inputs.examples/matched_instruction_family_ifbench/contract-draft.jsonstill hasknown_workshop_aggregate_before_usd_lte,workshop_aggregate_after_usd_lteandauthorized_workshop_ceiling_usd. It is a preregistration record with no readers, and the names describe a spend ceiling rather than any other repository.bench/imp/benchmark_truth/hover_gepa_no_merge_plan.ex,test/hover_gepa_no_merge_plan_test.exs,test/local_simba_feedback_trec_example_test.exs), "haven"/"haven't" in the GEPA stopword list, and the"# Haven"markdown heading fixture intest/rlm_interpreter_test.exs.benchmarks/config/playbook-live.jsonwas named in the brief but writes tobenchmarks/runs/andbenchmarks/checkpoints/, notbenchmarks/results/; nothing to change.Verification, locally, all passing:
mix format --check-formatted;MIX_ENV=test mix compile --warnings-as-errors;mix testover the eight named files, 81 tests 0 failures;python3 examples/matched_instruction_optimizers_trec/paired_coordinator_test.py, 6 tests OK;python3 test/python_upstream_authority_registry_test.py, 4 tests OK;MIX_ENV=test mix docs.check, 5 livebooks passed; and the TREC recompute command fromdocs/BENCHMARKS.md, which agrees with the committed aggregate in full.