Skip to content

cat-mode: lead the multi-emitter backfill done-gate with the general lesson. - #1312

Merged
mergify[bot] merged 5 commits into
mainfrom
stack/EdbertChan/automate/dashboard-health-receipt/cat-mode-lead-multi-emitter-backfill-done-gate--aa84683f
Oct 13, 2026
Merged

mergify[bot] merged 5 commits into
mainfrom
stack/EdbertChan/automate/dashboard-health-receipt/cat-mode-lead-multi-emitter-backfill-done-gate--aa84683f

Conversation

@EdbertChan

@EdbertChan EdbertChan commented Oct 8, 2026 •

Copy link
Copy Markdown
Owner

Depends-On: #1311

Summary

This updates a working-style lesson for agents who are asked to audit a broad backfill. It tells them to prove every writer and source, not just quote the request.

The problem was that the rule led with example phrases. That made the lesson sound tied to those phrases instead of the wider failure.

The fix leads with the general backfill requirement, then keeps the old phrases as examples. The nearby checks now lock that wording.

Review Claim

Review that the backfill lesson now leads with the general requirement while preserving the examples that trigger it.

Review Lane

behavior

Review Unit

corpus-lesson

Safety Invariant

This slice changes lesson wording and regression locks only. It does not alter runtime code.

Assumption: this invariant is unconfirmed because this is a headless CI repair.

Slice Rationale

This stays in the lesson slice so the reviewer can approve the wording rule and its checks together.

Non-goals

  • No runtime behavior change.
  • No new lesson trigger.
  • No publishing flow change.

Measured

none: Wording-only lesson slice; there is no runtime metric to compare.

Test Plan

Test Plan
  • node engine/skills/draft-pr/scripts/validate-pr-body.mjs --body "$BODY" --changed-files-file <(git diff --name-only origin/stack/EdbertChan/automate/dashboard-health-receipt/cat-mode-align-publishing-analytics-fixture--debc0f0b...origin/stack/EdbertChan/automate/dashboard-health-receipt/cat-mode-lead-multi-emitter-backfill-done-gate--aa84683f)
  • python3 engine/skills/make-pr/scripts/preflight.py --base origin/stack/EdbertChan/automate/dashboard-health-receipt/cat-mode-align-publishing-analytics-fixture--debc0f0b --commit-only --dry-run

Revert Plan

Revert Plan
  • Safe to revert? Yes
  • Revert command: git revert 07f048683f699506e7f071d9528e23a849c6225c
  • Post-revert steps: None
  • Data migration? No

Note

Low Risk
Wording-only changes to cat-mode skill docs and unittest fixtures; no runtime or product behavior changes.

Overview
cat-mode publishing-analytics rules are tightened so agents fix data at emit/resolve/backfill instead of patching charts, and prove coverage before claiming “done.”

Emit/resolve: Blank or synthetic model is an emit or resolve defect (not emit-only). Chart filters that exclude or include blanks are not fixes; prove the live resolve path first.

Backfill done-gate: “Full backfill” is satisfied only with a harness×source matrix plus fill table, with example phrases (full backfill, all harnesses) secondary to that general requirement.

New gates: Require a health receipt (harness×action, filled_model_rate, volume/fill/resolve/filter ownership) before narrating by-model health; split those four before L3 or insight/notebook PATCH when a harness is “missing” or real-model is zero; when /automate-me (or similar) lands mid thrash, lock the class catch before more analytics PRs.

references/verify.md mirrors the same text; new fires_/stays_silent_ fixtures and expanded TestPublishingAnalyticsDefaults / TestAnalyticsThrashCheckpoints lock the wording.

Reviewed by Cursor Bugbot for commit 30840ff. Bugbot is set up for automated code reviews on this repo. Configure here.

EdbertChan and others added 4 commits October 8, 2026 06:55
Agents kept narrating by-model health and patching charts before splitting volume, fill, resolve, and filter — lock that check into standing defaults.

Co-authored-by: Cursor <cursoragent@cursor.com>
Change-Id: I8efc336daf9878751865cf3c4bac4d55cd50733f
Phrase-judge rejected leads that named only the trigger phrase; state the split and the stop-first catch as the lesson, with the triggers as examples.

Co-authored-by: Cursor <cursoragent@cursor.com>
Change-Id: I7dad9e54f9d280bb4d8f41ac06662293e39c9e05
…mple phrasing.

Co-authored-by: Cursor <cursoragent@cursor.com>
Change-Id: Idebc0f0ba74e47864bf479446a8e27b0574cdcde
…lesson.

Co-authored-by: Cursor <cursoragent@cursor.com>
Change-Id: Iaa84683fcf5a66e8f70a0dbf99ebcf19acc1d7a7
@cursor

cursor Bot commented Oct 8, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: 5476db25-4bba-4cb4-9d61-b54e687e53f6)

@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/automate/dashboard-health-receipt/cat-mode-align-publishing-analytics-fixture--debc0f0b branch from 477d078 to 632eb92 Compare October 13, 2026 03:10
@cursor

cursor Bot commented Oct 13, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: d5c1e6d3-b0ee-4268-8912-c65d22301187)

@mergify

mergify Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Queued — the merge queue status continues in this comment ↓.

@EdbertChan
EdbertChan changed the base branch from stack/EdbertChan/automate/dashboard-health-receipt/cat-mode-align-publishing-analytics-fixture--debc0f0b to main October 13, 2026 08:18
@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@mergify

mergify Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 46 minutes 45 seconds in the queue, including 46 minutes 22 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Oct 13, 2026
@mergify mergify Bot mentioned this pull request Oct 13, 2026
8 tasks done
@mergify
mergify Bot merged commit ff82cf8 into main Oct 13, 2026
8 checks passed
@mergify mergify Bot removed the queued label Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant