CONN-5815 | CONN-5842 - Claude Compliance C2C Source and App docs#6713
Conversation
* 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>
… 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>
|
Latest commit: CONN-5815 image path and content corrections
— via Claude Code |
Co-authored-by: Kim Pohas <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim Pohas <56411016+kimsauce@users.noreply.github.com>
amee-sumo
left a comment
There was a problem hiding this comment.
Review Comments
Must Fix
-
Broken image in product list —
docs/integrations/product-list/product-list-a-l.mdreferencesimg/send-data/claude-compliance.pngbut the icon was added atimg/integrations/saas-cloud/claude-compliance.png. This will show a broken image. -
JSON/Terraform parameter naming mismatch — The C2C source doc's Configuration Object table lists
organizationUUIDandpollingInterval, butexample.jsonandexample.tfuseorgUUIDandpollingIntervalHour. These need to match what the backend actually expects. -
Terraform resource name —
example.tfuses"anthropic-compliance-source"as the resource name. Consider renaming to"claude-compliance-source"for consistency with the doc naming. -
Missing newline at end of file in:
blog-service/2026-05-11-apps.mdstatic/files/c2c/claude-compliance/example.jsonstatic/files/c2c/claude-compliance/example.tf
Moderate
-
Inconsistent vendor name —
claude-compliance-source.mdsays "contact your Claude representative" butclaude-compliance.mdsays "contact your Anthropic representative." Should be "Anthropic" in both places (Anthropic is the company; Claude is the product). -
Run-on sentence in
claude-compliance-source.mdintro:"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.
-
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
-
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"
-
Trailing whitespace on line 7 of
blog-service/2026-05-11-apps.md. -
Release note date — File is
2026-05-11-apps.md. Confirm this is the intended publish date. -
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).
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
amee-sumo
left a comment
There was a problem hiding this comment.
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
-
Missing newline at end of file — Still present in:
blog-service/2026-05-21-apps.mdstatic/files/c2c/claude-compliance/example.jsonstatic/files/c2c/claude-compliance/example.tf
-
pollingIntervalMinutestype vs. value mismatch — The Configuration Object table says the type isInteger(in minutes) with default5 minutes, but the example JSON/TF use the string"60m". If the type is truly an integer, examples should be60(no quotes, nomsuffix). If the API accepts duration strings, update the type in the table. -
Terraform resource name —
example.tfstill uses"anthropic-compliance-source". Should be"claude-compliance-source"for consistency with how the doc names the integration.
Minor
-
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.
-
Double blank lines before screenshots in a few dashboard sections (API Key Monitoring, Data Access and Exfiltration Monitoring, Session Management) — inconsistent with the rest.
-
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.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ling interval to string
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…c2c - in c2c doc as well as app doc
Purpose of this pull request
This pull request documents new app and c2c source - Claude Compliance.
Select the type of change
Ticket (if applicable)
https://sumologic.atlassian.net/browse/CONN-5842
https://sumologic.atlassian.net/browse/CONN-5815