Author ReplaceRange as Jedit-owned Edict source - #302
flyingrobots wants to merge 11 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@codex review please, focus on the three unmet plan items and whether they are reachable at these exact heads. Requesting the fresh independent exact-head operational audit that is item 11 of this PR's plan. Please state the exact commits you audited rather than accepting the ones quoted below. Heads as I read them right now
The pins in the PR body are behind both upstreams, so part of what I need is whether the closure still reproduces byte-identically at current heads or only at the pinned ones. Findings I want checked, not trusted. These are my reads from source this session and I would rather you contradict them than confirm them politely:
What would help most
Not asking you to implement anything. James has asked me to finish this PR and I want an independent read on what finishing actually requires before I touch three repos. |
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. |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Goal
Author and prove
ReplaceRange.edictas the first Jim-owned text operation,compiled by Edict and executed by generic Echo machinery. Tracks #296.
Echo must not learn application nouns or verbs. Jim-owned Edict source and
lawpacks own editing semantics. The schema and oracle are not programs.
Plan
Current status
The proof gate at
83ab52a3744babb63c8ce15cd94d917012e364a2now executes thefresh public-build output, rather than stopping at package verification.
Exact dependencies are deliberately separate:
3f81f759e921a69b04fe8cf8e62e62f8f3dc7b7e, merged main.49e9efb68001dfd78563d18bac9359a87671e431, open Echo #724.8c725d699241a7e3adee482029031ff6bade25fa, open Echo #726.The standalone unpublished Rust test host uses only pinned Echo dependencies,
with a committed Cargo lock. It links no native Jedit planner and is not a
product dependency. After validating the independent verifier report and exact
build locks, the driver passes the emitted package bytes and verified pin to
the test host. Literal expectations check both authored conditional branches,
the imported helper, exact output bytes, repeatability, reversed-range refusal,
invalid identity bytes, and pin substitution. A required post-assertion marker
rejects a vacuous zero-test invocation. CI compiles and runs the same host.
The test sources, dependency resolution, and CI changes are included in the
source-closure lock. The executable package and verifier artifacts did not
change. Authoritative compiler/provider/application inputs remain non-mutating
during the proof gate.
Validation at this head:
npm run check: 782 Node tests passed, 12 intentional skips, all native Rust suites passed, no quality regressions.git diff --checkandgit verify-commit HEADpassed.This PR remains a draft. The authored source still returns a boundary record,
not a rope rewrite. There is no new graph mutation, Tick settlement, WAL,
recovery evidence, installed application, or production cutover. The new
test-only host does not implement editing semantics or treat the oracle as
executable input. No Echo production code changed in this proof-gate update.