Content
88%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A dense, highly actionable runbook with excellent workflow sequencing and validation feedback loops for risky live-cluster operations. Its main weaknesses are minor: version-sensitive statements sit inline rather than in a deprecated section, and several referenced bundle paths (scripts/, docs/) are missing while Phase 3 duplicates content already in the reference file.
Suggestions
Move version-sensitive claims (kube-hetzner v2.20 eth1 naming, RKE2 4GB cx23 floor) into a dedicated 'Version notes / deprecated patterns' section so they can be updated without cluttering the main workflow.
Create the referenced scripts/destroy.sh, scripts/cleanup.sh, and docs/selinux.md (or remove the references) so all cited paths resolve.
Trim the inlined Phase 3 subsections (Capacity, Stateful Workloads, Worker/Control-Plane Replacement) to a concise summary and defer detail to references/immutable-node-replacement.md to avoid duplication.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 380-line body is mostly lean, actionable bash with tight domain-specific prose, but includes inline version-sensitive assertions ('Kube-hetzner v2.20 expects the private network interface to be named eth1', 'RKE2 no longer implies an 8GB control-plane floor') that are not segregated into a deprecated/old-patterns section, a minor conciseness penalty. | 4 / 5 |
Actionability | Provides fully executable, copy-paste-ready commands throughout (terraform init/plan/apply flags, kubectl queries, the jq no-destroy gate pipeline, hcloud and ssh commands) with appropriate placeholders, covering the common upgrade cases concretely. | 5 / 5 |
Workflow Clarity | Clear multi-phase sequence (Preflight, Phase 1 Module Convergence, Phase 2 Runtime Upgrade, Phase 3 Immutable Replacement, Firewall Closure, Final Report) with explicit validation checkpoints, stop conditions ('Any output is a stop condition'), feedback loops (validate -> repair -> re-prove), and a Final Report checklist, appropriate for destructive/batch live-cluster operations. | 5 / 5 |
Progressive Disclosure | Good section structure with a well-signaled one-level-deep reference ('For the full field-proven workflow, read references/immutable-node-replacement.md'), but the body references scripts/destroy.sh, scripts/cleanup.sh, and docs/selinux.md that do not exist, and Phase 3 inlines substantial detail that overlaps the existing reference file. | 4 / 5 |
Total | 18 / 20 Passed |