Skip to content

Agent classification enrichment - #1822

Open
Ian Streeter (istreeter) wants to merge 1 commit into
mainfrom
agent-classification-enrichment
Open

Agent classification enrichment#1822
Ian Streeter (istreeter) wants to merge 1 commit into
mainfrom
agent-classification-enrichment

Conversation

@istreeter

Copy link
Copy Markdown
Contributor

What changed?

Why?

Reviewer guidance

AI reviews

Claude will automatically review this PR against the docs style guide.

If you have questions or want it to look again at something specific, tag @claude in a comment.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 10, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
documentation 80a9a80 Commit Preview URL

Branch Preview URL
Jun 10 2026, 09:55 AM

@claude

claude Bot commented Jun 10, 2026

Copy link
Copy Markdown

Style & writing review

This PR is well-written and compliant with the style guide and CLAUDE.md. Frontmatter is complete and correctly formatted, terminology is correct (Snowplow CDI, Console with no article, YAUAA, Enrich, entity/event/schema casing), headings use sentence case, lists follow the capitalization/no-terminal-period rules, all internal links end in /index.md and resolve, the SchemaProperties and Tabs/TabItem components are used correctly, and ✅/❌ are used appropriately in tables. The external iglucentral schema reference resolves. Self-Hosted as a tab label matches the existing convention across the other enrichment pages, so that is fine.

One minor, optional item:

For Self-Hosted, [provide a complete JSON]...

Using JSON as a standalone count noun ("a complete JSON") reads a little awkwardly. Consider "provide a complete JSON configuration" (or "JSON document") for clarity. Not a blocker.

Nothing else material to flag.

<Tabs groupId="deployment" queryString>
<TabItem value="console" label="Console" default>

Configure the parameters in the Console enrichment editor. Keep the Console defaults for the `database` and `uri` fields. For example:

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.

This pending console support.


| Field | Type | Description |
| --- | --- | --- |
| `uri` | string | Base URI where the file is hosted. Supports `http:`, `s3:`, and `gs:` schemes. Must not end with a trailing slash. |

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.

abfs also.

"database": "<use default value from Console>"
},
"classification": {
"MyInternalBot": {"operator": "Acme Corp", "purpose": "Monitoring"}

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.

Suggested change
"MyInternalBot": {"operator": "Acme Corp", "purpose": "Monitoring"}
"SomeBot": {"operator": "Acme Corp", "purpose": "Monitoring"}

That's a detail but IMO MyInternalBot could let people think that it is possible to add our own bot somewhere, whereas it is not possible as the value comes from YAUAA.

@stanch Nick (stanch) added the do not merge Flag to denote a Issue or PR which should not yet be merged (usually pending a release) label Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Flag to denote a Issue or PR which should not yet be merged (usually pending a release)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants