Skip to content

fix: keep ghost payment identifiers collision-free - #22

Merged
rogerchappel merged 3 commits into
mainfrom
agent/oss-116792b5b8c5-ghost-payment-ids
Aug 31, 2026
Merged

fix: keep ghost payment identifiers collision-free#22
rogerchappel merged 3 commits into
mainfrom
agent/oss-116792b5b8c5-ghost-payment-ids

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • add regression coverage for collisions with PAY-GHOST-9001, INV-NOT-FOUND, and VEN-404
  • allocate unique payment and deliberately missing relationship identifiers
  • keep expected finding evidence synchronized with generated identifiers
  • document the scenario behavior change

Commits

  • cf70b27 test: cover ghost payment identifier collisions
  • e408190 fix: allocate ghost payment identifiers safely
  • 9868459 docs: note collision-safe ghost payments

Verification

  • node --test tests/scenarios.test.js
  • npm run release:check
  • git diff --check

All 48 tests pass; syntax, CLI smoke, documentation install smoke, and package dry run pass.

@rogerchappel

Copy link
Copy Markdown
Owner Author

Automated merge note

Triage class: auto-merge

Summary: Makes the ghost-payment scenario allocate collision-free payment and missing-reference identifiers, with regression coverage and a changelog note (3 files, +59/-2).

Checks run: CI / Repository hygiene — SUCCESS; contributor verification: node --test tests/scenarios.test.js (9/9), npm run release:check (48/48 tests, CLI/docs/package smoke), and git diff --check; final diff, mergeability, branch protection, and all Roger Chappel author/committer identities re-verified.

Rebased/CI-repaired: no.

Verified head SHA: 9868459.

@rogerchappel
rogerchappel merged commit 5a6064c into main Aug 31, 2026
1 check passed
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