You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Propose an incremental daemon modularity architecture centered on narrow capability ports,
daemon-owned adapters, and module façades for native replay, Maestro, and replay-test.
Adds measurable R7/R9/import-boundary migration ratchets, documents the benefits and tradeoffs, and
includes two executable design probes. The session probe now pins operation-keyed repair-close
receipts, target/force persistence, and publication failure/retry transitions. Updates the dependency
findings with the measured Apple-perf SCC counterfactual. Eight files are touched; there are no
production-source or command-behavior changes.
Validation
Both boundary prototypes pass on current main. Tooling, layering, dependency-graph, Fallow, build,
and smoke gates pass. The unrestricted unit run passed 4,663 of 4,664 tests; the lone load-sensitive
device-claim test then passed 7/7 in isolation. Runtime device verification does not apply because
this PR changes architecture documentation and throwaway design probes only.
Design blocker — The proposed ScriptPublicationState cannot preserve the repair close/commit transaction it claims to consolidate. Its repair variant omits the persisted output target and force authorization, committed state, and repairPlatformCloseReceipt. Today, after platform close succeeds, the receipt is stored before atomic publication; if publication fails, retry reuses that receipt rather than dispatching platform close again. explicit-close { platformCloseSucceeded } is an input event, not durable retry state, so the proposed aggregate either loses idempotency or leaves hidden parallel state behind. Model the explicit repair phases and durable fields—including target/force, close receipt, committed/abort state—state where they live, and pin the failure/retry transitions in the prototype before treating this as a migration target. Exact-head prototype and layering checks are green, but the design is not ready.
Implementation handoff: #1478. It captures the post-review interface decisions, compatibility constraints, dependency-ordered worker briefs, STOP conditions, success metrics, and Phase 7 docs cleanup. Where it differs, #1478 supersedes the proposal's provisional TypeScript sketches and unanswered grilling section; the proposal's rationale, ADR constraints, measurements, tradeoffs, and probes remain the design context. No files in this reviewed seven-artifact PR were changed.
Architecture blocker: #1478 contains the corrected implementation handoff, but this PR’s checked-in docs still present the incompatible ScriptPublicationState / explicit-close { platformCloseSucceeded } sketches as the revised recommendation, and the session probe claims repair completion without modeling durable close/publication retry state. A PR comment does not supersede versioned guidance. Replace/remove those sketches and adjust/remove the probe, or prominently mark the exact sections as superseded by #1478 with an explicit do-not-implement warning. Exact-head checks are green; #1478 remains unimplemented residual risk.
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
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.
Summary
Propose an incremental daemon modularity architecture centered on narrow capability ports,
daemon-owned adapters, and module façades for native replay, Maestro, and replay-test.
Adds measurable R7/R9/import-boundary migration ratchets, documents the benefits and tradeoffs, and
includes two executable design probes. The session probe now pins operation-keyed repair-close
receipts, target/force persistence, and publication failure/retry transitions. Updates the dependency
findings with the measured Apple-perf SCC counterfactual. Eight files are touched; there are no
production-source or command-behavior changes.
Validation
Both boundary prototypes pass on current
main. Tooling, layering, dependency-graph, Fallow, build,and smoke gates pass. The unrestricted unit run passed 4,663 of 4,664 tests; the lone load-sensitive
device-claim test then passed 7/7 in isolation. Runtime device verification does not apply because
this PR changes architecture documentation and throwaway design probes only.