fix(file-search): bound dispatcher database work

[icecrasher321]

Summary

• Bound search-dispatch preparation with transaction-local database deadlines, using PostgreSQL 17's native transaction timeout, matching the documented deployment baseline.
• Add phase timings and PostgreSQL error codes. Preserve claim cleanup under contention and retain both failures if enqueue and cleanup fail.
• Keep raw revision timestamps in UTC so failed enqueue cleanup matches the claimed rows, and register the database regressions in CI.

Type of Change

• Bug fix

Testing

13 focused unit tests passed. All 3 integration tests passed on PostgreSQL 17 in a non-UTC process timezone, covering cancellation, rollback, advisory-lock release, and claim cleanup with preparation deadlines cleared. App type-check, lint, API validation, repository audits, and generated-artifact checks passed.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Sep 17, 2026 3:03am UTC

[cubic-dev-ai]

All reported issues were addressed across 4 files

<sub>Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic</sub>

[greptile-apps]

Confidence Score: 5/5

The PR appears safe to merge; the previous findings are resolved and no new actionable failures remain.

Summary

Bounds file-search dispatch preparation with transaction-local PostgreSQL deadlines while preserving enqueue-failure cleanup and diagnostic context.

• Applies statement, lock, and transaction deadlines before acquiring the dispatcher advisory lock.
• Preserves UTC revision identity and retains both enqueue and cleanup errors when both operations fail.
• Adds PostgreSQL 17 integration coverage for cancellation, rollback, advisory-lock release, claim cleanup, and timeout isolation.
• Replaces the cleanup contention test’s real-time delay with deterministic inspection of the cleanup transaction’s active timeout settings.

_{Reviews (4) · Last reviewed commit: "test(file-search): observe cleanup deadl..."}

[icecrasher321]

@greptile

[icecrasher321]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@icecrasher321 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 5 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}

[icecrasher321]

@greptile

[icecrasher321]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@icecrasher321 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 5 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}

[icecrasher321]

@greptile

[icecrasher321]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@icecrasher321 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 5 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}