Skip to content

chore(analytics): remove Profound request tracking - #7326

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/remove-profound-analytics
Sep 1, 2026
Merged

chore(analytics): remove Profound request tracking#7326
waleedlatif1 merged 1 commit into
stagingfrom
codex/remove-profound-analytics

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Remove Profound request log batching and the proxy hook
  • Drop the unused analytics environment configuration

Type of Change

  • Maintenance

Testing

  • bun run lint
  • bun run apps/sim/scripts/check-block-registry.ts origin/staging
  • bun run check:audits
  • bun run --cwd apps/sim test proxy.test.ts

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

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 1, 2026 12:28am UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 3 files

Confidence score: 5/5

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

Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This maintenance PR removes Profound request-log collection while preserving the proxy’s existing indexing policy.

  • Deletes the Profound analytics buffering and export module.
  • Removes the unused Profound environment-schema entries.
  • Replaces the tracking wrapper with direct indexing-policy application on every proxy response path.

Confidence Score: 5/5

The PR appears safe to merge with no actionable correctness, security, or repository-rule issues identified.

The deleted analytics module and environment keys have no remaining references, and every proxy branch continues to return the same response after applying the unchanged indexing policy.

Important Files Changed

Filename Overview
apps/sim/lib/analytics/profound.ts Deletes the Profound request batching implementation; no remaining repository references to its module or exports were found.
apps/sim/lib/core/config/env.ts Removes the two optional Profound analytics environment properties, with no remaining typed consumers.
apps/sim/proxy.ts Removes Profound tracking while preserving response identity, status, return paths, and indexing-policy mutations.

Reviews (1): Last reviewed commit: "chore(analytics): remove Profound reques..." | Re-trigger Greptile

@waleedlatif1
waleedlatif1 merged commit 898ecdb into staging Sep 1, 2026
21 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/remove-profound-analytics branch September 1, 2026 00:32
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.

1 participant