From 0b13fb73d06ef69f5a2bdff764ba01974009d595 Mon Sep 17 00:00:00 2001 From: Kevin Veen-Birkenbach Date: Wed, 2 Sep 2026 01:24:47 +0200 Subject: [PATCH] style: apply make autoformat Two leftovers of my own. The sweep that removed the reverse client entries appended a newline where it cut a block, which at end of file leaves a blank line after the last key in four services.yml. And the two cache-read nocheck comments in the bond editor were written past the line limit, so the formatter breaks their read_text calls across three lines and carries the comment to the closing paren. The wrap does not cost the suppression: test_no_uncached_file_reads matches the statement, not the line the directive sits on, and passes on this form. Co-Authored-By: Claude Opus 5 (1M context) --- cli/meta/roles/applications/bond/edit.py | 4 +++- roles/web-app-flowise/meta/services.yml | 1 - roles/web-app-hermes/meta/services.yml | 1 - roles/web-app-openclaw/meta/services.yml | 1 - roles/web-app-openwebui/meta/services.yml | 1 - .../unit/python/cli/meta/roles/applications/bond/test_edit.py | 4 +++- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/cli/meta/roles/applications/bond/edit.py b/cli/meta/roles/applications/bond/edit.py index 808fda52a7..281128c99b 100644 --- a/cli/meta/roles/applications/bond/edit.py +++ b/cli/meta/roles/applications/bond/edit.py @@ -70,7 +70,9 @@ def set_bond(roles_dir: Path, role: str, service_key: str, bond: float | None) - if not path.is_file(): raise EditError(f"no such file: {path}") - raw = path.read_text(encoding="utf-8") # nocheck: cache-read — a cached read would serve the text from before the previous edit and silently revert it + raw = path.read_text( + encoding="utf-8" + ) # nocheck: cache-read — a cached read would serve the text from before the previous edit and silently revert it lines = raw.splitlines(keepends=True) start, end = _block(lines, service_key) diff --git a/roles/web-app-flowise/meta/services.yml b/roles/web-app-flowise/meta/services.yml index 6032c0d923..110c4d759d 100644 --- a/roles/web-app-flowise/meta/services.yml +++ b/roles/web-app-flowise/meta/services.yml @@ -127,4 +127,3 @@ container_backup: # nocheck: bond-baseline backup host joins outside the variant bond: 1 enabled: "{{ 'svc-bkp-volume-2-local' in group_names }}" shared: true - diff --git a/roles/web-app-hermes/meta/services.yml b/roles/web-app-hermes/meta/services.yml index 84c84cacc9..51c3558129 100644 --- a/roles/web-app-hermes/meta/services.yml +++ b/roles/web-app-hermes/meta/services.yml @@ -95,4 +95,3 @@ container_backup: # nocheck: bond-baseline bond: 1 enabled: "{{ 'svc-bkp-volume-2-local' in group_names }}" shared: true - diff --git a/roles/web-app-openclaw/meta/services.yml b/roles/web-app-openclaw/meta/services.yml index 7ddeff107f..10cfd62dba 100644 --- a/roles/web-app-openclaw/meta/services.yml +++ b/roles/web-app-openclaw/meta/services.yml @@ -89,4 +89,3 @@ container_backup: # nocheck: bond-baseline bond: 1 enabled: "{{ 'svc-bkp-volume-2-local' in group_names }}" shared: true - diff --git a/roles/web-app-openwebui/meta/services.yml b/roles/web-app-openwebui/meta/services.yml index 93f785fa91..2898f3c4c3 100644 --- a/roles/web-app-openwebui/meta/services.yml +++ b/roles/web-app-openwebui/meta/services.yml @@ -108,4 +108,3 @@ container_backup: # nocheck: bond-baseline backup host joins outside the variant bond: 1 enabled: "{{ 'svc-bkp-volume-2-local' in group_names }}" shared: true - diff --git a/tests/unit/python/cli/meta/roles/applications/bond/test_edit.py b/tests/unit/python/cli/meta/roles/applications/bond/test_edit.py index ce7e178e60..21eec53488 100644 --- a/tests/unit/python/cli/meta/roles/applications/bond/test_edit.py +++ b/tests/unit/python/cli/meta/roles/applications/bond/test_edit.py @@ -28,7 +28,9 @@ class TestSetBond(unittest.TestCase): self.addCleanup(self._tmp.cleanup) def read(self) -> str: - return self.path.read_text(encoding="utf-8") # nocheck: cache-read — re-reads the file set_bond just rewrote + return self.path.read_text( + encoding="utf-8" + ) # nocheck: cache-read — re-reads the file set_bond just rewrote def test_rewrites_the_value_in_place(self) -> None: self.assertEqual(set_bond(self.roles, "web-app-demo", "litellm", 0.25), "0.25")