Dev to Main

[lakshayman]

PRs going in #243

Description by Korbit AI

What change is being made?

Refactor Lambda handlers to use an injected Firestore client through a new deps struct, update main handlers to initialize and pass the client, and expand tests and test scaffolding to support integration tests, health checks, and verification flows across multiple modules (with updated test script and environment handling); also broaden .gitignore and minor test scaffolding changes.

Why are these changes being made?

Introduce dependency injection for Firestore to improve testability and reusability, and extend test coverage to exercise integration paths (including Firebase emulator flows) across health-check, call-profile, call-profiles, and verify modules. This enables more robust end-to-end validation while keeping existing code paths intact.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch develop

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[korbit-ai]

Review by Korbit AI

Korbit automatically attempts to detect when you fix issues in new commits.

Category	Issue	Status
	Missing port range rationale ▹ view
	Inefficient polling with fixed sleep ▹ view
	Inefficient array management in polling loop ▹ view
	Random port generation can cause collisions in parallel execution ▹ view
	Unclear Concurrency Limit Rationale ▹ view
	Extract Parallel Execution Logic ▹ view
	Lack of proper dependency injection ▹ view

Files scanned

​

File Path	Reviewed
verify/main.go	✅
health-check/main.go	✅
call-profiles/main.go	✅
scripts/test.sh	✅
call-profile/main.go	✅

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X