fix: address review feedback - move test to test/, add test-reports/ to .gitignore, fix rate-limit header check

- Move rest-api.test.ts from src/__tests__/ to test/ per repo conventions
- Add test-reports/ to .gitignore to prevent committing generated reports
- Fix rate-limit header check: toBeDefined() doesn't catch null from Headers.get()
  Record missing headers as schema violation instead of hard-failing

Co-Authored-By: Chris K <ckorhonen@gmail.com>

Author: devin-ai-integration[bot]

.gitignore

@@ -3,3 +3,4 @@ dist
 .env
 *.tgz
 .tsbuildinfo
+test-reports/

src/__tests__/rest-api.test.ts test/rest-api.test.tssrc/__tests__/rest-api.test.ts renamed to test/rest-api.test.ts

@@ -1170,9 +1170,9 @@ describeIfLive(
             res.headers.get("ratelimit-remaining") ??
             res.headers.get("x-ratelimit-limit") ??
             res.headers.get("ratelimit-limit")
-          // Rate limiting headers should be present
-          expect(rateLimitHeader).toBeDefined()
-          record("headers.rateLimit", "GET", path, res)
+          // Record whether rate-limit headers are present (not all endpoints expose them)
+          const violations = rateLimitHeader === null ? ["no rate-limit headers found"] : []
+          record("headers.rateLimit", "GET", path, res, violations)
         } catch (e) {
           recordError("headers.rateLimit", "GET", path, e)
           throw e