fix(ci): keep schema auto-sync green when prod lags apps_with_preview

[riderx]

Summary (AI generated)

• Extend GlobalStatsSnapshotPatch with optional apps_with_preview so backend typecheck still passes when prod-generated types omit the new migration column
• Retry global_stats snapshot updates without apps_with_preview when PostgREST reports the column missing
• Soft-skip sync_schema_types auto-sync push (exit 0) when regenerated prod types fail typecheck, instead of reddening migration-touching releases

Motivation (AI generated)

sync_schema_types regenerates types from prod. Prod is still behind apps_with_preview, so regeneration dropped the column and bun typecheck failed on logsnag_insights.ts, failing Bump version after #2713 / #2937.

Business Impact (AI generated)

Unblocks release/auto-sync CI while preview QR daily stats remain written once the migration is on prod, and avoids core-shard failures if functions deploy before the column exists.

Test Plan (AI generated)

• bun run typecheck:backend passes with current types
• bun run typecheck:backend passes after temporarily removing apps_with_preview from backend generated types
• bunx vitest run tests/logsnag-insights-revenue.unit.test.ts (53 passed)
• Confirm next sync_schema_types run either syncs cleanly or soft-skips without failing Bump version

Generated with AI

  

---

^{Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.}

Summary by CodeRabbit

• Bug Fixes

  • Improved insights reporting to include preview-app metrics when available.
  • Added graceful handling for environments where preview-app metrics are not yet supported.
  • Preserved existing organization statistics updates during partial schema compatibility.

• Tests

  • Added coverage for optional preview-app metrics and missing-column handling.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: ff6914b1-c512-4161-9995-65ee534ff9f1

📥 Commits

Reviewing files that changed from the base of the PR and between fd9a223 and ce12145.

📒 Files selected for processing (3)

• .github/workflows/bump_version.yml
• supabase/functions/_backend/triggers/logsnag_insights.ts
• tests/logsnag-insights-revenue.unit.test.ts

---

📝 Walkthrough

Walkthrough

The snapshot trigger now supports optional apps_with_preview data and retries updates when its database column is unavailable. The version-bump workflow now treats type-check failures as non-blocking and restores generated files.

Changes

Preview metrics persistence

Layer / File(s)	Summary
Preview metric typing and persistence supabase/functions/_backend/triggers/logsnag_insights.ts, tests/logsnag-insights-revenue.unit.test.ts	Global snapshot types accept optional apps_with_preview values. Shared column-error detection supports build metrics and preview metrics. Snapshot updates retry without the preview field when its column is missing. Tests verify the new source behavior.

Typecheck workflow handling

Layer / File(s)	Summary
Non-blocking generated-file sync .github/workflows/bump_version.yml	The auto-sync step logs type-check failures, restores generated files, and exits successfully without committing or pushing.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• Cap-go/capgo.app#2937: Adds the apps_with_preview snapshot implementation that this change extends with typing, fallback handling, persistence, and tests.

Suggested labels: codex

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

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

[codspeed-hq]

Merging this PR will not alter performance

✅ 43 untouched benchmarks
⏩ 2 skipped benchmarks¹

---

_{Comparing cursor/fix-apps-with-preview-types-sync-d57e (ce12145) with main (fd9a223)}

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

[cursor]

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_291a1029-7c2b-4873-bfa1-5c0806930ce2)