fix: Bulk file query

[yuvrajjsingh0]

Summary by CodeRabbit

• Refactor
  • Optimized file lookup query strategy for improved database performance and efficiency.

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
airborne_server/src/release/utils.rs	45% smaller
airborne_server/src/utils/db/models.rs	6% smaller
airborne_dashboard/package-lock.json	0% smaller

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

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.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a16b73d9-0cb4-4911-b70d-458a85960701

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

Use the checkbox below for a quick retry:

• 🔍 Trigger review

Walkthrough

FileEntry gains the QueryableByName derive. get_files_by_file_keys_async is rewritten to replace per-key Diesel OR predicates with a single sql_query that binds versioned and tagged file keys as SQL arrays, uses unnest() joins, and combines results with UNION, scoped by org_id and app_id. Invalid key formats now return a bad-request error.

Changes

File-key lookup rewrite

Layer / File(s)	Summary
Add QueryableByName to FileEntry airborne_server/src/utils/db/models.rs	FileEntry's #[derive(...)] list gains QueryableByName, enabling Diesel to deserialize sql_query results into this struct.
Rewrite get_files_by_file_keys_async with sql_query + unnest() airborne_server/src/release/utils.rs	Diesel imports are updated from BoxableExpression/Bool to sql_query/Array/Integer/Text; schema DSL imports are removed. get_files_by_file_keys_async now partitions file_paths into versioned and tagged key arrays, rejects invalid formats with ABError::BadRequest, and issues a single sql_query using unnest() joins combined via UNION constrained by org_id and app_id.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 Hop, hop, no more loops of OR,
Arrays and unnest do the chore!
Two key types split with care,
One UNION joins them there.
The rabbit's queries soar! 🚀

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: Bulk file query' directly aligns with the main change: rewriting the bulk file query lookup strategy in get_files_by_file_keys_async from Diesel predicates to SQL unnest/array approach.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-bulk-file-query

---

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.}