fix(openapi): document GET /fees, and make the coverage test deterministic and fast (W082)

[Glorious21]

Summary

Makes src/__tests__/openapiCoverage.test.ts green on main, deterministic and fast.

Routes that were missing

Only one: GET /fees. It shipped in f89a303 without a spec entry. It is now documented, including the optional ?amount= query and the per-asset fee shape (enabled, fees keyed by asset code), and openapi.json is regenerated.

Why it flapped

The route check itself is deterministic: 10 runs in a row on main gave the same result. What changed between runs was the line endings in openapi.json, not the route table.

On Windows, git's core.autocrlf=true checks openapi.json out with CRLF line endings, so the byte-for-byte comparison failed. npm run docs:openapi writes LF, so it then passed until the next checkout. I reproduced this on one commit: 1 failure after running the generator, 2 after a fresh checkout.

Fixes:

• .gitattributes pins openapi.json (and docs/openapi.json) to eol=lf.
• The test compares content with line endings normalised.

Checked both: a fresh Windows checkout is now LF, and a file forced to CRLF still passes.

Speed

About 90% of the suite's time is loading the app (Stellar SDK, Apollo, Prisma). Walking the routes and building the spec takes about 100 ms. ts-jest was also type-checking that whole import graph on every cold run, which CI already does in npm run typecheck (it covers src/**/*, tests included) before npm test. tsconfig.jest.json now sets isolatedModules, so Jest only transpiles.

	before	after
cold, with coverage (as CI runs it)	23 s	13 s
warm	~9 s	~2 s
full npm test	—	37 s

This setting applies to every suite, not just this one. Flagging it in case you'd rather keep ts-jest type checking.

Acceptance criteria

• The suite passes on main. Full npm test: 55/55 suites, 700 passed, 1 skipped.
• Ten consecutive runs give the same result. 10/10 passed, each in about 1.8 s.
• Adding an undocumented route fails the suite. efadd0d adds GET /w082-undocumented-probe, and the suite fails with + "get /w082-undocumented-probe". 4a130f5 reverts it.
• Every missing route is named: GET /fees was the only one.

Found along the way, not fixed here

src/__tests__/routes/accounts.test.ts makes a real network call on every GET: the stale-data middleware calls checkRpcHealth → getLatestLedger against the Soroban RPC, and nothing mocks it. On a slow connection, the balance tests hit Jest's 5 s timeout. I saw this once during this work; it then passed 4 times in a row. It probably needs its own issue.

Related issue

Closes #239

Type of change

• Bug fix
• New feature
• Refactor
• Docs
• Tests
• CI / tooling

Checklist

• I have read CONTRIBUTING.md
• npx tsc --noEmit passes
• npm run build passes
• I added / updated tests where relevant
• I updated docs where relevant

🤖 Generated with Claude Code

[drips-wave]

@Glorious21 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits