Skip to content

Conversation

@lmcdowell
Copy link
Contributor

Description

Adding the ability to pull crash groups.

Added using https://app.bugsplat.com/api/keycrash?database={DATABASE}&stackKeyId={GROUP_NUMBER} api call.

Checklist

  • Tested manually
  • Unit tests pass with no errors or warnings
  • Documentation updated (if applicable)
  • Reviewed by at least 1 other contributor

@bobbyg603
Copy link
Member

Thanks for the PR @lmcdowell ! I looked over this briefly and things look good, we'll do a full review soon.

@bobbyg603 bobbyg603 changed the title feat:adding get-key-crashes feat: adding get-key-crashes Sep 23, 2025
@bobbyg603 bobbyg603 requested review from bobbyg603 and Copilot and removed request for Copilot September 23, 2025 22:28
Copy link

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

This PR adds functionality to retrieve and format crash groups by Stack Key ID through a new get-key-crashes tool. The feature allows users to analyze patterns within specific crash types by listing all individual crashes that belong to the same crash group.

  • Implements getKeyCrashes function to fetch crashes for a specific Stack Key ID using the BugSplat API
  • Adds formatKeyCrashesOutput function to format crash data into readable text output
  • Integrates the new functionality as a server tool with input validation and error handling

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/key-crash.ts Core implementation of key crash functionality with API client integration and output formatting
src/index.ts Server tool registration with parameter validation and error handling
spec/key-crash.spec.ts Comprehensive test coverage including integration tests and edge case handling
README.md Documentation update describing the new get-key-crashes tool and its parameters

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@bobbyg603
Copy link
Member

I ran the tests locally and everything passed. An action triggered by a forked repo can't use secrets which makes sense, so I'm going to override our checks and admin merge this.

@bobbyg603 bobbyg603 merged commit 939d584 into BugSplat-Git:main Sep 23, 2025
1 of 2 checks passed
@bobbyg603
Copy link
Member

Thanks for the PR @lmcdowell 🚀

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