[posthog]

Problem

The hourly timing collector repeats a full duration-map scan for each product.

Why: Remove repeated processing work while keeping sharding weights stable.

Origin

• Scout: a custom scout
• First signal: 2026-11-07
• Inbox report: open
• Task started by: a person, from the inbox

Changes

• Group product keys once with the existing module parser, preserving key order and summation order.
• Expected effect: less product-scaling CPU work. Complete-job savings remain unverified.

How did you test this code?

• Ran the optimizer unit suite, Ruff, and CI preflight.
• Paired local probes preserved ordered output; raw-artifact benchmarking needs Depot access.

Test rationale: Existing scaling, scoping, marker, missing-JUnit, and drift tests cover this internal optimization.

Post-merge check: compare five matched hourly jobs, all attempts, memory, and downstream maps.

Release status

• No feature flag controls this change

Docs update

• None. Internal performance only.

🤖 Agent context

Autonomy: Fully autonomous

Agent: Codex, gpt-6.1-sol

More context for AI agents

• Session: Scout report; GitHub CLI and PostHog tools.
• Decisions: No matching open optimizer PR found. No timing data or logs added to the repository.
• Follow-ups: Run complete raw-artifact cold/warm comparisons when Depot read access is available. No fleet or cash savings claim.
• Read closely: Product grouping preserves exact directory boundaries and per-module insertion order.

Skills: /verifying-changes, /writing-tests, /splitting-oversized-modules, /running-ci-preflight, /writing-pr-descriptions, /reducing-ci-costs, /pr-shepherd.

---

Created with PostHog Desktop

[trunk-io]

😎 Merged successfully - details.

[github-actions]

🤖 CI report

⚠️ Trunk lane — backend Python lane

This PR is assigned to the backend Python lane. It runs backend Python tests and may merge in parallel with PRs in other lanes.

✅ Duplication (Python) — clean

New Python code duplication introduced by this branch. Fails at 70+ tokens in app code, or 150+ tokens when both copies live in test files. Advisory while the gate proves itself: extract a shared helper instead of copying.

✅ Duplication (TypeScript) — clean

New TypeScript code duplication introduced by this branch. Fails at 70+ tokens in app code, or 150+ tokens when both copies live in test files. Advisory while the gate proves itself: extract a shared helper instead of copying.

[posthog]

PostHog Review · Standard · reviewed 643f3c6: no findings.

[posthog]

Robot: QA Swarm review complete.

Note

🤖 Automated comment by QA Swarm. This is not a human review.

Verdict: ✅ APPROVE _{(round 1 @ 643f3c6)}

No actionable findings. Grouping preserves key order and scaling order.

Key findings

None.

Convergence

No findings to compare.

Reviewer summaries

Reviewer	Assessment
Router (gpt-6.1-sol)	Low risk, high confidence. No specialist review needed.

---

Automated by QA Swarm. This is not a human review.

[parameterai]

Risk: No findings

The PR replaces a per-module startswith("products/<module>/") scan of the duration map in scale_products_to_junit with a single pass that groups keys by product_module(key). It is a behavior-preserving internal CI optimization with no security-relevant surface, and I verified the new grouping is exactly equivalent to the old prefix filter for all edge cases I could construct.

_{Sentinel reviewed 643f3c6 · Review settings}

[stamphog]

stamphog does not review bot-authored pull requests, so the trigger label has been removed. This change needs a human reviewer.

[deployment-status-posthog]

Deploy status

Environment	Status	Deployed At	Workflow
dev	✅ Deployed	2026-11-07 20:43 UTC	Run
prod-us	✅ Deployed	2026-11-07 20:59 UTC	Run
prod-eu	✅ Deployed	2026-11-07 20:58 UTC	Run