chore: CLAUDE.md proposals for 5 repos with recent merged PRs (May–Jun 2026)#6
Open
shrivastavakapil2000 wants to merge 2 commits into
Open
chore: CLAUDE.md proposals for 5 repos with recent merged PRs (May–Jun 2026)#6shrivastavakapil2000 wants to merge 2 commits into
shrivastavakapil2000 wants to merge 2 commits into
Conversation
Reviews merged PRs (May–June 2026) from SayaliPat, shrivastavakapil2000, JoeVsVolcano, mike-brant, and nathan-resonate across the organization and produces CLAUDE.md files capturing current architecture, key concepts, and recent changes for: - step-function-workflow-orchestrator (new) — EMR 7 migration, 12 pipelines, dynamic dates, integration test patterns - batch-expression-modeling (update) — formatter metrics lambda, stitch throttle fix, delta_with_full_fallback fix, new formatter path layout - identity-graph (new) — PRISM design, 11 Spark jobs, shared utilities - batch-audience-delivery-syndication (new) — OpenX/BlockGraph lambdas, csv.gz extension fix, source path partition order - dos-data-pipeline (new) — district_source provenance, IP-inferred fallback, zip→district CSV namespace requirements https://claude.ai/code/session_0121mjajLMWrWCnCUnjvKWZH
There was a problem hiding this comment.
Pull request overview
This PR stages CLAUDE.md documentation updates (generated from recent May–June 2026 merged PR activity) for five other repositories under claude-md-updates/, along with a local README describing how to copy/apply them in their target repos.
Changes:
- Added new
CLAUDE.mddrafts for: step-function-workflow-orchestrator, identity-graph, batch-audience-delivery-syndication, and dos-data-pipeline. - Replaced/updated the
CLAUDE.mddraft for batch-expression-modeling. - Added
claude-md-updates/README.mdwith copy/apply instructions.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
claude-md-updates/README.md |
Documents which staged CLAUDE.md maps to which target repo and how to copy/apply them. |
claude-md-updates/step-function-workflow-orchestrator/CLAUDE.md |
Draft guidance for Step Functions + EMR pipeline orchestrator repo (pipelines, EMR7 migration notes, testing patterns). |
claude-md-updates/batch-expression-modeling/CLAUDE.md |
Draft guidance for BEM system (architecture, workflows, JSONata notes, recent operational fixes). |
claude-md-updates/identity-graph/CLAUDE.md |
Draft guidance for Identity Graph / PRISM Spark jobs and build/deploy/test workflows. |
claude-md-updates/batch-audience-delivery-syndication/CLAUDE.md |
Draft guidance for syndication workflows/lambdas and vendor-specific gotchas. |
claude-md-updates/dos-data-pipeline/CLAUDE.md |
Draft guidance for DOS geo/district pipeline concepts and recent changes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,65 @@ | |||
| # CLAUDE.md Updates | |||
|
|
|||
| This directory contains proposed CLAUDE.md files for 5 repositories that had active PRs merged into main (May–June 2026) by team members: SayaliPat, shrivastavakapil2000, JoeVsVolcano, PallaviJagarlamudi, mike-brant, nathan-resonate. | |||
| # Example for step-function-workflow-orchestrator | ||
| cd /path/to/step-function-workflow-orchestrator | ||
| git checkout -b chore/add-claude-md | ||
| cp /path/to/this/step-function-workflow-orchestrator/CLAUDE.md ./CLAUDE.md |
- Remove PallaviJagarlamudi from author list (no merged PRs found in scope) - Fix copy command to reference actual path in this repo (claude-md-updates/<repo>/CLAUDE.md) with raw GitHub URL alternative https://claude.ai/code/session_0121mjajLMWrWCnCUnjvKWZH
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reviews merged PRs from May–June 2026 by team members
SayaliPat,shrivastavakapil2000,JoeVsVolcano,mike-brant, andnathan-resonateacross the organization, and producesCLAUDE.mdfiles capturing current architecture, key concepts, and recent changes.Since this session only has write access to
resonate/.github, the files are staged here underclaude-md-updates/. Each needs to be copied to the root of its target repo and a separate PR opened there.Files to Apply
claude-md-updates/step-function-workflow-orchestrator/CLAUDE.mdresonate/step-function-workflow-orchestratorclaude-md-updates/batch-expression-modeling/CLAUDE.mdresonate/batch-expression-modelingclaude-md-updates/identity-graph/CLAUDE.mdresonate/identity-graphclaude-md-updates/batch-audience-delivery-syndication/CLAUDE.mdresonate/batch-audience-delivery-syndicationclaude-md-updates/dos-data-pipeline/CLAUDE.mdresonate/dos-data-pipelineWhat's Documented in Each File
step-function-workflow-orchestrator (NEW)
fusion-behavior-preprocessremoved, geo district namespace fixes, L2 TapAd stitch wiringbatch-expression-modeling (UPDATE)
batch-delivery-formatter-publish-metrics)MaxConcurrency=2+ index-staggered Wait (CDP-118972)delta_with_full_fallbackrefresh type requirement in_annotate_previous_date_partitionsvendor=*/method=*/)identity-graph (NEW)
application.conf)--add-opensflagsbatch-audience-delivery-syndication (NEW)
openx-publish-data-files: hardcoded.csv.gzextension — do not revert (CDP-118955 fix)blockgraph-create-taxonomy-file: taxonomy generation, SPI=Nconstant, initial vs. refresh routingvendor=*/method=av/(not legacymethod=av/vendor=*/)dos-data-pipeline (NEW)
district_sourceprovenance enum (L2_CONFIRMED, L2_UNCONFIRMED, IP_INFERRED)L2_CONFIRMEDGeoLocationFullBackfillalways re-derives all 4 districtsHow to Apply to Target Repos
Generated by Claude Code