Skip to content

Document AI Shopping Agent events in README - #289

Merged
pedro-lb merged 2 commits into
masterfrom
docs/readme-asa-events
Oct 21, 2026
Merged

pedro-lb merged 2 commits into
masterfrom
docs/readme-asa-events

Conversation

@pedro-lb

Copy link
Copy Markdown
Contributor

What

Adds an "AI Shopping Agent Events" subsection to "12. Instrument Behavioral Events". It shows the six trackAgent* methods that shipped in v4.7.0 (#278), with the signatures from ConstructorIO.swift.

Why

The README had no ASA section, so people could not tell that the iOS client tracks ASA events. README only. No code changes.

Tracker: https://github.com/pedro-lb/constructor-area-51/blob/main/work-tracker/entries/20261020-agentic-sdk-docs-findability/README.md

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 20, 2026 16:57
@pedro-lb
pedro-lb requested a review from a team as a code owner October 20, 2026 16:57

Copilot AI 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.

🟢 Approval recommended

The documentation matches the existing public APIs, with no unresolved findings.

No open issues 🎉

What changed in this PR

Documents the existing AI Shopping Agent tracking APIs so iOS SDK users can discover and use them. No code changes.

Changes:

  • Adds AI Shopping Agent events to the behavioral event list.
  • Provides examples for all six trackAgent* methods, version availability, and result ID guidance.
File Description
README.md Adds AI Shopping Agent event documentation and usage examples.

Review effort: Balanced

constructor-claude-bedrock[bot]

This comment was marked as outdated.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@constructor-claude-bedrock constructor-claude-bedrock 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.

Code Review

This PR adds a README section documenting the six trackAgent* methods introduced in v4.7.0, which is a welcome and needed addition. The examples are mostly correct and follow the style of surrounding sections, but there are a few accuracy and clarity issues worth addressing before merging.

Inline comments: 4 discussions added

Overall Assessment: ⚠️ Needs Work

Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread README.md

@Mudaafi Mudaafi 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.

thanks for adding this!

@pedro-lb
pedro-lb merged commit 6241571 into master Oct 21, 2026
3 checks passed
@pedro-lb
pedro-lb deleted the docs/readme-asa-events branch October 21, 2026 19:21
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.

3 participants