Deferred from PR #558 review.
Original reviewer comment: #558 (comment)
Context: rolesData in src/domain/analysis/roles.ts uses dynamic SQL (conditions array varies per call), so cachedStmt can't apply directly. However, the static SQL in buildTestFileIds in module-map.ts could use the cachedStmt pattern. Explore caching strategies for the dynamic case as well.
Deferred from PR #558 review.
Original reviewer comment: #558 (comment)
Context:
rolesDatainsrc/domain/analysis/roles.tsuses dynamic SQL (conditions array varies per call), socachedStmtcan't apply directly. However, the static SQL inbuildTestFileIdsinmodule-map.tscould use thecachedStmtpattern. Explore caching strategies for the dynamic case as well.