Skip to content

Add ARCHITECTURE.md for agent and contributor onboarding#924

Merged
drbenvincent merged 1 commit into
mainfrom
cursor/add-architecture-md
May 25, 2026
Merged

Add ARCHITECTURE.md for agent and contributor onboarding#924
drbenvincent merged 1 commit into
mainfrom
cursor/add-architecture-md

Conversation

@drbenvincent
Copy link
Copy Markdown
Collaborator

Summary

  • Adds ARCHITECTURE.md — a living reference covering the backend dispatch system, experiment lifecycle, formula/data pipeline, data contracts, experiment & model inventories, extension guide, and key conventions/gotchas.
  • Updates AGENTS.md to point agents at the architecture doc and require it to be kept in sync with structural changes.

This eliminates the repeated cost of agents re-discovering CausalPy's internal architecture from scratch in every fresh session.

Test plan

  • Verify ARCHITECTURE.md renders correctly on GitHub
  • Verify claims are accurate against current codebase
  • Confirm AGENTS.md reference is clear and actionable

Made with Cursor

Agents and contributors currently spend significant time re-discovering
the same structural facts about CausalPy (backend dispatch, experiment
lifecycle, data contracts, formula pipeline). This adds a living
architecture reference that can be read once at the start of a session.

AGENTS.md now points to ARCHITECTURE.md and requires it to be updated
alongside structural changes.

Co-authored-by: Cursor <cursoragent@cursor.com>
@drbenvincent drbenvincent added the agents Agent related issues specifically for use by developers label May 25, 2026
@drbenvincent drbenvincent changed the title docs: add ARCHITECTURE.md for agent and contributor onboarding Add ARCHITECTURE.md for agent and contributor onboarding May 25, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.07%. Comparing base (d56b781) to head (20781a2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #924   +/-   ##
=======================================
  Coverage   95.07%   95.07%           
=======================================
  Files          87       87           
  Lines       13701    13701           
  Branches      812      812           
=======================================
  Hits        13026    13026           
  Misses        479      479           
  Partials      196      196           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@drbenvincent
Copy link
Copy Markdown
Collaborator Author

I'm going to YOLO merge this, but tagging @cetagostini and @juanitorduz

@drbenvincent drbenvincent marked this pull request as ready for review May 25, 2026 09:22
@drbenvincent drbenvincent merged commit e40af27 into main May 25, 2026
18 checks passed
@drbenvincent drbenvincent deleted the cursor/add-architecture-md branch May 25, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent related issues specifically for use by developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant