Skip to content

ref: remove AgentClient from agent repo#37

Merged
Litarnus merged 6 commits intomainfrom
cleanup-agent-client
May 6, 2026
Merged

ref: remove AgentClient from agent repo#37
Litarnus merged 6 commits intomainfrom
cleanup-agent-client

Conversation

@Litarnus
Copy link
Copy Markdown
Contributor

Removes the AgentClient from this repo because it was moved to the PHP SDK.

The structure remained unchanged for now

Comment thread agent/composer.json Outdated
Comment thread composer.json
Comment thread composer.json
Comment on lines 14 to +15
"php": "^7.2|^8",
"ext-json": "*",
"sentry/sentry": "^4.19.1"
},
"autoload": {
"psr-4": {
"Sentry\\Agent\\": "src/"
}
"ext-json": "*"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: The composer.json for sentry/sentry-agent is missing a required dependency on sentry/sentry, which will cause a fatal error when AgentClientBuilder is not found.
Severity: CRITICAL

Suggested Fix

Add "sentry/sentry": "^4.19.1" back to the require section of the composer.json file. This will ensure that Composer correctly installs the sentry/sentry package, making the AgentClientBuilder class available.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: composer.json#L14-L15

Potential issue: The `composer.json` for the `sentry/sentry-agent` package omits a
required dependency on `sentry/sentry`. The `AgentClientBuilder` class, essential for
using the agent, was moved to the `sentry/sentry` package. Without this declared
dependency, Composer will not automatically install `sentry/sentry`. This will lead to a
fatal "Class not found" error for `AgentClientBuilder` at runtime if a user only
installs the `sentry/sentry-agent` package, breaking the package's functionality upon
installation.

Did we get this right? 👍 / 👎 to inform future reviews.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit cff8e42. Configure here.

Comment thread agent/composer.json Outdated
@Litarnus Litarnus merged commit a79d8fb into main May 6, 2026
45 checks passed
@Litarnus Litarnus deleted the cleanup-agent-client branch May 6, 2026 08:57
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