Skip to content

CONN-5815 | CONN-5842 - Claude Compliance C2C Source and App docs#6713

Merged
amee-sumo merged 33 commits into
mainfrom
anthropiccompliancechats
May 21, 2026
Merged

CONN-5815 | CONN-5842 - Claude Compliance C2C Source and App docs#6713
amee-sumo merged 33 commits into
mainfrom
anthropiccompliancechats

Conversation

@sumoanema
Copy link
Copy Markdown
Collaborator

@sumoanema sumoanema commented May 19, 2026

Purpose of this pull request

This pull request documents new app and c2c source - Claude Compliance.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/CONN-5842
https://sumologic.atlassian.net/browse/CONN-5815

@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label May 19, 2026
@sumoanema sumoanema requested review from npande and parth-sumo May 19, 2026 13:04
* CONN-5815:created new doc for Anthropic Compliance App

* Update anthropic-compliance.md

* added additional files

* CONN-5815: added source configuration section

* Update anthropic-compliance.md

* Update anthropic-compliance.md

* minor changes

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Update anthropic-compliance.md

* Update index.md

* CONN-5815: Added the Monitor and updated the dashboards img

* CONN-5844: Rename anthropic compliance app to claude compliance app

---------

Co-authored-by: Parth Langalia <parth.langalia.ctr@parth.langalia.ctr-mac>
Co-authored-by: Amee Lepcha <amee.lepcha.ctr@sumologic.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@kimsauce kimsauce requested a review from a team as a code owner May 19, 2026 20:21
@kimsauce kimsauce changed the title Claude Compliance c2c source doc Claude Compliance C2C Source and App docs May 19, 2026
@kimsauce kimsauce changed the title Claude Compliance C2C Source and App docs CONN-5815 | CONN-5842 - Claude Compliance C2C Source and App docs May 19, 2026
Comment thread static/files/c2c/claude-compliance/example.json
Comment thread static/files/c2c/claude-compliance/example.json Outdated
Comment thread static/files/c2c/claude-compliance/example.tf
Comment thread static/files/c2c/claude-compliance/example.tf Outdated
… Claude Compliance docs

- Move all Claude Compliance images from img/send-data/ to img/integrations/saas-cloud/
- Rename C2C config screenshots with claude-compliance- prefix
- Fix grammar: "an Claude" → "a Claude" in source doc
- Fix alphabetical ordering of Claude Compliance in sidebar and index
- Add Claude Compliance Source link to product list; update icon size to width 40
- Update all image references in claude-compliance.md and claude-compliance-source.md

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kimsauce
Copy link
Copy Markdown
Collaborator

Latest commit: CONN-5815 image path and content corrections

  • Moved all Claude Compliance images from img/send-data/ to img/integrations/saas-cloud/ (correct location for app docs)
  • Renamed C2C config screenshots with claude-compliance- prefix for consistency
  • Fixed grammar: "an Claude" → "a Claude" in the source doc
  • Fixed alphabetical ordering of Claude Compliance in sidebars.ts and index.md
  • Added Claude Compliance Source link to the product list; updated icon size to width="40"
  • Updated all image references in claude-compliance.md and claude-compliance-source.md

— via Claude Code

Comment thread cid-redirects.json
Comment thread docs/integrations/saas-cloud/claude-compliance.md Outdated
Comment thread docs/integrations/saas-cloud/claude-compliance.md Outdated
Comment thread docs/integrations/saas-cloud/claude-compliance.md Outdated
sumoanema and others added 4 commits May 20, 2026 13:55
Copy link
Copy Markdown
Collaborator

@amee-sumo amee-sumo left a comment

Choose a reason for hiding this comment

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

Review Comments

Must Fix

  1. Broken image in product listdocs/integrations/product-list/product-list-a-l.md references img/send-data/claude-compliance.png but the icon was added at img/integrations/saas-cloud/claude-compliance.png. This will show a broken image.

  2. JSON/Terraform parameter naming mismatch — The C2C source doc's Configuration Object table lists organizationUUID and pollingInterval, but example.json and example.tf use orgUUID and pollingIntervalHour. These need to match what the backend actually expects.

  3. Terraform resource nameexample.tf uses "anthropic-compliance-source" as the resource name. Consider renaming to "claude-compliance-source" for consistency with the doc naming.

  4. Missing newline at end of file in:

    • blog-service/2026-05-11-apps.md
    • static/files/c2c/claude-compliance/example.json
    • static/files/c2c/claude-compliance/example.tf

