[PR #38] p3: fork test comment references non-existent issue #22.x — broken tracker reference

[obchain]

Summary

test_executeLiquidation_endToEndOnFork in Section E contains:

/// Status: SKIPPED pending issue #22.x

and later:

/// TODO(#22.x): unmocked PCS swap once a stable pair + amount is identified.

No issue #22 exists in the obchain/Charon tracker. The PR itself closes #23. The sub-issue notation "#22.x" is not a valid GitHub issue reference format. This creates a permanently broken tracker reference embedded in source code that:

1. Cannot be auto-closed by GitHub
2. Points developers to a non-existent issue when they try to un-skip the test
3. Makes it unclear which milestone or issue actually gates the fork test

Location

contracts/test/CharonLiquidator.t.sol — test_executeLiquidation_endToEndOnFork NatSpec comment

Fix

Replace #22.x with the correct issue number that tracks fork test completion (likely the issue that supersedes #123, or a new dedicated issue). Update both the Status: line and the TODO line.

Refs #38

[obchain]

Fixed via commit 6447016. The skipped fork test's docblock and TODO now reference issue #53 (feat/25-foundry-fork-tests) instead of the non-existent #22.x placeholder. The natspec block lists the exact prep work needed to un-skip it: pin a BSC block, use a known-underwater borrower from the scanner output, mock only vToken.liquidateBorrow + vToken.redeem, and assert profit > 0 with LiquidationExecuted emitted.