Skip to content

10140 Add Previous Month Targets#1

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

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
@github-actions github-actions Bot closed this Apr 13, 2026
Hareet added a commit that referenced this pull request Apr 13, 2026
…CiV fleet

Analyzed 6,662 telemetry entries (2,611 users) from devices.json to
re-evaluate the PowerSync hardware analysis against actual device data.
Cross-referenced with PowerSync MCP docs to identify 5 critical gaps
between our Node.js benchmarks and the Web SDK reality on Android Go.

Key findings:
- OPFS is NOT the default VFS — explicit OPFSCoopSyncVFS config is #1 fix
- fetchStrategy/cacheSizeKb not in Web SDK API — use Prioritized Sync instead
- Benchmarks used @powersync/node + better-sqlite3; expect 1.5-2x slower on WASM
- NOT EXISTS subquery may not work in Sync Streams — needs testing
- Prioritized Sync (priority 0-3) enables usable UI before full sync on Go edition

Updated all 7 agent assignments for Iteration 2 device-ready focus,
added device fleet reference data, and wrote benchmark gap analysis.
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