janpeter/scrum4me-workers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make Worker Insights incident-first and actionable #70

Closed
janpeter wants to merge 0 commits from codex/worker-insights-incident-first into main
pull from: codex/worker-insights-incident-first
merge into: janpeter:main
Conversation 1 Commits - Files changed -
janpeter commented 2026-07-18 01:18:01 +02:00
Owner
Copy link

Wat verandert

  • Corrigeert toolfrictie, guardrail-, latency-, retry-loop- en rate-limitsemantiek.
  • Groepeert gevalideerde triageverdicts tot actiegerichte incidenten met bewijslinks.
  • Voegt gedeelde filters, begrensde job-kindscope en vorige-periodevergelijkingen toe.
  • Isoleert queryfouten per sectie en onderscheidt lege van onbeschikbare data.
  • Voegt deeplinks naar runs en jobs toe en documenteert het volledige metriekcontract.

Waarom

De oude pagina was metriek- en tabelgedreven, kon guardrailrejections als fouten tellen en verloor alle data wanneer één loader faalde. Hierdoor waren oorzaken, impact en onderzoekspaden onvoldoende betrouwbaar.

Impact

Beheerders zien eerst actuele incidenten met concrete verbeterhefboom en bewijs. Alle datasets delen dezelfde scope; bemonsterde totalen en onbeschikbare trends worden expliciet gelabeld.

Cross-repo contract

Companion producer-PR: janpeter/Ops-dashboard#111

De Ops-dashboard producer-PR moet vóór of atomair met deze consumer-PR worden geïntegreerd.

Verificatie

  • npm run verify: lint 0 errors (1 bestaande warning), typecheck geslaagd.
  • 113 testbestanden geslaagd, 1 overgeslagen.
  • 777 tests geslaagd, 10 overgeslagen.
  • Geen Prisma-migratie of wijziging aan vendor/scrum4me-shared.
  • Onafhankelijke review: Ready: yes.
## Wat verandert - Corrigeert toolfrictie, guardrail-, latency-, retry-loop- en rate-limitsemantiek. - Groepeert gevalideerde triageverdicts tot actiegerichte incidenten met bewijslinks. - Voegt gedeelde filters, begrensde job-kindscope en vorige-periodevergelijkingen toe. - Isoleert queryfouten per sectie en onderscheidt lege van onbeschikbare data. - Voegt deeplinks naar runs en jobs toe en documenteert het volledige metriekcontract. ## Waarom De oude pagina was metriek- en tabelgedreven, kon guardrailrejections als fouten tellen en verloor alle data wanneer één loader faalde. Hierdoor waren oorzaken, impact en onderzoekspaden onvoldoende betrouwbaar. ## Impact Beheerders zien eerst actuele incidenten met concrete verbeterhefboom en bewijs. Alle datasets delen dezelfde scope; bemonsterde totalen en onbeschikbare trends worden expliciet gelabeld. ## Cross-repo contract Companion producer-PR: https://git.jp-visser.nl/janpeter/Ops-dashboard/pulls/111 De Ops-dashboard producer-PR moet vóór of atomair met deze consumer-PR worden geïntegreerd. ## Verificatie - `npm run verify`: lint 0 errors (1 bestaande warning), typecheck geslaagd. - 113 testbestanden geslaagd, 1 overgeslagen. - 777 tests geslaagd, 10 overgeslagen. - Geen Prisma-migratie of wijziging aan `vendor/scrum4me-shared`. - Onafhankelijke review: Ready: yes.
fix(insights): retain token-expired incidents
Some checks failed
CI / Verify (pull_request) Failing after 0s
Details
cb8b24b8c1
janpeter changed title from WIP: Make Worker Insights incident-first and actionable to Make Worker Insights incident-first and actionable 2026-07-18 01:22:22 +02:00
janpeter closed this pull request 2026-07-18 01:26:17 +02:00
s4m-codex-reviewer approved these changes 2026-07-18 01:26:52 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • Geen blokkerende of error-severity findings gevonden.

De diff is in lijn met de relevante productdoc Worker Insights — betrouwbare incident-first analyse: de belangrijkste metriekcontracten zijn verwerkt (UNCLASSIFIED los van OTHER, guardrail-rejections uitgesloten van foutcategorieën/retry-loops, rate-limit impact als unieke runs), de pagina is incident-first opgebouwd, deeplinks naar worker logs/jobs zijn toegevoegd, secties degraderen onafhankelijk, filters/trends zijn URL-gestuurd en de paginadocumentatie is bijgewerkt.

Let op: er was geen lokale checkout beschikbaar in /opt/agent, dus ik heb de frozen PR-diff en productdocs gereviewd maar geen npm run verify kunnen uitvoeren.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende of error-severity findings gevonden. De diff is in lijn met de relevante productdoc `Worker Insights — betrouwbare incident-first analyse`: de belangrijkste metriekcontracten zijn verwerkt (`UNCLASSIFIED` los van `OTHER`, guardrail-rejections uitgesloten van foutcategorieën/retry-loops, rate-limit impact als unieke runs), de pagina is incident-first opgebouwd, deeplinks naar worker logs/jobs zijn toegevoegd, secties degraderen onafhankelijk, filters/trends zijn URL-gestuurd en de paginadocumentatie is bijgewerkt. Let op: er was geen lokale checkout beschikbaar in `/opt/agent`, dus ik heb de frozen PR-diff en productdocs gereviewd maar geen `npm run verify` kunnen uitvoeren.
Some checks failed
CI / Verify (pull_request) Failing after 0s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/scrum4me-workers!70
No description provided.