Skip to content

[SDK-398] Ai integration files#1043

Merged
franco-zalamena-iterable merged 8 commits intomasterfrom
ai-integration-files
Apr 16, 2026
Merged

[SDK-398] Ai integration files#1043
franco-zalamena-iterable merged 8 commits intomasterfrom
ai-integration-files

Conversation

@franco-zalamena-iterable
Copy link
Copy Markdown
Contributor

@franco-zalamena-iterable franco-zalamena-iterable commented Apr 8, 2026

🔹 Jira Ticket(s) if any

✏️ Description

Have a guideline for users integrating our sdk, the integration guide can be provided manually or through the Iterable MCP.

…fixes

- Auth handler must read email fresh from source of truth on every
  onAuthTokenRequested() call. Caching at startup causes AUTH_TOKEN_NULL.
- Added full runtime POST_NOTIFICATIONS permission code for Android 13+.
  Without it, push silently fails.
- Updated traps list from 10 to 12 with both new issues.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Comment thread docs/ai/integration-guide.md Outdated
.setCustomActionHandler(AppCustomActionHandler())
.setUrlHandler(AppUrlHandler(this))
.setAllowedProtocols(arrayOf("yourscheme")) // REQUIRED for custom deep link schemes
.setAuthHandler(AppAuthHandler { getFreshEmail() }) // REQUIRED if JWT key — see note below
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.

Shouldn't this be generateJwtForEmail(getFreshEmail()) instead? Because onAuthTokenRequested() must return a JWT token and not an email right?

Suggested change
.setAuthHandler(AppAuthHandler { getFreshEmail() }) // REQUIRED if JWT key — see note below
.setAuthHandler(AppAuthHandler { generateJwtForEmail(getFreshEmail()) }) // REQUIRED if JWT key — see note below

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed

Comment thread CLAUDE.md
@@ -0,0 +1,22 @@
# Iterable Android SDK
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.

Would it make sense to rename this file to AGENTS.md so that all AI agents could read it besides Claude?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am analyzing this on the following ticket, apparently we should use llms.txt and llms-full.txt as well as CLAUDE.md.

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.

Can you link the ticket here too please?

@franco-zalamena-iterable franco-zalamena-iterable changed the title Ai integration files [SDK-398] Ai integration files Apr 16, 2026
@franco-zalamena-iterable franco-zalamena-iterable merged commit 5e44cba into master Apr 16, 2026
9 of 11 checks passed
@franco-zalamena-iterable franco-zalamena-iterable deleted the ai-integration-files branch April 16, 2026 11:41
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