Skip to content

Agent B: add a receipt preview for carts#8

Closed
RollNuts wants to merge 3 commits into
mainfrom
demo/receipt-preview
Closed

Agent B: add a receipt preview for carts#8
RollNuts wants to merge 3 commits into
mainfrom
demo/receipt-preview

Conversation

@RollNuts

Copy link
Copy Markdown
Contributor

Adds orders/receipt.py with a preview that calls calculate_total(items) exactly as main defines it today. Green on its own — but if the currency PR lands first, this call shape no longer matches.

New receipt module calling the pricing contract as it exists on main.
Green on its own.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@veripsa-core

veripsa-core Bot commented Jul 17, 2026

Copy link
Copy Markdown

Veripsa — heading to main

❓ Unknown — not enough signal to call this clear

This PR reserves: orders/receipt.py, tests/test_receipt.py

New paths in this PR — coupling will be computable after merge. These paths are NEW in this PR, so there is no main-side graph to compare against yet (expected, not a warning): orders/receipt.py, tests/test_receipt.py. Veripsa will compute their coupling once this PR lands.

Veripsa records what is heading to main and who reserved what — it does not assert correctness. Advisory by default; your branch-protection policy decides what blocks.

@RollNuts

Copy link
Copy Markdown
Contributor Author

Closing — superseded by the fresh canonical collision pair #13 / #14 (2026-07-20)

Part of the earlier demo round. The current canonical demo is the clean two-PR collision pair #13 (Agent A) / #14 (Agent B) on orders/pricing.py. This PR (orders/receipt.py) is kept out of active demo traffic to keep the demo to exactly two PRs.

Historical evidence preserved (not deleted): this PR and its Veripsa output — Veripsa — Unknown (neutral) — remain in-thread: #8 (comment)

@RollNuts RollNuts closed this Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant