[codex] Drop unused channel devices owner org index

[riderx]

Summary (AI generated)

• Adds a Supabase migration to drop public.finx_channel_devices_owner_org.
• Keeps the change scoped to the unused channel_devices.owner_org index only.

Motivation (AI generated)

Live index stats showed finx_channel_devices_owner_org had only a handful of scans after the channel_devices reindex, and code search found runtime query paths use app_id, device_id, or channel_id instead of owner_org as a leading lookup.

Business Impact (AI generated)

This removes a low-value index from a churn-heavy table, reducing write overhead and keeping database maintenance focused on the indexes used by customer-facing channel/device flows.

Test Plan (AI generated)

• Ran bun run lint:backend.
• Ran bun run lint.
• Ran bun typecheck.
• Ran bunx supabase migration list --linked and confirmed this migration is the only local pending migration.

Screenshots (AI generated)

Not applicable. Database migration only.

Checklist (AI generated)

• My code follows the code style of this project and passes bun run lint:backend && bun run lint.
• My change does not require a change to the documentation.
• My change does not need E2E test coverage because it only removes an unused database index.
• I have tested my code manually, and I have provided steps how to reproduce my tests.

Generated with AI

Summary by CodeRabbit

• Chores
  • Removed an index from the channel devices table to optimize database performance and update runtime lookup patterns.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: e226f166-5960-48c6-a56d-57856d1d844e

📥 Commits

Reviewing files that changed from the base of the PR and between 3227120 and 1a083a2.

📒 Files selected for processing (2)

• read_replicate/schema_replicate.sql
• supabase/migrations/20260524123635_drop_channel_devices_owner_org_index.sql

💤 Files with no reviewable changes (1)

• read_replicate/schema_replicate.sql

---

📝 Walkthrough

Walkthrough

This PR removes the finx_channel_devices_owner_org index from the channel_devices table. A Supabase migration drops the index from production databases, and the schema definition file is updated to prevent its recreation during future database initialization.

Changes

Index Removal

Layer / File(s)	Summary
Remove finx_channel_devices_owner_org index supabase/migrations/20260524123635_drop_channel_devices_owner_org_index.sql, read_replicate/schema_replicate.sql	Migration drops the finx_channel_devices_owner_org index due to changing lookup patterns for channel_devices. Schema file removes the index definition to prevent recreation on new database initialization.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

codex

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: dropping an unused index on the channel_devices table named finx_channel_devices_owner_org, which matches the changeset.
Description check	✅ Passed	The description includes all required sections (Summary, Test plan, Screenshots, Checklist) with substantive content and demonstrates the PR was tested thoroughly before submission.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.}

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

---

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

[codspeed-hq]

Merging this PR will not alter performance

✅ 43 untouched benchmarks
⏩ 2 skipped benchmarks¹

---

_{Comparing codex/drop-channel-devices-owner-org-index (1a083a2) with main (3227120)}

Footnotes

1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.