Auto-run rowCount backfill during make dev

[urugondavishnu]

Summary

Follow-up to #89. The rowCount migration is idempotent but currently has to be run manually — contributors who pulled main after #89 still see curated seed datasets capped at "5 rows" on their local dashboards, because seed datasets never receive writes so the self-heal in the row mutations never fires for them.

Hooks npx convex run datasets:backfillRowCounts into make dev right after convex-push, so a fresh clone reaches a fully-migrated state with zero manual steps.

Uses the env-var-only invocation shape (matching seed-public-datasets) instead of explicit --admin-key flags, because the admin key contains a | that gets re-parsed by Windows cmd when passed via the CLI.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: fc252e08-a395-4bd4-ab98-e84f9f5cae74

📥 Commits

Reviewing files that changed from the base of the PR and between a796bab and a8c102d.

📒 Files selected for processing (1)

• makefiles/Makefile

🚧 Files skipped from review as they are similar to previous changes (1)

• makefiles/Makefile

---

📝 Walkthrough

Walkthrough

Adds a new phony Makefile target backfill-row-counts that verifies the root .env and runs npx convex run datasets:backfillRowCounts via frontend/../scripts/with-root-env.mjs. The dev target now invokes $(MAKE) backfill-row-counts after convex-push, causing the Convex dataset rowCount backfill to run automatically during development startup.

Sequence Diagram(s)

sequenceDiagram
  participant Developer
  participant Makefile
  participant with_root_env_mjs as with-root-env.mjs
  participant Convex
  Developer->>Makefile: run `make dev`
  Makefile->>Makefile: run `convex-push`
  Makefile->>with_root_env_mjs: invoke `backfill-row-counts`
  with_root_env_mjs->>Convex: `npx convex run datasets:backfillRowCounts` (supplies CONVEX_SELF_HOSTED_ADMIN_KEY)

Loading

Possibly related PRs

• tinyfish-io/bigset#109: Overlaps Makefile dev workflow changes around convex-push and environment/Convex validation.
• tinyfish-io/bigset#19: Earlier modifications to makefiles/Makefile affecting the convex-push flow that this change extends.
• tinyfish-io/bigset#89: Adds the backfillRowCounts mutation and rowCount migration which this Makefile target invokes.

Suggested reviewers

• giaphutran12
• hwennnn
• MMeteorL

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and accurately summarizes the main change: automatically running the rowCount backfill during the make dev workflow.
Description check	✅ Passed	The description provides clear context about the follow-up to #89, explains the idempotent migration, the rationale for auto-running it, and technical implementation details including the Windows cmd parsing workaround.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[giaphutran12]

Holding for launch safety. This branch is merge-conflicting and stale against latest main. The intended Makefile change is small, but the branch needs to be rebased or merged with current main so the diff does not conflict with the newer dev setup changes, then checks need to rerun.

[urugondavishnu]

Rebased onto current main (head 3222058). The conflicts were entirely from #109's Makefile restructure (self-healing admin key, install-deps, the BigSet is ready! block).

Resolution:

• Kept all of Simplify dev setup: one-command start with self-healing admin key #109's structural changes intact.
• Added backfill-row-counts to .PHONY.
• Inserted $(MAKE) backfill-row-counts immediately after $(MAKE) convex-push in the dev target, before the app-container docker compose up. So the backfill runs once the schema is deployed, but before the frontend/backend/mastra come up — dashboard cards show correct counts on the very first load instead of needing a refresh.
• The new backfill-row-counts target itself was untouched by Simplify dev setup: one-command start with self-healing admin key #109 and applied cleanly.

Verification:

• Live-ran node ../scripts/with-root-env.mjs npx convex run datasets:backfillRowCounts against the dev Convex; got { alreadyCorrect: 29, patched: 0, total: 29 }. Idempotent on a fully migrated DB as designed.
• git diff main is now +16/-1 in makefiles/Makefile only.

Single commit, force-pushed with --force-with-lease. Ready for re-review. @giaphutran12

[MMeteorL]

LGTM