feat: add first-party Tinybird analytics - #2003
Closed
richiemcilroy wants to merge 151 commits into
Closed
Conversation
richiemcilroy
marked this pull request as ready for review
July 12, 2026 15:56
Member
Author
|
hey @greptileai, please re-review the PR |
Member
Author
|
hey @greptileai, please re-review the PR |
Member
Author
|
hey @greptileai, please re-review the PR |
Member
Author
|
hey @greptileai, please re-review the PR |
| --artifact "${{ steps.checkpoint.outputs.artifact }}" | ||
| - name: Upload redacted containment evidence | ||
| if: always() && steps.checkpoint.outputs.artifact != '' | ||
| uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 |
There was a problem hiding this comment.
P1: actions/upload-artifact@ea165f8 has 5 supply-chain risks
Superagent correlated 5 risks for this dependency. Top signal: Detects suspicious autostart persistence mechanisms. Location: upload-artifact-ea165f8d65b6e75b540449e92b4886f43607f…
Review actions/upload-artifact@ea165f8 before merging. Remove or replace it if this behavior is unexpected.
AI prompt
Check if this security scanner issue is valid. If so, understand the root cause and fix it. If appropriate, update or add tests. Keep the change focused and preserve intended behavior.
<file name=".github/workflows/analytics-recovery-30731913704.yml">
<violation number="1" location=".github/workflows/analytics-recovery-30731913704.yml:141">
<priority>P1</priority>
<title>actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 has 5 supply-chain risks</title>
<evidence>Superagent correlated 5 risks for this dependency. Top signal: Detects suspicious autostart persistence mechanisms. Location: upload-artifact-ea165f8d65b6e75b540449e92b4886f43607fa02/dist/merge/index.js. Rules: threat-filesystem-autostart. Rules: threat-filesystem-autostart, threat-runtime-obfuscation-base64exec, threat-runtime-obfuscation-steganography
Dependency path: .github/workflows/analytics-recovery-30731913704.yml → jobs → recover → steps → 9 → actions/upload-artifact</evidence>
<recommendation>Review actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 before merging. Remove or replace it if this behavior is unexpected.</recommendation>
</violation>
</file>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Validation
pnpm analytics:test— 152 tests passedpnpm --filter @cap/web test --run __tests__/unit/mobile-checkout.test.ts— 7 tests passedpnpm typecheckcargo fmt --allcargo check -p cap-desktopwith the repository FFmpeg 7 toolchaincargo test -p cap-desktop product_analytics --lib— 9 tests passedGreptile Summary
This PR replaces PostHog with first-party Tinybird product analytics. The main changes are:
Confidence Score: 4/5
This is close, but the remaining replay path should be fixed before merging.
Files Needing Attention: apps/web/app/api/events/route.ts
Security Review
The anonymous analytics endpoint still accepts replayed browser tokens for allowlisted web events, which can let raw clients append fabricated anonymous product events.
Important Files Changed
Prompt To Fix All With AI
Reviews (13): Last reviewed commit: "fix: normalize Tinybird health query tim..." | Re-trigger Greptile
Context used: