Files
ietf-draft-analyzer/workspace/draft-team/cycles/agent-error-recovery-rollback/50-reviews-v1/architecture.md
Christian Nennemann 2506b6325a
Some checks failed
CI / test (3.11) (push) Failing after 1m37s
CI / test (3.12) (push) Failing after 57s
feat: add draft data, gap analysis report, and workspace config
2026-04-06 18:47:15 +02:00

1.5 KiB

Architecture Review

Findings

Medium: the draft is mostly well scoped, but it wavers between abstract event semantics and protocol behavior

The document says it is carrier-agnostic and not a transport binding, which is correct. However, several MUST-level statements already imply protocol behavior. That is acceptable, but the architecture should acknowledge that the document defines an abstract protocol model, not only vocabulary.

Medium: coordinator role is introduced but not integrated into the model

The coordinator is defined as optional, yet no section explains how peers distinguish coordinator-computed scope from sender-local scope. That leaves a conceptual hole in the actor model.

Medium: cancellation is declared out of scope, but the boundary with rollback is not fully clean

The text says cancellation of work not yet started is out of scope, except when used internally to satisfy rollback. That line is defensible, but it should be expressed more rigorously to prevent readers from assuming cancellation semantics are standardized here.

Open questions

  • Should the draft describe itself as an abstract recovery protocol profile rather than only "semantics"?
  • Does the optional coordinator need one or two normative constraints, or should it be deferred entirely?

Residual risk

Scope discipline is good overall. The main remaining architectural risk is ambiguity about whether this document is merely descriptive or actually defines interoperable protocol behavior. It should explicitly choose the latter in a carefully bounded way.