Skip to content

Conversation

@lempira
Copy link
Contributor

@lempira lempira commented Nov 22, 2025

This adds the recordings for indexer endpoints to be used in the mock server for indexer testing.

@joe-p joe-p self-requested a review November 23, 2025 19:42
@joe-p
Copy link
Contributor

joe-p commented Nov 23, 2025

I somehow managed to approve this PR despite reviewing the other. Is this PR based on #11 ?

@lempira
Copy link
Contributor Author

lempira commented Nov 24, 2025

I somehow managed to approve this PR despite reviewing the other. Is this PR based on #11 ?

Yeah, sorry, I forgot to mention that in the description. I had to merge that branch in, in order to get the new file structure. That's the only thing it depends on, tho. The indexer requests are independent of the algod requests.

This enables debugging the recording process and captures real testnet
API responses for mock server playback.

Adds VS Code Bun debugger configuration for server
- Strip transfer-encoding and content-encoding headers in beforeReplay hook to prevent conflicts with content-length and decompression errors
- Replace server tests with replay-focused tests
- Update unrecorded endpoint test to use non-existent account address
- Re-record HAR files with consistent API token format
- Split bin/server.ts (replay-only) and bin/record.ts (recording-only)
- Update startServer() to remove mode parameter, always replays
- Created tests for replay and recording
- Update tests to use new API signatures
- Add debug configurations for both tools
Removed the bin record file since recording happens on every server start
- Add comprehensive algod request functions in src/requests/algodRequests.ts
- Add beforePersist hook to ensure msgpack responses are base64 encoded
- Refactor record.ts to use modular request functions
- Update HAR recordings with complete endpoint coverage
- Add indexer and kmd request modules
@lempira lempira force-pushed the feature/add-indexer-recordings branch from ddb159f to 6d38803 Compare November 24, 2025 19:04
@lempira lempira merged commit e155ebc into main Nov 24, 2025
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.

3 participants