Moderate

  1. Inconsistent vendor nameclaude-compliance-source.md says "contact your Claude representative" but claude-compliance.md says "contact your Anthropic representative." Should be "Anthropic" in both places (Anthropic is the company; Claude is the product).

  2. Run-on sentence in claude-compliance-source.md intro:

    "The Sumo Logic source for Claude Compliance enables you to collect chat messages data from Claude into Sumo Logic Claude provides advanced AI solutions..."

    Missing period after "Sumo Logic." — two sentences run together.

  3. Duplicate vendor configuration — The key creation steps (Console/API and Claude.ai sections) are duplicated verbatim in both docs. Consider having the C2C doc link to the app doc's vendor config section instead.

Minor / Style

  1. Grammar:

    • "Click the Create key to name your key" → "Click Create key to name your key" (remove "the")
    • "collect chat messages data" → "collect chat message data"
    • "Claude compliances Messages API" → "Claude Compliance Messages API"
  2. Trailing whitespace on line 7 of blog-service/2026-05-11-apps.md.

  3. Release note date — File is 2026-05-11-apps.md. Confirm this is the intended publish date.

  4. Index page description — "Monitor and manage your compliance and security" is quite generic. Suggest something more specific like: "Monitor Claude platform activities, security events, and compliance governance."


Overall the content is comprehensive and well-structured. Dashboard sections, monitors table, and sidebar/redirect placements all look good. The critical items to address before merge are #1 (broken image path) and #2 (parameter naming mismatch).

Comment thread static/files/c2c/claude-compliance/example.json
Comment thread static/files/c2c/claude-compliance/example.tf
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@amee-sumo amee-sumo left a comment

Choose a reason for hiding this comment

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

Updated Review (May 21)

Fixed from previous review ✓

  • Release note date corrected to 2026-05-21
  • Product list image path fixed (now uses correct img/integrations/saas-cloud/ path)
  • Run-on sentence in C2C source doc intro fixed
  • "Claude compliances Messages API" corrected
  • "Click the Create key" grammar fixed
  • Index page description improved
  • Dashboard sections now include structured "Use this dashboard to:" bullet points
  • Parameter names aligned between doc table and examples (orgUUID, pollingIntervalMinutes)

Remaining Issues

Must Fix

  1. Missing newline at end of file — Still present in:

    • blog-service/2026-05-21-apps.md
    • static/files/c2c/claude-compliance/example.json
    • static/files/c2c/claude-compliance/example.tf
  2. pollingIntervalMinutes type vs. value mismatch — The Configuration Object table says the type is Integer (in minutes) with default 5 minutes, but the example JSON/TF use the string "60m". If the type is truly an integer, examples should be 60 (no quotes, no m suffix). If the API accepts duration strings, update the type in the table.

  3. Terraform resource nameexample.tf still uses "anthropic-compliance-source". Should be "claude-compliance-source" for consistency with how the doc names the integration.

Minor

  1. Duplicate vendor configuration — Key creation steps are duplicated verbatim in both docs. Consider having the C2C doc reference the app doc's vendor config section.

  2. Double blank lines before screenshots in a few dashboard sections (API Key Monitoring, Data Access and Exfiltration Monitoring, Session Management) — inconsistent with the rest.

  3. Trailing whitespace on line 7 of blog-service/2026-05-21-apps.md (hide_table_of_contents: true ).


Verdict

Much improved. The critical items left are #1 (EOF newlines) and #2 (polling interval type mismatch). Would be good to merge after those are fixed.

@amee-sumo amee-sumo enabled auto-merge May 21, 2026 16:25
@amee-sumo amee-sumo added this pull request to the merge queue May 21, 2026
Merged via the queue into main with commit a590223 May 21, 2026
4 checks passed
@amee-sumo amee-sumo deleted the anthropiccompliancechats branch May 21, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants