open/portfolio-optimiser
1
0
Fork 0
Code Issues Releases 3 Activity

style(format): ruff format on the three files P7/P8 left drifting [skip-docs]

Browse source 
Pure formatting, no behaviour change. ruff 0.15.18 reported
"3 files would be reformatted, 195 files already formatted" on HEAD 455d611;
after this commit "198 files already formatted". Suite 1570 passed / 5 skipped
before and after, golden demo-transcript.stdout content sha1 unchanged
(ea8c534773acdbe41ae68f2c55724d69aaf8be4f).

Two of the three came from P7 (277bb95), one from P8 (455d611).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Kjell Tore Guttormsen 2026-09-10 06:23:35 +02:00
commit 2e04c00d6e
3 changed files with 7 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/portfolio_optimiser/run.py
View file

@ -1111,8 +1111,7 @@ async def run_project(
if bundle_dir is not None: if bundle_dir is not None:
bundle = okf.navigate_bundle(bundle_dir) bundle = okf.navigate_bundle(bundle_dir)
bundle_grounding = "\n".join( bundle_grounding = "\n".join(
"\n".join([f.name, *f.frontmatter.values(), f.body]) "\n".join([f.name, *f.frontmatter.values(), f.body]) for f in bundle.context_files
for f in bundle.context_files
) )
# ONE bundle-id rule (Step 10, slackened S7a-3 pkt. 1): the DECLARED id is the identity and # ONE bundle-id rule (Step 10, slackened S7a-3 pkt. 1): the DECLARED id is the identity and
# the mount is carried alongside, so a base delivered under a directory name of its own is # the mount is carried alongside, so a base delivered under a directory name of its own is

7
tests/test_grounding_offer_loadbearing.py
View file

@ -104,8 +104,7 @@ def _project(*codes: str) -> Project:
description="", description="",
currency="NOK", currency="NOK",
cost_items=tuple( cost_items=tuple(
CostItem(code=c, description=c, quantity=1.0, unit="stk", unit_cost=1.0) CostItem(code=c, description=c, quantity=1.0, unit="stk", unit_cost=1.0) for c in codes
for c in codes
), ),
docs_dir="", docs_dir="",
) )
@ -226,9 +225,7 @@ async def test_the_full_run_carries_the_offer() -> None:
docs_dir=str(UNANCHORED), docs_dir=str(UNANCHORED),
bundle_dir=str(UNANCHORED), bundle_dir=str(UNANCHORED),
verdict_input=_VERDICT_INPUT, verdict_input=_VERDICT_INPUT,
client_factory=scripted_factory( client_factory=scripted_factory({"proposer": _VALID_REPLY, "checker": _CHECKER_REPLY}, []),
{"proposer": _VALID_REPLY, "checker": _CHECKER_REPLY}, []
),
) )
offer = getattr(result, "grounding_offer", None) offer = getattr(result, "grounding_offer", None)

8
tests/test_identifier_grounding_loadbearing.py
View file

@ -47,9 +47,7 @@ def _proposal(*codes: str, unit_cost: float = 100.0) -> SavingsProposal:
return SavingsProposal( return SavingsProposal(
project_id="K2", project_id="K2",
measure="reduce marking scope", measure="reduce marking scope",
affected_items=[ affected_items=[AffectedItem(code=c, quantity=10.0, unit_cost=unit_cost) for c in codes],
AffectedItem(code=c, quantity=10.0, unit_cost=unit_cost) for c in codes
],
claimed_saving_nok=1.0, claimed_saving_nok=1.0,
) )
@ -74,7 +72,9 @@ def _project():
("s7c-k2-generation-prompt.txt", ("PRD-001", "PRD-002")), ("s7c-k2-generation-prompt.txt", ("PRD-001", "PRD-002")),
], ],
) )
def test_a_fabricated_code_from_a_real_recording_falls(fixture: str, codes: tuple[str, ...]) -> None: def test_a_fabricated_code_from_a_real_recording_falls(
fixture: str, codes: tuple[str, ...]
) -> None:
"""(a)+(b). Both recordings' codes are absent from the prompt that produced them (denominator """(a)+(b). Both recordings' codes are absent from the prompt that produced them (denominator
2 of 2 in each), and the verdict must now fall — with ``baseline=None``, which is exactly the 2 of 2 in each), and the verdict must now fall — with ``baseline=None``, which is exactly the
configuration under which økt 108 said ``validated``.""" configuration under which økt 108 said ``validated``."""