Skip to content

specs/functions: crm-proactive-sweep -- one step per seller after the 502 runs - #4113

Merged
gHashTag merged 4 commits into
masterfrom
sweep-spec
Oct 2, 2026
Merged

gHashTag merged 4 commits into
masterfrom
sweep-spec

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Closes #4112

New spec specs/functions/crm-proactive-sweep.t27 for the Inngest cron that had none. Three FAILED runs on 2026-09-16 (15 s / 2m02s / 5m00s) all read "HTTP 502 before the SDK responded" on the self-hosted dashboard: the tick was one step.run('sweep'), one HTTP request carrying every seller (each up to 170 s). Spec says STEPS = resolve-sellers, sweep-${owner}, RETRIES 0, and NOTE records the finding and what was not verified.

  • README DOMAIN vocabulary gains crm.
  • docs/now/2026-09-17-the-sweep-gets-one-step-per-seller.md.
  • Compiler verdict clean: typecheckOk, 0 errors, hirOk. ASCII only.
  • published_figures.py --check drifts 6 figures on master already (test blocks 13218 vs pin 12783); this PR adds no test blocks and does not touch the pin.

Code follows in 999-multibots-telegraf branch sweep-per-seller-steps.

This was referenced Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-17 09:17:49 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 0
PRs with All Checks Green 50
READY 0
FAILING 0
PENDING 0
NO CHECKS YET 0

These columns do not partition: 0 + 0 + 0 + 0 = 0, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=8932aae0c5a9 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

gHashTag and others added 2 commits October 3, 2026 03:19
…01 (999 main @7d48227)

The sweep-per-seller change landed on 999-multibots-telegraf main and the
createFunction call now sits at line 101, not 41. STEPS (resolve-sellers,
sweep-${owner}), RETRIES 0 and concurrency 1 re-read on that revision.

Refs #4112

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-02 20:19:40 UTC

Summary

Status Count
Total Open PRs 34
PRs with Failing Checks 31
PRs with All Checks Green 3
READY 1
FAILING 31
PENDING 0
NO CHECKS YET 0

These columns do not partition: 1 + 31 + 0 + 0 = 32, and there are 34 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=b45a356c2eb6 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-02 20:21:25 UTC

Summary

Status Count
Total Open PRs 34
PRs with Failing Checks 33
PRs with All Checks Green 1
READY 1
FAILING 33
PENDING 0
NO CHECKS YET 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=b45a356c2eb6 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@gHashTag

gHashTag commented Oct 2, 2026

Copy link
Copy Markdown
Owner Author

bee review (not merged) -- head 344157c29add58bc3853dc5c2e50cae0324952be, master 7079823ca7c5

Content is sound:

  • Anchor verified on gHashTag/999-multibots-telegraf main 6a6dfbe4: crmProactiveSweep.ts:101 is export const crmProactiveSweep = inngest.createFunction(; retries: 0 (L105), concurrency: { limit: 1 } (L106), onFailure handler (L107), cron CRM_SWEEP_CRON = '*/30 * * * *' (L40).
  • STEPS verified: resolve-sellers (L114), sweep-${owner} (L145). The cited test src/__tests__/services/crmProactive.sweepSteps.test.ts exists on main. No crm-proactive-sweep.t27 on master yet.
  • Closes #4112 is the matching issue. README DOMAIN gains crm. Required validate, check-linked-issue, parse-ratchet: green.
  • build, integrity-gate, pack-index-consistency: same failures on specs/functions: payment-ai-server-process -- claim-invoice before crediting #4116/classify: the browser could not tell a Markdown file from a broken spec (refs #2133) #4541, about files this PR does not touch. Not blockers.

Blocker: check-now-freshness is red and the PR causes it -- docs/now/2026-09-17-...md is outside the window 2026-10-01..2026-10-03. Master is green on it.

To clear: re-date the NOW entry (or add one dated today) on the branch.

The NOW sync gate reads the entry date from the filename and accepts only
yesterday..tomorrow UTC. The entry was dated 2026-09-17; it is moved to
2026-10-03 with its heading, and a line says it was re-dated for re-review.

Refs #4112

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-02 20:54:36 UTC

Summary

Status Count
Total Open PRs 34
PRs with Failing Checks 28
PRs with All Checks Green 6
READY 1
FAILING 28
PENDING 0
NO CHECKS YET 0

These columns do not partition: 1 + 28 + 0 + 0 = 29, and there are 34 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=b45a356c2eb6 != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@gHashTag

gHashTag commented Oct 2, 2026

Copy link
Copy Markdown
Owner Author

Bee review evidence -- #4113 at ffe0f67c55dce2abfb4c4360ddb1a45949f77c02

  • Diff re-read: specs/functions/crm-proactive-sweep.t27 (new function spec, anchor crmProactiveSweep.ts:101 @7d48227, STEPS resolve-sellers, sweep-${owner}, RETRIES 0), README DOMAIN vocabulary gains crm, plus the NOW entry.
  • Re-date commit ffe0f67c5: a pure rename docs/now/2026-09-17-the-sweep-gets-one-step-per-seller.md -> 2026-10-03-..., the heading date 2026-09-17 -> 2026-10-03, and an appended ## Re-dated section stating the original date (2026-09-17, matching the old filename). Nothing else changed.
  • Mergeable: MERGEABLE. Required checks (validate, parse-ratchet, check-linked-issue) pass; check-now-freshness, Check L1 TRACEABILITY and the L1 summary pass.
  • Red non-required: untrusted-input, spec-guards (also red on master 7079823ca), and emit-bitexact (on the known master-wide list). No red check is specific to this PR.

@t27-bees t27-bees Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bee review: see evidence comment

@gHashTag
gHashTag merged commit 2dc66ff into master Oct 2, 2026
32 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bee-reviewed A reviewer bee reviewed and verified this PR at its current head; the only merge signal (#5525)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Spec crm-proactive-sweep: one step per seller after the 502 failures of 2026-09-16

1 participant