Skip to content

Commit 6383dbe

Browse files
cursoragentmorluto
andcommitted
Fix branch lint and formatting leftovers
Resolve import order, unused MCP imports, nested-with SIM117 in the demoted SAT harness, and ruff format drift left by earlier cleanup commits. Co-authored-by: morluto <morluto@users.noreply.github.com>
1 parent 3f9a08e commit 6383dbe

10 files changed

Lines changed: 26 additions & 24 deletions

File tree

src/jacobian/contracts/polynomial_operations.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,10 @@ def require_canonical_irreducible_records(self) -> Self:
229229
key=lambda record: (
230230
record.multiplicity,
231231
max(
232-
(sum(term.exponents) for term in record.factor.polynomial.terms),
232+
(
233+
sum(term.exponents)
234+
for term in record.factor.polynomial.terms
235+
),
233236
default=0,
234237
),
235238
tuple(

src/jacobian/domains/polynomial/invariants.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
from jacobian.contracts.polynomial_operations import (
44
PolynomialDiscriminantRequest,
55
PolynomialDiscriminantResult,
6-
PolynomialFactorRequest,
76
PolynomialFactorizationResult,
7+
PolynomialFactorRequest,
88
PolynomialGcdRequest,
99
PolynomialGcdResult,
1010
PolynomialResultantRequest,

src/jacobian/domains/polynomial/operations.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@
88
PolynomialBezoutIdentity,
99
PolynomialDiscriminantRequest,
1010
PolynomialDiscriminantResult,
11-
PolynomialFactorRequest,
1211
PolynomialFactorizationResult,
12+
PolynomialFactorRequest,
1313
PolynomialGcdRequest,
1414
PolynomialGcdResult,
1515
PolynomialGroebnerBasisRequest,

tests/boundary/mcp/test_mcp_entrypoint.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,12 @@
99
from importlib.metadata import version
1010
from pathlib import Path
1111

12-
from jacobian.adapters.mcp.server import create_server
13-
1412
MCP_TOOL_NAMES = {
1513
"math.find",
1614
"math.run",
1715
}
1816

17+
1918
def test_mcp_stdio_entrypoint_exposes_stable_math_tools(
2019
tmp_path: Path,
2120
) -> None:
@@ -67,5 +66,3 @@ def test_mcp_entrypoint_reports_distribution_version() -> None:
6766

6867
assert completed.returncode == 0
6968
assert completed.stdout.strip() == f"jacobian-mcp {version('jacobian')}"
70-
71-

tests/boundary/mcp/test_mcp_errors_and_tracing.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -170,5 +170,3 @@ async def scenario() -> None:
170170
)
171171

172172
asyncio.run(scenario())
173-
174-

tests/boundary/storage/recovery/test_search_plugin_fail_closed.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,8 @@ def test_search_batch_respects_archive_parent_limit(attached_complete_runtime) -
267267
assert snapshot.accounting.iterations == 2
268268
for page_uri in snapshot.archive_page_uris:
269269
assert (
270-
len(attached_complete_runtime.core.store.get(page_uri).manifest.parents) <= 6
270+
len(attached_complete_runtime.core.store.get(page_uri).manifest.parents)
271+
<= 6
271272
)
272273

273274

@@ -336,7 +337,8 @@ def test_refiner_can_fit_previous_nominations_to_archive_parent_limit(
336337
assert checkpoint.state["observed_lineage_parent_limit"] == 2
337338
for page_uri in snapshot.archive_page_uris:
338339
assert (
339-
len(attached_complete_runtime.core.store.get(page_uri).manifest.parents) <= 6
340+
len(attached_complete_runtime.core.store.get(page_uri).manifest.parents)
341+
<= 6
340342
)
341343

342344

tests/component/sat/test_sat_artifacts.py

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -177,13 +177,15 @@ def test_model_backed_schema_rejects_noncanonical_generic_artifact_put(
177177
payload=payload,
178178
)
179179

180-
with open_domain_services(state_root) as restarted:
181-
with pytest.raises(ArtifactValidationError):
182-
restarted.core.artifacts.put(
183-
schema_uri=restarted.core.sat.installation.cnf_schema_uri,
184-
semantics_uri=restarted.core.sat.installation.semantics_uri,
185-
payload=payload,
186-
)
180+
with (
181+
open_domain_services(state_root) as restarted,
182+
pytest.raises(ArtifactValidationError),
183+
):
184+
restarted.core.artifacts.put(
185+
schema_uri=restarted.core.sat.installation.cnf_schema_uri,
186+
semantics_uri=restarted.core.sat.installation.semantics_uri,
187+
payload=payload,
188+
)
187189

188190

189191
def test_assignment_and_raw_proof_bind_exact_cnf_identity_and_lineage(

tests/domain/graph/test_graph_core_capabilities.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,9 @@ def test_generic_graph_artifacts_use_the_authoritative_bounded_model(
4242
},
4343
)
4444

45-
assert services.core.store.get(accepted.artifact_uri).payload["vertices"] == vertices
45+
assert (
46+
services.core.store.get(accepted.artifact_uri).payload["vertices"] == vertices
47+
)
4648
with pytest.raises(ArtifactValidationError, match="does not match its schema"):
4749
services.core.artifacts.put(
4850
schema_uri=installation.graph_schema_uri,

tests/unit/support/test_runtime_profiles.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@
55
from pathlib import Path
66

77
import pytest
8-
9-
from jacobian.runtime import CheckerAuthorityMode
108
from tests.support.runtime_profiles import (
119
ATTACHED_COMPUTE,
1210
ATTACHED_COMPUTE_READ_ONLY,
@@ -18,6 +16,8 @@
1816
)
1917
from tests.support.state import copy_template, publish_template
2018

19+
from jacobian.runtime import CheckerAuthorityMode
20+
2121

2222
def test_read_only_attach_still_copies_away_from_session_template(
2323
tmp_path: Path,

tests/unit/tooling/test_plan_local_tests.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -213,9 +213,7 @@ def _planner_tree(tmp_path: Path, monkeypatch) -> ModuleType:
213213
return planner
214214

215215

216-
def _append_exact_override(
217-
manifest: Path, path: str, selectors: list[str]
218-
) -> None:
216+
def _append_exact_override(manifest: Path, path: str, selectors: list[str]) -> None:
219217
lines = [
220218
"",
221219
"[[local_planning.exact_overrides]]",

0 commit comments

Comments
 (0)