Skip to content

fix: capture transactional factsheet anchors#78

Merged
teamchong merged 1 commit into
teamchong:mainfrom
ousamabenyounes:fix/issue-55
Jul 19, 2026
Merged

fix: capture transactional factsheet anchors#78
teamchong merged 1 commit into
teamchong:mainfrom
ousamabenyounes:fix/issue-55

Conversation

@ousamabenyounes

Copy link
Copy Markdown
Contributor

Summary

  • Extend factsheet extraction for issue 55 exact-token classes: email addresses, IBAN-like strings, and currency amounts such as $14,360.
  • Keep those tokens in the protected priority tier alongside existing UUID, hex, flag, and number anchors.
  • Add a focused factsheet regression test for transactional text.

Fix #55

Test verification (RED -> GREEN)

RED, new test before production extractor change:

> pxpipe-proxy@0.8.0 test
> vitest run tests/factsheet.test.ts

RUN  v4.1.6

❯ tests/factsheet.test.ts (15 tests | 1 failed) 65ms
    × captures issue 55 exact-token classes for transactional text 8ms

Test Files  1 failed (1)
Tests  1 failed | 14 passed (15)

RED, production factsheet changes temporarily reverted while keeping the test:

> pxpipe-proxy@0.8.0 test
> vitest run tests/factsheet.test.ts

RUN  v4.1.6

❯ tests/factsheet.test.ts (15 tests | 1 failed) 51ms
    × captures issue 55 exact-token classes for transactional text 5ms

Test Files  1 failed (1)
Tests  1 failed | 14 passed (15)

GREEN, focused test after fix:

> pxpipe-proxy@0.8.0 test
> vitest run tests/factsheet.test.ts

✓ tests/factsheet.test.ts (15 tests) 81ms

Test Files  1 passed (1)
Tests  15 passed (15)

GREEN, required checks:

> pxpipe-proxy@0.8.0 typecheck
> tsc --noEmit

> pxpipe-proxy@0.8.0 test
> vitest run

Test Files  32 passed (32)
Tests  646 passed (646)

> pxpipe-proxy@0.8.0 build
> node scripts/build.mjs

✓ emitted dist/ library modules + declarations
✓ built dist/node.js
✓ version smoke check: --version prints 0.8.0

@teamchong
teamchong merged commit b754d95 into teamchong:main Jul 19, 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.

Proposal: "anchor sidecar" — pass exact identifiers as text alongside imaged blocks

2 participants