Transaction cycle

[jotelfootball-tech]

I have successfully pushed your latest changes to the transaction-cycle branch on GitHub.

Changes Committed
The following files were committed:

backend/package-lock.json
: Updated dependencies.
backend/src/
tests
/eventIndexer.test.ts
: Fixes and improvements to event indexing tests.
frontend/src/app/[locale]/repay/[loanId]/page.tsx
: UI enhancements for the loan repayment page.
frontend/src/app/hooks/useContractMutation.ts
: Refactored and improved contract mutation hook.

closes #566

[ogazboiz]

Hey, nice work on the transaction tracking flow. A few things need to be addressed before this can go in:

Must fix:

1. Remove status.txt - This is a git status terminal dump that got committed by accident. It's not a real project file.

2. Stale closure bug in setTimeout - In both repay/[loanId]/page.tsx and StepFinalSignature.tsx, the 60-second setTimeout captures trackerState at the time it's scheduled, not when it fires. So the check trackerState === "confirming" is always looking at a stale value. Use a ref to track current state instead.

3. Fake recovery handler - handleRecovery just fakes a success after 2 seconds. Users clicking "Transaction stuck? Try recovery" will get a "Recovery successful" message that does nothing. Either wire it up to real logic, remove the button, or clearly disable it as a placeholder.

4. enableEnhancedPolling defaults to true - This means every existing mutation using useContractMutation will now start polling Horizon automatically. This is a breaking behavior change for existing callers. Default should be false so callers opt in.

5. Countdown timer never updates - getEstimatedWaitTime() computes based on Date.now() but nothing triggers re-renders, so the "Xs remaining" text is frozen. You need a useEffect with a setInterval to make it tick, or just remove it.

Nice to have:

• The recovery/countdown/timeout logic is nearly identical in both repay/[loanId]/page.tsx and StepFinalSignature.tsx. Would be cleaner as a shared hook.
• The commit message "Implement feature X to enhance user experience and optimize performance" looks like a placeholder, might want to squash that.

[ogazboiz]

Friendly reminder to rebase your branch onto the latest main and address the review feedback when you get a chance. The codebase had some fixes recently so a rebase is needed before we can move forward. Let me know if you need help.

[ogazboiz]

heads up, a few important changes just landed on main that affect your PR:

1. axios pinned to 1.13.5 - there's an active supply chain attack on axios 1.14.1 (pulls in confirmed malware). we added overrides in all package.json files to block it.

2. CI now runs a supply chain audit before backend/frontend jobs. if your lockfile has a compromised package, CI will fail with a clear error.

3. backend test fixes - loanEndpoints tests now use valid Stellar addresses and base64 strings. if your PR was failing backend CI but you didn't touch backend code, this should fix it after rebase.

please rebase on latest main:

git fetch upstream
git rebase upstream/main
git push --force-with-lease