Skip to content

10140 Add Previous Month Targets#1

Draft
Hareet wants to merge 5 commits intomasterfrom
10140-prev-targets-with-claude
Draft

10140 Add Previous Month Targets#1
Hareet wants to merge 5 commits intomasterfrom
10140-prev-targets-with-claude

Conversation

@Hareet
Copy link
Copy Markdown
Owner

@Hareet Hareet commented Sep 4, 2025

Trials and Tribulations with Agent Claude.

Locally:

  • Unit tests pass
  • Integration tests pass
  • e2e flakes out on unrelated tests

To Do:

  • More tests, clean up Claude over-eagerness
  • Manually test Supervisor Target Aggregates still functions
  • Follow along to this Squad doc
  • SonarCloud

Some screenshots of success:
Screenshot 2025-09-04 002637
Screenshot 2025-09-04 002657

…ggregate Feature. Working stop point for CHW Target Previous Month. Design choices around keeping current targets as RulesEngine and not adding that to TargetAggregatorService. Interval calc functions and month could be pulled from elsewhere.
…ed to get something out. Useful to guide the next agent to catch up to narrow changes and background info
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

This PR is now marked "stale" after 30 days without activity. It will be closed automatically in 10 days unless you add a comment, push new changes or remove the "stale" label.

@github-actions github-actions bot added the Stale label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant