Skip to content

docs(laravel): Add AI agent monitoring onboarding#118363

Open
constantinius wants to merge 1 commit into
masterfrom
constantinius/docs/laravel/laravel-ai-agent-monitoring-onboarding
Open

docs(laravel): Add AI agent monitoring onboarding#118363
constantinius wants to merge 1 commit into
masterfrom
constantinius/docs/laravel/laravel-ai-agent-monitoring-onboarding

Conversation

@constantinius

@constantinius constantinius commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Adds Laravel-specific agent monitoring onboarding for laravel/ai, including setup for sentry/sentry-laravel, tracing, prompt/response capture, and an example Laravel AI agent/tool. The agent monitoring and Conversations onboarding flows now recognize php-laravel as a supported platform and show Laravel-specific conversation guidance using Conversational, RemembersConversations, and ->continue().

Screenshot 2026-06-24 at 19 14 30 Screenshot 2026-06-24 at 19 14 58 Screenshot 2026-06-24 at 19 15 15
Screenshot 2026-06-24 at 19 15 42 Screenshot 2026-06-24 at 19 16 00 Screenshot 2026-06-24 at 19 16 14 Screenshot 2026-06-24 at 19 16 27

Closes TET-2515

@constantinius constantinius requested review from a team as code owners June 24, 2026 16:47
@linear-code

linear-code Bot commented Jun 24, 2026

Copy link
Copy Markdown

TET-2515

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 24, 2026
projectId={params.project.id}
minVersion={MIN_REQUIRED_VERSION}
packageName="sentry/sentry-laravel"
/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Laravel SDK alert never shows

Low Severity

The Laravel agent monitoring intro uses SdkUpdateAlert with packageName sentry/sentry-laravel, but SdkUpdateAlert only maps Python and JavaScript SDK names to package names. Laravel project SDK rows never match, so the minimum-version warning never appears for php-laravel projects.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 77e4af9. Configure here.

@constantinius constantinius force-pushed the constantinius/docs/laravel/laravel-ai-agent-monitoring-onboarding branch from 77e4af9 to d09a42c Compare June 24, 2026 17:06

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

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.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

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

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d09a42c. Configure here.

),
...(isPhpPlatform
? [getPhpConversationVerifyStep()]
: agentMonitoringDocs.verify?.(docParams) || []),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PHP conversations skips verify

Medium Severity

For php-laravel projects, Conversations onboarding replaces agentMonitoringDocs.verify with only getPhpConversationVerifyStep(), so the guided /ai-test route and initial prompt() never appear. The final step uses ->continue('my-conversation-123', …) without any prior step that creates that conversation, so the flow cannot be completed from the docs alone.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit d09a42c. Configure here.

@constantinius constantinius force-pushed the constantinius/docs/laravel/laravel-ai-agent-monitoring-onboarding branch from d09a42c to 1bb5088 Compare June 24, 2026 17:10
@constantinius constantinius force-pushed the constantinius/docs/laravel/laravel-ai-agent-monitoring-onboarding branch from 1bb5088 to 2d681cc Compare June 24, 2026 17:13
Add Laravel-specific agent monitoring setup instructions for laravel/ai and expose php-laravel as a supported platform for the agent monitoring and conversations onboarding flows.

Refs TET-2515
@constantinius constantinius force-pushed the constantinius/docs/laravel/laravel-ai-agent-monitoring-onboarding branch from 2d681cc to 29f3501 Compare June 24, 2026 17:14

@cleptric cleptric left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is way too much text for "update SDK". This should all work by default. Conversation IDs are the exception, this part is good to have. Options that are enabled by default can be omitted. I would generally boil this down massively.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants