From 5c4a148b36bca7d538f72468c0b52f40311929d7 Mon Sep 17 00:00:00 2001 From: Jan Novak Date: Thu, 8 Oct 2026 20:47:53 +0200 Subject: [PATCH] docs: close out the conventions and listener-name plans Co-Authored-By: Claude Opus 5.5 --- .../2026-10-08-2028-house-conventions.md | 7 ++++--- .../2026-10-08-2046-listener-name-sanitize.md | 9 ++++++++- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/docs/plans-executions/2026-10-08-2028-house-conventions.md b/docs/plans-executions/2026-10-08-2028-house-conventions.md index cdac887..bbfe425 100644 --- a/docs/plans-executions/2026-10-08-2028-house-conventions.md +++ b/docs/plans-executions/2026-10-08-2028-house-conventions.md @@ -9,7 +9,7 @@ Plan: `docs/plans/2026-10-08-2028-house-conventions.md` - [x] Step 4 — Module path and version identity - [x] Step 5 — Baseline tests - [x] Step 6 — CI build workflow -- [ ] Step 7 — Close out +- [x] Step 7 — Close out ## Step 1 — CLAUDE.md @@ -151,5 +151,6 @@ bugs deferred to the bug-fix phase, one test simplification declined. Worth noting: the reviewer agents were not registered as agent types when the review started (they were added in this session), so they were run as general-purpose agents told to follow the agent files. The `:latest` dispatch hole came in with the -workflow copied from egress and is still open there. Step 7 stays unchecked until -the branch is pushed and the MR is open. +workflow copied from egress and is still open there. Pushed together with the listener-name fix +(`docs/plans/2026-10-08-2046-listener-name-sanitize.md`) as one MR. The CHANGELOG +entry is still owed, after the owner confirms it works. diff --git a/docs/plans-executions/2026-10-08-2046-listener-name-sanitize.md b/docs/plans-executions/2026-10-08-2046-listener-name-sanitize.md index 8080a53..3aa6e5f 100644 --- a/docs/plans-executions/2026-10-08-2046-listener-name-sanitize.md +++ b/docs/plans-executions/2026-10-08-2046-listener-name-sanitize.md @@ -6,7 +6,7 @@ Plan: `docs/plans/2026-10-08-2046-listener-name-sanitize.md` - [x] Step 1 — Regression tests first - [x] Step 2 — Fix - [x] Step 3 — Docs -- [ ] Step 4 — Ship +- [x] Step 4 — Ship ## Steps 1–3 — Tests, fix, docs @@ -32,3 +32,10 @@ Deviations from the plan: Worth noting: names that were valid before are unchanged, so existing listeners are not renamed. Not verified against a cluster; the production case is covered by the test using the exact cert and DNS name from the error log. + +## Step 4 — Ship + +Pushed `feat/house-conventions` and opened one MR covering the conventions port and +this fix. Still owed after merge: tag `v0.1.0` with `/release`, restart the +deployment (it pins `:latest`), confirm the reconcile error is gone, then the +CHANGELOG entries.