Skip to content

gather data from user submitted long_call.txt files#1695

Draft
sandboxcoder wants to merge 1 commit into
stagingfrom
rno/profile-long-calls
Draft

gather data from user submitted long_call.txt files#1695
sandboxcoder wants to merge 1 commit into
stagingfrom
rno/profile-long-calls

Conversation

@sandboxcoder
Copy link
Copy Markdown
Contributor

@sandboxcoder sandboxcoder commented May 12, 2026

Description

Motivation and Context

Check user submitted logs to AWS S3 bucket and generate a report.

How Has This Been Tested?

Ran it manually over a 24 hour period.

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a standalone TypeScript utility under profile-long-calls/ to download user-cache zips from S3, extract long_calls.txt, ingest the data into a local SQLite DB, and print aggregate “worst offender” IPC-call performance reports.

Changes:

  • Introduces long-calls-analyzer.ts (S3 listing/download, zip extraction, SQLite ingest, and reporting).
  • Adds local tooling/config for building/running/debugging the analyzer (tsconfig, package.json, VS Code launch config, Yarn config).
  • Updates .gitignore to exclude build output (dist/) and the generated SQLite DB.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
profile-long-calls/tsconfig.json TypeScript compiler config for building the analyzer into dist/.
profile-long-calls/README.md Documents purpose and usage for the analyzer.
profile-long-calls/package.json Declares dependencies and run/build scripts for the analyzer.
profile-long-calls/long-calls-analyzer.ts Core implementation: S3 fetch, unzip, SQLite ingest, and stats output.
profile-long-calls/.yarnrc.yml Yarn configuration for installing/running the tool in this subfolder.
profile-long-calls/.vscode/launch.json Debug configuration for running the analyzer with ts-node.
.gitignore Ignores generated analyzer artifacts (dist/, SQLite DB files).
Comments suppressed due to low confidence (1)

profile-long-calls/long-calls-analyzer.ts:396

  • Same as above: affected_users is computed as COUNT(DISTINCT source_id), which counts sources/zips rather than users. If the output is intended to be per-user, join sources and count distinct username (handling NULLs appropriately) or rename the output to avoid implying it’s user count.
        `SELECT
          call_name,
          SUM(is_freeze)            AS freezes,
          COUNT(DISTINCT source_id) AS affected_users,
          MAX(total_ms)             AS max_ms,
          ROUND(AVG(total_ms))      AS avg_ms

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread profile-long-calls/README.md Outdated
Comment thread profile-long-calls/README.md Outdated
Comment thread profile-long-calls/package.json
Comment thread profile-long-calls/long-calls-analyzer.ts Outdated
Comment thread profile-long-calls/long-calls-analyzer.ts Outdated
Comment thread profile-long-calls/long-calls-analyzer.ts Outdated
Comment thread profile-long-calls/long-calls-analyzer.ts
Comment thread profile-long-calls/long-calls-analyzer.ts Outdated
Comment thread profile-long-calls/long-calls-analyzer.ts Outdated
* Analyzes the long_calls.txt file produced by obs-studio-node and helps
identify which methods takes the longest.
* Asana: https://app.asana.com/1/1083097041131/project/1207748235152476/task/1214564097102925
@sandboxcoder sandboxcoder force-pushed the rno/profile-long-calls branch from 41a9a46 to a73522d Compare May 13, 2026 01:19
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