Skip to content

Optimize web schema indexes#143

Merged
arjunkomath merged 1 commit into
mainfrom
codex/web-schema-indexes
Jul 15, 2026
Merged

Optimize web schema indexes#143
arjunkomath merged 1 commit into
mainfrom
codex/web-schema-indexes

Conversation

@techulus-agent

Copy link
Copy Markdown
Collaborator

Summary

  • remove redundant and low-value indexes from deployments, builds, GitHub repositories, and service revisions
  • add missing foreign-key lookup indexes for service-owned resources and GitHub installations
  • align backup, rollout, build, and work-queue indexes with their filtering and ordering patterns

Why

The schema mixed redundant indexes with gaps around common relationship lookups and history queries. This reduces unnecessary index maintenance while improving the paths used to load service resources, recent history, backups, and queued work.

Impact

This changes database index definitions only. It does not change application behavior and intentionally does not include a database push or generated migration.

Validation

  • pnpm exec biome check db/schema.ts
  • pnpm exec tsc --noEmit
  • pnpm test — 21 files and 115 tests passed
  • git diff --check

@github-actions

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit c34303b.

@arjunkomath arjunkomath marked this pull request as ready for review July 15, 2026 07:37

@arjunkomath arjunkomath left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the index changes against the actual Drizzle query patterns. The added FK/composite indexes match service-resource, history, backup, build, and queue lookups; removed indexes are redundant or low-value. Schema-only delivery also matches the repository's drizzle-kit push workflow. No blocking findings.

CI is green, including build, CodeQL, and React Doctor.

@arjunkomath arjunkomath merged commit 262f7f5 into main Jul 15, 2026
9 checks passed
@arjunkomath arjunkomath deleted the codex/web-schema-indexes branch July 15, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants